Re: [survey] sysfs layout for btrfs

[Gabriel de Perthuis <g2p.code@gmail.com>]

On Sat, 15 Aug 2015 07:40:40 +0800, Anand Jain wrote:

> Hello,
> 
> as of now btrfs sysfs does not include the attributes for the volume 
> manager part in its sysfs layout, so its being developed and there are 
> two types of layout here below, so I have a quick survey to know which 
> will be preferred. contenders are:
> 1. FS and VM (volume manager) attributes[1] merged sysfs layout
> 
>    /sys/fs/btrfs/<fsid> <-- holds FS attr, VM attr will be added here.
>    /sys/fs/btrfs/<fsid>/devices/<uuid [2] > <-- btrfs_devices attr here

My vote is for the first one.
Lengthening the UI/API with /pools/ seems unnecessary, and it's
better to get attributes exposed earlier.

> 2. FS and VM attributes separated sysfs layout.
> 
>   /sys/fs/btrfs/<fsid> <--- as is, will continue to hold fs attributes.
>   /sys/fs/btrfs/pools/<fsid>/ <-- will hold VM attributes
>   /sys/fs/btrfs/pools/<fsid>/devices/<sdx> <-- btrfs_devices attr here