Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-19 Thread Faith Ekstrand
On Mon, Aug 19, 2024 at 10:00 AM Bas Nieuwenhuizen wrote: > > > On Mon, Aug 19, 2024 at 4:51 PM Christian König < > ckoenig.leichtzumer...@gmail.com> wrote: > >> Am 07.08.24 um 22:33 schrieb Bas Nieuwenhuizen: >> >> On Wed, Aug 7, 2024 at 10:25 PM Faith Ekstrand >> wrote: >> >>> On Wed, Aug 7, 2

Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-19 Thread Bas Nieuwenhuizen
On Mon, Aug 19, 2024 at 4:51 PM Christian König < ckoenig.leichtzumer...@gmail.com> wrote: > Am 07.08.24 um 22:33 schrieb Bas Nieuwenhuizen: > > On Wed, Aug 7, 2024 at 10:25 PM Faith Ekstrand > wrote: > >> On Wed, Aug 7, 2024 at 2:23 PM Joshua Ashton wrote: >> >>> I was thinking about this more

Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-19 Thread Christian König
Am 07.08.24 um 22:33 schrieb Bas Nieuwenhuizen: On Wed, Aug 7, 2024 at 10:25 PM Faith Ekstrand wrote: On Wed, Aug 7, 2024 at 2:23 PM Joshua Ashton wrote: I was thinking about this more recently. I was initially considering "maybe this should be a per-BO import," but I

Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-08 Thread Michel Dänzer
On 2024-08-07 21:23, Joshua Ashton wrote: > I was thinking about this more recently. I was initially considering "maybe > this should be a per-BO import," but I couldn't think of anything in the GL > model that would actually benefit given its not "true" bindless and there's > no update-after-bi

Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-07 Thread Bas Nieuwenhuizen
On Wed, Aug 7, 2024 at 10:25 PM Faith Ekstrand wrote: > On Wed, Aug 7, 2024 at 2:23 PM Joshua Ashton wrote: > >> I was thinking about this more recently. I was initially considering >> "maybe this should be a per-BO import," but I couldn't think of anything in >> the GL model that would actually

Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-07 Thread Faith Ekstrand
On Wed, Aug 7, 2024 at 2:23 PM Joshua Ashton wrote: > I was thinking about this more recently. I was initially considering > "maybe this should be a per-BO import," but I couldn't think of anything in > the GL model that would actually benefit given its not "true" bindless and > there's no update

Re: [RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-07 Thread Joshua Ashton
I was thinking about this more recently. I was initially considering "maybe this should be a per-BO import," but I couldn't think of anything in the GL model that would actually benefit given its not "true" bindless and there's no update-after-bind there. Worth others more familiar with GL aski

[RFC] amdgpu: Add a context flag to disable implicit sync

2024-08-07 Thread Faith Ekstrand
Previously, AMDGPU_GEM_CREATE_EXPLICIT_SYNC was used to disable implicit synchronization on BOs when explicit synchronization can be used. The problem is that this flag is per-BO and affects all amdgpu users in the system, not just the usermode drver which sets it. This can lead to some unintende