From owner-svn-src-head@FreeBSD.ORG Thu Aug 8 09:23:47 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 0F2B49A0; Thu, 8 Aug 2013 09:23:47 +0000 (UTC) (envelope-from glebius@FreeBSD.org) Received: from cell.glebius.int.ru (glebius.int.ru [81.19.69.10]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 8B3AE2268; Thu, 8 Aug 2013 09:23:45 +0000 (UTC) Received: from cell.glebius.int.ru (localhost [127.0.0.1]) by cell.glebius.int.ru (8.14.7/8.14.7) with ESMTP id r789Nh54071797; Thu, 8 Aug 2013 13:23:43 +0400 (MSK) (envelope-from glebius@FreeBSD.org) Received: (from glebius@localhost) by cell.glebius.int.ru (8.14.7/8.14.7/Submit) id r789NhTH071796; Thu, 8 Aug 2013 13:23:43 +0400 (MSK) (envelope-from glebius@FreeBSD.org) X-Authentication-Warning: cell.glebius.int.ru: glebius set sender to glebius@FreeBSD.org using -f Date: Thu, 8 Aug 2013 13:23:43 +0400 From: Gleb Smirnoff To: Adrian Chadd Subject: Re: svn commit: r254082 - head/sys/net80211 Message-ID: <20130808092343.GP20104@FreeBSD.org> References: <201308080509.r7859aWv084743@svn.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <201308080509.r7859aWv084743@svn.freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 08 Aug 2013 09:23:47 -0000 On Thu, Aug 08, 2013 at 05:09:36AM +0000, Adrian Chadd wrote: A> Author: adrian A> Date: Thu Aug 8 05:09:35 2013 A> New Revision: 254082 A> URL: http://svnweb.freebsd.org/changeset/base/254082 A> A> Log: A> Convert net80211 over to using if_transmit for the dispatch from the A> upper layer(s). A> A> This eliminates the if_snd queue from net80211. Yay! A> A> This unfortunately has a few side effects: A> A> * It breaks ALTQ to net80211 for now - sorry everyone, but fixing A> parallelism and eliminating the if_snd queue is more important A> than supporting this broken traffic scheduling model. :-) A> A> * There's no VAP and IC flush methods just yet - I think I'll add A> some NULL methods for now just as placeholders. A> A> * It reduces throughput a little because now net80211 will drop packets A> rather than buffer them if the driver doesn't do its own buffering. A> This will be addressed in the future as I implement per-node software A> queues. Now access to IFF_DRV_OACTIVE bit isn't locked. Is this okay? Ideally drivers should be written in such way that they don't need to set and clear this flag at all. Is it still needed for net80211? -- Totus tuus, Glebius.