All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH  v1 0/8] misc fixes pre-PR (docs, plugins, tests)
@ 2021-06-23 10:27 Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
                   ` (7 more replies)
  0 siblings, 8 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel; +Cc: Alex Bennée

Hi,

This is a pre-PR series which collects up some miscellaneous random
bits and pieces. This includes various documentation tweaks, a minor
plugin helper API and a temporary suppression of the signals test for
s390x/hppa to keep CI green.

I should roll the PR on Friday morning so shout now if you want to
object to anything.

The following patches are un-reviewed:

 - tests/tcg: skip the signals test for hppa/s390x for now
 - tests/acceptance: tweak the tcg/kvm tests for virt


Alex Bennée (4):
  tests/acceptance: tweak the tcg/kvm tests for virt
  scripts/checkpatch: roll diff tweaking into checkpatch itself
  tests/tcg: skip the signals test for hppa/s390x for now
  plugins/api: expose symbol lookup to plugins

John Snow (2):
  GitLab: Add "Bug" issue reporting template
  GitLab: Add "Feature Request" issue template.

Luis Pires (1):
  docs/devel: Add a single top-level header to MTTCG's doc

Stefan Weil (1):
  Update documentation to refer to new location for issues

 docs/devel/multi-thread-tcg.rst            |  5 +-
 README.rst                                 |  6 +-
 include/qemu/qemu-plugin.h                 |  9 +++
 plugins/api.c                              |  6 ++
 .github/lockdown.yml                       |  6 +-
 .gitlab-ci.d/static_checks.yml             |  3 -
 .gitlab/issue_templates/bug.md             | 64 ++++++++++++++++++++++
 .gitlab/issue_templates/feature_request.md | 32 +++++++++++
 .patchew.yml                               |  3 -
 scripts/checkpatch.pl                      |  7 ++-
 tests/acceptance/boot_linux.py             | 24 ++++----
 tests/tcg/hppa/Makefile.target             |  4 ++
 tests/tcg/s390x/Makefile.target            |  4 ++
 13 files changed, 146 insertions(+), 27 deletions(-)
 create mode 100644 .gitlab/issue_templates/bug.md
 create mode 100644 .gitlab/issue_templates/feature_request.md

-- 
2.20.1



^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH  v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 11:18   ` Willian Rampazzo
  2021-06-23 10:27 ` [PATCH v1 2/8] docs/devel: Add a single top-level header to MTTCG's doc Alex Bennée
                   ` (6 subsequent siblings)
  7 siblings, 1 reply; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel
  Cc: Philippe Mathieu-Daudé,
	Alex Bennée, Wainer dos Santos Moschetta, Cleber Rosa

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>
---
 tests/acceptance/boot_linux.py | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

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	[flat|nested] 11+ messages in thread

* [PATCH v1 2/8] docs/devel: Add a single top-level header to MTTCG's doc
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 3/8] scripts/checkpatch: roll diff tweaking into checkpatch itself Alex Bennée
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel; +Cc: Luis Pires, Alex Bennée

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>
---
 docs/devel/multi-thread-tcg.rst | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

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	[flat|nested] 11+ messages in thread

* [PATCH v1 3/8] scripts/checkpatch: roll diff tweaking into checkpatch itself
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 2/8] docs/devel: Add a single top-level header to MTTCG's doc Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 4/8] GitLab: Add "Bug" issue reporting template Alex Bennée
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel
  Cc: Thomas Huth, Daniel P . Berrangé,
	Philippe Mathieu-Daudé, Philippe Mathieu-Daudé,
	Wainer dos Santos Moschetta, 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: <20210607171829.25111-1-alex.bennee@linaro.org>
---
 .gitlab-ci.d/static_checks.yml | 3 ---
 .patchew.yml                   | 3 ---
 scripts/checkpatch.pl          | 7 ++++++-
 3 files changed, 6 insertions(+), 7 deletions(-)

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	[flat|nested] 11+ messages in thread

* [PATCH  v1 4/8] GitLab: Add "Bug" issue reporting template
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
                   ` (2 preceding siblings ...)
  2021-06-23 10:27 ` [PATCH v1 3/8] scripts/checkpatch: roll diff tweaking into checkpatch itself Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 5/8] GitLab: Add "Feature Request" issue template Alex Bennée
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel; +Cc: Alex Bennée, Peter Krempa, John Snow, 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>
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>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .gitlab/issue_templates/bug.md | 64 ++++++++++++++++++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 .gitlab/issue_templates/bug.md

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	[flat|nested] 11+ messages in thread

* [PATCH  v1 5/8] GitLab: Add "Feature Request" issue template.
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
                   ` (3 preceding siblings ...)
  2021-06-23 10:27 ` [PATCH v1 4/8] GitLab: Add "Bug" issue reporting template Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 6/8] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel
  Cc: Alex Bennée, Thomas Huth, Peter Krempa, John Snow, Stefan Hajnoczi

From: John Snow <jsnow@redhat.com>

Based on Peter Krempa's libvirt template, feature.md.

Signed-off-by: John Snow <jsnow@redhat.com>
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>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .gitlab/issue_templates/feature_request.md | 32 ++++++++++++++++++++++
 1 file changed, 32 insertions(+)
 create mode 100644 .gitlab/issue_templates/feature_request.md

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	[flat|nested] 11+ messages in thread

* [PATCH v1 6/8] tests/tcg: skip the signals test for hppa/s390x for now
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
                   ` (4 preceding siblings ...)
  2021-06-23 10:27 ` [PATCH v1 5/8] GitLab: Add "Feature Request" issue template Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 10:43   ` Cornelia Huck
  2021-06-23 10:27 ` [PATCH v1 7/8] plugins/api: expose symbol lookup to plugins Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 8/8] Update documentation to refer to new location for issues Alex Bennée
  7 siblings, 1 reply; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel
  Cc: Thomas Huth, David Hildenbrand, Cornelia Huck, Richard Henderson,
	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>
Cc: Richard Henderson <richard.henderson@linaro.org>
---
 tests/tcg/hppa/Makefile.target  | 4 ++++
 tests/tcg/s390x/Makefile.target | 4 ++++
 2 files changed, 8 insertions(+)

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	[flat|nested] 11+ messages in thread

* [PATCH  v1 7/8] plugins/api: expose symbol lookup to plugins
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
                   ` (5 preceding siblings ...)
  2021-06-23 10:27 ` [PATCH v1 6/8] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  2021-06-23 10:27 ` [PATCH v1 8/8] Update documentation to refer to new location for issues Alex Bennée
  7 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel; +Cc: Mahmoud Mandour, Alex Bennée

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>
Message-Id: <20210601145824.3849-1-alex.bennee@linaro.org>
Signed-off-by: Mahmoud Mandour <ma.mandourr@gmail.com>
Message-Id: <20210608040532.56449-2-ma.mandourr@gmail.com>
---
 include/qemu/qemu-plugin.h | 9 +++++++++
 plugins/api.c              | 6 ++++++
 2 files changed, 15 insertions(+)

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	[flat|nested] 11+ messages in thread

* [PATCH v1 8/8] Update documentation to refer to new location for issues
  2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
                   ` (6 preceding siblings ...)
  2021-06-23 10:27 ` [PATCH v1 7/8] plugins/api: expose symbol lookup to plugins Alex Bennée
@ 2021-06-23 10:27 ` Alex Bennée
  7 siblings, 0 replies; 11+ messages in thread
From: Alex Bennée @ 2021-06-23 10:27 UTC (permalink / raw)
  To: qemu-devel
  Cc: Thomas Huth, Stefan Weil, Philippe Mathieu-Daudé,
	Wainer dos Santos Moschetta, Willian Rampazzo, Alex Bennée

From: Stefan Weil <sw@weilnetz.de>

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20210619154253.553446-1-sw@weilnetz.de>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 README.rst           | 6 +++---
 .github/lockdown.yml | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/README.rst b/README.rst
index a92c7394b7..79b19f1481 100644
--- a/README.rst
+++ b/README.rst
@@ -131,16 +131,16 @@ will be tagged as my-feature-v2.
 Bug reporting
 =============
 
-The QEMU project uses Launchpad as its primary upstream bug tracker. Bugs
+The QEMU project uses GitLab issues to track bugs. Bugs
 found when running code built from QEMU git or upstream released sources
 should be reported via:
 
-* `<https://bugs.launchpad.net/qemu/>`_
+* `<https://gitlab.com/qemu-project/qemu/-/issues>`_
 
 If using QEMU via an operating system vendor pre-built binary package, it
 is preferable to report bugs to the vendor's own bug tracker first. If
 the bug is also known to affect latest upstream code, it can also be
-reported via launchpad.
+reported via GitLab.
 
 For additional information on bug reporting consult:
 
diff --git a/.github/lockdown.yml b/.github/lockdown.yml
index 07fc2f31ee..d3546bd2bc 100644
--- a/.github/lockdown.yml
+++ b/.github/lockdown.yml
@@ -14,11 +14,11 @@ issues:
     at https://gitlab.com/qemu-project/qemu.git.
     The project does not process issues filed on GitHub.
 
-    The project issues are tracked on Launchpad:
-    https://bugs.launchpad.net/qemu
+    The project issues are tracked on GitLab:
+    https://gitlab.com/qemu-project/qemu/-/issues
 
     QEMU welcomes bug report contributions. You can file new ones on:
-    https://bugs.launchpad.net/qemu/+filebug
+    https://gitlab.com/qemu-project/qemu/-/issues/new
 
 pulls:
   comment: |
-- 
2.20.1



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 6/8] tests/tcg: skip the signals test for hppa/s390x for now
  2021-06-23 10:27 ` [PATCH v1 6/8] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
@ 2021-06-23 10:43   ` Cornelia Huck
  0 siblings, 0 replies; 11+ messages in thread
From: Cornelia Huck @ 2021-06-23 10:43 UTC (permalink / raw)
  To: Alex Bennée, qemu-devel
  Cc: Alex Bennée, Thomas Huth, Richard Henderson,
	open list:S390 TCG CPUs, David Hildenbrand

On Wed, Jun 23 2021, 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>
> Cc: Richard Henderson <richard.henderson@linaro.org>
> ---
>  tests/tcg/hppa/Makefile.target  | 4 ++++
>  tests/tcg/s390x/Makefile.target | 4 ++++
>  2 files changed, 8 insertions(+)

Acked-by: Cornelia Huck <cohuck@redhat.com>



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt
  2021-06-23 10:27 ` [PATCH v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
@ 2021-06-23 11:18   ` Willian Rampazzo
  0 siblings, 0 replies; 11+ messages in thread
From: Willian Rampazzo @ 2021-06-23 11:18 UTC (permalink / raw)
  To: Alex Bennée
  Cc: Philippe Mathieu-Daudé,
	qemu-devel, Wainer dos Santos Moschetta, Cleber Rosa

On Wed, Jun 23, 2021 at 7:30 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> 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>
> ---
>  tests/acceptance/boot_linux.py | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
>

Reviewed-by: Willian Rampazzo <willianr@redhat.com>



^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2021-06-23 11:20 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-23 10:27 [PATCH v1 0/8] misc fixes pre-PR (docs, plugins, tests) Alex Bennée
2021-06-23 10:27 ` [PATCH v1 1/8] tests/acceptance: tweak the tcg/kvm tests for virt Alex Bennée
2021-06-23 11:18   ` Willian Rampazzo
2021-06-23 10:27 ` [PATCH v1 2/8] docs/devel: Add a single top-level header to MTTCG's doc Alex Bennée
2021-06-23 10:27 ` [PATCH v1 3/8] scripts/checkpatch: roll diff tweaking into checkpatch itself Alex Bennée
2021-06-23 10:27 ` [PATCH v1 4/8] GitLab: Add "Bug" issue reporting template Alex Bennée
2021-06-23 10:27 ` [PATCH v1 5/8] GitLab: Add "Feature Request" issue template Alex Bennée
2021-06-23 10:27 ` [PATCH v1 6/8] tests/tcg: skip the signals test for hppa/s390x for now Alex Bennée
2021-06-23 10:43   ` Cornelia Huck
2021-06-23 10:27 ` [PATCH v1 7/8] plugins/api: expose symbol lookup to plugins Alex Bennée
2021-06-23 10:27 ` [PATCH v1 8/8] Update documentation to refer to new location for issues Alex Bennée

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.