tigros/Whisperer

getfreegpumem returns negative value

Closed this issue · 8 comments

Vaiz commented

Hi @tigros,

I've noticed that the 'getfreegpumem' function returns a negative value, even though I have more than enough memory. When I hardcode a different value instead, everything works fine. It seems that the method used for calculation does not return valid numbers. 'initperfcounter' detects two counters: one of them returns 0, and the other returns a crazy big value similar to 1.125565E+10.

My config:
CPU: intel 13700k
GPU: AMD 7900xt
RAM: 64 GB

Please try C:\Windows\SysWOW64\LODCTR /R from elevated cmd prompt. get that out of the way.

Vaiz commented

Please try C:\Windows\SysWOW64\LODCTR /R from elevated cmd prompt. get that out of the way.

I've tried it. The issue persist even after reboot :(

Vaiz commented

I did a bit of testing, and it looks like the problem is on my side.
GPUmembyproc.exe I built returns wrong amount of total GPU memory.
It does that because I built win32 version. After I switched to x64 the problem has gone.

x86 build: Total GPU memory: 3,221,225,472
x64 build: Total GPU memory: 21,408,108,544

ha! i did the same thing last week #51 (comment)

thanks for the mods, it's true now that it saves the file list someone can click the Go button much quicker, good catch!

Do you have VS 2017 installed? It would be better for me, I'm on win 7 usually.

Vaiz commented

Do you have VS 2017 installed? It would be better for me, I'm on win 7 usually.

I use VS 2022, but the pull request doesn't contain any changes in project's configuration. It should be safe to merge it.

it's the project files, especially PlatformToolset, .net version changed too, inconvenient, can't you install VS 2017? or out of room?

Vaiz commented

I did some changes in project's files, but I rolled them back before making pull request.
You can check the final diff between main branch and mine: https://github.com/tigros/Whisperer/pull/61/files

If you want, I can make a clean branch and cherry-pick commits that doesn't contain unnecessary changes.

oh ok guess can merge then, thanks again!