[PATCH 01/10] drm/i915/gvt: parse init context to update cmd accessible reg whitelist

[Yan Zhao]

<...> 
> > > > +
> > > > +	if (intel_gt_wait_for_idle(&dev_priv->gt,
> > > > +				I915_GEM_IDLE_TIMEOUT) == -ETIME) {
> > > > +		ret = -EIO;
> > > > +		goto out;
> > > > +	}
> > > > +
> > > > +	for_each_engine(engine, &dev_priv->gt, id) {
> > > > +		struct i915_request *rq;
> > > > +
> > > > +		rq = requests[id];
> > > > +		GEM_BUG_ON(!i915_request_completed(rq));
> > > > +		GEM_BUG_ON(!intel_context_is_pinned(rq->context));
> > > > +		ctx_obj[id] = rq->context->state->obj;
> > > > +	}
> > > > +
> > > 
> > > and these two iterations?
> > This iteration cannot be combined with previous one because there's only
> > one intel_gt_wait_for_idle for all rings.
> >
> 
> yeah, I mean above this and below which handles for each engine's ctx,
> should be in one, right?
> 
yes, I'll try to merge them.

Thanks for spotting it :)


Yan