lkml.org 
[lkml]   [2014]   [May]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] i915: Add module option to support VGA arbiter on HD devices
From
On Thu, May 15, 2014 at 11:43 PM, Alex Williamson
<alex.williamson@redhat.com> wrote:
> I don't know what to do with this. It seems like a lot of wishful
> thinking that in the best case would drag on for years. Even if we get
> VGA arbitration out of Xorg, the bit about making the userspace VGA
> arbiter interface lie depending on current->comm sounds tricky and
> horrible. If we can lie to Xorg there, why don't we do that now? If we
> can't lie to Xorg now, then what deprecation event or detection of the
> caller is going to allow us to do so in the future?

Well we wouldn't necessarily need to lie to X, but could instead look
whether all the vga devices in a system are claimed by kms drivers. If
that's the case the userspace doesn't have an awful lot of business
touching the VGA registers and we could simply not obey a vga arb
request from userspace.

More advanced would be if we still obey it for those devices not
claimed by kms drivers. So not really a need to key on current->comm.

> Meanwhile anyone that wants i915 to participate in arbitration like it
> should have from the start needs to patch their kernel with forward
> ports of the reverted commits.
>
> I just don't see this moving forward, which is why I thought a module
> option at least gives us a workaround. Thanks,

I know that this is awful, but a module option means that the few
people interested in moving this forward will be happy enough to no
longer bother. Which is even worse from my pov as driver maintainer. I
highly consider module options used in production evil.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


\
 
 \ /
  Last update: 2014-05-16 01:01    [W:1.893 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site