Hi Goffredo, On Sun, Mar 10, 2013 at 1:17 PM, Goffredo Baroncelli <goffredo.baronce...@yahoo.com> wrote: > Unfortunately I noticed a regression which passed all the reviews until now: > the command btrfs fi df previous didn't require the root capability, > now with my patches it is required, because I need to know some info > about the chunks so I need to use the "BTRFS_IOC_TREE_SEARCH". > > I think that there are the following possibilities: > 1) accept this regresssion > 2) remove the command "btrfs fi df" and leave only "btrfs fi disk-usage" and > "btrfs dev disk-usage" > 3) adding a new ioctl which could be used without root capability. Of course > this ioctl would return only a subset of the BTRFS_IOC_TREE_SEARCH info
I'd like to vote for option 1) here. "btrfs fi df" gives useful, condensed information that I can't get from the others at first glance. I'd argue that many read-only, informational functions of the btrfs utility already require root privileges even where that might not be strictly necessary from a security point of view. I'd also like to bump this thread in the hope that, if no one else steps up with new, specific problems, we can now merge this patchset. We can keep bikeshedding about the display details forever, but I really think this is a significant improvement over what we had previously. Regards, Bart -- 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
