[PATCH 2/2] drm/i915/gvt: use atomic operations to change the vGPU status

Tian, Kevin kevin.tian at intel.com
Fri Nov 11 02:57:32 UTC 2022


> From: Zhi Wang <zhi.wang.linux2 at gmail.com>
> Sent: Thursday, November 10, 2022 8:21 PM
> 
> Several vGPU status are used to decide the availability of GVT-g core logics
> when creating a vGPU. Use atomic operations on changing the vGPU status to
> avoid the racing.

please also mark out that the unnecessary WARN_ON is also removed
in intel_vgpu_remove().

> 
> v3:
> 
> - Merge the patch 3 into this patch. (Kevin)
> - Remove unnecessary checks. (Kevin)
> 
> v2:
> 
> - Fixed an error reported by Zhenyu.

this is not the right format. You should have a coverletter with all
version history put there. It's not good to have it in commit msg.

> 
> Cc: Zhenyu Wang <zhenyuw at linux.intel.com>
> Cc: Kevin Tian <kevin.tian at intel.com>
> Cc: Jason Gunthorpe <jgg at nvidia.com>
> Cc: intel-gvt-dev at lists.freedesktop.org
> Suggested-by: Alex Williamson <alex.williamson at redhat.com>
> Signed-off-by: Zhi Wang <zhi.a.wang at intel.com>
> Reviewed-by: Zhenyu Wang <zhenyuw at linux.intel.com>

apart from those:

Reviewed-by: Kevin Tian <kevin.tian at intel.com>

btw please check whether this conflicts with Sean's fix:

https://lore.kernel.org/all/20221111002225.2418386-1-seanjc@google.com/


More information about the intel-gvt-dev mailing list