Re: [PATCH v3 0/9] gpu: nova-core: gsp: add RM control command infrastructure
From: Eliot Courtney
Date: Sun Apr 05 2026 - 22:29:19 EST
On Mon Apr 6, 2026 at 4:48 AM JST, John Hubbard wrote:
> On 3/25/26 5:13 AM, Eliot Courtney wrote:
>> Add the infrastructure for sending RM control RPCs. This is needed e.g.
>> for channel allocation.
>>
>> This series adds:
>> - `NV_STATUS` bindings and wrapping `NvStatus` enum (used by RM control
>> RPCs)
>> - The necessary bindings for the RM control RPCs.
>> - `RmControlMsgFunction` to identify individual control commands, like
>> `MsgFunction` for GSP commands.
>> - `SBufferIter::read_to_vec` (using KVVec) for reading large RPC payloads
>> - A typed `RmControl` command that can send RM control commands.
>> - One usage of `RmControl`: the `FaultMethodBufferSize`
>> RPC. This is useful for channel allocation later.
>>
>> Each new RM control command can be added by extending
>> `RmControlMsgFunction`, adding the bindings and wrappers for their
>> parameters, and writing a type-safe wrapper to send and receive the
>> reply for the RM control rpc, using `RmControl`.
>>
>> This series applies on latest drm-rust-next with the listed
>> pre-requisites.
>
> Hi Eliot,
>
> Nice clean series, I appreciate the work that went into making
> it easily reviewable!
>
>
> thanks,
Thanks a lot John, and thanks for the reviews.