s/funtion/functions/
Plus the sentence reconstructed for better readability.
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
Changes from V1:
Randy's suggestions incorporated.
block/blk-mq-tag.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
index 9c92053e704d..c2bef283db63 100644
--- a/block/blk-mq-tag.c
+++ b/block/blk-mq-tag.c
@@ -373,8 +373,8 @@ static bool blk_mq_tagset_count_completed_rqs(struct request *rq,
}
/**
- * blk_mq_tagset_wait_completed_request - wait until all completed req's
- * complete funtion is run
+ * blk_mq_tagset_wait_completed_request - wait until all the req's
+ * functions completed their run
* @tagset: Tag set to drain completed request
*
* Note: This function has to be run after all IO queues are shutdown
--
2.26.2
Hm, needs some spacing fixes IMO. See below.
On Sat, 20 Mar 2021, Bhaskar Chowdhury wrote:
>
> s/funtion/functions/
>
> Plus the sentence reconstructed for better readability.
>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> Changes from V1:
> Randy's suggestions incorporated.
>
> block/blk-mq-tag.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
> index 9c92053e704d..c2bef283db63 100644
> --- a/block/blk-mq-tag.c
> +++ b/block/blk-mq-tag.c
> @@ -373,8 +373,8 @@ static bool blk_mq_tagset_count_completed_rqs(struct request *rq,
> }
>
> /**
> - * blk_mq_tagset_wait_completed_request - wait until all completed req's
> - * complete funtion is run
> + * blk_mq_tagset_wait_completed_request - wait until all the req's
the req's
> + * functions completed their run
and more indentation + wording on that line above:
* functions have completed their run
> * @tagset: Tag set to drain completed request
> *
> * Note: This function has to be run after all IO queues are shutdown
> --
Thanks.
On 13:03 Fri 19 Mar 2021, Randy Dunlap wrote:
>
>Hm, needs some spacing fixes IMO. See below.
>
>
>On Sat, 20 Mar 2021, Bhaskar Chowdhury wrote:
>
>>
>> s/funtion/functions/
>>
>> Plus the sentence reconstructed for better readability.
>>
>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>> ---
>> Changes from V1:
>> Randy's suggestions incorporated.
>>
>> block/blk-mq-tag.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
>> index 9c92053e704d..c2bef283db63 100644
>> --- a/block/blk-mq-tag.c
>> +++ b/block/blk-mq-tag.c
>> @@ -373,8 +373,8 @@ static bool blk_mq_tagset_count_completed_rqs(struct request *rq,
>> }
>>
>> /**
>> - * blk_mq_tagset_wait_completed_request - wait until all completed req's
>> - * complete funtion is run
>> + * blk_mq_tagset_wait_completed_request - wait until all the req's
>
> the req's
>
>> + * functions completed their run
>
>and more indentation + wording on that line above:
> * functions have completed their run
>
Apology...my bad ...fixing ...V3 ...in quick time ...
>> * @tagset: Tag set to drain completed request
>> *
>> * Note: This function has to be run after all IO queues are shutdown
>> --
>
>Thanks.
On Fri, Mar 19, 2021 at 01:03:02PM -0700, Randy Dunlap wrote:
>
> Hm, needs some spacing fixes IMO. See below.
>
>
> On Sat, 20 Mar 2021, Bhaskar Chowdhury wrote:
>
> >
> > s/funtion/functions/
> >
> > Plus the sentence reconstructed for better readability.
> >
> > Signed-off-by: Bhaskar Chowdhury <[email protected]>
> > ---
> > Changes from V1:
> > Randy's suggestions incorporated.
> >
> > block/blk-mq-tag.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
> > index 9c92053e704d..c2bef283db63 100644
> > --- a/block/blk-mq-tag.c
> > +++ b/block/blk-mq-tag.c
> > @@ -373,8 +373,8 @@ static bool blk_mq_tagset_count_completed_rqs(struct request *rq,
> > }
> >
> > /**
> > - * blk_mq_tagset_wait_completed_request - wait until all completed req's
> > - * complete funtion is run
completion function
That's my read of it.
--Tom
> > + * blk_mq_tagset_wait_completed_request - wait until all the req's
>
> the req's
>
> > + * functions completed their run
>
> and more indentation + wording on that line above:
> * functions have completed their run
>
> > * @tagset: Tag set to drain completed request
> > *
> > * Note: This function has to be run after all IO queues are shutdown
> > --
>
> Thanks.
On Fri, 19 Mar 2021, Tom Saeger wrote:
> On Fri, Mar 19, 2021 at 01:03:02PM -0700, Randy Dunlap wrote:
>>
>> Hm, needs some spacing fixes IMO. See below.
>>
>>
>> On Sat, 20 Mar 2021, Bhaskar Chowdhury wrote:
>>
>>>
>>> s/funtion/functions/
>>>
>>> Plus the sentence reconstructed for better readability.
>>>
>>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>>> ---
>>> Changes from V1:
>>> Randy's suggestions incorporated.
>>>
>>> block/blk-mq-tag.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
>>> index 9c92053e704d..c2bef283db63 100644
>>> --- a/block/blk-mq-tag.c
>>> +++ b/block/blk-mq-tag.c
>>> @@ -373,8 +373,8 @@ static bool blk_mq_tagset_count_completed_rqs(struct request *rq,
>>> }
>>>
>>> /**
>>> - * blk_mq_tagset_wait_completed_request - wait until all completed req's
>>> - * complete funtion is run
>
> completion function
>
> That's my read of it.
Sounds good. Thanks.
Bhaskar, please use this wording.
> --Tom
>
>>> + * blk_mq_tagset_wait_completed_request - wait until all the req's
>>
>> the req's
>>
>>> + * functions completed their run
>>
>> and more indentation + wording on that line above:
>> * functions have completed their run
>>
>>> * @tagset: Tag set to drain completed request
>>> *
>>> * Note: This function has to be run after all IO queues are shutdown
>>> --
>>
>> Thanks.
>
On 3/19/21 2:07 PM, Bhaskar Chowdhury wrote:
> On 13:03 Fri 19 Mar 2021, Randy Dunlap wrote:
>>
>> Hm, needs some spacing fixes IMO. See below.
>>
>>
>> On Sat, 20 Mar 2021, Bhaskar Chowdhury wrote:
>>
>>>
>>> s/funtion/functions/
>>>
>>> Plus the sentence reconstructed for better readability.
>>>
>>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>>> ---
>>> Changes from V1:
>>> Randy's suggestions incorporated.
>>>
>>> block/blk-mq-tag.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
>>> index 9c92053e704d..c2bef283db63 100644
>>> --- a/block/blk-mq-tag.c
>>> +++ b/block/blk-mq-tag.c
>>> @@ -373,8 +373,8 @@ static bool blk_mq_tagset_count_completed_rqs(struct request *rq,
>>> }
>>>
>>> /**
>>> - * blk_mq_tagset_wait_completed_request - wait until all completed req's
>>> - * complete funtion is run
>>> + * blk_mq_tagset_wait_completed_request - wait until all the req's
>>
>> the req's
>>
>>> + * functions completed their run
>>
>> and more indentation + wording on that line above:
>> * functions have completed their run
>>
> Apology...my bad ...fixing ...V3 ...in quick time ...
Easy, let's get this right instead of spinning tons of versions of a
trivial patch. The text should be:
Wait until all scheduled request completions have finished.
If we're fixing a typo, let's actually make it understandable too...
--
Jens Axboe