Re: [PATCH] scsi: Delete scsi_{get,free}_host_dev()

From: Hannes Reinecke
Date: Sat Jun 26 2021 - 07:55:22 EST


On 6/25/21 6:58 PM, John Garry wrote:
Functions scsi_{get,free}_host_dev() no longer have any in-tree users, so
delete them.

Signed-off-by: John Garry <john.garry@xxxxxxxxxx>
---
An alt agenda of this patch is to get clarification on whether this API
should be used for Hannes' reserved commands series.

Originally the recommendation was to use it, but now it seems to be to
not use it:
https://lore.kernel.org/linux-scsi/55918d68-7385-0153-0bd9-d822d3ce4c21@xxxxxxx/

Please don't.

Before we're doing something like this I would like to have clarification from Christoph which way he prefers for reserved commands. Personally I _do_ like the host dev approach for reserved commands as it allows us to re-use existing infrastructure.

Christoph?

Cheers,

Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare@xxxxxxx +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), Geschäftsführer: Felix Imendörffer