On 04/22/15 at 12:36P, Hiren Panchasara wrote: > On 04/21/15 at 03:05P, Hiren Panchasara wrote: > > On 04/21/15 at 08:24P, Hiren Panchasara wrote: > > > Author: hiren > > > Date: Tue Apr 21 20:24:15 2015 > > > New Revision: 281838 > > > URL: https://svnweb.freebsd.org/changeset/base/281838 > > > > > > Log: > > > For igb(4), when we are doing multiqueue, we are all setup to have full > > > 32bit > > > RSS hash from the card. We do not need to hide that under "ifdef RSS" > > > and should > > > expose that by default so others like lagg(4) can use that and avoid > > > hashing the > > > traffic by themselves. > > > While here, improve comments and get rid of hidden/unimplemented RSS > > > support > > > code for UDP. > > > > > > Differential Revision: https://reviews.freebsd.org/D2296 > > > Reviewed by: jfv, erj > > > Discussed with: adrian > > > Sponsored by: Limelight Networks > > > > > > Modified: > > > head/sys/dev/e1000/if_igb.c > > > > I intend to MFC this in a week or so. But for that, I'd have to first > > MFC a part of https://svnweb.freebsd.org/base?view=revision&revision=268028 > > which sets rss hash types. > > Apparently another missing MFC is r275358. I'll work on them.
So, here is the patch that I intend to commit to stable/10: https://people.freebsd.org/~hiren/patches/igb_flowid_mfc.patch RSS is not (yet) ready to be brought back into stable/10 so for now I am only exposing full 32bit flowid and we will still return flowtype OPAQUE. cheers, Hiren
pgpdcdpyin3b9.pgp
Description: PGP signature
