On 6 June 2014 14:49, Sean Bruno <sbr...@freebsd.org> wrote:

Your commit message is inaccurate -- this patch isn't
implementing a syscall, it's just adding support to QEMU's
strace code for it. Something like
"bsd-user: Implement strace support for 'syscall' syscall"

would be better.

If you fix that, then
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>

(I notice you haven't put in a Copyright: line when you added
the file header, but I'm a bit unconvinced that they're really
useful for anything so I think that's ok.)

thanks
-- PMM

Reply via email to