diff options
-rw-r--r-- | drivers/scsi/libata-core.c | 4 | ||||
-rw-r--r-- | drivers/scsi/libata-scsi.c | 18 | ||||
-rw-r--r-- | drivers/scsi/sata_sil.c | 4 | ||||
-rw-r--r-- | include/linux/libata.h | 4 |
4 files changed, 13 insertions, 17 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 1a373b4d0e22..61cba39a6834 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c | |||
@@ -1147,9 +1147,7 @@ void ata_dev_config(struct ata_port *ap, unsigned int i) | |||
1147 | printk(KERN_INFO "ata%u(%u): applying bridge limits\n", | 1147 | printk(KERN_INFO "ata%u(%u): applying bridge limits\n", |
1148 | ap->id, i); | 1148 | ap->id, i); |
1149 | ap->udma_mask &= ATA_UDMA5; | 1149 | ap->udma_mask &= ATA_UDMA5; |
1150 | ap->host->max_sectors = ATA_MAX_SECTORS; | 1150 | ap->device[i].max_sectors = ATA_MAX_SECTORS; |
1151 | ap->host->hostt->max_sectors = ATA_MAX_SECTORS; | ||
1152 | ap->device[i].flags |= ATA_DFLAG_LOCK_SECTORS; | ||
1153 | } | 1151 | } |
1154 | 1152 | ||
1155 | if (ap->ops->dev_config) | 1153 | if (ap->ops->dev_config) |
diff --git a/drivers/scsi/libata-scsi.c b/drivers/scsi/libata-scsi.c index 3628fedc9865..86da46502b3e 100644 --- a/drivers/scsi/libata-scsi.c +++ b/drivers/scsi/libata-scsi.c | |||
@@ -684,23 +684,23 @@ int ata_scsi_slave_config(struct scsi_device *sdev) | |||
684 | if (sdev->id < ATA_MAX_DEVICES) { | 684 | if (sdev->id < ATA_MAX_DEVICES) { |
685 | struct ata_port *ap; | 685 | struct ata_port *ap; |
686 | struct ata_device *dev; | 686 | struct ata_device *dev; |
687 | unsigned int max_sectors; | ||
687 | 688 | ||
688 | ap = (struct ata_port *) &sdev->host->hostdata[0]; | 689 | ap = (struct ata_port *) &sdev->host->hostdata[0]; |
689 | dev = &ap->device[sdev->id]; | 690 | dev = &ap->device[sdev->id]; |
690 | 691 | ||
691 | /* TODO: 1024 is an arbitrary number, not the | 692 | /* TODO: 2048 is an arbitrary number, not the |
692 | * hardware maximum. This should be increased to | 693 | * hardware maximum. This should be increased to |
693 | * 65534 when Jens Axboe's patch for dynamically | 694 | * 65534 when Jens Axboe's patch for dynamically |
694 | * determining max_sectors is merged. | 695 | * determining max_sectors is merged. |
695 | */ | 696 | */ |
696 | if ((dev->flags & ATA_DFLAG_LBA48) && | 697 | max_sectors = ATA_MAX_SECTORS; |
697 | ((dev->flags & ATA_DFLAG_LOCK_SECTORS) == 0)) { | 698 | if (dev->flags & ATA_DFLAG_LBA48) |
698 | /* | 699 | max_sectors = 2048; |
699 | * do not overwrite sdev->host->max_sectors, since | 700 | if (dev->max_sectors) |
700 | * other drives on this host may not support LBA48 | 701 | max_sectors = dev->max_sectors; |
701 | */ | 702 | |
702 | blk_queue_max_sectors(sdev->request_queue, 2048); | 703 | blk_queue_max_sectors(sdev->request_queue, max_sectors); |
703 | } | ||
704 | 704 | ||
705 | /* | 705 | /* |
706 | * SATA DMA transfers must be multiples of 4 byte, so | 706 | * SATA DMA transfers must be multiples of 4 byte, so |
diff --git a/drivers/scsi/sata_sil.c b/drivers/scsi/sata_sil.c index 61c4ac7ff9db..6c482c8be254 100644 --- a/drivers/scsi/sata_sil.c +++ b/drivers/scsi/sata_sil.c | |||
@@ -354,9 +354,7 @@ static void sil_dev_config(struct ata_port *ap, struct ata_device *dev) | |||
354 | (quirks & SIL_QUIRK_MOD15WRITE))) { | 354 | (quirks & SIL_QUIRK_MOD15WRITE))) { |
355 | printk(KERN_INFO "ata%u(%u): applying Seagate errata fix (mod15write workaround)\n", | 355 | printk(KERN_INFO "ata%u(%u): applying Seagate errata fix (mod15write workaround)\n", |
356 | ap->id, dev->devno); | 356 | ap->id, dev->devno); |
357 | ap->host->max_sectors = 15; | 357 | dev->max_sectors = 15; |
358 | ap->host->hostt->max_sectors = 15; | ||
359 | dev->flags |= ATA_DFLAG_LOCK_SECTORS; | ||
360 | return; | 358 | return; |
361 | } | 359 | } |
362 | 360 | ||
diff --git a/include/linux/libata.h b/include/linux/libata.h index 853c98859a9f..afe46457124e 100644 --- a/include/linux/libata.h +++ b/include/linux/libata.h | |||
@@ -122,8 +122,7 @@ enum { | |||
122 | /* struct ata_device stuff */ | 122 | /* struct ata_device stuff */ |
123 | ATA_DFLAG_LBA48 = (1 << 0), /* device supports LBA48 */ | 123 | ATA_DFLAG_LBA48 = (1 << 0), /* device supports LBA48 */ |
124 | ATA_DFLAG_PIO = (1 << 1), /* device currently in PIO mode */ | 124 | ATA_DFLAG_PIO = (1 << 1), /* device currently in PIO mode */ |
125 | ATA_DFLAG_LOCK_SECTORS = (1 << 2), /* don't adjust max_sectors */ | 125 | ATA_DFLAG_LBA = (1 << 2), /* device supports LBA */ |
126 | ATA_DFLAG_LBA = (1 << 3), /* device supports LBA */ | ||
127 | 126 | ||
128 | ATA_DEV_UNKNOWN = 0, /* unknown device */ | 127 | ATA_DEV_UNKNOWN = 0, /* unknown device */ |
129 | ATA_DEV_ATA = 1, /* ATA device */ | 128 | ATA_DEV_ATA = 1, /* ATA device */ |
@@ -348,6 +347,7 @@ struct ata_device { | |||
348 | 347 | ||
349 | unsigned int multi_count; /* sectors count for | 348 | unsigned int multi_count; /* sectors count for |
350 | READ/WRITE MULTIPLE */ | 349 | READ/WRITE MULTIPLE */ |
350 | unsigned int max_sectors; /* per-device max sectors */ | ||
351 | unsigned int cdb_len; | 351 | unsigned int cdb_len; |
352 | 352 | ||
353 | /* for CHS addressing */ | 353 | /* for CHS addressing */ |