* [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability @ 2021-03-19 20:23 Bhaskar Chowdhury 2021-03-19 20:27 ` Jens Axboe 0 siblings, 1 reply; 6+ messages in thread From: Bhaskar Chowdhury @ 2021-03-19 20:23 UTC (permalink / raw) To: axboe, linux-block, linux-kernel; +Cc: rdunlap, Bhaskar Chowdhury A typo fix and sentence reconstruction for better readability. Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> --- Changes from V2: Thanks, Randy and Tom for the suggestion,mould it. Missed the subject line prefix of pattern,so added back 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..9da426d20f12 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 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability 2021-03-19 20:23 [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability Bhaskar Chowdhury @ 2021-03-19 20:27 ` Jens Axboe 2021-03-19 20:39 ` Bhaskar Chowdhury 0 siblings, 1 reply; 6+ messages in thread From: Jens Axboe @ 2021-03-19 20:27 UTC (permalink / raw) To: Bhaskar Chowdhury, linux-block, linux-kernel; +Cc: rdunlap On 3/19/21 2:23 PM, Bhaskar Chowdhury wrote: > > A typo fix and sentence reconstruction for better readability. > > Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> > --- > Changes from V2: > Thanks, Randy and Tom for the suggestion,mould it. > Missed the subject line prefix of pattern,so added back > > 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..9da426d20f12 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 This is still nonsense, see reply to previous version. -- Jens Axboe ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability 2021-03-19 20:27 ` Jens Axboe @ 2021-03-19 20:39 ` Bhaskar Chowdhury 2021-03-19 22:19 ` Jens Axboe 0 siblings, 1 reply; 6+ messages in thread From: Bhaskar Chowdhury @ 2021-03-19 20:39 UTC (permalink / raw) To: Jens Axboe; +Cc: linux-block, linux-kernel, rdunlap [-- Attachment #1: Type: text/plain, Size: 1104 bytes --] On 14:27 Fri 19 Mar 2021, Jens Axboe wrote: >On 3/19/21 2:23 PM, Bhaskar Chowdhury wrote: >> >> A typo fix and sentence reconstruction for better readability. >> >> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> >> --- >> Changes from V2: >> Thanks, Randy and Tom for the suggestion,mould it. >> Missed the subject line prefix of pattern,so added back >> >> 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..9da426d20f12 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 > >This is still nonsense, see reply to previous version. > Well, I was just trying get a sense of your sense...so ...it's all yours fella,take on ... >-- >Jens Axboe > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability 2021-03-19 20:39 ` Bhaskar Chowdhury @ 2021-03-19 22:19 ` Jens Axboe 2021-03-19 22:55 ` Bhaskar Chowdhury 0 siblings, 1 reply; 6+ messages in thread From: Jens Axboe @ 2021-03-19 22:19 UTC (permalink / raw) To: Bhaskar Chowdhury, linux-block, linux-kernel, rdunlap On 3/19/21 2:39 PM, Bhaskar Chowdhury wrote: > On 14:27 Fri 19 Mar 2021, Jens Axboe wrote: >> On 3/19/21 2:23 PM, Bhaskar Chowdhury wrote: >>> >>> A typo fix and sentence reconstruction for better readability. >>> >>> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> >>> --- >>> Changes from V2: >>> Thanks, Randy and Tom for the suggestion,mould it. >>> Missed the subject line prefix of pattern,so added back >>> >>> 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..9da426d20f12 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 >> >> This is still nonsense, see reply to previous version. >> > Well, I was just trying get a sense of your sense...so ...it's all yours > fella,take on ... It's not my sense, I didn't write that function or comment. Just seems pointless to me to update it and not get it actually legible and correct, which is why I sent you a suggestion to what should be. From that point of view, the suggested change actually makes it _worse_, because "requests functions completed their run" doesn't mean anything. At least the current one is kind of legible, since the "complete function" refers to the IPI completion function, which is what we're waiting for here. In any case, what I replied in v2 should be generally readable, and avoids the weird req's thing too which I really dislike. Just uses requests, that's correct and avoids a nonsensical possessive. So do send a v4 if you want with that wording. -- Jens Axboe ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability 2021-03-19 22:19 ` Jens Axboe @ 2021-03-19 22:55 ` Bhaskar Chowdhury 2021-03-19 22:56 ` Jens Axboe 0 siblings, 1 reply; 6+ messages in thread From: Bhaskar Chowdhury @ 2021-03-19 22:55 UTC (permalink / raw) To: Jens Axboe; +Cc: linux-block, linux-kernel, rdunlap [-- Attachment #1: Type: text/plain, Size: 2155 bytes --] On 16:19 Fri 19 Mar 2021, Jens Axboe wrote: >On 3/19/21 2:39 PM, Bhaskar Chowdhury wrote: >> On 14:27 Fri 19 Mar 2021, Jens Axboe wrote: >>> On 3/19/21 2:23 PM, Bhaskar Chowdhury wrote: >>>> >>>> A typo fix and sentence reconstruction for better readability. >>>> >>>> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> >>>> --- >>>> Changes from V2: >>>> Thanks, Randy and Tom for the suggestion,mould it. >>>> Missed the subject line prefix of pattern,so added back >>>> >>>> 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..9da426d20f12 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 >>> >>> This is still nonsense, see reply to previous version. >>> >> Well, I was just trying get a sense of your sense...so ...it's all yours >> fella,take on ... > >It's not my sense, I didn't write that function or comment. Just seems >pointless to me to update it and not get it actually legible and >correct, which is why I sent you a suggestion to what should be. From >that point of view, the suggested change actually makes it _worse_, >because "requests functions completed their run" doesn't mean anything. >At least the current one is kind of legible, since the "complete >function" refers to the IPI completion function, which is what we're >waiting for here. > >In any case, what I replied in v2 should be generally readable, and >avoids the weird req's thing too which I really dislike. Just uses >requests, that's correct and avoids a nonsensical possessive. > >So do send a v4 if you want with that wording. > I am apologetic about the pain I caused you to take this long route. I shall be prudent in the future. Thanks for standing, Jens. >-- >Jens Axboe > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability 2021-03-19 22:55 ` Bhaskar Chowdhury @ 2021-03-19 22:56 ` Jens Axboe 0 siblings, 0 replies; 6+ messages in thread From: Jens Axboe @ 2021-03-19 22:56 UTC (permalink / raw) To: Bhaskar Chowdhury, linux-block, linux-kernel, rdunlap On 3/19/21 4:55 PM, Bhaskar Chowdhury wrote: > On 16:19 Fri 19 Mar 2021, Jens Axboe wrote: >> On 3/19/21 2:39 PM, Bhaskar Chowdhury wrote: >>> On 14:27 Fri 19 Mar 2021, Jens Axboe wrote: >>>> On 3/19/21 2:23 PM, Bhaskar Chowdhury wrote: >>>>> >>>>> A typo fix and sentence reconstruction for better readability. >>>>> >>>>> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> >>>>> --- >>>>> Changes from V2: >>>>> Thanks, Randy and Tom for the suggestion,mould it. >>>>> Missed the subject line prefix of pattern,so added back >>>>> >>>>> 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..9da426d20f12 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 >>>> >>>> This is still nonsense, see reply to previous version. >>>> >>> Well, I was just trying get a sense of your sense...so ...it's all yours >>> fella,take on ... >> >> It's not my sense, I didn't write that function or comment. Just seems >> pointless to me to update it and not get it actually legible and >> correct, which is why I sent you a suggestion to what should be. From >> that point of view, the suggested change actually makes it _worse_, >> because "requests functions completed their run" doesn't mean anything. >> At least the current one is kind of legible, since the "complete >> function" refers to the IPI completion function, which is what we're >> waiting for here. >> >> In any case, what I replied in v2 should be generally readable, and >> avoids the weird req's thing too which I really dislike. Just uses >> requests, that's correct and avoids a nonsensical possessive. >> >> So do send a v4 if you want with that wording. >> > I am apologetic about the pain I caused you to take this long route. I shall be > prudent in the future. Thanks for standing, Jens. Well, at least the end result pulled it to completion :-) -- Jens Axboe ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-03-19 22:56 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-03-19 20:23 [PATCH V3] blk-mq: Trivial typo fix and sentence construction for better readability Bhaskar Chowdhury 2021-03-19 20:27 ` Jens Axboe 2021-03-19 20:39 ` Bhaskar Chowdhury 2021-03-19 22:19 ` Jens Axboe 2021-03-19 22:55 ` Bhaskar Chowdhury 2021-03-19 22:56 ` Jens Axboe
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox