Hi, Liu Thank you for your explanation, and I'm sorry for this long silence.
Liu Bo <bo.li....@oracle.com> writes: > You may think of it twice, commit 53ee1bccf99cd5b474fe1aa857b7dd176e3a1407 > is to fix a bug of assigning a free block to two different extent buffers, ie. > two different extent buffers' share the same eb->start, so it's not just > bumping > a reference cnt. > > Right now we want to be consistent with the kernel side, decreasing eb->refs=0 > means it'd be freed, so droping free_some_buffer() can be a good choice. Now I understand the reason why @refs = 1 initially. I'll post a patch to drop free_some_buffer() after this. > And for caching extent buffer, we've increased eb->refs by 1 to keep it in the > cache rbtree. > > thanks, > -liubo Regards, Naohiro -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
