[PATCH 0/3] Fix initialization when iopl is forbidden
Adam Jackson
ajax at redhat.com
Thu Sep 20 08:13:17 PDT 2012
On 9/20/12 9:42 AM, Keith Packard wrote:
> Adam Jackson <ajax at redhat.com> writes:
>> Will resend with better commentary, thanks.
>
> Do you have this series with new commentary added? Otherwise, I'll just
> merge it as-is.
I do, for a loose definition of "added" and/or "commentary":
http://cgit.freedesktop.org/~ajax/xserver/log/?h=ioperm
What that actually does is remove the "do we want it" comment in favor
of a new Bool want_hw_access. We set that based on the result of the
GET_REQUIRED_HW_INTERFACES loop, and only later set the (global)
xorgHWAccess based on whether xf86EnableIO() succeeded.
Why make the comments speak better if you can make the code speak
better, after all.
- ajax
More information about the xorg-devel
mailing list