-----Original Message----- > On the other hand, I would like to make sure that > users who don't know about "log -s" don't overlook > the logs of safe buffers as possible. > > Is it acceptable to make "log" output look like the following, in addition to > implementing "log -s"? > > --- > [ 6.261629] Key type id_resolver registered > [ 6.261924] Key type id_legacy registered > [1717970.628952] safebuffer: loading out-of-tree module taints kernel. > [1717970.629299] safebuffer: module verification failed: signature and/or required key missing - tainting > kernel > [nmi_print_seq] message2 message2 message2 message2 > [nmi_print_seq] Uhhuh. NMI received for unknown reason 20 on CPU 0. > [nmi_print_seq] Do you have a strange power saving mode enabled? > [nmi_print_seq] Dazed and confused, but trying to continue > --- OK, makes sense. This looks better to me than the next one. Please add help description about this, too. > > or > > --- > [ 6.261629] Key type id_resolver registered > [ 6.261924] Key type id_legacy registered > [1717970.628952] safebuffer: loading out-of-tree module taints kernel. > [1717970.629299] safebuffer: module verification failed: signature and/or required key missing - tainting > kernel > message2 message2 message2 message2 > Uhhuh. NMI received for unknown reason 20 on CPU 0. > Do you have a strange power saving mode enabled? > Dazed and confused, but trying to continue > --- > > > > > + > > > + n = (len <= buffer_size) ? len : buffer_size; > > > + for (i = 0, p = buffer; i < n; i++, p++) { > > > + if (*p == 0x1) { //SOH > > > + i++; p++; > > > + continue; > > > > According to include/linux/kern_levels.h, the byte right after the SOH > > can be printed as log level? > > Ok, I'll add log level to the output when -m is specified to log command. Thanks. Kazu -- Crash-utility mailing list Crash-utility@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/crash-utility