U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » U++ Library support » U++ MT-multithreading and servers » CPU_Cores() error

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

CPU_Cores() error [message #14546]

Sun, 02 March 2008 17:14

kodos
Messages: 111
Registered: March 2008

Experienced Member

Hi,

I have found one or two bugs in the function CPU_Cores() under Win32.

1)
The for loop in the 32 bit version of the function should only run trough 32 times not 64 times.

2)
Shouldn't we use the ProcessAffinityMask instead of the SystemAffinityMask to get the available cores?

Report message to a moderator

Re: CPU_Cores() error [message #14564 is a reply to message #14546]

Mon, 03 March 2008 20:11

mirek
Messages: 14257
Registered: November 2005

Ultimate Member

kodos wrote on Sun, 02 March 2008 11:14

Hi,

I have found one or two bugs in the function CPU_Cores() under Win32.

1)
The for loop in the 32 bit version of the function should only run trough 32 times not 64 times.

Correct and fixed (but actually not a bug, it worked).

Quote:

2)
Shouldn't we use the ProcessAffinityMask instead of the SystemAffinityMask to get the available cores?

Maybe... Should these be the samein most cases?

Mirek

[Updated on: Mon, 03 March 2008 20:12]

Report message to a moderator

Re: CPU_Cores() error [message #14573 is a reply to message #14564]

Mon, 03 March 2008 21:01

kodos
Messages: 111
Registered: March 2008

Experienced Member

luzr wrote on Mon, 03 March 2008 20:11

Quote:

2)
Shouldn't we use the ProcessAffinityMask instead of the SystemAffinityMask to get the available cores?

Maybe... Should these be the samein most cases?

I haven't tested it but I think that they are not the same if you assign the process to an explicit processor with the taskmanager.

And I think the return value of this function should return the cores it can use and not the cores that are in the system..

Report message to a moderator