<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 18, 2020 at 6:34 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Jun 17, 2020 at 11:17 PM Utkarsh Rai <<a href="mailto:utkarsh.rai60@gmail.com" target="_blank">utkarsh.rai60@gmail.com</a>> wrote:<br>
><br>
> Hello,<br>
> For my GSoC project, I need to set/unset the memory attributes of the thread stacks on each context switch.<br>
> Right now I am making changes to the CPU-specific context switch assembly code. The arm/../cpu.h file has the Context_Control structure, which is used to store the relevant registers at each context switch.<br>
>  I have been trying to add the stack attributes to this structure from the 'rtems/score/stackmanagement.h' header (This header has been defined by me and has the stack attributes structure).<br>
<br>
When you change the context structure, there might be other places you<br>
need to update besides the context switch itself. One that comes to<br>
mind is _CPU_Context_validate. Maybe chase down where the context is<br>
referenced.<br>
<br>
>  After that, I would initialize the stack attributes in _CPU_Context_Initialize.<br>
>  For unsetting stack memories, I plan on making changes to _CPU_Context_Switch().<br>
> The problem is when I try to include the header in cpu.h it breaks the build with the following error -<br>
><br>
> '.......<br>
> ........<br>
> /home/utkarsh/sandbox/rtems/5/arm-rtems5/include/machine/_default_types.h: Assembler messages:<br>
> /home/utkarsh/sandbox/rtems/cpukit/include/rtems/score/stackmanagement.h:42: Error: junk at end of line, first unrecognized character is `}'<br>
> /home/utkarsh/sandbox/rtems/cpukit/include/rtems/score/stackmanagement.h:44: Error: bad instruction `typedef struct stack_attr_prot'<br>
> /home/utkarsh/sandbox/rtems/cpukit/include/rtems/score/stackmanagement.h:45: Error: junk at end of line, first unrecognized character is `{'<br>
> /home/utkarsh/sandbox/rtems/cpukit/include/rtems/score/stackmanagement.h:46: Error: bad instruction `stack_attr_shared *shared_stacks'<br>
> /home/utkarsh/sandbox/rtems/cpukit/include/rtems/score/stackmanagement.h:47: Error: bad instruction `stack_attr Base'<br>
> /home/utkarsh/sandbox/rtems/cpukit/include/rtems/score/stackmanagement.h:48: Error: bad instruction `_bool current_stack'<br>
> {standard input}: Error: invalid operands (*UND* and *ABS* sections) for `*' when setting `iter'<br>
> Makefile:11563: recipe for target 'score/cpu/arm/arm-context-validate.o' failed<br>
> make[4]: *** [score/cpu/arm/arm-context-validate.o] Error 1<br>
> '<br>
> I have defined the stackmanagement.h header in the cpukit/<a href="http://headers.am" rel="noreferrer" target="_blank">headers.am</a> and this is obviously an assembler error.  Can someone point out what is it that I am doing wrong?<br>
><br>
<br>
When a .h file can be included in both C and ASM, you need to separate<br>
some of it, and disable some of it in ASM scope that doesn't<br>
understand C data types. See, for example,<br>
cpukit/include/rtems/score/percpu.h<br>
<br>
> I also would like to verify if the above way of handling context switching is the appropriate way to proceed.<br>
<br>
Seems like the right general idea. We probably need a CPP switch to<br>
enable/disable this stuff though.<br></blockquote><div><br></div><div>Can you please elaborate on CPP switch? I am not sure if I am clear about this. </div></div></div>