On Wed 15-08-12 02:11:30, Joel Becker wrote:
> EDQUOT is a user-visible error, not a logic problem. Teach mlog_errno()
> to ignore it like it ignores -ENOSPC, etc.
Looks sane. You can add:
Reviewed-by: Jan Kara <jack@xxxxxxx>
Honza
>
> Signed-off-by: Joel Becker <jlbec@xxxxxxxxxxxx>
> Cc: Jan Kara <jack@xxxxxxx>
> Reported-by: Marek Królikowski <admin@xxxxxxxxxxx>
> ---
> fs/ocfs2/cluster/masklog.h | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/fs/ocfs2/cluster/masklog.h b/fs/ocfs2/cluster/masklog.h
> index baa2b9e..2260fb9 100644
> --- a/fs/ocfs2/cluster/masklog.h
> +++ b/fs/ocfs2/cluster/masklog.h
> @@ -199,7 +199,8 @@ extern struct mlog_bits mlog_and_bits, mlog_not_bits;
> #define mlog_errno(st) do { \
> int _st = (st); \
> if (_st != -ERESTARTSYS && _st != -EINTR && \
> - _st != AOP_TRUNCATED_PAGE && _st != -ENOSPC) \
> + _st != AOP_TRUNCATED_PAGE && _st != -ENOSPC && \
> + _st != -EDQUOT) \
> mlog(ML_ERROR, "status = %lld\n", (long long)_st); \
> } while (0)
>
> --
> 1.7.7.3
>
--
Jan Kara <jack@xxxxxxx>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
