by NeilBrown[permalink] [raw]
On Thursday February 7, [email protected] wrote:
> Hi Neil-
> I don't have a problem with removing the variant expiry behavior --
> in fact, I think it might be better if NLM host garbage collection
> was done only under memory pressure.
> > - if (++nrhosts > NLM_HOST_MAX)
> > - next_gc = 0;
> > -
> AFAICT this is the only place where nrhosts is bumped. So you should
> either get rid of nrhosts all together, or leave in a ++nrhosts;
It is still used for a sanity check, and would be needed (I think) if
we introduced a memory-pressure handler, so I'll leave it there and
restore the "++nrhosts" - thanks for catching that.