Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for per-vGPU mode profiling #166

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jiaopengyuan
Copy link
Contributor

These 3 commits are about per virtual GPU counter profiling, which are usage for graphics workloads running inside virtual machines. User can select a specific vGPU on the web ui or through gputop-csv to observe each vGPU performance.

Let me briefly introduce my idea. If we create 2 vGPUs, there are 3 states. ctx_mode = ["Global", "vGPU1", "vGPU2"], and corresponding hw_id = ["0", hw_id1, hw_id2]. The "Global" state means the current state, without applying my patch, that is, it shows the whole counter values for any context id. All of the vgpu_id and hw_id starts from non-zero value. Hence I define the hw_id as 0 to map the "Global" state. So in the beginning without selecting any hw id, the current_hw_id is 0, it works in the "Global" state.

For profiling counters in per-vGPU mode, this menu requests
for vgpu_id and hw_id of each vGPU when vGPUs are created

Signed-off-by: Pengyuan Jiao <[email protected]>
When a vGPU is selected on the menu, the related hw_id is
passed to client-c through webui, then per-vgpu filtering
by hw_id is achieved in gputop-client-c.c in userspace.

Signed-off-by: Pengyuan Jiao <[email protected]>
add new option "--vgpu" for gputop-csv to print counter
data in per-vGPU mode.

Signed-off-by: Pengyuan Jiao <[email protected]>
@djdeath
Copy link
Collaborator

djdeath commented Jan 30, 2018

The new UI has gained the ability to list processes associated to a hw_id (requires kernel 4.15+).
I supposed that would display the name of the hypervisor process in your use case.
Is that enough?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants