From owner-svn-src-user@FreeBSD.ORG Sat Aug 27 06:30:02 2011 Return-Path: Delivered-To: svn-src-user@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3FE6D106566B; Sat, 27 Aug 2011 06:30:02 +0000 (UTC) (envelope-from adrian@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id 2BE918FC08; Sat, 27 Aug 2011 06:30:02 +0000 (UTC) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.4/8.14.4) with ESMTP id p7R6U2bj087851; Sat, 27 Aug 2011 06:30:02 GMT (envelope-from adrian@svn.freebsd.org) Received: (from adrian@localhost) by svn.freebsd.org (8.14.4/8.14.4/Submit) id p7R6U2uG087849; Sat, 27 Aug 2011 06:30:02 GMT (envelope-from adrian@svn.freebsd.org) Message-Id: <201108270630.p7R6U2uG087849@svn.freebsd.org> From: Adrian Chadd Date: Sat, 27 Aug 2011 06:30:01 +0000 (UTC) To: src-committers@freebsd.org, svn-src-user@freebsd.org X-SVN-Group: user MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r225207 - user/adrian/if_ath_tx/sys/dev/ath X-BeenThere: svn-src-user@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the experimental " user" src tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 27 Aug 2011 06:30:02 -0000 Author: adrian Date: Sat Aug 27 06:30:01 2011 New Revision: 225207 URL: http://svn.freebsd.org/changeset/base/225207 Log: Give in and just zero bf_state when allocating a new buffer. The lingering bits of state that are floating around were causing occasional issues in hostap mode where the buffers allocated for beacon frames would be incorrectly accounted for (eg they'd still have the bfs_aggr flag set.) Modified: user/adrian/if_ath_tx/sys/dev/ath/if_ath.c Modified: user/adrian/if_ath_tx/sys/dev/ath/if_ath.c ============================================================================== --- user/adrian/if_ath_tx/sys/dev/ath/if_ath.c Fri Aug 26 23:45:27 2011 (r225206) +++ user/adrian/if_ath_tx/sys/dev/ath/if_ath.c Sat Aug 27 06:30:01 2011 (r225207) @@ -1891,6 +1891,7 @@ _ath_getbuf_locked(struct ath_softc *sc) bf->bf_next = NULL; /* XXX just to be sure */ bf->bf_last = NULL; /* XXX again, just to be sure */ bf->bf_comp = NULL; /* XXX again, just to be sure */ + bzero(&bf->bf_state, sizeof(bf->bf_state)); return bf; }