diff options
author | vignesh babu <vignesh.babu@wipro.com> | 2007-12-13 13:43:41 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-01-11 19:28:51 -0500 |
commit | 118365721768f29d74718a59895ed7bd3c10b68e (patch) | |
tree | 3bd814b509c6b9d1abaf7185d17588b88f208775 | |
parent | 4545a88fc1ae2c0cd1da6e35f3adcbc56ff27b07 (diff) |
[SCSI] libiscsi: use is_power_of_2
Replacing n & (n - 1) for power of 2 check by is_power_of_2(n)
Signed-off-by: vignesh babu <vignesh.babu@wipro.com>
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
-rw-r--r-- | drivers/scsi/libiscsi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c index 6573223cb346..553168ae44f1 100644 --- a/drivers/scsi/libiscsi.c +++ b/drivers/scsi/libiscsi.c | |||
@@ -24,6 +24,7 @@ | |||
24 | #include <linux/types.h> | 24 | #include <linux/types.h> |
25 | #include <linux/kfifo.h> | 25 | #include <linux/kfifo.h> |
26 | #include <linux/delay.h> | 26 | #include <linux/delay.h> |
27 | #include <linux/log2.h> | ||
27 | #include <asm/unaligned.h> | 28 | #include <asm/unaligned.h> |
28 | #include <net/tcp.h> | 29 | #include <net/tcp.h> |
29 | #include <scsi/scsi_cmnd.h> | 30 | #include <scsi/scsi_cmnd.h> |
@@ -1700,7 +1701,7 @@ iscsi_session_setup(struct iscsi_transport *iscsit, | |||
1700 | qdepth = ISCSI_DEF_CMD_PER_LUN; | 1701 | qdepth = ISCSI_DEF_CMD_PER_LUN; |
1701 | } | 1702 | } |
1702 | 1703 | ||
1703 | if (cmds_max < 2 || (cmds_max & (cmds_max - 1)) || | 1704 | if (!is_power_of_2(cmds_max) || |
1704 | cmds_max >= ISCSI_MGMT_ITT_OFFSET) { | 1705 | cmds_max >= ISCSI_MGMT_ITT_OFFSET) { |
1705 | if (cmds_max != 0) | 1706 | if (cmds_max != 0) |
1706 | printk(KERN_ERR "iscsi: invalid can_queue of %d. " | 1707 | printk(KERN_ERR "iscsi: invalid can_queue of %d. " |