Re: [PATCH 19/23] advansys: Remove cmd_per_lun setting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday 24 April 2015 13:18:38 Hannes Reinecke wrote:
> Ancient, and pretty much obsolete by now.
>
> Signed-off-by: Hannes Reinecke <hare@xxxxxxx>
> ---
>  drivers/scsi/advansys.c | 18 ------------------
>  1 file changed, 18 deletions(-)
>
> diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c
> index 74e5518..5a55272 100644
> --- a/drivers/scsi/advansys.c
> +++ b/drivers/scsi/advansys.c
> @@ -11212,24 +11212,6 @@ static int advansys_board_found(struct Scsi_Host
> *shost, unsigned int iop, }
>
>  	/*
> -	 * Following v1.3.89, 'cmd_per_lun' is no longer needed
> -	 * and should be set to zero.
> -	 *
> -	 * But because of a bug introduced in v1.3.89 if the driver is
> -	 * compiled as a module and 'cmd_per_lun' is zero, the Mid-Level
> -	 * SCSI function 'allocate_device' will panic. To allow the driver
> -	 * to work as a module in these kernels set 'cmd_per_lun' to 1.
> -	 *
> -	 * Note: This is wrong.  cmd_per_lun should be set to the depth
> -	 * you want on untagged devices always.
> -	 #ifdef MODULE
> -	 */
> -	shost->cmd_per_lun = 1;
> -/* #else
> -            shost->cmd_per_lun = 0;
> -#endif */
> -
> -	/*
>  	 * Set the maximum number of scatter-gather elements the
>  	 * adapter can handle.
>  	 */

This patch breaks my setup: "modprobe advansys" hangs.

It works when all other patches are applied except this one.

-- 
Ondrej Zary
--
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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux