On Tue, 17 Dec 2024 at 10:04, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
> I'm not sure what you mean. If the kernels are the same, then the pointers
> should also be the same, as KASLR just shifts them. This no longer uses
> module code. It only traces core kernel code which should always have the
> same offsets.
(a) the shifting is what caused problems with you having to hack
round the format string and %pS.
(b) the data addresses are more than shifted, so that "data_delta" is
*completely* bogus
(c) neither of them are AT ALL valid for modules regardless
Stop using the delta fields. They are broken. Even for the same
kernel. It's literally a "I made sh*t up and it sometimes works"
situation.
That "sometimes works" is not how we do kernel development. Stop it.
What *woiuld* have been an acceptable model is to actually modify the
boot-time buffers in place, using actual real heuristics that look at
whether a pointer was IN THE CODE SECTION OR THE STATIC DATA section
of the previous boot.
But you never did that. All this delta code has always been complete
and utter garbage, and complete hacks.
Remove it.
Then, if at some point you can do it *right* (see above), maybe you
can try to re-introduce it. But the current delta code is pure and
utter garbage and needs to die. No more of this "hacking shit up to
make it sometimes work".
Linus
