Re: [PATCH 3/8] ext4: Warn if direct reclaim tries to writeback pages

From: Johannes Weiner
Date: Wed Aug 03 2011 - 07:06:55 EST


On Wed, Aug 03, 2011 at 12:58:19PM +0200, Johannes Weiner wrote:
> On Thu, Jul 21, 2011 at 05:28:45PM +0100, Mel Gorman wrote:
> > Direct reclaim should never writeback pages. Warn if an attempt
> > is made.
> >
> > Signed-off-by: Mel Gorman <mgorman@xxxxxxx>
>
> Acked-by: Johannes Weiner <jweiner@xxxxxxxxxx>

Oops, too fast.

Shouldn't the WARN_ON() be at the top of the function, rather than
just warn when the write is deferred due to delalloc?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/