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

Brian Foster
2016-05-11T14:16:02Z

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?

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