[PATCH 0/1] objdump: Add callx support for BPF CPU v1

Nick Clifton nickc@redhat.com
Mon Feb 12 17:01:34 GMT 2024


Hi Will,

> First time patch submitter but long time GNU enthusiast. Thank you for
> all the work that you all do to maintain these vital pieces of software
> for the community.

Welcome on board and thank you contributing.


> I tried very hard to follow best practices for creating this patch
> submission but I am sure that I did something wrong. Please let me know
> what I can do differently or better.

No, no, what you did was excellent.  My only suggestions is that
with your v2 patch you consider adding a new assembler test (or
extending an already existing one) that checks that the instruction
can be assembled and then disassembled.  For example you might
like to add it to gas/testsuite/gas/bpf/call.s (and the associated
call.d file for the disassembly).

Cheers
   Nick



More information about the Binutils mailing list