Re: [9fans] venti and "contrib": RFC

2012-01-07 Thread Bakul Shah
On Thu, 05 Jan 2012 15:03:11 EST erik quanstrom wrote: > > > > venti doesn't have a "scrub" command, does it? zfs scrub was > > > > instrumental in warning me that I needed new disks. > > > > > > they're using coraid storage. all this is taken care of for them > > > by the SR appliance. > > >

Re: [9fans] BUG: disk/fdisk.c: corrupts MBR

2012-01-07 Thread tlaronde
On Sat, Jan 07, 2012 at 02:30:52PM -0500, erik quanstrom wrote: > > the code assumes c/h/s. and if it (libdisk) can't find the geometry, it > makes stuff > up. that seems like a dubious assumption. not very clean at all. > > i was thinking that setting cyc2cyl=1 should cleanly remove this fea

Re: [9fans] BUG: disk/fdisk.c: corrupts MBR

2012-01-07 Thread erik quanstrom
> > can you just set sec2cyl=1? > > This wouls probably do the trick. But the code is clean and setting just > this is not---to my taste. > > The ideal would be to simply patch the existing with changed values (no > need to be efficient for 4 * 16 bytes), and leave clearly alone values > set by o

Re: [9fans] BUG: disk/fdisk.c: corrupts MBR

2012-01-07 Thread tlaronde
On Sat, Jan 07, 2012 at 02:04:51PM -0500, erik quanstrom wrote: > On Sat Jan 7 13:30:33 EST 2012, tlaro...@polynum.com wrote: > > The explanation is arithmetic truncation. disk/fdisk.c does this: > > > > p->start = lba/sec2cyl; > > > > and, if modifications are done (for example creating a Plan9

Re: [9fans] BUG: disk/fdisk.c: corrupts MBR

2012-01-07 Thread erik quanstrom
On Sat Jan 7 13:30:33 EST 2012, tlaro...@polynum.com wrote: > Hello, > > Since I'm playing with the installation, I have been hit once again > with disk/fdisk corrupting the MBR and rendering other systems > unbootable. > > The explanation is arithmetic truncation. disk/fdisk.c does this: > > p

[9fans] BUG: disk/fdisk.c: corrupts MBR

2012-01-07 Thread tlaronde
Hello, Since I'm playing with the installation, I have been hit once again with disk/fdisk corrupting the MBR and rendering other systems unbootable. The explanation is arithmetic truncation. disk/fdisk.c does this: p->start = lba/sec2cyl; and, if modifications are done (for example creating a