bpf.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Luigi Rizzo <rizzo@iet.unipi.it>
To: bpf@vger.kernel.org, ppenkov@google.com,
	Luigi Rizzo <lrizzo@google.com>,
	andriin@fb.com, sdf@google.com
Subject: libbpf/bpftool inconsistent handling og .data and .bss ?
Date: Wed, 7 Oct 2020 16:01:34 +0200	[thread overview]
Message-ID: <CA+hQ2+gb_y7TViv13K_JpJTP=yHFqORmY+=6PrO4eAjgrBSitw@mail.gmail.com> (raw)

I am experiencing some weirdness in global variables handling
in bpftool and libbpf, as described below.

This happens happen with code in foo_bpf.c compiled with
   clang-10 -O2 -Wall -Werror -target bpf ...
and subsequently exported with
   bpftool gen skeleton ...
(i have tried bpftool 5.8.7 and 5.9.0-rc6)

1. uninitialized globals are not recognised
   The following code in the bpf program

     int x;
     SEC("fentry/bar")
     int BPF_PROG(bar) { return 0;}

   compiles ok but bpftool then complains

      libbpf: prog 'bar': invalid relo against 'x' in special section
0xfff2; forgot to initialize global var?..

   The error disappears if I initialize x=0 or x=1
   (in the skeleton, x=0 ends up in .bss, x=1 ends up in .data)

2. .bss overrides from userspace are not seen in bpf at runtime

    In foo_bpf.c I have "int x = 0;"
    In the userspace program, before foo_bpf__load(), I do
       obj->bss->x = 1
    but after attach, the bpf code does not see the change, ie
        "if (x == 0) { .. } else { .. }"
    always takes the first branch.

    If I initialize "int x = 2" and then do
       obj->data->x = 1
    the update is seen correctly ie
          "if (x == 2) { .. } else { .. }"
     takes one or the other depending on whether userspace overrides
     the value before foo_bpf__load()

3. .data overrides do not seem to work for non-scalar types
    In foo_bpf.c I have
          struct one { int a; }; // type also visible to userspace
          struct one x { .a = 2 }; // avoid bugs #1 and #2
    If in userspace I do
          obj->data->x.a = 1
    the update is not seen in the kernel, ie
            "if (x.a == 2) { .. } else { .. }"
     always takes the first branch

Are these known issues ?

thanks
luigi

             reply	other threads:[~2020-10-07 14:01 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-07 14:01 Luigi Rizzo [this message]
2020-10-07 15:58 ` libbpf/bpftool inconsistent handling og .data and .bss ? Yonghong Song
2020-10-07 18:35 ` Andrii Nakryiko
2020-10-07 20:31   ` Luigi Rizzo
2020-10-07 20:40     ` Andrii Nakryiko
2020-10-07 21:29       ` Luigi Rizzo
2020-10-07 22:26         ` Andrii Nakryiko
2020-10-08  1:33           ` Yonghong Song
2020-10-10 22:49         ` Luigi Rizzo
2020-10-10 23:11           ` Andrii Nakryiko
2020-10-11  0:31             ` Luigi Rizzo
2020-10-11  1:36               ` 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='CA+hQ2+gb_y7TViv13K_JpJTP=yHFqORmY+=6PrO4eAjgrBSitw@mail.gmail.com' \
    --to=rizzo@iet.unipi.it \
    --cc=andriin@fb.com \
    --cc=bpf@vger.kernel.org \
    --cc=lrizzo@google.com \
    --cc=ppenkov@google.com \
    --cc=sdf@google.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).