[PATCH 0/5 v1] Processing java JIT dumps from containers

Arnaldo Carvalho de Melo posted 5 patches 1 year ago
tools/perf/Documentation/perf-config.txt |  2 +-
tools/perf/util/jitdump.c                | 15 ++++++++++++---
tools/perf/util/namespaces.c             |  7 ++++++-
tools/perf/util/namespaces.h             |  3 ++-
4 files changed, 21 insertions(+), 6 deletions(-)
[PATCH 0/5 v1] Processing java JIT dumps from containers
Posted by Arnaldo Carvalho de Melo 1 year ago
From: Arnaldo Carvalho de Melo <acme@redhat.com>

Hi,

	This is trying to move the needle on supporting jitdump done
from a container while perf is running from the outside, that is not
working right now.

	I tried to collect as much details on what is being done in the
commit logs to document further work that needs to be done to support
such a scenario in a streamlined way, but what in this patchkit at least
helped in a real case, allowing us to get 'perf annotate' to work and
confirm suspicions.

- Arnaldo

Arnaldo Carvalho de Melo (5):
  perf config: Fix trival typo 'an' -> 'can'
  perf jitdump: Accept jitdump mmaps emitted from inside containers
  perf namespaces: Introduce nsinfo__set_in_pidns()
  perf jitdump: Fixup in_pidns member when java agent and 'perf record' are not in the same pidns
  perf namespaces: Fixup the nsinfo__in_pidns() return type, its bool

 tools/perf/Documentation/perf-config.txt |  2 +-
 tools/perf/util/jitdump.c                | 15 ++++++++++++---
 tools/perf/util/namespaces.c             |  7 ++++++-
 tools/perf/util/namespaces.h             |  3 ++-
 4 files changed, 21 insertions(+), 6 deletions(-)

-- 
2.47.0
Re: [PATCH 0/5 v1] Processing java JIT dumps from containers
Posted by Arnaldo Carvalho de Melo 11 months, 4 weeks ago
On Fri, Dec 06, 2024 at 05:48:23PM -0300, Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> Hi,
> 
> 	This is trying to move the needle on supporting jitdump done
> from a container while perf is running from the outside, that is not
> working right now.
> 
> 	I tried to collect as much details on what is being done in the
> commit logs to document further work that needs to be done to support
> such a scenario in a streamlined way, but what in this patchkit at least
> helped in a real case, allowing us to get 'perf annotate' to work and
> confirm suspicions.

I'm going to apply this to perf-tools-next, please holler if you
disagree.

- Arnaldo
 
> - Arnaldo
> 
> Arnaldo Carvalho de Melo (5):
>   perf config: Fix trival typo 'an' -> 'can'
>   perf jitdump: Accept jitdump mmaps emitted from inside containers
>   perf namespaces: Introduce nsinfo__set_in_pidns()
>   perf jitdump: Fixup in_pidns member when java agent and 'perf record' are not in the same pidns
>   perf namespaces: Fixup the nsinfo__in_pidns() return type, its bool
> 
>  tools/perf/Documentation/perf-config.txt |  2 +-
>  tools/perf/util/jitdump.c                | 15 ++++++++++++---
>  tools/perf/util/namespaces.c             |  7 ++++++-
>  tools/perf/util/namespaces.h             |  3 ++-
>  4 files changed, 21 insertions(+), 6 deletions(-)
> 
> -- 
> 2.47.0
Re: [PATCH 0/5 v1] Processing java JIT dumps from containers
Posted by Arnaldo Carvalho de Melo 11 months, 1 week ago
On Mon, Dec 23, 2024 at 04:51:49PM -0300, Arnaldo Carvalho de Melo wrote:
> On Fri, Dec 06, 2024 at 05:48:23PM -0300, Arnaldo Carvalho de Melo wrote:
> > From: Arnaldo Carvalho de Melo <acme@redhat.com>
> > 
 > Hi,
> > 
> > 	This is trying to move the needle on supporting jitdump done
> > from a container while perf is running from the outside, that is not
> > working right now.
> > 
> > 	I tried to collect as much details on what is being done in the
> > commit logs to document further work that needs to be done to support
> > such a scenario in a streamlined way, but what in this patchkit at least
> > helped in a real case, allowing us to get 'perf annotate' to work and
> > confirm suspicions.
> 
> I'm going to apply this to perf-tools-next, please holler if you
> disagree.

Applied.

- Arnaldo