On Monday, June 18, 2012 2:56:30 pm Daniil Cherednik wrote: > Hi! > > I am trying to continue the work started by DavidXu on implemention of fast > syscalls via sysenter/sysexit. > http://people.freebsd.org/~davidxu/sysenter/kernel/ > I have ported it on FreeBSD9. It looks like it works. Unfortunately I am a > beginner in kernel so I have some questions: > > 1. see http://people.freebsd.org/~davidxu/sysenter/kernel/kernel.patch > /* > * If %edx was changed, we can not use sysexit, because it > * needs %edx to restore userland %eip. > */ > if (orig_edx != frame.tf_edx) > td->td_pcb->pcb_flags |= PCB_FULLCTX; > > What is the reason why we have to do this additional check? In > http://people.freebsd.org/~davidxu/sysenter/kernel/sysenter.s > we store %edx to the stack in > pushl %edx /* ring 3 next %eip */ > and we restore the register in > popl %edx /* ring 3 %eip */
Some system calls return two return values (pipe(2)) or return a 64-bit off_t (lseek(2)). Those system calls change %edx's value and need that changed value to make it out to userland. > 2. see http://people.freebsd.org/~davidxu/sysenter/kernel/sysenter.s > movl PCPU(CURPCB),%esi > call syscall > > Why do we movl PCPU(CURPCB),%esi before calling syscall? syscall is just c- > function. No clue on this one, looks like it is not needed. -- John Baldwin _______________________________________________ freebsd-hackers@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-hackers To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"