Refactor thread counts

[skmono]

Remove numa dependency and acquire cores/threads by parsing lscpu

[skmono]

@justalittlenoob What do you think of this approach? This PR can remove the numa dependency

[justalittlenoob]

Hi @skmono
This PR looks good.
I have only one concern. If using two different machines compile code and execute code separately, IPCL_THREAD_NUM may not be correct at runtime.

[skmono]

@justalittlenoob You're right. Also hard coded num_sockets could pose the same issue. Let me dig into it a bit more.

[justalittlenoob]

@justalittlenoob You're right. Also hard coded num_sockets could pose the same issue. Let me dig into it a bit more.

I have an idea FYI. How about set several options in the compilation options, such as IPCL_ COMPILE_ ONLY IPCL_ EXCUTE_ ONLY and IPCL_ COMPILE_ AND_ EXECUTE?
If this machine is only responsible for compiling, then we can put the detection of avx512, rdseed, rdrand, num_thread, etc. into the compilation period(IPCL_COMPILE_ONLY).
If this machine is only responsible for executing code(IPCL_EXECUTE_ONLY), then detecting those features in the cpp code.
etc..

[skmono]

@justalittlenoob I added a /proc/cpuinfo parser to detect number of sockets (nodes)

So there will be following cases:

Case	IPCL_THREAD_COUNT explicitly set	IPCL_THREAD_COUNT not set
IPCL_DETECT_CPU_RUNTIME=OFF	cpus = IPCL_THREAD_COUNT	cpus = std::thread::hardware_concurrency
	nodes = IPCL_NUM_NODES	nodes = IPCL_NUM_NODES
IPCL_DETECT_CPU_RUNTIME=ON	cpus = IPCL_THREAD_COUNT	cpus = std::thread::hardware_concurrency
	nodes from parser	nodes from parser

nodes CMake compile def can be found in:

pailliercryptolib/CMakeLists.txt

Line 105 in 745f02d

ipcl_get_core_thread_count(num_cores num_threads num_sockets)

pailliercryptolib/CMakeLists.txt

Lines 120 to 121 in 745f02d

	# set cpu socket count parsed from lscpu precompile
	add_compile_definitions(IPCL_NUM_NODES=${num_sockets})

nodes parser from:

pailliercryptolib/ipcl/include/ipcl/utils/util.hpp

Lines 78 to 81 in f121da2

	#ifdef IPCL_RUNTIME_DETECT_CPU_FEATURES
	static const linuxCPUInfo cpuinfo;
	static const linuxCPUInfo getLinuxCPUInfo() { return GetLinuxCPUInfo(); }
	#endif

pailliercryptolib/ipcl/utils/util.cpp

Lines 11 to 13 in f121da2

	#ifdef IPCL_RUNTIME_DETECT_CPU_FEATURES
	const linuxCPUInfo OMPUtilities::cpuinfo = OMPUtilities::getLinuxCPUInfo();
	#endif // IPCL_RUNTIME_DETECT_CPU_FEATURES

What do you think?

[skmono]

There is little difference in performance, as /proc/cpuinfo will be parsed only once

[justalittlenoob]

It's great.