Re: [PATCH v2 3/4] fs/dcache: Enable automatic pruning of negative dentries

2017-07-24 Thread Waiman Long
On 07/21/2017 07:07 PM, James Bottomley wrote: > On Fri, 2017-07-21 at 16:17 -0400, Waiman Long wrote: >> On 07/21/2017 03:30 PM, James Bottomley wrote: >>> On Fri, 2017-07-21 at 09:43 -0400, Waiman Long wrote: Having a limit for the number of negative dentries does have an undesirable si

Re: [PATCH v2 3/4] fs/dcache: Enable automatic pruning of negative dentries

2017-07-21 Thread James Bottomley
On Fri, 2017-07-21 at 16:17 -0400, Waiman Long wrote: > On 07/21/2017 03:30 PM, James Bottomley wrote: > > > > On Fri, 2017-07-21 at 09:43 -0400, Waiman Long wrote: > > > > > > Having a limit for the number of negative dentries does have an > > > undesirable side effect that no new negative dentr

Re: [PATCH v2 3/4] fs/dcache: Enable automatic pruning of negative dentries

2017-07-21 Thread Waiman Long
On 07/21/2017 03:30 PM, James Bottomley wrote: > On Fri, 2017-07-21 at 09:43 -0400, Waiman Long wrote: >> Having a limit for the number of negative dentries does have an >> undesirable side effect that no new negative dentries will be allowed >> when the limit is reached. This will have performance

Re: [PATCH v2 3/4] fs/dcache: Enable automatic pruning of negative dentries

2017-07-21 Thread James Bottomley
On Fri, 2017-07-21 at 09:43 -0400, Waiman Long wrote: > Having a limit for the number of negative dentries does have an > undesirable side effect that no new negative dentries will be allowed > when the limit is reached. This will have performance implication > for some types of workloads. This re

[PATCH v2 3/4] fs/dcache: Enable automatic pruning of negative dentries

2017-07-21 Thread Waiman Long
Having a limit for the number of negative dentries does have an undesirable side effect that no new negative dentries will be allowed when the limit is reached. This will have performance implication for some types of workloads. So we need a way to prune the negative dentries so that new ones can