Re: [PATCH 2/2] writeback: allow for dirty metadata accounting

2016-08-10 Thread Tejun Heo
Hello, Josef. On Wed, Aug 10, 2016 at 05:16:03PM -0400, Josef Bacik wrote: > > It bothers me a bit that sb's can actually be off bdi->sb_list while > > sb_list_lock is released. Can we make this explicit? e.g. keep > > separate bdi sb list for sb's pending metadata writeout (like b_dirty) > > or

Re: [PATCH 2/2] writeback: allow for dirty metadata accounting

2016-08-10 Thread Jan Kara
On Tue 09-08-16 15:08:27, Josef Bacik wrote: > Provide a mechanism for file systems to indicate how much dirty metadata they > are holding. This introduces a few things > > 1) Zone stats for dirty metadata, which is the same as the NR_FILE_DIRTY. > 2) WB stat for dirty metadata. This way we know

Re: [PATCH 2/2] writeback: allow for dirty metadata accounting

2016-08-10 Thread Josef Bacik
On 08/10/2016 04:12 PM, Tejun Heo wrote: Hello, Josef. On Tue, Aug 09, 2016 at 03:08:27PM -0400, Josef Bacik wrote: Provide a mechanism for file systems to indicate how much dirty metadata they are holding. This introduces a few things 1) Zone stats for dirty metadata, which is the same as th

Re: [PATCH 2/2] writeback: allow for dirty metadata accounting

2016-08-10 Thread Tejun Heo
Hello, Josef. On Tue, Aug 09, 2016 at 03:08:27PM -0400, Josef Bacik wrote: > Provide a mechanism for file systems to indicate how much dirty metadata they > are holding. This introduces a few things > > 1) Zone stats for dirty metadata, which is the same as the NR_FILE_DIRTY. > 2) WB stat for di

Re: [PATCH 2/2] writeback: allow for dirty metadata accounting

2016-08-10 Thread Josef Bacik
On 08/10/2016 06:09 AM, Jan Kara wrote: On Tue 09-08-16 15:08:27, Josef Bacik wrote: Provide a mechanism for file systems to indicate how much dirty metadata they are holding. This introduces a few things 1) Zone stats for dirty metadata, which is the same as the NR_FILE_DIRTY. 2) WB stat for

[PATCH 2/2] writeback: allow for dirty metadata accounting

2016-08-09 Thread Josef Bacik
Provide a mechanism for file systems to indicate how much dirty metadata they are holding. This introduces a few things 1) Zone stats for dirty metadata, which is the same as the NR_FILE_DIRTY. 2) WB stat for dirty metadata. This way we know if we need to try and call into the file system to wri