Excerpts from Michael Neuling's message of 2015-05-19 21:22:32 +1000:
> + if (ctx->kernel) {
> + sr |= CXL_PSL_SR_An_R | (mfmsr() & MSR_SF) | CXL_PSL_SR_An_HV;
> + pid = 0;
> + } else {
> + sr |= CXL_PSL_SR_An_PR | CXL_PSL_SR_An_R;
CXL_PSL_SR_An_R is common between these two cases - you could probably
move it to the initial sr = ... line instead of here.
> + set_endian(sr);
We should call set_endian(sr) for kernel contexts as well.
> + if (ctx->kernel) {
> + sr |= CXL_PSL_SR_An_R | (mfmsr() & MSR_SF) | CXL_PSL_SR_An_HV;
> + pid = 0;
> + } else { /* User space */
> + sr |= CXL_PSL_SR_An_PR | CXL_PSL_SR_An_R;
CXL_PSL_SR_An_R is common here as well.
Cheers,
-Ian
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
