theChaosCoder / VSRepoGUI

A GUI for VSRepo
https://forum.doom9.org/showthread.php?t=176313
MIT License
36 stars 3 forks source link

CPU core reporting is wrong. #4

Closed gcs8 closed 4 years ago

gcs8 commented 4 years ago

I have dual E5-2673 v4's so it should report 40 cores / 80 threads depending on how you are counting.

OS: Microsoft Windows 10 Pro Is 64Bit OS?: True GPU: NVIDIA Quadro P4000 CPU: Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz CPU Cores: 32

image

theChaosCoder commented 4 years ago

Seems to be the same problem as described here VSRepoGUI also uses System.Environment.ProcessorCount

This test version VSRepoGUI_cpu_core_test.zip should report the same numbers as shown in your image. Please test and report.

gcs8 commented 4 years ago

So, not quite, it looks like it is not counting / adding quantities. Looks like it is only counting GPU and CPU 0 but not 1.

image

theChaosCoder commented 4 years ago

Ahh yes Sockets = 2

Can you show me what the output looks like on your machine with this: VSRepoGUI_Win32_Processor_output.zip

It will list everything from the Win32_Processor WMI class. I need to know how the output looks like on your machine,

gcs8 commented 4 years ago

`/!\ Only Plugins and no Scripts are tested /!\ VapourSynth Video Processing Library Copyright (c) 2012-2020 Fredrik Mellbin Core R50 API R3.6 Options: -

OS: Microsoft Windows 10 Pro Is 64Bit OS: Yes GPU: NVIDIA Quadro P4000 CPU: Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz CPU Cores: 20 Logical Processors: 40

Win32_Processor Output

AddressWidth: 64 Architecture: 9 AssetTag: Availability: 3 Caption: Intel64 Family 6 Model 79 Stepping 1 Characteristics: 252 ConfigManagerErrorCode: ConfigManagerUserConfig: CpuStatus: 1 CreationClassName: Win32_Processor CurrentClockSpeed: 2295 CurrentVoltage: 7 DataWidth: 64 Description: Intel64 Family 6 Model 79 Stepping 1 DeviceID: CPU0 ErrorCleared: ErrorDescription: ExtClock: 100 Family: 179 InstallDate: L2CacheSize: 5120 L2CacheSpeed: L3CacheSize: 51200 L3CacheSpeed: 0 LastErrorCode: Level: 6 LoadPercentage: 100 Manufacturer: GenuineIntel MaxClockSpeed: 2295 Name: Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz NumberOfCores: 20 NumberOfEnabledCore: 20 NumberOfLogicalProcessors: 40 OtherFamilyDescription: PartNumber: PNPDeviceID: PowerManagementCapabilities: PowerManagementSupported: False ProcessorId: BFEBFBFF000406F1 ProcessorType: 3 Revision: 20225 Role: CPU SecondLevelAddressTranslationExtensions: False SerialNumber: SocketDesignation: CPU0 Status: OK StatusInfo: 3 Stepping: SystemCreationClassName: Win32_ComputerSystem SystemName: WASHU ThreadCount: 40 UniqueId: UpgradeMethod: 43 Version: VirtualizationFirmwareEnabled: False VMMonitorModeExtensions: False VoltageCaps: AddressWidth: 64 Architecture: 9 AssetTag: Availability: 3 Caption: Intel64 Family 6 Model 79 Stepping 1 Characteristics: 252 ConfigManagerErrorCode: ConfigManagerUserConfig: CpuStatus: 1 CreationClassName: Win32_Processor CurrentClockSpeed: 2295 CurrentVoltage: 7 DataWidth: 64 Description: Intel64 Family 6 Model 79 Stepping 1 DeviceID: CPU1 ErrorCleared: ErrorDescription: ExtClock: 100 Family: 179 InstallDate: L2CacheSize: 5120 L2CacheSpeed: L3CacheSize: 51200 L3CacheSpeed: 0 LastErrorCode: Level: 6 LoadPercentage: 100 Manufacturer: GenuineIntel MaxClockSpeed: 2295 Name: Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz NumberOfCores: 20 NumberOfEnabledCore: 20 NumberOfLogicalProcessors: 40 OtherFamilyDescription: PartNumber: PNPDeviceID: PowerManagementCapabilities: PowerManagementSupported: False ProcessorId: BFEBFBFF000406F1 ProcessorType: 3 Revision: 20225 Role: CPU SecondLevelAddressTranslationExtensions: False SerialNumber: SocketDesignation: CPU1 Status: OK StatusInfo: 3 Stepping: SystemCreationClassName: Win32_ComputerSystem SystemName: WASHU ThreadCount: 40 UniqueId: UpgradeMethod: 43 Version: VirtualizationFirmwareEnabled: False VMMonitorModeExtensions: False VoltageCaps: `

theChaosCoder commented 4 years ago

Last test I hope VSRepoGUI_cpu2.zip

gcs8 commented 4 years ago

woooo, guess my only other question is would it be worth your time to report the qty of CPUs/GPUs?

`/!\ Only Plugins and no Scripts are tested /!\ VapourSynth Video Processing Library Copyright (c) 2012-2020 Fredrik Mellbin Core R50 API R3.6 Options: -

OS: Microsoft Windows 10 Pro Is 64Bit OS: Yes GPU: NVIDIA Quadro P4000 CPU: Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz CPU Cores: 40 Logical Processors: 80`

theChaosCoder commented 4 years ago

Something like this?

GPU1: NVIDIA Quadro P4000
GPU2: NVIDIA GeForce GTX 1070
CPU: Intel(R) Xeon(R) CPU E5-2673 v4 @ 2.30GHz       
CPU Cores: 40
Logical Processors: 80
Sockets: 2 or CPUs: 2
gcs8 commented 4 years ago

Maybe something like this? (sorry for the bad pseudo code)


if GPUs = identical print Model x#
else print list

CPU: mfg + series + model @ speed + qty if <1
Cores #
LPs (threads)#
Sockets AND/OR NUMA nodes
RAM in GiB```
theChaosCoder commented 4 years ago

I don't want to show to much details. There are better tools for showing detailed cpu gpu information.

Please check if the output looks ok and it doesn't crash. Can't really test here with multiple cpus/gpus...

This is how it looks on my machine

OS: Microsoft Windows 10 Enterprise
64Bit OS: Yes
GPU: NVIDIA GeForce GTX 1070
CPU: AMD Ryzen 5 2600 Six-Core Processor            
CPU Cores: 6
Logical Processors: 12
RAM: 32 GB
gcs8 commented 4 years ago

Did you mean to attach another test file?

theChaosCoder commented 4 years ago

Yes :D VSRepoGUI_cpu3.zip

gcs8 commented 4 years ago

looks like it is not counting right

image

theChaosCoder commented 4 years ago

That's a lot of ram 😲 VSRepoGUI_cpu4.zip

gcs8 commented 4 years ago

Yeah, it is nice to have.

image

gcs8 commented 4 years ago

Cool, anything else I can test or make extra work for you? If not I will close this out.

theChaosCoder commented 4 years ago

hehe, I think you can close it. At least I don't miss any feature.