to enable vgpu pv feature, pv capability is introduced for guest by
new pv_caps member in struct i915_virtual_gpu and for host GVT by
new pv_caps register in struct vgt_if.

both of them are used to control different pv feature support in each
domain and the final pv caps runtime negotiated between guest and host.

it also adds VGT_CAPS_PV capability BIT useb by guest to query host GVTg
whether support any PV feature or not.

v2: addressed dim checkpatch issues and Jani Nikula's comment.

Signed-off-by: Xiaolin Zhang <xiaolin.zh...@intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h    |  1 +
 drivers/gpu/drm/i915/i915_pvinfo.h |  5 ++-
 drivers/gpu/drm/i915/i915_vgpu.c   | 64 +++++++++++++++++++++++++++++++++++++-
 drivers/gpu/drm/i915/i915_vgpu.h   | 10 ++++++
 4 files changed, 78 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index eef9a82..b854a53 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -799,6 +799,7 @@ struct i915_virtual_gpu {
        struct mutex lock; /* serialises sending of g2v_notify command pkts */
        bool active;
        u32 caps;
+       u32 pv_caps;
 };
 
 struct intel_cdclk_config {
diff --git a/drivers/gpu/drm/i915/i915_pvinfo.h 
b/drivers/gpu/drm/i915/i915_pvinfo.h
index 683e97a..8b0dc25 100644
--- a/drivers/gpu/drm/i915/i915_pvinfo.h
+++ b/drivers/gpu/drm/i915/i915_pvinfo.h
@@ -57,6 +57,7 @@ enum vgt_g2v_type {
 #define VGT_CAPS_FULL_PPGTT            BIT(2)
 #define VGT_CAPS_HWSP_EMULATION                BIT(3)
 #define VGT_CAPS_HUGE_GTT              BIT(4)
+#define VGT_CAPS_PV                    BIT(5)
 
 struct vgt_if {
        u64 magic;              /* VGT_MAGIC */
@@ -109,7 +110,9 @@ struct vgt_if {
        u32 execlist_context_descriptor_lo;
        u32 execlist_context_descriptor_hi;
 
-       u32  rsv7[0x200 - 24];    /* pad to one page */
+       u32 pv_caps;
+
+       u32  rsv7[0x200 - 25];    /* pad to one page */
 } __packed;
 
 #define vgtif_offset(x) (offsetof(struct vgt_if, x))
diff --git a/drivers/gpu/drm/i915/i915_vgpu.c b/drivers/gpu/drm/i915/i915_vgpu.c
index 70fca72..d0c9cee 100644
--- a/drivers/gpu/drm/i915/i915_vgpu.c
+++ b/drivers/gpu/drm/i915/i915_vgpu.c
@@ -98,7 +98,15 @@ void intel_vgpu_detect(struct drm_i915_private *dev_priv)
 
        dev_priv->vgpu.active = true;
        mutex_init(&dev_priv->vgpu.lock);
-       drm_info(&dev_priv->drm, "Virtual GPU for Intel GVT-g detected.\n");
+
+       if (intel_vgpu_pv_detect_caps(dev_priv, shared_area)) {
+               drm_info(&dev_priv->drm,
+                        "Virtual GPU for Intel GVT-g detected with PV 
Capabilities.\n");
+               drm_dbg(&dev_priv->drm, "VGPU pv caps: 0x%x\n", 
dev_priv->vgpu.pv_caps);
+       } else {
+               drm_info(&dev_priv->drm,
+                        "Virtual GPU for Intel GVT-g detected.\n");
+       }
 
 out:
        pci_iounmap(pdev, shared_area);
@@ -134,6 +142,19 @@ bool intel_vgpu_has_huge_gtt(struct drm_i915_private 
*dev_priv)
        return dev_priv->vgpu.caps & VGT_CAPS_HUGE_GTT;
 }
 
+static bool intel_vgpu_pv_check_cap(struct drm_i915_private *dev_priv,
+                                   enum intel_vgpu_pv_caps cap)
+{
+       return (dev_priv->vgpu.active &&
+               (dev_priv->vgpu.caps & VGT_CAPS_PV) &&
+               (dev_priv->vgpu.pv_caps & cap));
+}
+
+static bool intel_vgpu_has_pv_caps(struct drm_i915_private *dev_priv)
+{
+       return dev_priv->vgpu.caps & VGT_CAPS_PV;
+}
+
 struct _balloon_info_ {
        /*
         * There are up to 2 regions per mappable/unmappable graphic
@@ -336,3 +357,44 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt)
        drm_err(&dev_priv->drm, "VGT balloon fail\n");
        return ret;
 }
+
+/*
+ * i915 vgpu PV support for Linux
+ */
+
+/*
+ * Config vgpu PV ops for different PV capabilities
+ */
+void intel_vgpu_pv_config_caps(struct drm_i915_private *i915,
+                              enum intel_vgpu_pv_caps cap, void *data)
+{
+       if (!intel_vgpu_pv_check_cap(i915, cap))
+               return;
+}
+
+/*
+ * intel_vgpu_pv_detect_caps - detect virtual GPU PV capabilities
+ *
+ * This function is called at the initialization stage to detect VGPU
+ * PV capabilities
+ */
+bool intel_vgpu_pv_detect_caps(struct drm_i915_private *i915,
+                              void __iomem *shared_area)
+{
+       u32 gvt_pvcaps;
+       u32 pvcaps = 0;
+
+       if (!intel_vgpu_has_pv_caps(i915))
+               return false;
+
+       /* PV capability negotiation between PV guest and GVT */
+       gvt_pvcaps = readl(shared_area + vgtif_offset(pv_caps));
+       pvcaps = i915->vgpu.pv_caps & gvt_pvcaps;
+       i915->vgpu.pv_caps = pvcaps;
+       writel(pvcaps, shared_area + vgtif_offset(pv_caps));
+
+       if (!pvcaps)
+               return false;
+
+       return true;
+}
diff --git a/drivers/gpu/drm/i915/i915_vgpu.h b/drivers/gpu/drm/i915/i915_vgpu.h
index ffbb77d..1dccea6 100644
--- a/drivers/gpu/drm/i915/i915_vgpu.h
+++ b/drivers/gpu/drm/i915/i915_vgpu.h
@@ -29,6 +29,11 @@
 struct drm_i915_private;
 struct i915_ggtt;
 
+/* intel vGPU PV capabilities */
+enum intel_vgpu_pv_caps {
+       PV_NONE = 0,
+};
+
 void intel_vgpu_detect(struct drm_i915_private *i915);
 bool intel_vgpu_active(struct drm_i915_private *i915);
 void intel_vgpu_register(struct drm_i915_private *i915);
@@ -39,4 +44,9 @@ bool intel_vgpu_has_huge_gtt(struct drm_i915_private *i915);
 int intel_vgt_balloon(struct i915_ggtt *ggtt);
 void intel_vgt_deballoon(struct i915_ggtt *ggtt);
 
+/* i915 vgpu pv related functions */
+bool intel_vgpu_pv_detect_caps(struct drm_i915_private *i915,
+                              void __iomem *shared_area);
+void intel_vgpu_pv_config_caps(struct drm_i915_private *i915,
+                              enum intel_vgpu_pv_caps cap, void *data);
 #endif /* _I915_VGPU_H_ */
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to