Re: [PATCH 2/3] Better log messages for PQ3 devs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Mar 29, 2006 at 10:12:47PM +0200, Kurt Garloff wrote:
>  		 */
> -		SCSI_LOG_SCAN_BUS(3, printk(KERN_INFO
> -					"scsi scan: peripheral qualifier of 3,"
> -					" no device added\n"));
> +		SCSI_LOG_SCAN_BUS(2, printk(KERN_INFO "scsi scan: periph.qual. of 3,"
> +				" device %s not added\n",
> +				sdev->sdev_gendev.bus_id));

You seem to be suffering from long-line-itis, probably as a result of
too much exposure to ia64 code.  How about:

		SCSI_LOG_SCAN_BUS(2, printk(KERN_INFO "scsi scan: peripheral "
				"qualifier of 3, device %s not added\n",
				sdev->sdev_gendev.bus_id));

> +		if (lun == 0) {
> +			unsigned char vend[9], mod[17];
> +			SCSI_LOG_SCAN_BUS(1, printk(KERN_INFO "scsi scan: consider passing "
> +					"scsi_mod.dev_flags=%s:%s:0x240 or 0x800240\n",


Likewise:

			SCSI_LOG_SCAN_BUS(1, printk(KERN_INFO "scsi scan: "
					"consider passing scsi_mod.dev_flags="
					"%s:%s:0x240 or 0x800240\n",

> +					scsi_inq_str(vend, result, 8, 16),
> +					scsi_inq_str(mod, result, 16, 32)));
> +		}
> +		
>  		res = SCSI_SCAN_TARGET_PRESENT;
>  		goto out_free_result;
>  	}
>  
> -- 
> Kurt Garloff, Head Architect Linux R&D, Novell Inc.


-
: 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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux