Re: [Intel-gfx] [PATCH v5] drm/i915/gvt: Change KVMGT as self load module

2018-12-06 Thread Yuan, Hang
Looks good to me. Thanks. Reviewed-by: Yuan, Hang > -Original Message- > From: Zhenyu Wang [mailto:zhen...@linux.intel.com] > Sent: Thursday, December 6, 2018 4:03 PM > To: intel-gfx@lists.freedesktop.org > Cc: intel-gvt-...@lists.freedesktop.org; Yuan, Hang ; &g

Re: [Intel-gfx] [PATCH v4] drm/i915/gvt: Change KVMGT as self load module

2018-12-05 Thread Yuan, Hang
l_gvt_register_hypervisor(struct intel_gvt_mpt *m) { > + int ret; > + void *gvt; > + > + if (!intel_gvt_host.initialized) > + return -ENODEV; > + > + if (m->type != INTEL_GVT_HYPERVISOR_KVM && > + m->type != INT

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gvt: Avoid use-after-free iterating the gtt list

2018-11-20 Thread Yuan, Hang
Sorry I missed it. Thanks for the correction! Regards, Henry > -Original Message- > From: Zhenyu Wang [mailto:zhen...@linux.intel.com] > Sent: Wednesday, November 21, 2018 10:29 AM > To: Chris Wilson > Cc: intel-gfx@lists.freedesktop.org; Zhenyu Wang > ; Yuan, H

Re: [Intel-gfx] [FW: [PATCH 3/6] drm/i915/gtt: Disable read-only support under GVT]

2018-10-25 Thread Yuan, Hang
> -Original Message- > From: intel-gvt-dev [mailto:intel-gvt-dev-boun...@lists.freedesktop.org] On > Behalf Of Chris Wilson > Sent: Thursday, October 25, 2018 8:51 PM > To: Yuan, Hang > Cc: intel-gfx@lists.freedesktop.org; joonas.lahti...@linux.intel.com; > matthew.w

Re: [Intel-gfx] [FW: [PATCH 3/6] drm/i915/gtt: Disable read-only support under GVT]

2018-10-25 Thread Yuan, Hang
Hi Chris, I checked GVT shadow PPGTT handling codes and think GVT does propagate guest PTE flags to shadow PPGTT entry. Then I run i915 self-test case "igt_ctx_readonly" in KVMGT's guest. The case doesn't report any errors. Can I know if there are any other reasons that you think GVT breaks