Thread (44 messages) 44 messages, 6 authors, 2008-08-19

Re: qdisc_enqueue, NET_XMIT_SUCCESS and kfree_skb

From: Patrick McHardy <hidden>
Date: 2008-08-19 13:20:08

Herbert Xu wrote:
On Tue, Aug 19, 2008 at 03:08:57PM +0200, Patrick McHardy wrote:
quoted
quoted
Another possibility would to replace requeue by a peek+force_dequeue
interface, where you can peek at the next packet, and you could then
dequeue that particular packet if you're satisfied.
That would be fine for TBF since it only needs to check whether
the current packet exceeds the limit and reschedule otherwise.
HFSC OTOH really needs to know the length of the next packet for
calculating the deadline.
You mean a peek interface is insufficient for HFSC? Could you
elaborate?
I might have misunderstood you, but the way I imagine force_dequeue
is that it would give you the packet peeked at, even if a higher
priority packet is available.

But actually I don't understand the use for force_dequeue at all,
assuming ->peek behaves correctly ->dequeue should already hand
out the correct packet.

(Note: Its OK to hand out a different packet if we had a ->enqueue
operation after ->peek since the upper qdisc can just re-peek and
recalculate based on the new highest priority packet).

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help