Re: [patch 4/5] btrfs: add nofail variant of set_extent_dirty

From: Peter Zijlstra
Date: Tue Aug 24 2010 - 09:31:12 EST


On Tue, 2010-08-24 at 03:50 -0700, David Rientjes wrote:
> Add set_extent_dirty_nofail(). This function is equivalent to
> set_extent_dirty(), except that it will never fail because of allocation
> failure and instead loop forever trying to allocate memory.
>
> If the first allocation attempt fails, a warning will be emitted,
> including a call trace. Subsequent failures will suppress this warning.
>
> This was added as a helper function for documentation and auditability.
> No future callers should be added.
>
> Signed-off-by: David Rientjes <rientjes@xxxxxxxxxx>
> ---
> fs/btrfs/extent-tree.c | 8 ++++----
> fs/btrfs/extent_io.c | 19 +++++++++++++++++++
> fs/btrfs/extent_io.h | 2 ++
> 3 files changed, 25 insertions(+), 4 deletions(-)

I'd much rather someone help mason to clean this up.
--
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/