Index | Thread | Search

From:
"Theo de Raadt" <deraadt@openbsd.org>
Subject:
Re: disable ix(4) LRO by default on sparc64
To:
Vitaliy Makkoveev <otto@bsdbox.dev>
Cc:
jan@openbsd.org, tech@openbsd.org
Date:
Thu, 07 Mar 2024 10:27:38 -0700

Download raw body.

Thread
I think the idea is that someone working in this area can try to use it
without compiling a custom kernel.

> "ifp_>if_capabilities |= IFCAP_LRO" should be wrapped too.
> 
> > On 7 Mar 2024, at 19:25, jan@openbsd.org wrote:
> > 
> > Hi,
> > 
> > As discussed on hackers, we'll disable LRO in ix(4) on sparc64 by
> > default, to avoid further unknown bugs here.  Thus, the upcoming release
> > has a more stable ix(4) driver.
> > 
> > ok?
> > 
> > bye,
> > Jan
> > 
> > Index: dev/pci/if_ix.c
> > ===================================================================
> > RCS file: /cvs/src/sys/dev/pci/if_ix.c,v
> > diff -u -p -r1.209 if_ix.c
> > --- dev/pci/if_ix.c	15 Feb 2024 10:56:53 -0000	1.209
> > +++ dev/pci/if_ix.c	7 Mar 2024 15:43:48 -0000
> > @@ -1932,7 +1932,9 @@ ixgbe_setup_interface(struct ix_softc *s
> > 
> > 	ifp->if_capabilities |= IFCAP_TSOv4 | IFCAP_TSOv6;
> > 	if (sc->hw.mac.type != ixgbe_mac_82598EB) {
> > +#ifndef __sparc64__
> > 		ifp->if_xflags |= IFXF_LRO;
> > +#endif
> > 		ifp->if_capabilities |= IFCAP_LRO;
> > 	}
> > 
> > 
>