Re: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-12-03 Thread Brian Geffon
On Tue, Dec 3, 2024 at 3:25 AM Ville Syrjälä wrote: > > On Tue, Dec 03, 2024 at 10:20:23AM +0200, Ville Syrjälä wrote: > > On Mon, Dec 02, 2024 at 10:40:36AM -0500, Brian Geffon wrote: > > > On Wed, Nov 27, 2024 at 1:11 AM Ville Syrjala > > > wrote: > > > > > > > > From: Ville Syrjälä > > > > >

Re: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-12-03 Thread Ville Syrjälä
On Tue, Dec 03, 2024 at 10:20:23AM +0200, Ville Syrjälä wrote: > On Mon, Dec 02, 2024 at 10:40:36AM -0500, Brian Geffon wrote: > > On Wed, Nov 27, 2024 at 1:11 AM Ville Syrjala > > wrote: > > > > > > From: Ville Syrjälä > > > > > > Currently intel_dpt_resume() tries to blindly rewrite all the > >

Re: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-12-03 Thread Ville Syrjälä
On Mon, Dec 02, 2024 at 10:40:36AM -0500, Brian Geffon wrote: > On Wed, Nov 27, 2024 at 1:11 AM Ville Syrjala > wrote: > > > > From: Ville Syrjälä > > > > Currently intel_dpt_resume() tries to blindly rewrite all the > > PTEs for currently bound DPT VMAs. That is problematic because > > the CPU m

Re: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-12-02 Thread Brian Geffon
On Wed, Nov 27, 2024 at 1:11 AM Ville Syrjala wrote: > > From: Ville Syrjälä > > Currently intel_dpt_resume() tries to blindly rewrite all the > PTEs for currently bound DPT VMAs. That is problematic because > the CPU mapping for the DPT is only really guaranteed to exist > while the DPT object h

RE: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-11-27 Thread Srinivas, Vidya
Hello Ville, Thank you so much. Changes Look Good to me. Reviewed-by: Vidya Srinivas > -Original Message- > From: Ville Syrjala > Sent: 27 November 2024 11:41 > To: intel-gfx@lists.freedesktop.org > Cc: Brian Geffon ; Srinivas, Vidya > > Subject: [PATCH 3/4] dr

RE: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-11-27 Thread Srinivas, Vidya
> -Original Message- > From: Ville Syrjala > Sent: 27 November 2024 11:41 > To: intel-gfx@lists.freedesktop.org > Cc: Brian Geffon ; Srinivas, Vidya > > Subject: [PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend > > From: Ville Syrjälä > > Curr

[PATCH 3/4] drm/i915/dpt: Evict all DPT VMAs on suspend

2024-11-26 Thread Ville Syrjala
From: Ville Syrjälä Currently intel_dpt_resume() tries to blindly rewrite all the PTEs for currently bound DPT VMAs. That is problematic because the CPU mapping for the DPT is only really guaranteed to exist while the DPT object has been pinned. In the past we worked around this issue by making D