On Fri, May 2, 2014 at 4:46 PM, David Sterba <dste...@suse.cz> wrote:
> On Sun, Apr 20, 2014 at 10:40:03PM +0100, Filipe David Borba Manana wrote:
>> So that applications can find out what's the highest send stream
>> version supported/implemented by the running kernel:
>>
>>     $ cat /sys/fs/btrfs/send/stream_version
>>     2
>>
>> Signed-off-by: Filipe David Borba Manana <fdman...@gmail.com>
>> ---
>>
>> V2: Renamed /sys/fs/btrfs/send_stream_version to 
>> /sys/fs/btrfs/send/stream_version,
>>     as in the future it might be useful to add other sysfs attrbutes related 
>> to
>>     send (other ro information or tunables like internal buffer sizes, etc).
>
> Sounds good, I don't see any issue with the separate directory. Mixing
> it with /sys/fs/btrfs/features does not seem suitable for that if you
> intend adding more entries.

Yeah, I only didn't mix it with the features subdir because that
relates to features that are settable, plus there's 2 versions of it,
one global and one per fs (uuid) subdirectory (and it felt odd to me
to add it to one of those subdirs and not the other).

Thanks David

>
> Reviewed-by: David Sterba <dste...@suse.cz>



-- 
Filipe David Manana,

"Reasonable men adapt themselves to the world.
 Unreasonable men adapt the world to themselves.
 That's why all progress depends on unreasonable men."
--
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

Reply via email to