On Fri, Jul 7, 2023 at 1:37 AM Shyam Prasad N <nspmangalore@xxxxxxxxx> wrote: > > On Fri, Jul 7, 2023 at 11:20 AM ronnie sahlberg > <ronniesahlberg@xxxxxxxxx> wrote: > > > > We can only cache a limited amount of entries. > > We need to force entries to be dropped from the cache at regular > > intervals to make room for potential new entries/different > > directories. > > > > Access patterns change over time and what is the "hot" directory will > > also change over time so we need to drop entries to make sure that > > when some directory becomes hot there will be decent chance that it > > will be able to become cached. > > > > If a directory becomes "cold" we no longer want it to take up entries > > in our cache. > > > > Makes sense. > > However, the value of MAX_CACHED_FIDS to 16 seems very restrictive. > And as Steve suggested, 30s expiry seems very aggressive. > I think we can increase both. At least for the short term we can make this configurable (which will make it easier to test the best default values in the long run) e.g. via mount parm. Will make it much easier to experiment to optimize the value -- Thanks, Steve
