From: Denys Dmytriyenko <denis@denix.org>
To: Saul Wold <sgw@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 3/4] cryptodev-module: recipe for out-of-tree cryptodev device driver
Date: Thu, 10 Apr 2014 13:58:15 -0400 [thread overview]
Message-ID: <20140410175815.GU3370@denix.org> (raw)
In-Reply-To: <20140408005039.GK3370@denix.org>
On Mon, Apr 07, 2014 at 08:50:39PM -0400, Denys Dmytriyenko wrote:
> On Mon, Apr 07, 2014 at 03:13:14PM -0700, Saul Wold wrote:
> > On 04/06/2014 07:29 PM, Denys Dmytriyenko wrote:
> > >From: Denys Dmytriyenko <denys@ti.com>
> > >
> > >Depends on cryptodev-linux for providing a header file
> > >
> >
> > I am seeing failure on the Autobuilder with multilib enabled
> > (lib32), I am not sure if this is a multilib issue, but it shows up
> > here
>
> Ah, only tested it on 3.12 and 3.13+ requires this patch:
> http://patchwork.ozlabs.org/patch/312943/
>
> Will re-submit shortly...
Sorry for the delay - BBB BSP took priority. I just re-submitted the entire
set with an updated version of this patch #3, please let me know the results.
--
Denys
> > >NOTE: Running task 14174 of 19826 (ID: 13044, virtual:multilib:lib32:/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/libcgroup/libcgroup_0.41.bb, do_configure)
> > >ERROR: Function failed: do_compile (log file is located at /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/temp/log.do_compile.19322)
> > >ERROR: Logfile of failure stored in: /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/temp/log.do_compile.19322
> > >Log data follows:
> > >| DEBUG: Executing shell function do_compile
> > >| NOTE: make -j 16 KERNEL_DIR=/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/sysroots/qemux86/usr/src/kernel PREFIX=/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/image KERNEL_PATH=/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/sysroots/qemux86/usr/src/kernel KERNEL_SRC=/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/sysroots/qemux86/usr/src/kernel KERNEL_VERSION=3.14.0-yocto-standard CC=i586-poky-linux-gcc LD=i586-poky-linux-ld.bfd AR=i586-poky-linux-ar
> > >| make -C /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/sysroots/qemux86/usr/src/kernel SUBDIRS=`pwd` modules
> > >| make[1]: Entering directory `/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/sysroots/qemux86/usr/src/kernel'
> > >| make[1]: warning: jobserver unavailable: using -j1. Add `+' to parent make rule.
> > >| CC [M] /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/ioctl.o
> > >| /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/ioctl.c:710:21: warning: 'is_known_accelerated' defined but not used [-Wunused-function]
> > >| static unsigned int is_known_accelerated(struct crypto_tfm *tfm)
> > >| ^
> > >| CC [M] /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/main.o
> > >| CC [M] /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/cryptlib.o
> > >| /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/cryptlib.c: In function 'cryptodev_cipher_encrypt':
> > >| /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/cryptlib.c:223:2: error: implicit declaration of function 'INIT_COMPLETION' [-Werror=implicit-function-declaration]
> > >| INIT_COMPLETION(cdata->async.result->completion);
> > >| ^
> > >| cc1: some warnings being treated as errors
> > >| make[2]: *** [/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6/cryptlib.o] Error 1
> > >| make[1]: *** [_module_/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/cryptodev-linux-1.6] Error 2
> > >| make[1]: Leaving directory `/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/sysroots/qemux86/usr/src/kernel'
> > >| make: *** [build] Error 2
> > >| ERROR: oe_runmake failed
> > >| WARNING: /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/temp/run.do_compile.19322:1 exit 1 from
> > >| exit 1
> > >| ERROR: Function failed: do_compile (log file is located at /home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/build/tmp/work/qemux86-poky-linux/cryptodev-module/1.6-r0/temp/log.do_compile.19322)
> > >NOTE: recipe cryptodev-module-1.6-r0: task do_compile: Failed
> > >ERROR: Task 13808 (/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-kernel/cryptodev/cryptodev-module_1.6.bb, do_compile) failed with exit code '1'
> > >NOTE: Running task 14175 of 19826 (ID: 13002, virtual:multilib:lib32:/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-devtools/python/python-mako_0.9.1.bb, do_configure)
> > >NOTE: recipe liburcu-0.8.1-r0: task do_package_write_deb: Succeeded
> >
> >
> >
> > >Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > >---
> > > .../cryptodev/cryptodev-module_1.6.bb | 15 ++++++++++++
> > > ...talling-header-file-provided-by-another-p.patch | 28 ++++++++++++++++++++++
> > > 2 files changed, 43 insertions(+)
> > > create mode 100644 meta/recipes-kernel/cryptodev/cryptodev-module_1.6.bb
> > > create mode 100644 meta/recipes-kernel/cryptodev/files/0001-Disable-installing-header-file-provided-by-another-p.patch
> > >
> > >diff --git a/meta/recipes-kernel/cryptodev/cryptodev-module_1.6.bb b/meta/recipes-kernel/cryptodev/cryptodev-module_1.6.bb
> > >new file mode 100644
> > >index 0000000..fdf07b2
> > >--- /dev/null
> > >+++ b/meta/recipes-kernel/cryptodev/cryptodev-module_1.6.bb
> > >@@ -0,0 +1,15 @@
> > >+require cryptodev_${PV}.inc
> > >+
> > >+SUMMARY = "A /dev/crypto device driver kernel module"
> > >+
> > >+inherit module
> > >+
> > >+# Header file provided by a separate package
> > >+DEPENDS += "cryptodev-linux"
> > >+
> > >+SRC_URI += "file://0001-Disable-installing-header-file-provided-by-another-p.patch"
> > >+
> > >+EXTRA_OEMAKE='KERNEL_DIR="${STAGING_KERNEL_DIR}" PREFIX="${D}"'
> > >+
> > >+RCONFLICTS_${PN} = "ocf-linux"
> > >+RREPLACES_${PN} = "ocf-linux"
> > >diff --git a/meta/recipes-kernel/cryptodev/files/0001-Disable-installing-header-file-provided-by-another-p.patch b/meta/recipes-kernel/cryptodev/files/0001-Disable-installing-header-file-provided-by-another-p.patch
> > >new file mode 100644
> > >index 0000000..a580fc6
> > >--- /dev/null
> > >+++ b/meta/recipes-kernel/cryptodev/files/0001-Disable-installing-header-file-provided-by-another-p.patch
> > >@@ -0,0 +1,28 @@
> > >+From 8a884f55bd1527baa82fab68c186ba546273860c Mon Sep 17 00:00:00 2001
> > >+From: Denys Dmytriyenko <denys@ti.com>
> > >+Date: Sun, 6 Apr 2014 19:51:39 -0400
> > >+Subject: [PATCH] Disable installing header file provided by another package
> > >+
> > >+Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > >+
> > >+Upstream-Status: Inappropriate [ OE specific ]
> > >+---
> > >+ Makefile | 2 --
> > >+ 1 file changed, 2 deletions(-)
> > >+
> > >+diff --git a/Makefile b/Makefile
> > >+index d66ef26..8e97c6a 100644
> > >+--- a/Makefile
> > >++++ b/Makefile
> > >+@@ -23,8 +23,6 @@ install: modules_install
> > >+
> > >+ modules_install:
> > >+ make -C $(KERNEL_DIR) SUBDIRS=`pwd` modules_install
> > >+- @echo "Installing cryptodev.h in $(PREFIX)/usr/include/crypto ..."
> > >+- @install -D crypto/cryptodev.h $(PREFIX)/usr/include/crypto/cryptodev.h
> > >+
> > >+ clean:
> > >+ make -C $(KERNEL_DIR) SUBDIRS=`pwd` clean
> > >+--
> > >+1.9.1
> > >+
> > >
> >
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
next prev parent reply other threads:[~2014-04-10 17:58 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-07 2:29 [PATCH 1/4] cryptodev-linux: move to recipes-kernel to be shared with module and tests Denys Dmytriyenko
2014-04-07 2:29 ` [PATCH 2/4] cryptodev-linux: create common .inc file to be shared by " Denys Dmytriyenko
2014-04-07 2:29 ` [PATCH 3/4] cryptodev-module: recipe for out-of-tree cryptodev device driver Denys Dmytriyenko
2014-04-07 22:13 ` Saul Wold
2014-04-08 0:50 ` Denys Dmytriyenko
2014-04-10 17:58 ` Denys Dmytriyenko [this message]
2014-04-07 2:29 ` [PATCH 4/4] cryptodev-tests: recipe for cryptodev test suite based on OpenSSL Denys Dmytriyenko
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=20140410175815.GU3370@denix.org \
--to=denis@denix.org \
--cc=openembedded-core@lists.openembedded.org \
--cc=sgw@linux.intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.