Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <347d0e04-65d1-b735-75eb-e8358c91c08a@oracle.com>
Date:   Mon, 21 Nov 2022 13:48:08 +0000
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.5.0
Subject: Re: [External] : Re: [PATCH 2/2] scsi: core: Use SCSI_SCAN_INITIAL in
 do_scsi_scan_host()
To:     Damien Le Moal <damien.lemoal@opensource.wdc.com>,
        martin.petersen@oracle.com, jejb@linux.ibm.com
Cc:     linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20221121121725.1910795-1-john.g.garry@oracle.com>
 <20221121121725.1910795-3-john.g.garry@oracle.com>
 <83c29cc2-9bf4-265f-4f8a-ab83d8b6271b@opensource.wdc.com>
Content-Language: en-US
From:   John Garry <john.g.garry@oracle.com>
Organization: Oracle Corporation
In-Reply-To: <83c29cc2-9bf4-265f-4f8a-ab83d8b6271b@opensource.wdc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cHZpTHFTNEVaZ3JIZGFZeVFKOTF1RG41RFF3bldIQUJHbEFhWG9OMCtTVHpY?=
 =?utf-8?B?NWFtWmI2eUlYMDBDRVF3QkxKVVphWkExMnFWL1R5UUZaakh5dmxqOXlVQVFo?=
 =?utf-8?B?Nk5xbnlBTDRYNEtNdEh1cTRCOFF2OUoyajNPOTdPSTJqdnBDTlliUHdGNGZr?=
 =?utf-8?B?OEdVT3RjcXF6ZDZxcnNPM0Vzb3ZaNGc5aVZvanhYQWpXQzVHQ3dsaXo4ZHhj?=
 =?utf-8?B?VTZSL1FBR1Y3QTYvaTk1VUdmS3VmdzN1dTVGMFFpZG5ramxtRGhRYkZYNlNC?=
 =?utf-8?B?VktSQXZPT2xGQU1oNjJydmdnK2VXMjJsb3ZWaFhRbUVQWjFqWU5yWXUzclp3?=
 =?utf-8?B?SFBudUFET29RNGpRSTZqeEsyZk1welRUMk4wYmlxTnlPZk9uV2M2aklRTjJ0?=
 =?utf-8?B?dTNsOFJnZnpPTU5pS0Y1eWFNZzQreE95cEp6THVibmd5SkRMa1VMV0diN1No?=
 =?utf-8?B?YlZEcWUvYkZEcFBxQ2VVQ1lqMlE4c0JxVkJRYm1tRGlmdDZQdGpwMU1abnRE?=
 =?utf-8?B?d1V4emQ2OThWeklmcisxVGcrUmtEVHNTb05KZWxPNWh2dWs3bzNjWlREdHo0?=
 =?utf-8?B?NTBvVzJzWE5SQWxIYW9pbnlPMHVhNWgyU085MFUwWE5nMnROdU5zLzFOUHVB?=
 =?utf-8?B?akMxSmxhRGVzajBhc1NReFV1UFRxVDRhc3ZoWWFaMTlTY21ZeEhKcU5nVDJU?=
 =?utf-8?B?MnhzNnZaWk1Dc01Lbk9pZVNrNDdNTndycENRajNrRHVrVVhYaVE4UWVGbWZU?=
 =?utf-8?B?VTZ0bDJldHVzRml5QzJzbXA5Mm41bGRXSFF1NXUrS0VOTG9OdDd5bFloZEFy?=
 =?utf-8?B?aE9RZ1dWSTduVlJkV1FYU2VqMFNoT09kTEFDeWRmWW5UcHUyaUY0eXlsMHVE?=
 =?utf-8?B?alMybDNrOE44blNwSkErcGlFZjZBeG5QV1EwTDgvWTBSaStSeUgzOG1UM3I4?=
 =?utf-8?B?ME1KZEg3c3FKWEhpQ0duZkQ4OXR3d0Vjak52OFRybWN2amJQT0c0dzJ0RWxl?=
 =?utf-8?B?YStydGxlTitjbU0zSWlybVV3Q2RjaXhjNmg3OXFPVTNLemRGcjdFUkZiTFpX?=
 =?utf-8?B?bi9tNml5c1ZoZWo5enZXWm1jWmt3UStUR1FVV05OS0Z5dWo2WXVvK09jckhn?=
 =?utf-8?B?dGwxL3BxMWdueWV1YzVtQTltazhtSkhoT0ViT1d4d3RVOEFUYnlzcVNOVGtG?=
 =?utf-8?B?L1lOckVmaVBJUzl1aWs4bVMzc1ZlNXBpYWZmUFV2Yi8zUDVaOUVNRlJnYnB0?=
 =?utf-8?B?bDJFcVA4aVp2SHU5b1Z5VFlTV0tzYXdabitkSWdKOWptdGM2MXpHZ003aGhW?=
 =?utf-8?B?VEpaWGRVbnZITHRyOTk2WXBHYUJqbUVDNTZ5TmVKZkFybG5Jc3hINVErQTd4?=
 =?utf-8?B?aWZIQUFtQThzakNEaEVOOEFocWgwWFlKbUhndlZ4aVo3Wnd4THgwdmYzeVVI?=
 =?utf-8?B?S2NzRU9oamVTcExCLzNFa21VdHVuZDV2L1J2bi8zeWhwV0dkenQxODRrSVJw?=
 =?utf-8?B?NWphdThhNGtZQ0pua0xUbFJXdFlqSy9TS1FZTitReGpBbWpVdVM4QnhNTkRZ?=
 =?utf-8?B?NnFWaHlVMmN4RFFJWCtUd1ZSYy9lQ2w2ek4wYytrMHNuOEk5YjFHS1ZvWDNM?=
 =?utf-8?B?M3pmcjNoak52a1dPdDNlRG96bzBzSXU5aDRDaXgvSlNyRURacHgxWU53TFhz?=
 =?utf-8?B?QmlUY3lJVUV4VDVPa1ZEV3phUnQyUGsyZ3cxOFBHQ0U4NDVzVHk0OHY5d2Rx?=
 =?utf-8?B?TUo1d1U2My9tc2xYeFZubnJ6VWV0RnIra2hTV0dCUEJVVkZ5b1QzZkhCanAy?=
 =?utf-8?B?ZElFNENKS01OVkc1WXMyR05MVG5UWkpHRDk0RTFzQTVwVGZKajEyMUZlcGNl?=
 =?utf-8?B?Y3kyNjNVcGs0RVk3RHorSDJBeCtvWGFaalp3dnptWU4wdHk0OTZjQXpSSWtE?=
 =?utf-8?B?STJKSTZwNU5aM0hkNVZTZ0htNCtwUktoaHJyY2gyaTVhMHpTR3pOV0h1MHpM?=
 =?utf-8?B?TUJzaVcwNm1iS0hDK3BzLzZRY0RzaXdlNzlyRW01RDY4SWpuSXUydkdsK0Zo?=
 =?utf-8?B?NkJhQ3hQa21GSGcwWjVXNnV5YVBxeXJBREFlV3ZJUDFJT1crSjJBa2N5WTk2?=
 =?utf-8?B?dk8rQ1ZIMUpQOHAzWlNYSkh3eGhkUDdCZFB6cTdYL1JtcUtVM01oSGFBSlJM?=
 =?utf-8?B?ZGc9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 7EyCY0BMtPB5ahfwo/YcVf9scCNHHbknYZ8hDYSKTuB9jG+ZZ6qbnfqSI7pm5+3dglVrFdlOBM4aX4ru+Id8XamAOlOxv7L7lL0Ahm5r2P1GdgJAByLm5V8pGGBb1w4hNU7QFpm9hHr2Rfye7BRCDpDToLpc7fm2rtUWtZlfCwOQOSkysCvWopkZ7WezoOpERd0VNZBmsSEcyJ3N4ebNNTU5/FuOhjkkneE72ct6QemxJsnpezvNL5/MfOwoLSfm4Lva/QwbSrUkNlZZsL1jZAPpKJtMhzktXwQ7DcEiHli5PtR0owRhhxZryBkE/LN8Bds8JBJTP8koewy2kZxQIp2r4NdN04Y0uYg9sJRWkUC4/qWwmB7RVtES3awrAv2WfwfsyjvmHDBor8OAuOSDUGL6OzuELqZc1xgjoqxQ/PimJh5EuwV03HmIsshGhjq2zZYnpgOTf0vG766E20yU+bmg8LEUFGUwdYNkHcsJfOZw4SmtaL2J5PSgP7YF/NF02Rmax84/j9uFnfsu0TKAjGPRx40zpk+6L3EP34BDJf58Kmb5sssUCBfD+Lsnpg4nevq1JFzsY1/8AcBRKj8wKuuCNzN6pdMvJdL+RCqbrUch3HuBkqs6a81OBv+ZDutOsFwjcqcca+DHw/vt7cLe7tPWYRT3jrJ1nSWnvQpCQYZ/3LNgapDlg1pKd7TWuBdyjoKAbQeCCRqYHyxnTueIsUWbhHL6zifqw0da1CpCi+8beHaFUlorheR3nnwsxjrapZuDBbajmub7qHQij/ifM2tURWloR8KIhj/yAo2qbyqsExp71qEBRcT99B6TAkKDAMBRUlDUJLGK6nKKEuX/RBhb54q5pILYa0rBMi8UpANW1B2hjw/YoHvQZN+tI8QayKADAbj8WkgWUpKcIduolw==
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2884f8b9-804a-4cf0-91a0-08dacbc70a16
X-MS-Exchange-CrossTenant-AuthSource: DM6PR10MB4313.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Nov 2022 13:48:16.9350
 (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: pgSGBan3Ku5RYPvAqmXnxLyAYvpfnnCbi2co1OP2o0awOnkK8JTFoRhYXdgTL5Awqr2gnYIS7mKZUasbT1s9Qw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR10MB6542
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2022-11-21_13,2022-11-18_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 spamscore=0 malwarescore=0
 phishscore=0 mlxlogscore=999 bulkscore=0 suspectscore=0 adultscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000
 definitions=main-2211210108
X-Proofpoint-ORIG-GUID: xr13xj-0ibmWTfddJOccKvofvUyiMV4X
X-Proofpoint-GUID: xr13xj-0ibmWTfddJOccKvofvUyiMV4X
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 21/11/2022 12:33, Damien Le Moal wrote:
> On 11/21/22 21:17, John Garry wrote:
>> Instead of using hardcoded '0' as the do_scsi_scan_host() ->
>> scsi_scan_host_selected() rescan arg, use proper macro SCSI_SCAN_INITIAL.
>>
>> Signed-off-by: John Garry<john.g.garry@oracle.com>
> Looks good.
> 
> Reviewed-by: Damien Le Moal<damien.lemoal@opensource.wdc.com>
> 

cheers. As mentioned, I can combine these patches if people prefer.

John