2019-01-15 04:43:10

by Marcos Paulo de Souza

[permalink] [raw]
Subject: [PATCH] block/blk-sysfs.c: Remove last reference of blk_init_queue

blk_init_queue was removed in a1ce35fa4985.

Signed-off-by: Marcos Paulo de Souza <[email protected]>
---
There are more two references in Documentation/block/biodoc.txt, but maybe that
file needs a rewrite in rst anyway?

block/blk-sysfs.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/block/blk-sysfs.c b/block/blk-sysfs.c
index 590d1ef2f961..dd9b50226c6e 100644
--- a/block/blk-sysfs.c
+++ b/block/blk-sysfs.c
@@ -821,10 +821,9 @@ static void blk_free_queue_rcu(struct rcu_head *rcu_head)
* @work: pointer to the release_work member of the request queue to be released
*
* Description:
- * blk_release_queue is the counterpart of blk_init_queue(). It should be
- * called when a request queue is being released; typically when a block
- * device is being de-registered. Its primary task it to free the queue
- * itself.
+ * Should be called when a request queue is being released;
+ * typically when a block device is being de-registered. Its primary task it
+ * to free the queue itself.
*
* Notes:
* The low level driver must have finished any outstanding requests first
--
2.16.4



2019-01-15 04:25:34

by Jens Axboe

[permalink] [raw]
Subject: Re: [PATCH] block/blk-sysfs.c: Remove last reference of blk_init_queue

On 1/14/19 8:06 PM, Marcos Paulo de Souza wrote:
> blk_init_queue was removed in a1ce35fa4985.

Honestly, most of that comment is wrong anyway. Since this isn't
a visible API, I'd just kill the comment completely.

> Signed-off-by: Marcos Paulo de Souza <[email protected]>
> ---
> There are more two references in Documentation/block/biodoc.txt, but maybe that
> file needs a rewrite in rst anyway?

It probably needs a rewrite - period :-)

--
Jens Axboe


2019-01-15 12:02:29

by Marcos Paulo de Souza

[permalink] [raw]
Subject: Re: [PATCH] block/blk-sysfs.c: Remove last reference of blk_init_queue

HI Jeans,

On 1/15/19 1:54 AM, Jens Axboe wrote:
> On 1/14/19 8:06 PM, Marcos Paulo de Souza wrote:
>> blk_init_queue was removed in a1ce35fa4985.
>
> Honestly, most of that comment is wrong anyway. Since this isn't
> a visible API, I'd just kill the comment completely.

I will send a v2 trying to describe better what the function does and
where it is used (or removed it completely as you suggested).

>
>> Signed-off-by: Marcos Paulo de Souza <[email protected]>
>> ---
>> There are more two references in Documentation/block/biodoc.txt, but maybe that
>> file needs a rewrite in rst anyway?
>
> It probably needs a rewrite - period :-)
>


Attachments:
pEpkey.asc (1.77 kB)