From 992d959e239d034b68120aa393340c8ce9578785 Mon Sep 17 00:00:00 2001 From: John Spray Date: Fri, 24 Jul 2015 09:37:15 +0100 Subject: [PATCH] mds: fix val used in inode->last_journaled This was getting assigned with LogEvent::get_start_offset on an uncommitted LogEvent, which is junk. During replay last_journaled is compared with the metablob's event_seq, so that's what should be used here. This change just from code inspection -- haven't seen this manifest as an actual misbehaviour. Signed-off-by: John Spray --- src/mds/Migrator.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mds/Migrator.cc b/src/mds/Migrator.cc index e7e0e0a39a72..eff1025a9af2 100644 --- a/src/mds/Migrator.cc +++ b/src/mds/Migrator.cc @@ -2932,7 +2932,8 @@ int Migrator::decode_import_dir(bufferlist::iterator& blp, else if (icode == 'I') { // inode assert(le); - decode_import_inode(dn, blp, oldauth, ls, le->get_start_off(), peer_exports, updated_scatterlocks); + decode_import_inode(dn, blp, oldauth, ls, le->get_metablob()->event_seq, + peer_exports, updated_scatterlocks); } // add dentry to journal entry -- 2.47.3