On Mon, Nov 17, 2025 at 3:29 PM HAGIO KAZUHITO(萩尾 一仁) <[email protected]> wrote:
> 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. > Good to know, thank you, Kazu. Lianbo > 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
