On Tue 2020-08-11 12:40:22, Orson Zhai wrote: > From: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > > Timestamps printed in kernel log are retrieved by local_clock which reads > jiffies as a referrence clock source. > But it is diffcult to be synchronized with logs generated out of kernel, > say some remote processors (Modem) in the Soc of mobile phones. > Jiffies will be unchanged when system is in suspend mode but Modem will > not. > > So timestamps are required to be compensated with suspend time in some > complecated scenarios especially for Android system. As an initial > implementation was rejected by Linus, Thomas made this patch [1] 2 yeas > ago to replace ts_nsec with a struct ts that consists 3 types of time: > mono, boot and real. > > This is an updated version which comes from patch [1] written by Thomas > and suggestion [2] about VMCORE_INFO given by Linus. > > It provides the prerequisite code to print kernel logs with boot or real > timestamp in the future. > > This patch has been tested in qemu-x86-system. One problem is the timestamp > in kernel log will be printed [ 0.000000] for longer time than before. This would be a regression. People put huge effort into having early boot timestamps, see https://lore.kernel.org/lkml/20180719205545.16512-1-pasha.tatashin@xxxxxxxxxx/ Adding some active people from this patchset into CC. I wonder if we could have these early timestamps also in the mono clock. > 1 [ 0.000000] microcode: microcode updated early to revision 0x28, date = 2019-11-12 > 2 [ 0.000000] Linux version 5.8.0+ (root@spreadtrum) (gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0, GNU ld (GNU Binutils for Ubuntu) 2.30) #2 SMP Fri Aug 7 21:30:51 CST 2020 > 3 [ 0.000000] Command line: BOOT_IMAGE=/vmlinuz-5.8.0+ root=UUID=4d889bca-be18-433d-9b86-c1c1714cc293 ro quiet splash vt.handoff=1 > 4 [ 0.000000] KERNEL supported cpus: > 5 [ 0.000000] Intel GenuineIntel > .... > 223 [ 0.000000] Tracing variant of Tasks RCU enabled. > 224 [ 0.000000] rcu: RCU calculated value of scheduler-enlistment delay is 25 jiffies. > 225 [ 0.000000] rcu: Adjusting geometry for rcu_fanout_leaf=16, nr_cpu_ids=4 > 226 [ 0.000000] NR_IRQS: 524544, nr_irqs: 456, preallocated irqs: 16 > 227 [ 0.059662] random: crng done (trusting CPU's manufacturer) > ^^^^^^^^ > 228 [ 0.059662] Console: colour dummy device 80x25 > 229 [ 0.059662] printk: console [tty0] enabled > 230 [ 0.059662] ACPI: Core revision 20200528 > > compared to old log: > > 69 [ 0.000000] reserve setup_data: [mem 0x0000000100000000-0x000000021fdfffff] usable > 70 [ 0.000000] efi: EFI v2.31 by American Megatrends > 71 [ 0.000000] efi: ESRT=0xdbf69818 ACPI=0xdaef8000 ACPI 2.0=0xdaef8000 SMBIOS=0xf0480 > 72 [ 0.000000] SMBIOS 2.8 present. > 73 [ 0.000000] DMI: LENOVO ThinkCentre M4500t-N000/, BIOS FCKT58AUS 09/17/2014 > 74 [ 0.000000] tsc: Fast TSC calibration using PIT > 75 [ 0.000000] tsc: Detected 3292.477 MHz processor > 76 [ 0.000503] e820: update [mem 0x00000000-0x00000fff] usable ==> reserved > ^^^^^^^^ > 77 [ 0.000504] e820: remove [mem 0x000a0000-0x000fffff] usable > 78 [ 0.000510] last_pfn = 0x21fe00 max_arch_pfn = 0x400000000 > 79 [ 0.000513] MTRR default type: uncachable [...] > --- a/kernel/printk/printk.c > +++ b/kernel/printk/printk.c > @@ -366,7 +366,8 @@ enum log_flags { > }; > > struct printk_log { > - u64 ts_nsec; /* timestamp in nanoseconds */ > + /* Timestamps in nanoseconds */ > + struct timestamps ts; > u16 len; /* length of entire record */ > u16 text_len; /* length of text buffer */ > u16 dict_len; /* length of dictionary buffer */ > @@ -1090,7 +1091,9 @@ void log_buf_vmcoreinfo_setup(void) > * parse it and detect any changes to structure down the line. > */ > VMCOREINFO_STRUCT_SIZE(printk_log); > - VMCOREINFO_OFFSET(printk_log, ts_nsec); > + /* ts.mono used to be called "ts_nsec" */ > + VMCOREINFO_FIELD_OFFSET(printk_log, ts_nsec, ts.mono); > + VMCOREINFO_OFFSET(printk_log, ts); > VMCOREINFO_OFFSET(printk_log, len); > VMCOREINFO_OFFSET(printk_log, text_len); > VMCOREINFO_OFFSET(printk_log, dict_len); At least "crash" tool would need an update anyway. AFAIK, it checks the size of struct printk_log and refuses to read it when it changes. It means that the hack with VMCOREINFO_FIELD_OFFSET probably is not needed because we would need to update the crashdump-related tools anyway. Well, the timing is good. We are about to switch the printk ring buffer into a lockless one. It requires updating the crashdump tools as well. We could do this at the same time. The lockless ring buffer already is in linux-next. It is aimed for 5.10 or 5.11. We are still discussion handling of continuous lines that might need another change in the internal structures, see https://lore.kernel.org/r/20200717234818.8622-1-john.ogness@xxxxxxxxxxxxx It would be great to synchronize all these changes changes of the printk log buffer structures. Best Regards, Petr _______________________________________________ kexec mailing list kexec@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/kexec