Re: [PATCH v5 09/11] gpu: nova-core: add FalconUCodeDescV2 support

[John Hubbard <jhubbard@nvidia.com>]

On 1/9/26 10:59 AM, John Hubbard wrote:
> On 1/9/26 10:11 AM, Timur Tabi wrote:
>> On Thu, 2026-01-08 at 18:53 -0800, John Hubbard wrote:
>>> It may have its own imperfections, but this is *approximately* what I'd
>>> prefer (below). I believe that it strikes the right balance between
>>> simplicity and Rust idiomatic code--and it's not any larger than what
>>> you started with.
>>>
>>> And it won't cost you any time to implement, since this already works:
>>
>> Do you realize that this is almost identical to my V1 of this patchset?  We've come full circle.
>>
> 
> I hope I didn't review that one in detail. This is really pretty funny. :)
> 

But more seriously, we are all still calibrating ourselves about how
far to go with various Rust abstractions, in various situations. Here,
"we" means all of us involved in writing and reviewing the various
Nova patchsets.

And we have different levels of experience with Rust, kernel, and
GPU drivers. *And* that's changing as we continue to work together.

So I do expect this sort of situation to start converging over time.
Meanwhile I want to apologize for you getting jerked around on
reviews. It really will improve, I'm confident of that.

thanks,
-- 
John Hubbard