Re: [PATCH] drm/i915: Pre-populate the cursor physical dma address

2024-03-26 Thread Ville Syrjälä
On Mon, Mar 25, 2024 at 07:21:35PM +0100, Borislav Petkov wrote: > On Mon, Mar 25, 2024 at 07:57:38PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Calling i915_gem_object_get_dma_address() from the vblank > > evade critical section triggers might_sleep(). > > > > While we know

Re: [PATCH] drm/i915: Pre-populate the cursor physical dma address

2024-03-26 Thread Ville Syrjälä
day, March 25, 2024 11:28 PM > > > To: intel-gfx@lists.freedesktop.org > > > Cc: sta...@vger.kernel.org; Borislav Petkov > > > Subject: [PATCH] drm/i915: Pre-populate the cursor physical dma address > > > > > > From: Ville Syrjälä > > >

Re: [PATCH] drm/i915: Pre-populate the cursor physical dma address

2024-03-26 Thread Ville Syrjälä
el.org; Borislav Petkov > > Subject: [PATCH] drm/i915: Pre-populate the cursor physical dma address > > > > From: Ville Syrjälä > > > > Calling i915_gem_object_get_dma_address() from the vblank evade critical > > section triggers might_sleep(). &

RE: [PATCH] drm/i915: Pre-populate the cursor physical dma address

2024-03-25 Thread Borah, Chaitanya Kumar
> -Original Message- > From: Intel-gfx On Behalf Of Ville > Syrjala > Sent: Monday, March 25, 2024 11:28 PM > To: intel-gfx@lists.freedesktop.org > Cc: sta...@vger.kernel.org; Borislav Petkov > Subject: [PATCH] drm/i915: Pre-populate the cursor physical dma add

Re: [PATCH] drm/i915: Pre-populate the cursor physical dma address

2024-03-25 Thread Borislav Petkov
On Mon, Mar 25, 2024 at 07:57:38PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Calling i915_gem_object_get_dma_address() from the vblank > evade critical section triggers might_sleep(). > > While we know that we've already pinned the framebuffer > and thus

[PATCH] drm/i915: Pre-populate the cursor physical dma address

2024-03-25 Thread Ville Syrjala
From: Ville Syrjälä Calling i915_gem_object_get_dma_address() from the vblank evade critical section triggers might_sleep(). While we know that we've already pinned the framebuffer and thus i915_gem_object_get_dma_address() will in fact not sleep in this case, it seems reasonable to keep the