On Mon, Mar 3, 2025 at 7:09 AM Liang, Prike <[email protected]> wrote: > > [Public] > > Question: Why use the CONFIG_DRM_AMDGPU_NAVI3X_USERQ to guard all the various > GFX user mode queue enablement paths? It seems more generic and reasonable to > use the config name CONFIG_DRM_AMDGPU_NAVI3X_USERQ. >
I assume you mean something more generic like CONFIG_DRM_AMDGPU_USERQ? At this point it would make more sense, but at the time it was added there was only gfx11 support. We could change it, but at this point, I'd be more inclined to just remove it. > Except for that question, the series patch is Reviewed-by: Prike Liang > <[email protected]> . Thanks. Alex > > Regards, > Prike > > > -----Original Message----- > > From: amd-gfx <[email protected]> On Behalf Of Alex > > Deucher > > Sent: Saturday, March 1, 2025 3:54 AM > > To: [email protected] > > Cc: Deucher, Alexander <[email protected]> > > Subject: [PATCH 5/5] drm/amdgpu/userq: remove BROKEN from config > > > > This can be enabled now. We have the firmware checks in place. > > > > Signed-off-by: Alex Deucher <[email protected]> > > --- > > drivers/gpu/drm/amd/amdgpu/Kconfig | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig > > b/drivers/gpu/drm/amd/amdgpu/Kconfig > > index a614320114796..23c014da0f324 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/Kconfig > > +++ b/drivers/gpu/drm/amd/amdgpu/Kconfig > > @@ -98,7 +98,6 @@ config DRM_AMDGPU_WERROR config > > DRM_AMDGPU_NAVI3X_USERQ > > bool "Enable amdgpu usermode queues" > > depends on DRM_AMDGPU > > - depends on BROKEN > > default n > > help > > Choose this option to enable GFX usermode queue support for > > GFX/SDMA/Compute > > -- > > 2.48.1 >
