From: Paolo Bonzini <pbonzini@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>, Claudio Fontana <cfontana@suse.de>
Cc: qemu-devel@nongnu.org, richard.henderson@linaro.org,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
"Jose R. Ziviani" <jziviani@suse.de>
Subject: Re: modular tcg (was: Re: [PATCH v2 1/1] modules: Improve error message when module is not) found
Date: Thu, 29 Jul 2021 18:40:17 +0200 [thread overview]
Message-ID: <80e5ceaf-a425-2343-ef2e-d33b6ff87389@redhat.com> (raw)
In-Reply-To: <20210729091407.n7bdlyw5rsievdch@sirius.home.kraxel.org>
On 29/07/21 11:14, Gerd Hoffmann wrote:
> The common functions could be added TCGCPUOps to allow them being called via
> CPUClass->tcg_ops->$name instead of a direct symbol reference. Not sure this
> is a good idea though. Experimental patch covering tcg_exec_realizefn +
> tcg_exec_unrealizefn below.
A lot of these (though probably not all) are already stubbed out as
"static inline" in include/exec/exec-all.h. I think they can be changed
to function pointers in the style of ui/spice-module.c
(accel/tcg/tcg-module.c?).
> No idea yet how to handle arch-specific bits best. Seems there is no existing
> infrastructure to untangle target-specific code and tcg, so this probably needs
> something new.
If they are few enough, I would just move them out of target/i386/tcg
and into target/i386/cpu-sysemu.c.
> Noticed softmmu/physmem.c has lots of CONFIG_TCG #ifdefs, splitting this into
> softmmu/physmem-{common,tcg}.c is probably a good idea.
I only count one, and those function should be easily moved to
accel/tcg/cputlb.c (after all both physmem.c and cputlb.c used to be a
single file, exec.c, so this is just an oversight).
Paolo
next prev parent reply other threads:[~2021-07-29 16:41 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-22 22:09 [PATCH v2 0/1] Improve module accelerator error message Jose R. Ziviani
2021-07-22 22:09 ` [PATCH v2 1/1] modules: Improve error message when module is not found Jose R. Ziviani
2021-07-23 6:25 ` Gerd Hoffmann
2021-07-23 8:04 ` Claudio Fontana
2021-07-23 8:28 ` Markus Armbruster
2021-07-23 8:32 ` Claudio Fontana
2021-07-23 9:41 ` Claudio Fontana
2021-07-23 9:52 ` Gerd Hoffmann
2021-07-23 11:20 ` Claudio Fontana
2021-07-23 12:20 ` Claudio Fontana
2021-07-23 12:48 ` Gerd Hoffmann
2021-07-29 9:14 ` modular tcg (was: Re: [PATCH v2 1/1] modules: Improve error message when module is not) found Gerd Hoffmann
2021-07-29 9:40 ` modular tcg Claudio Fontana
2021-07-29 10:26 ` Gerd Hoffmann
2021-07-29 10:42 ` Claudio Fontana
2021-07-29 9:42 ` Claudio Fontana
2021-07-29 10:29 ` Gerd Hoffmann
2021-07-29 10:44 ` Claudio Fontana
2021-07-29 11:34 ` Philippe Mathieu-Daudé
2021-07-29 11:39 ` Claudio Fontana
2021-07-29 14:22 ` Claudio Fontana
2021-07-29 14:59 ` Philippe Mathieu-Daudé
2021-07-29 16:35 ` Claudio Fontana
2021-07-29 16:40 ` Paolo Bonzini [this message]
2021-07-30 9:05 ` Gerd Hoffmann
2021-07-30 10:02 ` Claudio Fontana
2022-09-02 9:50 ` modular tcg (was: Re: [PATCH v2 1/1] modules: Improve error message when module is not) found Claudio Fontana
2021-07-23 13:50 ` [PATCH v2 1/1] modules: Improve error message when module is not found Jose R. Ziviani
2021-07-23 14:02 ` Claudio Fontana
2021-07-23 14:14 ` Philippe Mathieu-Daudé
2021-07-23 14:36 ` Jose R. Ziviani
2021-07-23 15:27 ` Claudio Fontana
2021-07-23 15:46 ` Jose R. Ziviani
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=80e5ceaf-a425-2343-ef2e-d33b6ff87389@redhat.com \
--to=pbonzini@redhat.com \
--cc=cfontana@suse.de \
--cc=f4bug@amsat.org \
--cc=jziviani@suse.de \
--cc=kraxel@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.