Re: [PATCH] drm/nouveau: select FW caching

2025-02-18 Thread Dave Airlie
On Wed, 19 Feb 2025 at 07:39, Luis Chamberlain wrote: > > On Tue, Feb 18, 2025 at 06:24:43PM +0100, Danilo Krummrich wrote: > > On Tue, Feb 18, 2025 at 06:28:53AM -0800, Luis Chamberlain wrote: > > > > > > What I recommend is to look into why we disable it by default, I think > > > > I think FW_CA

Re: [PATCH] drm/nouveau: select FW caching

2025-02-18 Thread Luis Chamberlain
On Tue, Feb 18, 2025 at 06:24:43PM +0100, Danilo Krummrich wrote: > On Tue, Feb 18, 2025 at 06:28:53AM -0800, Luis Chamberlain wrote: > > > > What I recommend is to look into why we disable it by default, I think > > I think FW_CACHE is enabled by default, no? > > > its sold old obscure reasonin

Re: [PATCH] drm/nouveau: select FW caching

2025-02-18 Thread Danilo Krummrich
On Tue, Feb 18, 2025 at 06:28:53AM -0800, Luis Chamberlain wrote: > > What I recommend is to look into why we disable it by default, I think I think FW_CACHE is enabled by default, no? > its sold old obscure reasoning but now suspect it was udev being dumb, > in line with why we also try to defe

Re: [PATCH] drm/nouveau: select FW caching

2025-02-18 Thread Luis Chamberlain
On Fri, Feb 14, 2025 at 02:10:27PM +0100, Danilo Krummrich wrote: > On Fri, Feb 14, 2025 at 02:05:36PM +0100, Danilo Krummrich wrote: > > only force FW_CACHE if DRM_NOUVEAU_GSP_DEFAULT? > > Please scratch that, it was a horrible idea. What I recommend is to look into why we disable it by default,

Re: [PATCH] drm/nouveau: select FW caching

2025-02-14 Thread Danilo Krummrich
On Fri, Feb 14, 2025 at 02:05:36PM +0100, Danilo Krummrich wrote: > only force FW_CACHE if DRM_NOUVEAU_GSP_DEFAULT? Please scratch that, it was a horrible idea.

Re: [PATCH] drm/nouveau: select FW caching

2025-02-14 Thread Danilo Krummrich
(+ Luis, Russ) On Fri, Feb 07, 2025 at 11:25:31AM +1000, Dave Airlie wrote: > From: Dave Airlie > > nouveau tries to load some firmware during suspend that it loaded earlier, > but with > fw caching disabled it hangs suspend, so just rely on FW cache enabling > instead of > working around it i