All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] A possible hotfix for aliasing PPGTT in kernel 4.11.x or 4.10.x
@ 2017-02-08 13:03 Zhi Wang
  2017-02-08 13:03 ` [PATCH] drm/i915: A hotfix for making aliasing PPGTT work Zhi Wang
                   ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: Zhi Wang @ 2017-02-08 13:03 UTC (permalink / raw)
  To: intel-gfx

Redhat guys have choose kernel stable 4.10 as their target. As you know,
currently the aliasing PPGTT mode required by GVT-g doesn't work now.

Thanks Chris who has already pushed the fixes, but it seems that the
kernel 4.11 merging window has already open and those fixes might be
merged in 4.12.x possibly which is too late for us. T_T

After some discussions with Zhiyuan and Zhenyu, we wonder that if it's
possible to send this hotfix patch to kernel 4.11 *fixes* to make
aliasing PPGTT work temporarily and after Chris' fixes merged in future,
we or he can revert or drop this patch then. Is this possible?

Or we can see if there is any option for us. Thank you guys!

Zhi Wang (1):
  drm/i915: A hotfix for making aliasing PPGTT work

 drivers/gpu/drm/i915/i915_gem_gtt.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

-- 
1.9.1

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

^ permalink raw reply	[flat|nested] 13+ messages in thread
* [PATCH] drm/i915: A hotfix for making aliasing PPGTT work
@ 2017-02-09  9:33 Zhi Wang
  0 siblings, 0 replies; 13+ messages in thread
From: Zhi Wang @ 2017-02-09  9:33 UTC (permalink / raw)
  To: intel-gfx; +Cc: Daniel Vetter, drm-intel-fixes, Daniel Vetter, Zhiyuan Lv

This patch makes PPGTT page table non-shrinkable when using aliasing PPGTT
mode. It's just a temporary solution for making aliasing PPGTT mode work.

v2:
- Add fixes tag. (Chris and Daniel)

Fixes commit 2ce5179fe826 ("drm/i915/gtt: Free unused lower-level page tables")
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
Cc: Michel Thierry <michel.thierry@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
Cc: Zhiyuan Lv <zhiyuan.lv@intel.com>
Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1486559013-25251-2-git-send-email-zhi.a.wang@intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: <drm-intel-fixes@lists.freedesktop.org> # v4.10-rc1+
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/i915_gem_gtt.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 22b3374..3233f12 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -756,8 +756,10 @@ static bool gen8_ppgtt_clear_pt(struct i915_address_space *vm,
 
 	bitmap_clear(pt->used_ptes, pte, num_entries);
 
-	if (bitmap_empty(pt->used_ptes, GEN8_PTES))
-		return true;
+	if (USES_FULL_PPGTT(vm->i915)) {
+		if (bitmap_empty(pt->used_ptes, GEN8_PTES))
+			return true;
+	}
 
 	pt_vaddr = kmap_px(pt);
 
-- 
1.9.1

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

^ permalink raw reply related	[flat|nested] 13+ messages in thread
* [PATCH] drm/i915: A hotfix for making aliasing PPGTT work
@ 2017-02-10  1:44 ` Zhi Wang
  0 siblings, 0 replies; 13+ messages in thread
From: Zhi Wang @ 2017-02-10  1:44 UTC (permalink / raw)
  To: intel-gfx
  Cc: Zhi Wang, Jani Nikula, Daniel Vetter, Tvrtko Ursulin,
	Michal Winiarski, Michel Thierry, Joonas Lahtinen, Chris Wilson,
	Daniel Vetter, Zhenyu Wang, Zhiyuan Lv, drm-intel-fixes, stable

This patch makes PPGTT page table non-shrinkable when using aliasing PPGTT
mode. It's just a temporary solution for making aliasing PPGTT mode work.

v4:
- Add Cc to stable kernel.

v3:
- Fix the format of "Fixes line".

v2:
- Add fixes tag. (Chris and Daniel)

Fixes: 2ce5179fe826 ("drm/i915/gtt: Free unused lower-level page tables")
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
Cc: Michel Thierry <michel.thierry@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
Cc: Zhiyuan Lv <zhiyuan.lv@intel.com>
Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1486559013-25251-2-git-send-email-zhi.a.wang@intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: <drm-intel-fixes@lists.freedesktop.org> # v4.10-rc1+
Cc: <stable@vger.kernel.org> # v4.10
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/i915_gem_gtt.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 22b3374..3233f12 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -756,8 +756,10 @@ static bool gen8_ppgtt_clear_pt(struct i915_address_space *vm,
 
 	bitmap_clear(pt->used_ptes, pte, num_entries);
 
-	if (bitmap_empty(pt->used_ptes, GEN8_PTES))
-		return true;
+	if (USES_FULL_PPGTT(vm->i915)) {
+		if (bitmap_empty(pt->used_ptes, GEN8_PTES))
+			return true;
+	}
 
 	pt_vaddr = kmap_px(pt);
 
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2017-02-10  1:54 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-08 13:03 [PATCH] A possible hotfix for aliasing PPGTT in kernel 4.11.x or 4.10.x Zhi Wang
2017-02-08 13:03 ` [PATCH] drm/i915: A hotfix for making aliasing PPGTT work Zhi Wang
2017-02-09  7:57   ` Chris Wilson
2017-02-09  8:27     ` Chris Wilson
2017-02-09  8:10   ` Chris Wilson
2017-02-09  8:46     ` Zhi Wang
2017-02-08 13:22 ` [PATCH] A possible hotfix for aliasing PPGTT in kernel 4.11.x or 4.10.x Chris Wilson
2017-02-09  2:32   ` Zhi Wang
2017-02-09  7:51     ` Chris Wilson
2017-02-09  7:54 ` ✓ Fi.CI.BAT: success for drm/i915: A hotfix for making aliasing PPGTT work Patchwork
2017-02-09  9:33 [PATCH] " Zhi Wang
2017-02-10  1:44 Zhi Wang
2017-02-10  1:44 ` Zhi Wang

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.