Thread (16 messages) 16 messages, 4 authors, 2026-02-06

Re: [PATCH net] bonding: Fix header_ops type confusion

From: Eric Dumazet <edumazet@google.com>
Date: 2026-02-02 17:11:39
Also in: lkml

On Wed, Jan 28, 2026 at 11:46 AM 戸田晃太 [off-list ref] wrote:
Here is the patch with the barriers added, based on v6.12.67.

However, as Yuki said, we are wondering if this would be considered an
acceptable change
from the perspective of the maintainers (or in terms of Linux kernel
culture). This is because
the patch adds `READ_ONCE` to several locations outside of bonding subsystem.
Please let me know if you have any concerns regarding this point.
quoted
 Also, please clarify what happens with stacks of two or more bonding devices ?
To clarify, currently the `header_ops` of the bottom-most
interface are used regardless of the number of bonding layers.
This patch changes it so that `&bond->bond_header_ops` is used
as the bond device's `header_ops`, regardless of the stack depth.
Could you try to cook a patch series perhaps ?

The READ_ONCE()/WRITE_ONCE() on dev->header_ops->cache could be done separately.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help