[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Zv1ZN8XZmSZTD-78@gallifrey>
Date: Wed, 2 Oct 2024 14:31:19 +0000
From: "Dr. David Alan Gilbert" <linux@...blig.org>
To: Andy Shevchenko <andy@...nel.org>
Cc: kees@...nel.org, akpm@...ux-foundation.org, pmladek@...e.com,
rostedt@...dmis.org, linux@...musvillemoes.dk,
senozhatsky@...omium.org, linux-hardening@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] printf: Remove unused 'bprintf'
* Andy Shevchenko (andy@...nel.org) wrote:
> On Wed, Oct 02, 2024 at 02:21:25AM +0100, linux@...blig.org wrote:
> > From: "Dr. David Alan Gilbert" <linux@...blig.org>
> >
> > bprintf is unused. Remove it.
>
> bprintf()
OK.
> > It was added in
> > commit 4370aa4aa753 ("vsprintf: add binary printf")
> >
> > but as far as I can see was never used, unlike the other
> > two functions in that patch.
>
> Please, rewrap these lines to use more room on each line.
Something like this? :
bprintf() is unused. Remove it. It was added in
commit 4370aa4aa753 ("vsprintf: add binary printf")
but as far as I can see was never used, unlike the other two functions in
that patch.
> ...
>
> I am not familiar with tricks in BPF or ftrace code where this actually might
> be implicitly called via a macro, but brief grep gives nothing that might point
> to that.
I've got an all-yes build (well, most after I took out broken stuff) booting
with it, and it has CONFIG_BINARY_PRINTF=y and CONFIG_FTRACE=y .
trace_seq.c uses seq_buf_bprintf which uses bstr_printf rather than the plain
bprintf() that I've deleted.
Not sure where to dig in BPF, but I've had a fairly good grep around.
> Hence, with the amended commit message
>
> Reviewed-by: Andy Shevchenko <andy@...nel.org>
Thanks!
> P.S. I hope we may rely in CIs to report issues soon, if any.
Sure
Dave
> --
> With Best Regards,
> Andy Shevchenko
>
>
>
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
Powered by blists - more mailing lists