netdev - Re: [PATCH 0/9] Make dev_hard_start_xmit() work fundamentall on lists

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] [day] [month] [year] [list]

Date:	Tue, 2 Sep 2014 09:51:20 +0200
From:	Jesper Dangaard Brouer <brouer@...hat.com>
To:	David Miller <davem@...emloft.net>
Cc:	brouer@...hat.com, netdev@...r.kernel.org
Subject: Re: [PATCH 0/9] Make dev_hard_start_xmit() work fundamentall on
 lists

On Mon, 01 Sep 2014 15:24:30 -0700 (PDT)
David Miller <davem@...emloft.net> wrote:

> After this patch set, dev_hard_start_xmit() will work fundemantally
> on any and all SKB lists.

This is really excellent work, thank you!
dev_hard_start_xmit() looks so clean now :-)

 
> This opens the path for a clean implementation of pulling multiple
> packets out during qdisc_restart(), and then passing that blob
> in one shot to dev_hard_start_xmit().

Sounds perfect.  I'll start to experiment with qdisc stuff.

> And with those two issues out of the way, it should now be trivial to
> build experiments on top of this patch set, all of the framework
> should be there now.  You could do something as simple as:

As xmit_list()/dev_hard_start_xmit() depend on all SKBs belonging to
the same TXQ, below code should also take this into account.

Thus, bulk dequeue will only see the benefit (of taking the qdisc lock
less) when packets are for the same TXQ.  But I guess this is a good
design choice, as this makes the rest of the code simpler to work with.

> 	skb = q->dequeue(q);
> 	if (skb)
> 		skb = validate_xmit_skb(skb, qdisc_dev(q));
> 	if (skb) {
> 		struct sk_buff *new, *head = skb;
> 		int limit = 5;
> 
> 		do {
> 			new = q->dequeue(q);
> 			if (new)
> 				new = validate_xmit_skb(new, qdisc_dev(q));
> 			if (new) {
> 				skb->next = new;
> 				skb = new;
> 			}
> 		} while (new && --limit);
> 		skb = head;
> 	}
> 
> inside of the else branch of dequeue_skb().
> 
> Signed-off-by: David S. Miller <davem@...emloft.net>


-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Sr. Network Kernel Developer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer
--
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