I pulled the commit, which, in the git log of that file, was just before yours, and it was fine there. Shouldn't we switch_to() to tmp?
On Tue, Dec 15, 2015 at 8:14 AM, Barret Rhoden <[email protected]> wrote: > On 2015-12-15 at 06:51 "'Davide Libenzi' via Akaros" > <[email protected]> wrote: > > For the uaccess one, /me looks at Barrett ☺ > > > > > https://github.com/brho/akaros/commit/a1c75135e810a7b645a398e6fc8da16bf62af041 > > The test was working for me after that commit. Maybe we are running > the test differently? > > > This was not failing before. This is the code: > > munmap(tmp, (uintptr_t) addr, mmap_size); > > > > KT_ASSERT_M("Copy to user (u8) to not mapped address should fail", > > copy_to_user(addr, buf, 1) == -EFAULT); > > From looking at the code, that should be failing. Maybe the munmap > failed? > > -- > You received this message because you are subscribed to the Google Groups > "Akaros" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "Akaros" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. For more options, visit https://groups.google.com/d/optout.
