netdev - Re: [PATCH net-next-2.6 1/2] net: add IFLA_NUM

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 for Android: free password hash cracker in your pocket

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

Message-ID: <1284787983.4373.560.camel@edumazet-laptop>
Date:	Sat, 18 Sep 2010 07:33:03 +0200
From:	Eric Dumazet <eric.dumazet@...il.com>
To:	David Miller <davem@...emloft.net>
Cc:	netdev@...r.kernel.org
Subject: Re: [PATCH net-next-2.6 1/2] net: add IFLA_NUM_TXQ attribute

Le vendredi 17 septembre 2010 à 16:29 -0700, David Miller a écrit :
> From: Eric Dumazet <eric.dumazet@...il.com>
> Date: Fri, 17 Sep 2010 10:31:28 +0200
> 
> > In order to enable multiqueue support on some devices,
> > add IFLA_NUM_TXQ attribute, number of transmit queues, that "ip link"
> > can use, at creation and show time :
> > 
> > # ip link add gre34 txqueues 8 type gre remote 192.168.20.80
> > 
> > # ip link sho dev gre34
> > 8: gre34: <POINTOPOINT,NOARP> mtu 1476 qdisc noop state DOWN txqueues 8 
> >     link/gre 0.0.0.0 peer 192.168.20.80
> > 
> > Drivers not yet multiqueue aware are supported, because core network
> > temporary sets real_num_tx_queues to one.
> > 
> > Multiqueue enabled drivers must then sets real_num_tx_queues to
> > num_tx_queues in their newlink() method.
> > 
> > Limits number of queues to 256 for the moment.
> > 
> > Signed-off-by: Eric Dumazet <eric.dumazet@...il.com>
> 
> This is one way to solve the problem, but I think we can do a lot
> better.
> 
> What is the true barrier for full parallel processing over GRE tunnels
> at the moment?
> 
> It seems to me that the only issue that exists is the TXQ->lock done
> by dev_queue_xmit() for the GRE tunnel xmit.
> 
> This is something we should have fixed ages ago, and we tried with the
> ugly LLTX thing.  In my opinion all paths leading to a non-queueing
> device should not take the TX lock, because by definition there is no
> queueing state or synchronization to be cognizant of.
> 
> Actually, statistics can matter but we already have to address that
> problem seperately for the sake of 64-bit stats on 32-bit machines.
> 

Agreed, and even before 64bit stats, we did percpu stats on
loopback/bridge...

> Alexey even open condones this in the huge comment that sits in
> the "!q->enqueue" path of dev_queue_xmit().
> 
> If we take care of this, then TX multi-queue works transparently for
> all software devices layered on top of suitably capable hardware,
> without us having to make any explicit multi-queue changes to the
> software device code.
> 
> Eric, if you can demonstrate a real need for this once we solve the
> fundamental issue, as I have outlined above, I am happy to add this
> netlink attribute and tunable.  But for now I'm deferring these
> two patches.
> 
> Thanks!

Hmm, in my case, I was interested in RX processing and RPS.
(Asymetric routing)

The only way to have more than one rx queue today is to have more than
one tx queue.

But this probably can be addressed separately.

Thanks !


--
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