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

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, May 30, 2025 at 09:58:02AM +0900, Alexandre Courbot wrote:
> > But for now, doing it in generic code, that all systems end up loading,
> > yet very very very few would ever actually use makes no sense.  And
> > adding it to a driver also doesn't make sense as you can define your
> > user/kernel api now, it's not set in stone at all given that there is no
> > existing code merged.
> 
> Eschewing this from the driver would require duplicating the GSP
> firmware (a healthy 26MB compressed binary) in linux-firmware to provide
> both ELF and non-ELF versions of the same code, and also store the other
> ELF sections as their own files. I expect this to be a hard sell for
> linux-firmware.

Why would the linux-firmware people care about the size of firmware
blobs being given to them?  That's the whole reason for their existance,
to put them in one place instead of having to download them from random
locations on the internet, or to have them in the kernel tree itself.

It's already 300MB or so for the whole project, what's 26MB more?  It's
not like the collection is ever going to get smaller :)

thanks,

greg k-h