On 2018年04月24日 13:52, Su Yue wrote: > After call of ref_for_same_block, ref1->parent must equals to > ref2->parent, the block of exchange is never reached. > > So remove the block of exchange.
Reviewed-by: Qu Wenruo <w...@suse.com> The patch looks good, but considering how much difference the kernel and btrfs-progs have in backref.c, I think it would be much better to start merging code for it. Thanks, Qu > > Signed-off-by: Su Yue <suy.f...@cn.fujitsu.com> > --- > backref.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/backref.c b/backref.c > index c144dbf060f2..51553c702187 100644 > --- a/backref.c > +++ b/backref.c > @@ -497,7 +497,6 @@ static void __merge_refs(struct pref_state *prefstate, > int mode) > for (pos2 = pos1->next, n2 = pos2->next; pos2 != head; > pos2 = n2, n2 = pos2->next) { > struct __prelim_ref *ref2; > - struct __prelim_ref *xchg; > struct extent_inode_elem *eie; > > ref2 = list_entry(pos2, struct __prelim_ref, list); > @@ -505,11 +504,6 @@ static void __merge_refs(struct pref_state *prefstate, > int mode) > if (mode == 1) { > if (!ref_for_same_block(ref1, ref2)) > continue;> - > if (!ref1->parent && ref2->parent) { > - xchg = ref1; > - ref1 = ref2; > - ref2 = xchg; > - } > } else { > if (ref1->parent != ref2->parent) > continue; >
signature.asc
Description: OpenPGP digital signature