Date: Sat, 27 Aug 2011 06:30:01 +0000 (UTC) From: Adrian Chadd <adrian@FreeBSD.org> To: src-committers@freebsd.org, svn-src-user@freebsd.org Subject: svn commit: r225207 - user/adrian/if_ath_tx/sys/dev/ath Message-ID: <201108270630.p7R6U2uG087849@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
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; }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201108270630.p7R6U2uG087849>