On Tue, Jan 12, 2010 at 11:06:33PM +0100, Christoph Hellwig wrote: > On Tue, Jan 12, 2010 at 09:50:55PM +0200, Michael S. Tsirkin wrote: > > So the issue is that wrong block size (0xffffffff) was passed > > to guest. Would it make sense to add some sanity checking in virtio-blk > > to make it not crash but fail in probe? Which block size values > > are sane? > > Yes, I'll cook up a patch. Basically powers of two up from 512 bytes > are theoretically sane. In practice I doubt we'll ever see anything > other than 512 or 4095 bytes.
I also noticed that 0x8000 causes a crash. You can play with other sizes and see what happens. -- MST