qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: stefanha@redhat.com, "Alex Bennée" <alex.bennee@linaro.org>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Alexandre Iooss" <erdnaxe@crans.org>,
	"Mahmoud Mandour" <ma.mandourr@gmail.com>
Subject: [PULL 28/31] contrib/plugins: enable debug on CONFIG_DEBUG_TCG
Date: Mon, 31 Oct 2022 13:10:07 +0000	[thread overview]
Message-ID: <20221031131010.682984-29-alex.bennee@linaro.org> (raw)
In-Reply-To: <20221031131010.682984-1-alex.bennee@linaro.org>

We used to rely on QEMU_CFLAGS to expose the debug flags but now this
is synthesised by meson and only available to the main build. Add our
own flags if we detect the build has been enabled with
CONFIG_DEBUG_TCG (which is the default for --enable-debug anyway).

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20221027183637.2772968-28-alex.bennee@linaro.org>

diff --git a/contrib/plugins/Makefile b/contrib/plugins/Makefile
index df3499f4f2..23e0396687 100644
--- a/contrib/plugins/Makefile
+++ b/contrib/plugins/Makefile
@@ -29,6 +29,7 @@ SONAMES := $(addsuffix .so,$(addprefix lib,$(NAMES)))
 CFLAGS = $(GLIB_CFLAGS)
 CFLAGS += -fPIC -Wall $(filter -W%, $(QEMU_CFLAGS))
 CFLAGS += $(if $(findstring no-psabi,$(QEMU_CFLAGS)),-Wpsabi)
+CFLAGS += $(if $(CONFIG_DEBUG_TCG), -ggdb -O0)
 CFLAGS += -I$(SRC_PATH)/include/qemu
 
 all: $(SONAMES)
-- 
2.34.1



  parent reply	other threads:[~2022-10-31 13:23 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-31 13:09 [PULL for 7.2 00/31] testing and plugins updates Alex Bennée
2022-10-31 13:09 ` [PULL 01/31] tests/lcitool: Rename non-Debian specific helper Alex Bennée
2022-10-31 13:09 ` [PULL 02/31] tests/lcitool: Refresh to latest libvirt-ci module Alex Bennée
2022-10-31 13:09 ` [PULL 03/31] tests/docker: update fedora-win[32|64]-cross with lcitool Alex Bennée
2022-10-31 13:09 ` [PULL 04/31] tests/docker: update test-mingw to run single build Alex Bennée
2022-10-31 13:09 ` [PULL 05/31] tests/docker: Add flex/bison to `debian-all-test` Alex Bennée
2022-10-31 13:09 ` [PULL 06/31] configure: don't enable cross compilers unless in target_list Alex Bennée
2022-10-31 13:09 ` [PULL 07/31] configure: fix the --enable-static --disable-pie case Alex Bennée
2022-10-31 13:09 ` [PULL 08/31] tests/avocado: extend the timeout for x86_64 tcg tests Alex Bennée
2022-10-31 13:09 ` [PULL 09/31] tests/docker: Add flex/bison to `debian-hexagon-cross` Alex Bennée
2022-10-31 13:09 ` [PULL 10/31] tests/tcg: use regular semihosting for nios2-softmmu Alex Bennée
2022-10-31 13:09 ` [PULL 11/31] tests/tcg/nios2: Tweak 10m50-ghrd.ld Alex Bennée
2022-10-31 13:09 ` [PULL 12/31] semihosting/arm-compat-semi: Avoid using hardcoded /tmp Alex Bennée
2022-10-31 13:09 ` [PULL 13/31] tcg: " Alex Bennée
2022-10-31 13:09 ` [PULL 14/31] block/vvfat: Unify the mkdir() call Alex Bennée
2022-10-31 13:09 ` [PULL 15/31] hw/usb: dev-mtp: Use g_mkdir() Alex Bennée
2022-10-31 13:09 ` [PULL 16/31] MAINTAINERS: add entries for the key build bits Alex Bennée
2022-10-31 13:09 ` [PULL 17/31] MAINTAINERS: add features_to_c.sh to gdbstub files Alex Bennée
2022-10-31 13:09 ` [PULL 18/31] MAINTAINERS: fix-up for check-tcg Makefile changes Alex Bennée
2022-10-31 13:09 ` [PULL 19/31] tests/avocado: set -machine none for userfwd and vnc tests Alex Bennée
2022-10-31 13:09 ` [PULL 20/31] tests/avocado: raspi2_initrd: Wait for guest shutdown message before stopping Alex Bennée
2022-10-31 13:10 ` [PULL 21/31] tests/avocado: disable sh4 rd2 tests on Gitlab Alex Bennée
2022-10-31 13:10 ` [PULL 22/31] tests/tcg: re-enable linux-test for sh4 Alex Bennée
2022-10-31 13:10 ` [PULL 23/31] tests/tcg: re-enable threadcount " Alex Bennée
2022-10-31 13:10 ` [PULL 24/31] target/s390x: don't use ld_code2 to probe next pc Alex Bennée
2022-10-31 13:10 ` [PULL 25/31] target/s390x: don't probe next pc for EXecuted insns Alex Bennée
2022-10-31 13:10 ` [PULL 26/31] target/s390x: fake instruction loading when handling 'ex' Alex Bennée
2022-10-31 13:10 ` [PULL 27/31] tests/tcg: include CONFIG_PLUGIN in config-host.mak Alex Bennée
2022-10-31 13:10 ` Alex Bennée [this message]
2022-10-31 13:10 ` [PULL 29/31] contrib/plugins: protect execlog's last_exec expansion Alex Bennée
2022-10-31 13:10 ` [PULL 30/31] tests/unit: cleanups for test-io-channel-command Alex Bennée
2022-10-31 13:10 ` [PULL 31/31] tests/vm: use -o IdentitiesOnly=yes for ssh Alex Bennée
2022-10-31 18:47 ` [PULL for 7.2 00/31] testing and plugins updates Stefan Hajnoczi

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=20221031131010.682984-29-alex.bennee@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=erdnaxe@crans.org \
    --cc=ma.mandourr@gmail.com \
    --cc=pbonzini@redhat.com \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.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).