Boost C++ Libraries: Ticket #10788: GetLogicalProcessor isn't available for Windows platform less or equals to 0x0502

attachment set

steve.tousignant@… — Wed, 12 Nov 2014 19:47:04 GMT

attachment → boost-1.57.0-WinXP-SP2-PhysicalConcurrency-Return0.patch

Proposed patch for boost threading

component changed

viboes — Sun, 07 Jun 2015 08:51:29 GMT

component threads → thread

Please, don't use the threads component but the thread one.

owner, status changed

viboes — Sun, 28 Jun 2015 09:07:10 GMT

owner changed from Anthony Williams to viboes
status new → assigned

The current code is

    unsigned thread::physical_concurrency() BOOST_NOEXCEPT
    {
#if BOOST_PLAT_WINDOWS_RUNTIME || (defined(__MINGW32__) && !defined(__MINGW64_VERSION_MAJOR))
        return hardware_concurrency();
#else
        unsigned cores = 0;
        DWORD size = 0;
        GetLogicalProcessorInformation(NULL, &size);
        if (ERROR_INSUFFICIENT_BUFFER != GetLastError())
            return 0;
        std::vector<SYSTEM_LOGICAL_PROCESSOR_INFORMATION> buffer(size);
        if (GetLogicalProcessorInformation(&buffer.front(), &size) == FALSE)
            return 0;
        const size_t Elements = size / sizeof(SYSTEM_LOGICAL_PROCESSOR_INFORMATION);
        for (size_t i = 0; i < Elements; ++i) {
            if (buffer[i].Relationship == RelationProcessorCore)
                ++cores;
        }
        return cores;
#endif
    }

Does it works for you?

viboes — Sat, 05 Sep 2015 14:45:28 GMT

PING !!!

type changed

viboes — Wed, 23 Sep 2015 22:48:11 GMT

type Bugs → Support Requests

type changed

viboes — Sun, 27 Sep 2015 13:01:48 GMT

type Support Requests → Bugs

milestone changed

viboes — Sun, 27 Sep 2015 13:18:36 GMT

milestone To Be Determined → Boost 1.60.0

#11683 describes the history of this function and how we rolled back a fix.

I've update the develop branch.

https://github.com/boostorg/thread/commit/9a8759a3dda2c63342d55d5e97ed906826cee981

Daniel — Mon, 28 Sep 2015 07:07:56 GMT

Please also take into account the issue with comments after preprocessor continuation for VS2010 (comment:6:ticket:10824): commit/8e1be55c6d560c44fb07cc497a3480527957576c

status changed; resolution set

viboes — Wed, 21 Oct 2015 22:17:52 GMT

status assigned → closed
resolution → fixed