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

Carlos Maiolino
2016-05-11T14:40:26Z

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@redhat.com>
>
> > 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@oss.sgi.com
> > http://oss.sgi.com/mailman/listinfo/xfs

--
Carlos