Messages in this thread Patch in this message |  | | | Date | Thu, 17 Jun 2004 22:21:27 +0900 | | From | Takao Indoh <> | | Subject | Re: [PATCH 2/4]Diskdump Update |
| |
On Thu, 17 Jun 2004 13:49:57 +0100, Christoph Hellwig wrote:
>my old comments for this are still valid, please add the actual dumping >methods directly to scsi_host_template instead of a pointer to another >method vector,
I have already done in the latest patch.
diff -Nur linux-2.6.6.org/drivers/scsi/aic7xxx/aic7xxx_osm.c linux-2.6.6/drivers/scsi/aic7xxx/aic7xxx_osm.c --- linux-2.6.6.org/drivers/scsi/aic7xxx/aic7xxx_osm.c 2004-06-04 21:22:20.000000000 +0900 +++ linux-2.6.6/drivers/scsi/aic7xxx/aic7xxx_osm.c 2004-06-16 19:34:16.000000000 +0900 @@ -774,6 +774,10 @@ static int ahc_linux_bus_reset(Scsi_Cmnd *); static int ahc_linux_dev_reset(Scsi_Cmnd *); static int ahc_linux_abort(Scsi_Cmnd *); +#if defined(CONFIG_SCSI_DUMP) || defined(CONFIG_SCSI_DUMP_MODULE) +static int ahc_linux_sanity_check(struct scsi_device *); +static void ahc_linux_poll(struct scsi_device *); +#endif /* * Calculate a safe value for AHC_NSEG (as expressed through ahc_linux_nseg). @@ -1313,6 +1317,10 @@ .slave_alloc = ahc_linux_slave_alloc, .slave_configure = ahc_linux_slave_configure, .slave_destroy = ahc_linux_slave_destroy, +#if defined(CONFIG_SCSI_DUMP) || defined(CONFIG_SCSI_DUMP_MODULE) + .dump_sanity_check = ahc_linux_sanity_check, + .dump_poll = ahc_linux_poll, +#endif }; /**************************** Tasklet Handler *********************************/
>please make it not a module of it's own but part of the >scsi code,
Do you mean scsi_dump module should be merged with sd_mod.o or scsi_mod.o?
>please don't use "scsi.h" in new code,
Oh, I missed it. Thanks.
>and the find gendisk >by dev_t code in sd.c is still no good.
In the latest source, sd_find_scsi_device, which I added, does not use dev_t. Here is the latest version.
struct scsi_device *sd_find_scsi_device(struct block_device *bdev) { struct gendisk *disk = bdev->bd_disk; /* Check whether dev is scsi or not */ if (!disk || (disk->fops != &sd_fops)) return NULL; if(disk->private_data) return scsi_disk(disk)->device; else return NULL; } I'll try sysfs attribute instead of this.
>On Thu, 27 May 2004 14:48:40 +0100, Christoph Hellwig wrote: >>Not the kind of interface we want exported. IMHO you shouldn't find >>device by dev_t but add a dumpdevice sysfs attribute to the scsi_device >>where you can echo 1 to to make it a possible dump device.
Best Regards, Takao Indoh - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |