iovisor / iovisor/bcc

Adding new feature to track user all processes on memleak

Open
#4,296 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
22.7k
Forks
4.1k
Avg merge
10d 4h
Merged PRs (30d)
3

Description

Current memleak need to specify the user process pid or command to track all user processes.
Which means, it could track only one user process at a time.
How about add `-u` option for memleak to track every user process memory allocation?
Or make new tool name `allocsnoop`?

It seems pretty hard to add `-u` option on memleak,
since memleak uses `address` as a key of `allocs` `BPF_HASH` but address could conflict between user processes.
It could be possible to implement `-u` option easily, if the key of `combined_allocs` `BPF_HASH`(=`stack_id`) doesn't conflict between user processes.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the memleak implementation and the allocs and combined_allocs BPF_HASH definitions mentioned in the issue. Compare the proposed -u option with the alternative allocsnoop tool, then establish a decided design for tracking every process owned by a user without key collisions. Done should include an agreed implementation scope and validation for multi-process tracking.

Written by the indexing model from the issue text.

Assessment

Tech stack
c, linux
Domain
devtools, operating-systems
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.