On 06/25/2018 02:18 PM, Max Filippov wrote: > On Mon, Jun 25, 2018 at 10:10 AM, Philippe Mathieu-Daudé > <f4...@amsat.org> wrote: >> On 06/25/2018 01:59 PM, Emilio G. Cota wrote: >>> On Sat, Jun 23, 2018 at 20:24:09 -0700, Max Filippov wrote: >>>> # first bad commit: [0b5c91f74f3c83a36f37740969df8c775c997e69] >>>> translate-all: use per-page locking in !user-mode >>>> >>>> Emilio, could you please take a look? The following test locks up QEMU: >>>> >>>> qemu-system-xtensa -M sim -cpu dc232b -nographic -semihosting -icount >>>> 6 -kernel ./test_mmu.tst >>>> >>>> test_mmu.tst binary may be found here: >>>> http://jcmvbkbc.spb.ru/~dumb/tmp/201806232022/test_mmu.tst >>> >>> Thanks for the test case. >>> >>> I submitted a patch that fixes this for me--see this thread: >>> https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg07236.html >> >> Thanks, this works :) >> >> Alex nicer test output using: >> >> tests/tcg/xtensa/Makefile.system >> run-%.tst: %.tst >> - $(SIM) $(SIMFLAGS) ./$< >> + $(SIM) -monitor null $(SIMFLAGS) ./$< > > This makefile is also usable with another simulator (xtensa ISS) which would > not understand this option. Could it be added to the proper SIMFLAGS instead?
OK, good to know.