On 4/21/22 10:03 AM, Dan Carpenter wrote:
> These flags are supposed to be bit numbers. Right now they cause a
> double shift bug where we use BIT(BIT(2)) instead of BIT(2).
> Fortunately, the bit numbers are small and it's done consistently so it
> does not cause an issue at run time.
>
> Fixes: 5bd856256f8c ("scsi: iscsi: Merge suspend fields")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> ---
> include/scsi/libiscsi.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/scsi/libiscsi.h b/include/scsi/libiscsi.h
> index d0a24779c52d..c0703cd20a99 100644
> --- a/include/scsi/libiscsi.h
> +++ b/include/scsi/libiscsi.h
> @@ -54,9 +54,9 @@ enum {
> #define ISID_SIZE 6
>
> /* Connection flags */
> -#define ISCSI_CONN_FLAG_SUSPEND_TX BIT(0)
> -#define ISCSI_CONN_FLAG_SUSPEND_RX BIT(1)
> -#define ISCSI_CONN_FLAG_BOUND BIT(2)
> +#define ISCSI_CONN_FLAG_SUSPEND_TX 0
> +#define ISCSI_CONN_FLAG_SUSPEND_RX 1
> +#define ISCSI_CONN_FLAG_BOUND 2
>
> #define ISCSI_ITT_MASK 0x1fff
> #define ISCSI_TOTAL_CMDS_MAX 4096
Thanks.
Reviewed-by: Mike Christie <michael.christie@xxxxxxxxxx>
