On Wed, Mar 23, 2011 at 02:22:58PM +0100, David Coppa wrote: > On Tue, 18 Jan 2011, Vadim Zhukov wrote: > > > Hello all (again). > > > > So here is a bit reworked patch, taking into account suggestions from > > tedu@, halex@ and jmc@. Particularily, there is no more huge phrase > > "This value is multiplied by the number of 512\-byte blocks in a sector > > to yield the size...": just sectors being converted to (512-byte) > > blocks. I added explicit checks for the sector size be multiply of 512 > > as it's what the kernel does - but the kernel does not print such nice > > error messages and, we need this for our math being correct anyway. > > > > In addition to all suggestions I also fixed ambiguity in mkfs.c, > > adding word "sectors" to error message. > > > > Thanks to all reviewers. Hope this'll be commited. > > Has this been forgotten? > > I'm using it since January and it works fine: > > $ grep mfs /etc/fstab > 4d700a703f68291d.b /tmp mfs rw,nodev,nosuid,-s=256m 0 0 > > $ mount | grep mfs > mfs:5635 on /tmp type mfs (asynchronous, local, nodev, nosuid, size=524288 > 512-blocks) > > > Cheers, > David
Sorry, was going to look at this diff, but forgot. Can somebody send me the latest version? -Otto