[PATCH 0/2] GVT-g guest patch for 4.8

[PATCH 0/2] GVT-g guest patch for 4.8

[Zhenyu Wang]

Hi,

Here're two patches for GVT-g guest to fix run against future GVT-g
host driver, which try to ensure 4.8 will be ready to use for VM.

thanks.

Ping Gao (1):
  drm/i915: enable vGPU detection for all

Zhi Wang (1):
  drm/i915: disable 48bit full PPGTT when vGPU is active

 drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
 drivers/gpu/drm/i915/i915_vgpu.c    | 3 ---
 2 files changed, 6 insertions(+), 6 deletions(-)

-- 
2.9.3

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

[PATCH 1/2] drm/i915: enable vGPU detection for all

[Zhenyu Wang]

From: Ping Gao <ping.a.gao@intel.com>

vGPU capability is handled by GVT-g host driver, not needed to
put extra HW check for vGPU detection. And we'll actually support
vGPU from BDW.

Signed-off-by: Ping Gao <ping.a.gao@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
---
 drivers/gpu/drm/i915/i915_vgpu.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_vgpu.c b/drivers/gpu/drm/i915/i915_vgpu.c
index ca2e912..dae340c 100644
--- a/drivers/gpu/drm/i915/i915_vgpu.c
+++ b/drivers/gpu/drm/i915/i915_vgpu.c
@@ -65,9 +65,6 @@ void i915_check_vgpu(struct drm_i915_private *dev_priv)
 
 	BUILD_BUG_ON(sizeof(struct vgt_if) != VGT_PVINFO_SIZE);
 
-	if (!IS_HASWELL(dev_priv))
-		return;
-
 	magic = __raw_i915_read64(dev_priv, vgtif_reg(magic));
 	if (magic != VGT_MAGIC)
 		return;
-- 
2.9.3

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

[PATCH 2/2] drm/i915: disable 48bit full PPGTT when vGPU is active

[Zhenyu Wang]

From: Zhi Wang <zhi.a.wang@intel.com>

Disable 48bit full PPGTT on vGPU too for now.

Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
---
 drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 570e731..e16c380 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -124,8 +124,11 @@ int intel_sanitize_enable_ppgtt(struct drm_i915_private *dev_priv,
 	has_full_48bit_ppgtt =
 	       	IS_BROADWELL(dev_priv) || INTEL_GEN(dev_priv) >= 9;
 
-	if (intel_vgpu_active(dev_priv))
-		has_full_ppgtt = false; /* emulation is too hard */
+	if (intel_vgpu_active(dev_priv)) {
+		/* emulation is too hard */
+		has_full_ppgtt = false;
+		has_full_48bit_ppgtt = false;
+	}
 
 	if (!has_aliasing_ppgtt)
 		return 0;
@@ -160,7 +163,7 @@ int intel_sanitize_enable_ppgtt(struct drm_i915_private *dev_priv,
 		return 0;
 	}
 
-	if (INTEL_GEN(dev_priv) >= 8 && i915.enable_execlists)
+	if (INTEL_GEN(dev_priv) >= 8 && i915.enable_execlists && has_full_ppgtt)
 		return has_full_48bit_ppgtt ? 3 : 2;
 	else
 		return has_aliasing_ppgtt ? 1 : 0;
-- 
2.9.3

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

✗ Fi.CI.BAT: warning for GVT-g guest patch for 4.8

[Patchwork]

== Series Details ==

Series: GVT-g guest patch for 4.8
URL   : https://patchwork.freedesktop.org/series/12026/
State : warning

== Summary ==

Series 12026v1 GVT-g guest patch for 4.8
http://patchwork.freedesktop.org/api/1.0/series/12026/revisions/1/mbox/

Test drv_module_reload_basic:
                pass       -> SKIP       (fi-skl-6260u)
Test kms_pipe_crc_basic:
        Subgroup nonblocking-crc-pipe-b:
                skip       -> PASS       (fi-hsw-4770r)
        Subgroup read-crc-pipe-b:
                skip       -> PASS       (fi-skl-6260u)
        Subgroup read-crc-pipe-c:
                skip       -> PASS       (fi-hsw-4770r)

fi-bdw-5557u     total:252  pass:233  dwarn:2   dfail:1   fail:1   skip:15 
fi-bsw-n3050     total:252  pass:203  dwarn:1   dfail:1   fail:1   skip:46 
fi-byt-n2820     total:252  pass:206  dwarn:2   dfail:1   fail:2   skip:41 
fi-hsw-4770k     total:252  pass:226  dwarn:2   dfail:1   fail:1   skip:22 
fi-hsw-4770r     total:252  pass:222  dwarn:2   dfail:1   fail:1   skip:26 
fi-ivb-3520m     total:252  pass:217  dwarn:2   dfail:1   fail:1   skip:31 
fi-skl-6260u     total:252  pass:233  dwarn:2   dfail:1   fail:1   skip:15 
fi-skl-6700k     total:252  pass:219  dwarn:3   dfail:1   fail:1   skip:28 
fi-snb-2520m     total:252  pass:204  dwarn:2   dfail:1   fail:2   skip:43 
fi-snb-2600      total:252  pass:205  dwarn:2   dfail:1   fail:1   skip:43 

Results at /archive/results/CI_IGT_test/Patchwork_2470/

9baa666b3e48f71b46c5f63541f57d2a95a1b1c0 drm-intel-nightly: 2016y-09m-03d-12h-12m-15s UTC integration manifest
82e85de drm/i915: disable 48bit full PPGTT when vGPU is active
5d4cfb7 drm/i915: enable vGPU detection for all

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

Re: [PATCH 0/2] GVT-g guest patch for 4.8

[Chris Wilson]

On Tue, Sep 06, 2016 at 12:04:10PM +0800, Zhenyu Wang wrote:
> Hi,
> 
> Here're two patches for GVT-g guest to fix run against future GVT-g
> host driver, which try to ensure 4.8 will be ready to use for VM.
> 
> thanks.
> 
> Ping Gao (1):
>   drm/i915: enable vGPU detection for all
> 
> Zhi Wang (1):
>   drm/i915: disable 48bit full PPGTT when vGPU is active
> 
>  drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
>  drivers/gpu/drm/i915/i915_vgpu.c    | 3 ---
>  2 files changed, 6 insertions(+), 6 deletions(-)

Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH 1/2] drm/i915: enable vGPU detection for all

[Joonas Lahtinen]

On ti, 2016-09-06 at 12:04 +0800, Zhenyu Wang wrote:
> From: Ping Gao <ping.a.gao@intel.com>
> 
> vGPU capability is handled by GVT-g host driver, not needed to
> put extra HW check for vGPU detection. And we'll actually support
> vGPU from BDW.
> 
> Signed-off-by: Ping Gao <ping.a.gao@intel.com>
> Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>

> ---
>  drivers/gpu/drm/i915/i915_vgpu.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_vgpu.c b/drivers/gpu/drm/i915/i915_vgpu.c
> index ca2e912..dae340c 100644
> --- a/drivers/gpu/drm/i915/i915_vgpu.c
> +++ b/drivers/gpu/drm/i915/i915_vgpu.c
> @@ -65,9 +65,6 @@ void i915_check_vgpu(struct drm_i915_private *dev_priv)
>  
>  	BUILD_BUG_ON(sizeof(struct vgt_if) != VGT_PVINFO_SIZE);
>  
> -	if (!IS_HASWELL(dev_priv))
> -		return;
> -
>  	magic = __raw_i915_read64(dev_priv, vgtif_reg(magic));
>  	if (magic != VGT_MAGIC)
>  		return;
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH 2/2] drm/i915: disable 48bit full PPGTT when vGPU is active

[Joonas Lahtinen]

On ti, 2016-09-06 at 12:04 +0800, Zhenyu Wang wrote:
> From: Zhi Wang <zhi.a.wang@intel.com>
> 
> Disable 48bit full PPGTT on vGPU too for now.
> 
> Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
> Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH 0/2] GVT-g guest patch for 4.8

[Jani Nikula]

On Tue, 06 Sep 2016, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> On Tue, Sep 06, 2016 at 12:04:10PM +0800, Zhenyu Wang wrote:
>> Hi,
>> 
>> Here're two patches for GVT-g guest to fix run against future GVT-g
>> host driver, which try to ensure 4.8 will be ready to use for VM.
>> 
>> thanks.
>> 
>> Ping Gao (1):
>>   drm/i915: enable vGPU detection for all
>> 
>> Zhi Wang (1):
>>   drm/i915: disable 48bit full PPGTT when vGPU is active
>> 
>>  drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
>>  drivers/gpu/drm/i915/i915_vgpu.c    | 3 ---
>>  2 files changed, 6 insertions(+), 6 deletions(-)
>
> Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
> -Chris

So these are needed for 4.8?

Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH 0/2] GVT-g guest patch for 4.8

[Zhenyu Wang]

[-- Attachment #1.1: Type: text/plain, Size: 1059 bytes --]

On 2016.09.06 11:33:57 +0300, Jani Nikula wrote:
> On Tue, 06 Sep 2016, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> > On Tue, Sep 06, 2016 at 12:04:10PM +0800, Zhenyu Wang wrote:
> >> Hi,
> >> 
> >> Here're two patches for GVT-g guest to fix run against future GVT-g
> >> host driver, which try to ensure 4.8 will be ready to use for VM.
> >> 
> >> thanks.
> >> 
> >> Ping Gao (1):
> >>   drm/i915: enable vGPU detection for all
> >> 
> >> Zhi Wang (1):
> >>   drm/i915: disable 48bit full PPGTT when vGPU is active
> >> 
> >>  drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
> >>  drivers/gpu/drm/i915/i915_vgpu.c    | 3 ---
> >>  2 files changed, 6 insertions(+), 6 deletions(-)
> >
> > Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
> > -Chris
> 
> So these are needed for 4.8?
> 

Yeah, we'd like to support any distribution that will base on 4.8 kernel
for guest VM. So would require these changes.

thanks

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

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

Re: [PATCH 0/2] GVT-g guest patch for 4.8

[Jani Nikula]

On Tue, 06 Sep 2016, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> [ Unknown signature status ]
> On 2016.09.06 11:33:57 +0300, Jani Nikula wrote:
>> On Tue, 06 Sep 2016, Chris Wilson <chris@chris-wilson.co.uk> wrote:
>> > On Tue, Sep 06, 2016 at 12:04:10PM +0800, Zhenyu Wang wrote:
>> >> Hi,
>> >> 
>> >> Here're two patches for GVT-g guest to fix run against future GVT-g
>> >> host driver, which try to ensure 4.8 will be ready to use for VM.
>> >> 
>> >> thanks.
>> >> 
>> >> Ping Gao (1):
>> >>   drm/i915: enable vGPU detection for all
>> >> 
>> >> Zhi Wang (1):
>> >>   drm/i915: disable 48bit full PPGTT when vGPU is active
>> >> 
>> >>  drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
>> >>  drivers/gpu/drm/i915/i915_vgpu.c    | 3 ---
>> >>  2 files changed, 6 insertions(+), 6 deletions(-)
>> >
>> > Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
>> > -Chris
>> 
>> So these are needed for 4.8?
>> 
>
> Yeah, we'd like to support any distribution that will base on 4.8 kernel
> for guest VM. So would require these changes.

Both pushed to dinq with Cc: drm-intel-fixes, thanks for the patches and
review.

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH 0/2] GVT-g guest patch for 4.8

[Jani Nikula]

On Tue, 06 Sep 2016, Jani Nikula <jani.nikula@linux.intel.com> wrote:
> On Tue, 06 Sep 2016, Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
>> [ Unknown signature status ]
>> On 2016.09.06 11:33:57 +0300, Jani Nikula wrote:
>>> On Tue, 06 Sep 2016, Chris Wilson <chris@chris-wilson.co.uk> wrote:
>>> > On Tue, Sep 06, 2016 at 12:04:10PM +0800, Zhenyu Wang wrote:
>>> >> Hi,
>>> >> 
>>> >> Here're two patches for GVT-g guest to fix run against future GVT-g
>>> >> host driver, which try to ensure 4.8 will be ready to use for VM.
>>> >> 
>>> >> thanks.
>>> >> 
>>> >> Ping Gao (1):
>>> >>   drm/i915: enable vGPU detection for all
>>> >> 
>>> >> Zhi Wang (1):
>>> >>   drm/i915: disable 48bit full PPGTT when vGPU is active
>>> >> 
>>> >>  drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
>>> >>  drivers/gpu/drm/i915/i915_vgpu.c    | 3 ---
>>> >>  2 files changed, 6 insertions(+), 6 deletions(-)
>>> >
>>> > Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
>>> > -Chris
>>> 
>>> So these are needed for 4.8?
>>> 
>>
>> Yeah, we'd like to support any distribution that will base on 4.8 kernel
>> for guest VM. So would require these changes.
>
> Both pushed to dinq with Cc: drm-intel-fixes, thanks for the patches and
> review.

And picked up to fixes to be queued for 4.8.

>
> BR,
> Jani.

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx