]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
xfs: don't shutdown log recovery on validation errors
authorDave Chinner <dchinner@redhat.com>
Wed, 12 Jun 2013 02:19:06 +0000 (12:19 +1000)
committerBen Myers <bpm@sgi.com>
Fri, 14 Jun 2013 20:29:31 +0000 (15:29 -0500)
commit9222a9cf86c0d64ffbedf567412b55da18763aa3
treee233bb133d288cd23545df3a9a6026489dff4c89
parentade1335afef556df6538eb02e8c0dc91fbd9cc37
xfs: don't shutdown log recovery on validation errors

Unfortunately, we cannot guarantee that items logged multiple times
and replayed by log recovery do not take objects back in time. When
they are taken back in time, the go into an intermediate state which
is corrupt, and hence verification that occurs on this intermediate
state causes log recovery to abort with a corruption shutdown.

Instead of causing a shutdown and unmountable filesystem, don't
verify post-recovery items before they are written to disk. This is
less than optimal, but there is no way to detect this issue for
non-CRC filesystems If log recovery successfully completes, this
will be undone and the object will be consistent by subsequent
transactions that are replayed, so in most cases we don't need to
take drastic action.

For CRC enabled filesystems, leave the verifiers in place - we need
to call them to recalculate the CRCs on the objects anyway. This
recovery problem can be solved for such filesystems - we have a LSN
stamped in all metadata at writeback time that we can to determine
whether the item should be replayed or not. This is a separate piece
of work, so is not addressed by this patch.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Ben Myers <bpm@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
fs/xfs/xfs_log_recover.c