Openembedded Core Discussions
 help / color / mirror / Atom feed
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


  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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox