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

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/ (local)

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(-)