On Thu, 2012-09-20 at 03:39 -0400, Saurav Kashyap wrote:
> From: Andrew Vasquez <andrew.vasq...@qlogic.com>
> 
> Original code incorrectly assigned LOOPID_MAP_SIZE to be the
> allocation size in bytes rather than total bit size.
> Additionally corrected code to check for bit-allocation failure
> in qla2x00_find_new_loop_id().
> 
> Signed-off-by: Andrew Vasquez <andrew.vasq...@qlogic.com>
> Signed-off-by: Saurav Kashyap <saurav.kash...@qlogic.com>
> ---
>  drivers/scsi/qla2xxx/qla_def.h  |    2 +-
>  drivers/scsi/qla2xxx/qla_init.c |    4 ++--
>  drivers/scsi/qla2xxx/qla_os.c   |    3 ++-
>  3 files changed, 5 insertions(+), 4 deletions(-)

This is correcting a bug introduced by

commit 07f2a8c179aa81075a161e6d0c522aa1289d994f
Author: Chad Dupuis <chad.dup...@qlogic.com>
Date:   Wed Aug 22 14:21:00 2012 -0400

    [SCSI] qla2xxx: Use bitmap to store loop_id's for fcports.

Isn't it?  So rather than introducing a bug and later fixing it, we
should just merge the two, since it's not yet upstream, right?

James


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to