linux-kernel - Re: [PATCH 0/3] perf annotate-data: Implement folding in TUI browser

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Zrp0m53X7STRB8gU@x1>
Date: Mon, 12 Aug 2024 17:46:19 -0300
From: Arnaldo Carvalho de Melo <acme@...nel.org>
To: Namhyung Kim <namhyung@...nel.org>
Cc: Ian Rogers <irogers@...gle.com>, Kan Liang <kan.liang@...ux.intel.com>,
	Jiri Olsa <jolsa@...nel.org>,
	Adrian Hunter <adrian.hunter@...el.com>,
	Peter Zijlstra <peterz@...radead.org>,
	Ingo Molnar <mingo@...nel.org>, LKML <linux-kernel@...r.kernel.org>,
	linux-perf-users@...r.kernel.org
Subject: Re: [PATCH 0/3] perf annotate-data: Implement folding in TUI browser

On Mon, Aug 12, 2024 at 12:44:44PM -0700, Namhyung Kim wrote:
> Hello,
> 
> I've implemented the folding logic to expand/collapse the current entry
> in the TUI data type browser.  Users can use 'e' or 'E' to toggle the
> entry (with or without children, respectively).
> 
> Ax example output looks like below.
> 
>   Annotate type: 'struct socket' (1 samples)
>         Percent     Offset       Size  Field
>   -      100.00          0        128  struct socket {                                  ◆
>            0.00          0          4      socket_state   state;                        ▒
>            0.00          4          2      short int      type;                         ▒
>            0.00          8          8      long unsigned int      flags;                ▒
>            0.00         16          8      struct file*   file;                         ▒
>          100.00         24          8      struct sock*   sk;                           ▒
>            0.00         32          8      struct proto_ops*      ops;                  ▒
>   -        0.00         64         64      struct socket_wq       wq {                  ▒
>   -        0.00         64         24          wait_queue_head_t  wait {                ▒
>   +        0.00         64          4              spinlock_t     lock;                 ▒
>   -        0.00         72         16              struct list_head       head {        ▒
>            0.00         72          8                  struct list_head*  next;         ▒
>            0.00         80          8                  struct list_head*  prev;         ▒
>                                                    };                                   ▒
>                                                };                                       ▒
>            0.00         88          8          struct fasync_struct*      fasync_list;  ▒
>            0.00         96          8          long unsigned int  flags;                ▒
>   +        0.00        104         16          struct callback_head       rcu;          ▒
>                                            };                                           ▒
>                                        };                                               ▒
> 
> It hides the detail of spinlock_t and struct callback_head.

Thanks, useful feature! All tested and applied.

- Arnaldo