On 2025/11/13 19:20, lijiang wrote: > On Thu, Nov 13, 2025 at 11:10 AM <[email protected] > <mailto:[email protected]>> wrote: > > Date: Thu, 13 Nov 2025 01:35:47 +0000 > From: HAGIO KAZUHITO(萩尾 一仁) <[email protected] > <mailto:[email protected]>> > Subject: [Crash-utility] Bug report: "timer -r" option fails on > 6.18-rc kernels > To: "[email protected] > <mailto:[email protected]>" > <[email protected] > <mailto:[email protected]>> > Message-ID: <[email protected] > <mailto:[email protected]>> > Content-Type: text/plain; charset="utf-8" > > Hi Lianbo, Tao, > > The following error is observed on 6.18-rc kernels. > > crash> timer -r > > timer: invalid structure member offset: hrtimer_clock_base_get_time > FILE: kernel.c LINE: 7953 FUNCTION: dump_hrtimer_clock_base() > ... > > This is due to [1]. > > [1] > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=009eb5da29a9 > > <https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=009eb5da29a9> > > > Thank you for reporting the current issue, Kazu. > > > The "timer -r" option displays the base->get_time function, but now kernel > switches the function by base->clockid, which are CLOCK_* macro values. > We can copy the function names and macro values from the kernel, but if > they are modified, crash cannot follow the changes automatically. > > How should we fix this? or in the first place, maybe we can change what > the "timer -r" displays there, instead of .get_time function name. > For example, if we show enum hrtimer_base_type values by base->index: > > crash> timer -r > CPU: 0 HRTIMER_CPU_BASE: ffff8b7aefc20640 > CLOCK: 0 HRTIMER_CLOCK_BASE: ffff8b7aefc20680 > [HRTIMER_BASE_MONOTONIC] > ... > CLOCK: 1 HRTIMER_CLOCK_BASE: ffff8b7aefc206c0 > [HRTIMER_BASE_REALTIME] > ... > > What do you think? > > > I agree with you to display the clock type name there, this looks reasonable.
ok, thank you for the comment. > But I haven't investigated the details, can you show your code(if any)? I don't have a patch yet, I just edited an output as an example above. I think probably we can use dump_enumerator_list() to convert base->index to the enum name, I'll try it. Thanks, Kazu -- Crash-utility mailing list -- [email protected] To unsubscribe send an email to [email protected] https://${domain_name}/admin/lists/devel.lists.crash-utility.osci.io/ Contribution Guidelines: https://github.com/crash-utility/crash/wiki
