From:   John Garry <john.g.garry@oracle.com>
To:     jejb@linux.ibm.com, martin.petersen@oracle.com
Cc:     linux-scsi@vger.kernel.org, bvanassche@acm.org,
        linux-kernel@vger.kernel.org, dgilbert@interlog.com,
        John Garry <john.g.garry@oracle.com>
Subject: [PATCH RESEND v2 00/11] scsi_debug: Some minor improvements
Date:   Mon, 13 Mar 2023 09:31:03 +0000
Message-Id: <20230313093114.1498305-1-john.g.garry@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?/WnfQXnr7UwUdz0HDFz1CBQNm/defix5DHQ54soXGP1AZ5lMFfhTA+QzW4zV?=
 =?us-ascii?Q?4/Z9FLBJy35+LZ5glYMnkXD3ZnWzCGWEdpKa5YbSQ1abpORTCjzbYNKiA14E?=
 =?us-ascii?Q?lDBp/pz41NpVOaD3jHQQzx5nvnuH0oegapCmGx2KiBjis2MPWlJBQmZXy6mx?=
 =?us-ascii?Q?PXybLI0oI0LecgQqqrlYO2aaFat5cI88LSJMC62tsyIrNrb71m+tEYuVqjoB?=
 =?us-ascii?Q?obBdLKrTdHkYvLEmr25lcFbES0vHpukEQA+weX647J/ljZZNMrKrA2pPjcGd?=
 =?us-ascii?Q?NkHnWesD8dqrYcshaldKRC2IajxI5+ACaO6KG13VFEQvs7vfQO/lZIVuZvAu?=
 =?us-ascii?Q?hx2YAafjT8wvUzFdvN1tO020SwlXFjjHTXTdXU/vIt1vL5DHEaX543ydXb/Y?=
 =?us-ascii?Q?nIooB5Ck3QHclziToAEYC7I66+Pt9uGJupNeS+ulvDolfpF2nLsfhL/PMk9H?=
 =?us-ascii?Q?j0X+Hdx++V5lVtxeOiJQevXCohH1NYXNOB7fdWR49YEfjGWNhcU16BBZNe0p?=
 =?us-ascii?Q?A9IE6FdXnNHci+B8kyBEdrSZTfvvPvhkM1lgEhe8I3RR2ocwiNwrGrljqIqO?=
 =?us-ascii?Q?VrBYfbWAzyRZqF4gCXNrqxfPRu3AGisZ6w+XlZPCnbOB8P4QRxhLwFo6aVr+?=
 =?us-ascii?Q?MIGnqe8qw25XmeSQVHtcV/80rI1Qj5bllPss42M8T4jzdbXCF8qIhupTuZ6g?=
 =?us-ascii?Q?rHZ8almrmsYpIKoo2SzfKiP7A/RdxenmLtLfHvqzzobvaXnjBDdYaLXMQnhi?=
 =?us-ascii?Q?0NXyjWQSYOiX2j5R2mO8cIdO/A+OUMgOP8jbmUk6LFRp/RrP9E4pJUkDTAH7?=
 =?us-ascii?Q?e6MGBSoiu14nkDJ9ghmlnN2uYuLZVA0RO6ru8kK96iCZ/rwulTWi+p7U6U97?=
 =?us-ascii?Q?V00jO1MVveiJ8MvxE/UEmFbYvxZeDCKkFuBaqyFVOQTdD7vzhmjTTTIcAm0G?=
 =?us-ascii?Q?Mjm+ERSg+IPmTm5q+2YFEYW7IHU5yi+h5qVRujgLjbc18hHjkV8D3hDj3o8z?=
 =?us-ascii?Q?kYJwjlX7TWS1rh4q2UgTL40xzBtFkgzsa8melIS62Nf6KqDKbSVBALtf6kLm?=
 =?us-ascii?Q?fyZyzoBzfBJ0VqcKvE3EhHalfAbczTBwDTKdCcJ0IyHC2C8KZvS1NYGHU5zv?=
 =?us-ascii?Q?8u8MmchvT+BRxgnkxR/kOAdMyPkCuHJfj/EblR9UhHCUBMI08xyvTAHx5sRR?=
 =?us-ascii?Q?43CICYiauXhTfhgq9gzO2t3yxExc8zo72WJKGxPqNI6pR+ZRDGSUCYhssaM9?=
 =?us-ascii?Q?9ZYCr0bNVCiGJ/3Wliu/bDcXVT6CojBrTFh9F2KYN2TrZutOFTh1fzUnF62j?=
 =?us-ascii?Q?+BUBQlSA5ytDmPltjruQJQMJGrjZ6oVi8XeXa5AQK21UgcMfM/1IbDtWw88f?=
 =?us-ascii?Q?Z/whOLBIQ1wcb4rNBprwbpMitA/fqNOiZEuuZ0g2nXHf0OgRS+YLlFbHsMIF?=
 =?us-ascii?Q?XMk5MHPARp0QkTIQ+vSk0Hmg/M/osCTcknUtAzYSfI1wEGO8Ux0t8qV2Ufic?=
 =?us-ascii?Q?u3j+hxl6lfytTKjOzrellgSPgpFGjIV3QU44U1k/43sst0Tof4lBsIMmeEGN?=
 =?us-ascii?Q?gbTPERNDmHZEs0ITA9DHu/1q5vacRhkWM4DeC4ucCRKAak5UOGoJ1x5PD+r9?=
 =?us-ascii?Q?ZQ=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 6YwD1KC9kFmmLPhfQ9zWVELbXzRiSZZiWSbquehfrBKoVV7b3IAVGbr5b0X7HPhtnH28MrEnC5XkDuBUNICP0jSayybY2FGvzPJP0Qd+xYwY+IeKzZtHAuLhVIIzSfLNsQYCJiz7qQfop/+zAQVQJDcys1d7L2sugktzslFtYX2pjEF8YDyp+kh1VQceplB2FP5kolYsU/ZC+w1mf5Q8JHYaQ+t2Q0jqzbZHXkweO+8Py1QneV/P/OZ5winwMGlPk4SySXpL17OBjZhprmBXRPRTIATF6/+dpUc8WdWiGMALZzZeyjhbwekMN553i6YuUDQkaVg/XzwWsCMDC9oP16eUHG56Xd/L+TpaaytH6s1B7V58r9Y6vxQK/eJ6jXHReeiol0aeiZyf0q4XQQKs0lyfSkPlmcbkeFNVv/lN8wT87qSAGFtI29S54iObG60kNfo8hCj/aX3XfysedRvdmFZN/5ZELlbCCDtx8/hKjRu1Sap+rb/wUReL0283/9olYt//uQ0giVrXfSppzrBHZrL6Y77AT4FmQFMSDjB0BvmwxfkdmDG4pPSSSYAMOWHebA3Emj94dPeU1S+4Yr3xJAnv0DnNWw7vLWk0yn7i4WUkPW/7NyD5wcTOXI21oox0IGNO4RDBqdZV04AFWy9u/qr+dHoa+7HP6mj5Lu2qo+le4lJysi8lK+cz4zZ5wsG0cWOP6BgK+Xt1xVPaTpvOJIay+YQw4MW8a6VAQs2ASBbvOXgZYXlOm5dKr1l74h7iNp8V1xo4HY1Q+gX5LRkr3vsV4wjcny1eoHzm/aPfyKrE9c/1kuDRCRO9V1/Kh8Bu4+rE+ZoZlUnu1XT+dPIoAY3rup7+knuBgLpsnjCjz1Q=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: aea00997-a6ad-480f-7711-08db23a5b823
X-MS-Exchange-CrossTenant-AuthSource: DM6PR10MB4313.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Mar 2023 09:31:26.9902
 (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: LJjkpYanIMvfVpXYnrxGg9i67dhXje4+k/HniKPXixX8m2o1w6RYkF/HofqrWrSma/M3STQL6smGbbpdxcgz5g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR10MB7048
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22
 definitions=2023-03-13_02,2023-03-10_01,2023-02-09_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 adultscore=0 malwarescore=0
 mlxlogscore=999 mlxscore=0 suspectscore=0 bulkscore=0 phishscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000
 definitions=main-2303130077
X-Proofpoint-GUID: Z7imLzIrBjI-LOW6LAH0KMqGQttTTJE6
X-Proofpoint-ORIG-GUID: Z7imLzIrBjI-LOW6LAH0KMqGQttTTJE6
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

This series contains a bunch of minor improvements to the driver. I have
another bunch waiting with more major changes.

Most of the changes are quite straightforward, and the only patches of note are
as follows:
- Fix the command abort feature, enabled with host option SDEBUG_OPT_CMD_ABORT
- Drop driver count of queued commands per device
- Add poll mode completions to statistics. We already have poll mode callback
  call count, so maybe it was intentional to omit poll mode from the
  statistics.
  
Difference to v1:
- rename function to get sdebug host from shost (Bart)

Based on scsi-staging 6.4 @ commit 0b31b77f281a ("Merge patch series "PCI/AER: ...")

John Garry (11):
  scsi: scsi_debug: Don't hold driver host struct pointer in
    host->hostdata[]
  scsi: scsi_debug: Stop setting devip->sdbg_host twice
  scsi: scsi_debug: Drop scsi_debug_abort() NULL pointer checks
  scsi: scsi_debug: Drop scsi_debug_device_reset() NULL pointer checks
  scsi: scsi_debug: Drop scsi_debug_target_reset() NULL pointer checks
  scsi: scsi_debug: Drop scsi_debug_bus_reset() NULL pointer checks
  scsi: scsi_debug: Drop scsi_debug_host_reset() device NULL pointer
    check
  scsi: scsi_debug: Drop check for num_in_q exceeding queue depth
  scsi: scsi_debug: Drop sdebug_dev_info.num_in_q
  scsi: scsi_debug: Get command abort feature working again
  scsi: scsi_debug: Add poll mode deferred completions to statistics

 drivers/scsi/scsi_debug.c | 209 ++++++++++++++------------------------
 1 file changed, 76 insertions(+), 133 deletions(-)

-- 
2.35.3