[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b4516819-8cc7-46eb-b4ea-9ffd1a0c51e6@huawei.com>
Date: Thu, 7 Mar 2024 21:56:37 +0800
From: Pu Lehui <pulehui@...wei.com>
To: Daniel Borkmann <daniel@...earbox.net>
CC: <bpf@...r.kernel.org>, <netdev@...r.kernel.org>,
<magnus.karlsson@...el.com>, <prbatra.mail@...il.com>, <toke@...hat.com>,
<kuba@...nel.org>
Subject: Re: [PATCH bpf 1/2] xdp, bonding: Fix feature flags when there are no
slave devs anymore
On 2024/3/6 8:30, patchwork-bot+netdevbpf@...nel.org wrote:
> Hello:
>
> This series was applied to bpf/bpf.git (master)
> by Alexei Starovoitov <ast@...nel.org>:
>
> On Tue, 5 Mar 2024 10:08:28 +0100 you wrote:
>> Commit 9b0ed890ac2a ("bonding: do not report NETDEV_XDP_ACT_XSK_ZEROCOPY")
>> changed the driver from reporting everything as supported before a device
>> was bonded into having the driver report that no XDP feature is supported
>> until a real device is bonded as it seems to be more truthful given
>> eventually real underlying devices decide what XDP features are supported.
>>
>> The change however did not take into account when all slave devices get
>> removed from the bond device. In this case after 9b0ed890ac2a, the driver
>> keeps reporting a feature mask of 0x77, that is, NETDEV_XDP_ACT_MASK &
>> ~NETDEV_XDP_ACT_XSK_ZEROCOPY whereas it should have reported a feature
>> mask of 0.
>>
>> [...]
>
> Here is the summary with links:
> - [bpf,1/2] xdp, bonding: Fix feature flags when there are no slave devs anymore
> https://git.kernel.org/bpf/bpf/c/f267f2628150
> - [bpf,2/2] selftests/bpf: Fix up xdp bonding test wrt feature flags
> https://git.kernel.org/bpf/bpf/c/0bfc0336e134
I had encountered the same issue during riscv bpf selftest regression.
Happy to see this fixes.
>
> You are awesome, thank you!
Powered by blists - more mailing lists