[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <8CCB36E4-4C5D-4E2F-BC58-255770AA4904@collabora.com>
Date: Wed, 30 Jul 2025 10:36:33 -0300
From: Daniel Almeida <daniel.almeida@...labora.com>
To: Miguel Ojeda <ojeda@...nel.org>
Cc: Alex Gaynor <alex.gaynor@...il.com>,
Andreas Hindborg <a.hindborg@...nel.org>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>,
Boqun Feng <boqun.feng@...il.com>,
Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <lossin@...nel.org>,
Alice Ryhl <aliceryhl@...gle.com>,
Trevor Gross <tmgross@...ch.edu>,
Danilo Krummrich <dakr@...nel.org>,
rust-for-linux@...r.kernel.org,
linux-block@...r.kernel.org,
dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org,
patches@...ts.linux.dev
Subject: Re: [PATCH 0/3] Fix broken `srctree/` links and warn about them
Hi Miguel, this is indeed nice!
> On 30 Jul 2025, at 10:07, Miguel Ojeda <ojeda@...nel.org> wrote:
>
> This fixes a handful of broken links and introduces a warning to
> prevent them from happening in the future.
>
> Relatedly, we could also perhaps check the other side of the links, but
> perhaps there are cases we want to customize. Alternatively, we could
> also in the future introduce custom syntax for these that avoids
> repetition or a fancier preprocessing step.
>
> Miguel Ojeda (3):
> rust: block: fix `srctree/` links
> rust: drm: fix `srctree/` links
> rust: warn if `srctree/` links do not exist
>
> rust/Makefile | 6 ++++++
> rust/kernel/block/mq/gen_disk.rs | 2 +-
> rust/kernel/drm/device.rs | 2 +-
> rust/kernel/drm/driver.rs | 2 +-
> rust/kernel/drm/file.rs | 2 +-
> rust/kernel/drm/gem/mod.rs | 2 +-
> rust/kernel/drm/ioctl.rs | 2 +-
> 7 files changed, 12 insertions(+), 6 deletions(-)
>
> —
> 2.50.1
Reviewed-by: Daniel Almeida <daniel.almeida@...labora.com>
Patch 3 alone indeed produces the following warnings:
warning: srctree/ link to include/linux/blk_mq.h does not exist
warning: srctree/ link to include/linux/drm/drm_device.h does not exist
warning: srctree/ link to include/linux/drm/drm_ioctl.h does not exist
warning: srctree/ link to include/linux/drm/drm_file.h does not exist
warning: srctree/ link to include/linux/drm/drm_drv.h does not exist
warning: srctree/ link to include/linux/drm/drm_gem.h does not exist
So you can add my Tested-by for that one.
Cheers,
— Daniel
Powered by blists - more mailing lists