On Wed, Nov 11, 2020 at 09:45:38AM +0000, Satya Tangirala wrote: > On Wed, Nov 11, 2020 at 09:23:05AM +0000, Christoph Hellwig wrote: > > On Tue, Nov 10, 2020 at 06:14:27PM -0800, Eric Biggers wrote: > > > + * hash_ptr() assumes bits != 0, so ensure the hash table has at least 2 > > > + * buckets. This only makes a difference when there is only 1 keyslot. > > > + */ > > > + slot_hashtable_size = max(slot_hashtable_size, 2U); > > > > shouldn't this be a min()? > I think it should be max(), since we want whichever is larger between 2 > and the original slot_hashtable_size :) Yes, of course. Sorry for the noise.