On Sun, Aug 06, 2006 at 06:15:22PM -0500, James Bottomley wrote:
> Caused the bus to be added to dev_printk, so now our SCSI inquiry short
> messages print like this:
>
> scsiscsi 2:0:0:0: Direct access IBM-ESXS ST973401SS B519 PQ: 0 ANSI: 5
>
> Just remove the "scsi" from the sdev_printk to compensate.
Yes, that's clearly the right thing to do, thanks for fixing that up.
I think we should probably switch sdev_printk() to its own custom
implementation rather than just calling dev_printk(). Greg didn't seem
to be interested in fixing the problem that we have with dev_printk();
presumably he has some other users in mind for which the current
dev_printk behaviour is useful.
How does something like the following look (compile tested):
Index: include/scsi/scsi_device.h
===================================================================
RCS file: /var/cvs/linux-2.6/include/scsi/scsi_device.h,v
retrieving revision 1.28
diff -u -p -r1.28 scsi_device.h
--- include/scsi/scsi_device.h 19 May 2006 02:43:19 -0000 1.28
+++ include/scsi/scsi_device.h 7 Aug 2006 02:45:24 -0000
@@ -151,10 +151,10 @@ struct scsi_device {
to_scsi_device(class_dev->dev)
#define sdev_printk(prefix, sdev, fmt, a...) \
- dev_printk(prefix, &(sdev)->sdev_gendev, fmt, ##a)
+ printk(prefix "scsi %s: " fmt, (sdev)->sdev_gendev.bus_id , ## a)
#define scmd_printk(prefix, scmd, fmt, a...) \
- dev_printk(prefix, &(scmd)->device->sdev_gendev, fmt, ##a)
+ sdev_printk(prefix, (scmd)->device, fmt, ##a)
enum scsi_target_state {
STARGET_RUNNING = 1,
-
: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
