<div dir="ltr">Sorry, forgot to mention, the branch test1 is the one i'm currently using. Well im working on as6 but didn't want to push it yet since it doesn't really work.<div><br></div><div>So we don't need to jump again, that's great. Wasn't sure if it is needed or not, but it looked like it got better with the jump. </div><div>But without the jump, the syscall is called again for some reason and it tries to print again.

 And looks like it tries to print something, it doesn't have access to, since it crashes there.

 But this time it causes a kernel panic loop (the one my colleague mention earlier this week). With this im even more confused.</div><div><br></div><div>- PS</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">ne 16. 2. 2020 o 16:03 Vojtech Horky <<a href="mailto:horky@d3s.mff.cuni.cz">horky@d3s.mff.cuni.cz</a>> napísal(a):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello.<br>
<br>
Dne 16. 02. 20 v 15:25 Patrik Smelik napsal(a):<br>
> Hi,<br>
> <br>
> Since yesterday, we are stuck on this problem. Userspace process is <br>
<br>
It would be extremely useful to send commit hash where the bug is <br>
reproducible, your master is four weeks old, I do not see implementation <br>
of print in branch as6...<br>
<br>
<br>
> created fine, syscall handling also works, e.g. for print it prints out <br>
> what it needs to, but when returning from syscall, i.e. jumping back to <br>
> userspace, the application suddenly terminates. I fed the <br>
> cpu_jump_to_userspace function context->sp and context->epc as <br>
> arguments.The problem is that the context->sp is set to 0 for some <br>
> reason, which i believe isn't the correct input for the jump. I traced <br>
> the change to the memcpy method, so to when the binary image is being <br>
> copied, but i'm currently stuck here.<br>
<br>
Not sure I got it right, but when the syscall is handled, the thread <br>
returns to userspace normally as with any other exception. That is, you <br>
need to call cpu_jump_to_userspace only once per process.<br>
<br>
And you only use context->epc to advance to next instruction as <br>
otherwise the syscall would be restarted after being handled.<br>
<br>
Also note that this function would typically receive two constants: <br>
address of the top of stack (so something like 0x3f00) and entry point <br>
as specified in the linker script (0x4000).<br>
<br>
As a side note: in one of your commits I see that you panic when the <br>
syscall id is invalid. That is definitely a bad behavior because any <br>
application would be easily able to terminate your kernel.<br>
<br>
Hope this helps,<br>
- VH<br>
_______________________________________________<br>
NSWI004 mailing list<br>
<a href="mailto:NSWI004@d3s.mff.cuni.cz" target="_blank">NSWI004@d3s.mff.cuni.cz</a><br>
<a href="https://d3s.mff.cuni.cz/mailman/listinfo/nswi004" rel="noreferrer" target="_blank">https://d3s.mff.cuni.cz/mailman/listinfo/nswi004</a><br>
</blockquote></div>