Re: [PATCH v2] log: output logs of printk safe buffers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----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




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux