Re: [PATCH v2 0/5] powerpc/rtas: miscellaneous cleanups

[Christophe Leroy <christophe.leroy@csgroup.eu>]

Le 08/04/2021 à 16:06, Nathan Lynch a écrit :
> This is a reroll of the series posted here:
> https://lore.kernel.org/linuxppc-dev/20210114220004.1138993-1-nathanl@linux.ibm.com/
> 
> Originally this work was prompted by failures on radix MMU PowerVM
> guests when passing buffers to RTAS that lay outside of its idea of
> the RMA. In v1 I approached this as a problem to be solved in Linux,
> but RTAS development has since decided to change their code so that
> the RMA restriction does not apply with radix.
> 
> So in v2 I retain the cleanups and discard the more significant change
> which accommodated the misbehaving RTAS versions.

Is there a link with https://github.com/linuxppc/issues/issues/252 ?

> 
> Changes since v1:
> - Correct missing conversion of RTAS_RMOBUF_MAX ->
>    RTAS_USER_REGION_SIZE in in_rmo_buf().
> - Remove unnecessary braces in rtas_syscall_filter_init().
> - Leave expression of RTAS_WORK_AREA_SIZE as-is instead of changing
>    the factors in a confusing way, per discussion with Alexey.
> - Drop "powerpc/rtas: constrain user region allocation to RMA"
> 
> Nathan Lynch (5):
>    powerpc/rtas: improve ppc_rtas_rmo_buf_show documentation
>    powerpc/rtas-proc: remove unused RMO_READ_BUF_MAX
>    powerpc/rtas: remove ibm_suspend_me_token
>    powerpc/rtas: move syscall filter setup into separate function
>    powerpc/rtas: rename RTAS_RMOBUF_MAX to RTAS_USER_REGION_SIZE
> 
>   arch/powerpc/include/asm/rtas.h |  6 +++---
>   arch/powerpc/kernel/rtas-proc.c | 15 +++++++++++----
>   arch/powerpc/kernel/rtas.c      | 34 +++++++++++++++++----------------
>   3 files changed, 32 insertions(+), 23 deletions(-)
>