Nouveau - Aug 2025 - [PATCH v2 0/8] gpu: nova-core: process and prepare more firmwares to boot GSP

On 8/25/25 9:07 PM, Alexandre Courbot wrote:
...
> The base of this series is today's nova-next, but there a few more
> unmerged dependencies required:
> 
> - BorrowedPage, AsPageIter and VmallocPageIter [2]
> - Rust infrastructure for sg_table and scatterlist [3]
> - FromBytes [4]
> - The Alignment series [5]
Hi Alex!

For future patchsets that have complex and/or lengthy prerequisite
patchsets, let's please please make it easier for people to
locally apply the series. Very important. 

The very nicest experience is by simply providing a link to your git
branch that exactly matches the patchset you just posted.

In this particular case, it would also have worked to just say, "this 
series can be applied by first applying FromBytes [4], to the base commit."

Because that works, and is simpler than hunting down the other three
items, which are not actually necessary for git am (although they
might be for actual testing).

But the point is that the admin can be made simpler for the reviewers--even
those of us who know exactly what you're up to. And we should keep
that in mind, especially because there are many more of these situations
coming soon.


thanks,
-- 
John Hubbard

On Wed Aug 27, 2025 at 9:29 AM JST, John Hubbard wrote:
> On 8/25/25 9:07 PM, Alexandre Courbot wrote:
> ...
>> The base of this series is today's nova-next, but there a few more
>> unmerged dependencies required:
>> 
>> - BorrowedPage, AsPageIter and VmallocPageIter [2]
>> - Rust infrastructure for sg_table and scatterlist [3]
>> - FromBytes [4]
>> - The Alignment series [5]
>
> Hi Alex!
>
> For future patchsets that have complex and/or lengthy prerequisite
> patchsets, let's please please make it easier for people to
> locally apply the series. Very important. 
>
> The very nicest experience is by simply providing a link to your git
> branch that exactly matches the patchset you just posted.
>
> In this particular case, it would also have worked to just say, "this 
> series can be applied by first applying FromBytes [4], to the base
commit."
>
> Because that works, and is simpler than hunting down the other three
> items, which are not actually necessary for git am (although they
> might be for actual testing).
>
> But the point is that the admin can be made simpler for the reviewers--even
> those of us who know exactly what you're up to. And we should keep
> that in mind, especially because there are many more of these situations
> coming soon.
Right, b4 is supposed to be able to help with this as well, but indeed a
link to a git tree should be the standard with patchsets requiring such
dependencies. :) Will try to keep this in mind.