[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202507022055.C7y9sGGN-lkp@intel.com>
Date: Wed, 2 Jul 2025 20:24:11 +0800
From: kernel test robot <lkp@...el.com>
To: Shankari Anand <shankari.ak0208@...il.com>,
linux-kernel@...r.kernel.org, rust-for-linux@...r.kernel.org,
patches@...ts.linux.dev
Cc: oe-kbuild-all@...ts.linux.dev, Miguel Ojeda <ojeda@...nel.org>,
Alex Gaynor <alex.gaynor@...il.com>,
Boqun Feng <boqun.feng@...il.com>, Gary Guo <gary@...yguo.net>,
Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <lossin@...nel.org>,
Andreas Hindborg <a.hindborg@...nel.org>,
Alice Ryhl <aliceryhl@...gle.com>, Trevor Gross <tmgross@...ch.edu>,
Danilo Krummrich <dakr@...nel.org>,
Shankari Anand <shankari.ak0208@...il.com>
Subject: Re: [PATCH v2 2/2] rust: update ARef and AlwaysRefCounted call sites
to import from sync::aref
Hi Shankari,
kernel test robot noticed the following build errors:
[auto build test ERROR on 0303584766b7bdb6564c7e8f13e0b59b6ef44984]
url: https://github.com/intel-lab-lkp/linux/commits/Shankari-Anand/rust-update-ARef-and-AlwaysRefCounted-call-sites-to-import-from-sync-aref/20250625-191416
base: 0303584766b7bdb6564c7e8f13e0b59b6ef44984
patch link: https://lore.kernel.org/r/20250625111133.698481-2-shankari.ak0208%40gmail.com
patch subject: [PATCH v2 2/2] rust: update ARef and AlwaysRefCounted call sites to import from sync::aref
config: x86_64-rhel-9.4-rust (https://download.01.org/0day-ci/archive/20250702/202507022055.C7y9sGGN-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
rustc: rustc 1.78.0 (9b00956e5 2024-04-29)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250702/202507022055.C7y9sGGN-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202507022055.C7y9sGGN-lkp@intel.com/
All errors (new ones prefixed by >>):
PATH=/opt/cross/clang-18/bin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
INFO PATH=/opt/cross/rustc-1.78.0-bindgen-0.65.1/cargo/bin:/opt/cross/clang-18/bin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
/usr/bin/timeout -k 100 12h /usr/bin/make KCFLAGS= -fno-crash-diagnostics -Wno-error=return-type -Wreturn-type -funsigned-char -Wundef W=1 --keep-going LLVM=1 -j32 -C source O=/kbuild/obj/consumer/x86_64-rhel-9.4-rust ARCH=x86_64 SHELL=/bin/bash rustfmtcheck
make: Entering directory '/kbuild/src/consumer'
make[1]: Entering directory '/kbuild/obj/consumer/x86_64-rhel-9.4-rust'
>> Diff in drivers/gpu/drm/nova/driver.rs at line 1:
// SPDX-License-Identifier: GPL-2.0
-use kernel::{auxiliary, c_str, device::Core, drm, drm::gem, drm::ioctl, prelude::*, sync::aref::ARef};
+use kernel::{
+ auxiliary, c_str, device::Core, drm, drm::gem, drm::ioctl, prelude::*, sync::aref::ARef,
+};
use crate::file::File;
use crate::gem::NovaObject;
>> Diff in drivers/gpu/drm/nova/driver.rs at line 1:
// SPDX-License-Identifier: GPL-2.0
-use kernel::{auxiliary, c_str, device::Core, drm, drm::gem, drm::ioctl, prelude::*, sync::aref::ARef};
+use kernel::{
+ auxiliary, c_str, device::Core, drm, drm::gem, drm::ioctl, prelude::*, sync::aref::ARef,
+};
use crate::file::File;
use crate::gem::NovaObject;
Diff in rust/kernel/sync/aref.rs at line 10:
//!
//! For Rust-managed objects, prefer using [`Arc`](crate::sync::Arc) instead.
-use core::{
- marker::PhantomData,
- mem::ManuallyDrop,
- ops::Deref,
- ptr::NonNull,
-};
+use core::{marker::PhantomData, mem::ManuallyDrop, ops::Deref, ptr::NonNull};
/// Trait for types that are _always_ reference-counted.
///
>> Diff in rust/kernel/cred.rs at line 8:
//!
//! Reference: <https://www.kernel.org/doc/html/latest/security/credentials.html>
-use crate::{
- bindings,
- task::Kuid,
- sync::aref::AlwaysRefCounted,
- types::Opaque,
-};
+use crate::{bindings, sync::aref::AlwaysRefCounted, task::Kuid, types::Opaque};
/// Wraps the kernel's `struct cred`.
///
>> Diff in rust/kernel/device.rs at line 4:
//!
//! C header: [`include/linux/device.h`](srctree/include/linux/device.h)
-use crate::{
- bindings,
- str::CStr,
- sync::aref::ARef,
- types::Opaque,
-};
+use crate::{bindings, str::CStr, sync::aref::ARef, types::Opaque};
use core::{fmt, marker::PhantomData, ptr};
#[cfg(CONFIG_PRINTK)]
Diff in rust/kernel/pid_namespace.rs at line 7:
//! C header: [`include/linux/pid_namespace.h`](srctree/include/linux/pid_namespace.h) and
//! [`include/linux/pid.h`](srctree/include/linux/pid.h)
-use crate::{
- bindings,
- sync::aref::AlwaysRefCounted,
- types::Opaque,
-};
+use crate::{bindings, sync::aref::AlwaysRefCounted, types::Opaque};
use core::ptr;
/// Wraps the kernel's `struct pid_namespace`. Thread safe.
Diff in rust/kernel/sync/aref.rs at line 10:
//!
//! For Rust-managed objects, prefer using [`Arc`](crate::sync::Arc) instead.
-use core::{
- marker::PhantomData,
- mem::ManuallyDrop,
- ops::Deref,
- ptr::NonNull,
-};
+use core::{marker::PhantomData, mem::ManuallyDrop, ops::Deref, ptr::NonNull};
/// Trait for types that are _always_ reference-counted.
///
Diff in rust/kernel/pid_namespace.rs at line 7:
//! C header: [`include/linux/pid_namespace.h`](srctree/include/linux/pid_namespace.h) and
//! [`include/linux/pid.h`](srctree/include/linux/pid.h)
-use crate::{
- bindings,
- sync::aref::AlwaysRefCounted,
- types::Opaque,
-};
+use crate::{bindings, sync::aref::AlwaysRefCounted, types::Opaque};
use core::ptr;
/// Wraps the kernel's `struct pid_namespace`. Thread safe.
Diff in rust/kernel/sync/aref.rs at line 10:
//!
//! For Rust-managed objects, prefer using [`Arc`](crate::sync::Arc) instead.
-use core::{
- marker::PhantomData,
- mem::ManuallyDrop,
- ops::Deref,
- ptr::NonNull,
-};
+use core::{marker::PhantomData, mem::ManuallyDrop, ops::Deref, ptr::NonNull};
/// Trait for types that are _always_ reference-counted.
///
>> Diff in rust/kernel/cred.rs at line 8:
//!
//! Reference: <https://www.kernel.org/doc/html/latest/security/credentials.html>
-use crate::{
- bindings,
- task::Kuid,
- sync::aref::AlwaysRefCounted,
- types::Opaque,
-};
+use crate::{bindings, sync::aref::AlwaysRefCounted, task::Kuid, types::Opaque};
/// Wraps the kernel's `struct cred`.
///
>> Diff in rust/kernel/device.rs at line 4:
//!
//! C header: [`include/linux/device.h`](srctree/include/linux/device.h)
-use crate::{
- bindings,
- str::CStr,
- sync::aref::ARef,
- types::Opaque,
-};
+use crate::{bindings, str::CStr, sync::aref::ARef, types::Opaque};
use core::{fmt, marker::PhantomData, ptr};
#[cfg(CONFIG_PRINTK)]
make[2]: *** [Makefile:1831: rustfmt] Error 123
make[2]: Target 'rustfmtcheck' not remade because of errors.
make[1]: Leaving directory '/kbuild/obj/consumer/x86_64-rhel-9.4-rust'
make[1]: *** [Makefile:248: __sub-make] Error 2
make[1]: Target 'rustfmtcheck' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target 'rustfmtcheck' not remade because of errors.
make: Leaving directory '/kbuild/src/consumer'
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists