mirror of
https://github.com/RIOT-OS/RIOT.git
synced 2024-12-29 04:50:03 +01:00
eb1279005c
Replace direct accesses to sched_active_thread and sched_active_pid with the helper functions thread_getpid() and thread_get_active(). This serves two purposes: 1. It makes accidental writes to those variable from outside core less likely. 2. Casting off the volatile qualifier is now well contained to those two functions
116 lines
2.9 KiB
C
116 lines
2.9 KiB
C
/*
|
|
* Copyright (C) 2019 Freie Universität Berlin
|
|
*
|
|
* This file is subject to the terms and conditions of the GNU Lesser
|
|
* General Public License v2.1. See the file LICENSE in the top level
|
|
* directory for more details.
|
|
*/
|
|
|
|
/**
|
|
* @ingroup tests
|
|
* @{
|
|
*
|
|
* @file
|
|
* @brief Thread race condition test application to reproduce
|
|
* https://github.com/RIOT-OS/RIOT/issues/10881
|
|
*
|
|
* @author Martine Lenders <m.lenders@fu-berlin.de>
|
|
*
|
|
* @}
|
|
*/
|
|
|
|
#include <stdio.h>
|
|
#include <unistd.h>
|
|
|
|
#include "periph/timer.h"
|
|
#include "random.h"
|
|
#include "test_utils/expect.h"
|
|
#include "thread.h"
|
|
#include "msg.h"
|
|
|
|
#define CANARY_TYPE (0x21fd)
|
|
|
|
#define TIMER_FREQ (1000000LU)
|
|
#define TIMER_TIMEOUT_MIN (1U)
|
|
#define TIMER_TIMEOUT_MAX (100U)
|
|
|
|
static char _stack[THREAD_STACKSIZE_DEFAULT];
|
|
|
|
static kernel_pid_t _pid_main = KERNEL_PID_UNDEF;
|
|
|
|
/**
|
|
* @brief Schedule next timer event in TIMER_TIMEOUT_MIN to TIMER_TIMEOUT_MAX
|
|
* ticks.
|
|
*/
|
|
static void _sched_next(void)
|
|
{
|
|
timer_set(TIMER_DEV(0), 0, random_uint32_range(TIMER_TIMEOUT_MIN,
|
|
TIMER_TIMEOUT_MAX));
|
|
}
|
|
|
|
/**
|
|
* @brief The timer interrupt
|
|
*/
|
|
static void _timer(void *arg, int channel)
|
|
{
|
|
(void)arg;
|
|
(void)channel;
|
|
/* just continue rescheduling interrupt triggering at random time */
|
|
_sched_next();
|
|
}
|
|
|
|
/**
|
|
* @brief The sending thread
|
|
*/
|
|
static void *_thread(void *arg)
|
|
{
|
|
(void) arg;
|
|
|
|
while (1) {
|
|
msg_t msg = { .type = 0U };
|
|
|
|
write(STDOUT_FILENO, ".", 1U);
|
|
/* send without blocking */
|
|
msg_try_send(&msg, _pid_main);
|
|
thread_yield();
|
|
}
|
|
|
|
return NULL;
|
|
}
|
|
|
|
int main(void)
|
|
{
|
|
kernel_pid_t pid;
|
|
|
|
timer_init(TIMER_DEV(0), TIMER_FREQ, _timer, NULL);
|
|
random_init(timer_read(TIMER_DEV(0)));
|
|
puts("Test is \"successful\" if it runs forever without halting\n"
|
|
"on any of the assertion in this file\n");
|
|
_pid_main = thread_getpid();
|
|
|
|
puts("I will try to trigger an interrupt at random intervals. When an\n"
|
|
"interrupt is fired while ISR is disable in the thread_yield_higher()\n"
|
|
"function some platform-specific implementations used to not call\n"
|
|
"sched_run() which was the cause of the bug tested here");
|
|
_sched_next();
|
|
pid = thread_create(_stack, sizeof(_stack), THREAD_PRIORITY_MAIN + 1,
|
|
THREAD_CREATE_WOUT_YIELD | THREAD_CREATE_STACKTEST,
|
|
_thread, NULL, "nr2");
|
|
expect(pid != KERNEL_PID_UNDEF);
|
|
|
|
while (1) {
|
|
msg_t msg = { .type = CANARY_TYPE };
|
|
|
|
/* receive blocked */
|
|
msg_receive(&msg);
|
|
/* check msg_receive() returned without blocking (i.e. the sending
|
|
* thread did not get a chance to copy the message over) */
|
|
if (msg.type == CANARY_TYPE) {
|
|
puts("Message was not written");
|
|
return 1;
|
|
}
|
|
write(STDOUT_FILENO, "\b", 1U);
|
|
}
|
|
return 0;
|
|
}
|