fprintf: Support skipping modifier
When doing BTF comparisons between functions defined in multiple CUs, it
was noticed a few critical functions failed prototype comparisons due to
multiple "const" modifiers; for example:
function mismatch for 'memchr_inv'('memchr_inv'): 'void * ()(const const void * , int, size_t)' != 'void * ()(const void *, int, size_t)'
function mismatch for 'strnlen'('strnlen'): '__kernel_size_t ()(const const char * , __kernel_size_t)' != '__kernel_size_t ()(const char *, size_t)'
(note the "const const" in the first parameter.)
As such it would be useful to omit modifiers for comparison
purposes. Also noted was the fact that for the "no_parm_names"
case, an extra space was being emitted in some cases, also
throwing off string comparisons of prototypes.
Committer notes:
We need to check if conf is NULL in tag__ptr_name() before accessing
conf->skip_emitting_modifier, to avoid a segfault noticed when testing
with 'btfdiff vmlinux'.
Ditto for __tag__name(), but there we can use pconf-> instead as that
one is guaranteed to have the 'conf' contents or point to a default
conf_fprintf struct.
Also put back the removed whitespace after "const", making it "const "
in __tag__name(), to fix a problem reported by Eduard Zingerman.
Link: https://lore.kernel.org/bpf/87964239858beb2fe8e2d625953a3606161c85b3.camel@gmail.com
Signed-off-by: Alan Maguire <alan.maguire@oracle.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Tested-by: Jiri Olsa <jolsa@kernel.org>
Cc: Alan Maguire <alan.maguire@oracle.com>
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Andrii Nakryiko <andrii@kernel.org>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: Eduard Zingerman <eddyz87@gmail.com>
Cc: Hao Luo <haoluo@google.com>
Cc: John Fastabend <john.fastabend@gmail.com>
Cc: KP Singh <kpsingh@chromium.org>
Cc: Kui-Feng Lee <sinquersw@gmail.com>
Cc: Martin KaFai Lau <martin.lau@kernel.org>
Cc: Song Liu <songliubraving@fb.com>
Cc: Stanislav Fomichev <sdf@google.com>
Cc: Timo Beckers <timo@incline.eu>
Cc: Yonghong Song <yhs@fb.com>
Cc: bpf@vger.kernel.org
Link: https://lore.kernel.org/bpf/1678459850-16140-3-git-send-email-alan.maguire@oracle.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
2 files changed
