Commit 5990533a authored by Tamir Duberstein's avatar Tamir Duberstein Committed by Miguel Ojeda
Browse files

rust: seq_file: use `kernel::{fmt,prelude::fmt!}` 



Reduce coupling to implementation details of the formatting machinery by
avoiding direct use for `core`'s formatting traits and macros.

Acked-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: default avatarAlice Ryhl <aliceryhl@google.com>
Reviewed-by: default avatarBenno Lossin <lossin@kernel.org>
Signed-off-by: default avatarTamir Duberstein <tamird@gmail.com>
Signed-off-by: default avatarMiguel Ojeda <ojeda@kernel.org>
parent aa2417c1

rust/kernel/seq_file.rs

+3 −3
Original line number Diff line number Diff line
@@ -4,7 +4,7 @@ 
//!

//! C header: [`include/linux/seq_file.h`](srctree/include/linux/seq_file.h)



use crate::{bindings, c_str, types::NotThreadSafe, types::Opaque};

use crate::{bindings, c_str, fmt, types::NotThreadSafe, types::Opaque};



/// A utility for generating the contents of a seq file.

#[repr(transparent)]
@@ -31,7 +31,7 @@ pub unsafe fn from_raw<'a>(ptr: *mut bindings::seq_file) -> &'a SeqFile { 


    /// Used by the [`seq_print`] macro.

    #[inline]

    pub fn call_printf(&self, args: core::fmt::Arguments<'_>) {

    pub fn call_printf(&self, args: fmt::Arguments<'_>) {

        // SAFETY: Passing a void pointer to `Arguments` is valid for `%pA`.

        unsafe {

            bindings::seq_printf(
@@ -47,7 +47,7 @@ pub fn call_printf(&self, args: core::fmt::Arguments<'_>) { 
#[macro_export]

macro_rules! seq_print {

    ($m:expr, $($arg:tt)+) => (

        $m.call_printf(format_args!($($arg)+))

        $m.call_printf($crate::prelude::fmt!($($arg)+))

    );

}

pub use seq_print;