From: "Alexandre Courbot" <acourbot@nvidia.com>
To: "Joel Fernandes" <joelagnelf@nvidia.com>
Cc: linux-kernel@vger.kernel.org,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Simona Vetter" <simona@ffwll.ch>,
"Jonathan Corbet" <corbet@lwn.net>,
"Alex Deucher" <alexander.deucher@amd.com>,
"Christian König" <christian.koenig@amd.com>,
"Jani Nikula" <jani.nikula@linux.intel.com>,
"Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
"Rodrigo Vivi" <rodrigo.vivi@intel.com>,
"Tvrtko Ursulin" <tursulin@ursulin.net>,
"Huang Rui" <ray.huang@amd.com>,
"Matthew Auld" <matthew.auld@intel.com>,
"Matthew Brost" <matthew.brost@intel.com>,
"Lucas De Marchi" <lucas.demarchi@intel.com>,
"Thomas Hellström" <thomas.hellstrom@linux.intel.com>,
"Helge Deller" <deller@gmx.de>,
"Danilo Krummrich" <dakr@kernel.org>,
"Alice Ryhl" <aliceryhl@google.com>,
"Miguel Ojeda" <ojeda@kernel.org>,
"Alex Gaynor" <alex.gaynor@gmail.com>,
"Boqun Feng" <boqun.feng@gmail.com>,
"Gary Guo" <gary@garyguo.net>,
"Björn Roy Baron" <bjorn3_gh@protonmail.com>,
"Benno Lossin" <lossin@kernel.org>,
"Andreas Hindborg" <a.hindborg@kernel.org>,
"Trevor Gross" <tmgross@umich.edu>,
"Alistair Popple" <apopple@nvidia.com>,
"Andrea Righi" <arighi@nvidia.com>, "Zhi Wang" <zhiw@nvidia.com>,
"Philipp Stanner" <phasta@kernel.org>,
"Elle Rhumsaa" <elle@weathered-steel.dev>,
"Daniel Almeida" <daniel.almeida@collabora.com>,
nouveau@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
rust-for-linux@vger.kernel.org, linux-doc@vger.kernel.org,
amd-gfx@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
intel-xe@lists.freedesktop.org, linux-fbdev@vger.kernel.org
Subject: Re: [PATCH -next v9 1/3] rust: clist: Add support to interface with C linked lists
Date: Thu, 19 Feb 2026 11:47:23 +0900 [thread overview]
Message-ID: <DGILFRPORZFW.31EV2LX1MGXMK@nvidia.com> (raw)
In-Reply-To: <1771462209.c70cb872032e4abe@nvidia.com>
On Thu Feb 19, 2026 at 9:59 AM JST, Joel Fernandes wrote:
> On Thu, 19 Feb 2026 09:35:31 +0900, Alexandre Courbot wrote:
>
> [...]
>
>> I asked this a couple of times ([1], [2]) but got no reply, so let me
>> try again. :) Given that `list_head` is doubly-linked, can we also
>> implement `DoubleEndedIterator`?
>>
>> This can be done in a follow-up patch but should be there eventually as
>> C lists are often parsed in both directions.
>>
>> [1] https://lore.kernel.org/all/DEGQCMSX1SGZ.2NQDPG5KUNA9D@nvidia.com/
>> [2] https://lore.kernel.org/all/DEOLRLCZQMBG.1WHBR4YL8SKYR@nvidia.com/
>
> There was a lot of redesign and you only mentioned it as an optional
> suggestion and there several higher priority comments.
>
> Anyway, I disagree. I don't think there's a use case for
> DoubleEndedIterator at the moment -- none of the current users of clist need
> reverse iteration. I'd prefer to keep the interface minimal and add it when
> there's an actual need for it, rather than adding API surface that nobody
> uses. If a usecase comes up, we can always trivially add it.
It's a double-linked list still, and `DoubleEndedIterator` a standard
Rust trait. Adding it now would spare us the trouble of deciding which
tree takes the feature when it eventually lands.
But even without it the series still stands, so your call.
next prev parent reply other threads:[~2026-02-19 2:47 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-10 23:32 [PATCH -next v9 0/3] rust: Add CList and GPU buddy allocator bindings Joel Fernandes
2026-02-10 23:32 ` [PATCH -next v9 1/3] rust: clist: Add support to interface with C linked lists Joel Fernandes
2026-02-11 9:23 ` Danilo Krummrich
2026-02-11 17:28 ` Joel Fernandes
2026-02-19 0:35 ` Alexandre Courbot
2026-02-19 0:59 ` Joel Fernandes
2026-02-19 2:47 ` Alexandre Courbot [this message]
2026-02-19 15:29 ` Joel Fernandes
2026-02-19 16:20 ` Joel Fernandes
2026-02-20 1:57 ` Alexandre Courbot
2026-02-10 23:32 ` [PATCH -next v9 2/3] rust: gpu: Add GPU buddy allocator bindings Joel Fernandes
2026-02-10 23:32 ` [PATCH -next v9 3/3] nova-core: mm: Select GPU_BUDDY for VRAM allocation Joel Fernandes
2026-02-11 9:21 ` Danilo Krummrich
2026-02-11 9:19 ` [PATCH -next v9 0/3] rust: Add CList and GPU buddy allocator bindings Danilo Krummrich
2026-02-11 17:30 ` Joel Fernandes
[not found] <20260211-v9-nova-mm-v9-0-a8e261c2e734@nvidia.com>
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DGILFRPORZFW.31EV2LX1MGXMK@nvidia.com \
--to=acourbot@nvidia.com \
--cc=a.hindborg@kernel.org \
--cc=alex.gaynor@gmail.com \
--cc=alexander.deucher@amd.com \
--cc=aliceryhl@google.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=apopple@nvidia.com \
--cc=arighi@nvidia.com \
--cc=bjorn3_gh@protonmail.com \
--cc=boqun.feng@gmail.com \
--cc=christian.koenig@amd.com \
--cc=corbet@lwn.net \
--cc=dakr@kernel.org \
--cc=daniel.almeida@collabora.com \
--cc=deller@gmx.de \
--cc=dri-devel@lists.freedesktop.org \
--cc=elle@weathered-steel.dev \
--cc=gary@garyguo.net \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
--cc=jani.nikula@linux.intel.com \
--cc=joelagnelf@nvidia.com \
--cc=joonas.lahtinen@linux.intel.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lossin@kernel.org \
--cc=lucas.demarchi@intel.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=matthew.auld@intel.com \
--cc=matthew.brost@intel.com \
--cc=mripard@kernel.org \
--cc=nouveau@lists.freedesktop.org \
--cc=ojeda@kernel.org \
--cc=phasta@kernel.org \
--cc=ray.huang@amd.com \
--cc=rodrigo.vivi@intel.com \
--cc=rust-for-linux@vger.kernel.org \
--cc=simona@ffwll.ch \
--cc=thomas.hellstrom@linux.intel.com \
--cc=tmgross@umich.edu \
--cc=tursulin@ursulin.net \
--cc=zhiw@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox