Skip to content

Conversation

@adi-benz
Copy link
Contributor

@adi-benz adi-benz commented Jan 16, 2022

Description

Use ns.get_process_nspid() from granulate_utils, it supported older kernels too (kernels older than 4.1 that doesn't expose the NSpid through /proc/pid/status).

How Has This Been Tested?

Tested Java profiling correctly identifies hotspot's error file using the ns pid
Tested on Ubuntu 14.04 3.13.0-170-generic

Checklist:

@adi-benz adi-benz added the enhancement New feature or request label Jan 16, 2022
@adi-benz adi-benz requested a review from Jongy January 16, 2022 14:51
Copy link
Contributor

@Jongy Jongy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

Jongy
Jongy previously approved these changes Jan 17, 2022
Copy link
Contributor

@Jongy Jongy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, we'll just update granulate-utils once that's ready

@adi-benz adi-benz marked this pull request as ready for review January 19, 2022 17:57
@adi-benz adi-benz merged commit 4683673 into master Jan 19, 2022
@adi-benz adi-benz deleted the get-process-nspid-support-old-kernels branch January 19, 2022 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants