Re: [PATCH 5/6] fs-writeback: move nr_pages == 0 logic to one location

2017-09-20 Thread Jens Axboe
On 09/20/2017 09:36 AM, Jan Kara wrote: > On Wed 20-09-17 09:05:51, Jens Axboe wrote: >> On 09/20/2017 08:41 AM, Jan Kara wrote: >>> On Tue 19-09-17 13:53:06, Jens Axboe wrote: Now that we have no external callers of wb_start_writeback(), we can move the nr_pages == 0 logic into that func

Re: [PATCH 5/6] fs-writeback: move nr_pages == 0 logic to one location

2017-09-20 Thread Jan Kara
On Wed 20-09-17 09:05:51, Jens Axboe wrote: > On 09/20/2017 08:41 AM, Jan Kara wrote: > > On Tue 19-09-17 13:53:06, Jens Axboe wrote: > >> Now that we have no external callers of wb_start_writeback(), > >> we can move the nr_pages == 0 logic into that function. > >> > >> Signed-off-by: Jens Axboe

Re: [PATCH 5/6] fs-writeback: move nr_pages == 0 logic to one location

2017-09-20 Thread Jens Axboe
On 09/20/2017 08:41 AM, Jan Kara wrote: > On Tue 19-09-17 13:53:06, Jens Axboe wrote: >> Now that we have no external callers of wb_start_writeback(), >> we can move the nr_pages == 0 logic into that function. >> >> Signed-off-by: Jens Axboe > > ... > >> +static unsigned long get_nr_dirty_pages(

Re: [PATCH 5/6] fs-writeback: move nr_pages == 0 logic to one location

2017-09-20 Thread Jan Kara
On Tue 19-09-17 13:53:06, Jens Axboe wrote: > Now that we have no external callers of wb_start_writeback(), > we can move the nr_pages == 0 logic into that function. > > Signed-off-by: Jens Axboe ... > +static unsigned long get_nr_dirty_pages(void) > +{ > + return global_node_page_state(NR_

Re: [PATCH 5/6] fs-writeback: move nr_pages == 0 logic to one location

2017-09-19 Thread Johannes Weiner
On Tue, Sep 19, 2017 at 01:53:06PM -0600, Jens Axboe wrote: > Now that we have no external callers of wb_start_writeback(), > we can move the nr_pages == 0 logic into that function. > > Signed-off-by: Jens Axboe Acked-by: Johannes Weiner

[PATCH 5/6] fs-writeback: move nr_pages == 0 logic to one location

2017-09-19 Thread Jens Axboe
Now that we have no external callers of wb_start_writeback(), we can move the nr_pages == 0 logic into that function. Signed-off-by: Jens Axboe --- fs/fs-writeback.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/fs/fs-writeback.c b/fs/fs