[Intel-gfx] [PATCH v4] drm/i915: Slaughter the thundering i915_wait_request herd

Chris Wilson chris at chris-wilson.co.uk
Mon Nov 30 08:40:34 PST 2015


On Mon, Nov 30, 2015 at 04:30:57PM +0000, Chris Wilson wrote:
> On Mon, Nov 30, 2015 at 02:34:40PM +0000, Chris Wilson wrote:
> > +bool intel_breadcrumbs_add_waiter(struct drm_i915_gem_request *request)
> > +{
> > +	struct intel_breadcrumbs *b = &request->i915->breadcrumbs;
> > +	struct intel_breadcrumbs_engine *be = &b->engine[request->ring->id];
> > +	bool first = false;
> > +
> > +	spin_lock(&be->lock);
> > +	if (request->waiters_count++ == 0) {
> > +		struct rb_node **p, *parent;
> > +
> > +		if (be->first_waiter == NULL)
> > +			wake_up_process(b->task);
> 
> Oh, the irony.
> 
> Converting the other side to a lockless READ_ONCE() means that I do know

And that's the second mispelling of "now" today!
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list