Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
To:     bvanassche@acm.org, linux-scsi@vger.kernel.org,
        ming.lei@redhat.com, john.garry@huawei.com, haiyangz@microsoft.com,
        Dexuan Cui <decui@microsoft.com>, mikelley@microsoft.com,
        linux-hyperv@vger.kernel.org, kys@microsoft.com,
        longli@microsoft.com, jejb@linux.ibm.com, sthemmin@microsoft.com,
        wei.liu@kernel.org
Cc:     "Martin K . Petersen" <martin.petersen@oracle.com>,
        linux-kernel@vger.kernel.org, stable@vger.kernel.org
Subject: Re: [PATCH v3] scsi: core: Fix shost->cmd_per_lun calculation in scsi_add_host_with_dma()
Date:   Tue, 12 Oct 2021 16:35:23 -0400
Message-Id: <163407065460.24401.12829276973949007468.b4-ty@oracle.com>
In-Reply-To: <20211008043546.6006-1-decui@microsoft.com>
References: <20211008043546.6006-1-decui@microsoft.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RG1aZThRaVJnWkVGQTc2UEE0bFJaV3g1a24wN2pTZExOR1UzZnNYcXRtOVVa?=
 =?utf-8?B?WWRsNXBjOERWeENOMEdYU3BLZEZyM3c4ZUh1WnZuNFVvSnFsMkdOZUFEcnhG?=
 =?utf-8?B?VFZqVC90bWxqUGs3aVRLOGJLdDdQcWFlVVBMRDZMTFg3bGdCZE0rak5aWkpB?=
 =?utf-8?B?dHNUNmN1RlRKMjd2NEZVemRzdEoxUlJXV3ptTUNONjNyY2lkdUdHNDJFbFVO?=
 =?utf-8?B?RzhXTk9ZbW9FdmlTMkdzTUs0Q0s3MStUUW16WG1qWGNNelM3cmw0aFJHZW90?=
 =?utf-8?B?QmNjS2l2ekR3NmJFdzQrcGdIK0FhNWhzMWlPODg2WHFOcEljTmhpSlB4Z3VW?=
 =?utf-8?B?OGR4bmxxSDBBYlR2SitkZk1hZ1RqYWNoODJORERlcnd6RjRnWHNuL0owOTk3?=
 =?utf-8?B?aVVncWJQVFBaeFZkTVQ4c1lQNk94QlBhVitFQndSbjZkRTZLdEx5VG8zV3ZU?=
 =?utf-8?B?QnpYa040NEd0V0cxYmhkQ2R6SkphaVNzNkw4TUw1OSt2a1pzbmk3NEd3VWlL?=
 =?utf-8?B?eHEzK1BtTDZlUzJJMXA2VFR6Y2l4MG81aTgvZzNXeWc1Mi8xai8rbXRFRERk?=
 =?utf-8?B?eUR1czdsRGFQbjlEa29iK2tZUG91VWcrRkZwNnlOaHpMeFhKaW4xeTM1NTZP?=
 =?utf-8?B?SE9PMDB5Z0JLR2JJVzYxVTZrSkE0RGNFWEtoak5zcnVNZi92eEp6Tys2Vms2?=
 =?utf-8?B?Y2JNdjdUWkxsb0hGOEYwaE9PVnA1YlF1eUMyQzFoRFZvbzRjbGtIQkJleGNQ?=
 =?utf-8?B?d3l2R2RIZ3Y0VzEvTVB2c2ZoVmo2dmFyMDEvcUdqdDM0TmZlOFdHZUhid1pH?=
 =?utf-8?B?YkpMdWZSUG5BcjNKalpKWjZLNEpMOHQweVRsbWxFdWRrQjQ3YjFLOUlQVDJT?=
 =?utf-8?B?YUpuVjZ2NFhFRzZWaUoyNUdNVytJaXFsekdkSmI4bDM2dVFqRHh6WTBSOG44?=
 =?utf-8?B?NXZMMHhSRmxPTFYxS1BVM2RYU3FqTzFiSkluOHRRbGEwelBkcU9NSll0VzN4?=
 =?utf-8?B?NzR6WkM2bGw3RFZYUGZpc0pYczkzVElRN2JXaHAxTkZxWGpxTDBJRUl3Ym9Z?=
 =?utf-8?B?U3BHbjdNOUlnZDVVR3B2YUdOVndlS3pOS2Z6VE0yWGFHYkFEVTJyY2lmaVN4?=
 =?utf-8?B?ZlBKWmxqVVN4b0tPc0FrWFhwa2l4dkFNYng4VVVtSmphSlNxK1FFYkx2bmU2?=
 =?utf-8?B?ZG0yWnYxWmNDRHJNUENvNVl4TDZXRXdDMGMwc2FTUjlybVVYcXVYdDdPMGs3?=
 =?utf-8?B?bEZqWXd4bXhxYmhmNVF4eEs5SERQMVNiem5zNkVnK0k0bGlTdm8zWTExd3JG?=
 =?utf-8?B?S3JGdlAzcmFZWGVvQXg1bWtSZzBIWFJZS3NKZFJqSGR0bkRrcUNERGRWREt0?=
 =?utf-8?B?NU9HVm80dUdzMlRsRmNiQUtvWDh4TTcwRXI2bzNGT1dyT09ucGNlQllJRytM?=
 =?utf-8?B?RnFtSVB6dDlHZ2NOYU1hTFlaK1FYUUdDd1Uxc1NvVkJ1eWpsQ09SdW9TNDY5?=
 =?utf-8?B?SGdoajFvcTViOTNQNGVIS01kTFI1WFU4MFRJaWNOMTlXc0dMN0xLbmN6UkFH?=
 =?utf-8?B?OU5NNVIxemIwcHlvckZ0a2hvRG04Zk9LK21sVTU5WWFOY1pkSVZWd3laRkNt?=
 =?utf-8?B?aDBUNCt3WVpVTkErUEZxTWc5dDhGT3hhMUZybVNwMS9zdUpJdjNOZUFSZXc5?=
 =?utf-8?B?bFozVmd4RUxQQWxYWFNoSnVPQmZ4dk9hUEhUazZCSEVaVUhPYTAyejhtYmYw?=
 =?utf-8?Q?7jpJ5+Hk/cqi24efve1NtoLWIfdbI3Yt0a8bYql?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 35e57796-8dde-4ac0-83ba-08d98dbfd373
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Oct 2021 20:35:27.7888
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: LdDnKjMiCYN2QTgWMiVj8hfoZwF6b/O/yrPcQ/zh6Hz1K5jRJtGkcr2FY5EdV2/PxhHbFQWEOkpz7KSQZZQo4j4zfv8ITWkiQnYYJSNJfHE=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB5514
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10135 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 phishscore=0 bulkscore=0
 malwarescore=0 adultscore=0 mlxscore=0 spamscore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2109230001
 definitions=main-2110120109
X-Proofpoint-GUID: 9liN9mXFRbjQQ8NzL7gghSbCD2fvYmQl
X-Proofpoint-ORIG-GUID: 9liN9mXFRbjQQ8NzL7gghSbCD2fvYmQl
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, 7 Oct 2021 21:35:46 -0700, Dexuan Cui wrote:

> After commit ea2f0f77538c, a 416-CPU VM running on Hyper-V hangs during
> boot because the hv_storvsc driver sets scsi_driver.can_queue to an "int"
> value that exceeds SHRT_MAX, and hence scsi_add_host_with_dma() sets
> shost->cmd_per_lun to a negative "short" value.
> 
> Use min_t(int, ...) to fix the issue.
> 
> [...]

Applied to 5.15/scsi-fixes, thanks!

[1/1] scsi: core: Fix shost->cmd_per_lun calculation in scsi_add_host_with_dma()
      https://git.kernel.org/mkp/scsi/c/50b6cb351636

-- 
Martin K. Petersen	Oracle Linux Engineering