Archive-only list for patches
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Shankari Anand <shankari.ak0208@gmail.com>,
	linux-kernel@vger.kernel.org, rust-for-linux@vger.kernel.org,
	patches@lists.linux.dev
Cc: oe-kbuild-all@lists.linux.dev, Miguel Ojeda <ojeda@kernel.org>,
	Alex Gaynor <alex.gaynor@gmail.com>,
	Boqun Feng <boqun.feng@gmail.com>, Gary Guo <gary@garyguo.net>,
	Roy Baron <bjorn3_gh@protonmail.com>,
	Benno Lossin <lossin@kernel.org>,
	Andreas Hindborg <a.hindborg@kernel.org>,
	Alice Ryhl <aliceryhl@google.com>,
	Trevor Gross <tmgross@umich.edu>,
	Danilo Krummrich <dakr@kernel.org>,
	Shankari Anand <shankari.ak0208@gmail.com>
Subject: Re: [PATCH v2 2/2] rust: update ARef and AlwaysRefCounted call sites to import from sync::aref
Date: Wed, 2 Jul 2025 20:24:11 +0800	[thread overview]
Message-ID: <202507022055.C7y9sGGN-lkp@intel.com> (raw)
In-Reply-To: <20250625111133.698481-2-shankari.ak0208@gmail.com>

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@intel.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

  reply	other threads:[~2025-07-02 12:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-25 11:11 [PATCH v2 1/2] rust: move ARef and AlwaysRefCounted to sync::aref Shankari Anand
2025-06-25 11:11 ` [PATCH v2 2/2] rust: update ARef and AlwaysRefCounted call sites to import from sync::aref Shankari Anand
2025-07-02 12:24   ` kernel test robot [this message]
2025-06-25 22:29 ` [PATCH v2 1/2] rust: move ARef and AlwaysRefCounted to sync::aref Benno Lossin
2025-06-26 13:55   ` Shankari Anand
2025-07-02 11:01 ` kernel test robot

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=202507022055.C7y9sGGN-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=a.hindborg@kernel.org \
    --cc=alex.gaynor@gmail.com \
    --cc=aliceryhl@google.com \
    --cc=bjorn3_gh@protonmail.com \
    --cc=boqun.feng@gmail.com \
    --cc=dakr@kernel.org \
    --cc=gary@garyguo.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lossin@kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=ojeda@kernel.org \
    --cc=patches@lists.linux.dev \
    --cc=rust-for-linux@vger.kernel.org \
    --cc=shankari.ak0208@gmail.com \
    --cc=tmgross@umich.edu \
    /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