Thank you for the feedback on my first attempt. I believe that this attempt is better. Given the feedback, I 0. Removed the incorrect use of linkage_name field; 1. Tried to maintain the semantics of DWARF as much as possible; 2. Reworked how BTF information is loaded into a new function based on my understanding of the semantics of BTF_FUNC_STATIC, BTF_FUNC_GLOBAL, and BTF_FUNC_EXTERN; and 3. Modified (slightly) the way functions are printed in pfunc to show linkage where appropriate (based, in particular, on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45153#c6. I saw that Arnaldo and Namhyung have made similar changes to the printing code so I hope that I am not stepping on their work (sorry if I am!). Again, I appreciate the feedback and hope that I was able to address it with this v2. Sincerely, Will Will Hawkins (1): btf_loader: Support linkages for BTF subprograms btf_loader.c | 4 +++- dwarves.h | 2 +- pfunct.c | 15 ++++++++++++++- 3 files changed, 18 insertions(+), 3 deletions(-) -- 2.45.2
