Re: [OE-core] [PATCH 34/42] lttng-modules: update to 2.12.1

2020-06-09 Thread Alexander Kanavin
On Tue, 9 Jun 2020 at 18:24, Peter Kjellerstedt 
wrote:

> > devupstream variant was incorrectly updated in a059fc67da4
> > (changed to 2.12, but without updating SRCREV), so revert it back
> > to 2.11.2.
>
> I don't understand this. If you update the recipe to 2.12.1, why do
> you not update the devupstream variant to match?
>

Because I have no interest in that variant, other than making sure it is
not misleading; it was added and is (supposedly) maintained by others who
use it to test the bleeding edge.

Generally those 'development' variants tend to bit rot quickly due to lack
of regular updates and testing, oe-core used to have plenty of them and
they were all dropped, except this one.

Alex
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#139378): 
https://lists.openembedded.org/g/openembedded-core/message/139378
Mute This Topic: https://lists.openembedded.org/mt/74747463/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 34/42] lttng-modules: update to 2.12.1

2020-06-09 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core@lists.openembedded.org  c...@lists.openembedded.org> On Behalf Of Alexander Kanavin
> Sent: den 8 juni 2020 09:47
> To: openembedded-core@lists.openembedded.org
> Cc: Alexander Kanavin 
> Subject: [OE-core] [PATCH 34/42] lttng-modules: update to 2.12.1
> 
> Drop backports.
> 
> devupstream variant was incorrectly updated in a059fc67da4
> (changed to 2.12, but without updating SRCREV), so revert it back
> to 2.11.2.

I don't understand this. If you update the recipe to 2.12.1, why do 
you not update the devupstream variant to match?

> Signed-off-by: Alexander Kanavin 
> ---
>  ...l-5.7-use-vmalloc_sync_mappings-on-k.patch | 853 --
>  ...per-rename-to-wrapper_vmalloc_sync_m.patch |  32 -
>  ..._sync_mappings-on-kernel-5.6-as-well.patch |  88 --
>  ...loc_sync_mappings-for-stable-kernels.patch |  67 --
>  ...ules_2.12.0.bb => lttng-modules_2.12.1.bb} |  11 +-
>  5 files changed, 3 insertions(+), 1048 deletions(-)
>  delete mode 100644 meta/recipes-kernel/lttng/lttng-modules/0001-Update-
> for-kernel-5.7-use-vmalloc_sync_mappings-on-k.patch
>  delete mode 100644 meta/recipes-kernel/lttng/lttng-modules/0002-Fix-
> missing-wrapper-rename-to-wrapper_vmalloc_sync_m.patch
>  delete mode 100644 meta/recipes-kernel/lttng/lttng-modules/0003-Fix-Use-
> vmalloc_sync_mappings-on-kernel-5.6-as-well.patch
>  delete mode 100644 meta/recipes-kernel/lttng/lttng-modules/0004-Update-
> Use-vmalloc_sync_mappings-for-stable-kernels.patch
>  rename meta/recipes-kernel/lttng/{lttng-modules_2.12.0.bb => lttng-
> modules_2.12.1.bb} (75%)

[snip]

> diff --git a/meta/recipes-kernel/lttng/lttng-modules_2.12.0.bb 
> b/meta/recipes-kernel/lttng/lttng-modules_2.12.1.bb
> similarity index 75%
> rename from meta/recipes-kernel/lttng/lttng-modules_2.12.0.bb
> rename to meta/recipes-kernel/lttng/lttng-modules_2.12.1.bb
> index 4a6c42e3bf..8d3fb475c0 100644
> --- a/meta/recipes-kernel/lttng/lttng-modules_2.12.0.bb
> +++ b/meta/recipes-kernel/lttng/lttng-modules_2.12.1.bb
> @@ -11,14 +11,9 @@ COMPATIBLE_HOST = 
> '(x86_64|i.86|powerpc|aarch64|mips|nios2|arm|riscv).*-linux'
>  SRC_URI = "https://lttng.org/files/${BPN}/${BPN}-${PV}.tar.bz2 \
> file://Makefile-Do-not-fail-if-CONFIG_TRACEPOINTS-is-not-en.patch 
> \
> file://BUILD_RUNTIME_BUG_ON-vs-gcc7.patch \
> -   
> file://0001-Update-for-kernel-5.7-use-vmalloc_sync_mappings-on-k.patch \
> -   
> file://0002-Fix-missing-wrapper-rename-to-wrapper_vmalloc_sync_m.patch \
> -   
> file://0003-Fix-Use-vmalloc_sync_mappings-on-kernel-5.6-as-well.patch \
> -   
> file://0004-Update-Use-vmalloc_sync_mappings-for-stable-kernels.patch \
> "
> 
> -SRC_URI[md5sum] = "be252df5013ea03894921d0b493c7b6c"
> -SRC_URI[sha256sum] = 
> "efeb2ae60e96dbe9a9b342d3a80dfc7642f06a8c09686089d491ad696bb98f4d"
> +SRC_URI[sha256sum] = 
> "639c03bf8f0e920567f45c96cc7ef749a8fc29ff7fb3fb97263d67f66fb71aef"
> 
>  export INSTALL_MOD_DIR="kernel/lttng-modules"
> 
> @@ -37,11 +32,11 @@ python do_package_prepend() {
>  BBCLASSEXTEND = "devupstream:target"
>  LIC_FILES_CHKSUM_class-devupstream = 
> "file://LICENSE;md5=3f882d431dc0f32f1f44c0707aa41128"
>  DEFAULT_PREFERENCE_class-devupstream = "-1"
> -SRC_URI_class-devupstream = 
> "git://git.lttng.org/lttng-modules;branch=stable-2.12 \
> +SRC_URI_class-devupstream = 
> "git://git.lttng.org/lttng-modules;branch=stable-2.11 \
> file://Makefile-Do-not-fail-if-CONFIG_TRACEPOINTS-is-not-en.patch 
> \
> file://BUILD_RUNTIME_BUG_ON-vs-gcc7.patch \
> "
>  SRCREV_class-devupstream = "17c413953603f063f2a9d6c3788bec914ce6f955"
> -PV_class-devupstream = "2.12.0+git${SRCPV}"
> +PV_class-devupstream = "2.11.2+git${SRCPV}"
>  S_class-devupstream = "${WORKDIR}/git"
>  SRCREV_FORMAT ?= "lttng_git"
> --
> 2.26.2

//Peter

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#139376): 
https://lists.openembedded.org/g/openembedded-core/message/139376
Mute This Topic: https://lists.openembedded.org/mt/74747463/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 34/42] lttng-modules: update to 2.12.1

2020-06-08 Thread Alexander Kanavin
Drop backports.

devupstream variant was incorrectly updated in a059fc67da4
(changed to 2.12, but without updating SRCREV), so revert it back
to 2.11.2.

Signed-off-by: Alexander Kanavin 
---
 ...l-5.7-use-vmalloc_sync_mappings-on-k.patch | 853 --
 ...per-rename-to-wrapper_vmalloc_sync_m.patch |  32 -
 ..._sync_mappings-on-kernel-5.6-as-well.patch |  88 --
 ...loc_sync_mappings-for-stable-kernels.patch |  67 --
 ...ules_2.12.0.bb => lttng-modules_2.12.1.bb} |  11 +-
 5 files changed, 3 insertions(+), 1048 deletions(-)
 delete mode 100644 
meta/recipes-kernel/lttng/lttng-modules/0001-Update-for-kernel-5.7-use-vmalloc_sync_mappings-on-k.patch
 delete mode 100644 
meta/recipes-kernel/lttng/lttng-modules/0002-Fix-missing-wrapper-rename-to-wrapper_vmalloc_sync_m.patch
 delete mode 100644 
meta/recipes-kernel/lttng/lttng-modules/0003-Fix-Use-vmalloc_sync_mappings-on-kernel-5.6-as-well.patch
 delete mode 100644 
meta/recipes-kernel/lttng/lttng-modules/0004-Update-Use-vmalloc_sync_mappings-for-stable-kernels.patch
 rename meta/recipes-kernel/lttng/{lttng-modules_2.12.0.bb => 
lttng-modules_2.12.1.bb} (75%)

diff --git 
a/meta/recipes-kernel/lttng/lttng-modules/0001-Update-for-kernel-5.7-use-vmalloc_sync_mappings-on-k.patch
 
b/meta/recipes-kernel/lttng/lttng-modules/0001-Update-for-kernel-5.7-use-vmalloc_sync_mappings-on-k.patch
deleted file mode 100644
index 58f4d29193..00
--- 
a/meta/recipes-kernel/lttng/lttng-modules/0001-Update-for-kernel-5.7-use-vmalloc_sync_mappings-on-k.patch
+++ /dev/null
@@ -1,853 +0,0 @@
-From a2cc0e06330e55eefe668f7d6370de6d3177a907 Mon Sep 17 00:00:00 2001
-From: Mathieu Desnoyers 
-Date: Tue, 5 May 2020 13:38:31 -0400
-Subject: [PATCH] Update for kernel 5.7: use vmalloc_sync_mappings on kernels
- >= 5.7
-
-Upstream-Status: Backport 
[https://github.com/lttng/lttng-modules/commit/da0fcb1497ff2437407883647a8a0bba12bd0f91]
-
-Signed-off-by: Mathieu Desnoyers 
-Signed-off-by: Ovidiu Panait 

- lib/ringbuffer/ring_buffer_backend.c |  4 +--
- lttng-abi.c  |  4 +--
- lttng-context-callstack.c|  2 +-
- lttng-context-cgroup-ns.c|  2 +-
- lttng-context-cpu-id.c   |  2 +-
- lttng-context-egid.c |  2 +-
- lttng-context-euid.c |  2 +-
- lttng-context-gid.c  |  2 +-
- lttng-context-hostname.c |  2 +-
- lttng-context-interruptible.c|  2 +-
- lttng-context-ipc-ns.c   |  2 +-
- lttng-context-migratable.c   |  2 +-
- lttng-context-mnt-ns.c   |  2 +-
- lttng-context-need-reschedule.c  |  2 +-
- lttng-context-net-ns.c   |  2 +-
- lttng-context-nice.c |  2 +-
- lttng-context-perf-counters.c|  2 +-
- lttng-context-pid-ns.c   |  2 +-
- lttng-context-pid.c  |  2 +-
- lttng-context-ppid.c |  2 +-
- lttng-context-preemptible.c  |  2 +-
- lttng-context-prio.c |  2 +-
- lttng-context-procname.c |  2 +-
- lttng-context-sgid.c |  2 +-
- lttng-context-suid.c |  2 +-
- lttng-context-tid.c  |  2 +-
- lttng-context-uid.c  |  2 +-
- lttng-context-user-ns.c  |  2 +-
- lttng-context-uts-ns.c   |  2 +-
- lttng-context-vegid.c|  2 +-
- lttng-context-veuid.c|  2 +-
- lttng-context-vgid.c |  2 +-
- lttng-context-vpid.c |  2 +-
- lttng-context-vppid.c|  2 +-
- lttng-context-vsgid.c|  2 +-
- lttng-context-vsuid.c|  2 +-
- lttng-context-vtid.c |  2 +-
- lttng-context-vuid.c |  2 +-
- lttng-context.c  |  2 +-
- lttng-events.c   | 10 +++---
- lttng-ring-buffer-client.h   |  4 +--
- lttng-ring-buffer-metadata-client.h  |  4 +--
- lttng-syscalls.c |  2 +-
- probes/lttng-kprobes.c   |  2 +-
- probes/lttng-kretprobes.c|  2 +-
- probes/lttng-tracepoint-event-impl.h |  4 +--
- probes/lttng-uprobes.c   |  2 +-
- probes/lttng.c   |  2 +-
- tests/probes/lttng-test.c|  2 +-
- wrapper/vmalloc.h| 49 ++--
- 50 files changed, 104 insertions(+), 61 deletions(-)
-
-diff --git a/lib/ringbuffer/ring_buffer_backend.c 
b/lib/ringbuffer/ring_buffer_backend.c
-index d4bec25..d232b7f 100644
 a/lib/ringbuffer/ring_buffer_backend.c
-+++ b/lib/ringbuffer/ring_buffer_backend.c
-@@ -17,7 +17,7 @@
- #include 
- 
- #include 
--#include   /* for wrapper_vmalloc_sync_all() */
-+#include   /* for wrapper_vmalloc_sync_mappings() */
- #include 
- #include 
- #include 
-@@ -156,7 +156,7 @@ int lib_ring_buffer_backend_allocate(const struct 
lib_ring_buffer_config *config
-* If kmalloc ever uses vmalloc underneath, make sure the buffer pages
-