From: Jakub Sitnicki <jakub@cloudflare.com>
To: Alexei Starovoitov <alexei.starovoitov@gmail.com>
Cc: kernel test robot <lkp@intel.com>, bpf <bpf@vger.kernel.org>,
kbuild-all@lists.01.org,
Network Development <netdev@vger.kernel.org>,
Alexei Starovoitov <ast@kernel.org>,
Daniel Borkmann <daniel@iogearbox.net>,
Andrii Nakryiko <andrii@kernel.org>,
kernel-team <kernel-team@cloudflare.com>
Subject: Re: [PATCH bpf-next 1/2] bpf: Make remote_port field in struct bpf_sk_lookup 16-bit wide
Date: Tue, 08 Feb 2022 21:42:04 +0100 [thread overview]
Message-ID: <87y22lqeeb.fsf@cloudflare.com> (raw)
In-Reply-To: <CAADnVQKGF=YaKvzWZFO1c9bO63XHoiD=i-w-chCeSbaNoRfdwg@mail.gmail.com>
On Tue, Feb 08, 2022 at 08:28 PM CET, Alexei Starovoitov wrote:
> On Mon, Feb 7, 2022 at 2:05 PM kernel test robot <lkp@intel.com> wrote:
>> 7c32e8f8bc33a5f Lorenz Bauer 2021-03-03 1148
>> 7c32e8f8bc33a5f Lorenz Bauer 2021-03-03 @1149 if (user_ctx->local_port > U16_MAX || user_ctx->remote_port > U16_MAX) {
>
> Jakub,
> are you planning to respin and remove that check?
Yes, certainly. Just didn't get to it today.
Can either respin the series or send a follow-up. Whatever works.
WARNING: multiple messages have this Message-ID (diff)
From: Jakub Sitnicki <jakub@cloudflare.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH bpf-next 1/2] bpf: Make remote_port field in struct bpf_sk_lookup 16-bit wide
Date: Tue, 08 Feb 2022 21:42:04 +0100 [thread overview]
Message-ID: <87y22lqeeb.fsf@cloudflare.com> (raw)
In-Reply-To: <CAADnVQKGF=YaKvzWZFO1c9bO63XHoiD=i-w-chCeSbaNoRfdwg@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 496 bytes --]
On Tue, Feb 08, 2022 at 08:28 PM CET, Alexei Starovoitov wrote:
> On Mon, Feb 7, 2022 at 2:05 PM kernel test robot <lkp@intel.com> wrote:
>> 7c32e8f8bc33a5f Lorenz Bauer 2021-03-03 1148
>> 7c32e8f8bc33a5f Lorenz Bauer 2021-03-03 @1149 if (user_ctx->local_port > U16_MAX || user_ctx->remote_port > U16_MAX) {
>
> Jakub,
> are you planning to respin and remove that check?
Yes, certainly. Just didn't get to it today.
Can either respin the series or send a follow-up. Whatever works.
next prev parent reply other threads:[~2022-02-08 22:23 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-07 13:14 [PATCH bpf-next 0/2] Split bpf_sk_lookup remote_port field Jakub Sitnicki
2022-02-07 13:14 ` [PATCH bpf-next 1/2] bpf: Make remote_port field in struct bpf_sk_lookup 16-bit wide Jakub Sitnicki
2022-02-07 19:14 ` Yonghong Song
2022-02-07 22:04 ` kernel test robot
2022-02-07 22:04 ` kernel test robot
2022-02-08 19:28 ` Alexei Starovoitov
2022-02-08 19:28 ` Alexei Starovoitov
2022-02-08 20:42 ` Jakub Sitnicki [this message]
2022-02-08 20:42 ` Jakub Sitnicki
2022-02-08 20:58 ` Alexei Starovoitov
2022-02-08 20:58 ` Alexei Starovoitov
2022-02-07 13:14 ` [PATCH bpf-next 2/2] selftests/bpf: Cover 4-byte load from remote_port in bpf_sk_lookup Jakub Sitnicki
2022-02-07 19:20 ` Yonghong Song
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=87y22lqeeb.fsf@cloudflare.com \
--to=jakub@cloudflare.com \
--cc=alexei.starovoitov@gmail.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=kbuild-all@lists.01.org \
--cc=kernel-team@cloudflare.com \
--cc=lkp@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.