On Thu, Nov 15, 2012 at 09:14:48AM -0800, Ben Widawsky wrote: > On Wed, 14 Nov 2012 11:06:42 -0800 > Ben Widawsky <ben at bwidawsk.net> wrote: > > > On Wed, 14 Nov 2012 09:15:14 +0000 > > Chris Wilson <chris at chris-wilson.co.uk> wrote: > > > > > The bspec was recently updated to remove the ability to update the > > > semaphore using the MI_SEMAPHORE_BOX command, the ability to wait upon > > > the semaphore value remained. Instead the advice is to update the > > > register using the MI_LOAD_REGISTER_IMM command. In cursory testing, > > > semaphores continue to function - the question is whether this fixes > > > some of the deadlocks where the semaphore registers contained stale > > > values? > > > > > > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk> > > > Cc: Daniel J Blueman <daniel at quora.org> > > Reviewed-by: Ben Widawsky <ben at bwidawsk.net> > Tested-by: Ben Widawsky <ben at bwidawsk.net> I seem to have forgotten to mention that this is merged. Bad, bad me ... Thanks for the patch & review. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
