On Tue, Jun 24, 2025 at 3:13 PM Timur Tabi <tt...@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.

Reply via email to