Cool, I had this on my stack too. But can you maybe remove the nonsensical return values, and instead of renaming & keeping the btrfsctl.c copy, why not just use a common copy in utils.c? It'd just be 2 checks for fd < 0 in the btrfsctl callers.
Thanks for the comments Eric. Though I agree, but it deviates from the purpose of this patch-set. It could be taken as a separate patch. Anand -- 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