[PATCH] lttng-modules: Update 2.7.3 -> 2.8.0+master

[PATCH] lttng-modules: Update 2.7.3 -> 2.8.0+master

[Richard Purdie]

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

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

diff --git a/meta/recipes-kernel/lttng/lttng-modules_git.bb b/meta/recipes-kernel/lttng/lttng-modules_git.bb
index d8a7d36..96bd945 100644
--- a/meta/recipes-kernel/lttng/lttng-modules_git.bb
+++ b/meta/recipes-kernel/lttng/lttng-modules_git.bb
@@ -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"
 

Re: [PATCH] lttng-modules: Update 2.7.3 -> 2.8.0+master

[Otavio Salvador]

On Wed, Aug 31, 2016 at 7:32 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> We need master for the changes to work with 4.8 kernels.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

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.

-- 
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

Re: [PATCH] lttng-modules: Update 2.7.3 -> 2.8.0+master

[Richard Purdie]

On Mon, 2016-09-05 at 11:40 -0300, Otavio Salvador wrote:
> On Wed, Aug 31, 2016 at 7:32 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> > 
> > We need master for the changes to work with 4.8 kernels.
> > 
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> 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/ca62d279e2e83fd46b8a04e
> 6ad4471e9f8fc7c47

Hmm, any idea how we should proceed then?

> 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

Re: [PATCH] lttng-modules: Update 2.7.3 -> 2.8.0+master

[Otavio Salvador]

On Mon, Sep 5, 2016 at 11:44 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Mon, 2016-09-05 at 11:40 -0300, Otavio Salvador wrote:
>> On Wed, Aug 31, 2016 at 7:32 AM, Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>> >
>> > We need master for the changes to work with 4.8 kernels.
>> >
>> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> 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/ca62d279e2e83fd46b8a04e
>> 6ad4471e9f8fc7c47
>
> Hmm, any idea how we should proceed then?

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

>> 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 ;-)

-- 
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