From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Subject: Re: tc linklayer ADSL calc broken after commit 56b765b79 (htb: improved accuracy at high rates) Date: Thu, 30 May 2013 07:39:10 -0700 Message-ID: <1369924750.5109.91.camel@edumazet-glaptop> References: <20130529151330.22c5c89e@redhat.com> <1369842724.5109.44.camel@edumazet-glaptop> <20130530095117.503eeee7@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Jesper Dangaard Brouer , Stephen Hemminger , David Miller , j.vimal@gmail.com, Michal Soltys , Mike Frysinger , Jussi Kivilinna , Patrick McHardy , Jiri Pirko , Toke =?ISO-8859-1?Q?H=F8iland-J=F8rgensen?= , Dave Taht , netdev@vger.kernel.org, bloat@lists.bufferbloat.net, Dan Siemon , Jim Gettys , Steven Barth , Felix Fietkau , Jiri Benc To: Jesper Dangaard Brouer Return-path: Received: from mail-pb0-f41.google.com ([209.85.160.41]:55971 "EHLO mail-pb0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932733Ab3E3OjN (ORCPT ); Thu, 30 May 2013 10:39:13 -0400 Received: by mail-pb0-f41.google.com with SMTP id xb12so467845pbc.28 for ; Thu, 30 May 2013 07:39:13 -0700 (PDT) In-Reply-To: <20130530095117.503eeee7@redhat.com> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 2013-05-30 at 09:51 +0200, Jesper Dangaard Brouer wrote: > On Wed, 29 May 2013 08:52:04 -0700 > Eric Dumazet wrote: > > I am not sure it will solve the ATM logic (with the 5 bytes overhead > > per 48 bytes cell) > > Are you talking about, that for GSO frames we are not adding a encap > overhead to each "sub" skb. This part is now done properly in qdisc_pkt_len_init() since linux-3.9 No I was really mentioning the DIV_ROUND_UP(pkt_len,48)*53 thing