On Thu 15-08-13 15:15:04, Andrew Morton wrote:
> On Mon, 15 Jul 2013 11:45:14 +0200 Jan Kara <[email protected]> wrote:
>
> > In case when system contains no dirty pages, wakeup_flusher_threads()
> > will submit WB_SYNC_NONE writeback for 0 pages so wb_writeback() exits
> > immediately without doing anything. Thus sync(1) will write all the
> > dirty inodes from a WB_SYNC_ALL writeback pass which is slow.
> >
> > Fix the problem by using get_nr_dirty_pages() in
> > wakeup_flusher_threads() instead of calculating number of dirty pages
> > manually. That function also takes number of dirty inodes into account.
>
> Fair enough.
>
> > CC: [email protected]
>
> Unexpected. What was your thinking here?
Right, probably the problem isn't serious enough. Just discard the CC
please. Thanks!
Honza
--
Jan Kara <[email protected]>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
