diff options
author | Tomas Henzl <thenzl@redhat.com> | 2012-07-03 04:08:41 -0400 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2013-01-28 22:15:34 -0500 |
commit | d4a0d607e22013446a63e024a80e9106b3e30407 (patch) | |
tree | 0ceb000dc5a8b3dcb2fcf78c4afa3d3fca88fa0e /drivers/scsi/bnx2i | |
parent | c0773b7cd5b1894d5a77dbe9b0c9a96d481fe485 (diff) |
[SCSI] bnx2i: fix the bit manipulation when setting the error mask
The intention in bnx2i_send_fw_iscsi_init_msg was to zero out
only the lower 32bits, but instead the whole mask64 is zeroed.
This patch fixes it.
Signed-off-by: Tomas Henzl <thenzl@redhat.com>
Acked-by: Eddie Wai <eddie.wai@broadcom.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/bnx2i')
-rw-r--r-- | drivers/scsi/bnx2i/bnx2i_hwi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/bnx2i/bnx2i_hwi.c b/drivers/scsi/bnx2i/bnx2i_hwi.c index 91eec60252ee..a28b03e5a5f6 100644 --- a/drivers/scsi/bnx2i/bnx2i_hwi.c +++ b/drivers/scsi/bnx2i/bnx2i_hwi.c | |||
@@ -1317,7 +1317,7 @@ int bnx2i_send_fw_iscsi_init_msg(struct bnx2i_hba *hba) | |||
1317 | (1ULL << ISCSI_KCQE_COMPLETION_STATUS_PROTOCOL_ERR_LUN)); | 1317 | (1ULL << ISCSI_KCQE_COMPLETION_STATUS_PROTOCOL_ERR_LUN)); |
1318 | if (error_mask1) { | 1318 | if (error_mask1) { |
1319 | iscsi_init2.error_bit_map[0] = error_mask1; | 1319 | iscsi_init2.error_bit_map[0] = error_mask1; |
1320 | mask64 &= (u32)(~mask64); | 1320 | mask64 ^= (u32)(mask64); |
1321 | mask64 |= error_mask1; | 1321 | mask64 |= error_mask1; |
1322 | } else | 1322 | } else |
1323 | iscsi_init2.error_bit_map[0] = (u32) mask64; | 1323 | iscsi_init2.error_bit_map[0] = (u32) mask64; |