[PATCH 1/6] drm/i915/gvt: Separate cmd scan from request allocation

[Zhenyu Wang]

On 2017.08.09 06:35:30 +0000, Gao, Fred wrote:
> thanks Zhenyu for the thoughtful review.
> 
> for the ctx pin issue:
>  ring = engine->context_pin(engine, shadow_ctx);
> 
> the above pin will be used in the below shadow ctx, 
> otherwise the null pointer err will be thrown.
> >       ret = populate_shadow_context(workload);
> 
> previously it is ok since the pin is also done in i915_gem_request_alloc() before scan
> now i915_gem_request_alloc() is moved after scan.

yeah, I was only concerned on guest ctx access but ignored
for shadow ctx, then that should be ok.

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20170809/4cf45c58/attachment.sig>