Bug 460192

Summary: Add epoll_pwait2
Product: [Developer tools] valgrind Reporter: Olli Vanhoja <olli.vanhoja>
Component: generalAssignee: Julian Seward <jseward>
Status: RESOLVED FIXED    
Severity: normal CC: mark, pjfloyd, ronnebaumfinances, sam
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
See Also: https://bugzilla.redhat.com/show_bug.cgi?id=2203636
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: A patch to add epoll_pwait2

Description Olli Vanhoja 2022-10-10 09:58:02 UTC
Created attachment 152684 [details]
A patch to add epoll_pwait2

SUMMARY

epoll_pwait2() is a fairly new syscall in Linux. The syscall should be available on all archs but I can only test it on Amd64, hence submitting a patch for only the platform that I can verify.


STEPS TO REPRODUCE

1. Use epoll_pwait2

OBSERVED RESULT

WARNING: unhandled amd64-linux syscall: 441

EXPECTED RESULT

No warning.

SOFTWARE/OS VERSIONS
 
Linux: 5.19.11-200.fc36.x86_64

ADDITIONAL INFORMATION
Comment 1 Paul Floyd 2022-10-11 15:15:34 UTC
Do you have a testcase for this?
Comment 2 Mark Wielaard 2023-05-17 11:22:23 UTC
Thanks. I tweaked your patch a little to explicitly check the timespec argument (when not NULL) and hook it up for all other linux arches.

commit 17f56ab4c4fce7b4778251e1fab2bc50e5c6dbda
Author: Olli Vanhoja <olli.vanhoja@gmail.com>
Date:   Mon Oct 10 11:44:22 2022 +0200

    Add epoll_pwait2
    
    The only difference between epoll_pwait and epoll_pwait2 is the
    timeout argument. For epoll_pwait2 this is a timespec (which is
    always 64bit).
    
    https://bugs.kde.org/show_bug.cgi?id=460192
Comment 3 Mark Wielaard 2023-06-27 10:56:26 UTC
*** Bug 471476 has been marked as a duplicate of this bug. ***