translate-all: Fix user-mode self-modifying code in 2 page long TB
commit7399a337e4126f7c8c8af3336726f001378c4798
authorStanislav Shmarov <snarpix@gmail.com>
Thu, 7 Jul 2016 08:33:12 +0000 (7 11:33 +0300)
committerRichard Henderson <rth@twiddle.net>
Fri, 8 Jul 2016 20:17:38 +0000 (8 13:17 -0700)
treee58f0235241cecdfe2f5b711dcd287dbd70a071e
parent81daabaf7a572f138a8b88ba6eea556bdb0cce46
translate-all: Fix user-mode self-modifying code in 2 page long TB

In user-mode emulation Translation Block can consist of 2 guest pages.
In that case QEMU also mprotects 2 host pages that are dedicated for
guest memory, containing instructions. QEMU detects self-modifying code
with SEGFAULT signal processing.

In case if instruction in 1st page is modifying memory of 2nd
page (or vice versa) QEMU will mark 2nd page with PAGE_WRITE,
invalidate TB, generate new TB contatining 1 guest instruction and
exit to CPU loop. QEMU won't call mprotect, and new TB will cause
same SEGFAULT. Page will have both PAGE_WRITE_ORG and PAGE_WRITE
flags, so QEMU will handle the signal as guest binary problem,
and exit with guest SEGFAULT.

Solution is to do following: In case if current TB was invalidated
continue to invalidate TBs from remaining guest pages and mark pages
as PAGE_WRITE. After that disable host page protection with mprotect.
If current tb was invalidated longjmp to main loop. That is more
efficient, since we won't get SEGFAULT when executing new TB.

Reviewed-by: Sergey Fedorov <sergey.fedorov@linaro.org>
Signed-off-by: Stanislav Shmarov <snarpix@gmail.com>
Message-Id: <1467880392-1043630-1-git-send-email-snarpix@gmail.com>
Signed-off-by: Richard Henderson <rth@twiddle.net>
translate-all.c