From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ie0-x22a.google.com (mail-ie0-x22a.google.com [IPv6:2607:f8b0:4001:c03::22a]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by huchra.bufferbloat.net (Postfix) with ESMTPS id 13D7021F204 for ; Sun, 19 Jan 2014 19:38:10 -0800 (PST) Received: by mail-ie0-f170.google.com with SMTP id u16so5976701iet.1 for ; Sun, 19 Jan 2014 19:38:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=UYiu81AGV7YcxTHInaInH3/6/DC7LB6HEBr3ZczYMXY=; b=C/ivncHYax/5+gZddAEx+REIL41puxUIMx/DQzGSW37lFiq1aeQAPRqEGYPGhN+4qh sJcTsxHeSo9BjvQ4RafjwH2Akm4ioNI+xsq8lBG893+QMQx+8+U9ahs5oiXSFDdMFFkt LyulptbmvKXSirvmmsXnNmkstSzZS8TUHkLx2ylFGKamB9oY6I3SL8FQyczlknyaB4by d0FrEJ6QR9vpo2BzXlSPWptyewR45S4MBnUxa8rQ4j5oHTu9bX14PzpZ/rWQaFod08wo jQukW1AlF3i7tnKkpi7hzeFZRL8rOBEXudxw+KUOjAVTTtAX/tJL2EBbSrskVAM11BA/ e+6Q== MIME-Version: 1.0 X-Received: by 10.50.79.228 with SMTP id m4mr10054710igx.47.1390189089210; Sun, 19 Jan 2014 19:38:09 -0800 (PST) Received: by 10.64.145.67 with HTTP; Sun, 19 Jan 2014 19:38:09 -0800 (PST) In-Reply-To: <21212.38828.910370.71395@gargle.gargle.HOWL> References: <21212.38828.910370.71395@gargle.gargle.HOWL> Date: Sun, 19 Jan 2014 22:38:09 -0500 Message-ID: From: Dave Taht To: Sujith Manoharan Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Cc: "cerowrt-devel@lists.bufferbloat.net" Subject: Re: [Cerowrt-devel] looking over ampdu stats X-BeenThere: cerowrt-devel@lists.bufferbloat.net X-Mailman-Version: 2.1.13 Precedence: list List-Id: Development issues regarding the cerowrt test router project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 20 Jan 2014 03:38:14 -0000 On Sun, Jan 19, 2014 at 10:27 PM, Sujith Manoharan wro= te: > Dave Taht wrote: >> in my tests, we almost never see more than 2 AMPDUs stacked up. (just >> running netperf, not rrul). > > 2 is the minimum queue depth to aggregate packets. From ath9k.h: > #define ATH_AGGR_MIN_QDEPTH 2 > >> This could be the fault of the client device I'm using... > > This is not a bug with the client. Gotcha. I went later (at felix's urging) to look at the rc_stats and that showed some decent averages for the actual number of packets in the AMPDU. Never managed to get it much above 11, regardless of what I set be_qlen to... > >> cat /sys/kernel/debug/ieee80211/phy1/ath9k/queues >> >> (note: have set the default be_qlen to the default here) >> >> (VO): qnum: 0 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 >> (VI): qnum: 1 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 >> (BE): qnum: 2 qdepth: 3 ampdu-depth: 2 pending: 60 stopped: 0 >> (BK): qnum: 3 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 >> (CAB): qnum: 8 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 > > qlen will not affect the minimum aggregate threshold, so adjusting > be_qlen will not affect aggregation. Well, it sort of does (or used to) in that cero holds this value low (12) to minimize latency. While this costs some throughput it seems to help in low rate situations. > > Sujith --=20 Dave T=E4ht Fixing bufferbloat with cerowrt: http://www.teklibre.com/cerowrt/subscribe.= html