drm_open doesnt lock before increasing dev->open_count

2011-02-09 Thread adam zeira
8, 2011 at 1:18 AM, Chris Wilson wrote: > On Tue, 08 Feb 2011 08:52:52 +1000, Dave Airlie > wrote: > > On Sun, 2011-02-06 at 19:24 +0200, adam zeira wrote: > > > is this a problem? before trying to submit a solution I wanted to make > sure with the experts > > > >

Re: drm_open doesnt lock before increasing dev->open_count

2011-02-09 Thread adam zeira
8, 2011 at 1:18 AM, Chris Wilson wrote: > On Tue, 08 Feb 2011 08:52:52 +1000, Dave Airlie > wrote: > > On Sun, 2011-02-06 at 19:24 +0200, adam zeira wrote: > > > is this a problem? before trying to submit a solution I wanted to make > sure with the experts > > > >

drm_open doesn't lock before increasing dev->open_count

2011-02-07 Thread adam zeira
is this a problem? before trying to submit a solution I wanted to make sure with the experts it seems open_count isn't locked and can cause problems and if it is a problem, and needs to be solved, should locking be done or is it better implemented with the (as I understand standard) kref? ht

drm_open doesn't lock before increasing dev->open_count

2011-02-06 Thread adam zeira
is this a problem? before trying to submit a solution I wanted to make sure with the experts it seems open_count isn't locked and can cause problems and if it is a problem, and needs to be solved, should locking be done or is it better implemented with the (as I understand standard) kref? ht