mirror of
https://github.com/FEX-Emu/linux.git
synced 2025-01-14 05:12:17 +00:00
target: remove the unused struct iblock_hba
There is no reason to allocate a struct just to store the host number for a debug printk in the detach path. I've simply removed the verbose debugging given that the calling code thinks the number passed in is something different from a host ID anyway. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
This commit is contained in:
parent
3d26fea01d
commit
6e315a066d
@ -56,51 +56,25 @@ static void iblock_bio_done(struct bio *, int);
|
|||||||
*/
|
*/
|
||||||
static int iblock_attach_hba(struct se_hba *hba, u32 host_id)
|
static int iblock_attach_hba(struct se_hba *hba, u32 host_id)
|
||||||
{
|
{
|
||||||
struct iblock_hba *ib_host;
|
|
||||||
|
|
||||||
ib_host = kzalloc(sizeof(struct iblock_hba), GFP_KERNEL);
|
|
||||||
if (!ib_host) {
|
|
||||||
pr_err("Unable to allocate memory for"
|
|
||||||
" struct iblock_hba\n");
|
|
||||||
return -ENOMEM;
|
|
||||||
}
|
|
||||||
|
|
||||||
ib_host->iblock_host_id = host_id;
|
|
||||||
|
|
||||||
hba->hba_ptr = ib_host;
|
|
||||||
|
|
||||||
pr_debug("CORE_HBA[%d] - TCM iBlock HBA Driver %s on"
|
pr_debug("CORE_HBA[%d] - TCM iBlock HBA Driver %s on"
|
||||||
" Generic Target Core Stack %s\n", hba->hba_id,
|
" Generic Target Core Stack %s\n", hba->hba_id,
|
||||||
IBLOCK_VERSION, TARGET_CORE_MOD_VERSION);
|
IBLOCK_VERSION, TARGET_CORE_MOD_VERSION);
|
||||||
|
|
||||||
pr_debug("CORE_HBA[%d] - Attached iBlock HBA: %u to Generic\n",
|
|
||||||
hba->hba_id, ib_host->iblock_host_id);
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void iblock_detach_hba(struct se_hba *hba)
|
static void iblock_detach_hba(struct se_hba *hba)
|
||||||
{
|
{
|
||||||
struct iblock_hba *ib_host = hba->hba_ptr;
|
|
||||||
|
|
||||||
pr_debug("CORE_HBA[%d] - Detached iBlock HBA: %u from Generic"
|
|
||||||
" Target Core\n", hba->hba_id, ib_host->iblock_host_id);
|
|
||||||
|
|
||||||
kfree(ib_host);
|
|
||||||
hba->hba_ptr = NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void *iblock_allocate_virtdevice(struct se_hba *hba, const char *name)
|
static void *iblock_allocate_virtdevice(struct se_hba *hba, const char *name)
|
||||||
{
|
{
|
||||||
struct iblock_dev *ib_dev = NULL;
|
struct iblock_dev *ib_dev = NULL;
|
||||||
struct iblock_hba *ib_host = hba->hba_ptr;
|
|
||||||
|
|
||||||
ib_dev = kzalloc(sizeof(struct iblock_dev), GFP_KERNEL);
|
ib_dev = kzalloc(sizeof(struct iblock_dev), GFP_KERNEL);
|
||||||
if (!ib_dev) {
|
if (!ib_dev) {
|
||||||
pr_err("Unable to allocate struct iblock_dev\n");
|
pr_err("Unable to allocate struct iblock_dev\n");
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
ib_dev->ibd_host = ib_host;
|
|
||||||
|
|
||||||
pr_debug( "IBLOCK: Allocated ib_dev for %s\n", name);
|
pr_debug( "IBLOCK: Allocated ib_dev for %s\n", name);
|
||||||
|
|
||||||
|
@ -19,11 +19,6 @@ struct iblock_dev {
|
|||||||
u32 ibd_flags;
|
u32 ibd_flags;
|
||||||
struct bio_set *ibd_bio_set;
|
struct bio_set *ibd_bio_set;
|
||||||
struct block_device *ibd_bd;
|
struct block_device *ibd_bd;
|
||||||
struct iblock_hba *ibd_host;
|
|
||||||
} ____cacheline_aligned;
|
|
||||||
|
|
||||||
struct iblock_hba {
|
|
||||||
int iblock_host_id;
|
|
||||||
} ____cacheline_aligned;
|
} ____cacheline_aligned;
|
||||||
|
|
||||||
#endif /* TARGET_CORE_IBLOCK_H */
|
#endif /* TARGET_CORE_IBLOCK_H */
|
||||||
|
Loading…
x
Reference in New Issue
Block a user