[PATCH 0/6 v2] mkfs: use user values saved in opts table

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

 



Second version of this set. All issues raised so far should be addressed.
The first patch of this set was moved from the other set ("save user input into
opts table"), see specific patches for description of changes.

Text from v1 follows:

The previous set (mkfs: save user input into opts table) added infrastructure,
which we are going to use now. I split it into two patchsets, because this
part is more ugly, so if there is an issue with this part, it doesn't stop
the other patches from merging.

The only thing this set is doing is that it replaces various ad-hoc variables
with set/get functions. As a result, we now have just a single place for these
values to be.  Once all the values are in a single place, we can expand the
automated validation and use it e.g.  for loading up optional config file like
Luis wants to do.

I'm not removing all variables, because for now, we need e.g. flags which are
not in the opts table. This is something that can happen later on, but for now,
I'm just replacing variables that can be done 1:1.

Jan Tulak (6):
  mkfs: save user input values into opts
  mkfs: replace variables with opts table: -b,d,s options
  mkfs: replace variables with opts table: -i options
  mkfs: replace variables with opts table: -l options
  mkfs: replace variables with opts table: -n options
  mkfs: replace variables with opts table: -r options

 mkfs/xfs_mkfs.c | 1246 ++++++++++++++++++++++++++++++++-----------------------
 1 file changed, 728 insertions(+), 518 deletions(-)

-- 
2.13.3

--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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