[PR #89] [MERGED] fix(api/libnvml): fix removal for process info v3 APIs on the upstream 535.98 driver #155

New issue

Closed

opened 2026-05-05 03:26:55 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 03:26:55 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/XuehaiPan/nvitop/pull/89
Author: @XuehaiPan
Created: 8/16/2023
Status: ✅ Merged
Merged: 8/17/2023
Merged by: @XuehaiPan

Base: main ← Head: fix-process-api

📝 Commits (2)

4e448c3 fix(api/libnvml): fix upstream removal for process info v3 APIs on 535.98 driver
c89acaf docs(CHANGELOG): update CHANGELOG

📊 Changes

2 files changed (+30 additions, -7 deletions)

View changed files

📝 CHANGELOG.md (+1 -1)
📝 nvitop/api/libnvml.py (+29 -6)

📄 Description

Issue Type

Bug fix

Description

Add an extra handler to check the removal of the process info v3 APIs.

Copied from https://github.com/XuehaiPan/nvitop/issues/88#issuecomment-1680052592:

Version change:

495.46 -> 510.39.01: b2f0e7f437

Add process info v3 APIs but use v2 nvmlProcessInfo_st struct type
default:
- nvmlDeviceGetComputeRunningProcesses -> nvmlDeviceGetComputeRunningProcesses_v3
- nvmlProcessInfo_st -> nvmlProcessInfo_v2_st

530.41.03 -> 535.43.02: 39c3e28e84

Process info v3 APIs use v3 nvmlProcessInfo_st struct type without a version bump
default:
- nvmlDeviceGetComputeRunningProcesses -> nvmlDeviceGetComputeRunningProcesses_v3
- nvmlProcessInfo_st -> nvmlProcessInfo_v3_st

535.86.05 -> 535.98: 0cb3beffa0

Remove process info v3 APIs and v3 nvmlProcessInfo_st struct type
default:
- nvmlDeviceGetComputeRunningProcesses -> nvmlDeviceGetComputeRunningProcesses_v2
- nvmlProcessInfo_st -> nvmlProcessInfo_v2_st

UPDATE:

535.98 -> 535.104.05: 74cae7fa6a

Re-add process info v3 APIs but use v2 nvmlProcessInfo_st struct type
default:
- nvmlDeviceGetComputeRunningProcesses -> nvmlDeviceGetComputeRunningProcesses_v3
- nvmlProcessInfo_st -> nvmlProcessInfo_v2_st

Motivation and Context

Fixes #88

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/XuehaiPan/nvitop/pull/89 **Author:** [@XuehaiPan](https://github.com/XuehaiPan) **Created:** 8/16/2023 **Status:** ✅ Merged **Merged:** 8/17/2023 **Merged by:** [@XuehaiPan](https://github.com/XuehaiPan) **Base:** `main` ← **Head:** `fix-process-api` --- ### 📝 Commits (2) - [`4e448c3`](https://github.com/XuehaiPan/nvitop/commit/4e448c344f771560aa9a4b59bc266dd174ca7aac) fix(api/libnvml): fix upstream removal for process info v3 APIs on 535.98 driver - [`c89acaf`](https://github.com/XuehaiPan/nvitop/commit/c89acafc1ec6387f32dd123e6a9e9b35a162d768) docs(CHANGELOG): update CHANGELOG ### 📊 Changes **2 files changed** (+30 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -1) 📝 `nvitop/api/libnvml.py` (+29 -6) </details> ### 📄 Description  #### Issue Type  - Bug fix #### Description  Add an extra handler to check the removal of the process info v3 APIs. Copied from https://github.com/XuehaiPan/nvitop/issues/88#issuecomment-1680052592: Version change: 495.46 -> 510.39.01: https://github.com/NVIDIA/nvidia-settings/commit/b2f0e7f437c42d92ed58120ec8d880f5f4b90d60 - Add process info v3 APIs but use v2 `nvmlProcessInfo_st` struct type - default: - `nvmlDeviceGetComputeRunningProcesses` -> `nvmlDeviceGetComputeRunningProcesses_v3` - `nvmlProcessInfo_st` -> `nvmlProcessInfo_v2_st` 530.41.03 -> 535.43.02: https://github.com/NVIDIA/nvidia-settings/commit/39c3e28e84f3ffb034abaf1ae92dbb570c207d05 - Process info v3 APIs use v3 `nvmlProcessInfo_st` struct type without a version bump - default: - `nvmlDeviceGetComputeRunningProcesses` -> `nvmlDeviceGetComputeRunningProcesses_v3` - `nvmlProcessInfo_st` -> `nvmlProcessInfo_v3_st` 535.86.05 -> 535.98: https://github.com/NVIDIA/nvidia-settings/commit/0cb3beffa0cb8a1f8cb405291b11a1e2eb7a4786 - Remove process info v3 APIs and v3 `nvmlProcessInfo_st` struct type - default: - `nvmlDeviceGetComputeRunningProcesses` -> `nvmlDeviceGetComputeRunningProcesses_v2` - `nvmlProcessInfo_st` -> `nvmlProcessInfo_v2_st` UPDATE: 535.98 -> 535.104.05: https://github.com/NVIDIA/nvidia-settings/commit/74cae7fa6a3da595a1bd87918ef0a67bb4326925 - Re-add process info v3 APIs but use v2 `nvmlProcessInfo_st` struct type - default: - `nvmlDeviceGetComputeRunningProcesses` -> `nvmlDeviceGetComputeRunningProcesses_v3` - `nvmlProcessInfo_st` -> `nvmlProcessInfo_v2_st` #### Motivation and Context    Fixes #88 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>