Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-04 Thread Eric D. Mudama
On 8/4/05, Jan Engelhardt <[EMAIL PROTECTED]> wrote: > All of these numbers are virtual, since CHS is not really used anymore, as > we know. But, which of these fake CHS values (16383/16/63 | 65535/16/63 | > 1023/255/63) is the right one? 255/63/4982 is another matter, since it > [almost] matches t

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-04 Thread Mark Bellon
Jan Engelhardt wrote: Simple: do not use BIOS values. [ Yes, there should be some warning from kernel. ] On that matter, I get a warning from LILO wrt cyls and stuff: 07:47 spectre:~ # cat /proc/ide/hda/geometry physical 16383/16/63 logical 65535/16/63 07:58 spectre:~ # lilo W

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Jan Engelhardt
>Simple: do not use BIOS values. >[ Yes, there should be some warning from kernel. ] On that matter, I get a warning from LILO wrt cyls and stuff: 07:47 spectre:~ # cat /proc/ide/hda/geometry physical 16383/16/63 logical 65535/16/63 07:58 spectre:~ # lilo Warning: Kernel & BIOS return

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Bartlomiej Zolnierkiewicz
On 8/3/05, Mark Bellon <[EMAIL PROTECTED]> wrote: > Bartlomiej Zolnierkiewicz wrote: > > >On 8/3/05, Mark Bellon <[EMAIL PROTECTED]> wrote: > > > > > >>Bartlomiej Zolnierkiewicz wrote: > >> > >> > >> > >>>Hi, > >>> > >>>The topic was discussed to death on linux-kernel. > >>> > >>>Mark, you need to

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Mark Bellon
Bartlomiej Zolnierkiewicz wrote: On 8/3/05, Mark Bellon <[EMAIL PROTECTED]> wrote: Bartlomiej Zolnierkiewicz wrote: Hi, The topic was discussed to death on linux-kernel. Mark, you need to fix your applications and stop using /proc/ide/hd*/geometry or/and HDIO_GET_GEO ioctl (which BT

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Bartlomiej Zolnierkiewicz
On 8/3/05, Mark Bellon <[EMAIL PROTECTED]> wrote: > Bartlomiej Zolnierkiewicz wrote: > > >Hi, > > > >The topic was discussed to death on linux-kernel. > > > >Mark, you need to fix your applications and stop using /proc/ide/hd*/geometry > >or/and HDIO_GET_GEO ioctl (which BTW your patch also affect

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Mark Bellon
Bartlomiej Zolnierkiewicz wrote: Hi, The topic was discussed to death on linux-kernel. Mark, you need to fix your applications and stop using /proc/ide/hd*/geometry or/and HDIO_GET_GEO ioctl (which BTW your patch also affects). Fixing the applications I can understand but the patch still se

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Bartlomiej Zolnierkiewicz
Hi, The topic was discussed to death on linux-kernel. Mark, you need to fix your applications and stop using /proc/ide/hd*/geometry or/and HDIO_GET_GEO ioctl (which BTW your patch also affects). Bartlomiej On 8/3/05, Andre Hedrick <[EMAIL PROTECTED]> wrote: > > Did you read ATA-1 through ATA-7

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Mark Bellon
Andre Hedrick wrote: Did you read ATA-1 through ATA-7 to understand all the variations? Regardless of all of the geometry returns by the drives and their ATA compliance, the existing code will fail for some drives and return values. For instance, the existing code attempts to "fix up" LBA 4

Re: [PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-03 Thread Andre Hedrick
Did you read ATA-1 through ATA-7 to understand all the variations? On Tue, 2 Aug 2005, Mark Bellon wrote: > The ATA specification tells large disk drives to return C/H/S data of > 16383/16/63 regardless of their actual size (other variations on this > return include 15 heads and/or 4092 cylind

[PATCH] IDE disks show invalid geometries in /proc/ide/hd*/geometry

2005-08-02 Thread Mark Bellon
The ATA specification tells large disk drives to return C/H/S data of 16383/16/63 regardless of their actual size (other variations on this return include 15 heads and/or 4092 cylinders). Unfortunately these CHS data confuse the existing IDE code and cause it to report invalid geometries in /pr