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 <[email protected]> wrote: > From: "Yan, Zheng" <[email protected]> > > 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 <[email protected]> > --- > 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 [email protected] > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
