Re: [PATCH 020/102] xfs: warn if direct reclaim tries to writeback pages

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 020/102] xfs: warn if direct reclaim tries to writeback pages
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Mon, 27 Aug 2012 14:17:42 -0400
Cc:	xfs@xxxxxxxxxxx, Mel Gorman <mgorman@xxxxxxx>
In-reply-to:	<1345698180-13612-21-git-send-email-david@xxxxxxxxxxxxx>
References:	<1345698180-13612-1-git-send-email-david@xxxxxxxxxxxxx> <1345698180-13612-21-git-send-email-david@xxxxxxxxxxxxx>
User-agent:	Mutt/1.5.21 (2010-09-15)

On Thu, Aug 23, 2012 at 03:01:38PM +1000, Dave Chinner wrote:
> From: Mel Gorman <mgorman@xxxxxxx>
> 
> Upstream commit: 94054fa3fca1fd78db02cb3d68d5627120f0a1d4
> 
> Direct reclaim should never writeback pages.  For now, handle the
> situation and warn about it.  Ultimately, this will be a BUG_ON.

Is this actually the case on 3.0-stable?

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 011/102] xfs: XFS_TRANS_SWAPEXT is not a valid flag for, (continued) [PATCH 011/102] xfs: XFS_TRANS_SWAPEXT is not a valid flag for, Dave Chinner [PATCH 021/102] xfs: fix force shutdown handling in xfs_end_io, Dave Chinner [PATCH 025/102] xfs: make sure to really flush all dquots in xfs_qm_quotacheck, Dave Chinner [PATCH 027/102] xfs: mark the xfssyncd workqueue as non-reentrant, Dave Chinner [PATCH 023/102] xfs: fix the logspace waiting algorithm, Dave Chinner [PATCH 018/102] xfs: do not flush data workqueues in xfs_flush_buftarg, Dave Chinner [PATCH 024/102] xfs: untangle SYNC_WAIT and SYNC_TRYLOCK meanings for xfs_qm_dqflush, Dave Chinner [PATCH 026/102] xfs: simplify xfs_qm_detach_gdquots, Dave Chinner [PATCH 031/102] xfs: always return with the iolock held from, Dave Chinner [PATCH 020/102] xfs: warn if direct reclaim tries to writeback pages, Dave Chinner Re: [PATCH 020/102] xfs: warn if direct reclaim tries to writeback pages, Christoph Hellwig <= [PATCH 028/102] xfs: make i_flags an unsigned long, Dave Chinner [PATCH 030/102] xfs: remove the i_new_size field in struct xfs_inode, Dave Chinner [PATCH 033/102] xfs: pass KM_SLEEP flag to kmem_realloc() in, Dave Chinner [PATCH 034/102] xfs: show uuid when mount fails due to duplicate uuid, Dave Chinner [PATCH 029/102] xfs: remove the i_size field in struct xfs_inode, Dave Chinner [PATCH 038/102] xfs: remove xfs_trans_unlocked_item, Dave Chinner [PATCH 032/102] xfs: cleanup xfs_file_aio_write, Dave Chinner [PATCH 036/102] xfs: split tail_lsn assignments from log space wakeups, Dave Chinner [PATCH 037/102] xfs: do exact log space wakeups in xlog_ungrant_log_space, Dave Chinner [PATCH 039/102] xfs: cleanup xfs_log_space_wake, Dave Chinner

Previous by Date:	Re: [PATCH] xfstests: test dump/restore of 32-bit projids, Eric Sandeen
Next by Date:	[RFC PATCH 1/4] xfs: add EOFBLOCKS inode tagging/untagging, Brian Foster
Previous by Thread:	[PATCH 020/102] xfs: warn if direct reclaim tries to writeback pages, Dave Chinner
Next by Thread:	[PATCH 028/102] xfs: make i_flags an unsigned long, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]