svn commit: r260363 - head/sys/dev/ath

Adrian Chadd adrian at freebsd.org
Mon Jan 6 19:49:06 UTC 2014


On 6 January 2014 09:37, Konstantin Belousov <kostikbel at gmail.com> wrote:
>> > This is a NOP, right ?
>>
>> Nope. the previous version didn't remove the entries from the list.
>> that's the bug.
> Hm, you mean that the previous version did not TAILQ_INIT() after the loop ?
> Then I agree.

Well, it's that the frames just get moved between the RX queue active
list(s) or the rxbuf inactive list. The correct thing to do is to
TAILQ_REMOVE them from one list before adding them to the free list.

-a


More information about the svn-src-head mailing list