[lkml]   [2004]   [Jun]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
SubjectRe: [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/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 *);
+static int ahc_linux_sanity_check(struct scsi_device *);
+static void ahc_linux_poll(struct scsi_device *);

* 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,
+ .dump_sanity_check = ahc_linux_sanity_check,
+ .dump_poll = ahc_linux_poll,

/**************************** 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;
return scsi_disk(disk)->device;
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
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 14:03    [W:0.056 / U:18.952 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site