netdev - RE: gianfar select

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 24 Dec 2009 12:29:47 +0530
From:	Krishna Kumar2 <krkumar2@...ibm.com>
To:	"Kumar Gopalpet-B05799" <B05799@...escale.com>
Cc:	avorontsov@...mvista.com, "David Miller" <davem@...emloft.net>,
	e1000-devel@...ts.sourceforge.net, galak@...nel.crashing.org,
	netdev@...r.kernel.org, netdev-owner@...r.kernel.org,
	yinghai@...nel.org
Subject: RE: gianfar select_queue bogosity



"Kumar Gopalpet-B05799" <B05799@...escale.com> wrote on 12/24/2009 12:06:09
PM:

> "Kumar Gopalpet-B05799" <B05799@...escale.com>
> 12/24/2009 12:06 PM
>
> To
>
> "David Miller" <davem@...emloft.net>
>
> cc
>
> Krishna Kumar2/India/IBM@...IN, <yinghai@...nel.org>, <e1000-
> devel@...ts.sourceforge.net>, <netdev@...r.kernel.org>, <netdev-
> owner@...r.kernel.org>, <avorontsov@...mvista.com>,
> <galak@...nel.crashing.org>
>
> Subject
>
> RE: gianfar select_queue bogosity
>
>
>
> >-----Original Message-----
> >From: David Miller [mailto:davem@...emloft.net]
> >Sent: Thursday, December 24, 2009 12:00 PM
> >To: Kumar Gopalpet-B05799
> >Cc: krkumar2@...ibm.com; yinghai@...nel.org;
> >e1000-devel@...ts.sourceforge.net; netdev@...r.kernel.org;
> >netdev-owner@...r.kernel.org; avorontsov@...mvista.com;
> >galak@...nel.crashing.org
> >Subject: Re: gianfar select_queue bogosity
> >
> >From: "Kumar Gopalpet-B05799" <B05799@...escale.com>
> >Date: Thu, 24 Dec 2009 11:34:23 +0530
> >
> >> What if I want to maintaing a 1-1 mapping b/w the Rx/Tx queues.
> >> For eg., I want a packet received on queue-1 on eth0 to be forwarded
> >> on to queue-1 on eth1.
> >
> >That's what the default code does when forwarding/bridging!
> >
> >And for locally generated packets it uses the flow hash.
> >
> >What do you think we do by default?  Go read the code :-)
> >
>
> OOPS ..I am really sorry. I should have given a little thought before
> providing the gfar_select_queue( ) function. Thanks for the pointer.
>
> But then, on the Rx-side, we should also set the "queue_mapping" as
> "queue_mapping +1", since skb_get_rx_queue( ) returns "queue_mapping
> -1". Is this correct ?

Don't use +1/-1 anywhere. Instead of calling skb_set_queue_mapping,
call skb_record_rx_queue which does the +1.

- KK

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html