From: Laurent Vivier <laurent@vivier.eu>
To: qemu-devel@nongnu.org
Cc: "Philippe Mathieu-Daudé" <f4bug@amsat.org>,
"Thomas Huth" <huth@tuxfamily.org>,
"Richard Henderson" <richard.henderson@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Edgar E . Iglesias" <edgar.iglesias@gmail.com>,
"Eduardo Habkost" <ehabkost@redhat.com>,
"Michael Walle" <michael@walle.cc>,
"Aurelien Jarno" <aurelien@aurel32.net>,
"Anthony Green" <green@moxielogic.com>,
"Chris Wulff" <crwulff@gmail.com>,
"Stafford Horne" <shorne@gmail.com>,
"Alexander Graf" <agraf@suse.de>,
"Artyom Tarasenko" <atar4qemu@gmail.com>,
"Bastian Koppelmann" <kbastian@mail.uni-paderborn.de>,
"Guan Xuetao" <gxt@mprc.pku.edu.cn>,
"Max Filippov" <jcmvbkbc@gmail.com>,
qemu-arm@nongnu.org, qemu-ppc@nongnu.org, qemu-s390x@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v4 1/7] accel/tcg: add size paremeter in tlb_fill()
Date: Tue, 23 Jan 2018 23:11:28 +0100 [thread overview]
Message-ID: <a6bc9bdb-57cd-4b9d-eb60-0feec05bcbf6@vivier.eu> (raw)
In-Reply-To: <20180118193846.24953-2-laurent@vivier.eu>
Le 18/01/2018 à 20:38, Laurent Vivier a écrit :
> The MC68040 MMU provides the size of the access that
> triggers the page fault.
>
> This size is set in the Special Status Word which
> is written in the stack frame of the access fault
> exception.
>
> So we need the size in m68k_cpu_unassigned_access() and
> m68k_cpu_handle_mmu_fault().
>
> To be able to do that, this patch modifies the prototype of
> handle_mmu_fault handler, tlb_fill() and probe_write().
> do_unassigned_access() already includes a size parameter.
I'd like to send a pull request for m68k MMU series, and 68040 MMU
relies on this patch.
If someone disagrees with this change, it's time to say it!
Thanks,
Laurent
next prev parent reply other threads:[~2018-01-23 22:12 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-18 19:38 [Qemu-devel] [PATCH v4 0/7] target/m68k: supervisor mode (part 2) Laurent Vivier
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 1/7] accel/tcg: add size paremeter in tlb_fill() Laurent Vivier
2018-01-23 22:11 ` Laurent Vivier [this message]
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 2/7] target/m68k: add MC68040 MMU Laurent Vivier
2018-01-19 5:34 ` Richard Henderson
2018-01-21 19:57 ` Rob Landley
2018-01-22 7:16 ` Laurent Vivier
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 3/7] target/m68k: add Transparent Translation Laurent Vivier
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 4/7] target/m68k: add index parameter to gen_load()/gen_store() and Co Laurent Vivier
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 5/7] target/m68k: add moves Laurent Vivier
2018-01-19 5:37 ` Richard Henderson
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 6/7] target/m68k: add pflush/ptest Laurent Vivier
2018-01-18 19:38 ` [Qemu-devel] [PATCH v4 7/7] target/m68k: add HMP command "info tlb" Laurent Vivier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a6bc9bdb-57cd-4b9d-eb60-0feec05bcbf6@vivier.eu \
--to=laurent@vivier.eu \
--cc=agraf@suse.de \
--cc=atar4qemu@gmail.com \
--cc=aurelien@aurel32.net \
--cc=crwulff@gmail.com \
--cc=edgar.iglesias@gmail.com \
--cc=ehabkost@redhat.com \
--cc=f4bug@amsat.org \
--cc=green@moxielogic.com \
--cc=gxt@mprc.pku.edu.cn \
--cc=huth@tuxfamily.org \
--cc=jcmvbkbc@gmail.com \
--cc=kbastian@mail.uni-paderborn.de \
--cc=michael@walle.cc \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=shorne@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).