Source-Changes-D archive

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

Re: CVS commit: src/sys/ufs/ffs



> Module Name:  src
> Committed By: pooka
> Date:         Mon Aug  9 15:50:13 UTC 2010
> 
> Modified Files:
>       src/sys/ufs/ffs: ffs_vfsops.c
> 
> Log Message:
> Return error if we try to mount a file system with block size > MAXBSIZE.
> 
> Note: there is a billion ways to make the kernel panic by trying
> to mount a garbage file system and I don't imagine we'll ever get
> close to fixing even half of them.  However, for this one failing
> gracefully is a bonus since Xen DomU only does 32k MAXBSIZE and
> the 64k MAXBSIZE file systems are out there (PR port-xen/43727).
> 
> Tested by compiling sys/rump with CPPFLAGS+=-DMAXPHYS=32768 (all
> tests in tests/fs still pass).  I don't know how we're going to
> translate this into an easy regression test, though.  Maybe with
> a hacked newfs?

No. What you need is dd, /dev/zero, some disk space, vnconfig, newfs, mount, 
umount and rm in this order
to have an easy regression test. :)

Christoph


Home | Main Index | Thread Index | Old Index