Mark Kettenis <mark.kette...@xs4all.nl> writes: > Most of my objections to this Tiago's patch still apply. Threads > severely impact debuggability and this needs a full audit of all the > drivers; not just the server.
I added the change which makes -dumbSched disable input threading to ease debugging, although I haven't had any issues debugging multithreaded code in a long time. As for the driver audit, the only ones which need review are the few active input drivers which weren't allowing SIGIO-based event processing (libinput and keyboard). Not much to audit in either of those, fortunately. Given that evdev already allowed SIGIO handling for all devices, I think we've got good coverage for the X server itself. Output drivers haven't always had to support sprite position changes at SIGIO time, so they're all ready for this change. The reason for this change is that libinput isn't willing to tolerate reading events at SIGIO time as that means not being able to use most of libc. In the process of doing this work, I uncovered numerous existing places which were calling inappropriate libc functions from the SIGIO context; this change will fix those as well as allowing us to eliminate kludges working around this limitation. -- -keith
signature.asc
Description: PGP signature
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
