Messages in this thread | | | From | Namhyung Kim <> | Date | Fri, 25 Jul 2014 00:42:04 +0900 | Subject | Re: [PATCHSET 0/6] perf tools: Honor column width setting |
| |
On Thu, Jul 24, 2014 at 10:59 PM, Arnaldo Carvalho de Melo <acme@kernel.org> wrote: > Em Wed, Jul 09, 2014 at 02:28:08PM +0900, Namhyung Kim escreveu: >> Hello, >> >> This patchset is to control perf report/top output column width by >> -w/--column-widths option so that it can fit into the terminal size. >> The -w option is there for perf report but it ignored by recent output >> field changed due to some reason. This patchset fixes it and supports >> perf top also. >> >> This is sometimes useful if your terminal is small and there's some >> C++ applications which have amazingly long symbol names. Without this >> patchset user might not see those symbols on TUI, since it maps >> left/right arrow keys to other functions. >> >> The -w option sets column width starting from the first column >> (overhead or optional overhead_children column unless -F option is >> given). It doesn't make sense to limit those overhead columns so it's >> not a hard-limit for them. But it *is* a hard-limit for other columns >> such as comm, dso, symbol, and so on. One can use 0 not to >> limit/force a width for those columns. > > Due to the issues pointed out, will wait for a v2 patchset, ok?
Okay. Will send v2 tomorrow.
Thanks, Namhyung
| |