Re: [Intel-gfx] [PATCH] drm/i915: don't update the dri1 breadcrumb with modesetting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Dec 10, 2013 at 02:48:23PM +0100, Daniel Vetter wrote:
> On Tue, Dec 10, 2013 at 12:44:12PM +0000, Chris Wilson wrote:
> > On Tue, Dec 10, 2013 at 01:24:01PM +0100, Daniel Vetter wrote:
> > > The update is horribly racy since it doesn't protect at all against
> > > concurrent closing of the master fd. And it can't really since that
> > > requires us to grab a mutex.
> > > 
> > > Instead of jumping through hoops and offloading this to a worker
> > > thread just block this bit of code for the modesetting driver.
> > > 
> > > Reported-by: Eugene Shatokhin <eugene.shatokhin@xxxxxxxxxx>
> > > Cc: Eugene Shatokhin <eugene.shatokhin@xxxxxxxxxx>
> > > Cc: stable@xxxxxxxxxxxxxxx
> > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> > 
> > We should also not be calling update_dri1_breadcrumb unless we have a
> > USER_INTERRUPT. The other updates to the breadcrumb hws index do not
> > appear to be serialised by anything other than polling.
> 
> Well, this is what's been there before and I don't really want to touch
> dri1/ums code at all. So I think we'll just keep this here until we'll all
> put it on the big pyre ;-)

Chris and I discusssed the commit message a bit on irc and now the hack
also spurts a code comment. Merged to -fixes for now, but I'd still like
to get a tested by from Eugene before sending off the pull request.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]