On Wed, 2022-11-23 at 13:32 -0600, Nick Child wrote: > On 11/22/22 20:51, Andrew Donnellan wrote: > > On Fri, 2022-11-18 at 09:07 -0600, Nathan Lynch wrote: > > > +enum rtas_function_flags { > > > + RTAS_FN_FLAG_BANNED_FOR_SYSCALL_ON_LE = (1 << 0), > > > +}; > > > > This seems to be new, what's the justification? > > > > Seems to be a run-time replacement of: > #ifdef CONFIG_CPU_BIG_ENDIAN > { "ibm,suspend-me", -1, -1, -1, -1, -1 }, > { "ibm,update-nodes", -1, 0, -1, -1, -1, 4096 }, > { "ibm,update-properties", -1, 0, -1, -1, -1, 4096 }, > #endif > > It looks to be handled logically: > + if (IS_ENABLED(CONFIG_CPU_LITTLE_ENDIAN) && > + (func->flags & RTAS_FN_FLAG_BANNED_FOR_SYSCALL_ON_LE)) > + goto err; > > Perhaps, also allow the addition of any future special cases > for rtas functions easier to maintain?
Makes sense, though I'm slightly confused about the original rationale for the ifdef and why it's not being fixed in userspace. Slightly clunky name though, something like RTAS_FN_FLAG_SYSCALL_BE_ONLY might be less clunky? -- Andrew Donnellan OzLabs, ADL Canberra a...@linux.ibm.com IBM Australia Limited