On Mon 2021-03-01 09:47:47, Stephen Boyd wrote: > The %pS printk format (among some others) is used to print kernel > addresses symbolically. When the kernel prints an address inside of a > module, the kernel prints the addresses' symbol name along with the > module's name that contains the address. Let's make kernel stacktraces > easier to identify on KALLSYMS builds by including the build ID of a > module when we print the address. > > This is especially helpful for crash debugging with pstore or crashdump > kernels. If we have the build ID for the module in the stacktrace we can > request the debug symbols for the module from a remote debuginfod server I have read the thread so for. IMHO, all mentioned variants complicate the logs a lot. Either the backtrace lines are hard to parse. Or the OOps/panic blocks gets too long when the ID is mentioned for every loaded module. IMHO, neither proposed solution is acceptable to be always used. First, I think that only I some developers would actually use this information. Many of them either know what module was used or they do not have an easy way to get the debugging information by the ID. So, it should be configurable at minimum. Second, I am not sure that it should be part of each OOps/panic blob. One solution would be to print the ID by the module loader when the module gets loaded. It would be mentioned earlier in the log then. Or we could make it available, for example, via /proc. It is a kind of information that might be gathered later on a rebooted system. SUSE has "supportconfig" command that allows to gather a lot of similar information about the system. We use it when analyzing crashdumps and kernel bugs in general. Anyway, I consider this a very detailed information that is not suitable for everyone. It should be availabe on request, like for example, backtraces from all CPUs, list of tasks, memory info. Alternative solution would be to minimize the information, for example, by printing only the modules that appear in the backtrace. But this might be complicated to implement. Best Regards, Petr
