From: Andrii Nakryiko <andrii.nakryiko@gmail.com>
To: Song Liu <songliubraving@fb.com>
Cc: Andrii Nakryiko <andrii@kernel.org>,
"bpf@vger.kernel.org" <bpf@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
Alexei Starovoitov <ast@fb.com>,
"daniel@iogearbox.net" <daniel@iogearbox.net>,
Kernel Team <Kernel-team@fb.com>,
Andrii Nakryiko <andriin@fb.com>
Subject: Re: [PATCH bpf-next 06/11] selftests/bpf: add checking of raw type dump in BTF writer APIs selftests
Date: Mon, 2 Nov 2020 21:14:39 -0800 [thread overview]
Message-ID: <CAEf4BzbZtoyFYbQnF7WvyT2xRcbZCPO5u61wzxaz-h42z4WPNQ@mail.gmail.com> (raw)
In-Reply-To: <507AB3B7-50BF-43CA-82CA-7C24CD5DF8A4@fb.com>
On Mon, Nov 2, 2020 at 4:08 PM Song Liu <songliubraving@fb.com> wrote:
>
>
>
> > On Oct 28, 2020, at 5:58 PM, Andrii Nakryiko <andrii@kernel.org> wrote:
> >
> > From: Andrii Nakryiko <andriin@fb.com>
> >
> > Add re-usable btf_helpers.{c,h} to provide BTF-related testing routines. Start
> > with adding a raw BTF dumping helpers.
> >
> > Raw BTF dump is the most succinct and at the same time a very human-friendly
> > way to validate exact contents of BTF types. Cross-validate raw BTF dump and
> > writable BTF in a single selftest. Raw type dump checks also serve as a good
> > self-documentation.
> >
> > Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
>
> Acked-by: Song Liu <songliubraving@fb.com>
>
> with a couple nits:
>
> [...]
>
> > +
> > +/* Print raw BTF type dump into a local buffer and return string pointer back.
> > + * Buffer *will* be overwritten by subsequent btf_type_raw_dump() calls
> > + */
> > +const char *btf_type_raw_dump(const struct btf *btf, int type_id)
> > +{
> > + static char buf[16 * 1024];
> > + FILE *buf_file;
> > +
> > + buf_file = fmemopen(buf, sizeof(buf) - 1, "w");
> > + if (!buf_file) {
> > + fprintf(stderr, "Failed to open memstream: %d\n", errno);
> > + return NULL;
> > + }
> > +
> > + fprintf_btf_type_raw(buf_file, btf, type_id);
> > + fflush(buf_file);
> > + fclose(buf_file);
> > +
> > + return buf;
> > +}
> > diff --git a/tools/testing/selftests/bpf/btf_helpers.h b/tools/testing/selftests/bpf/btf_helpers.h
> > new file mode 100644
> > index 000000000000..2c9ce1b61dc9
> > --- /dev/null
> > +++ b/tools/testing/selftests/bpf/btf_helpers.h
> > @@ -0,0 +1,12 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +/* Copyright (c) 2020 Facebook */
> > +#ifndef __BTF_HELPERS_H
> > +#define __BTF_HELPERS_H
> > +
> > +#include <stdio.h>
> > +#include <bpf/btf.h>
> > +
> > +int fprintf_btf_type_raw(FILE *out, const struct btf *btf, __u32 id);
> > +const char *btf_type_raw_dump(const struct btf *btf, int type_id);
> > +
> > +#endif
> > diff --git a/tools/testing/selftests/bpf/prog_tests/btf_write.c b/tools/testing/selftests/bpf/prog_tests/btf_write.c
> > index 314e1e7c36df..bc1412de1b3d 100644
> > --- a/tools/testing/selftests/bpf/prog_tests/btf_write.c
> > +++ b/tools/testing/selftests/bpf/prog_tests/btf_write.c
> > @@ -2,6 +2,7 @@
> > /* Copyright (c) 2020 Facebook */
> > #include <test_progs.h>
> > #include <bpf/btf.h>
> > +#include "btf_helpers.h"
> >
> > static int duration = 0;
> >
> > @@ -11,12 +12,12 @@ void test_btf_write() {
> > const struct btf_member *m;
> > const struct btf_enum *v;
> > const struct btf_param *p;
> > - struct btf *btf;
> > + struct btf *btf = NULL;
>
> No need to initialize btf.
>
> > int id, err, str_off;
> >
> > btf = btf__new_empty();
> > if (CHECK(IS_ERR(btf), "new_empty", "failed: %ld\n", PTR_ERR(btf)))
> > - return;
> > + goto err_out;
>
> err_out is not needed either.
eagle eye ;) fixed both
>
> [...]
next prev parent reply other threads:[~2020-11-03 5:14 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-29 0:58 [PATCH bpf-next 00/11] libbpf: split BTF support Andrii Nakryiko
2020-10-29 0:58 ` [PATCH bpf-next 01/11] libbpf: factor out common operations in BTF writing APIs Andrii Nakryiko
2020-10-30 0:36 ` Song Liu
2020-10-29 0:58 ` [PATCH bpf-next 02/11] selftest/bpf: relax btf_dedup test checks Andrii Nakryiko
2020-10-30 16:43 ` Song Liu
2020-10-30 18:44 ` Andrii Nakryiko
2020-10-30 22:30 ` Song Liu
2020-10-29 0:58 ` [PATCH bpf-next 03/11] libbpf: unify and speed up BTF string deduplication Andrii Nakryiko
2020-10-30 23:32 ` Song Liu
2020-11-03 4:51 ` Andrii Nakryiko
2020-11-03 4:59 ` Alexei Starovoitov
2020-11-03 6:01 ` Andrii Nakryiko
2020-10-29 0:58 ` [PATCH bpf-next 04/11] libbpf: implement basic split BTF support Andrii Nakryiko
2020-11-02 23:23 ` Song Liu
2020-11-03 5:02 ` Andrii Nakryiko
2020-11-03 5:41 ` Song Liu
2020-11-04 23:51 ` Andrii Nakryiko
2020-10-29 0:58 ` [PATCH bpf-next 05/11] selftests/bpf: add split BTF basic test Andrii Nakryiko
2020-11-02 23:36 ` Song Liu
2020-11-03 5:10 ` Andrii Nakryiko
2020-10-29 0:58 ` [PATCH bpf-next 06/11] selftests/bpf: add checking of raw type dump in BTF writer APIs selftests Andrii Nakryiko
2020-11-03 0:08 ` Song Liu
2020-11-03 5:14 ` Andrii Nakryiko [this message]
2020-10-29 0:58 ` [PATCH bpf-next 07/11] libbpf: fix BTF data layout checks and allow empty BTF Andrii Nakryiko
2020-11-03 0:51 ` Song Liu
2020-11-03 5:18 ` Andrii Nakryiko
2020-11-03 5:44 ` Song Liu
2020-10-29 0:58 ` [PATCH bpf-next 08/11] libbpf: support BTF dedup of split BTFs Andrii Nakryiko
2020-11-03 2:49 ` Song Liu
2020-11-03 5:25 ` Andrii Nakryiko
2020-11-03 5:59 ` Song Liu
2020-11-03 6:31 ` Andrii Nakryiko
2020-11-03 17:15 ` Song Liu
2020-11-03 5:10 ` Alexei Starovoitov
2020-11-03 6:27 ` Andrii Nakryiko
2020-11-03 17:55 ` Alexei Starovoitov
2020-10-29 0:59 ` [PATCH bpf-next 09/11] libbpf: accomodate DWARF/compiler bug with duplicated identical arrays Andrii Nakryiko
2020-11-03 2:52 ` Song Liu
2020-10-29 0:59 ` [PATCH bpf-next 10/11] selftests/bpf: add split BTF dedup selftests Andrii Nakryiko
2020-11-03 5:35 ` Song Liu
2020-11-03 6:05 ` Andrii Nakryiko
2020-11-03 6:30 ` Song Liu
2020-10-29 0:59 ` [PATCH bpf-next 11/11] tools/bpftool: add bpftool support for split BTF Andrii Nakryiko
2020-11-03 6:03 ` Song Liu
2020-10-30 0:33 ` [PATCH bpf-next 00/11] libbpf: split BTF support Song Liu
2020-10-30 2:33 ` Andrii Nakryiko
2020-10-30 6:45 ` Song Liu
2020-10-30 12:04 ` Alan Maguire
2020-10-30 18:30 ` Andrii Nakryiko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAEf4BzbZtoyFYbQnF7WvyT2xRcbZCPO5u61wzxaz-h42z4WPNQ@mail.gmail.com \
--to=andrii.nakryiko@gmail.com \
--cc=Kernel-team@fb.com \
--cc=andrii@kernel.org \
--cc=andriin@fb.com \
--cc=ast@fb.com \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=netdev@vger.kernel.org \
--cc=songliubraving@fb.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).