Re: [PATCH 01/10] xfs: mount-api - add fs parameter description

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 8/22/19 4:05 AM, Ian Kent wrote:
> On Wed, 2019-08-21 at 09:52 -0500, Eric Sandeen wrote:
>> On 6/23/19 9:58 PM, Ian Kent wrote:
>>> The new mount-api uses an array of struct fs_parameter_spec for
>>> parameter parsing, create this table populated with the xfs mount
>>> parameters.
>>>
>>> The new mount-api table definition is wider than the token based
>>> parameter table and interleaving the option description comments
>>> between each table line is much less readable than adding them to
>>> the end of each table entry. So add the option description comment
>>> to each entry line even though it causes quite a few of the entries
>>> to be longer than 80 characters.
>>>
>>> Signed-off-by: Ian Kent <raven@xxxxxxxxxx>
>>
>> Ian, I saw hints about a V2 in replies, is that still in the works?
> 
> I have a v2 that should address the comments made on the initial
> post.
> 
> My reluctance to post the v2 is because the series uses a patch
> that's currently sitting in linux-next and that will cause a
> conflict if the series is accepted.
> 
> I'm thinking about doing an RFC post with that patch included
> and a caution about the possible conflict.

That should be fine, just let everyone know what the patchset is based
on and we can do another round of review & sort out the dependencies
as needed, I think.

Thanks
-Eric
 
> Not sure yet, I've only just yesterday been able to successfully
> run xfstests on a kernel with the changes so deciding what to do
> next is upon me now, ;)



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux