Re: [PATCH 2/8] lttng-modules: update to 2.5.0

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Fri, 2014-07-25 at 09:48 -0400, Bruce Ashfield wrote:
> On Fri, Jul 25, 2014 at 9:46 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > On Fri, Jul 25, 2014 at 09:36:12AM -0400, Bruce Ashfield wrote:
> >> On Fri, Jul 25, 2014 at 9:23 AM, Bruce Ashfield
> >> <bruce.ashfield@gmail.com> wrote:
> >> > On Fri, Jul 25, 2014 at 9:21 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> >> >> On Thu, Jul 24, 2014 at 04:41:48PM -0400, Bruce Ashfield wrote:
> >> >>> During the uprev of the yocto kernel to 3.16, lttng-modules failed to build.
> >> >>> To grab the latest stable content, we update to 2.5.0, and add two patches
> >> >>> to also make it build against 3.16+.
> >> >>>
> >> >>> We also drop the older 2.3.3 lttng-modules, since it is no longer required
> >> >>> to support ARM builds.
> >> >>>
> >> >>> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
> >> >>> ---
> >> >>>  ...compaction-instrumentation-to-3.16-kernel.patch | 83 ++++++++++++++++++++++
> >> >>>  ...ate-vmscan-instrumentation-to-3.16-kernel.patch | 70 ++++++++++++++++++
> >> >>>  meta/recipes-kernel/lttng/lttng-modules_2.3.3.bb   | 36 ----------
> >> >>>  ...tng-modules_2.4.1.bb => lttng-modules_2.5.0.bb} |  7 +-
> >> >>>  4 files changed, 157 insertions(+), 39 deletions(-)
> >> >>>  create mode 100644 meta/recipes-kernel/lttng/lttng-modules/Update-compaction-instrumentation-to-3.16-kernel.patch
> >> >>>  create mode 100644 meta/recipes-kernel/lttng/lttng-modules/Update-vmscan-instrumentation-to-3.16-kernel.patch
> >> >>>  delete mode 100644 meta/recipes-kernel/lttng/lttng-modules_2.3.3.bb
> >> >>>  rename meta/recipes-kernel/lttng/{lttng-modules_2.4.1.bb => lttng-modules_2.5.0.bb} (85%)
> >> >>>
> >> >>
> >> >> ...
> >> >>
> >> >>> diff --git a/meta/recipes-kernel/lttng/lttng-modules_2.4.1.bb b/meta/recipes-kernel/lttng/lttng-modules_2.5.0.bb
> >> >>> similarity index 85%
> >> >>> rename from meta/recipes-kernel/lttng/lttng-modules_2.4.1.bb
> >> >>> rename to meta/recipes-kernel/lttng/lttng-modules_2.5.0.bb
> >> >>> index 5e05ffc69853..5a99a5adae8b 100644
> >> >>> --- a/meta/recipes-kernel/lttng/lttng-modules_2.4.1.bb
> >> >>> +++ b/meta/recipes-kernel/lttng/lttng-modules_2.5.0.bb
> >> >>> @@ -10,14 +10,15 @@ DEPENDS = "virtual/kernel"
> >> >>
> >> >> Is the LICENSE = "LGPLv2.1 & GPLv2" correct?
> >> >
> >> > Let me double check. I was doing an update and test, and didn't notice if
> >> > this had changed.
> >> >
> >> > Even if it did change, and I'm just uprev'ing .. I'll fix it, since it
> >> > being wrong
> >> > in the past doesn't give me a free pass to keep it wrong in an update :)
> >>
> >> Too funny. There have been the three MIT licence files since 2011 .. so I agree
> >> that the recipe's summary is long.  I'll tack a patch onto the end of the series
> >> and wait to hear from RP/Saul about anything else I may have broken.
> >
> > Sorry I thought you're maintainer of lttng-*, it seems like Laurentiu is
> 
> No worries. I just get caught in the blast when lttng blows up during
> kernel uprevs, and I fix the wreckage so I can send out my updates :)
> 
> If we don't hear from Laurentiu, I can still do the update, so I'll hold it
> as a separate patch here.

I'm having fun with email, this thread got split into several different
folders. I've merged the series after seeing the nightly builds pass ok
(nice work Bruce!) so I'll take any follow up patches to fix anything
like the LICENSE fields.

Cheers,

Richard