[PATCH 1/3] net/nfp: add the elf module
Ferruh Yigit
ferruh.yigit at amd.com
Fri Mar 1 18:22:46 CET 2024
On 2/28/2024 10:18 PM, Stephen Hemminger wrote:
> On Tue, 27 Feb 2024 19:15:49 +0800
> Chaoyong He <chaoyong.he at corigine.com> wrote:
>
>> From: Peng Zhang <peng.zhang at corigine.com>
>>
>> Add the elf module, which can get mip information from the
>> firmware ELF file.
>>
>> Signed-off-by: Peng Zhang <peng.zhang at corigine.com>
>> Reviewed-by: Chaoyong He <chaoyong.he at corigine.com>
>> Reviewed-by: Long Wu <long.wu at corigine.com>
>> ---
>
> Why are you rolling your own ELF parser?
> There are libraries to do this such as libelf.
> Libelf is already used in the BPF part of DPDK.
>
There cons and pros to depend external library, as this is in the
limited scope of the driver I am less concerned about local code.
Chaoyong, what is your take on the issue, did you consider using libelf
library option?
More information about the dev
mailing list