On Sun, Aug 30, 2015 at 03:47:17PM +0800, Herbert Xu wrote:

Phil Sutter [off-list ref] wrote:

quoted

Should we introduce a new field to struct rhashtable to track the
internal state? This might allow to clean up some rather obscure tests,
e.g. whether a table resize is in progress or not.

Why would we want to do that? The deferred expansion is done
on a best effort basis so its failure has nothing to do with
the failure of a subsequent insertion.

The variable would be used to track if the worker has failed to allocate
memory in background.

Since the failing insertion will be retried, subsequent inserts are not
necessary unrelated.

The insertion must have tried its own last-ditch synchronous
expansion and only fail if that fails.

Who do you mean with "the insertion"? The user or the API?

Cheers, Phil