[lkml]   [2009]   [Mar]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[patch 43/96] libata: Dont trust current capacity values in identify words 57-58
    2.6.27-stable review patch.  If anyone has any objections, please let us know.


    From: Robert Hancock <>

    commit 968e594afdbc40b4270f9d4032ae8350475749d6 upstream.

    Hanno Böck reported a problem where an old Conner CP30254 240MB hard drive
    was reported as 1.1TB in capacity by libata:

    This was caused by libata trusting the drive's reported current capacity in
    sectors in identify words 57 and 58 if the drive does not support LBA and the
    current CHS translation values appear valid. Unfortunately it seems older
    ATA specs were vague about what this field should contain and a number of drives
    used values with wrong byte order or that were totally bogus. There's no
    unique information that it conveys and so we can just calculate the number
    of sectors from the reported current CHS values.

    While we're at it, clean up this function to use named constants for the
    identify word values.

    Signed-off-by: Robert Hancock <>
    Signed-off-by: Jeff Garzik <>
    Signed-off-by: Greg Kroah-Hartman <>

    drivers/ata/libata-core.c | 10 ++++++----
    1 file changed, 6 insertions(+), 4 deletions(-)

    --- a/drivers/ata/libata-core.c
    +++ b/drivers/ata/libata-core.c
    @@ -1190,14 +1190,16 @@ static u64 ata_id_n_sectors(const u16 *i
    if (ata_id_has_lba(id)) {
    if (ata_id_has_lba48(id))
    - return ata_id_u64(id, 100);
    + return ata_id_u64(id, ATA_ID_LBA_CAPACITY_2);
    - return ata_id_u32(id, 60);
    + return ata_id_u32(id, ATA_ID_LBA_CAPACITY);
    } else {
    if (ata_id_current_chs_valid(id))
    - return ata_id_u32(id, 57);
    + return id[ATA_ID_CUR_CYLS] * id[ATA_ID_CUR_HEADS] *
    - return id[1] * id[3] * id[6];
    + return id[ATA_ID_CYLS] * id[ATA_ID_HEADS] *
    + id[ATA_ID_SECTORS];

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2009-03-14 01:37    [W:0.024 / U:37.120 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site