xfs: rework log recovery to submit buffers on LSN boundaries
authorBrian Foster <bfoster@redhat.com>
Sun, 25 Sep 2016 22:22:16 +0000 (08:22 +1000)
committerDave Chinner <david@fromorbit.com>
Sun, 25 Sep 2016 22:22:16 +0000 (08:22 +1000)
commit12818d24db8ab01836bf423a7c1be639c3135572
treea481144f3f078ba6e545576511cbe7213f47f653
parentddeb14f4fb2fa1242829a76edc821f087e50bcdf
xfs: rework log recovery to submit buffers on LSN boundaries

The fix to log recovery to update the metadata LSN in recovered buffers
introduces the requirement that a buffer is submitted only once per
current LSN. Log recovery currently submits buffers on transaction
boundaries. This is not sufficient as the abstraction between log
records and transactions allows for various scenarios where multiple
transactions can share the same current LSN. If independent transactions
share an LSN and both modify the same buffer, log recovery can
incorrectly skip updates and leave the filesystem in an inconsisent
state.

In preparation for proper metadata LSN updates during log recovery,
update log recovery to submit buffers for write on LSN change boundaries
rather than transaction boundaries. Explicitly track the current LSN in
a new struct xlog field to handle the various corner cases of when the
current LSN may or may not change.

Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
fs/xfs/xfs_log_priv.h
fs/xfs/xfs_log_recover.c