Browse Source
Merge pull request #4786 from ClaymorePT/boost_1.65.0_r1
boost: 1.65.0 Update + Fixeslilik-openwrt-22.03
champtar
7 years ago
committed by
GitHub
GitHub
5 changed files with 395 additions and 121 deletions
Split View
Diff Options
-
+39 -14libs/boost/Makefile
-
+344 -0libs/boost/patches/010-boost-1.65.0_after_release_patches.patch
-
+0 -26libs/boost/patches/01_fiber_fix.patch
-
+12 -0libs/boost/patches/020-fiber_fix_0506e46640989e1f919388ad36890bd6063cd43a.patch
-
+0 -81libs/boost/patches/02_fix_context_overload_ambiguity.patch
+ 39
- 14
libs/boost/Makefile
View File
+ 344
- 0
libs/boost/patches/010-boost-1.65.0_after_release_patches.patch
View File
| @ -0,0 +1,344 @@ | |||
| Index: boost_1_65_0/boost/context/continuation_fcontext.hpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/boost/context/continuation_fcontext.hpp | |||
| +++ boost_1_65_0/boost/context/continuation_fcontext.hpp | |||
| @@ -95,7 +95,7 @@ transfer_t context_ontop( transfer_t t) | |||
| t.data = nullptr; | |||
| Ctx c{ t.fctx }; | |||
| // execute function, pass continuation via reference | |||
| - fn( std::move( c) ); | |||
| + c = fn( std::move( c) ); | |||
| #if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| return { exchange( c.fctx_, nullptr), nullptr }; | |||
| #else | |||
| Index: boost_1_65_0/boost/context/continuation_ucontext.hpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/boost/context/continuation_ucontext.hpp | |||
| +++ boost_1_65_0/boost/context/continuation_ucontext.hpp | |||
| @@ -84,18 +84,18 @@ static void entry_func( void * data) noe | |||
| struct BOOST_CONTEXT_DECL activation_record { | |||
| thread_local static activation_record * current_rec; | |||
| - ucontext_t uctx{}; | |||
| - stack_context sctx{}; | |||
| - bool main_ctx{ true }; | |||
| - activation_record * from{ nullptr }; | |||
| - std::function< void(activation_record*&) > ontop{}; | |||
| - bool terminated{ false }; | |||
| - bool force_unwind{ false }; | |||
| + ucontext_t uctx{}; | |||
| + stack_context sctx{}; | |||
| + bool main_ctx{ true }; | |||
| + activation_record * from{ nullptr }; | |||
| + std::function< activation_record*(activation_record*&) > ontop{}; | |||
| + bool terminated{ false }; | |||
| + bool force_unwind{ false }; | |||
| #if defined(BOOST_USE_ASAN) | |||
| - void * fake_stack{ nullptr }; | |||
| - void * stack_bottom{ nullptr }; | |||
| - std::size_t stack_size{ 0 }; | |||
| - bool started{ false }; | |||
| + void * fake_stack{ nullptr }; | |||
| + void * stack_bottom{ nullptr }; | |||
| + std::size_t stack_size{ 0 }; | |||
| + bool started{ false }; | |||
| #endif | |||
| static activation_record *& current() noexcept; | |||
| @@ -168,20 +168,30 @@ struct BOOST_CONTEXT_DECL activation_rec | |||
| current()->ontop = std::bind( | |||
| [](typename std::decay< Fn >::type & fn, activation_record *& ptr){ | |||
| Ctx c{ ptr }; | |||
| - fn( std::move( c) ); | |||
| + c = fn( std::move( c) ); | |||
| if ( ! c) { | |||
| ptr = nullptr; | |||
| } | |||
| +#if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| + return exchange( c.ptr_, nullptr); | |||
| +#else | |||
| + return std::exchange( c.ptr_, nullptr); | |||
| +#endif | |||
| }, | |||
| std::forward< Fn >( fn), | |||
| std::placeholders::_1); | |||
| #else | |||
| current()->ontop = [fn=std::forward<Fn>(fn)](activation_record *& ptr){ | |||
| Ctx c{ ptr }; | |||
| - fn( std::move( c) ); | |||
| + c = fn( std::move( c) ); | |||
| if ( ! c) { | |||
| ptr = nullptr; | |||
| } | |||
| +#if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| + return exchange( c.ptr_, nullptr); | |||
| +#else | |||
| + return std::exchange( c.ptr_, nullptr); | |||
| +#endif | |||
| }; | |||
| #endif | |||
| #if defined(BOOST_USE_SEGMENTED_STACKS) | |||
| @@ -408,7 +418,7 @@ public: | |||
| if ( BOOST_UNLIKELY( detail::activation_record::current()->force_unwind) ) { | |||
| throw detail::forced_unwind{ ptr}; | |||
| } else if ( BOOST_UNLIKELY( nullptr != detail::activation_record::current()->ontop) ) { | |||
| - detail::activation_record::current()->ontop( ptr); | |||
| + ptr = detail::activation_record::current()->ontop( ptr); | |||
| detail::activation_record::current()->ontop = nullptr; | |||
| } | |||
| return continuation{ ptr }; | |||
| @@ -426,7 +436,7 @@ public: | |||
| if ( BOOST_UNLIKELY( detail::activation_record::current()->force_unwind) ) { | |||
| throw detail::forced_unwind{ ptr}; | |||
| } else if ( BOOST_UNLIKELY( nullptr != detail::activation_record::current()->ontop) ) { | |||
| - detail::activation_record::current()->ontop( ptr); | |||
| + ptr = detail::activation_record::current()->ontop( ptr); | |||
| detail::activation_record::current()->ontop = nullptr; | |||
| } | |||
| return continuation{ ptr }; | |||
| Index: boost_1_65_0/boost/context/continuation_winfib.hpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/boost/context/continuation_winfib.hpp | |||
| +++ boost_1_65_0/boost/context/continuation_winfib.hpp | |||
| @@ -65,13 +65,13 @@ static VOID WINAPI entry_func( LPVOID da | |||
| struct BOOST_CONTEXT_DECL activation_record { | |||
| thread_local static activation_record * current_rec; | |||
| - LPVOID fiber{ nullptr }; | |||
| - stack_context sctx{}; | |||
| - bool main_ctx{ true }; | |||
| - activation_record * from{ nullptr }; | |||
| - std::function< void(activation_record*&) > ontop{}; | |||
| - bool terminated{ false }; | |||
| - bool force_unwind{ false }; | |||
| + LPVOID fiber{ nullptr }; | |||
| + stack_context sctx{}; | |||
| + bool main_ctx{ true }; | |||
| + activation_record * from{ nullptr }; | |||
| + std::function< activation_record*(activation_record*&) > ontop{}; | |||
| + bool terminated{ false }; | |||
| + bool force_unwind{ false }; | |||
| static activation_record *& current() noexcept; | |||
| @@ -142,20 +142,30 @@ struct BOOST_CONTEXT_DECL activation_rec | |||
| current()->ontop = std::bind( | |||
| [](typename std::decay< Fn >::type & fn, activation_record *& ptr){ | |||
| Ctx c{ ptr }; | |||
| - fn( std::move( c) ); | |||
| + c = fn( std::move( c) ); | |||
| if ( ! c) { | |||
| ptr = nullptr; | |||
| } | |||
| +#if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| + return exchange( c.ptr_, nullptr); | |||
| +#else | |||
| + return std::exchange( c.ptr_, nullptr); | |||
| +#endif | |||
| }, | |||
| std::forward< Fn >( fn), | |||
| std::placeholders::_1); | |||
| #else | |||
| current()->ontop = [fn=std::forward<Fn>(fn)](activation_record *& ptr){ | |||
| Ctx c{ ptr }; | |||
| - fn( std::move( c) ); | |||
| + c = fn( std::move( c) ); | |||
| if ( ! c) { | |||
| ptr = nullptr; | |||
| } | |||
| +#if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| + return exchange( c.ptr_, nullptr); | |||
| +#else | |||
| + return std::exchange( c.ptr_, nullptr); | |||
| +#endif | |||
| }; | |||
| #endif | |||
| // context switch | |||
| @@ -336,7 +346,7 @@ public: | |||
| if ( BOOST_UNLIKELY( detail::activation_record::current()->force_unwind) ) { | |||
| throw detail::forced_unwind{ ptr}; | |||
| } else if ( BOOST_UNLIKELY( nullptr != detail::activation_record::current()->ontop) ) { | |||
| - detail::activation_record::current()->ontop( ptr); | |||
| + ptr = detail::activation_record::current()->ontop( ptr); | |||
| detail::activation_record::current()->ontop = nullptr; | |||
| } | |||
| return continuation{ ptr }; | |||
| @@ -354,7 +364,7 @@ public: | |||
| if ( BOOST_UNLIKELY( detail::activation_record::current()->force_unwind) ) { | |||
| throw detail::forced_unwind{ ptr}; | |||
| } else if ( BOOST_UNLIKELY( nullptr != detail::activation_record::current()->ontop) ) { | |||
| - detail::activation_record::current()->ontop( ptr); | |||
| + ptr = detail::activation_record::current()->ontop( ptr); | |||
| detail::activation_record::current()->ontop = nullptr; | |||
| } | |||
| return continuation{ ptr }; | |||
| Index: boost_1_65_0/boost/thread/win32/condition_variable.hpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/boost/thread/win32/condition_variable.hpp | |||
| +++ boost_1_65_0/boost/thread/win32/condition_variable.hpp | |||
| @@ -211,7 +211,7 @@ namespace boost | |||
| {} | |||
| #endif | |||
| - void remove_waiter() | |||
| + void remove_waiter_and_reset() | |||
| { | |||
| if (entry) { | |||
| boost::lock_guard<boost::mutex> internal_lock(internal_mutex); | |||
| @@ -221,7 +221,7 @@ namespace boost | |||
| } | |||
| ~entry_manager() BOOST_NOEXCEPT_IF(false) | |||
| { | |||
| - remove_waiter(); | |||
| + remove_waiter_and_reset(); | |||
| } | |||
| list_entry* operator->() | |||
| @@ -250,7 +250,7 @@ namespace boost | |||
| woken=entry->woken(); | |||
| } | |||
| // do it here to avoid throwing on the destructor | |||
| - entry->remove_waiter(); | |||
| + entry.remove_waiter_and_reset(); | |||
| locker.lock(); | |||
| return woken; | |||
| } | |||
| Index: boost_1_65_0/libs/context/doc/callcc.qbk | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/context/doc/callcc.qbk | |||
| +++ boost_1_65_0/libs/context/doc/callcc.qbk | |||
| @@ -176,6 +176,7 @@ return `void`. | |||
| c=c.resume_with([&data](ctx::continuation && c){ | |||
| std::cout << "f2: entered: " << data << std::endl; | |||
| data=-1; | |||
| + return std::move( c); | |||
| }); | |||
| std::cout << "f1: returned third time" << std::endl; | |||
| @@ -221,6 +222,7 @@ an exception. | |||
| c = c.resume_with( | |||
| [](ctx::continuation && c){ | |||
| throw my_exception(std::move(c),"abc"); | |||
| + return std::move( c); | |||
| }); | |||
| output: | |||
| @@ -527,11 +529,11 @@ e.g. ['continuation::operator bool()] re | |||
| [variablelist | |||
| [[Effects:] [Captures current continuation and resumes `*this`. | |||
| -The function `resume_with`, is used to execute function `fn` in continuation | |||
| +The function `resume_with`, is used to execute function `fn` in the execution context of | |||
| `*this` (e.g. the stack frame of `fn` is allocated on stack of `*this`).]] | |||
| [[Returns:] [The continuation representing the continuation that has been | |||
| suspended.]] | |||
| -[[Note:] [Function `fn` needs to return void.]] | |||
| +[[Note:] [Function `fn` needs to return `continuation`.]] | |||
| [[Note:] [The returned continuation indicates if the suspended continuation has | |||
| terminated (return from context-function) via `bool operator()`.]] | |||
| ] | |||
| Index: boost_1_65_0/libs/context/example/ontop.cpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/context/example/ontop.cpp | |||
| +++ boost_1_65_0/libs/context/example/ontop.cpp | |||
| @@ -32,6 +32,7 @@ int main() { | |||
| c = c.resume_with( [&data](ctx::continuation && c){ | |||
| std::cout << "f2: entered: " << data << std::endl; | |||
| data = -1; | |||
| + return std::move( c); | |||
| }); | |||
| std::cout << "f1: returned third time" << std::endl; | |||
| std::cout << "main: done" << std::endl; | |||
| Index: boost_1_65_0/libs/context/example/ontop_void.cpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/context/example/ontop_void.cpp | |||
| +++ boost_1_65_0/libs/context/example/ontop_void.cpp | |||
| @@ -21,8 +21,9 @@ ctx::continuation f1( ctx::continuation | |||
| return std::move( c); | |||
| } | |||
| -void f2( ctx::continuation && c) { | |||
| +ctx::continuation f2( ctx::continuation && c) { | |||
| std::cout << "f2: entered" << std::endl; | |||
| + return std::move( c); | |||
| } | |||
| int main() { | |||
| Index: boost_1_65_0/libs/context/example/throw.cpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/context/example/throw.cpp | |||
| +++ boost_1_65_0/libs/context/example/throw.cpp | |||
| @@ -38,6 +38,7 @@ int main() { | |||
| c = c.resume_with( | |||
| [](ctx::continuation && c){ | |||
| throw my_exception(std::move( c), "abc"); | |||
| + return std::move( c); | |||
| }); | |||
| std::cout << "main: done" << std::endl; | |||
| Index: boost_1_65_0/libs/context/test/test_callcc.cpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/context/test/test_callcc.cpp | |||
| +++ boost_1_65_0/libs/context/test/test_callcc.cpp | |||
| @@ -252,6 +252,7 @@ void test_ontop() { | |||
| c = c.resume_with( | |||
| [&i](ctx::continuation && c){ | |||
| i -= 10; | |||
| + return std::move( c); | |||
| }); | |||
| BOOST_CHECK( c); | |||
| BOOST_CHECK_EQUAL( i, 200); | |||
| @@ -266,6 +267,7 @@ void test_ontop() { | |||
| c = c.resume_with( | |||
| [&c1](ctx::continuation && c){ | |||
| c1 = std::move( c); | |||
| + return std::move( c); | |||
| }); | |||
| } | |||
| } | |||
| @@ -290,7 +292,8 @@ void test_ontop_exception() { | |||
| const char * what = "hello world"; | |||
| c.resume_with( | |||
| [what](ctx::continuation && c){ | |||
| - throw my_exception( std::move( c), what); | |||
| + throw my_exception( std::move( c), what); | |||
| + return std::move( c); | |||
| }); | |||
| BOOST_CHECK_EQUAL( 3, value1); | |||
| BOOST_CHECK_EQUAL( std::string( what), value2); | |||
| Index: boost_1_65_0/libs/fiber/src/context.cpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/fiber/src/context.cpp | |||
| +++ boost_1_65_0/libs/fiber/src/context.cpp | |||
| @@ -145,6 +145,7 @@ context::resume() noexcept { | |||
| // pass pointer to the context that resumes `this` | |||
| c_.resume_with([prev](boost::context::continuation && c){ | |||
| prev->c_ = std::move( c); | |||
| + return boost::context::continuation{}; | |||
| }); | |||
| } | |||
| @@ -158,6 +159,7 @@ context::resume( detail::spinlock_lock & | |||
| c_.resume_with([prev,&lk](boost::context::continuation && c){ | |||
| prev->c_ = std::move( c); | |||
| lk.unlock(); | |||
| + return boost::context::continuation{}; | |||
| }); | |||
| } | |||
| @@ -171,6 +173,7 @@ context::resume( context * ready_ctx) no | |||
| c_.resume_with([prev,ready_ctx](boost::context::continuation && c){ | |||
| prev->c_ = std::move( c); | |||
| context::active()->schedule( ready_ctx); | |||
| + return boost::context::continuation{}; | |||
| }); | |||
| } | |||
| @@ -218,6 +221,7 @@ context::suspend_with_cc() noexcept { | |||
| // pass pointer to the context that resumes `this` | |||
| return c_.resume_with([prev](boost::context::continuation && c){ | |||
| prev->c_ = std::move( c); | |||
| + return boost::context::continuation{}; | |||
| }); | |||
| } | |||
+ 0
- 26
libs/boost/patches/01_fiber_fix.patch
View File
| @ -1,26 +0,0 @@ | |||
| Index: boost_1_63_0/libs/fiber/build/Jamfile.v2 | |||
| =================================================================== | |||
| --- boost_1_63_0.orig/libs/fiber/build/Jamfile.v2 | |||
| +++ boost_1_63_0/libs/fiber/build/Jamfile.v2 | |||
| @@ -44,20 +44,6 @@ lib boost_fiber | |||
| recursive_timed_mutex.cpp | |||
| timed_mutex.cpp | |||
| scheduler.cpp | |||
| - : <link>shared:<library>../../context/build//boost_context | |||
| - [ requires cxx11_auto_declarations | |||
| - cxx11_constexpr | |||
| - cxx11_defaulted_functions | |||
| - cxx11_final | |||
| - cxx11_hdr_mutex | |||
| - cxx11_hdr_tuple | |||
| - cxx11_lambdas | |||
| - cxx11_noexcept | |||
| - cxx11_nullptr | |||
| - cxx11_rvalue_references | |||
| - cxx11_template_aliases | |||
| - cxx11_thread_local | |||
| - cxx11_variadic_templates ] | |||
| - ; | |||
| + : <link>shared:<library>../../context/build//boost_context ; | |||
| boost-install boost_fiber ; | |||
+ 12
- 0
libs/boost/patches/020-fiber_fix_0506e46640989e1f919388ad36890bd6063cd43a.patch
View File
| @ -0,0 +1,12 @@ | |||
| Index: boost_1_65_0/libs/fiber/src/numa/linux/pin_thread.cpp | |||
| =================================================================== | |||
| --- boost_1_65_0.orig/libs/fiber/src/numa/linux/pin_thread.cpp | |||
| +++ boost_1_65_0/libs/fiber/src/numa/linux/pin_thread.cpp | |||
| @@ -9,6 +9,7 @@ | |||
| extern "C" { | |||
| #include <pthread.h> | |||
| #include <sched.h> | |||
| +#include <string.h> | |||
| } | |||
| #include <system_error> | |||
+ 0
- 81
libs/boost/patches/02_fix_context_overload_ambiguity.patch
View File
| @ -1,81 +0,0 @@ | |||
| Index: boost_1_64_0/boost/context/continuation.hpp | |||
| =================================================================== | |||
| --- boost_1_64_0.orig/boost/context/continuation.hpp 2017-04-24 12:36:55.987412031 +0100 | |||
| +++ boost_1_64_0/boost/context/continuation.hpp 2017-04-24 12:37:49.730416122 +0100 | |||
| @@ -56,7 +56,7 @@ | |||
| namespace context { | |||
| namespace detail { | |||
| -template< int N > | |||
| +template<typename U> | |||
| struct helper { | |||
| template< typename T > | |||
| static T convert( T && t) noexcept { | |||
| @@ -64,8 +64,8 @@ | |||
| } | |||
| }; | |||
| -template<> | |||
| -struct helper< 1 > { | |||
| +template<typename U> | |||
| +struct helper< std::tuple<U> > { | |||
| template< typename T > | |||
| static std::tuple< T > convert( T && t) noexcept { | |||
| return std::make_tuple( std::forward< T >( t) ); | |||
| @@ -239,15 +239,16 @@ | |||
| } | |||
| -template< typename Ctx, typename Fn, typename ... Arg > | |||
| +template< typename Ctx, typename Fn, typename Arg > | |||
| detail::transfer_t context_ontop( detail::transfer_t t) { | |||
| - auto p = static_cast< std::tuple< Fn, std::tuple< Arg ... > > * >( t.data); | |||
| + auto p = static_cast< Arg * >( t.data); | |||
| BOOST_ASSERT( nullptr != p); | |||
| typename std::decay< Fn >::type fn = std::forward< Fn >( std::get< 0 >( * p) ); | |||
| t.data = & std::get< 1 >( * p); | |||
| Ctx c{ t }; | |||
| // execute function, pass continuation via reference | |||
| - std::get< 1 >( * p) = detail::helper< sizeof ... (Arg) >::convert( fn( std::move( c) ) ); | |||
| + typedef typename std::decay<decltype(std::get<1>(*p))>::type H; | |||
| + std::get< 1 >(* p) = detail::helper<H>::convert( fn( std::move( c) ) ); | |||
| #if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| return { detail::exchange( c.t_.fctx, nullptr), & std::get< 1 >( * p) }; | |||
| #else | |||
| @@ -275,7 +276,7 @@ | |||
| template< typename Ctx, typename StackAlloc, typename Fn > | |||
| friend class detail::record; | |||
| - template< typename Ctx, typename Fn, typename ... Arg > | |||
| + template< typename Ctx, typename Fn, typename Arg > | |||
| friend detail::transfer_t | |||
| context_ontop( detail::transfer_t); | |||
| @@ -354,7 +355,7 @@ | |||
| template< typename Fn, typename ... Arg > | |||
| continuation resume_with( Fn && fn, Arg ... arg) { | |||
| BOOST_ASSERT( nullptr != t_.fctx); | |||
| - auto tpl = std::make_tuple( std::forward< Fn >( fn), std::forward< Arg >( arg) ... ); | |||
| + auto tpl = std::make_tuple( std::forward< Fn >( fn), std::make_tuple( std::forward< Arg >( arg) ... )); | |||
| return detail::ontop_fcontext( | |||
| #if defined(BOOST_NO_CXX14_STD_EXCHANGE) | |||
| detail::exchange( t_.fctx, nullptr), | |||
| @@ -362,7 +363,7 @@ | |||
| std::exchange( t_.fctx, nullptr), | |||
| #endif | |||
| & tpl, | |||
| - context_ontop< continuation, Fn, Arg ... >); | |||
| + context_ontop< continuation, Fn, decltype(tpl) >); | |||
| } | |||
| continuation resume() { | |||
| @@ -451,7 +452,8 @@ | |||
| template< | |||
| typename Fn, | |||
| typename ... Arg, | |||
| - typename = detail::disable_overload< continuation, Fn > | |||
| + typename = detail::disable_overload< continuation, Fn >, | |||
| + typename = detail::disable_overload< std::allocator_arg_t, Fn > | |||
| > | |||
| continuation | |||
| callcc( Fn && fn, Arg ... arg) { | |||