On Tue, 28 Jun 2011 11:27:47 +0100, Chris Wilson <[email protected]> 
wrote:
> As pointed out by Dan Carpenter, it was seemingly possible to hit an error
> whilst mapping the buffer for the regs (except the only likely error
> returns should not happen during init) and so leak a pin count on the
> bo. To handle this we would need to reacquire the struct mutex, so for
> simplicity rearrange for the lock to be held for the entire function.
> For extra pedagogy, test that we only call init once.
> 
> Signed-off-by: Chris Wilson <[email protected]>

Reviewed-by: Keith Packard <[email protected]>

I'll get this into drm-intel-fixes once I can fast forward and pick up
Hugh's original patch.

-- 
[email protected]

Attachment: pgpyulDe7ZQo0.pgp
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to