Received-SPF: pass (google.com: domain of linux-kernel+bounces-21926-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 10 Jan 2024 09:56:22 +0000
To: Charalampos Mitrodimas <charmitro@posteo.net>, ojeda@kernel.org, alex.gaynor@gmail.com, wedsonaf@gmail.com, boqun.feng@gmail.com, gary@garyguo.net, bjorn3_gh@protonmail.com, aliceryhl@google.com, a.hindborg@samsung.com
From: Benno Lossin <benno.lossin@proton.me>
Cc: rust-for-linux@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] rust: sync: `CondVar` rename "wait_list" to "wait_queue_head"
Message-ID: <b11521cc-b873-4734-bb51-83f313300509@proton.me>
In-Reply-To: <20240105012930.1426214-1-charmitro@posteo.net>
References: <20240105012930.1426214-1-charmitro@posteo.net>
Feedback-ID: 71780778:user:proton
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

On 05.01.24 02:29, Charalampos Mitrodimas wrote:
> Fields named "wait_list" usually are of type "struct list_head". To
> avoid confusion and because it is of type
> "Opaque<bindings::wait_queue_head>" we are renaming "wait_list" to
> "wait_queue_head".
>=20
> Signed-off-by: Charalampos Mitrodimas <charmitro@posteo.net>
> ---
>   rust/kernel/sync/condvar.rs | 20 ++++++++++++--------
>   1 file changed, 12 insertions(+), 8 deletions(-)

Reviewed-by: Benno Lossin <benno.lossin@proton.me>

Thanks for the patch!

--=20
Cheers,
Benno