Re: [tip:perf/urgent] perf symbols: Handle /proc/sys/kernel/kptr_restrict

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

 



* Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:

> > Had you done this you'd have immediately noticed that the individual 
> > lines are *way* too large to fit on even large terminals. Please 
> > break such messages in a much shorter fashion - we want the error 
> > messages we output to be readable even on relatively small (80col) 
> > terminals.
> 
> But it would reflow, no?

Not on all terminals. For example my terminal will just cut anything 
beyond the col limit:

 # perf report --stdio
 No kallsyms or vmlinux with build-id 122214021a666675f6e5ff97d70a85ce7139c0e7 was found
 [kernel.kallsyms] with build id 122214021a666675f6e5ff97d70a85ce7139c0e7 not found, continuing without s
 Kernel address maps (/proc/{kallsyms,modules}) were restricted, check /proc/sys/kernel/kptr_restrict bef

So we generally want to generate error messages to look good on col80 
- it should not be hard here, right?

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Stable Commits]     [Linux Stable Kernel]     [Linux Kernel]     [Linux USB Devel]     [Linux Video &Media]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux