tech-userlevel archive

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

Re: resize_ffs improvements

On Oct,Friday 29 2010, at 5:31 PM, Christos Zoulas wrote:

> In article <>,
> Adam Hamsik  <> wrote:
>> hi,
>> I have done some small improvements to resize_ffs. With [1] patch it ask
>> for confirmation if user has run fsck on filesystem (requested by agc@)
>> and I have changed default behaviour of tool to grow filesystem to
>> device size by default, shrinking is still possible with -s parameter. I
>> would like to commit this patch and add resize_ffs to release build to
>> have at least ffsv1 grow/shrink working for users.
>> [1]
> - How will the code after you call err(3) get reached in get_dev_size?
> - Why return 0 size instead of erroring out when the partition is not found?
> - whitespace in while for getopt
> - whitespace around = in strtoul in main.
> - whitespace in strcasecmp in main.
> - whitespace in exit
> - knf the printf
> - why do you use err(1, and err(EXIT_FAILURE, ?
> - usage should be noreturn.

Please have a look at [1]

I fixed your comments and changed manual page to respect my changes. 

If you have better idea about needed ffsv2 changes, can you describe them to me 




Home | Main Index | Thread Index | Old Index