[Crash-utility] Re: Bug report: "timer -r" option fails on 6.18-rc kernels

On 2025/11/13 19:20, lijiang wrote: > On Thu, Nov 13, 2025 at 11:10 AM < devel-request(a)lists.crash-utility.osci.io <mailto: devel-request(a)lists.crash-utility.osci.io&gt;&gt; wrote: > > Date: Thu, 13 Nov 2025 01:35:47 +0000 > From: HAGIO KAZUHITO(萩尾 一仁) <k-hagio-ab(a)nec.com <mailto: k-hagio-ab(a)nec.com&gt;&gt; > Subject: [Crash-utility] Bug report: "timer -r" option fails on > 6.18-rc kernels > To: &quot;devel(a)lists.crash-utility.osci.io <mailto: devel(a)lists.crash-utility.osci.io&gt;&quot; > <devel(a)lists.crash-utility.osci.io <mailto: devel(a)lists.crash-utility.osci.io&gt;&gt; > Message-ID: <011ccc03-c0ca-4a26-8ffe-97081e2d2dd6(a)nec.com <mailto: 011ccc03-c0ca-4a26-8ffe-97081e2d2dd6(a)nec.com&gt;&gt; > 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... < https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit... > > > > 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