[Intel-gfx] [PATCH 1/2] drm/i915: Dumb down the semaphore logic
Ben Widawsky
ben at bwidawsk.net
Thu Sep 8 03:19:56 CEST 2011
On Sep 7, 2011, at 5:30 PM, Andrew Lutomirski <luto at mit.edu> wrote:
> On Wed, Sep 7, 2011 at 4:12 PM, Ben Widawsky <ben at bwidawsk.net> wrote:
>> While I think the previous code is correct, it was hard to follow and
>> hard to debug. Since we already have a ring abstraction, might as well
>> use it to handle the semaphore updates and compares.
>>
>> I don't expect this code to make semaphores better or worse, but you
>> never know...
>>
>> Cc: Chris Wilson <chris at chris-wilson.co.uk>
>> Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
>> Cc: Eric Anholt <eric at anholt.net>
>> Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
>> ---
>> drivers/gpu/drm/i915/i915_gem_execbuffer.c | 3 +-
>> drivers/gpu/drm/i915/i915_reg.h | 7 +
>> drivers/gpu/drm/i915/intel_ringbuffer.c | 176 +++++++++++++++++++++-------
>> drivers/gpu/drm/i915/intel_ringbuffer.h | 7 +-
>> 4 files changed, 145 insertions(+), 48 deletions(-)
>>
>
> Sadly, it still instantly crashes.
>
> --Andy
Remind me again... Does ssh still work?
More information about the Intel-gfx
mailing list