* [PATCH V3] Save command pool address of Scsi_Host
@ 2014-08-04 4:26 jgross
2014-08-04 11:08 ` Christoph Hellwig
0 siblings, 1 reply; 3+ messages in thread
From: jgross @ 2014-08-04 4:26 UTC (permalink / raw)
To: James.Bottomley, hch, linux-scsi, linux-kernel; +Cc: Juergen Gross
From: Juergen Gross <jgross@suse.com>
If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
private command pool is needed. scsi_find_host_cmd_pool() will locate it, but
scsi_alloc_host_cmd_pool() isn't saving the pool address in the host template.
This will result in an access error when the host is removed.
Avoid the problem by saving the address of a new allocated command pool where
it is expected and delete it again when the pool is destroyed.
Signed-off-by: Juergen Gross <jgross@suse.com>
---
drivers/scsi/scsi.c | 19 ++++++++++++-------
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/drivers/scsi/scsi.c b/drivers/scsi/scsi.c
index 88d46fe..11ea502 100644
--- a/drivers/scsi/scsi.c
+++ b/drivers/scsi/scsi.c
@@ -351,11 +351,12 @@ scsi_find_host_cmd_pool(struct Scsi_Host *shost)
}
static void
-scsi_free_host_cmd_pool(struct scsi_host_cmd_pool *pool)
+scsi_free_host_cmd_pool(struct scsi_host_cmd_pool **pool)
{
- kfree(pool->sense_name);
- kfree(pool->cmd_name);
- kfree(pool);
+ kfree(pool[0]->sense_name);
+ kfree(pool[0]->cmd_name);
+ kfree(pool[0]);
+ pool[0] = NULL;
}
static struct scsi_host_cmd_pool *
@@ -371,7 +372,7 @@ scsi_alloc_host_cmd_pool(struct Scsi_Host *shost)
pool->cmd_name = kasprintf(GFP_KERNEL, "%s_cmd", hostt->name);
pool->sense_name = kasprintf(GFP_KERNEL, "%s_sense", hostt->name);
if (!pool->cmd_name || !pool->sense_name) {
- scsi_free_host_cmd_pool(pool);
+ scsi_free_host_cmd_pool(&pool);
return NULL;
}
@@ -380,6 +381,10 @@ scsi_alloc_host_cmd_pool(struct Scsi_Host *shost)
pool->slab_flags |= SLAB_CACHE_DMA;
pool->gfp_mask = __GFP_DMA;
}
+
+ if (hostt->cmd_size)
+ hostt->cmd_pool = pool;
+
return pool;
}
@@ -425,7 +430,7 @@ out_free_slab:
kmem_cache_destroy(pool->cmd_slab);
out_free_pool:
if (hostt->cmd_size)
- scsi_free_host_cmd_pool(pool);
+ scsi_free_host_cmd_pool(&hostt->cmd_pool);
goto out;
}
@@ -448,7 +453,7 @@ static void scsi_put_host_cmd_pool(struct Scsi_Host *shost)
kmem_cache_destroy(pool->cmd_slab);
kmem_cache_destroy(pool->sense_slab);
if (hostt->cmd_size)
- scsi_free_host_cmd_pool(pool);
+ scsi_free_host_cmd_pool(&hostt->cmd_pool);
}
mutex_unlock(&host_cmd_pool_mutex);
}
--
1.8.4.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V3] Save command pool address of Scsi_Host
2014-08-04 4:26 [PATCH V3] Save command pool address of Scsi_Host jgross
@ 2014-08-04 11:08 ` Christoph Hellwig
2014-08-04 11:17 ` Juergen Gross
0 siblings, 1 reply; 3+ messages in thread
From: Christoph Hellwig @ 2014-08-04 11:08 UTC (permalink / raw)
To: jgross; +Cc: James.Bottomley, hch, linux-scsi, linux-kernel
On Mon, Aug 04, 2014 at 06:26:09AM +0200, jgross@suse.com wrote:
> From: Juergen Gross <jgross@suse.com>
>
> If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
> private command pool is needed. scsi_find_host_cmd_pool() will locate it, but
> scsi_alloc_host_cmd_pool() isn't saving the pool address in the host template.
>
> This will result in an access error when the host is removed.
>
> Avoid the problem by saving the address of a new allocated command pool where
> it is expected and delete it again when the pool is destroyed.
I don't really like the double pointer passing - just NULLing out the
pointer in the caller where needed seems cleaner.
Otherwise this looks good to me.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V3] Save command pool address of Scsi_Host
2014-08-04 11:08 ` Christoph Hellwig
@ 2014-08-04 11:17 ` Juergen Gross
0 siblings, 0 replies; 3+ messages in thread
From: Juergen Gross @ 2014-08-04 11:17 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: James.Bottomley, linux-scsi, linux-kernel
On 08/04/2014 01:08 PM, Christoph Hellwig wrote:
> On Mon, Aug 04, 2014 at 06:26:09AM +0200, jgross@suse.com wrote:
>> From: Juergen Gross <jgross@suse.com>
>>
>> If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
>> private command pool is needed. scsi_find_host_cmd_pool() will locate it, but
>> scsi_alloc_host_cmd_pool() isn't saving the pool address in the host template.
>>
>> This will result in an access error when the host is removed.
>>
>> Avoid the problem by saving the address of a new allocated command pool where
>> it is expected and delete it again when the pool is destroyed.
>
> I don't really like the double pointer passing - just NULLing out the
> pointer in the caller where needed seems cleaner.
I wanted to avoid to spread this, but I don't mind doing it. V4 is
coming...
Juergen
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-04 11:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-04 4:26 [PATCH V3] Save command pool address of Scsi_Host jgross
2014-08-04 11:08 ` Christoph Hellwig
2014-08-04 11:17 ` Juergen Gross
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.