All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0
@ 2021-12-19 23:26 Richard Purdie
  2021-12-19 23:26 ` [PATCH 2/2] lttng-tools: Upgrade 2.13.1 -> 2.13.2 Richard Purdie
  2021-12-29  3:26 ` [OE-core] [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Khem Raj
  0 siblings, 2 replies; 4+ messages in thread
From: Richard Purdie @ 2021-12-19 23:26 UTC (permalink / raw)
  To: openembedded-core

Add config option for pulseaudio
Change audio to default mode since quoting of list of devices is broken in meson
Drop patch where code changed completed upstream
Refresh other patches

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/conf/distro/include/tcmode-default.inc   |  2 +-
 ...u-native_6.1.0.bb => qemu-native_6.2.0.bb} |  0
 ...e_6.1.0.bb => qemu-system-native_6.2.0.bb} |  0
 meta/recipes-devtools/qemu/qemu.inc           |  6 ++--
 ...nux-user-Replace-__u64-with-uint64_t.patch | 31 -------------------
 ...d-use-relative-path-to-refer-to-file.patch | 13 +++++---
 ...-connect-socket-to-a-spawned-command.patch | 30 +++++++++---------
 .../qemu/{qemu_6.1.0.bb => qemu_6.2.0.bb}     |  0
 8 files changed, 27 insertions(+), 55 deletions(-)
 rename meta/recipes-devtools/qemu/{qemu-native_6.1.0.bb => qemu-native_6.2.0.bb} (100%)
 rename meta/recipes-devtools/qemu/{qemu-system-native_6.1.0.bb => qemu-system-native_6.2.0.bb} (100%)
 delete mode 100644 meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch
 rename meta/recipes-devtools/qemu/{qemu_6.1.0.bb => qemu_6.2.0.bb} (100%)

diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
index d858869bbd5..49e99af8f25 100644
--- a/meta/conf/distro/include/tcmode-default.inc
+++ b/meta/conf/distro/include/tcmode-default.inc
@@ -22,7 +22,7 @@ BINUVERSION ?= "2.37%"
 GDBVERSION ?= "11.%"
 GLIBCVERSION ?= "2.34"
 LINUXLIBCVERSION ?= "5.15%"
-QEMUVERSION ?= "6.1%"
+QEMUVERSION ?= "6.2%"
 GOVERSION ?= "1.17%"
 # This can not use wildcards like 8.0.% since it is also used in mesa to denote
 # llvm version being used, so always bump it with llvm recipe version bump
diff --git a/meta/recipes-devtools/qemu/qemu-native_6.1.0.bb b/meta/recipes-devtools/qemu/qemu-native_6.2.0.bb
similarity index 100%
rename from meta/recipes-devtools/qemu/qemu-native_6.1.0.bb
rename to meta/recipes-devtools/qemu/qemu-native_6.2.0.bb
diff --git a/meta/recipes-devtools/qemu/qemu-system-native_6.1.0.bb b/meta/recipes-devtools/qemu/qemu-system-native_6.2.0.bb
similarity index 100%
rename from meta/recipes-devtools/qemu/qemu-system-native_6.1.0.bb
rename to meta/recipes-devtools/qemu/qemu-system-native_6.2.0.bb
diff --git a/meta/recipes-devtools/qemu/qemu.inc b/meta/recipes-devtools/qemu/qemu.inc
index 584c9482e93..fe838276be1 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -26,12 +26,11 @@ SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
            file://mmap2.patch \
            file://determinism.patch \
            file://0001-tests-meson.build-use-relative-path-to-refer-to-file.patch \
-           file://0001-linux-user-Replace-__u64-with-uint64_t.patch \
            file://0001-riscv-Set-5.4-as-minimum-kernel-version-for-riscv32.patch \
            "
 UPSTREAM_CHECK_REGEX = "qemu-(?P<pver>\d+(\.\d+)+)\.tar"
 
-SRC_URI[sha256sum] = "eebc089db3414bbeedf1e464beda0a7515aad30f73261abc246c9b27503a3c96"
+SRC_URI[sha256sum] = "68e15d8e45ac56326e0b9a4afa8b49a3dfe8aba3488221d098c84698bca65b45"
 
 SRC_URI:append:class-target = " file://cross.patch"
 SRC_URI:append:class-nativesdk = " file://cross.patch"
@@ -173,7 +172,7 @@ PACKAGECONFIG[gcrypt] = "--enable-gcrypt,--disable-gcrypt,libgcrypt,"
 PACKAGECONFIG[nettle] = "--enable-nettle,--disable-nettle,nettle"
 PACKAGECONFIG[libusb] = "--enable-libusb,--disable-libusb,libusb1"
 PACKAGECONFIG[fdt] = "--enable-fdt,--disable-fdt,dtc"
-PACKAGECONFIG[alsa] = "--audio-drv-list='oss alsa',,alsa-lib"
+PACKAGECONFIG[alsa] = "--audio-drv-list=default,,alsa-lib"
 PACKAGECONFIG[glx] = "--enable-opengl,--disable-opengl,virtual/libgl"
 PACKAGECONFIG[lzo] = "--enable-lzo,--disable-lzo,lzo"
 PACKAGECONFIG[numa] = "--enable-numa,--disable-numa,numactl"
@@ -200,6 +199,7 @@ PACKAGECONFIG[seccomp] = "--enable-seccomp,--disable-seccomp,libseccomp"
 # libnfs is currently provided by meta-kodi
 PACKAGECONFIG[libnfs] = "--enable-libnfs,--disable-libnfs,libnfs"
 PACKAGECONFIG[pmem] = "--enable-libpmem,--disable-libpmem,pmdk"
+PACKAGECONFIG[pulsedio] = "--enable-pa,--disable-pa,pulseaudio"
 
 INSANE_SKIP:${PN} = "arch"
 
diff --git a/meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch b/meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch
deleted file mode 100644
index 3d5c890b0fb..00000000000
--- a/meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch
+++ /dev/null
@@ -1,31 +0,0 @@
-From 5cc7b63299602d0aa8b57c684bbd9829856d54ad Mon Sep 17 00:00:00 2001
-From: Khem Raj <raj.khem@gmail.com>
-Date: Mon, 8 Nov 2021 11:39:26 -0800
-Subject: [PATCH] linux-user: Replace __u64 with uint64_t
-
-uint64_t is available in all userspaces via compiler include stdint.h
-therefore use it instead of __u64 which is linux internal type, it fixes
-build on some platforms eg. aarch64 systems using musl C library
-
-Upstream-Status: Submitted [https://lists.gnu.org/archive/html/qemu-devel/2021-11/msg01955.html]
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
----
- linux-user/host/aarch64/hostdep.h | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/linux-user/host/aarch64/hostdep.h b/linux-user/host/aarch64/hostdep.h
-index a8d41a21ad..34d934f665 100644
---- a/linux-user/host/aarch64/hostdep.h
-+++ b/linux-user/host/aarch64/hostdep.h
-@@ -25,7 +25,7 @@ extern char safe_syscall_end[];
- static inline void rewind_if_in_safe_syscall(void *puc)
- {
-     ucontext_t *uc = puc;
--    __u64 *pcreg = &uc->uc_mcontext.pc;
-+    uint64_t *pcreg = &uc->uc_mcontext.pc;
- 
-     if (*pcreg > (uintptr_t)safe_syscall_start
-         && *pcreg < (uintptr_t)safe_syscall_end) {
--- 
-2.33.1
-
diff --git a/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch b/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch
index d5e1ab4d511..fcef1291817 100644
--- a/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch
+++ b/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch
@@ -16,16 +16,19 @@ Signed-off-by: Changqing Li <changqing.li@windriver.com>
  tests/meson.build   | 2 +-
  1 files changed, 1 insertions(+), 1 deletion(-)
 
-Index: qemu-6.0.0/tests/unit/meson.build
+Index: qemu-6.2.0/tests/unit/meson.build
 ===================================================================
---- qemu-6.0.0.orig/tests/unit/meson.build
-+++ qemu-6.0.0/tests/unit/meson.build
-@@ -42,7 +42,7 @@ tests = {
+--- qemu-6.2.0.orig/tests/unit/meson.build
++++ qemu-6.2.0/tests/unit/meson.build
+@@ -44,9 +44,9 @@ tests = {
    'test-keyval': [testqapi],
    'test-logging': [],
    'test-uuid': [],
--  'ptimer-test': ['ptimer-test-stubs.c', meson.source_root() / 'hw/core/ptimer.c'],
+-  'ptimer-test': ['ptimer-test-stubs.c', meson.project_source_root() / 'hw/core/ptimer.c'],
 +  'ptimer-test': ['ptimer-test-stubs.c', '../../hw/core/ptimer.c'],
    'test-qapi-util': [],
+-  'test-smp-parse': [qom, meson.project_source_root() / 'hw/core/machine-smp.c'],
++  'test-smp-parse': [qom, '../../hw/core/machine-smp.c'],
  }
  
+ if have_system or have_tools
diff --git a/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch b/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch
index 05dc849dada..cf8b0e7a452 100644
--- a/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch
+++ b/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch
@@ -51,10 +51,10 @@ Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
  qapi/char.json        |   5 +++
  3 files changed, 109 insertions(+)
 
-Index: qemu-6.0.0/chardev/char-socket.c
+Index: qemu-6.2.0/chardev/char-socket.c
 ===================================================================
---- qemu-6.0.0.orig/chardev/char-socket.c
-+++ qemu-6.0.0/chardev/char-socket.c
+--- qemu-6.2.0.orig/chardev/char-socket.c
++++ qemu-6.2.0/chardev/char-socket.c
 @@ -1362,6 +1362,67 @@ static bool qmp_chardev_validate_socket(
      return true;
  }
@@ -133,7 +133,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
      bool do_nodelay     = sock->has_nodelay ? sock->nodelay : false;
      bool is_listen      = sock->has_server  ? sock->server  : true;
      bool is_telnet      = sock->has_telnet  ? sock->telnet  : false;
-@@ -1446,6 +1510,14 @@ static void qmp_chardev_open_socket(Char
+@@ -1440,6 +1504,14 @@ static void qmp_chardev_open_socket(Char
  
      update_disconnected_filename(s);
  
@@ -148,7 +148,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
      if (s->is_listen) {
          if (qmp_chardev_open_socket_server(chr, is_telnet || is_tn3270,
                                             is_waitconnect, errp) < 0) {
-@@ -1465,6 +1537,9 @@ static void qemu_chr_parse_socket(QemuOp
+@@ -1459,6 +1531,9 @@ static void qemu_chr_parse_socket(QemuOp
      const char *host = qemu_opt_get(opts, "host");
      const char *port = qemu_opt_get(opts, "port");
      const char *fd = qemu_opt_get(opts, "fd");
@@ -158,7 +158,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
  #ifdef CONFIG_LINUX
      bool tight = qemu_opt_get_bool(opts, "tight", true);
      bool abstract = qemu_opt_get_bool(opts, "abstract", false);
-@@ -1472,6 +1547,20 @@ static void qemu_chr_parse_socket(QemuOp
+@@ -1466,6 +1541,20 @@ static void qemu_chr_parse_socket(QemuOp
      SocketAddressLegacy *addr;
      ChardevSocket *sock;
  
@@ -179,7 +179,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
      if ((!!path + !!fd + !!host) != 1) {
          error_setg(errp,
                     "Exactly one of 'path', 'fd' or 'host' required");
-@@ -1522,13 +1611,24 @@ static void qemu_chr_parse_socket(QemuOp
+@@ -1516,13 +1605,24 @@ static void qemu_chr_parse_socket(QemuOp
      sock->tls_creds = g_strdup(qemu_opt_get(opts, "tls-creds"));
      sock->has_tls_authz = qemu_opt_get(opts, "tls-authz");
      sock->tls_authz = g_strdup(qemu_opt_get(opts, "tls-authz"));
@@ -194,7 +194,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
      if (path) {
 +#endif
          UnixSocketAddress *q_unix;
-         addr->type = SOCKET_ADDRESS_LEGACY_KIND_UNIX;
+         addr->type = SOCKET_ADDRESS_TYPE_UNIX;
          q_unix = addr->u.q_unix.data = g_new0(UnixSocketAddress, 1);
 +#ifndef _WIN32
 +        q_unix->path = cmd ? g_strdup_printf("cmd:%s", cmd) : g_strdup(path);
@@ -204,11 +204,11 @@ Index: qemu-6.0.0/chardev/char-socket.c
  #ifdef CONFIG_LINUX
          q_unix->has_tight = true;
          q_unix->tight = tight;
-Index: qemu-6.0.0/chardev/char.c
+Index: qemu-6.2.0/chardev/char.c
 ===================================================================
---- qemu-6.0.0.orig/chardev/char.c
-+++ qemu-6.0.0/chardev/char.c
-@@ -840,6 +840,9 @@ QemuOptsList qemu_chardev_opts = {
+--- qemu-6.2.0.orig/chardev/char.c
++++ qemu-6.2.0/chardev/char.c
+@@ -836,6 +836,9 @@ QemuOptsList qemu_chardev_opts = {
              .name = "path",
              .type = QEMU_OPT_STRING,
          },{
@@ -218,10 +218,10 @@ Index: qemu-6.0.0/chardev/char.c
              .name = "host",
              .type = QEMU_OPT_STRING,
          },{
-Index: qemu-6.0.0/qapi/char.json
+Index: qemu-6.2.0/qapi/char.json
 ===================================================================
---- qemu-6.0.0.orig/qapi/char.json
-+++ qemu-6.0.0/qapi/char.json
+--- qemu-6.2.0.orig/qapi/char.json
++++ qemu-6.2.0/qapi/char.json
 @@ -250,6 +250,10 @@
  #
  # @addr: socket address to listen on (server=true)
diff --git a/meta/recipes-devtools/qemu/qemu_6.1.0.bb b/meta/recipes-devtools/qemu/qemu_6.2.0.bb
similarity index 100%
rename from meta/recipes-devtools/qemu/qemu_6.1.0.bb
rename to meta/recipes-devtools/qemu/qemu_6.2.0.bb
-- 
2.32.0



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

* [PATCH 2/2] lttng-tools: Upgrade 2.13.1 -> 2.13.2
  2021-12-19 23:26 [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Richard Purdie
@ 2021-12-19 23:26 ` Richard Purdie
  2021-12-29  3:26 ` [OE-core] [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Khem Raj
  1 sibling, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2021-12-19 23:26 UTC (permalink / raw)
  To: openembedded-core

Drop backported patches.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 ...1-src-common-correct-header-location.patch |  41 ----
 ...0ba19aec78f36e301494a03f5678fcb6fbb4.patch | 218 ------------------
 ...46a03fbf31c19b85ec367dc2c3db56e6dbf7.patch | 113 ---------
 ...-tools_2.13.1.bb => lttng-tools_2.13.2.bb} |   5 +-
 4 files changed, 1 insertion(+), 376 deletions(-)
 delete mode 100644 meta/recipes-kernel/lttng/lttng-tools/0001-src-common-correct-header-location.patch
 delete mode 100644 meta/recipes-kernel/lttng/lttng-tools/87250ba19aec78f36e301494a03f5678fcb6fbb4.patch
 delete mode 100644 meta/recipes-kernel/lttng/lttng-tools/8f0646a03fbf31c19b85ec367dc2c3db56e6dbf7.patch
 rename meta/recipes-kernel/lttng/{lttng-tools_2.13.1.bb => lttng-tools_2.13.2.bb} (96%)

diff --git a/meta/recipes-kernel/lttng/lttng-tools/0001-src-common-correct-header-location.patch b/meta/recipes-kernel/lttng/lttng-tools/0001-src-common-correct-header-location.patch
deleted file mode 100644
index 24fcce41cb0..00000000000
--- a/meta/recipes-kernel/lttng/lttng-tools/0001-src-common-correct-header-location.patch
+++ /dev/null
@@ -1,41 +0,0 @@
-From 790ef313364d1f1efb6f283705bb324eea2979ba Mon Sep 17 00:00:00 2001
-From: Alexander Kanavin <alex@linutronix.de>
-Date: Sun, 5 Sep 2021 10:13:08 +0200
-Subject: [PATCH] src/common: correct header location
-
-sys/unistd.h is not guaranteed to exist, and does not with
-e.g. musl C library.
-
-Upstream-Status: Backport [http://git.lttng.org/?p=lttng-tools.git;a=commitdiff;h=8f14767d33f786d895e50633fb46b880249b6c54]
-Signed-off-by: Alexander Kanavin <alex@linutronix.de>
----
- src/common/kernel-probe.c    | 2 +-
- src/common/userspace-probe.c | 2 +-
- 2 files changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/src/common/kernel-probe.c b/src/common/kernel-probe.c
-index 2beb53f..44100bc 100644
---- a/src/common/kernel-probe.c
-+++ b/src/common/kernel-probe.c
-@@ -20,7 +20,7 @@
- #include <lttng/kernel-probe.h>
- #include <sys/stat.h>
- #include <sys/types.h>
--#include <sys/unistd.h>
-+#include <unistd.h>
- 
- static
- int lttng_kernel_probe_location_address_serialize(
-diff --git a/src/common/userspace-probe.c b/src/common/userspace-probe.c
-index 67b0ee6..66afc6f 100644
---- a/src/common/userspace-probe.c
-+++ b/src/common/userspace-probe.c
-@@ -20,7 +20,7 @@
- #include <lttng/userspace-probe-internal.h>
- #include <sys/stat.h>
- #include <sys/types.h>
--#include <sys/unistd.h>
-+#include <unistd.h>
- 
- static
- int lttng_userspace_probe_location_function_set_binary_fd_handle(
diff --git a/meta/recipes-kernel/lttng/lttng-tools/87250ba19aec78f36e301494a03f5678fcb6fbb4.patch b/meta/recipes-kernel/lttng/lttng-tools/87250ba19aec78f36e301494a03f5678fcb6fbb4.patch
deleted file mode 100644
index f4db4f86fed..00000000000
--- a/meta/recipes-kernel/lttng/lttng-tools/87250ba19aec78f36e301494a03f5678fcb6fbb4.patch
+++ /dev/null
@@ -1,218 +0,0 @@
-Upstream-Status: Backport
-
-From 87250ba19aec78f36e301494a03f5678fcb6fbb4 Mon Sep 17 00:00:00 2001
-From: =?UTF-8?q?J=C3=A9r=C3=A9mie=20Galarneau?=
- <jeremie.galarneau@efficios.com>
-Date: Mon, 1 Nov 2021 15:43:55 -0400
-Subject: [PATCH] Fix: relayd: live: mishandled initial null trace chunk
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-Observed issue
-==============
-
-As reported in #1323 (https://bugs.lttng.org/issues/1323), crashes of
-the relay daemon are observed when running the user space clear tests.
-
-The crash occurs with the following stack trace:
-  #0  0x000055fbb861d6ae in urcu_ref_get_unless_zero (ref=0x28) at /usr/local/include/urcu/ref.h:85
-  #1  lttng_trace_chunk_get (chunk=0x0) at trace-chunk.c:1836
-  #2  0x000055fbb86051e2 in make_viewer_streams (relay_session=relay_session@entry=0x7f6ea002d540, viewer_session=<optimized out>, seek_t=seek_t@entry=LTTNG_VIEWER_SEEK_BEGINNING, nb_total=nb_total@entry=0x7f6ea9607b00, nb_unsent=nb_unsent@entry=0x7f6ea9607aec, nb_created=nb_created@entry=0x7f6ea9607ae8, closed=<optimized out>) at live.c:405
-  #3  0x000055fbb86061d9 in viewer_get_new_streams (conn=0x7f6e94000fc0) at live.c:1155
-  #4  process_control (conn=0x7f6e94000fc0, recv_hdr=0x7f6ea9607af0) at live.c:2353
-  #5  thread_worker (data=<optimized out>) at live.c:2515
-  #6  0x00007f6eae86a609 in start_thread () from /lib/x86_64-linux-gnu/libpthread.so.0
-  #7  0x00007f6eae78f293 in clone () from /lib/x86_64-linux-gnu/libc.so.6
-
-The race window during which this occurs seems very small as it can take
-hours to reproduce this crash. However, a minimal reproducer could be
-identified, as stated in the bug report.
-
-Essentially, the same crash can be reproduced by attaching a live viewer
-to a session that has seen events being produced, been stopped and been
-cleared.
-
-Cause
-=====
-
-The crash occurs as an attempt is made to take a reference to a viewer
-session’s trace chunk as viewer streams are created. The crux of the
-problem is that the code doesn’t expect a viewer session’s trace chunk
-to be NULL.
-
-The viewer session’s current trace chunk is initially set, when a viewer
-attaches to the viewer session, to a copy the corresponding
-relay_session’s current trace chunk.
-
-A live session always attempts to "catch-up" to the newest available
-trace chunk. This means that when a viewer reaches the end of a trace
-chunk, the viewer session may not transition to the "next" one: it jumps
-to the most recent trace chunk available (the one being produced by the
-relay_session). Hence, if the producer performs multiple rotations
-before a viewer completes the consumption of a trace chunk, it will skip
-over those "intermediary" trace chunks.
-
-A viewer session updates its current trace chunk when:
-  1) new viewer streams are created,
-  2) a new index is requested,
-  3) metadata is requested.
-
-Hence, as a general principle, the viewer session will reference the
-most recent trace chunk available _even if its streams do not point to
-it_. It indicates which trace chunk viewer streams should transition to
-when the end of their current trace chunk is reached.
-
-The live code properly handles transitions to a null chunk. This can be
-verified by attaching a viewer to a live session, stopping the session,
-clearing it (thus entering a null trace chunk), and resuming tracing.
-
-The only issue is that the case where the first trace chunk of a viewer
-session is "null" (no active trace chunk) is mishandled in two places:
-  1) in make_viewer_streams(), where the crash is observed,
-  2) in viewer_get_metadata().
-
-Solution
-========
-
-In make_viewer_streams(), it is assumed that a viewer session will have
-a non-null trace chunk whenever a rotation is not ongoing. This is
-reflected by the fact that a reference is always acquired on the viewer
-session’s trace chunk.
-
-That code is one of the three places that can cause a viewer session’s
-trace chunk to be updated. We still want to update the viewer session to
-the most recently seen trace chunk (null, in this case). However, there
-is no reference to acquire and the trace chunk to use for the creation
-of the viewer stream is NULL. This is properly handled by
-viewer_stream_create().
-
-The second site to change is viewer_get_metadata() which doesn’t handle
-a viewer metadata stream not having an active trace chunk at all.
-Thankfully, the protocol allows us to express this condition by
-returning the LTTNG_VIEWER_NO_NEW_METADATA status code when a viewer
-metadata stream doesn’t have an open file and doesn’t have a current
-trace chunk.
-
-Surprisingly, this bug didn’t trigger in the case where a transition to
-a null chunk occurred _after_ attaching to a viewer session.
-
-This is because viewers will typically ask for metadata as a result of an
-LTTNG_VIEWER_FLAG_NEW_METADATA reply to the GET_NEXT_INDEX command. When
-a session is stopped and all data was consumed, this command returns
-that no new data is available, causing the viewers to wait and ask again
-later.
-
-However, when attaching, babeltrace2 (at least, and probably babeltrace 1.x)
-always asks for an initial segment of metadata before asking for an
-index.
-
-Known drawbacks
-===============
-
-None.
-
-Fixes: #1323
-
-Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
-Change-Id: I516fca60755e6897f6b7170c12d706ef57ad61a5
----
- src/bin/lttng-relayd/live.c   | 47 ++++++++++++++++++++++++-----------
- src/bin/lttng-relayd/stream.h |  5 ++++
- 2 files changed, 38 insertions(+), 14 deletions(-)
-
-Index: lttng-tools-2.13.1/src/bin/lttng-relayd/live.c
-===================================================================
---- lttng-tools-2.13.1.orig/src/bin/lttng-relayd/live.c
-+++ lttng-tools-2.13.1/src/bin/lttng-relayd/live.c
-@@ -384,8 +384,6 @@ static int make_viewer_streams(struct re
- 						goto error_unlock;
- 					}
- 				} else {
--					bool reference_acquired;
--
- 					/*
- 					 * Transition the viewer session into the newest trace chunk available.
- 					 */
-@@ -402,11 +400,26 @@ static int make_viewer_streams(struct re
- 						}
- 					}
- 
--					reference_acquired = lttng_trace_chunk_get(
--							viewer_session->current_trace_chunk);
--					assert(reference_acquired);
--					viewer_stream_trace_chunk =
--							viewer_session->current_trace_chunk;
-+					if (relay_stream->trace_chunk) {
-+						/*
-+						 * If the corresponding relay
-+						 * stream's trace chunk is set,
-+						 * the viewer stream will be
-+						 * created under it.
-+						 *
-+						 * Note that a relay stream can
-+						 * have a NULL output trace
-+						 * chunk (for instance, after a
-+						 * clear against a stopped
-+						 * session).
-+						 */
-+						const bool reference_acquired = lttng_trace_chunk_get(
-+								viewer_session->current_trace_chunk);
-+
-+						assert(reference_acquired);
-+						viewer_stream_trace_chunk =
-+								viewer_session->current_trace_chunk;
-+					}
- 				}
- 
- 				viewer_stream = viewer_stream_create(
-@@ -2016,8 +2029,9 @@ int viewer_get_metadata(struct relay_con
- 		}
- 	}
- 
--	if (conn->viewer_session->current_trace_chunk !=
--			vstream->stream_file.trace_chunk) {
-+	if (conn->viewer_session->current_trace_chunk &&
-+			conn->viewer_session->current_trace_chunk !=
-+					vstream->stream_file.trace_chunk) {
- 		bool acquired_reference;
- 
- 		DBG("Viewer session and viewer stream chunk differ: "
-@@ -2034,11 +2048,16 @@ int viewer_get_metadata(struct relay_con
- 
- 	len = vstream->stream->metadata_received - vstream->metadata_sent;
- 
--	/*
--	 * Either this is the first time the metadata file is read, or a
--	 * rotation of the corresponding relay stream has occurred.
--	 */
--	if (!vstream->stream_file.handle && len > 0) {
-+	if (!vstream->stream_file.trace_chunk) {
-+		reply.status = htobe32(LTTNG_VIEWER_NO_NEW_METADATA);
-+		len = 0;
-+		goto send_reply;
-+	} else if (vstream->stream_file.trace_chunk &&
-+			!vstream->stream_file.handle && len > 0) {
-+		/*
-+		 * Either this is the first time the metadata file is read, or a
-+		 * rotation of the corresponding relay stream has occurred.
-+		 */
- 		struct fs_handle *fs_handle;
- 		char file_path[LTTNG_PATH_MAX];
- 		enum lttng_trace_chunk_status status;
-Index: lttng-tools-2.13.1/src/bin/lttng-relayd/stream.h
-===================================================================
---- lttng-tools-2.13.1.orig/src/bin/lttng-relayd/stream.h
-+++ lttng-tools-2.13.1/src/bin/lttng-relayd/stream.h
-@@ -174,6 +174,11 @@ struct relay_stream {
- 	/*
- 	 * The trace chunk to which the file currently being produced (if any)
- 	 * belongs.
-+	 *
-+	 * Note that a relay stream can have no output trace chunk. For
-+	 * instance, after a session stop followed by a session clear,
-+	 * streams will not have an output trace chunk until the session
-+	 * is resumed.
- 	 */
- 	struct lttng_trace_chunk *trace_chunk;
- 	LTTNG_OPTIONAL(struct relay_stream_rotation) ongoing_rotation;
diff --git a/meta/recipes-kernel/lttng/lttng-tools/8f0646a03fbf31c19b85ec367dc2c3db56e6dbf7.patch b/meta/recipes-kernel/lttng/lttng-tools/8f0646a03fbf31c19b85ec367dc2c3db56e6dbf7.patch
deleted file mode 100644
index db2fca03fe1..00000000000
--- a/meta/recipes-kernel/lttng/lttng-tools/8f0646a03fbf31c19b85ec367dc2c3db56e6dbf7.patch
+++ /dev/null
@@ -1,113 +0,0 @@
-Upstream-Status: Backport
-
-From 8f0646a03fbf31c19b85ec367dc2c3db56e6dbf7 Mon Sep 17 00:00:00 2001
-From: Francis Deslauriers <francis.deslauriers@efficios.com>
-Date: Mon, 25 Oct 2021 11:32:24 -0400
-Subject: [PATCH] Typo: occurences -> occurrences
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
-Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
-Change-Id: I719e26febd639f3b047b6aa6361fc6734088e871
----
- configure.ac                                             | 2 +-
- src/bin/lttng-relayd/live.c                              | 2 +-
- src/bin/lttng-sessiond/event-notifier-error-accounting.c | 2 +-
- src/bin/lttng-sessiond/ust-app.c                         | 2 +-
- tests/utils/utils.sh                                     | 8 ++++----
- 5 files changed, 8 insertions(+), 8 deletions(-)
-
-diff --git a/configure.ac b/configure.ac
-index 12cc7a17e..27148c105 100644
---- a/configure.ac
-+++ b/configure.ac
-@@ -253,7 +253,7 @@ AS_IF([test "x$libtool_fixup" = "xyes"],
- 	    [
- 	    libtool_m4="$srcdir/m4/libtool.m4"
- 	    libtool_flag_pattern=".*link_all_deplibs\s*,\s*\$1\s*)"
--	    AC_MSG_CHECKING([for occurence(s) of link_all_deplibs = no in $libtool_m4])
-+	    AC_MSG_CHECKING([for occurrence(s) of link_all_deplibs = no in $libtool_m4])
- 	    libtool_flag_pattern_count=$($GREP -c "$libtool_flag_pattern\s*=\s*no" $libtool_m4)
- 	    AS_IF([test $libtool_flag_pattern_count -ne 0],
- 	    [
-diff --git a/src/bin/lttng-relayd/live.c b/src/bin/lttng-relayd/live.c
-index 13078026b..42b0d947e 100644
---- a/src/bin/lttng-relayd/live.c
-+++ b/src/bin/lttng-relayd/live.c
-@@ -2036,7 +2036,7 @@ int viewer_get_metadata(struct relay_connection *conn)
- 
- 	/*
- 	 * Either this is the first time the metadata file is read, or a
--	 * rotation of the corresponding relay stream has occured.
-+	 * rotation of the corresponding relay stream has occurred.
- 	 */
- 	if (!vstream->stream_file.handle && len > 0) {
- 		struct fs_handle *fs_handle;
-diff --git a/src/bin/lttng-sessiond/event-notifier-error-accounting.c b/src/bin/lttng-sessiond/event-notifier-error-accounting.c
-index d3e3692f5..1488d801c 100644
---- a/src/bin/lttng-sessiond/event-notifier-error-accounting.c
-+++ b/src/bin/lttng-sessiond/event-notifier-error-accounting.c
-@@ -488,7 +488,7 @@ struct ust_error_accounting_entry *ust_error_accounting_entry_create(
- 	lttng_ust_ctl_destroy_counter(daemon_counter);
- error_create_daemon_counter:
- error_shm_alloc:
--	/* Error occured before per-cpu SHMs were handed-off to ustctl. */
-+	/* Error occurred before per-cpu SHMs were handed-off to ustctl. */
- 	if (cpu_counter_fds) {
- 		for (i = 0; i < entry->nr_counter_cpu_fds; i++) {
- 			if (cpu_counter_fds[i] < 0) {
-diff --git a/src/bin/lttng-sessiond/ust-app.c b/src/bin/lttng-sessiond/ust-app.c
-index b18988560..28c63e70c 100644
---- a/src/bin/lttng-sessiond/ust-app.c
-+++ b/src/bin/lttng-sessiond/ust-app.c
-@@ -1342,7 +1342,7 @@ static struct ust_app_event_notifier_rule *alloc_ust_app_event_notifier_rule(
- 	case LTTNG_EVENT_RULE_GENERATE_EXCLUSIONS_STATUS_NONE:
- 		break;
- 	default:
--		/* Error occured. */
-+		/* Error occurred. */
- 		ERR("Failed to generate exclusions from trigger while allocating an event notifier rule");
- 		goto error_put_trigger;
- 	}
-diff --git a/tests/utils/utils.sh b/tests/utils/utils.sh
-index e463e4fe3..42d99444f 100644
---- a/tests/utils/utils.sh
-+++ b/tests/utils/utils.sh
-@@ -1921,7 +1921,7 @@ function validate_trace
- 			pass "Validate trace for event $i, $traced events"
- 		else
- 			fail "Validate trace for event $i"
--			diag "Found $traced occurences of $i"
-+			diag "Found $traced occurrences of $i"
- 		fi
- 	done
- 	ret=$?
-@@ -1949,7 +1949,7 @@ function validate_trace_count
- 			pass "Validate trace for event $i, $traced events"
- 		else
- 			fail "Validate trace for event $i"
--			diag "Found $traced occurences of $i"
-+			diag "Found $traced occurrences of $i"
- 		fi
- 		cnt=$(($cnt + $traced))
- 	done
-@@ -1979,7 +1979,7 @@ function validate_trace_count_range_incl_min_excl_max
- 			pass "Validate trace for event $i, $traced events"
- 		else
- 			fail "Validate trace for event $i"
--			diag "Found $traced occurences of $i"
-+			diag "Found $traced occurrences of $i"
- 		fi
- 		cnt=$(($cnt + $traced))
- 	done
-@@ -2013,7 +2013,7 @@ function validate_trace_exp()
- 		pass "Validate trace for expression '${event_exp}', $traced events"
- 	else
- 		fail "Validate trace for expression '${event_exp}'"
--		diag "Found $traced occurences of '${event_exp}'"
-+		diag "Found $traced occurrences of '${event_exp}'"
- 	fi
- 	ret=$?
- 	return $ret
diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.13.1.bb b/meta/recipes-kernel/lttng/lttng-tools_2.13.2.bb
similarity index 96%
rename from meta/recipes-kernel/lttng/lttng-tools_2.13.1.bb
rename to meta/recipes-kernel/lttng/lttng-tools_2.13.2.bb
index 7ae02d92d2d..d6ce2c7e32e 100644
--- a/meta/recipes-kernel/lttng/lttng-tools_2.13.1.bb
+++ b/meta/recipes-kernel/lttng/lttng-tools_2.13.2.bb
@@ -36,13 +36,10 @@ SRC_URI = "https://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
            file://run-ptest \
            file://lttng-sessiond.service \
            file://determinism.patch \
-           file://0001-src-common-correct-header-location.patch \
-           file://8f0646a03fbf31c19b85ec367dc2c3db56e6dbf7.patch \
-           file://87250ba19aec78f36e301494a03f5678fcb6fbb4.patch \
            file://disable-tests.patch \
            "
 
-SRC_URI[sha256sum] = "cfe6df7da831fc07fd07ce46b442c2ec1074c167af73f3a1b1d2fba0c453c8b5"
+SRC_URI[sha256sum] = "ced2ba00020da77fd17ac92b5a53825502409a07e3aeca28f9515d4a844fd7bd"
 
 inherit autotools ptest pkgconfig useradd python3-dir manpages systemd
 
-- 
2.32.0


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

* Re: [OE-core] [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0
  2021-12-19 23:26 [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Richard Purdie
  2021-12-19 23:26 ` [PATCH 2/2] lttng-tools: Upgrade 2.13.1 -> 2.13.2 Richard Purdie
@ 2021-12-29  3:26 ` Khem Raj
  2021-12-30 17:35   ` Richard Purdie
  1 sibling, 1 reply; 4+ messages in thread
From: Khem Raj @ 2021-12-29  3:26 UTC (permalink / raw)
  To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer

I am seeing that it now fails to build for riscv64 target where it
fails to configure

ERROR: Unknown CPU family riscv, see
https://wiki.yoctoproject.org/wiki/Meson/UnknownCPU for directions.

ERROR: meson setup failed

On Sun, Dec 19, 2021 at 3:26 PM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> Add config option for pulseaudio
> Change audio to default mode since quoting of list of devices is broken in meson
> Drop patch where code changed completed upstream
> Refresh other patches
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  meta/conf/distro/include/tcmode-default.inc   |  2 +-
>  ...u-native_6.1.0.bb => qemu-native_6.2.0.bb} |  0
>  ...e_6.1.0.bb => qemu-system-native_6.2.0.bb} |  0
>  meta/recipes-devtools/qemu/qemu.inc           |  6 ++--
>  ...nux-user-Replace-__u64-with-uint64_t.patch | 31 -------------------
>  ...d-use-relative-path-to-refer-to-file.patch | 13 +++++---
>  ...-connect-socket-to-a-spawned-command.patch | 30 +++++++++---------
>  .../qemu/{qemu_6.1.0.bb => qemu_6.2.0.bb}     |  0
>  8 files changed, 27 insertions(+), 55 deletions(-)
>  rename meta/recipes-devtools/qemu/{qemu-native_6.1.0.bb => qemu-native_6.2.0.bb} (100%)
>  rename meta/recipes-devtools/qemu/{qemu-system-native_6.1.0.bb => qemu-system-native_6.2.0.bb} (100%)
>  delete mode 100644 meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch
>  rename meta/recipes-devtools/qemu/{qemu_6.1.0.bb => qemu_6.2.0.bb} (100%)
>
> diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
> index d858869bbd5..49e99af8f25 100644
> --- a/meta/conf/distro/include/tcmode-default.inc
> +++ b/meta/conf/distro/include/tcmode-default.inc
> @@ -22,7 +22,7 @@ BINUVERSION ?= "2.37%"
>  GDBVERSION ?= "11.%"
>  GLIBCVERSION ?= "2.34"
>  LINUXLIBCVERSION ?= "5.15%"
> -QEMUVERSION ?= "6.1%"
> +QEMUVERSION ?= "6.2%"
>  GOVERSION ?= "1.17%"
>  # This can not use wildcards like 8.0.% since it is also used in mesa to denote
>  # llvm version being used, so always bump it with llvm recipe version bump
> diff --git a/meta/recipes-devtools/qemu/qemu-native_6.1.0.bb b/meta/recipes-devtools/qemu/qemu-native_6.2.0.bb
> similarity index 100%
> rename from meta/recipes-devtools/qemu/qemu-native_6.1.0.bb
> rename to meta/recipes-devtools/qemu/qemu-native_6.2.0.bb
> diff --git a/meta/recipes-devtools/qemu/qemu-system-native_6.1.0.bb b/meta/recipes-devtools/qemu/qemu-system-native_6.2.0.bb
> similarity index 100%
> rename from meta/recipes-devtools/qemu/qemu-system-native_6.1.0.bb
> rename to meta/recipes-devtools/qemu/qemu-system-native_6.2.0.bb
> diff --git a/meta/recipes-devtools/qemu/qemu.inc b/meta/recipes-devtools/qemu/qemu.inc
> index 584c9482e93..fe838276be1 100644
> --- a/meta/recipes-devtools/qemu/qemu.inc
> +++ b/meta/recipes-devtools/qemu/qemu.inc
> @@ -26,12 +26,11 @@ SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
>             file://mmap2.patch \
>             file://determinism.patch \
>             file://0001-tests-meson.build-use-relative-path-to-refer-to-file.patch \
> -           file://0001-linux-user-Replace-__u64-with-uint64_t.patch \
>             file://0001-riscv-Set-5.4-as-minimum-kernel-version-for-riscv32.patch \
>             "
>  UPSTREAM_CHECK_REGEX = "qemu-(?P<pver>\d+(\.\d+)+)\.tar"
>
> -SRC_URI[sha256sum] = "eebc089db3414bbeedf1e464beda0a7515aad30f73261abc246c9b27503a3c96"
> +SRC_URI[sha256sum] = "68e15d8e45ac56326e0b9a4afa8b49a3dfe8aba3488221d098c84698bca65b45"
>
>  SRC_URI:append:class-target = " file://cross.patch"
>  SRC_URI:append:class-nativesdk = " file://cross.patch"
> @@ -173,7 +172,7 @@ PACKAGECONFIG[gcrypt] = "--enable-gcrypt,--disable-gcrypt,libgcrypt,"
>  PACKAGECONFIG[nettle] = "--enable-nettle,--disable-nettle,nettle"
>  PACKAGECONFIG[libusb] = "--enable-libusb,--disable-libusb,libusb1"
>  PACKAGECONFIG[fdt] = "--enable-fdt,--disable-fdt,dtc"
> -PACKAGECONFIG[alsa] = "--audio-drv-list='oss alsa',,alsa-lib"
> +PACKAGECONFIG[alsa] = "--audio-drv-list=default,,alsa-lib"
>  PACKAGECONFIG[glx] = "--enable-opengl,--disable-opengl,virtual/libgl"
>  PACKAGECONFIG[lzo] = "--enable-lzo,--disable-lzo,lzo"
>  PACKAGECONFIG[numa] = "--enable-numa,--disable-numa,numactl"
> @@ -200,6 +199,7 @@ PACKAGECONFIG[seccomp] = "--enable-seccomp,--disable-seccomp,libseccomp"
>  # libnfs is currently provided by meta-kodi
>  PACKAGECONFIG[libnfs] = "--enable-libnfs,--disable-libnfs,libnfs"
>  PACKAGECONFIG[pmem] = "--enable-libpmem,--disable-libpmem,pmdk"
> +PACKAGECONFIG[pulsedio] = "--enable-pa,--disable-pa,pulseaudio"
>
>  INSANE_SKIP:${PN} = "arch"
>
> diff --git a/meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch b/meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch
> deleted file mode 100644
> index 3d5c890b0fb..00000000000
> --- a/meta/recipes-devtools/qemu/qemu/0001-linux-user-Replace-__u64-with-uint64_t.patch
> +++ /dev/null
> @@ -1,31 +0,0 @@
> -From 5cc7b63299602d0aa8b57c684bbd9829856d54ad Mon Sep 17 00:00:00 2001
> -From: Khem Raj <raj.khem@gmail.com>
> -Date: Mon, 8 Nov 2021 11:39:26 -0800
> -Subject: [PATCH] linux-user: Replace __u64 with uint64_t
> -
> -uint64_t is available in all userspaces via compiler include stdint.h
> -therefore use it instead of __u64 which is linux internal type, it fixes
> -build on some platforms eg. aarch64 systems using musl C library
> -
> -Upstream-Status: Submitted [https://lists.gnu.org/archive/html/qemu-devel/2021-11/msg01955.html]
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ----
> - linux-user/host/aarch64/hostdep.h | 2 +-
> - 1 file changed, 1 insertion(+), 1 deletion(-)
> -
> -diff --git a/linux-user/host/aarch64/hostdep.h b/linux-user/host/aarch64/hostdep.h
> -index a8d41a21ad..34d934f665 100644
> ---- a/linux-user/host/aarch64/hostdep.h
> -+++ b/linux-user/host/aarch64/hostdep.h
> -@@ -25,7 +25,7 @@ extern char safe_syscall_end[];
> - static inline void rewind_if_in_safe_syscall(void *puc)
> - {
> -     ucontext_t *uc = puc;
> --    __u64 *pcreg = &uc->uc_mcontext.pc;
> -+    uint64_t *pcreg = &uc->uc_mcontext.pc;
> -
> -     if (*pcreg > (uintptr_t)safe_syscall_start
> -         && *pcreg < (uintptr_t)safe_syscall_end) {
> ---
> -2.33.1
> -
> diff --git a/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch b/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch
> index d5e1ab4d511..fcef1291817 100644
> --- a/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch
> +++ b/meta/recipes-devtools/qemu/qemu/0001-tests-meson.build-use-relative-path-to-refer-to-file.patch
> @@ -16,16 +16,19 @@ Signed-off-by: Changqing Li <changqing.li@windriver.com>
>   tests/meson.build   | 2 +-
>   1 files changed, 1 insertions(+), 1 deletion(-)
>
> -Index: qemu-6.0.0/tests/unit/meson.build
> +Index: qemu-6.2.0/tests/unit/meson.build
>  ===================================================================
> ---- qemu-6.0.0.orig/tests/unit/meson.build
> -+++ qemu-6.0.0/tests/unit/meson.build
> -@@ -42,7 +42,7 @@ tests = {
> +--- qemu-6.2.0.orig/tests/unit/meson.build
> ++++ qemu-6.2.0/tests/unit/meson.build
> +@@ -44,9 +44,9 @@ tests = {
>     'test-keyval': [testqapi],
>     'test-logging': [],
>     'test-uuid': [],
> --  'ptimer-test': ['ptimer-test-stubs.c', meson.source_root() / 'hw/core/ptimer.c'],
> +-  'ptimer-test': ['ptimer-test-stubs.c', meson.project_source_root() / 'hw/core/ptimer.c'],
>  +  'ptimer-test': ['ptimer-test-stubs.c', '../../hw/core/ptimer.c'],
>     'test-qapi-util': [],
> +-  'test-smp-parse': [qom, meson.project_source_root() / 'hw/core/machine-smp.c'],
> ++  'test-smp-parse': [qom, '../../hw/core/machine-smp.c'],
>   }
>
> + if have_system or have_tools
> diff --git a/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch b/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch
> index 05dc849dada..cf8b0e7a452 100644
> --- a/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch
> +++ b/meta/recipes-devtools/qemu/qemu/0006-chardev-connect-socket-to-a-spawned-command.patch
> @@ -51,10 +51,10 @@ Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
>   qapi/char.json        |   5 +++
>   3 files changed, 109 insertions(+)
>
> -Index: qemu-6.0.0/chardev/char-socket.c
> +Index: qemu-6.2.0/chardev/char-socket.c
>  ===================================================================
> ---- qemu-6.0.0.orig/chardev/char-socket.c
> -+++ qemu-6.0.0/chardev/char-socket.c
> +--- qemu-6.2.0.orig/chardev/char-socket.c
> ++++ qemu-6.2.0/chardev/char-socket.c
>  @@ -1362,6 +1362,67 @@ static bool qmp_chardev_validate_socket(
>       return true;
>   }
> @@ -133,7 +133,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
>       bool do_nodelay     = sock->has_nodelay ? sock->nodelay : false;
>       bool is_listen      = sock->has_server  ? sock->server  : true;
>       bool is_telnet      = sock->has_telnet  ? sock->telnet  : false;
> -@@ -1446,6 +1510,14 @@ static void qmp_chardev_open_socket(Char
> +@@ -1440,6 +1504,14 @@ static void qmp_chardev_open_socket(Char
>
>       update_disconnected_filename(s);
>
> @@ -148,7 +148,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
>       if (s->is_listen) {
>           if (qmp_chardev_open_socket_server(chr, is_telnet || is_tn3270,
>                                              is_waitconnect, errp) < 0) {
> -@@ -1465,6 +1537,9 @@ static void qemu_chr_parse_socket(QemuOp
> +@@ -1459,6 +1531,9 @@ static void qemu_chr_parse_socket(QemuOp
>       const char *host = qemu_opt_get(opts, "host");
>       const char *port = qemu_opt_get(opts, "port");
>       const char *fd = qemu_opt_get(opts, "fd");
> @@ -158,7 +158,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
>   #ifdef CONFIG_LINUX
>       bool tight = qemu_opt_get_bool(opts, "tight", true);
>       bool abstract = qemu_opt_get_bool(opts, "abstract", false);
> -@@ -1472,6 +1547,20 @@ static void qemu_chr_parse_socket(QemuOp
> +@@ -1466,6 +1541,20 @@ static void qemu_chr_parse_socket(QemuOp
>       SocketAddressLegacy *addr;
>       ChardevSocket *sock;
>
> @@ -179,7 +179,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
>       if ((!!path + !!fd + !!host) != 1) {
>           error_setg(errp,
>                      "Exactly one of 'path', 'fd' or 'host' required");
> -@@ -1522,13 +1611,24 @@ static void qemu_chr_parse_socket(QemuOp
> +@@ -1516,13 +1605,24 @@ static void qemu_chr_parse_socket(QemuOp
>       sock->tls_creds = g_strdup(qemu_opt_get(opts, "tls-creds"));
>       sock->has_tls_authz = qemu_opt_get(opts, "tls-authz");
>       sock->tls_authz = g_strdup(qemu_opt_get(opts, "tls-authz"));
> @@ -194,7 +194,7 @@ Index: qemu-6.0.0/chardev/char-socket.c
>       if (path) {
>  +#endif
>           UnixSocketAddress *q_unix;
> -         addr->type = SOCKET_ADDRESS_LEGACY_KIND_UNIX;
> +         addr->type = SOCKET_ADDRESS_TYPE_UNIX;
>           q_unix = addr->u.q_unix.data = g_new0(UnixSocketAddress, 1);
>  +#ifndef _WIN32
>  +        q_unix->path = cmd ? g_strdup_printf("cmd:%s", cmd) : g_strdup(path);
> @@ -204,11 +204,11 @@ Index: qemu-6.0.0/chardev/char-socket.c
>   #ifdef CONFIG_LINUX
>           q_unix->has_tight = true;
>           q_unix->tight = tight;
> -Index: qemu-6.0.0/chardev/char.c
> +Index: qemu-6.2.0/chardev/char.c
>  ===================================================================
> ---- qemu-6.0.0.orig/chardev/char.c
> -+++ qemu-6.0.0/chardev/char.c
> -@@ -840,6 +840,9 @@ QemuOptsList qemu_chardev_opts = {
> +--- qemu-6.2.0.orig/chardev/char.c
> ++++ qemu-6.2.0/chardev/char.c
> +@@ -836,6 +836,9 @@ QemuOptsList qemu_chardev_opts = {
>               .name = "path",
>               .type = QEMU_OPT_STRING,
>           },{
> @@ -218,10 +218,10 @@ Index: qemu-6.0.0/chardev/char.c
>               .name = "host",
>               .type = QEMU_OPT_STRING,
>           },{
> -Index: qemu-6.0.0/qapi/char.json
> +Index: qemu-6.2.0/qapi/char.json
>  ===================================================================
> ---- qemu-6.0.0.orig/qapi/char.json
> -+++ qemu-6.0.0/qapi/char.json
> +--- qemu-6.2.0.orig/qapi/char.json
> ++++ qemu-6.2.0/qapi/char.json
>  @@ -250,6 +250,10 @@
>   #
>   # @addr: socket address to listen on (server=true)
> diff --git a/meta/recipes-devtools/qemu/qemu_6.1.0.bb b/meta/recipes-devtools/qemu/qemu_6.2.0.bb
> similarity index 100%
> rename from meta/recipes-devtools/qemu/qemu_6.1.0.bb
> rename to meta/recipes-devtools/qemu/qemu_6.2.0.bb
> --
> 2.32.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#159850): https://lists.openembedded.org/g/openembedded-core/message/159850
> Mute This Topic: https://lists.openembedded.org/mt/87846474/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


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

* Re: [OE-core] [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0
  2021-12-29  3:26 ` [OE-core] [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Khem Raj
@ 2021-12-30 17:35   ` Richard Purdie
  0 siblings, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2021-12-30 17:35 UTC (permalink / raw)
  To: Khem Raj; +Cc: Patches and discussions about the oe-core layer

On Tue, 2021-12-28 at 19:26 -0800, Khem Raj wrote:
> I am seeing that it now fails to build for riscv64 target where it
> fails to configure
> 
> ERROR: Unknown CPU family riscv, see
> https://wiki.yoctoproject.org/wiki/Meson/UnknownCPU for directions.
> 
> ERROR: meson setup failed

Looks like we need something like:

https://git.qemu.org/?p=qemu.git;a=commitdiff;h=c94c239496256f1f1cb589825d052c2f3e26ebf6

Cheers,

Richard



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

end of thread, other threads:[~2021-12-30 17:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-19 23:26 [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Richard Purdie
2021-12-19 23:26 ` [PATCH 2/2] lttng-tools: Upgrade 2.13.1 -> 2.13.2 Richard Purdie
2021-12-29  3:26 ` [OE-core] [PATCH 1/2] qemu: Upgrade 6.1.0 -> 6.2.0 Khem Raj
2021-12-30 17:35   ` Richard Purdie

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.