* [PULL 1/7] tests/acceptance: tweak the tcg/kvm tests for virt
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-06-25 9:13 ` [PULL 2/7] docs/devel: Add a single top-level header to MTTCG's doc Alex Bennée
` (8 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell
Cc: Alex Bennée, qemu-devel, Wainer dos Santos Moschetta,
Willian Rampazzo, Cleber Rosa, Philippe Mathieu-Daudé
Really it's only TCG that can select which GIC model you want, KVM
guests should always be using the "host" version of the GIC for which
QEMU already provides a handy shortcut. Make the KVM test use this and
split the TCG test into it's two versions.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Message-Id: <20210623102749.25686-2-alex.bennee@linaro.org>
diff --git a/tests/acceptance/boot_linux.py b/tests/acceptance/boot_linux.py
index 314370fd1f..4c8a5994b2 100644
--- a/tests/acceptance/boot_linux.py
+++ b/tests/acceptance/boot_linux.py
@@ -75,10 +75,11 @@ def add_common_args(self):
self.vm.add_args('-device', 'virtio-rng-pci,rng=rng0')
self.vm.add_args('-object', 'rng-random,id=rng0,filename=/dev/urandom')
- def test_virt_tcg(self):
+ def test_virt_tcg_gicv2(self):
"""
:avocado: tags=accel:tcg
:avocado: tags=cpu:max
+ :avocado: tags=device:gicv2
"""
self.require_accelerator("tcg")
self.vm.add_args("-accel", "tcg")
@@ -87,29 +88,28 @@ def test_virt_tcg(self):
self.add_common_args()
self.launch_and_wait(set_up_ssh_connection=False)
- def test_virt_kvm_gicv2(self):
+ def test_virt_tcg_gicv3(self):
"""
- :avocado: tags=accel:kvm
- :avocado: tags=cpu:host
- :avocado: tags=device:gicv2
+ :avocado: tags=accel:tcg
+ :avocado: tags=cpu:max
+ :avocado: tags=device:gicv3
"""
- self.require_accelerator("kvm")
- self.vm.add_args("-accel", "kvm")
- self.vm.add_args("-cpu", "host")
- self.vm.add_args("-machine", "virt,gic-version=2")
+ self.require_accelerator("tcg")
+ self.vm.add_args("-accel", "tcg")
+ self.vm.add_args("-cpu", "max")
+ self.vm.add_args("-machine", "virt,gic-version=3")
self.add_common_args()
self.launch_and_wait(set_up_ssh_connection=False)
- def test_virt_kvm_gicv3(self):
+ def test_virt_kvm(self):
"""
:avocado: tags=accel:kvm
:avocado: tags=cpu:host
- :avocado: tags=device:gicv3
"""
self.require_accelerator("kvm")
self.vm.add_args("-accel", "kvm")
self.vm.add_args("-cpu", "host")
- self.vm.add_args("-machine", "virt,gic-version=3")
+ self.vm.add_args("-machine", "virt,gic-version=host")
self.add_common_args()
self.launch_and_wait(set_up_ssh_connection=False)
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PULL 2/7] docs/devel: Add a single top-level header to MTTCG's doc
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
2021-06-25 9:13 ` [PULL 1/7] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-06-25 9:13 ` [PULL 3/7] scripts/checkpatch: roll diff tweaking into checkpatch itself Alex Bennée
` (7 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell; +Cc: Luis Pires, Alex Bennée, qemu-devel
From: Luis Pires <luis.pires@eldorado.org.br>
Without a single top-level header in the .rst file, the index ended
up linking to all of the top-level headers separately. Now the index
links to the top-level header at the beginning of the document and
any inner headers are correctly linked as sub-items in the index.
Signed-off-by: Luis Pires <luis.pires@eldorado.org.br>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210528123526.144065-1-luis.pires@eldorado.org.br>
Message-Id: <20210623102749.25686-3-alex.bennee@linaro.org>
diff --git a/docs/devel/multi-thread-tcg.rst b/docs/devel/multi-thread-tcg.rst
index 92a9eba13c..5b446ee08b 100644
--- a/docs/devel/multi-thread-tcg.rst
+++ b/docs/devel/multi-thread-tcg.rst
@@ -4,8 +4,9 @@
This work is licensed under the terms of the GNU GPL, version 2 or
later. See the COPYING file in the top-level directory.
-Introduction
-============
+==================
+Multi-threaded TCG
+==================
This document outlines the design for multi-threaded TCG (a.k.a MTTCG)
system-mode emulation. user-mode emulation has always mirrored the
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PULL 3/7] scripts/checkpatch: roll diff tweaking into checkpatch itself
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
2021-06-25 9:13 ` [PULL 1/7] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
2021-06-25 9:13 ` [PULL 2/7] docs/devel: Add a single top-level header to MTTCG's doc Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-06-25 9:13 ` [PULL 4/7] GitLab: Add "Bug" issue reporting template Alex Bennée
` (6 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell
Cc: Thomas Huth, Daniel P . Berrangé,
Philippe Mathieu-Daudé,
qemu-devel, Wainer dos Santos Moschetta,
Philippe Mathieu-Daudé,
Willian Rampazzo, Alex Bennée
Rather than relying on external tweaks lets just do it inside
checkpatch's direct commitish handling which is QEMU specific code
anyway.
Suggested-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Message-Id: <20210623102749.25686-4-alex.bennee@linaro.org>
diff --git a/.gitlab-ci.d/static_checks.yml b/.gitlab-ci.d/static_checks.yml
index 7e685c6a65..c5fa4fce26 100644
--- a/.gitlab-ci.d/static_checks.yml
+++ b/.gitlab-ci.d/static_checks.yml
@@ -4,9 +4,6 @@ check-patch:
needs:
job: amd64-centos8-container
script:
- - git config --local diff.renamelimit 0
- - git config --local diff.renames True
- - git config --local diff.algorithm histogram
- .gitlab-ci.d/check-patch.py
variables:
GIT_DEPTH: 1000
diff --git a/.patchew.yml b/.patchew.yml
index 2638b7f564..1b78262ce5 100644
--- a/.patchew.yml
+++ b/.patchew.yml
@@ -138,9 +138,6 @@ testing:
script: |
#!/bin/bash
git rev-parse base > /dev/null || exit 0
- git config --local diff.renamelimit 0
- git config --local diff.renames True
- git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
docker-mingw@fedora:
enabled: true
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index bbcd25ae05..cb8eff233e 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -399,7 +399,12 @@ if ($chk_branch) {
my $num_patches = @patches;
for my $hash (@patches) {
my $FILE;
- open($FILE, '-|', "git", "show", "--patch-with-stat", $hash) ||
+ open($FILE, '-|', "git",
+ "-c", "diff.renamelimit=0",
+ "-c", "diff.renames=True",
+ "-c", "diff.algorithm=histogram",
+ "show",
+ "--patch-with-stat", $hash) ||
die "$P: git show $hash - $!\n";
while (<$FILE>) {
chomp;
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PULL 4/7] GitLab: Add "Bug" issue reporting template
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (2 preceding siblings ...)
2021-06-25 9:13 ` [PULL 3/7] scripts/checkpatch: roll diff tweaking into checkpatch itself Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-06-25 9:13 ` [PULL 5/7] GitLab: Add "Feature Request" issue template Alex Bennée
` (5 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell
Cc: Alex Bennée, Peter Krempa, John Snow, qemu-devel, Stefan Hajnoczi
From: John Snow <jsnow@redhat.com>
Based loosely on libvirt's template, written by Peter Krempa.
Signed-off-by: John Snow <jsnow@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
CC: Peter Krempa <pkrempa@redhat.com>
Message-Id: <20210607153155.1760158-2-jsnow@redhat.com>
Message-Id: <20210623102749.25686-5-alex.bennee@linaro.org>
diff --git a/.gitlab/issue_templates/bug.md b/.gitlab/issue_templates/bug.md
new file mode 100644
index 0000000000..e910f7b1c2
--- /dev/null
+++ b/.gitlab/issue_templates/bug.md
@@ -0,0 +1,64 @@
+<!--
+This is the upstream QEMU issue tracker.
+
+If you are able to, it will greatly facilitate bug triage if you attempt
+to reproduce the problem with the latest qemu.git master built from
+source. See https://www.qemu.org/download/#source for instructions on
+how to do this.
+
+QEMU generally supports the last two releases advertised on
+https://www.qemu.org/. Problems with distro-packaged versions of QEMU
+older than this should be reported to the distribution instead.
+
+See https://www.qemu.org/contribute/report-a-bug/ for additional
+guidance.
+
+If this is a security issue, please consult
+https://www.qemu.org/contribute/security-process/
+-->
+
+## Host environment
+ - Operating system: (Windows 10 21H1, Fedora 34, etc.)
+ - OS/kernel version: (For POSIX hosts, use `uname -a`)
+ - Architecture: (x86, ARM, s390x, etc.)
+ - QEMU flavor: (qemu-system-x86_64, qemu-aarch64, qemu-img, etc.)
+ - QEMU version: (e.g. `qemu-system-x86_64 --version`)
+ - QEMU command line:
+ <!--
+ Give the smallest, complete command line that exhibits the problem.
+
+ If you are using libvirt, virsh, or vmm, you can likely find the QEMU
+ command line arguments in /var/log/libvirt/qemu/$GUEST.log.
+ -->
+ ```
+ ./qemu-system-x86_64 -M q35 -m 4096 -enable-kvm -hda fedora32.qcow2
+ ```
+
+## Emulated/Virtualized environment
+ - Operating system: (Windows 10 21H1, Fedora 34, etc.)
+ - OS/kernel version: (For POSIX guests, use `uname -a`.)
+ - Architecture: (x86, ARM, s390x, etc.)
+
+
+## Description of problem
+<!-- Describe the problem, including any error/crash messages seen. -->
+
+
+## Steps to reproduce
+1.
+2.
+3.
+
+
+## Additional information
+
+<!--
+Attach logs, stack traces, screenshots, etc. Compress the files if necessary.
+If using libvirt, libvirt logs and XML domain information may be relevant.
+-->
+
+<!--
+The line below ensures that proper tags are added to the issue.
+Please do not remove it.
+-->
+/label ~"kind::Bug"
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PULL 5/7] GitLab: Add "Feature Request" issue template.
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (3 preceding siblings ...)
2021-06-25 9:13 ` [PULL 4/7] GitLab: Add "Bug" issue reporting template Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-06-25 9:13 ` [PULL 6/7] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
` (4 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell
Cc: Thomas Huth, Peter Krempa, John Snow, qemu-devel,
Stefan Hajnoczi, Alex Bennée
From: John Snow <jsnow@redhat.com>
Based on Peter Krempa's libvirt template, feature.md.
Signed-off-by: John Snow <jsnow@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
CC: Peter Krempa <pkrempa@redhat.com>
Message-Id: <20210607153155.1760158-3-jsnow@redhat.com>
Message-Id: <20210623102749.25686-6-alex.bennee@linaro.org>
diff --git a/.gitlab/issue_templates/feature_request.md b/.gitlab/issue_templates/feature_request.md
new file mode 100644
index 0000000000..7de02dcc2c
--- /dev/null
+++ b/.gitlab/issue_templates/feature_request.md
@@ -0,0 +1,32 @@
+<!--
+This is the upstream QEMU issue tracker.
+
+Please note that QEMU, like most open source projects, relies on
+contributors who have motivation, skills and available time to work on
+implementing particular features.
+
+Feature requests can be helpful for determining demand and interest, but
+they are not a guarantee that a contributor will volunteer to implement
+it. We welcome and encourage even draft patches to implement a feature
+be sent to the mailing list where it can be discussed and developed
+further by the community.
+
+Thank you for your interest in helping us to make QEMU better!
+-->
+
+## Goal
+<!-- Describe the final result you want to achieve. Avoid design specifics. -->
+
+
+## Technical details
+<!-- Describe technical details, design specifics, suggestions, versions, etc. -->
+
+
+## Additional information
+<!-- Patch or branch references, any other useful information -->
+
+<!--
+The line below ensures that proper tags are added to the issue.
+Please do not remove it.
+-->
+/label ~"kind::Feature Request"
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PULL 6/7] tests/tcg: skip the signals test for hppa/s390x for now
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (4 preceding siblings ...)
2021-06-25 9:13 ` [PULL 5/7] GitLab: Add "Feature Request" issue template Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-09-11 21:57 ` Philippe Mathieu-Daudé
2021-06-25 9:13 ` [PULL 7/7] plugins/api: expose symbol lookup to plugins Alex Bennée
` (3 subsequent siblings)
9 siblings, 1 reply; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell
Cc: Thomas Huth, David Hildenbrand, Cornelia Huck, Richard Henderson,
qemu-devel, open list:S390 TCG CPUs, Alex Bennée
There are fixes currently in flight but as this is getting in the way
of a green CI we might as well skip for now. For reference the fix
series are:
linux-user: Move signal trampolines to new page
20210616011209.1446045-1-richard.henderson@linaro.org
and
linux-user: Load a vdso for x86_64 and hppa
20210619034329.532318-1-richard.henderson@linaro.org
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Cornelia Huck <cohuck@redhat.com>
Cc: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20210623102749.25686-7-alex.bennee@linaro.org>
diff --git a/tests/tcg/hppa/Makefile.target b/tests/tcg/hppa/Makefile.target
index 8bf01966bd..71791235f6 100644
--- a/tests/tcg/hppa/Makefile.target
+++ b/tests/tcg/hppa/Makefile.target
@@ -4,3 +4,7 @@
# On parisc Linux supports 4K/16K/64K (but currently only 4k works)
EXTRA_RUNS+=run-test-mmap-4096 # run-test-mmap-16384 run-test-mmap-65536
+
+# There is a race that causes this to fail about 1% of the time
+run-signals: signals
+ $(call skip-test, $<, "BROKEN awaiting vdso support")
diff --git a/tests/tcg/s390x/Makefile.target b/tests/tcg/s390x/Makefile.target
index 241ef28f61..0036b8a505 100644
--- a/tests/tcg/s390x/Makefile.target
+++ b/tests/tcg/s390x/Makefile.target
@@ -8,3 +8,7 @@ TESTS+=exrl-trtr
TESTS+=pack
TESTS+=mvo
TESTS+=mvc
+
+# This triggers failures on s390x hosts about 4% of the time
+run-signals: signals
+ $(call skip-test, $<, "BROKEN awaiting sigframe clean-ups")
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PULL 6/7] tests/tcg: skip the signals test for hppa/s390x for now
2021-06-25 9:13 ` [PULL 6/7] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
@ 2021-09-11 21:57 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-09-11 21:57 UTC (permalink / raw)
To: Alex Bennée
Cc: Peter Maydell, Thomas Huth, David Hildenbrand, Cornelia Huck,
Richard Henderson, qemu-devel@nongnu.org Developers,
open list:S390 TCG CPUs
On Fri, Jun 25, 2021 at 11:21 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> There are fixes currently in flight but as this is getting in the way
> of a green CI we might as well skip for now. For reference the fix
> series are:
>
> linux-user: Move signal trampolines to new page
> 20210616011209.1446045-1-richard.henderson@linaro.org
>
> and
>
> linux-user: Load a vdso for x86_64 and hppa
> 20210619034329.532318-1-richard.henderson@linaro.org
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> Acked-by: Cornelia Huck <cohuck@redhat.com>
> Cc: Richard Henderson <richard.henderson@linaro.org>
> Message-Id: <20210623102749.25686-7-alex.bennee@linaro.org>
>
> diff --git a/tests/tcg/hppa/Makefile.target b/tests/tcg/hppa/Makefile.target
> index 8bf01966bd..71791235f6 100644
> --- a/tests/tcg/hppa/Makefile.target
> +++ b/tests/tcg/hppa/Makefile.target
> @@ -4,3 +4,7 @@
>
> # On parisc Linux supports 4K/16K/64K (but currently only 4k works)
> EXTRA_RUNS+=run-test-mmap-4096 # run-test-mmap-16384 run-test-mmap-65536
> +
> +# There is a race that causes this to fail about 1% of the time
> +run-signals: signals
> + $(call skip-test, $<, "BROKEN awaiting vdso support")
> diff --git a/tests/tcg/s390x/Makefile.target b/tests/tcg/s390x/Makefile.target
> index 241ef28f61..0036b8a505 100644
> --- a/tests/tcg/s390x/Makefile.target
> +++ b/tests/tcg/s390x/Makefile.target
> @@ -8,3 +8,7 @@ TESTS+=exrl-trtr
> TESTS+=pack
> TESTS+=mvo
> TESTS+=mvc
> +
> +# This triggers failures on s390x hosts about 4% of the time
> +run-signals: signals
> + $(call skip-test, $<, "BROKEN awaiting sigframe clean-ups")
Odd, still failing?
https://app.travis-ci.com/gitlab/qemu-project/qemu/jobs/536639512#L12456
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PULL 7/7] plugins/api: expose symbol lookup to plugins
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (5 preceding siblings ...)
2021-06-25 9:13 ` [PULL 6/7] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
@ 2021-06-25 9:13 ` Alex Bennée
2021-06-25 14:28 ` [PULL 0/7] misc fixes (docs, plugins, tests) Peter Maydell
` (2 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Alex Bennée @ 2021-06-25 9:13 UTC (permalink / raw)
To: peter.maydell; +Cc: Mahmoud Mandour, Alex Bennée, qemu-devel
This is a quality of life helper for plugins so they don't need to
re-implement symbol lookup when dumping an address. The strings are
constant so don't need to be duplicated. One minor tweak is to return
NULL instead of a zero length string to show lookup failed.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Mahmoud Mandour <ma.mandourr@gmail.com>
Message-Id: <20210608040532.56449-2-ma.mandourr@gmail.com>
Message-Id: <20210623102749.25686-8-alex.bennee@linaro.org>
diff --git a/include/qemu/qemu-plugin.h b/include/qemu/qemu-plugin.h
index 97cdfd7761..dc3496f36c 100644
--- a/include/qemu/qemu-plugin.h
+++ b/include/qemu/qemu-plugin.h
@@ -525,6 +525,15 @@ qemu_plugin_register_vcpu_syscall_ret_cb(qemu_plugin_id_t id,
char *qemu_plugin_insn_disas(const struct qemu_plugin_insn *insn);
+/**
+ * qemu_plugin_insn_symbol() - best effort symbol lookup
+ * @insn: instruction reference
+ *
+ * Return a static string referring to the symbol. This is dependent
+ * on the binary QEMU is running having provided a symbol table.
+ */
+const char *qemu_plugin_insn_symbol(const struct qemu_plugin_insn *insn);
+
/**
* qemu_plugin_vcpu_for_each() - iterate over the existing vCPU
* @id: plugin ID
diff --git a/plugins/api.c b/plugins/api.c
index 817c9b6b69..332e2c60e2 100644
--- a/plugins/api.c
+++ b/plugins/api.c
@@ -233,6 +233,12 @@ char *qemu_plugin_insn_disas(const struct qemu_plugin_insn *insn)
return plugin_disas(cpu, insn->vaddr, insn->data->len);
}
+const char *qemu_plugin_insn_symbol(const struct qemu_plugin_insn *insn)
+{
+ const char *sym = lookup_symbol(insn->vaddr);
+ return sym[0] != 0 ? sym : NULL;
+}
+
/*
* The memory queries allow the plugin to query information about a
* memory access.
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PULL 0/7] misc fixes (docs, plugins, tests)
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (6 preceding siblings ...)
2021-06-25 9:13 ` [PULL 7/7] plugins/api: expose symbol lookup to plugins Alex Bennée
@ 2021-06-25 14:28 ` Peter Maydell
2021-06-25 18:39 ` Peter Maydell
2021-06-28 17:42 ` Peter Maydell
9 siblings, 0 replies; 12+ messages in thread
From: Peter Maydell @ 2021-06-25 14:28 UTC (permalink / raw)
To: Alex Bennée; +Cc: QEMU Developers
On Fri, 25 Jun 2021 at 10:14, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> The following changes since commit ecba223da6215d6f6ce2d343b70b2e9a19bfb90b:
>
> Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20210624' into staging (2021-06-24 15:00:34 +0100)
>
> are available in the Git repository at:
>
> https://github.com/stsquad/qemu.git tags/pull-testing-updates-250621-1
>
> for you to fetch changes up to 7c4ab60f18f5257d37164df0ae0951ece4425b02:
>
> plugins/api: expose symbol lookup to plugins (2021-06-25 10:08:37 +0100)
>
> ----------------------------------------------------------------
> A few miscellaneous fixes
>
> - tweak tcg/kvm based GIC tests
> - add header to MTTCG docs
> - cleanup checkpatch handling
> - GitLab feature and bug request templates
> - symbol resolution helper for plugin API
> - skip hppa/s390x signals test until fixes arrive
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/6.1
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PULL 0/7] misc fixes (docs, plugins, tests)
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (7 preceding siblings ...)
2021-06-25 14:28 ` [PULL 0/7] misc fixes (docs, plugins, tests) Peter Maydell
@ 2021-06-25 18:39 ` Peter Maydell
2021-06-28 17:42 ` Peter Maydell
9 siblings, 0 replies; 12+ messages in thread
From: Peter Maydell @ 2021-06-25 18:39 UTC (permalink / raw)
To: Alex Bennée; +Cc: QEMU Developers
On Fri, 25 Jun 2021 at 10:14, Alex Bennée <alex.bennee@linaro.org> wrote:
> ----------------------------------------------------------------
> A few miscellaneous fixes
>
> - tweak tcg/kvm based GIC tests
> - add header to MTTCG docs
> - cleanup checkpatch handling
> - GitLab feature and bug request templates
> - symbol resolution helper for plugin API
> - skip hppa/s390x signals test until fixes arrive
>
> ----------------------------------------------------------------
Since I applied this, the gitlab CI has gone green again, which is nice.
However, it also seems to now only be running the "external" Travis
job for "master", not on the pre-merge "staging":
https://gitlab.com/qemu-project/qemu/-/pipelines
Could we make it do it the other way around, so we can detect
Travis CI failures before pushing the merges to master ?
thanks
-- PMM
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PULL 0/7] misc fixes (docs, plugins, tests)
2021-06-25 9:13 [PULL 0/7] misc fixes (docs, plugins, tests) Alex Bennée
` (8 preceding siblings ...)
2021-06-25 18:39 ` Peter Maydell
@ 2021-06-28 17:42 ` Peter Maydell
9 siblings, 0 replies; 12+ messages in thread
From: Peter Maydell @ 2021-06-28 17:42 UTC (permalink / raw)
To: Alex Bennée; +Cc: QEMU Developers
On Fri, 25 Jun 2021 at 10:14, Alex Bennée <alex.bennee@linaro.org> wrote:
> - skip hppa/s390x signals test until fixes arrive
FWIW, an intermittent I just saw (x86 host):
timeout --foreground 60
/home/petmay01/linaro/qemu-for-merges/build/all-linux-static/qemu-sh4
signals > signals.out
qemu: uncaught target signal 11 (Segmentation fault) - core dumped
timeout: the monitored command dumped core
Segmentation fault
../Makefile.target:158: recipe for target 'run-signals' failed
thanks
-- PMM
^ permalink raw reply [flat|nested] 12+ messages in thread