[Intel-gfx] [PATCH v3 4/5] drm/i915/fbdev: lock the fbdev obj before vma pin

[Sripada, Radhakrishna]

Thank you for the Review. Pushed.

-Radhakrishna Sripada

> -----Original Message-----
> From: Andi Shyti <andi.shyti at linux.intel.com>
> Sent: Thursday, March 9, 2023 9:18 AM
> To: Sripada, Radhakrishna <radhakrishna.sripada at intel.com>
> Cc: intel-gfx at lists.freedesktop.org; Auld, Matthew <matthew.auld at intel.com>;
> Wilson, Chris P <chris.p.wilson at intel.com>
> Subject: Re: [Intel-gfx] [PATCH v3 4/5] drm/i915/fbdev: lock the fbdev obj
> before vma pin
> 
> Hi Radhakrishna,
> 
> On Wed, Mar 01, 2023 at 12:10:52PM -0800, Radhakrishna Sripada wrote:
> > From: Tejas Upadhyay <tejas.upadhyay at intel.com>
> >
> > lock the fbdev obj before calling into
> > i915_vma_pin_iomap(). This helps to solve below :
> >
> > <7>[   93.563308] i915 0000:00:02.0: [drm:intelfb_create [i915]] no BIOS fb,
> allocating a new one
> > <4>[   93.581844] ------------[ cut here ]------------
> > <4>[   93.581855] WARNING: CPU: 12 PID: 625 at
> drivers/gpu/drm/i915/gem/i915_gem_pages.c:424
> i915_gem_object_pin_map+0x152/0x1c0 [i915]
> >
> > Fixes: f0b6b01b3efe ("drm/i915: Add ww context to intel_dpt_pin, v2.")
> > Cc: Chris Wilson <chris.p.wilson at intel.com>
> > Cc: Matthew Auld <matthew.auld at intel.com>
> > Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> > Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
> > Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada at intel.com>
> 
> Reviewed-by: Andi Shyti <andi.shyti at linux.intel.com>
> 
> Andi