* [PATCH] rust: block: fix comment referring to states by wrong labels @ 2026-04-28 7:24 ` Hsiu Che Yu 2026-04-28 11:42 ` Miguel Ojeda 2026-04-29 10:17 ` Andreas Hindborg 0 siblings, 2 replies; 5+ messages in thread From: Hsiu Che Yu @ 2026-04-28 7:24 UTC (permalink / raw) To: Andreas Hindborg, Boqun Feng, Miguel Ojeda, Gary Guo, Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross, Danilo Krummrich, Peter Zijlstra (Intel) Cc: linux-block, rust-for-linux, linux-kernel, Hsiu Che Yu The comment in `Request` describing the four ownership states referred to states 2 and 3 as "B" and "C", which are inconsistent with the numbering scheme used throughout the same comment block. Update the labels to use "2" and "3" to match the existing numbered list. Fixes: a307bf1db5448 ("rust: block: convert `block::mq` to use `Refcount`") Signed-off-by: Hsiu Che Yu <yu.whisper.personal@gmail.com> --- rust/kernel/block/mq/request.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rust/kernel/block/mq/request.rs b/rust/kernel/block/mq/request.rs index ce3e30c81cb5..75d3fdf896e0 100644 --- a/rust/kernel/block/mq/request.rs +++ b/rust/kernel/block/mq/request.rs @@ -39,7 +39,7 @@ /// back ownership to the block layer. /// /// Note that the driver can still obtain new `ARef` even if there is no `ARef`s in existence by -/// using `tag_to_rq`, hence the need to distinguish B and C. +/// using `tag_to_rq`, hence the need to distinguish 2 and 3. /// /// The states are tracked through the private `refcount` field of /// `RequestDataWrapper`. This structure lives in the private data area of the C --- base-commit: b4e07588e743c989499ca24d49e752c074924a9a change-id: 20260428-fix-docs-mq-req-d2a0fe1b6471 Best regards, -- Hsiu Che Yu <yu.whisper.personal@gmail.com> ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] rust: block: fix comment referring to states by wrong labels 2026-04-28 7:24 ` [PATCH] rust: block: fix comment referring to states by wrong labels Hsiu Che Yu @ 2026-04-28 11:42 ` Miguel Ojeda 2026-04-28 11:48 ` Hsiu Che Yu 2026-04-29 10:17 ` Andreas Hindborg 1 sibling, 1 reply; 5+ messages in thread From: Miguel Ojeda @ 2026-04-28 11:42 UTC (permalink / raw) To: Hsiu Che Yu Cc: Andreas Hindborg, Boqun Feng, Miguel Ojeda, Gary Guo, Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross, Danilo Krummrich, Peter Zijlstra (Intel), linux-block, rust-for-linux, linux-kernel On Tue, Apr 28, 2026 at 9:25 AM Hsiu Che Yu <yu.whisper.personal@gmail.com> wrote: > > Fixes: a307bf1db5448 ("rust: block: convert `block::mq` to use `Refcount`") Since this commit exists in stable kernels, and this is visible in generated docs, we generally tag them for backport, so I am doing so here: Cc: stable@vger.kernel.org Thanks! Cheers, Miguel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] rust: block: fix comment referring to states by wrong labels 2026-04-28 11:42 ` Miguel Ojeda @ 2026-04-28 11:48 ` Hsiu Che Yu 0 siblings, 0 replies; 5+ messages in thread From: Hsiu Che Yu @ 2026-04-28 11:48 UTC (permalink / raw) To: Miguel Ojeda Cc: Hsiu Che Yu, Andreas Hindborg, Boqun Feng, Miguel Ojeda, Gary Guo, Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross, Danilo Krummrich, Peter Zijlstra (Intel), linux-block, rust-for-linux, linux-kernel On Tue, Apr 28, 2026 at 01:42:31PM +0200, Miguel Ojeda wrote: >Since this commit exists in stable kernels, and this is visible in >generated docs, we generally tag them for backport, so I am doing so >here: > >Cc: stable@vger.kernel.org Thanks! ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] rust: block: fix comment referring to states by wrong labels 2026-04-28 7:24 ` [PATCH] rust: block: fix comment referring to states by wrong labels Hsiu Che Yu 2026-04-28 11:42 ` Miguel Ojeda @ 2026-04-29 10:17 ` Andreas Hindborg 2026-04-29 10:35 ` Hsiu Che Yu 1 sibling, 1 reply; 5+ messages in thread From: Andreas Hindborg @ 2026-04-29 10:17 UTC (permalink / raw) To: Hsiu Che Yu, Boqun Feng, Miguel Ojeda, Gary Guo, Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross, Danilo Krummrich, Peter Zijlstra (Intel) Cc: linux-block, rust-for-linux, linux-kernel, Hsiu Che Yu Hi, "Hsiu Che Yu" <yu.whisper.personal@gmail.com> writes: > The comment in `Request` describing the four ownership states referred > to states 2 and 3 as "B" and "C", which are inconsistent with the > numbering scheme used throughout the same comment block. > > Update the labels to use "2" and "3" to match the existing numbered list. > > Fixes: a307bf1db5448 ("rust: block: convert `block::mq` to use `Refcount`") > Signed-off-by: Hsiu Che Yu <yu.whisper.personal@gmail.com> > --- > rust/kernel/block/mq/request.rs | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/rust/kernel/block/mq/request.rs b/rust/kernel/block/mq/request.rs > index ce3e30c81cb5..75d3fdf896e0 100644 > --- a/rust/kernel/block/mq/request.rs > +++ b/rust/kernel/block/mq/request.rs > @@ -39,7 +39,7 @@ > /// back ownership to the block layer. > /// > /// Note that the driver can still obtain new `ARef` even if there is no `ARef`s in existence by > -/// using `tag_to_rq`, hence the need to distinguish B and C. > +/// using `tag_to_rq`, hence the need to distinguish 2 and 3. > /// > /// The states are tracked through the private `refcount` field of > /// `RequestDataWrapper`. This structure lives in the private data area of the C > > --- > base-commit: b4e07588e743c989499ca24d49e752c074924a9a > change-id: 20260428-fix-docs-mq-req-d2a0fe1b6471 > > Best regards, > -- > Hsiu Che Yu <yu.whisper.personal@gmail.com> Thanks for the patch. Please be aware that this mistake is fixed in the latest rust null block series [1]. Please take a look at that series if you have more patches for the rust block subsystem queued. Best regards, Andreas Hindborg [1] https://lore.kernel.org/rust-for-linux/20260216-rnull-v6-19-rc5-send-v1-0-de9a7af4b469@kernel.org/ ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] rust: block: fix comment referring to states by wrong labels 2026-04-29 10:17 ` Andreas Hindborg @ 2026-04-29 10:35 ` Hsiu Che Yu 0 siblings, 0 replies; 5+ messages in thread From: Hsiu Che Yu @ 2026-04-29 10:35 UTC (permalink / raw) To: Andreas Hindborg Cc: Hsiu Che Yu, Boqun Feng, Miguel Ojeda, Gary Guo, Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross, Danilo Krummrich, Peter Zijlstra (Intel), linux-block, rust-for-linux, linux-kernel On Wed, Apr 29, 2026 at 12:17:43PM +0200, Andreas Hindborg wrote: >Thanks for the patch. Please be aware that this mistake is fixed in the >latest rust null block series [1]. Please take a look at that series if >you have more patches for the rust block subsystem queued. Thank you for letting me know. I'll take a look at the series. Best regards, Hsiu ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-04-29 10:35 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <u8w2aT8RMDhNK-zCkHKofFcml9mE8ndgtCnpwTrMWy6Kby41yC9ODSFCSKRZF8QahViSy2ANwHyurTUyVlgroA==@protonmail.internalid>
2026-04-28 7:24 ` [PATCH] rust: block: fix comment referring to states by wrong labels Hsiu Che Yu
2026-04-28 11:42 ` Miguel Ojeda
2026-04-28 11:48 ` Hsiu Che Yu
2026-04-29 10:17 ` Andreas Hindborg
2026-04-29 10:35 ` Hsiu Che Yu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox