Re: [PATCH 03/19] fsinfo: Provide a bitmap of supported features [ver #16]

From: David Howells
Date: Thu Feb 20 2020 - 07:22:18 EST


Darrick J. Wong <darrick.wong@xxxxxxxxxx> wrote:

> > +struct fsinfo_features {
> > + __u8 features[(FSINFO_FEAT__NR + 7) / 8];
>
> Hm...the structure size is pretty small (56 bytes) and will expand as we
> add new _FEAT flags. Is this ok because the fsinfo code will truncate
> its response to userspace to whatever buffer size userspace tells it?

Yes. Also, you can ask fsinfo how many feature bits it supports.

I should put this in the struct first rather than putting it elsewhere.

David