On 03/30/2018 06:46 AM, Misono Tomohiro wrote:
> On 2018/03/30 2:35, Goffredo Baroncelli wrote:
>> Hi Misono,
>
> Hello,
[...]
>> My conclusion, is that your work is not consistent with the current
>> implementation; so I am suggesting to create a new subcommand ("btrfs sub
>> tree" ?) where you
On 03/30/2018 06:46 AM, Misono Tomohiro wrote:
> [Un]fortunately, the stock "btrfs sub list" has the capability to show all
> the subvolumes, even the ones not mounted, so this can be entirely replaced
> by your API.
s/can be entirely/can't be entirely/
--
gpg @keyserver.linux.it: Goffredo Bar
On 2018/03/30 2:35, Goffredo Baroncelli wrote:
> Hi Misono,
Hello,
>
> I tested you patch, and it seems to work. I verified that the output is
> correct and the permission check is performed. However the output of "btrfs
> sub list" in the "root" mode and the "user" mode are a bit different.
>
Hi Misono,
I tested you patch, and it seems to work. I verified that the output is correct
and the permission check is performed. However the output of "btrfs sub list"
in the "root" mode and the "user" mode are a bit different.
As reported before, I find your output more consistent, and unders
On Wed, Mar 28, 2018 at 09:45:02AM -0400, Zygo Blaxell wrote:
> On Mon, Mar 19, 2018 at 04:30:17PM +0900, Misono, Tomohiro wrote:
> > This is a part of RFC I sent last December[1] whose aim is to improve
> > normal users' usability.
> > The remaining works of RFC are:
> > - Allow "sub delete" f
On Mon, Mar 19, 2018 at 04:30:17PM +0900, Misono, Tomohiro wrote:
> This is a part of RFC I sent last December[1] whose aim is to improve normal
> users' usability.
> The remaining works of RFC are:
> - Allow "sub delete" for empty subvolume
I don't mean to scope creep on you, but I have a cou
changelog:
v2 -> v3
- use get_euid() to check the caller's privilege (and remove 3rd patch)
- improve error handling
v1 -> v2
- add independent error handling patch (1st patch)
- reimplement according to ioctl change
- various cleanup
===
This RFC implements user version of "subvolume l
