Mirror
/
ardupilot
mirror of https://github.com/ArduPilot/ardupilot
5
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

AP_HAL_SITL: disable SITL stack checking due to memory corruption 

There appears to be issues with pthread_attr_setstack.
This commit is contained in:
Peter Barker 2020-01-15 14:50:58 +11:00 committed by Andrew Tridgell
parent a1fb44b782
commit f504009287
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
libraries/AP_HAL_SITL/Scheduler.cpp
View File

@ -17,6 +17,13 @@ using namespace HALSITL;
extern const AP_HAL::HAL& hal;
#ifndef SITL_STACK_CHECKING_ENABLED
//#define SITL_STACK_CHECKING_ENABLED !defined(__CYGWIN__) && !defined(__CYGWIN64__)
// stack checking is disabled until the memory corruption issues are
// fixed with pthread_attr_setstack. These may be due to
// changes in the way guard pages are handled.
#define SITL_STACK_CHECKING_ENABLED 0
#endif
AP_HAL::Proc Scheduler::_failsafe = nullptr;
@ -233,7 +240,9 @@ void Scheduler::_run_io_procs()
hal.uartH->_timer_tick();
hal.storage->_timer_tick();
#if SITL_STACK_CHECKING_ENABLED
check_thread_stacks();
#endif
AP::RC().update();
}
@ -316,7 +325,7 @@ bool Scheduler::thread_create(AP_HAL::MemberProc proc, const char *name, uint32_
a->name = name;
pthread_attr_init(&a->attr);
#if !defined(__CYGWIN__) && !defined(__CYGWIN64__)
#if SITL_STACK_CHECKING_ENABLED
if (pthread_attr_setstack(&a->attr, a->stack, alloc_stack) != 0) {
AP_HAL::panic("Failed to set stack of size %u for thread %s", alloc_stack, name);
}