[PATCH 2/2] drm/nouveau: Remove nvkm_gsp_fwif.enable

M Henning mhenning at darkrefraction.com
Thu Jun 26 17:15:17 UTC 2025


On Tue, Jun 24, 2025 at 3:13 PM Timur Tabi <ttabi at nvidia.com> wrote:
> You have a good point, but I think your change, in effect, necessitates my request.  Previously, the
> default was no GSP-RM unless needed.  Now it's yes GSP-RM, and the concept of "need" has been
> removed.  So there's no indication any more that some GPUs need GSP-RM and some do not.
>
> So to address that, I think it makes sense to add a warning if someone tries disable GSP-RM on a GPU
> that is not supported in that configuration.
>
> Now, whether or not we should ignore NvGspRm=0 on Ada+ is up for debate.  If I understand the code
> correctly, today (and still with your patches), Ada+ would fail to boot.  I can't say whether or not
> that's a good idea.  But I think a warning should be printed either way.

This patch behaves exactly the same as DRM_NOUVEAU_GSP_DEFAULT=y
kernels already behave.

That being said, I'm not against the additional error checking here
and can add it to the next version of this series.


More information about the Nouveau mailing list