RE: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-11 Thread Zhang, Carl
Wilson ; Roper, Matthew D ; Justen, Jordan L ; Gu, Lihao ; Intel GFX ; DRI Devel Subject: Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation > Hi Carl, > >>>> besides this, ask a dumb question. >>>> How we retrieve the pat_index from a sha

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-09 Thread Yang, Fei
> Hi Carl, > besides this, ask a dumb question. How we retrieve the pat_index from a shared resource though dma_buf fd? maybe we need to know whether it could be CPU cached if we want map it. Of course, looks there are no real usage to access it though CPU. Just use it

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-09 Thread Andi Shyti
Hi Carl, > > > besides this, ask a dumb question. > > > How we retrieve the pat_index from a shared resource though dma_buf fd? > > > maybe we need to know whether it could be CPU cached if we want map it. > > > Of course, looks there are no real usage to access it though CPU. > > > Just use it

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-07 Thread Andi Shyti
Hi Fei, On Tue, Jun 06, 2023 at 12:00:42PM +0200, Andi Shyti wrote: > From: Fei Yang > > To comply with the design that buffer objects shall have immutable > cache setting through out their life cycle, {set, get}_caching ioctl's > are no longer supported from MTL onward. With that change

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-07 Thread Andi Shyti
On Tue, Jun 06, 2023 at 12:00:42PM +0200, Andi Shyti wrote: > From: Fei Yang > > To comply with the design that buffer objects shall have immutable > cache setting through out their life cycle, {set, get}_caching ioctl's > are no longer supported from MTL onward. With that change caching >

RE: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Zhang, Carl
DRI Devel > Subject: Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation > > Hi Carl, > > On Wed, Jun 07, 2023 at 03:40:20AM +, Zhang, Carl wrote: > > Media driver reverted previous patches, and file a new PR > > https://github.com/intel/media-d

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Andi Shyti
gt; > ; Justen, Jordan L ; > > Zhang, Carl ; Gu, Lihao ; Intel > > GFX ; DRI Devel > de...@lists.freedesktop.org> > > Subject: Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO > > creation > > > > > > > > To comply with the design that buffer objects

RE: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Zhang, Carl
nas Lahtinen > Cc: Andi Shyti ; Tvrtko Ursulin > ; Yang, Fei ; Chris > Wilson ; Roper, Matthew D > ; Justen, Jordan L ; > Zhang, Carl ; Gu, Lihao ; Intel > GFX ; DRI Devel de...@lists.freedesktop.org> > Subject: Re: [PATCH v17 1/1] drm/i915: Allow user to set cache a

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Andi Shyti
> > > > To comply with the design that buffer objects shall have immutable > > > > cache setting through out their life cycle, {set, get}_caching ioctl's > > > > are no longer supported from MTL onward. With that change caching > > > > policy can only be set at object creation time. The current

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Joonas Lahtinen
Quoting Andi Shyti (2023-06-06 13:18:06) > On Tue, Jun 06, 2023 at 11:10:04AM +0100, Tvrtko Ursulin wrote: > > > > On 06/06/2023 11:00, Andi Shyti wrote: > > > From: Fei Yang > > > > > > To comply with the design that buffer objects shall have immutable > > > cache setting through out their

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Andi Shyti
On Tue, Jun 06, 2023 at 11:10:04AM +0100, Tvrtko Ursulin wrote: > > On 06/06/2023 11:00, Andi Shyti wrote: > > From: Fei Yang > > > > To comply with the design that buffer objects shall have immutable > > cache setting through out their life cycle, {set, get}_caching ioctl's > > are no longer

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Tvrtko Ursulin
On 06/06/2023 11:00, Andi Shyti wrote: From: Fei Yang To comply with the design that buffer objects shall have immutable cache setting through out their life cycle, {set, get}_caching ioctl's are no longer supported from MTL onward. With that change caching policy can only be set at object

[PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-06 Thread Andi Shyti
From: Fei Yang To comply with the design that buffer objects shall have immutable cache setting through out their life cycle, {set, get}_caching ioctl's are no longer supported from MTL onward. With that change caching policy can only be set at object creation time. The current code applies a