Bug 460192 - Add epoll_pwait2
Summary: Add epoll_pwait2
Status: RESOLVED FIXED
Alias: None
Product: valgrind
Classification: Developer tools
Component: general (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Julian Seward
URL:
Keywords:
: 471476 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-10-10 09:58 UTC by Olli Vanhoja
Modified: 2023-10-12 15:43 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
A patch to add epoll_pwait2 (2.93 KB, application/mbox)
2022-10-10 09:58 UTC, Olli Vanhoja 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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. ***