Messages in this thread | | | From | "Wang, Zhi A" <> | Subject | Re: [PATCH v4 1/2] i915/gvt: Introduce the mmio_info_table.c to support VFIO new mdev API | Date | Fri, 17 Dec 2021 08:52:53 +0000 |
| |
On 11/30/2021 6:46 PM, Christoph Hellwig wrote: > I still think this goes into the wrong direction. > > Something closer to your first version that also saves away the > gvt->mmio.mmio_attribute flags in the core i915 module, and which > splits the MMIO table into one that contains just the offset, size > and flags (core i915) and one that has the read-only mask and handlers > (gvt) would be much simpler and not create this super-tight coupling > between core i915 and gvt. > > Bonus points for moving your new intel_gvt_hw_state structure out > of struct intel_gvt and into struct i915_virtual_gpu.
Hi Christoph:
Sorry for the late reply as I am supporting the customers recently. I will refresh this after the christmas.
Thanks,
Zhi.
| |