From: Tony Ambardar
The runqslower binary from a cross-endian build currently fails to run
because the included skeleton has host endianness. Fix this by passing the
target BPF endianness to the runqslower sub-make.
Fixes: 5a63c33d6f00 ("selftests/bpf: Support cross-endian building")
Signed-off-
On Fri, Jan 24, 2025 at 10:35:40AM +, Cosmin Ratiu wrote:
> On Fri, 2025-01-24 at 09:54 +, Hangbin Liu wrote:
> > On Fri, Jan 24, 2025 at 10:57:44AM +0200, Cosmin Ratiu wrote:
> > > ---
> > > drivers/net/bonding/bond_main.c | 19 ++-
> > > 1 file changed, 10 insertions(+),
On Fri, 2025-01-24 at 09:54 +, Hangbin Liu wrote:
> On Fri, Jan 24, 2025 at 10:57:44AM +0200, Cosmin Ratiu wrote:
> > ---
> > drivers/net/bonding/bond_main.c | 19 ++-
> > 1 file changed, 10 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/net/bonding/bond_main.c
> >
On Fri, Jan 24, 2025 at 10:57:44AM +0200, Cosmin Ratiu wrote:
> ---
> drivers/net/bonding/bond_main.c | 19 ++-
> 1 file changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
> index 7b78c2bada81..e45bba240cbc
The referenced fix is incomplete. It correctly computes
bond_dev->gso_partial_features across slaves, but unfortunately
netdev_fix_features discards gso_partial_features from the feature set
if NETIF_F_GSO_PARTIAL isn't set in bond_dev->features.
This is visible with ethtool -k bond0 | grep esp:
t
V2:
Removed unnecessary Change-Id
Changed "bond->features" to "bond_dev->features" in commit message.
On Fri, 2025-01-24 at 03:29 +, Hangbin Liu wrote:
>
> What's Change-Id here? Others looks good to me.
Apologies, I forgot to remove it. It's from gerrit, the internal code
review tool we use.
I'll resend v2
>
> Reviewed-by: Hangbin Liu
Thanks for the review!
