Re: [PATCH v3] powerpc/audit: Convert powerpc to AUDIT_ARCH_COMPAT_GENERIC
Venkat Rao Bagalkote <[email protected]> Wed, 1 Jul 2026 10:02:06 +0530
| Newsgroups | org.kernel.vger.audit,org.kernel.vger.linux-kernel,org.ozlabs.lists.linuxppc-dev |
|---|---|
| Message-ID | <[email protected]> |
On 01/07/26 7:44 am, Madhavan Srinivasan wrote: > > On 7/1/26 12:41 AM, Paul Moore wrote: >> On Wed, May 13, 2026 at 1:42 AM Madhavan Srinivasan >> <[email protected]> wrote: >>> On 5/13/26 10:05 AM, Harsh Prateek Bora wrote: >>>> On 11/03/26 12:49 am, Paul Moore wrote: >>>>> On Tue, Mar 10, 2026 at 11:08 AM Christophe Leroy (CS GROUP) >>>>> <[email protected]> wrote: >>>>>> From: Christophe Leroy <[email protected]> >>>>>> >>>>>> Commit e65e1fc2d24b ("[PATCH] syscall class hookup for all normal >>>>>> targets") added generic support for AUDIT but that didn't include >>>>>> support for bi-arch like powerpc. >>>>>> >>>>>> Commit 4b58841149dc ("audit: Add generic compat syscall support") >>>>>> added generic support for bi-arch. >>>>>> >>>>>> Convert powerpc to that bi-arch generic audit support. >>>>>> >>>>>> With this change generated text is similar. >>>>>> >>>>>> Thomas has confirmed that the previously failing filter_exclude/test >>>>>> is now successful both without and with this patch, see [1] >>>>>> >>>>>> [1] >>>>>> https://lore.kernel.org/all/20260306115350-ef265661-6d6b-4043-9bd0-8e6b437d0d67@linutronix.de/ >>>>>> >>>>>> >>>>>> Link: https://github.com/linuxppc/issues/issues/412 >>>>>> Signed-off-by: Christophe Leroy <[email protected]> >>>>>> Reviewed-by: Cédric Le Goater <[email protected]> >>>>>> --- >>>>>> Venkat, a test result with >>>>>> https://github.com/linux-audit/audit-testsuite would be appreciated. >>>>> Yes, I'd like to see confirmation that the audit test suite runs >>>>> clean >>>>> on ppc systems with this patch applied, and unfortunately without a >>>>> ppc system I have no way to test this myself. >>> My bad, this is a miss from my end. >>> Venkat is already on this and will update the results here. >> Do we have an update on this? Maybe I missed it, but I don't recall >> seeing any test results. > Venkat, did run the test but I guess he missed to respond here, > Details of his test run that he shared me in the internal chat from > May 13. Sorry, My bad. I missed updating here. Yes, this was tested in May. Please let me know, if a re-run is required? Regards, Venkat. > > backlog_wait_time_actual_reset/test .. ok > bpf/test ............................. ok > exec_execve/test ..................... ok > exec_name/test ....................... ok > fanotify/test ........................ ok > field_compare/test ................... ok > file_create/test ..................... ok > file_delete/test ..................... ok > file_permission/test ................. ok > file_rename/test ..................... ok > filter_exclude/test .................. ok > filter_exit/test ..................... ok > filter_saddr_fam/test ................ ok > filter_sessionid/test ................ ok > io_uring/test ........................ ok > login_tty/test ....................... ok > lost_reset/test ...................... ok > netfilter_pkt/test ................... Can't exec "nc": No such file > or directory at netfilter_pkt/test line 83. > > Venkat, can you please re-run if possible and paste the log here. > > Thanks > Maddy > >