On Tue, Jan 10, 2012 at 3:42 PM, <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> Subject: drivers/scsi/megaraid.c: fix sparse warnings
>
> Fix sparse warnings of right shift bigger than source value size:
>
> drivers/scsi/megaraid.c:311:65: warning: right shift by bigger than source value
> drivers/scsi/megaraid.c:313:65: warning: right shift by bigger than source value
> drivers/scsi/megaraid.c:317:67: warning: right shift by bigger than source value
> drivers/scsi/megaraid.c:319:67: warning: right shift by bigger than source value
>
> Patch suggestion from email by Al Viro:
>
> "Since both are claimed to be strings, I really suspect that this >> 8 is
> misspelled >> 4 and they have a character followed by pair of two-digit
> packed decimals in there..."
>
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> Cc: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
> Cc: Neela Syam Kolli <megaraidlinux@xxxxxxx>
> Cc: James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
>
> drivers/scsi/megaraid.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff -puN drivers/scsi/megaraid.c~drivers-scsi-megaraidc-fix-sparse-warnings drivers/scsi/megaraid.c
> --- a/drivers/scsi/megaraid.c~drivers-scsi-megaraidc-fix-sparse-warnings
> +++ a/drivers/scsi/megaraid.c
> @@ -310,15 +310,15 @@ mega_query_adapter(adapter_t *adapter)
> if (adapter->product_info.subsysvid == HP_SUBSYS_VID) {
> sprintf (adapter->fw_version, "%c%d%d.%d%d",
> adapter->product_info.fw_version[2],
> - adapter->product_info.fw_version[1] >> 8,
> + adapter->product_info.fw_version[1] >> 4,
> adapter->product_info.fw_version[1] & 0x0f,
> - adapter->product_info.fw_version[0] >> 8,
> + adapter->product_info.fw_version[0] >> 4,
> adapter->product_info.fw_version[0] & 0x0f);
> sprintf (adapter->bios_version, "%c%d%d.%d%d",
> adapter->product_info.bios_version[2],
> - adapter->product_info.bios_version[1] >> 8,
> + adapter->product_info.bios_version[1] >> 4,
> adapter->product_info.bios_version[1] & 0x0f,
> - adapter->product_info.bios_version[0] >> 8,
> + adapter->product_info.bios_version[0] >> 4,
> adapter->product_info.bios_version[0] & 0x0f);
> } else {
> memcpy(adapter->fw_version,
After a huge hunt, I was able to locate a single parallel scsi board
that this driver works with, however it has LSI firmware on it. I am
unable to find the HP firmware for my board, which this code change
affects. Additionally, checking hp.com for firmware for my controller
was unsuccessful.
Changing the driver version string for HP firmware on this board in
_any way_ other than what was intended will affect the following code
in the driver that does firmware string matching:
if ((subsysvid == HP_SUBSYS_VID) &&
((subsysid == 0x60E7) || (subsysid == 0x60E8))) {
/*
* which firmware
*/
if (!strcmp(adapter->fw_version, "H01.07") ||
!strcmp(adapter->fw_version, "H01.08") ||
!strcmp(adapter->fw_version, "H01.09") ) {
printk(KERN_WARNING
"megaraid: Firmware H.01.07, "
"H.01.08, and H.01.09 on 1M/2M "
"controllers\n"
"megaraid: do not support 64 bit "
"addressing.\nmegaraid: DISABLING "
"64 bit support.\n");
adapter->flag &= ~BOARD_64BIT;
}
}
Since I have no HP firmware on one of these boards, and I can see that
altering the firmware string will alter the behvior of the above code
WRT setting/clearing BOARD_64BIT in the adapter->flag field, I have to
NACK this patch. I hope we can live with the sparse warning since I
have no way to test this.
-Adam
--
To unsubscribe from this list: 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
