lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080103172101.GE30771@mit.edu>
Date:	Thu, 3 Jan 2008 12:21:01 -0500
From:	Theodore Tso <tytso@....edu>
To:	Josef Bacik <jbacik@...hat.com>
Cc:	linux-ext4@...r.kernel.org
Subject: Re: [PATCH] e2fsprogs: add minimal resize size option

On Wed, Jan 02, 2008 at 03:16:59PM -0500, Josef Bacik wrote:
> Hello,
> 
> People wishing to make live usb disks and such are looking for a way
> to get the minimum resize size for an ext fs in blocks so that they
> can just resize their image to that size and do with it what they
> will.  This patch adds that functionality, just pass -m option and
> it calculates the minimum number of blocks the fs can be resized to.

Three comments.  Instead of using a new option, why not simply let
resize2fs check to see if the optional parameter is something like
"min" or "0"?

Secondly, I'd suggest factoring out the calculation of the minimal
size into a separate function.

Finally, please don't fix whitespace issues, as it makes it harder for me
to merge between the "maint" and "master" development lines.  What I
plan to do is to do a massive fixup of trailing whitespace before the
next major (1.41) release of e2fsprogs.  Any new lines added by
patches should not introduce any new trailing whitespace, of course.

Thanks!

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ