xfs
[Top] [All Lists]

Re: [PATCH V4] xfs: Document error handlers behavior

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH V4] xfs: Document error handlers behavior
From: Carlos Maiolino <cmaiolino@xxxxxxxxxx>
Date: Thu, 15 Sep 2016 05:18:51 -0400
Cc: linux-xfs@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160914220951.GP30497@dastard>
Mail-followup-to: Dave Chinner <david@xxxxxxxxxxxxx>, linux-xfs@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx
References: <1473757385-81633-1-git-send-email-cmaiolino@xxxxxxxxxx> <20160914012334.GK30497@dastard> <20160914100219.5743534ofe6oktbb@xxxxxxxxxx> <20160914220951.GP30497@dastard>
User-agent: Mutt/1.5.24 (2015-08-30)
> > Also, I apologize if I misunderstand it, but being ignored doesn't look a 
> > proper
> > description here, it sounds to me something like 'we ignore the error and 
> > tell
> > nobody about it", in unmount example, we shut down the filesystem if any 
> > error
> > happens, for me it doesn't sound like ignoring an error, but I might be
> > interpreting it in the wrong way.
> 
> I think you're making the assumption that the only way we handle
> errors once retries are exhausted is to trigger a filesystem shutdown.
> That assumption was repeated throughout the documentation.
> 
> While that may be true for /metadata write IO errors/, it is not
> true for the generic error handling case. e.g. if we extend it to
> memory allocation contexts, we may end up returning ENOMEM to
> userspace. Or, in certain contexts, we might be able to fall back to
> doing a single operation at a time using the stack for storage, in
> which case there is no reason at all to report the allocation
> failure to anyone.
> 
> The infrastructure is generic, as is the documentation, and so it
> shouldn't assume anything about what is going to happen once the
> retries are exhausted and the error is propagated upwards. What
> happens with that error after it is returned is a subsystem and
> context dependent behaviour, not something that is defined by the
> error retry configuration infrastructure....
> 
> Cheers,

Thanks for the very detailed explanation Dave
> 
> Dave.
> -- 
> Dave Chinner
> david@xxxxxxxxxxxxx
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs

-- 
Carlos

<Prev in Thread] Current Thread [Next in Thread>