From: Jakub Kicinski <kubakici@wp.pl>
To: David Miller <davem@davemloft.net>
Cc: ast@fb.com, dsahern@gmail.com, kafai@fb.com,
hannes@stressinduktion.org, netdev@vger.kernel.org,
daniel@iogearbox.net, kernel-team@fb.com
Subject: Re: [PATCH v2 net-next 0/8] Introduce bpf ID
Date: Thu, 1 Jun 2017 11:57:15 -0700 [thread overview]
Message-ID: <20170601115715.1107113c@cakuba.lan> (raw)
In-Reply-To: <20170601.143336.2263901633118099409.davem@davemloft.net>
On Thu, 01 Jun 2017 14:33:36 -0400 (EDT), David Miller wrote:
> From: Alexei Starovoitov <ast@fb.com>
> Date: Thu, 1 Jun 2017 11:27:56 -0700
>
> > Original instructions are not useful for debugging, since they have
> > map_fds which can be long gone when program is running, therefore to
> > solve introspection problem we need to look into real kernel
> > instructions.
>
> The only possible legitimate use I see for the original instructions
> is checkpoint/restart.
>
> If 6wind wants this for "external acceleration" or whatever, like
> Alexei I am not interested in facilitating that at all.
+1
I will try disabling the inlining then.
next prev parent reply other threads:[~2017-06-01 18:57 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-31 18:58 [PATCH v2 net-next 0/8] Introduce bpf ID Martin KaFai Lau
2017-05-31 18:58 ` [PATCH v2 net-next 1/8] bpf: Introduce bpf_prog ID Martin KaFai Lau
2017-05-31 18:58 ` [PATCH v2 net-next 2/8] bpf: Introduce bpf_map ID Martin KaFai Lau
2017-05-31 18:58 ` [PATCH v2 net-next 3/8] bpf: Add BPF_(PROG|MAP)_GET_NEXT_ID command Martin KaFai Lau
2017-05-31 18:58 ` [PATCH v2 net-next 4/8] bpf: Add BPF_PROG_GET_FD_BY_ID Martin KaFai Lau
2017-05-31 18:58 ` [PATCH v2 net-next 5/8] bpf: Add BPF_MAP_GET_FD_BY_ID Martin KaFai Lau
2017-05-31 18:59 ` [PATCH v2 net-next 6/8] bpf: Add jited_len to struct bpf_prog Martin KaFai Lau
2017-05-31 18:59 ` [PATCH v2 net-next 7/8] bpf: Add BPF_OBJ_GET_INFO_BY_FD Martin KaFai Lau
2017-05-31 18:59 ` [PATCH v2 net-next 8/8] bpf: Test for bpf ID Martin KaFai Lau
2017-05-31 23:35 ` [PATCH v2 net-next 0/8] Introduce " David Miller
2017-06-03 1:08 ` Martin KaFai Lau
2017-06-01 2:22 ` Jakub Kicinski
2017-06-01 13:55 ` David Ahern
2017-06-01 17:24 ` Alexei Starovoitov
2017-06-01 17:51 ` David Ahern
2017-06-01 18:27 ` Alexei Starovoitov
2017-06-01 18:33 ` David Miller
2017-06-01 18:57 ` Jakub Kicinski [this message]
2017-06-01 18:52 ` David Ahern
2017-06-01 18:56 ` David Miller
2017-06-01 19:00 ` Jakub Kicinski
2017-06-01 21:48 ` Alexei Starovoitov
2017-06-02 11:39 ` David Ahern
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=20170601115715.1107113c@cakuba.lan \
--to=kubakici@wp.pl \
--cc=ast@fb.com \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=dsahern@gmail.com \
--cc=hannes@stressinduktion.org \
--cc=kafai@fb.com \
--cc=kernel-team@fb.com \
--cc=netdev@vger.kernel.org \
/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).