[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b9686644-e2dd-4abf-9dd7-fc12081f400f@nvidia.com>
Date: Wed, 29 Oct 2025 17:35:13 -0700
From: John Hubbard <jhubbard@...dia.com>
To: Timur Tabi <ttabi@...dia.com>, "dakr@...nel.org" <dakr@...nel.org>
Cc: Alexandre Courbot <acourbot@...dia.com>,
 "lossin@...nel.org" <lossin@...nel.org>,
 "a.hindborg@...nel.org" <a.hindborg@...nel.org>,
 "boqun.feng@...il.com" <boqun.feng@...il.com>,
 "aliceryhl@...gle.com" <aliceryhl@...gle.com>, Zhi Wang <zhiw@...dia.com>,
 "simona@...ll.ch" <simona@...ll.ch>,
 "alex.gaynor@...il.com" <alex.gaynor@...il.com>,
 "ojeda@...nel.org" <ojeda@...nel.org>, "tmgross@...ch.edu"
 <tmgross@...ch.edu>,
 "nouveau@...ts.freedesktop.org" <nouveau@...ts.freedesktop.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "rust-for-linux@...r.kernel.org" <rust-for-linux@...r.kernel.org>,
 "bjorn3_gh@...tonmail.com" <bjorn3_gh@...tonmail.com>,
 Edwin Peer <epeer@...dia.com>, "airlied@...il.com" <airlied@...il.com>,
 Joel Fernandes <joelagnelf@...dia.com>,
 "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
 "gary@...yguo.net" <gary@...yguo.net>, Alistair Popple <apopple@...dia.com>
Subject: Re: [PATCH v3 2/2] gpu: nova-core: add boot42 support for next-gen
 GPUs
On 10/29/25 5:31 PM, Timur Tabi wrote:
> On Wed, 2025-10-29 at 17:29 -0700, John Hubbard wrote:
>> That, combined with Timur's comment, made me realize that .is_next_gen()
>> can be made reliable enough that we don't even need is_nv04() at all.
> 
> Please, not "next gen".  Technically speaking, BOOT42 is "current gen" and BOOT0 is "last gen".
> 
> And this all assumes that BOOT42 won't be replaced by something even bigger for
> Blackwell+++++++++++.
OK, I think this wants to be called use_boot42_instead(), approximately.
...and I've fortified the code accordingly:
impl NV_PMC_BOOT_0 {
    pub(crate) fn use_boot42_instead(self) -> bool {
        // "Future" GPUs (some time after Rubin) will set `architecture_0`
        // to 0, and `architecture_1` to 1, and put the architecture details in
        // boot42 instead.
        self.architecture_0() == 0 && self.architecture_1() == 1
    }
thanks,
-- 
John Hubbard
Powered by blists - more mailing lists
 
