From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail2.candelatech.com ([208.74.158.173]:35788 "EHLO mail2.candelatech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759624AbcHELqp (ORCPT ); Fri, 5 Aug 2016 07:46:45 -0400 Message-ID: <57A47CA3.4010101@candelatech.com> (sfid-20160805_134652_065556_CC0CE652) Date: Fri, 05 Aug 2016 04:46:43 -0700 From: Ben Greear MIME-Version: 1.0 To: Johannes Berg , Herbert Xu CC: "David S. Miller" , netdev@vger.kernel.org, linux-wireless@vger.kernel.org, Thomas Graf , tom@herbertland.com Subject: Re: Buggy rhashtable walking References: <20160804071846.GA773@gondor.apana.org.au> <20160804074546.GA996@gondor.apana.org.au> <1470377813.2977.14.camel@sipsolutions.net> <20160805104858.GA9297@gondor.apana.org.au> <1470394233.2977.37.camel@sipsolutions.net> In-Reply-To: <1470394233.2977.37.camel@sipsolutions.net> Content-Type: text/plain; charset=UTF-8; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 08/05/2016 03:50 AM, Johannes Berg wrote: > On Fri, 2016-08-05 at 18:48 +0800, Herbert Xu wrote: >> On Fri, Aug 05, 2016 at 08:16:53AM +0200, Johannes Berg wrote: >>> >>> Hm. Would you rather allocate a separate head entry for the >>> hashtable, >>> or chain the entries? >> >> My plan is to build support for this directly into rhashtable. >> So I'm adding a struct rhlist_head that would be used in place >> of rhash_head for these cases and it'll carry an extra pointer >> for the list of identical entries. >> >> I will then add an additional layer of insert/lookup interfaces >> for rhlist_head. Herbert, thank you for fixing this! It would not be fun to have to revert to the old way of hashing stations in mac80211... I'll be happy to test the patches when you have them ready. Ben -- Ben Greear Candela Technologies Inc http://www.candelatech.com