[PATCH] Revert "scsi: core: Cap scsi_host cmd_per_lun at can_queue"

[Tim Gardner]

On 10/8/21 9:09 AM, Thadeu Lima de Souza Cascardo wrote:
> On Fri, Oct 08, 2021 at 08:53:51AM -0600, Tim Gardner wrote:
>> BugLink: https://bugs.launchpad.net/bugs/1940564
>>
>> This reverts commit ce0da9f6062f460c27a7fe6e566c330991e757d8.
>> (upstream commit dcb634e434b850287b31a117637cbf598563ce1c)
> 
> This is really upstream commit ea2f0f77538c50739b9fb4de4700cee5535e1f77.
> 

Of course it is. I was mistakenly looking at a stable branch. I'll fix 
it when I apply.

> Cascardo.
> 
>>
>> This patch causes boot failures on instances with large numbers
>> of CPUs and SCSI LUNs, the product of which overflows a 16 bit
>> quantity. To date, that combination exists in Azure instance
>> types Standard_M416s_v2 and Azure Standard_D48_v3.
>>
>> Signed-off-by: Tim Gardner <tim.gardner at canonical.com>
>> ---
>>   drivers/scsi/hosts.c | 3 ---
>>   1 file changed, 3 deletions(-)
>>
>> diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c
>> index da3920a19d53d..bd0dcb540f822 100644
>> --- a/drivers/scsi/hosts.c
>> +++ b/drivers/scsi/hosts.c
>> @@ -220,9 +220,6 @@ int scsi_add_host_with_dma(struct Scsi_Host *shost, struct device *dev,
>>   		goto fail;
>>   	}
>>   
>> -	shost->cmd_per_lun = min_t(short, shost->cmd_per_lun,
>> -				   shost->can_queue);
>> -
>>   	error = scsi_init_sense_cache(shost);
>>   	if (error)
>>   		goto fail;
>> -- 
>> 2.33.0
>>
>>
>> -- 
>> kernel-team mailing list
>> kernel-team at lists.ubuntu.com
>> https://lists.ubuntu.com/mailman/listinfo/kernel-team

-- 
-----------
Tim Gardner
Canonical, Inc