On 2020/6/24 下午7:41, Johannes Thumshirn wrote:
> On 24/06/2020 13:03, Qu Wenruo wrote:
>>> diff --git a/include/uapi/linux/btrfs.h b/include/uapi/linux/btrfs.h
>>> index e6b6cb0f8bc6..161d9100c2a6 100644
>>> --- a/include/uapi/linux/btrfs.h
>>> +++ b/include/uapi/linux/btrfs.h
>>> @@ -250,10 +250,21 @@ struct btrfs_ioctl_fs_info_args {
>>> __u32 nodesize; /* out */
>>> __u32 sectorsize; /* out */
>>> __u32 clone_alignment; /* out */
>>> + __u32 flags; /* out */
>> The flags looks a little too generic.
>> What about extra_members or things like that?
>>
>> This flag really indicates what extra info the ioctl args can provide,
>> so a better name would be easier to understand.
>
> Maybe version and for each new member it gets incremented?
>
That looks even better!
Thanks,
Qu
Attachment:
signature.asc
Description: OpenPGP digital signature
