Excerpts from Christophe Leroy's message of February 9, 2021 4:18 pm:

Le 09/02/2021 à 02:11, Nicholas Piggin a écrit :

quoted

Excerpts from Christophe Leroy's message of February 9, 2021 1:10 am:

quoted

regs->softe doesn't exist on PPC32.

Add irq_soft_mask_regs_set_state() helper to set regs->softe.
This helper will void on PPC32.

Signed-off-by: Christophe Leroy <redacted>
---

You could do the same with the kuap_ functions to change some ifdefs
to IS_ENABLED.

That's just my preference but if you prefer this way I guess that's
okay.

That's also my preference on the long term.

Here it is ephemeral, I have a follow up series implementing interrupt exit/entry in C and getting 
rid of all the assembly kuap hence getting rid of those ifdefs.

I thought it might have been because you hate ifdef more tha most :)
 

The issue I see when using IS_ENABLED() is that you have to indent to the right, then you interfere 
with the file history and 'git blame'

Valid point if it's just going to indent back the other way in your next 
series.

Thanks for reviewing my series and looking forward to your feedback on my series on the interrupt 
entry/exit that I will likely release later today.

Cool, I'm eager to see them.

Thanks,
Nick