[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAA93jw60L+mxEPnxBeruSsZUZKPBy=N3oWTvQQbsJFk9pwvhAw@mail.gmail.com>
Date: Fri, 23 Dec 2011 19:54:08 +0100
From: Dave Taht <dave.taht@...il.com>
To: Eric Dumazet <eric.dumazet@...il.com>
Cc: Stephen Hemminger <shemminger@...tta.com>,
"John A. Sullivan III" <jsullivan@...nsourcedevel.com>,
netdev@...r.kernel.org
Subject: Re: netem and hierarchical ingress traffic shaping
On Fri, Dec 23, 2011 at 7:28 PM, Eric Dumazet <eric.dumazet@...il.com> wrote:
> Le vendredi 23 décembre 2011 à 18:54 +0100, Dave Taht a écrit :
>
>> Are there any place where all 48 bytes of cb are used?
>>
>
> Yes, but on in qdisc layer.
>
> struct tcp_skb_cb is known to be 44 bytes (when IPv6 is enabled)
>
> In qdisc layer, we use a small part of it, for the moment.
>
>> I wouldn't mind if 'time_to_send' became a separate skb field
>> for a more generic 'time_in_queue'...
>>
>
> This wont happen.
>
> As I posted in an earlier patch, this can be added in "struct
> qdisc_skb_cb"
Ah... this patch?
http://patchwork.ozlabs.org/patch/125329/
I liked what I saw then, but nobody chirped up to review it...
I guess I know what I'm doing this weekend.
>
>
>
--
Dave Täht
SKYPE: davetaht
US Tel: 1-239-829-5608
FR Tel: 0638645374
http://www.bufferbloat.net
--
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