Commit a06d4b9e authored by Zhi Wang's avatar Zhi Wang Committed by Zhenyu Wang
Browse files

drm/i915/gvt: use atomic operations to change the vGPU status 



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.

Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
Cc: Kevin Tian <kevin.tian@intel.com>
Cc: Jason Gunthorpe <jgg@nvidia.com>
Cc: intel-gvt-dev@lists.freedesktop.org
Suggested-by: default avatarAlex Williamson <alex.williamson@redhat.com>
Signed-off-by: default avatarZhi Wang <zhi.a.wang@intel.com>
Reviewed-by: default avatarZhenyu Wang <zhenyuw@linux.intel.com>
Reviewed-by: default avatarKevin Tian <kevin.tian@intel.com>
Signed-off-by: default avatarZhenyu Wang <zhenyuw@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20221110122034.3382-2-zhi.a.wang@intel.com
parent 704f3384

drivers/gpu/drm/i915/gvt/debugfs.c

+18 −1
Original line number Diff line number Diff line
@@ -151,6 +151,22 @@ DEFINE_SIMPLE_ATTRIBUTE(vgpu_scan_nonprivbb_fops, 
			vgpu_scan_nonprivbb_get, vgpu_scan_nonprivbb_set,

			"0x%llx\n");



static int vgpu_status_get(void *data, u64 *val)

{

	struct intel_vgpu *vgpu = (struct intel_vgpu *)data;



	*val = 0;



	if (test_bit(INTEL_VGPU_STATUS_ATTACHED, vgpu->status))

		*val |= (1 << INTEL_VGPU_STATUS_ATTACHED);

	if (test_bit(INTEL_VGPU_STATUS_ACTIVE, vgpu->status))

		*val |= (1 << INTEL_VGPU_STATUS_ACTIVE);



	return 0;

}



DEFINE_SIMPLE_ATTRIBUTE(vgpu_status_fops, vgpu_status_get, NULL, "0x%llx\n");



/**

 * intel_gvt_debugfs_add_vgpu - register debugfs entries for a vGPU

 * @vgpu: a vGPU
@@ -162,11 +178,12 @@ void intel_gvt_debugfs_add_vgpu(struct intel_vgpu *vgpu) 
	snprintf(name, 16, "vgpu%d", vgpu->id);

	vgpu->debugfs = debugfs_create_dir(name, vgpu->gvt->debugfs_root);



	debugfs_create_bool("active", 0444, vgpu->debugfs, &vgpu->active);

	debugfs_create_file("mmio_diff", 0444, vgpu->debugfs, vgpu,

			    &vgpu_mmio_diff_fops);

	debugfs_create_file("scan_nonprivbb", 0644, vgpu->debugfs, vgpu,

			    &vgpu_scan_nonprivbb_fops);

	debugfs_create_file("status", 0644, vgpu->debugfs, vgpu,

			    &vgpu_status_fops);

}



/**

drivers/gpu/drm/i915/gvt/dmabuf.c

+2 −1
Original line number Diff line number Diff line
@@ -134,7 +134,8 @@ static void dmabuf_gem_object_free(struct kref *kref) 
	struct list_head *pos;

	struct intel_vgpu_dmabuf_obj *dmabuf_obj;



	if (vgpu && vgpu->active && !list_empty(&vgpu->dmabuf_obj_list_head)) {

	if (vgpu && test_bit(INTEL_VGPU_STATUS_ACTIVE, vgpu->status) &&

	    !list_empty(&vgpu->dmabuf_obj_list_head)) {

		list_for_each(pos, &vgpu->dmabuf_obj_list_head) {

			dmabuf_obj = list_entry(pos, struct intel_vgpu_dmabuf_obj, list);

			if (dmabuf_obj == obj) {

drivers/gpu/drm/i915/gvt/gtt.c

+2 −2
Original line number Diff line number Diff line
@@ -55,7 +55,7 @@ static bool intel_gvt_is_valid_gfn(struct intel_vgpu *vgpu, unsigned long gfn) 
	int idx;

	bool ret;



	if (!vgpu->attached)

	if (!test_bit(INTEL_VGPU_STATUS_ATTACHED, vgpu->status))

		return false;



	idx = srcu_read_lock(&kvm->srcu);
@@ -1178,7 +1178,7 @@ static int is_2MB_gtt_possible(struct intel_vgpu *vgpu, 
	if (!HAS_PAGE_SIZES(vgpu->gvt->gt->i915, I915_GTT_PAGE_SIZE_2M))

		return 0;



	if (!vgpu->attached)

	if (!test_bit(INTEL_VGPU_STATUS_ATTACHED, vgpu->status))

		return -EINVAL;

	pfn = gfn_to_pfn(vgpu->vfio_device.kvm, ops->get_pfn(entry));

	if (is_error_noslot_pfn(pfn))

drivers/gpu/drm/i915/gvt/gvt.h

+10 −5
Original line number Diff line number Diff line
@@ -172,13 +172,18 @@ struct intel_vgpu_submission { 


#define KVMGT_DEBUGFS_FILENAME		"kvmgt_nr_cache_entries"



enum {

	INTEL_VGPU_STATUS_ATTACHED = 0,

	INTEL_VGPU_STATUS_ACTIVE,

	INTEL_VGPU_STATUS_NR_BITS,

};



struct intel_vgpu {

	struct vfio_device vfio_device;

	struct intel_gvt *gvt;

	struct mutex vgpu_lock;

	int id;

	bool active;

	bool attached;

	DECLARE_BITMAP(status, INTEL_VGPU_STATUS_NR_BITS);

	bool pv_notified;

	bool failsafe;

	unsigned int resetting_eng;
@@ -467,7 +472,7 @@ void intel_vgpu_write_fence(struct intel_vgpu *vgpu, 


#define for_each_active_vgpu(gvt, vgpu, id) \

	idr_for_each_entry((&(gvt)->vgpu_idr), (vgpu), (id)) \

		for_each_if(vgpu->active)

		for_each_if(test_bit(INTEL_VGPU_STATUS_ACTIVE, vgpu->status))



static inline void intel_vgpu_write_pci_bar(struct intel_vgpu *vgpu,

					    u32 offset, u32 val, bool low)
@@ -725,7 +730,7 @@ static inline bool intel_gvt_mmio_is_cmd_write_patch( 
static inline int intel_gvt_read_gpa(struct intel_vgpu *vgpu, unsigned long gpa,

		void *buf, unsigned long len)

{

	if (!vgpu->attached)

	if (!test_bit(INTEL_VGPU_STATUS_ATTACHED, vgpu->status))

		return -ESRCH;

	return vfio_dma_rw(&vgpu->vfio_device, gpa, buf, len, false);

}
@@ -743,7 +748,7 @@ static inline int intel_gvt_read_gpa(struct intel_vgpu *vgpu, unsigned long gpa, 
static inline int intel_gvt_write_gpa(struct intel_vgpu *vgpu,

		unsigned long gpa, void *buf, unsigned long len)

{

	if (!vgpu->attached)

	if (!test_bit(INTEL_VGPU_STATUS_ATTACHED, vgpu->status))

		return -ESRCH;

	return vfio_dma_rw(&vgpu->vfio_device, gpa, buf, len, true);

}

drivers/gpu/drm/i915/gvt/interrupt.c

+1 −1
Original line number Diff line number Diff line
@@ -433,7 +433,7 @@ static int inject_virtual_interrupt(struct intel_vgpu *vgpu) 
	 * enabled by guest. so if msi_trigger is null, success is still

	 * returned and don't inject interrupt into guest.

	 */

	if (!vgpu->attached)

	if (!test_bit(INTEL_VGPU_STATUS_ATTACHED, vgpu->status))

		return -ESRCH;

	if (vgpu->msi_trigger && eventfd_signal(vgpu->msi_trigger, 1) != 1)

		return -EFAULT;