I came across this patch while going through my email backlog and it looks like we haven't pulled in this patch or anything like it. Did you do something about this problem in a different way? (The patch doesn't apply cleanly so I'll need to update it if this is still what we've got.) -Greg Software Engineer #42 @ http://inktank.com | http://ceph.com On Thu, Aug 22, 2013 at 6:42 PM, Yan, Zheng <zheng.z.yan@xxxxxxxxx> wrote: > From: "Yan, Zheng" <zheng.z.yan@xxxxxxxxx> > > We updated inode format version to 7 when introducing inode backtrace. > If we found a inode's format version < 7 when fetching the inode, set > the inode's backtrace_version to 0, to indicate the inode's backtrace > is out of data. When touching a inode with backtrace_version == 0, we > also update its backtrace. > > Signed-off-by: Yan, Zheng <zheng.z.yan@xxxxxxxxx> > --- > src/mds/Locker.cc | 4 ++++ > src/mds/Server.cc | 5 +++++ > src/mds/mdstypes.cc | 2 ++ > 3 files changed, 11 insertions(+) > > diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc > index 99bd761..e274204 100644 > --- a/src/mds/Locker.cc > +++ b/src/mds/Locker.cc > @@ -2898,6 +2898,10 @@ bool Locker::_do_cap_update(CInode *in, Capability *cap, > > wrlock_force(&in->xattrlock, mut); > } > + > + // update backtrace for old format inode. (see inode_t::decode) > + if (pi->backtrace_version == 0) > + pi->update_backtrace(); > > mut->auth_pin(in); > mdcache->predirty_journal_parents(mut, &le->metablob, in, 0, PREDIRTY_PRIMARY, 0, follows); > diff --git a/src/mds/Server.cc b/src/mds/Server.cc > index 466d481..b50169e 100644 > --- a/src/mds/Server.cc > +++ b/src/mds/Server.cc > @@ -3207,6 +3207,11 @@ void Server::handle_client_setattr(MDRequest *mdr) > > // log + wait > le->metablob.add_client_req(req->get_reqid(), req->get_oldest_client_tid()); > + > + // update backtrace for old format inode. (see inode_t::decode) > + if (pi->backtrace_version == 0) > + pi->update_backtrace(); > + > mdcache->predirty_journal_parents(mdr, &le->metablob, cur, 0, PREDIRTY_PRIMARY, false); > mdcache->journal_dirty_inode(mdr, &le->metablob, cur); > > diff --git a/src/mds/mdstypes.cc b/src/mds/mdstypes.cc > index 6886786..f8ec549 100644 > --- a/src/mds/mdstypes.cc > +++ b/src/mds/mdstypes.cc > @@ -294,6 +294,8 @@ void inode_t::decode(bufferlist::iterator &p) > ::decode(backtrace_version, p); > if (struct_v >= 7) > ::decode(old_pools, p); > + else > + backtrace_version = 0; // note inode which has no backtrace > > DECODE_FINISH(p); > } > -- > 1.8.1.4 > > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
