On Thu, April 25, 2013 at 04:16 (+0200), Wang Shilong wrote: > I just have an example in my mind, considering the following example: > > qgroup(1/1) > / \ > / \ > subv(257) snapshot(260) > | / > | / > shared_extent > > For the above example, your code can make qgroup(1/1)'s exclusive correct? > > In your second step, you update every qgroup's referenced correct. But you > only > updte every qgroup's refcnt only once, so in your last step, you won't update > qgroup > (1/1)'s exlusive...Or am i missing someting ^_^
You're absolutely right. Will be fixed in v4. I only don't see how that could possibly pass my previous tests. Thanks, -Jan -- 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