Re: [PATCH v7 1/3] i915/gvt: Separate the MMIO tracking table from GVT-g

Christoph Hellwig
2022-03-31T07:42:38Z

[-- Attachment #1: Type: text/plain, Size: 602 bytes --]

On Thu, Mar 31, 2022 at 07:11:07AM +0000, Wang, Zhi A wrote:
> Hi Jani and Joonas:
>
> Are you OK with these patches? I noticed I need to change the license of the new file. Will do that when check-in if you are OK with these.

So I actually ended up testing the patches, and first they fail to
compile against current mainline (trivial fix attached), but then
the guest (also latest mainline as of today) also hangs during the
initialization of th i915 driver on i7-8550U-based Thinkpad. Plain
mainline as the host boots fine, but spews a lot of warnings.

Host and guest configs are also attached.

[-- Attachment #2: gvt-compile-fix.diff --]
[-- Type: text/x-patch, Size: 575 bytes --]

commit 659b61b08dc3c66228a9dd139068bf2ca5096e13
Author: Christoph Hellwig <hch@lst.de>
Date: Thu Mar 31 09:17:09 2022 +0200

fix

diff --git a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
index 40b844eb07263..007b965067ea1 100644
--- a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
+++ b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
@@ -24,6 +24,7 @@

#include "i915_drv.h"
#include "i915_reg.h"
+#include "display/vlv_dsi_pll_regs.h"
#include "gt/intel_gt_regs.h"
#include "intel_mchbar_regs.h"
#include "i915_pvinfo.h"

[-- Attachment #3: config.host.gz --]
[-- Type: application/x-gzip, Size: 39896 bytes --]

[-- Attachment #4: config.guest.gz --]
[-- Type: application/x-gzip, Size: 36972 bytes --]