Re: do we still need ->is_dirty_writeback

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 18, 2018 at 07:08:12PM +0200, Christoph Hellwig wrote:
> Hi Mel,
> 
> you added the is_dirty_writeback callback a couple years ago mostly
> to work around the crazy ext3 writeback code, which is long gone now.
> We still use buffer_check_dirty_writeback on the block device, but
> without that ext3 case we really should not need it anymore.
> 
> That leaves NFS, where I don't understand why it doesn't simply
> use PageWrite?

If you mean PageWriteback the patch was to treat some pages as if they were
under writeback even if the page state didn't reflect that. The intent was
to catch the case when kswapd was scanning that it would not prematurely
skip over pages under writeback or were really dirty from being skipped and
clean pages being reclaimed instead. In an extreme case, it would avoid
a premature OOM if the number of clean pages that could be reclaimed was
too small. However, it was (is?) a corner case and the mechanisms that
control throttling have changed a lot since.

If the callback is problematic for some reason, I don't object to it
being removed. At worst, there will be rare cases where reclaim finds
clean pages and steals them prematurely. There are plenty of other cases
where page age inversion issues exist (e.g. NUMA machines that reclaim
young pages from local node when the working set is larger than a node)
and it's rare that people notice.

-- 
Mel Gorman
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux