Re: [PATCH 2/3] vfs: Add a super block operation to check for writeback errors

From: Vivek Goyal
Date: Tue Dec 22 2020 - 11:26:43 EST


On Tue, Dec 22, 2020 at 04:19:00PM +0000, Matthew Wilcox wrote:
> On Mon, Dec 21, 2020 at 02:50:54PM -0500, Vivek Goyal wrote:
> > - ret2 = errseq_check_and_advance(&sb->s_wb_err, &f.file->f_sb_err);
> > + if (sb->s_op->errseq_check_advance)
> > + ret2 = sb->s_op->errseq_check_advance(sb, f.file);
>
> What a terrible name for an fs operation. You don't seem to be able
> to distinguish between semantics and implementation. How about
> check_error()?

check_error() sounds better. I was not very happy with the name either.
Thought of starting with something.

Vivek