[Toybox] [PATCH] riscv: decode ELF header e_flags.

enh enh at google.com
Thu Nov 2 16:20:36 PDT 2023


On Thu, Nov 2, 2023 at 12:06 PM Rob Landley <rob at landley.net> wrote:
>
> On 11/1/23 18:45, enh wrote:
> > do you want a patch with the arm32 flags too?
>
> Yes please.

(sent.)

> I applied your patch and then moved stuff to elf.c and tidied it a
> little. Could you confirm the riscv flags still print right with the new trick?

yeah, lgtm. (though the arm32 patch fixes the arithmetic in file.c so
it works for 32-bit ELF files as well as 64-bit files, so now it'll
work for riscv32 too.)

> I haven't got a test binary here. (I should break down and add a riscv cross
> compiler to the heap and add stripped musl hello world binaries for each
> architecture to tests/files...)

yeah, i almost sent you one but i then started to wonder about which
specific variants we should have and so on.

i'll send you something when i send you the readelf attributes
support. (but let me know if you just want a riscv64 ELF file _now_
and corresponding file(1) test to improve coverage.)

> Rob


More information about the Toybox mailing list