Re: [PATCH] rust: add basic ELF sections parser

["Alexandre Courbot" <acourbot@nvidia.com>]

Hi Greg,

On Thu May 15, 2025 at 4:38 PM JST, Greg KH wrote:
> On Thu, May 15, 2025 at 03:03:51PM +0900, Alexandre Courbot wrote:
>> Add a simple ELF sections parser for unpacking loaded binaries from
>> user-space. This is not intended to become a fully-fledged ELF parser,
>> just a helper to parse firmwares packaged in that format.
>> 
>> This parser is notably helpful for NVIDIA's GSP firmware, which is
>> provided as an ELF binary using sections to separate the firmware code
>> to its other components like chipset-specific signatures.
>> 
>> Since the data source is likely to be user-space, checked arithmetic
>> operations and strict bound checking are used.
>> 
>> Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>> ---
>> This will soon be needed in order to load the GSP firmware in nova-core,
>> so sending this early for separate review.
>> ---
>>  rust/kernel/elf.rs | 322 +++++++++++++++++++++++++++++++++++++++++++++++++++++
>
> Why is this not just done in userspace and then have userspace feed the
> proper elf sections to the kernel through the firmware interface?
> Having to parse elf seems crazy for the kernel to be forced to do here
> as the kernel should NOT be touching anything in a firmware blob other
> than passing it off to the firmware directly.

FWIW, the GSP firmware in question is already in linux-firmware and
loaded by e.g. Nouveau.

I am not sure how userspace could feed the proper ELF sections otherwise
than by splitting the ELF binary into as many files as there are
sections. Is that what you imply, or is there another means that would
preserve the current firmware format?

Note also that in this particular case, the kernel cannot just pass the
firmware without modifying it anyway since the signatures relevant to
the chipset need to be patched into the right place before it is loaded.