From: Saul Wold <sgw@linux.intel.com>
To: Khem Raj <raj.khem@gmail.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/3] kmod: Upgrade to version 14
Date: Fri, 23 Aug 2013 13:37:14 -0700 [thread overview]
Message-ID: <5217C7FA.7090703@linux.intel.com> (raw)
In-Reply-To: <DBCC9871-A9D3-438D-95B2-7F6797313633@gmail.com>
On 08/22/2013 09:09 PM, Khem Raj wrote:
>
> On Aug 22, 2013, at 2:54 PM, Saul Wold <sgw@linux.intel.com> wrote:
>
>> On 08/21/2013 09:31 PM, Khem Raj wrote:
>>> The update is a requirement for systemd-206
>>>
>>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>>> ---
>>> meta/recipes-kernel/kmod/kmod-native_git.bb | 1 -
>>> meta/recipes-kernel/kmod/kmod.inc | 9 ++++----
>>> ...-man-page-generation-because-we-don-t-hav.patch | 26 ----------------------
>>> .../kmod/kmod/fix-undefined-O_CLOEXEC.patch | 16 ++++++-------
>>> meta/recipes-kernel/kmod/kmod_git.bb | 3 +--
>>> 5 files changed, 13 insertions(+), 42 deletions(-)
>>> delete mode 100644 meta/recipes-kernel/kmod/kmod/0001-man-disable-man-page-generation-because-we-don-t-hav.patch
>>>
>>
>> Did you build the native version? I am seeing failures in MUT on the autobuilder. It did pass my "sniff" test on my 2 local world builds.
>
>
> yes and multiple times. what distro do you run on your build boxes ? I wonder if its some sort of race condition
> I use -j20 and -j16
>
It's not a race condition, I built it stand alone with no parallel make.
I am not sure if a host contamination issue or something else, because
like you I was able to built it on two machine locally, but not on the AB.
Machine are CentOS 6.4, Debian 6.0.7, but it worked on my F19 and Ubuntu
12.04 machines!
Sau!
>
> % bitbake kmod-native
> Loading cache: 100% |###########################################| ETA: 00:00:00
> Loaded 1197 entries from dependency cache.
>
> Build Configuration:
> BB_VERSION = "1.19.1"
> BUILD_SYS = "x86_64-linux"
> NATIVELSBSTRING = "Ubuntu-12.04"
> TARGET_SYS = "x86_64-oe-linux"
> MACHINE = "qemux86-64"
> DISTRO_VERSION = "oe-core.0"
> TUNE_FEATURES = "m64"
> TARGET_FPU = ""
> meta = "master:2176bfd1099287056432780d0ee6b202e94bd1ee"
>
> NOTE: Resolving any missing task queue dependencies
> NOTE: Preparing runqueue
> NOTE: Executing SetScene Tasks
> NOTE: Executing RunQueue Tasks
> NOTE: Tasks Summary: Attempted 72 tasks of which 60 didn't need to be rerun and all succeeded.
>
>
>
>>
>>> | x86_64-linux-libtool: link: gcc -std=gnu99 -pipe -DANOTHER_BRICK_IN_THE -Wall -W -Wextra -Wno-inline -Wvla -Wundef -Wformat=2 -Wlogical-op -Wsign-compare -Wmissing-include-dirs -Wold-style-definition -Wpointer-arith -Winit-self -Wdeclaration-after-statement -Wfloat-equal -Wmissing-prototypes -Wstrict-prototypes -Wredundant-decls -Wmissing-declarations -Wmissing-noreturn -Wshadow -Wendif-labels -Wstrict-aliasing=2 -Wwrite-strings -Wno-long-long -Wno-overlength-strings -Wno-unused-parameter -Wno-missing-field-initializers -Wno-unused-result -Wnested-externs -Wchar-subscripts -Wtype-limits -Wuninitialized -fno-common -fdiagnostics-show-option -fvisibility=hidden -ffunction-sections -fdata-sections -isystem/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/include -O2 -pipe -Wl,--as-needed -Wl,--gc-sections -Wl,-rpath-link -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-lin
ux
>> /usr/lib -Wl,-rpath-link -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/lib -Wl,-rpath -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/lib -Wl,-rpath -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/lib -Wl,-O1 -o tools/test/kmod tools/kmod.o tools/lsmod.o tools/rmmod.o tools/insmod.o tools/modinfo.o tools/modprobe.o tools/depmod.o tools/log.o tools/static-nodes.o -L/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/lib -L/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/lib libkmod/.libs/libkmod-util.a libkmod/.libs/libkmod-private.a
>> Nice ester egg ^^^^^^^^^^^^^^^^
>>> | tools/kmod.o: In function `kmod_help':
>>> | kmod.c:(.text.kmod_help+0x27): undefined reference to `_Static_assert'
>>> | kmod.c:(.text.kmod_help+0x5c): undefined reference to `_Static_assert'
>>> | kmod.c:(.text.kmod_help+0x7f): undefined reference to `_Static_assert'
>>> | kmod.c:(.text.kmod_help+0xb9): undefined reference to `_Static_assert'
>>> | tools/kmod.o: In function `main':
>>> | kmod.c:(.text.main+0x5b): undefined reference to `_Static_assert'
>>> | tools/kmod.o:kmod.c:(.text.main+0x8c): more undefined references to `_Static_assert' follow
>>> | collect2: ld returned 1 exit status
>>> | make[2]: *** [tools/test/kmod] Error 1
>>> | make[2]: *** Waiting for unfinished jobs....
>>> | x86_64-linux-libtool: link: gcc -std=gnu99 -pipe -DANOTHER_BRICK_IN_THE -Wall -W -Wextra -Wno-inline -Wvla -Wundef -Wformat=2 -Wlogical-op -Wsign-compare -Wmissing-include-dirs -Wold-style-definition -Wpointer-arith -Winit-self -Wdeclaration-after-statement -Wfloat-equal -Wmissing-prototypes -Wstrict-prototypes -Wredundant-decls -Wmissing-declarations -Wmissing-noreturn -Wshadow -Wendif-labels -Wstrict-aliasing=2 -Wwrite-strings -Wno-long-long -Wno-overlength-strings -Wno-unused-parameter -Wno-missing-field-initializers -Wno-unused-result -Wnested-externs -Wchar-subscripts -Wtype-limits -Wuninitialized -fno-common -fdiagnostics-show-option -fvisibility=hidden -ffunction-sections -fdata-sections -isystem/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/include -O2 -pipe -Wl,--as-needed -Wl,--gc-sections -Wl,-rpath-link -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-lin
ux
>> /usr/lib -Wl,-rpath-link -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/lib -Wl,-rpath -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/lib -Wl,-rpath -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/lib -Wl,-O1 -o tools/.libs/kmod tools/kmod.o tools/lsmod.o tools/rmmod.o tools/insmod.o tools/modinfo.o tools/modprobe.o tools/depmod.o tools/log.o tools/static-nodes.o -L/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/lib -L/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/lib libkmod/.libs/libkmod-util.a libkmod/.libs/libkmod.so -Wl,-rpath -Wl,/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-x86/build/build/tmp/sysroots/x86_64-linux/usr/lib
>>> | tools/kmod.o: In function `kmod_help':
>>> | kmod.c:(.text.kmod_help+0x27): undefined reference to `_Static_assert'
>>> | kmod.c:(.text.kmod_help+0x5c): undefined reference to `_Static_assert'
>>> | kmod.c:(.text.kmod_help+0x7f): undefined reference to `_Static_assert'
>>> | kmod.c:(.text.kmod_help+0xb9): undefined reference to `_Static_assert'
>>> | tools/kmod.o: In function `main':
>>> | kmod.c:(.text.main+0x5b): undefined reference to `_Static_assert'
>>> | tools/kmod.o:kmod.c:(.text.main+0x8c): more undefined references to `_Static_assert' follow
>>> | collect2: ld returned 1 exit status
>>> | make[2]: *** [tools/kmod] Error 1
>>
>>
>> Sau!
>>
>>
>>> diff --git a/meta/recipes-kernel/kmod/kmod-native_git.bb b/meta/recipes-kernel/kmod/kmod-native_git.bb
>>> index 8cbcdf8..afa3ad0 100644
>>> --- a/meta/recipes-kernel/kmod/kmod-native_git.bb
>>> +++ b/meta/recipes-kernel/kmod/kmod-native_git.bb
>>> @@ -4,7 +4,6 @@
>>> require kmod.inc
>>> inherit native
>>>
>>> -PR = "${INC_PR}.1"
>>> SRC_URI += "file://fix-undefined-O_CLOEXEC.patch"
>>>
>>> do_install_append (){
>>> diff --git a/meta/recipes-kernel/kmod/kmod.inc b/meta/recipes-kernel/kmod/kmod.inc
>>> index a780b6c..1728a4e 100644
>>> --- a/meta/recipes-kernel/kmod/kmod.inc
>>> +++ b/meta/recipes-kernel/kmod/kmod.inc
>>> @@ -7,8 +7,6 @@ HOMEPAGE = "http://packages.profusion.mobi/kmod/"
>>> LICENSE = "GPL-2.0+ & LGPL-2.1+"
>>> LICENSE_libkmod = "LGPL-2.1+"
>>> SECTION = "base"
>>> -PV = "9"
>>> -INC_PR = "r0"
>>>
>>> DEPENDS += "pkgconfig-native"
>>>
>>> @@ -19,17 +17,18 @@ inherit autotools gtk-doc ptest
>>>
>>> SRC_URI = "git://git.kernel.org/pub/scm/utils/kernel/kmod/kmod.git \
>>> file://depmod-search.conf \
>>> - file://0001-man-disable-man-page-generation-because-we-don-t-hav.patch \
>>> file://run-ptest \
>>> file://ptest.patch \
>>> "
>>>
>>> -SRCREV = "62081c0f68905b22f375156d4532fd37fa5c8d33"
>>> +SRCREV = "3b38c7fcb58be4ddc34f90454c5f5dc3693d2d85"
>>> +# Lookout for PV bump too when SRCREV is changed
>>> +PV = "14"
>>>
>>> S = "${WORKDIR}/git"
>>>
>>> EXTRA_AUTORECONF += "--install --symlink"
>>> -EXTRA_OECONF +="--enable-debug --enable-logging --enable-tools"
>>> +EXTRA_OECONF +="--enable-debug --enable-logging --enable-tools --disable-manpages"
>>>
>>> do_configure_prepend () {
>>> gtkdocize --docdir ${S}/libkmod/docs || touch ${S}/libkmod/docs/gtk-doc.make
>>> diff --git a/meta/recipes-kernel/kmod/kmod/0001-man-disable-man-page-generation-because-we-don-t-hav.patch b/meta/recipes-kernel/kmod/kmod/0001-man-disable-man-page-generation-because-we-don-t-hav.patch
>>> deleted file mode 100644
>>> index 5361b84..0000000
>>> --- a/meta/recipes-kernel/kmod/kmod/0001-man-disable-man-page-generation-because-we-don-t-hav.patch
>>> +++ /dev/null
>>> @@ -1,26 +0,0 @@
>>> -From ce6f0cabb65e67dd4d31e1e555db8bc6eaf435d3 Mon Sep 17 00:00:00 2001
>>> -From: Martin Jansa <Martin.Jansa@gmail.com>
>>> -Date: Fri, 24 Feb 2012 07:35:38 +0100
>>> -Subject: [PATCH] man: disable man page generation because we don't have
>>> - working xsltproc
>>> -
>>> -Upstream-Status: Inappropriate [build system specific change]
>>> -
>>> -Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>> ----
>>> - Makefile.am | 2 +-
>>> - 1 files changed, 1 insertions(+), 1 deletions(-)
>>> -
>>> -diff --git a/Makefile.am b/Makefile.am
>>> -index 141c102..a8bdfd1 100644
>>> ---- a/Makefile.am
>>> -+++ b/Makefile.am
>>> -@@ -1,4 +1,4 @@
>>> --SUBDIRS = . libkmod/docs man
>>> -+SUBDIRS = . libkmod/docs
>>> -
>>> - DISTCLEAN_LOCAL_HOOKS =
>>> - EXTRA_DIST =
>>> ---
>>> -1.7.8.4
>>> -
>>> diff --git a/meta/recipes-kernel/kmod/kmod/fix-undefined-O_CLOEXEC.patch b/meta/recipes-kernel/kmod/kmod/fix-undefined-O_CLOEXEC.patch
>>> index 3177e9a..0268216 100644
>>> --- a/meta/recipes-kernel/kmod/kmod/fix-undefined-O_CLOEXEC.patch
>>> +++ b/meta/recipes-kernel/kmod/kmod/fix-undefined-O_CLOEXEC.patch
>>> @@ -2,16 +2,16 @@ Upstream-Status: Not applicable
>>>
>>> Index: git/libkmod/libkmod-private.h
>>> ===================================================================
>>> ---- git.orig/libkmod/libkmod-private.h
>>> -+++ git/libkmod/libkmod-private.h
>>> -@@ -1,6 +1,10 @@
>>> - #ifndef _LIBKMOD_PRIVATE_H_
>>> - #define _LIBKMOD_PRIVATE_H_
>>> +--- git.orig/libkmod/libkmod-private.h 2013-08-21 10:07:51.000000000 -0700
>>> ++++ git/libkmod/libkmod-private.h 2013-08-21 14:34:04.558278849 -0700
>>> +@@ -9,6 +9,10 @@
>>> + #include "macro.h"
>>> + #include "libkmod.h"
>>>
>>> +#ifndef O_CLOEXEC
>>> +# define O_CLOEXEC 0
>>> +#endif
>>> +
>>> - #include <stdbool.h>
>>> - #include <stdio.h>
>>> - #include <syslog.h>
>>> + static _always_inline_ _printf_format_(2, 3) void
>>> + kmod_log_null(struct kmod_ctx *ctx, const char *format, ...) {}
>>> +
>>> diff --git a/meta/recipes-kernel/kmod/kmod_git.bb b/meta/recipes-kernel/kmod/kmod_git.bb
>>> index f92ff72..b07c06e 100644
>>> --- a/meta/recipes-kernel/kmod/kmod_git.bb
>>> +++ b/meta/recipes-kernel/kmod/kmod_git.bb
>>> @@ -3,8 +3,7 @@
>>>
>>> require kmod.inc
>>>
>>> -PR = "${INC_PR}.0"
>>> -PV = "9+git${SRCPV}"
>>> +PV_append = "+git${SRCPV}"
>>>
>>> PROVIDES += "module-init-tools-insmod-static module-init-tools-depmod module-init-tools"
>>> RPROVIDES_${PN} += "module-init-tools-insmod-static module-init-tools-depmod module-init-tools"
>>>
>
>
>
next prev parent reply other threads:[~2013-08-23 20:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-22 4:30 [PATCH 0/3] Update systemd/kmod Khem Raj
2013-08-22 4:31 ` [PATCH 1/3] kmod: Upgrade to version 14 Khem Raj
2013-08-22 21:54 ` Saul Wold
2013-08-23 4:09 ` Khem Raj
2013-08-23 20:37 ` Saul Wold [this message]
2013-08-26 21:22 ` Khem Raj
2013-08-26 22:45 ` Khem Raj
2013-08-26 23:04 ` Saul Wold
2013-08-27 4:49 ` Khem Raj
2013-08-22 4:31 ` [PATCH 2/3] systemd: Upgrade to 206 Khem Raj
2013-08-27 2:40 ` Jonathan Liu
2013-08-27 4:53 ` Khem Raj
2013-08-22 4:31 ` [PATCH 3/3] glib-2.0: Specify libiconv when building on uclibc Khem Raj
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5217C7FA.7090703@linux.intel.com \
--to=sgw@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=raj.khem@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox