Re: [PATCH] rust: introduce sfile macro for easier code tracing

["gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>]

On Thu, Jun 05, 2025 at 05:38:56PM +0200, Miguel Ojeda wrote:
> On Thu, Jun 5, 2025 at 5:21 PM gregkh@linuxfoundation.org
> <gregkh@linuxfoundation.org> wrote:
> >
> > And I'm saying "don't use dbg!()" :)
> 
> You likely know all this, but in case it helps to clarify the
> discussion: `dbg!` is only meant for temporary code, i.e. so it
> shouldn't be used (in the sense of committing into Git).
> 
> That is, it is a development tool, unlike `pr_dbg!`.

Ah, I didn't realize this, so thanks.  The dbg!() implementation in the
kernel tree matches the userspace stuff, so the documentation refers to
"release builds" and the like.  It would be good if we could say
something like "Use ONLY for local debugging code, NEVER submit a patch
with this used in it."

Or we can add it to checkpatch to catch when people inevitably do submit
patches with it?  :)

thanks,

greg k-h