| Message ID | 1476259970-1866-5-git-send-email-robbieko@synology.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Oct 12, 2016 at 9:12 AM, robbieko <robbieko@synology.com> wrote: > From: Robbie Ko <robbieko@synology.com> > > There a some case similar as before. As before what? Each change log should be complete and the reader is not supposed to guess what's the previous patch or commit this is referring to. Imagine yourself or someone else reading the change log some time after this is committed to a git tree. How does he/she figures out what is "before", what commit or patch is it? > add check parent generation in the did_overwrite_ref. Please add some explanation of what problem is being fixed and how. This change log has absolutely no value. Also, can you please start sending xfstests too? Last batch of send fixes you've sent, I've asked you to do them, but you totally ignored it and later on I had to do them myself and rewrite all change logs (and remove some unnecessary code). Thanks. > > Signed-off-by: Robbie Ko <robbieko@synology.com> > --- > fs/btrfs/send.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c > index d908624..e090db2 100644 > --- a/fs/btrfs/send.c > +++ b/fs/btrfs/send.c > @@ -1939,6 +1939,19 @@ static int did_overwrite_ref(struct send_ctx *sctx, > if (ret <= 0) > goto out; > > + if (dir != BTRFS_FIRST_FREE_OBJECTID) { > + ret = get_inode_info(sctx->send_root, dir, NULL, &gen, NULL, > + NULL, NULL, NULL); > + if (ret < 0 && ret != -ENOENT) > + goto out; > + if (ret) { > + ret = 0; > + goto out; > + } > + if (gen != dir_gen) > + goto out; > + } > + > /* check if the ref was overwritten by another ref */ > ret = lookup_dir_item_inode(sctx->send_root, dir, name, name_len, > &ow_inode, &other_type); > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index d908624..e090db2 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -1939,6 +1939,19 @@ static int did_overwrite_ref(struct send_ctx *sctx, if (ret <= 0) goto out; + if (dir != BTRFS_FIRST_FREE_OBJECTID) { + ret = get_inode_info(sctx->send_root, dir, NULL, &gen, NULL, + NULL, NULL, NULL); + if (ret < 0 && ret != -ENOENT) + goto out; + if (ret) { + ret = 0; + goto out; + } + if (gen != dir_gen) + goto out; + } + /* check if the ref was overwritten by another ref */ ret = lookup_dir_item_inode(sctx->send_root, dir, name, name_len, &ow_inode, &other_type);