[PATCH v2 0/4] rust: list: remove HasListLinks::OFFSET

Tamir Duberstein posted 4 patches 2 weeks, 6 days ago
There is a newer version of this series
rust/kernel/list.rs                    |  18 +++--
rust/kernel/list/impl_list_item_mod.rs | 128 +++++++++++++++------------------
2 files changed, 67 insertions(+), 79 deletions(-)
[PATCH v2 0/4] rust: list: remove HasListLinks::OFFSET
Posted by Tamir Duberstein 2 weeks, 6 days ago
The bulk of this change occurs in the last commit, please its commit
messages for details. The first commit exists in 2 other series but was
picked into this series to allow using `container_of!` without the need
to cast from `*const Self` to `*mut Self`.

Signed-off-by: Tamir Duberstein <tamird@gmail.com>
---
Changes in v2:
- Change type parameter delimiter to `{}` for consistency. (Boqun Feng)
- Rebase on v6.15-rc1.
- Extract first commit to its own series as it is shared with other
  series.
- Link to v1: https://lore.kernel.org/r/20250324-list-no-offset-v1-0-afd2b7fc442a@gmail.com

---
Tamir Duberstein (4):
      rust: list: simplify macro capture
      rust: list: use consistent type parameter style
      rust: list: use consistent self parameter name
      rust: list: remove OFFSET constants

 rust/kernel/list.rs                    |  18 +++--
 rust/kernel/list/impl_list_item_mod.rs | 128 +++++++++++++++------------------
 2 files changed, 67 insertions(+), 79 deletions(-)
---
base-commit: 0af2f6be1b4281385b618cb86ad946eded089ac8
change-id: 20250324-list-no-offset-96ef65cb2a95
prerequisite-change-id: 20250409-container-of-mutness-b153dab4388d:v1
prerequisite-patch-id: 53d5889db599267f87642bb0ae3063c29bc24863

Best regards,
-- 
Tamir Duberstein <tamird@gmail.com>
Re: [PATCH v2 0/4] rust: list: remove HasListLinks::OFFSET
Posted by Tamir Duberstein 2 weeks, 6 days ago
On Wed, Apr 9, 2025 at 10:51 AM Tamir Duberstein <tamird@gmail.com> wrote:
>
> The bulk of this change occurs in the last commit, please its commit
> messages for details. The first commit exists in 2 other series but was
> picked into this series to allow using `container_of!` without the need
> to cast from `*const Self` to `*mut Self`.

That first commit is now in the prerequisite patch, not here.
Re: [PATCH v2 0/4] rust: list: remove HasListLinks::OFFSET
Posted by Tamir Duberstein 2 weeks, 1 day ago
On Wed, Apr 9, 2025 at 10:53 AM Tamir Duberstein <tamird@gmail.com> wrote:
>
> On Wed, Apr 9, 2025 at 10:51 AM Tamir Duberstein <tamird@gmail.com> wrote:
> >
> > The bulk of this change occurs in the last commit, please its commit
> > messages for details. The first commit exists in 2 other series but was
> > picked into this series to allow using `container_of!` without the need
> > to cast from `*const Self` to `*mut Self`.
>
> That first commit is now in the prerequisite patch, not here.

Gentle ping; this still needs a review. The first two patches are
trivial, and the last one is similar in spirit to
https://lore.kernel.org/all/20250411-no-offset-v3-1-c0b174640ec3@gmail.com/
which was reviewed by Benno Lossin, Alice Ryhl, and Andreas Hindborg.
Could you folks have a look please?