2 messages, 2 authors, 2016-09-19 · open the first message on its own page
From: Johannes Berg <hidden>
Date: 2016-09-19 10:10:27
Btw, for debug I put
BUG_ON(atomic_read(&ht->nelems) < 0);
after the atomic_dec() in __rhashtable_remove_fast_one(). That makes
the kernel crash instantly on the buggy code, and I just have to run a
single test ("wpas_ctrl_interface_add_many") to get there.
johannes
From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2016-09-19 10:48:13
On Mon, Sep 19, 2016 at 12:10:27PM +0200, Johannes Berg wrote:
Btw, for debug I put
BUG_ON(atomic_read(&ht->nelems) < 0);
after the atomic_dec() in __rhashtable_remove_fast_one(). That makes
the kernel crash instantly on the buggy code, and I just have to run a
single test ("wpas_ctrl_interface_add_many") to get there.Aha I see the problem now. The nelems logic on remove is broken. I'll send out a v3. Thanks! -- Email: Herbert Xu [off-list ref] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt