Re: [PATCH v3] powerpc/audit: Convert powerpc to AUDIT_ARCH_COMPAT_GENERIC

Le 01/07/2026 à 06:32, Venkat Rao Bagalkote a écrit :
> 
> 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://eur01.safelinks.protection.outlook.com/? 
>>>>>>> url=https%3A%2F%2Flore.kernel.org%2Fall%2F20260306115350- 
>>>>>>> ef265661-6d6b-4043-9bd0-8e6b437d0d67%40linutronix.de%2F&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7C81e1e4e3bae245103d3308ded729bb47%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C639184771399964577%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=%2FtCHXFR5np67gntCnXqv7Eemo5WuwaIEcywxZQzDADA%3D&reserved=0
>>>>>>>
>>>>>>> Link: https://eur01.safelinks.protection.outlook.com/? 
>>>>>>> url=https%3A%2F%2Fgithub.com%2Flinuxppc%2Fissues%2Fissues%2F412&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7C81e1e4e3bae245103d3308ded729bb47%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C639184771400190794%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=4fPEX7HuGBcxOcXlcJVQvDdP7Y9InhDpbz3z%2BS9lkCQ%3D&reserved=0
>>>>>>> Signed-off-by: Christophe Leroy <[email protected]>
>>>>>>> Reviewed-by: Cédric Le Goater <[email protected]>
>>>>>>> ---
>>>>>>> Venkat, a test result with
>>>>>>> https://eur01.safelinks.protection.outlook.com/? 
>>>>>>> url=https%3A%2F%2Fgithub.com%2Flinux-audit%2Faudit- 
>>>>>>> testsuite&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7C81e1e4e3bae245103d3308ded729bb47%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C639184771400214912%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=yHp3p5zNx%2BqQg2cKBsKJADWcUcKVtZstzNScBfWzF%2FQ%3D&reserved=0 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?

It looks like netcat (https://linux.die.net/man/1/nc) is not installed 
in you setup. Are you able to install it and rerun the test ?

Thanks
Christophe

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