From: "Alex Bennée" <alex.bennee@linaro.org>
To: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Cc: qemu-ppc@nongnu.org, david@gibson.dropbear.id.au,
rth@twiddle.net, qemu-devel@nongnu.org, intermediadc@hotmail.com,
programmingkidx@gmail.com, clg@kaod.org
Subject: Re: [Qemu-devel] [PATCH for 2.10] tcg: enable MTTCG by default for PPC64 on x86
Date: Mon, 10 Apr 2017 10:51:57 +0100 [thread overview]
Message-ID: <87mvbok2te.fsf@linaro.org> (raw)
In-Reply-To: <20170410081217.2897-1-nikunj@linux.vnet.ibm.com>
Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> writes:
> This enables the multi-threaded system emulation by default for PPC64
> guests using the x86_64 TCG back-end.
Technically this enables it for all backends that can meet the guests
default memory model requirements. So far only the x86 backend defines
one as:
#define TCG_TARGET_DEFAULT_MO (TCG_MO_ALL & ~TCG_MO_ST_LD)
>
> Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>
> Depends on following patch which fixes the define name:
>
> https://patchwork.ozlabs.org/patch/748840/
>
> ---
> configure | 2 ++
> target/ppc/cpu.h | 2 ++
> 2 files changed, 4 insertions(+)
>
> diff --git a/configure b/configure
> index 4b3b5cd..2a87495 100755
> --- a/configure
> +++ b/configure
> @@ -6008,12 +6008,14 @@ case "$target_name" in
> ppc64)
> TARGET_BASE_ARCH=ppc
> TARGET_ABI_DIR=ppc
> + mttcg=yes
> gdb_xml_files="power64-core.xml power-fpu.xml power-altivec.xml power-spe.xml power-vsx.xml"
> ;;
> ppc64le)
> TARGET_ARCH=ppc64
> TARGET_BASE_ARCH=ppc
> TARGET_ABI_DIR=ppc
> + mttcg=yes
> gdb_xml_files="power64-core.xml power-fpu.xml power-altivec.xml power-spe.xml power-vsx.xml"
> ;;
> ppc64abi32)
> diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
> index e0ff041..ece535d 100644
> --- a/target/ppc/cpu.h
> +++ b/target/ppc/cpu.h
> @@ -30,6 +30,8 @@
> #define TARGET_LONG_BITS 64
> #define TARGET_PAGE_BITS 12
>
> +#define TCG_GUEST_DEFAULT_MO 0
> +
> /* Note that the official physical address space bits is 62-M where M
> is implementation dependent. I've not looked up M for the set of
> cpus we emulate at the system level. */
--
Alex Bennée
next prev parent reply other threads:[~2017-04-10 9:51 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-10 8:12 [Qemu-devel] [PATCH for 2.10] tcg: enable MTTCG by default for PPC64 on x86 Nikunj A Dadhania
2017-04-10 9:51 ` Alex Bennée [this message]
2017-04-11 6:53 ` David Gibson
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=87mvbok2te.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=clg@kaod.org \
--cc=david@gibson.dropbear.id.au \
--cc=intermediadc@hotmail.com \
--cc=nikunj@linux.vnet.ibm.com \
--cc=programmingkidx@gmail.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=rth@twiddle.net \
/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).