On 3/27/19 5:35 PM, Bob Peterson wrote:
Hi,
Yesterday I posted a patch for this problem, but it was grossly inadequate.
This patch, version 2, is another attempt to fix it. Many thanks to Ross
Lagerwall for helping us identify, fix, and test the problem.
Regards,
Bob Peterson
---
gfs2: clean_journal was setting sd_log_flush_head replaying other journals
Function clean_journal was setting the value of sd_log_flush_head,
but that's only a valid thing to do if it is replaying its own journal.
If it's replaying another node's journal, that's completely wrong and
will lead to multiple problems. This patch tries to clean up the
mess by passing the value of head to gfs2_write_log_header so it can
treat non-owned journals fairly.
Signed-off-by: Bob Peterson <[email protected]>
This does seem to fix the issue. Thanks!
Tested-by: Ross Lagerwall <[email protected]>