errseq: Always report a writeback error once
authorMatthew Wilcox <willy@infradead.org>
Tue, 24 Apr 2018 21:02:57 +0000 (14:02 -0700)
committerJeff Layton <jlayton@redhat.com>
Fri, 27 Apr 2018 12:51:26 +0000 (08:51 -0400)
commitb4678df184b314a2bd47d2329feca2c2534aa12b
tree6c9da9eabee5ab70d5b06d996e380c858496c07e
parent0644f186fc9d77bb5bd198369e59fb28927a3692
errseq: Always report a writeback error once

The errseq_t infrastructure assumes that errors which occurred before
the file descriptor was opened are of no interest to the application.
This turns out to be a regression for some applications, notably Postgres.

Before errseq_t, a writeback error would be reported exactly once (as
long as the inode remained in memory), so Postgres could open a file,
call fsync() and find out whether there had been a writeback error on
that file from another process.

This patch changes the errseq infrastructure to report errors to all
file descriptors which are opened after the error occurred, but before
it was reported to any file descriptor.  This restores the user-visible
behaviour.

Cc: stable@vger.kernel.org
Fixes: 5660e13d2fd6 ("fs: new infrastructure for writeback error handling and reporting")
Signed-off-by: Matthew Wilcox <mawilcox@microsoft.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
lib/errseq.c