Skip to contentSkip to conversation

lttng-modules: Update 2.7.3 -> 2.8.0+master

PATCH In review Richard Purdie 2 participants 4 messages Aug 31, 2016 – Sep 5, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 lttng-modules: Update 2.7.3 -> 2.8.0+master

We need master for the changes to work with 4.8 kernels.

meta/recipes-kernel/lttng/lttng-modules_git.bb +3 -3
index d8a7d36..96bd945 100644
@@ -8,12 +8,12 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=362844633a08753bd96ab322a6c7f9f6 \
 
 inherit module
 
-SRCREV = "5362a1f26060e0d64464bb35cdd285c914ae6171"
-PV = "2.7.3+git${SRCPV}"
+SRCREV = "e36de50dd09527901339797a61a0a40d241c1a6d"
+PV = "2.8.0+git${SRCPV}"
 
 COMPATIBLE_HOST = '(x86_64|i.86|powerpc|aarch64|mips|nios2|arm).*-linux'
 
-SRC_URI = "git://git.lttng.org/lttng-modules.git;branch=stable-2.7"
+SRC_URI = "git://git.lttng.org/lttng-modules.git;branch=master"
 
 export INSTALL_MOD_DIR="kernel/lttng-modules"
 
#1 lore raw
Quoted 3 lines · Richard Purdie

We need master for the changes to work with 4.8 kernels.

This broken for kernels which does not enable CONFIG_TRACEPOINTS.

Up to now, the lttng-modules build were fine if this was disabled just printing a warning on the build. Now it makes build fail.

The behaviour changes was made on:

https://github.com/lttng/lttng-modules/commit/ca62d279e2e83fd46b8a04e6ad4471e9f8fc7c47

Another thing I noticed is that we ought to be using stable-2.8 branch or name the recipe accordingly as master is in fact 2.9-pre.

Signature · 3 lines
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
#2 lore raw
Quoted 8 lines · Otavio Salvador
Quoted 4 lines · Richard Purdie

We need master for the changes to work with 4.8 kernels.

This broken for kernels which does not enable CONFIG_TRACEPOINTS.

Quoted 8 lines

Up to now, the lttng-modules build were fine if this was disabled just
printing a warning on the build. Now it makes build fail.

The behaviour changes was made on:

https://github.com/lttng/lttng-modules/commit/ca62d279e2e83fd46b8a04e
6ad4471e9f8fc7c47

Hmm, any idea how we should proceed then?

Quoted 2 lines

Another thing I noticed is that we ought to be using stable-2.8 branch or name the recipe accordingly as master is in fact 2.9-pre.

We'll need to rename it then as there are changes in master needed for 4.8 kernels which is why I upgraded it in the first place.

Cheers,

Richard

#3 lore raw
Quoted 13 lines · Richard Purdie
Quoted 12 lines · Otavio Salvador
Quoted 4 lines · Richard Purdie

We need master for the changes to work with 4.8 kernels.

This broken for kernels which does not enable CONFIG_TRACEPOINTS.

Up to now, the lttng-modules build were fine if this was disabled just
printing a warning on the build. Now it makes build fail.

Quoted 6 lines
Quoted 4 lines

The behaviour changes was made on:

https://github.com/lttng/lttng-modules/commit/ca62d279e2e83fd46b8a04e
6ad4471e9f8fc7c47

Hmm, any idea how we should proceed then?

Is there a mechanism to check for a kernel config prior build?

Quoted 5 lines
Quoted 2 lines

Another thing I noticed is that we ought to be using stable-2.8 branch or name the recipe accordingly as master is in fact 2.9-pre.

We'll need to rename it then as there are changes in master needed for 4.8 kernels which is why I upgraded it in the first place.

No; the 2.8 branch does support 4.8 ;-)

Signature · 3 lines
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750