Skip to content

Commit

Permalink
nvapi-gpu: Do not include current usage in current available memory
Browse files Browse the repository at this point in the history
  • Loading branch information
jp7677 committed Aug 17, 2024
1 parent b33a244 commit 283e5f9
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions src/nvapi_gpu.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -424,7 +424,7 @@ extern "C" {
pMemoryInfoV2->systemVideoMemory = memoryInfo.DedicatedSystemMemory;
pMemoryInfoV2->sharedSystemMemory = memoryInfo.SharedSystemMemory;
pMemoryInfoV2->availableDedicatedVideoMemory = memoryInfo.DedicatedVideoMemory - memoryInfo.ReservedVideoMemory; // See comment above
pMemoryInfoV2->curAvailableDedicatedVideoMemory = memoryBudgetInfo.Budget - memoryBudgetInfo.CurrentUsage;
pMemoryInfoV2->curAvailableDedicatedVideoMemory = memoryBudgetInfo.Budget;
break;
}
case NV_DISPLAY_DRIVER_MEMORY_INFO_VER_3: {
Expand All @@ -433,7 +433,7 @@ extern "C" {
pMemoryInfoV3->systemVideoMemory = memoryInfo.DedicatedSystemMemory;
pMemoryInfoV3->sharedSystemMemory = memoryInfo.SharedSystemMemory;
pMemoryInfoV3->availableDedicatedVideoMemory = memoryInfo.DedicatedVideoMemory - memoryInfo.ReservedVideoMemory; // See comment above
pMemoryInfoV3->curAvailableDedicatedVideoMemory = memoryBudgetInfo.Budget - memoryBudgetInfo.CurrentUsage;
pMemoryInfoV3->curAvailableDedicatedVideoMemory = memoryBudgetInfo.Budget;
pMemoryInfoV3->dedicatedVideoMemoryEvictionsSize = 0;
pMemoryInfoV3->dedicatedVideoMemoryEvictionCount = 0;
break;
Expand Down Expand Up @@ -473,7 +473,7 @@ extern "C" {
pMemoryInfoV1->sharedSystemMemory = memoryInfo.SharedSystemMemory;
// See comment in NvAPI_GPU_GetMemoryInfo
pMemoryInfoV1->availableDedicatedVideoMemory = memoryInfo.DedicatedVideoMemory - memoryInfo.ReservedVideoMemory;
pMemoryInfoV1->curAvailableDedicatedVideoMemory = memoryBudgetInfo.Budget - memoryBudgetInfo.CurrentUsage;
pMemoryInfoV1->curAvailableDedicatedVideoMemory = memoryBudgetInfo.Budget;
pMemoryInfoV1->dedicatedVideoMemoryEvictionsSize = 0;
pMemoryInfoV1->dedicatedVideoMemoryEvictionCount = 0;
pMemoryInfoV1->dedicatedVideoMemoryPromotionsSize = 0;
Expand Down
6 changes: 3 additions & 3 deletions tests/nvapi_sysinfo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -580,7 +580,7 @@ TEST_CASE("Sysinfo methods succeed", "[.sysinfo]") {
REQUIRE(info.systemVideoMemory == 1014 * 1024);
REQUIRE(info.sharedSystemMemory == 16382 * 1024);
REQUIRE(info.availableDedicatedVideoMemory == 8191 * 1024);
REQUIRE(info.curAvailableDedicatedVideoMemory == 3072 * 1024);
REQUIRE(info.curAvailableDedicatedVideoMemory == 4096 * 1024);
}

SECTION("GetMemoryInfo (V3) returns OK") {
Expand All @@ -591,7 +591,7 @@ TEST_CASE("Sysinfo methods succeed", "[.sysinfo]") {
REQUIRE(info.systemVideoMemory == 1014 * 1024);
REQUIRE(info.sharedSystemMemory == 16382 * 1024);
REQUIRE(info.availableDedicatedVideoMemory == 8191 * 1024);
REQUIRE(info.curAvailableDedicatedVideoMemory == 3072 * 1024);
REQUIRE(info.curAvailableDedicatedVideoMemory == 4096 * 1024);
REQUIRE(info.dedicatedVideoMemoryEvictionsSize == 0);
REQUIRE(info.dedicatedVideoMemoryEvictionCount == 0);
}
Expand Down Expand Up @@ -639,7 +639,7 @@ TEST_CASE("Sysinfo methods succeed", "[.sysinfo]") {
REQUIRE(info.systemVideoMemory == 1014 * 1024);
REQUIRE(info.sharedSystemMemory == 16382 * 1024);
REQUIRE(info.availableDedicatedVideoMemory == 8191 * 1024);
REQUIRE(info.curAvailableDedicatedVideoMemory == 3072 * 1024);
REQUIRE(info.curAvailableDedicatedVideoMemory == 4096 * 1024);
REQUIRE(info.dedicatedVideoMemoryEvictionsSize == 0);
REQUIRE(info.dedicatedVideoMemoryEvictionCount == 0);
REQUIRE(info.dedicatedVideoMemoryPromotionsSize == 0);
Expand Down

0 comments on commit 283e5f9

Please sign in to comment.