<div dir="ltr">I hate to top post but I'm not sure where to insert this. <div><br></div><div>As a first step, I would ensure that paging catches stack overflow errors. Then move on to an optional capability where threads cannot access the stacks of other threads. POSIX does not say anything about whether that should work or not but there are cases (especially in RTEMS) where if a blocking thread has a message queue buffer on a stack and another thread does a send, then it will write to another thread's stack. This is accepted programming practice. </div><div><br></div><div>I'm not opposed to an option where per-thread stack protection is available. But making it mandatory is a bad step. Using the MMU to detect stack overflow is good. </div><div><br></div><div>As Hesham mentioned, this is hard on some architectures if you don't have a nice page management system. Can you make sure the minimum processor architectural requirements are documented. Not just in an email. This will ultimately be information in the CPU Supplement.</div><div><br></div><div>--joel</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 12, 2020 at 5:02 PM Hesham Almatary <<a href="mailto:hesham.almatary@cl.cam.ac.uk">hesham.almatary@cl.cam.ac.uk</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 Tue, 12 May 2020 at 04:57, Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
><br>
> On Thu, May 7, 2020 at 9:59 PM Hesham Almatary<br>
> <<a href="mailto:hesham.almatary@cl.cam.ac.uk" target="_blank">hesham.almatary@cl.cam.ac.uk</a>> wrote:<br>
> ><br>
> > Hello Utkarsh,<br>
> ><br>
> > I'd suggest you don't spend too much efforts on setting up BBB<br>
> > hardware if you haven't already. Debugging on QEMU with GDB is way<br>
> > easier, and you can consider either qemu-xilinx-zynq-a9 or rpi2 BSPs.<br>
> > Later, you can move your code to BBB if you want, since both are based<br>
> > on ARMv7.<br>
> +1<br>
><br>
> Past work has also used psim successfully I thought? Or am I mistaken there.<br>
><br>
Before my 2012 project (and part of it, yes), we used psim. The use of<br>
software TLBs wasn't very ideal/easy though, so we moved to ARM in<br>
2013. The development/testing was mainly on a RPi board. I don't<br>
remember there was a QEMU model for it yet.<br>
<br>
<br>
> ><br>
> > On Thu, 7 May 2020 at 18:26, Utkarsh Rai <<a href="mailto:utkarsh.rai60@gmail.com" target="_blank">utkarsh.rai60@gmail.com</a>> wrote:<br>
> > ><br>
> > > Hello,<br>
> > > This is to ensure that all the interested parties are on the same page before I start my project and can give their invaluable feedback.<br>
> Excellent, thank you for getting the initiative.<br>
><br>
> I'll be taking on the primary mentorship for your project, with<br>
> support from the co-mentors (Peter, Hesham, Sebastian). For now, I<br>
> prefer you to continue to make your presence on the mailing list. We<br>
> will establish other forms of communication as needed and will take on<br>
> IRC meetings once coding begins in earnest.<br>
><br>
> > > My GSoC project, providing user-configurable thread stack protection, requires adding architecture-specific low-level support as well as high-level API support. I will be starting my project with ARMv7-A (on BBB) based MMU since RTEMS already has quite mature support for it. As already mentioned in my proposal I will be focusing more on the High-level interface and let it drive whatever further low-level support is needed.<br>
> > > Once the application uses MMU for thread stack address generation each thread will be automatically protected as the page tables other than that of the executing thread would be made dormant. When the user has to share thread stacks they will have to obtain the stack attributes of the threads to be shared by pthread_attr_getstack() and then get a file descriptor of the memory to be mapped by a call to shm_open() and finally map this to the stack of the other thread through<br>
> > > mmap(), this is the POSIX compliant way I could think of. Now at the low level, it means mapping the page table of the thread to be shared into the address space of the executing thread. This is an area where the low-level support has to be provided. At the high-level, this means providing support to mmap and shared-memory interface as mmap provides support for a file by simply<br>
> > > copying the memory from the file to the destination. For shared memory objects it can<br>
> > > provide read/write access but cannot provide restriction of write/read access. One of the areas that I have to look into more detail is thread context-switch, as after every context switch the TLBs need to be flushed and reinitialized lest we get an invalid address for the executing thread. Since context-switch is low-level architecture-specific, this also has to be provided with more support.<br>
><br>
> This is really dense text. Try to break apart your writing a little<br>
> bit to help clarify your thoughts.  You should also translate some of<br>
> your proposal into a wiki page if you haven't started that yet, and a<br>
> blog post. Both of those will help to focus your thoughts into words.<br>
><br>
> "mapping the page table" is not meaningful to me. I think you mean<br>
> something like "mapping a page from the page table"?  Will the design<br>
> support sharing task stacks using MPUs with 4 regions? 8?  (It seems<br>
> challenging to me, but might be possible in some limited<br>
> configurations. Having support for those kinds of targets might still<br>
> be useful, with the caveat that sharing stacks is not possible.)<br>
><br>
> The first step is to get a BSP running that has sufficient<br>
> capabilities for you to test out memory protection with. Do a little<br>
> bit of digging, but definitely simulation is the way to go.<br>
><br>
> The second step from my perspective is to determine how to introduce<br>
> strict isolation between task stacks. Don't worry about sharing at<br>
> this stage, but rather can you completely isolate tasks? Then you can<br>
> start to poke holes in the isolation.<br>
><br>
> As you say, you'll also need to start to understand the context switch<br>
> code. Start looking into it to determine where you might think to<br>
> implement changing the address space of the executing thread. Another<br>
> challenge is that RTEMS can dispatch to a new task from the interrupt<br>
> handler, which may cause some problems for you as well to handle.<br>
><br>
> Have you figured out where in the code thread stacks are allocated?<br>
> How do you plan to make the thread stacks known to other threads?<br>
><br>
> TLB shootdown can be extremely expensive. Try to find ways to optimize<br>
> that cost earlier rather than later. (One of those cases where<br>
> premature optimization will be acceptable.) Tagged TLB architectures<br>
> or those with "superpages" may incur less overhead if you can<br>
> selectively shoot-down the entry (entries) used for task stacks.<br>
><br>
> A final thought is that the method to configure this support is<br>
> necessary. Configuration is undergoing some heavy changes lately, and<br>
> application-level configuration is going to be completely different in<br>
> rtems6. You may want to consider raising a new thread with CC to<br>
> Sebastian to get his input on how the best way to configure something<br>
> like this might look, now and in the future. I would have leaned<br>
> toward a high-level configure switch (--enable-task-protection) in the<br>
> past, but now I don't know.  This capability is however something that<br>
> should be considered disabled by default due to the extra overhead.<br>
><br>
> Gedare<br>
><br>
> > > Kindly provide your feedback if I have missed something or I have a wrong idea about it.<br>
> > ><br>
> > > Regards,<br>
> > > Utkarsh Rai.<br>
> > ><br>
> > > _______________________________________________<br>
> > > devel mailing list<br>
> > > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> > > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
> > _______________________________________________<br>
> > devel mailing list<br>
> > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>