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
| ||
|
Date: Thu, 03 Sep 2009 20:22:45 +0200 From: Patrick McHardy <kaber@...sh.net> To: Eric Dumazet <eric.dumazet@...il.com> CC: "David S. Miller" <davem@...emloft.net>, Linux Netdev List <netdev@...r.kernel.org> Subject: Re: [PATCH net-next-2.6] macvlan: add multiqueue capability Eric Dumazet wrote: > Patrick McHardy a écrit : >> The patch looks fine, but it just occured to me that this won't >> have any effect since both VLAN and macvlan use a tx_queue_len of 0, >> so they will by default have queueing disabled. In fact this >> will increase costs for the default case since we're now hashing >> every packet. > > Good point ! > > We'll have to hash the packet later when hitting the lowerdevice, > which is multiqueue. No ? Right. But we don't reuse that decision from what I can tell. > Also, what's wrong with > > ip link add link eth0 eth0.103 txqueuelen 100 type vlan id 103 There's nothing wrong, but its kind of pointless since with the default qdisc the queue will be bypassed, other qdiscs are shared between the queues and defeat multiqueue. I guess it could make sense if you want to apply TC actions or something like that once we support using different (non-shared) qdiscs for each queue. -- 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
Powered by blists - more mailing lists