[PATCH] xfree86: Use SA_SIGINFO if available for SIGIO handlers
Mark Kettenis
mark.kettenis at xs4all.nl
Tue Nov 10 12:15:07 PST 2009
> From: Adam Jackson <ajax at redhat.com>
> Date: Tue, 10 Nov 2009 15:03:31 -0500
>
> This one's slightly subtle, the fcntl(F_SETSIG) is required for ->si_fd
> to actually be filled in correctly, at least on Linux. I know this area
> has been sensitive to platform variation in the past, so I'd appreciate
> a second look from Solaris / BSD / Hurd people.
Alas, OpenBSD doesn't have si_fd. So if you really want to go this
route, you'll need to add an additional configure check for it.
> siginfo_t gives us the file descriptor that raised the signal directly,
> so we don't need to select() for it. This gets evdev event processing
> down to exactly one syscall in the common case.
Is that an optimizition that's really worth the added complexity?
More information about the xorg-devel
mailing list