xfs
[Top] [All Lists]

Re: [PATCH 0/7] Configurable error behavior [V4]

To: xfs@xxxxxxxxxxx
Subject: Re: [PATCH 0/7] Configurable error behavior [V4]
From: Carlos Maiolino <cmaiolino@xxxxxxxxxx>
Date: Wed, 11 May 2016 16:40:19 +0200
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160511141558.GE42410@xxxxxxxxxxxxxxx>
Mail-followup-to: xfs@xxxxxxxxxxx
References: <1462882581-30859-1-git-send-email-cmaiolino@xxxxxxxxxx> <20160511141558.GE42410@xxxxxxxxxxxxxxx>
User-agent: Mutt/1.5.24 (2015-08-30)
On Wed, May 11, 2016 at 10:15:59AM -0400, Brian Foster wrote:
> On Tue, May 10, 2016 at 02:16:14PM +0200, Carlos Maiolino wrote:
> > New revision ot the patchset according with comments for V3.
> > 
> > This patchset is aimed to implement a configurable error behavior in XFS, 
> > and,
> > most of the design has been done by Dave, so, I kept his signed-off.
> > 
> > Detailed changelog is written on each patch, but, major changes are:
> > 
> >     - make fail_at_unmount a global configuration option inside
> >       ../xfs/<dev>/err/fail_at_unmount
> > 
> 
> Any reason you didn't carry the reviewed-by tags from v3?

No reason Brian, I just completely forgot to carry the tags to the new version,
sorry for the extra work.

> 
> For Dave's convenience (and based on the reviews of v3 and v4), for the
> series:
> 
> Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx>
> 
> > Carlos Maiolino (7):
> >   xfs: configurable error behavior via sysfs
> >   xfs: introduce metadata IO error class
> >   xfs: add configurable error support to metadata buffers
> >   xfs: introduce table-based init for error behaviors
> >   xfs: add configuration of error failure speed
> >   xfs: add configuration handlers for specific errors
> >   xfs: add "fail at unmount" error handling configuration
> > 
> >  fs/xfs/xfs_buf.h      |  20 ++++
> >  fs/xfs/xfs_buf_item.c | 121 +++++++++++++--------
> >  fs/xfs/xfs_mount.c    |  22 +++-
> >  fs/xfs/xfs_mount.h    |  34 ++++++
> >  fs/xfs/xfs_sysfs.c    | 291 
> > +++++++++++++++++++++++++++++++++++++++++++++++++-
> >  fs/xfs/xfs_sysfs.h    |   3 +
> >  fs/xfs/xfs_trace.h    |   1 -
> >  7 files changed, 446 insertions(+), 46 deletions(-)
> > 
> > -- 
> > 2.4.11
> > 
> > _______________________________________________
> > xfs mailing list
> > xfs@xxxxxxxxxxx
> > http://oss.sgi.com/mailman/listinfo/xfs

-- 
Carlos

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