Alberto Nidasio authored 3 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 10 months ago and Federico committed 3 months ago

The problem is that Clang does not support assigning specific registers to variables.
This commit is a quick fix to solve this problem while adding support for Clang

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 10 months ago and Federico committed 3 months ago

For example in interrupts_cortexMx.cpp:
```cpp
static unsigned int getProgramCounter()
{
    unsigned int result;
    // Get program counter when the exception was thrown from stack frame
    asm volatile("mrs   %0,  psp    \n\t"
                 "add   %0, %0, #24 \n\t"
                 "ldr   %0, [%0]    \n\t":"=r"(result));
    return result;
}
```

To check differences in the generated code we disassempled the compiled source with and without the keyword, both for O0 and O2 and both for gcc and clang.
Only in a couple of cases with O0 and gcc there were some differences in the instructions, but non in the semantics

Removed `register` keyword and changed how the stack pointer is copied in `getCurrentFreeStack` function

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 10 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 10 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 10 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 9 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 9 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Alberto Nidasio authored 3 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Andrea Bontempi authored 3 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Daniele Cattaneo authored 3 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Daniele Cattaneo authored 3 months ago and Federico committed 3 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

Daniele Cattaneo authored 4 months ago and Federico committed 4 months ago

Signed-off-by: Terraneo Federico <fede.tft@miosix.org>

And use it to remove a dependency of libsyscalls from kernel headers.

In Miosix 3 if the timer interrupt causes a context switch, it will run the PendSV interrupt that will perform the stack overflow check anyway, so the call was also redundant.

Clear fault interrupt flags to fix the address causing the fault being stuck after the first fault, and remove wrong leftover IRQinvokeScheduler

Enable the MPU also in high priority interrupt handlers, as it also defines the cacheability of regions