On Mon, 2 May 2022 at 11:20, John Ogness john.ogness@linutronix.de wrote:
On 2022-05-02, Petr Mladek pmladek@suse.com wrote:
My proposal would be to fix the tracepoint like so:
[...]
This fixes the KFENCE and KCSAN tests.
Unless I hear objections, I'll prepare a patch explaining why we need to fix the tracepoint.
It makes perfect sense to me.
This is the easiest place for it. However, it should be clear that in the context of trace_console_rcuidle(), the message is not yet visible to any readers. The message _will_ get committed and definitely _will_ become visible at some point. But it is not (yet) visible at _this_ point. Maybe that is OK for what it is being used for.
If trace_console_rcuidle() must be called at the point of visibility for readers, it becomes more complicated.
I think that wasn't the original intent, so fixing it up to actually happen on printk() is probably the sanest thing. I sent the patch trying to explain: https://lore.kernel.org/all/20220503073844.4148944-1-elver@google.com/
Thank you both for the valuable pointers!
-- Marco
John