Re: [OE-core] [PATCH v2 2/2] recipes-support/nspr: Add RISC-V support

2018-06-21 Thread Martin Hundebøll



On 2018-06-21 23:26, Alistair Francis wrote:

Signed-off-by: Alistair Francis 
---
  ...initions-for-the-RISC-V-architecture.patch | 150 ++
  meta/recipes-support/nspr/nspr_4.19.bb|   1 +
  2 files changed, 151 insertions(+)
  create mode 100644 
meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch

diff --git 
a/meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch
 
b/meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch
new file mode 100644
index 00..aa32908d13
--- /dev/null
+++ 
b/meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch
@@ -0,0 +1,150 @@
+# HG changeset patch
+# User Karsten Merker 
+# Date 1523974333 -7200
+# Node ID f47871e2aeb16b39d4f516690e25c81b04d6d05a
+# Parent  776db96f834cb86e8863052201d55f60a2da91cb
+Bug 1308584, Add type definitions for the RISC-V architecture, r=kaie
+
+Signed-off-by: Alistair Francis 
+[ Changes by AF:
+ - Rebase on other Yocto patches
+]
+Upstream-Status: Merged [ 
https://hg.mozilla.org/projects/nspr/rev/f47871e2aeb1 ]


I don't think "merged" is a proper upstream status. Should be "backport" 
instead:

https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines#Patch_Header_Recommendations:_Upstream-Status


+
+diff --git a/pr/include/md/_linux.cfg b/pr/include/md/_linux.cfg
+--- a/pr/include/md/_linux.cfg
 b/pr/include/md/_linux.cfg
+@@ -1015,16 +1015,108 @@
+ #define PR_ALIGN_OF_FLOAT   4
+ #define PR_ALIGN_OF_DOUBLE  4
+ #define PR_ALIGN_OF_POINTER 4
+ #define PR_ALIGN_OF_WORD4
+
+ #define PR_BYTES_PER_WORD_LOG2   2
+ #define PR_BYTES_PER_DWORD_LOG2  3
+
++#elif defined(__riscv) && (__riscv_xlen == 32)
++
++#undef  IS_BIG_ENDIAN
++#define IS_LITTLE_ENDIAN 1
++#undef  IS_64
++
++#define PR_BYTES_PER_BYTE   1
++#define PR_BYTES_PER_SHORT  2
++#define PR_BYTES_PER_INT4
++#define PR_BYTES_PER_INT64  8
++#define PR_BYTES_PER_LONG   4
++#define PR_BYTES_PER_FLOAT  4
++#define PR_BYTES_PER_DOUBLE 8
++#define PR_BYTES_PER_WORD   4
++#define PR_BYTES_PER_DWORD  8
++
++#define PR_BITS_PER_BYTE8
++#define PR_BITS_PER_SHORT   16
++#define PR_BITS_PER_INT 32
++#define PR_BITS_PER_INT64   64
++#define PR_BITS_PER_LONG32
++#define PR_BITS_PER_FLOAT   32
++#define PR_BITS_PER_DOUBLE  64
++#define PR_BITS_PER_WORD32
++
++#define PR_BITS_PER_BYTE_LOG2   3
++#define PR_BITS_PER_SHORT_LOG2  4
++#define PR_BITS_PER_INT_LOG25
++#define PR_BITS_PER_INT64_LOG2  6
++#define PR_BITS_PER_LONG_LOG2   5
++#define PR_BITS_PER_FLOAT_LOG2  5
++#define PR_BITS_PER_DOUBLE_LOG2 6
++#define PR_BITS_PER_WORD_LOG2   5
++
++#define PR_ALIGN_OF_SHORT   2
++#define PR_ALIGN_OF_INT 4
++#define PR_ALIGN_OF_LONG4
++#define PR_ALIGN_OF_INT64   8
++#define PR_ALIGN_OF_FLOAT   4
++#define PR_ALIGN_OF_DOUBLE  8
++#define PR_ALIGN_OF_POINTER 4
++#define PR_ALIGN_OF_WORD4
++
++#define PR_BYTES_PER_WORD_LOG2  2
++#define PR_BYTES_PER_DWORD_LOG2 3
++
++#elif defined(__riscv) && (__riscv_xlen == 64)
++
++#undef  IS_BIG_ENDIAN
++#define IS_LITTLE_ENDIAN 1
++#define IS_64
++
++#define PR_BYTES_PER_BYTE   1
++#define PR_BYTES_PER_SHORT  2
++#define PR_BYTES_PER_INT4
++#define PR_BYTES_PER_INT64  8
++#define PR_BYTES_PER_LONG   8
++#define PR_BYTES_PER_FLOAT  4
++#define PR_BYTES_PER_DOUBLE 8
++#define PR_BYTES_PER_WORD   8
++#define PR_BYTES_PER_DWORD  8
++
++#define PR_BITS_PER_BYTE8
++#define PR_BITS_PER_SHORT   16
++#define PR_BITS_PER_INT 32
++#define PR_BITS_PER_INT64   64
++#define PR_BITS_PER_LONG64
++#define PR_BITS_PER_FLOAT   32
++#define PR_BITS_PER_DOUBLE  64
++#define PR_BITS_PER_WORD64
++
++#define PR_BITS_PER_BYTE_LOG2   3
++#define PR_BITS_PER_SHORT_LOG2  4
++#define PR_BITS_PER_INT_LOG25
++#define PR_BITS_PER_INT64_LOG2  6
++#define PR_BITS_PER_LONG_LOG2   6
++#define PR_BITS_PER_FLOAT_LOG2  5
++#define PR_BITS_PER_DOUBLE_LOG2 6
++#define PR_BITS_PER_WORD_LOG2   6
++
++#define PR_ALIGN_OF_SHORT   2
++#define PR_ALIGN_OF_INT 4
++#define PR_ALIGN_OF_LONG8
++#define PR_ALIGN_OF_INT64   8
++#define PR_ALIGN_OF_FLOAT   4
++#define PR_ALIGN_OF_DOUBLE  8
++#define PR_ALIGN_OF_POINTER 8
++#define PR_ALIGN_OF_WORD8
++
++#define PR_BYTES_PER_WORD_LOG2  3
++#define PR_BYTES_PER_DWORD_LOG2 3
++
+ #else
+
+ #error "Unknown CPU architecture"
+
+ #endif
+
+ #ifndef HAVE_LONG_LONG
+ #define   HAVE_LONG_LONG
+diff --git a/pr/include/md/_linux.h b/pr/include/md/_linux.h
+--- a/pr/include/md/_linux.h
 b/pr/include/md/_linux.h
+@@ -54,16 +54,20 @@
+ #elif defined(__avr32__)
+ #define _PR_SI_ARCHITECTURE "avr32"
+ #elif defined(__m32r__)
+ #define _PR_SI_ARCHITECTURE "m32r"
+ #elif defined(__nios2__)
+ #define _PR_SI_ARCHITECTURE "nios2"
+ #elif defined(__or1k__)
+ #define _PR_SI_ARCHITECTURE "or1k"
++#elif defined(__riscv) && (__riscv_xlen == 32)
++#define _PR_SI_ARCHITECTURE "riscv32"
++#elif defined(__riscv) && (__riscv_xlen == 64)

Re: [OE-core] [PATCH] systemd: quote SYSTEMD_SERVICE in prerm and postinst

2018-06-21 Thread Damien Riegel
Hi,

On Fri, Jun 22, 2018 at 09:29:37AM +0800, ChenQi wrote:
> Hi Damien,
> 
> If some recipe has more than one unit file, this patch will cause problem
> for it.

That's a good point, let me see if I can come up with a solution that
addresses that.

Regards,
-- 
Damien

> 
> Best Regards,
> Chen Qi
> 
> 
> On 06/22/2018 04:35 AM, Damien Riegel wrote:
> > Systemd mount configuration file must have a name that match the mount
> > point directory they control. So for instance, if a mount file contains
> > 
> >  [Mount]
> >  ...
> >  Where=/mnt/my-data
> > 
> > The file must be named `mnt-my\x2ddata.mount`, or systemd will refuse to
> > honour it.
> > 
> > If this config file contains an [Install] section, it will silently fail
> > because the unit file is not quoted when systemctl is called. To fix
> > that, quote `${SYSTEMD_SERVICE}` in prerm and postinst hooks.
> > 
> > Signed-off-by: Damien Riegel 
> > ---
> >   meta/classes/systemd.bbclass | 8 
> >   1 file changed, 4 insertions(+), 4 deletions(-)
> > 
> > diff --git a/meta/classes/systemd.bbclass b/meta/classes/systemd.bbclass
> > index 1b134322fb..589e73c855 100644
> > --- a/meta/classes/systemd.bbclass
> > +++ b/meta/classes/systemd.bbclass
> > @@ -34,10 +34,10 @@ if type systemctl >/dev/null 2>/dev/null; then
> > systemctl daemon-reload
> > fi
> > -   systemctl $OPTS ${SYSTEMD_AUTO_ENABLE} ${SYSTEMD_SERVICE}
> > +   systemctl $OPTS ${SYSTEMD_AUTO_ENABLE} "${SYSTEMD_SERVICE}"
> > if [ -z "$D" -a "${SYSTEMD_AUTO_ENABLE}" = "enable" ]; then
> > -   systemctl --no-block restart ${SYSTEMD_SERVICE}
> > +   systemctl --no-block restart "${SYSTEMD_SERVICE}"
> > fi
> >   fi
> >   }
> > @@ -51,10 +51,10 @@ fi
> >   if type systemctl >/dev/null 2>/dev/null; then
> > if [ -z "$D" ]; then
> > -   systemctl stop ${SYSTEMD_SERVICE}
> > +   systemctl stop "${SYSTEMD_SERVICE}"
> > fi
> > -   systemctl $OPTS disable ${SYSTEMD_SERVICE}
> > +   systemctl $OPTS disable "${SYSTEMD_SERVICE}"
> >   fi
> >   }
> 
> 
> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH V3 0/1] oeqa/runtime/cases/selftest.py: rename to _selftest.py

2018-06-21 Thread ChenQi

ping

I've tested this patch with `oe-selftest -r runtime_test', and the 
result is correct.


Best Regards,
Chen Qi

On 06/12/2018 05:08 PM, Chen Qi wrote:

Changes in V3:
* Also rename selftest.json to _selftest.json to avoid runtime_test.py failure 
in oe-selftest.
* Fixed a typo (sato -> full-cmdline)

Changes in V2:
* Fix to depend on _selftest.Selftest.test_install_package.


The following changes since commit fe6e6231441188c3ffe52ef5811a8f30d29ea952:

   local.conf.sample: update libsdl mentions to libsdl2 (2018-06-07 08:53:19 
+0100)

are available in the git repository at:

   git://git.pokylinux.org/poky-contrib ChenQi/rename_selftest
   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/rename_selftest

Chen Qi (1):
   oeqa/runtime/cases/selftest.py: rename to _selftest.py

  .../lib/oeqa/runtime/cases/{selftest.json => _selftest.json}  | 0
  meta-selftest/lib/oeqa/runtime/cases/{selftest.py => _selftest.py}| 2 +-
  meta/lib/oeqa/selftest/cases/runtime_test.py  | 4 ++--
  3 files changed, 3 insertions(+), 3 deletions(-)
  rename meta-selftest/lib/oeqa/runtime/cases/{selftest.json => _selftest.json} 
(100%)
  rename meta-selftest/lib/oeqa/runtime/cases/{selftest.py => _selftest.py} 
(94%)



--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] systemd: quote SYSTEMD_SERVICE in prerm and postinst

2018-06-21 Thread ChenQi

Hi Damien,

If some recipe has more than one unit file, this patch will cause 
problem for it.


Best Regards,
Chen Qi


On 06/22/2018 04:35 AM, Damien Riegel wrote:

Systemd mount configuration file must have a name that match the mount
point directory they control. So for instance, if a mount file contains

 [Mount]
 ...
 Where=/mnt/my-data

The file must be named `mnt-my\x2ddata.mount`, or systemd will refuse to
honour it.

If this config file contains an [Install] section, it will silently fail
because the unit file is not quoted when systemctl is called. To fix
that, quote `${SYSTEMD_SERVICE}` in prerm and postinst hooks.

Signed-off-by: Damien Riegel 
---
  meta/classes/systemd.bbclass | 8 
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/classes/systemd.bbclass b/meta/classes/systemd.bbclass
index 1b134322fb..589e73c855 100644
--- a/meta/classes/systemd.bbclass
+++ b/meta/classes/systemd.bbclass
@@ -34,10 +34,10 @@ if type systemctl >/dev/null 2>/dev/null; then
systemctl daemon-reload
fi
  
-	systemctl $OPTS ${SYSTEMD_AUTO_ENABLE} ${SYSTEMD_SERVICE}

+   systemctl $OPTS ${SYSTEMD_AUTO_ENABLE} "${SYSTEMD_SERVICE}"
  
  	if [ -z "$D" -a "${SYSTEMD_AUTO_ENABLE}" = "enable" ]; then

-   systemctl --no-block restart ${SYSTEMD_SERVICE}
+   systemctl --no-block restart "${SYSTEMD_SERVICE}"
fi
  fi
  }
@@ -51,10 +51,10 @@ fi
  
  if type systemctl >/dev/null 2>/dev/null; then

if [ -z "$D" ]; then
-   systemctl stop ${SYSTEMD_SERVICE}
+   systemctl stop "${SYSTEMD_SERVICE}"
fi
  
-	systemctl $OPTS disable ${SYSTEMD_SERVICE}

+   systemctl $OPTS disable "${SYSTEMD_SERVICE}"
  fi
  }
  



--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] mdadm: fix one more issue when building with gcc8

2018-06-21 Thread Martin Jansa
Signed-off-by: Martin Jansa 
---
 .../files/0001-Disable-gcc8-warnings.patch| 24 ++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/mdadm/files/0001-Disable-gcc8-warnings.patch 
b/meta/recipes-extended/mdadm/files/0001-Disable-gcc8-warnings.patch
index 8cf52406c4..739652cd55 100644
--- a/meta/recipes-extended/mdadm/files/0001-Disable-gcc8-warnings.patch
+++ b/meta/recipes-extended/mdadm/files/0001-Disable-gcc8-warnings.patch
@@ -6,6 +6,14 @@ Subject: [PATCH] Disable gcc8 warnings
 This is needed until
 https://bugzilla.redhat.com/show_bug.cgi?id=1553958
 
+Fix super0.c issue which is triggered only in DEBUG_BUILD with -O1 (or any
+configuration without -ftree-vrp or with -fno-tree-vrp)
+
+super0.c: In function 'examine_super0':
+super0.c:238:32: error: 'snprintf' output may be truncated before the last 
format character [-Werror=format-truncation=]
+   snprintf(nb, sizeof(nb), "%4d", d);
+^
+
 Upstream-Status: Inappropriate [Workaround]
 
 Signed-off-by: Khem Raj 
@@ -16,8 +24,22 @@ Signed-off-by: Khem Raj 
  super-ddf.c   | 4 
  super-intel.c | 6 ++
  super1.c  | 5 +
- 5 files changed, 22 insertions(+), 1 deletion(-)
+ super0.c  | 2 +-
+ 6 files changed, 22 insertions(+), 1 deletion(-)
 
+diff --git a/super0.c b/super0.c
+index f5b4507..264b598 100644
+--- a/super0.c
 b/super0.c
+@@ -231,7 +231,7 @@
+d++) {
+   mdp_disk_t *dp;
+   char *dv;
+-  char nb[11];
++  char nb[12];
+   int wonly, failfast;
+   if (d>=0) dp = >disks[d];
+   else dp = >this_disk;
 diff --git a/Monitor.c b/Monitor.c
 index 802a9d9..23b246b 100644
 --- a/Monitor.c
-- 
2.17.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-06-21 Thread Martin Jansa
I'm sorry, it's one of those days where "mtd-utils" and "mdadm" look
exactly the same to me - yeah both are in oe-core, both start with m and
are related to block devices.

And the build failure with mdadm is reproducible with DEBUG_BUILD, -O2 and
above work fine thanks to -ftree-vrp (similarly like systemd gcc8 issue
fixed in
http://git.openembedded.org/openembedded-core/commit/?id=b0fdaedc6e9c233f357022b0fb706cb19757f5c4
)



On Thu, Jun 21, 2018 at 1:30 PM Martin Jansa  wrote:

> It might be DEBUG_BUILD enabled in my builds, let me check.
>
> On Thu, Jun 21, 2018 at 1:20 PM Burton, Ross 
> wrote:
>
>> Master builds fine for me, but interestingly I don't see that in my
>> mtd-utils unpack.  Do you have patches?
>>
>> Ross
>>
>> On 21 June 2018 at 12:10, Martin Jansa  wrote:
>> > Does this upgrade fix the build failure with gcc8?
>> >
>> > With current master I'm seeing:
>> > super0.c:238:32: error: 'snprintf' output may be truncated before the
>> last
>> > format character [-Werror=format-truncation=]
>> >
>> > On Thu, Jun 21, 2018 at 12:33 PM Burton, Ross 
>> wrote:
>> >>
>> >> Sorry, no it doesn't.  I've just picked this in ross/mut but a
>> >> follow-up patch to remove that mystery patch (can't see it being used
>> >> in image generation anywhere) would be good.
>> >>
>> >> Ross
>> >>
>> >> On 20 June 2018 at 00:53, Denys Dmytriyenko  wrote:
>> >> > On Tue, May 22, 2018 at 01:16:34PM +0100, Burton, Ross wrote:
>> >> >> On 22 May 2018 at 08:49, Andrea Adami 
>> wrote:
>> >> >> > I am just wondering about this patch:we carry it since 2009  [1]
>> and
>> >> >> > is still marked as pending.
>> >> >> > Why was it necessary to ignore part of the filesystem? Can you
>> >> >> > remember?
>> >> >>
>> >> >> Make that at least 2004.  This commit in oe-classic touches it:
>> >> >>
>> >> >> commit c70605ebdca81412a93b4d867de7931543193f26
>> >> >> Author: Chris Larson 
>> >> >> Date:   Thu Dec 23 21:24:57 2004 +
>> >> >>
>> >> >> Update mtd to a current snapshot, change the mkfs.jffs2 -i
>> cmdline
>> >> >> option to
>> >> >> -I, and rename mtd and mtd-native to mtd-utils and
>> >> >> mtd-utils-native,
>> >> >> respectively.
>> >> >>
>> >> >> Prior history is lost to the git/bk migration unless anyone has a
>> >> >> BitKeeper clone to mine...
>> >> >
>> >> > So, does this prevent the version upgrade from being merged?
>> >> >
>> >> > --
>> >> > Denys
>> >> --
>> >> ___
>> >> Openembedded-core mailing list
>> >> Openembedded-core@lists.openembedded.org
>> >> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] iproute2: upgrade 4.16.0 -> 4.17.0

2018-06-21 Thread Changhyeok Bae
Signed-off-by: Changhyeok Bae 
---
 .../iproute2/{iproute2_4.16.0.bb => iproute2_4.17.0.bb}   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-connectivity/iproute2/{iproute2_4.16.0.bb => 
iproute2_4.17.0.bb} (72%)

diff --git a/meta/recipes-connectivity/iproute2/iproute2_4.16.0.bb 
b/meta/recipes-connectivity/iproute2/iproute2_4.17.0.bb
similarity index 72%
rename from meta/recipes-connectivity/iproute2/iproute2_4.16.0.bb
rename to meta/recipes-connectivity/iproute2/iproute2_4.17.0.bb
index 4b0e893..12812b0 100644
--- a/meta/recipes-connectivity/iproute2/iproute2_4.16.0.bb
+++ b/meta/recipes-connectivity/iproute2/iproute2_4.17.0.bb
@@ -6,8 +6,8 @@ SRC_URI = 
"${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BP}.tar.xz \
file://0001-ip-Remove-unneed-header.patch \
   "
 
-SRC_URI[md5sum] = "1f12a70d767ef77ffa2d1a0c4ce48f1a"
-SRC_URI[sha256sum] = 
"0c5c24020fd7349fe25728c5edee9fb6a1bc8a38f08e23be5c57a6301e55ee0a"
+SRC_URI[md5sum] = "6f3a87fe2d97c28214fc2faab2c257d6"
+SRC_URI[sha256sum] = 
"6fa991b092315887775b9e47dc6a89af7ae09dd3ad4ccff754d055c566b4be6e"
 
 # CFLAGS are computed in Makefile and reference CCOPTS
 #
-- 
2.7.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2 1/2] recipes-devtools/qemu: Add RISC-V support

2018-06-21 Thread Alistair Francis
Signed-off-by: Alistair Francis 
---
 meta/recipes-devtools/qemu/qemu.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/qemu/qemu.inc 
b/meta/recipes-devtools/qemu/qemu.inc
index 094a975177..b05c1cee9e 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -9,7 +9,7 @@ inherit pkgconfig bluetooth
 BBCLASSEXTEND = "native nativesdk"
 
 # QEMU_TARGETS is overridable variable
-QEMU_TARGETS ?= "arm aarch64 i386 mips mipsel mips64 mips64el ppc sh4 x86_64"
+QEMU_TARGETS ?= "arm aarch64 i386 mips mipsel mips64 mips64el ppc riscv32 
riscv64 sh4 x86_64"
 
 EXTRA_OECONF = " \
 --prefix=${prefix} \
-- 
2.17.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2 2/2] recipes-support/nspr: Add RISC-V support

2018-06-21 Thread Alistair Francis
Signed-off-by: Alistair Francis 
---
 ...initions-for-the-RISC-V-architecture.patch | 150 ++
 meta/recipes-support/nspr/nspr_4.19.bb|   1 +
 2 files changed, 151 insertions(+)
 create mode 100644 
meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch

diff --git 
a/meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch
 
b/meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch
new file mode 100644
index 00..aa32908d13
--- /dev/null
+++ 
b/meta/recipes-support/nspr/nspr/0003-Add-type-definitions-for-the-RISC-V-architecture.patch
@@ -0,0 +1,150 @@
+# HG changeset patch
+# User Karsten Merker 
+# Date 1523974333 -7200
+# Node ID f47871e2aeb16b39d4f516690e25c81b04d6d05a
+# Parent  776db96f834cb86e8863052201d55f60a2da91cb
+Bug 1308584, Add type definitions for the RISC-V architecture, r=kaie
+
+Signed-off-by: Alistair Francis 
+[ Changes by AF:
+ - Rebase on other Yocto patches
+]
+Upstream-Status: Merged [ 
https://hg.mozilla.org/projects/nspr/rev/f47871e2aeb1 ]
+
+diff --git a/pr/include/md/_linux.cfg b/pr/include/md/_linux.cfg
+--- a/pr/include/md/_linux.cfg
 b/pr/include/md/_linux.cfg
+@@ -1015,16 +1015,108 @@
+ #define PR_ALIGN_OF_FLOAT   4
+ #define PR_ALIGN_OF_DOUBLE  4
+ #define PR_ALIGN_OF_POINTER 4
+ #define PR_ALIGN_OF_WORD4
+ 
+ #define PR_BYTES_PER_WORD_LOG2   2
+ #define PR_BYTES_PER_DWORD_LOG2  3
+ 
++#elif defined(__riscv) && (__riscv_xlen == 32)
++
++#undef  IS_BIG_ENDIAN
++#define IS_LITTLE_ENDIAN 1
++#undef  IS_64
++
++#define PR_BYTES_PER_BYTE   1
++#define PR_BYTES_PER_SHORT  2
++#define PR_BYTES_PER_INT4
++#define PR_BYTES_PER_INT64  8
++#define PR_BYTES_PER_LONG   4
++#define PR_BYTES_PER_FLOAT  4
++#define PR_BYTES_PER_DOUBLE 8
++#define PR_BYTES_PER_WORD   4
++#define PR_BYTES_PER_DWORD  8
++
++#define PR_BITS_PER_BYTE8
++#define PR_BITS_PER_SHORT   16
++#define PR_BITS_PER_INT 32
++#define PR_BITS_PER_INT64   64
++#define PR_BITS_PER_LONG32
++#define PR_BITS_PER_FLOAT   32
++#define PR_BITS_PER_DOUBLE  64
++#define PR_BITS_PER_WORD32
++
++#define PR_BITS_PER_BYTE_LOG2   3
++#define PR_BITS_PER_SHORT_LOG2  4
++#define PR_BITS_PER_INT_LOG25
++#define PR_BITS_PER_INT64_LOG2  6
++#define PR_BITS_PER_LONG_LOG2   5
++#define PR_BITS_PER_FLOAT_LOG2  5
++#define PR_BITS_PER_DOUBLE_LOG2 6
++#define PR_BITS_PER_WORD_LOG2   5
++
++#define PR_ALIGN_OF_SHORT   2
++#define PR_ALIGN_OF_INT 4
++#define PR_ALIGN_OF_LONG4
++#define PR_ALIGN_OF_INT64   8
++#define PR_ALIGN_OF_FLOAT   4
++#define PR_ALIGN_OF_DOUBLE  8
++#define PR_ALIGN_OF_POINTER 4
++#define PR_ALIGN_OF_WORD4
++
++#define PR_BYTES_PER_WORD_LOG2  2
++#define PR_BYTES_PER_DWORD_LOG2 3
++
++#elif defined(__riscv) && (__riscv_xlen == 64)
++
++#undef  IS_BIG_ENDIAN
++#define IS_LITTLE_ENDIAN 1
++#define IS_64
++
++#define PR_BYTES_PER_BYTE   1
++#define PR_BYTES_PER_SHORT  2
++#define PR_BYTES_PER_INT4
++#define PR_BYTES_PER_INT64  8
++#define PR_BYTES_PER_LONG   8
++#define PR_BYTES_PER_FLOAT  4
++#define PR_BYTES_PER_DOUBLE 8
++#define PR_BYTES_PER_WORD   8
++#define PR_BYTES_PER_DWORD  8
++
++#define PR_BITS_PER_BYTE8
++#define PR_BITS_PER_SHORT   16
++#define PR_BITS_PER_INT 32
++#define PR_BITS_PER_INT64   64
++#define PR_BITS_PER_LONG64
++#define PR_BITS_PER_FLOAT   32
++#define PR_BITS_PER_DOUBLE  64
++#define PR_BITS_PER_WORD64
++
++#define PR_BITS_PER_BYTE_LOG2   3
++#define PR_BITS_PER_SHORT_LOG2  4
++#define PR_BITS_PER_INT_LOG25
++#define PR_BITS_PER_INT64_LOG2  6
++#define PR_BITS_PER_LONG_LOG2   6
++#define PR_BITS_PER_FLOAT_LOG2  5
++#define PR_BITS_PER_DOUBLE_LOG2 6
++#define PR_BITS_PER_WORD_LOG2   6
++
++#define PR_ALIGN_OF_SHORT   2
++#define PR_ALIGN_OF_INT 4
++#define PR_ALIGN_OF_LONG8
++#define PR_ALIGN_OF_INT64   8
++#define PR_ALIGN_OF_FLOAT   4
++#define PR_ALIGN_OF_DOUBLE  8
++#define PR_ALIGN_OF_POINTER 8
++#define PR_ALIGN_OF_WORD8
++
++#define PR_BYTES_PER_WORD_LOG2  3
++#define PR_BYTES_PER_DWORD_LOG2 3
++
+ #else
+ 
+ #error "Unknown CPU architecture"
+ 
+ #endif
+ 
+ #ifndef HAVE_LONG_LONG
+ #define   HAVE_LONG_LONG
+diff --git a/pr/include/md/_linux.h b/pr/include/md/_linux.h
+--- a/pr/include/md/_linux.h
 b/pr/include/md/_linux.h
+@@ -54,16 +54,20 @@
+ #elif defined(__avr32__)
+ #define _PR_SI_ARCHITECTURE "avr32"
+ #elif defined(__m32r__)
+ #define _PR_SI_ARCHITECTURE "m32r"
+ #elif defined(__nios2__)
+ #define _PR_SI_ARCHITECTURE "nios2"
+ #elif defined(__or1k__)
+ #define _PR_SI_ARCHITECTURE "or1k"
++#elif defined(__riscv) && (__riscv_xlen == 32)
++#define _PR_SI_ARCHITECTURE "riscv32"
++#elif defined(__riscv) && (__riscv_xlen == 64)
++#define _PR_SI_ARCHITECTURE "riscv64"
+ #else
+ #error "Unknown CPU architecture"
+ #endif
+ #define PR_DLL_SUFFIX ".so"
+ 
+ #define _PR_VMBASE  0x3000
+ #define _PR_STACK_VMBASE  0x5000
+ #define 

[OE-core] [PATCH 3/5] bbpath-intercepts.bbclass: add class

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

This class sets POSTINST_INTERCEPTS and POSTINST_INTERCEPTS_CHECKSUMS,
to allow us to pull intercepts from BBPATH. This is kept as a separate
class, as it's needed by both image construction and sdk construction,
the latter to support meta-toolchain & similar recipes.

Signed-off-by: Christopher Larson 
---
 meta/classes/bbpath-intercepts.bbclass | 23 +++
 1 file changed, 23 insertions(+)
 create mode 100644 meta/classes/bbpath-intercepts.bbclass

diff --git a/meta/classes/bbpath-intercepts.bbclass 
b/meta/classes/bbpath-intercepts.bbclass
new file mode 100644
index 000..ed30bbd98d9
--- /dev/null
+++ b/meta/classes/bbpath-intercepts.bbclass
@@ -0,0 +1,23 @@
+# Gather existing and candidate postinst intercepts from BBPATH
+POSTINST_INTERCEPTS_DIR ?= "${COREBASE}/scripts/postinst-intercepts"
+POSTINST_INTERCEPTS_PATHS ?= "${@':'.join('%s/postinst-intercepts' % p for p 
in '${BBPATH}'.split(':'))}:${POSTINST_INTERCEPTS_DIR}"
+
+python find_intercepts() {
+intercepts = {}
+search_paths = []
+paths = d.getVar('POSTINST_INTERCEPTS_PATHS').split(':')
+overrides = (':' + d.getVar('FILESOVERRIDES')).split(':') + ['']
+search_paths = [os.path.join(p, op) for p in paths for op in overrides]
+searched = oe.path.which_wild('*', ':'.join(search_paths), candidates=True)
+files, chksums = [], []
+for pathname, candidates in searched:
+if os.path.isfile(pathname):
+files.append(pathname)
+chksums.append('%s:True' % pathname)
+chksums.extend('%s:False' % c for c in candidates[:-1])
+
+d.setVar('POSTINST_INTERCEPT_CHECKSUMS', ' '.join(chksums))
+d.setVar('POSTINST_INTERCEPTS', ' '.join(files))
+}
+find_intercepts[eventmask] += "bb.event.RecipePreFinalise"
+addhandler find_intercepts
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 5/5] populate_sdk_base.bbclass: inherit and use bbpath-intercepts

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Signed-off-by: Christopher Larson 
---
 meta/classes/populate_sdk_base.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/populate_sdk_base.bbclass 
b/meta/classes/populate_sdk_base.bbclass
index 3da350747e2..8b576e1bfde 100644
--- a/meta/classes/populate_sdk_base.bbclass
+++ b/meta/classes/populate_sdk_base.bbclass
@@ -1,4 +1,4 @@
-inherit meta
+inherit meta bbpath-intercepts
 
 # Wildcards specifying complementary packages to install for every package 
that has been explicitly
 # installed into the rootfs
@@ -306,4 +306,5 @@ do_populate_sdk[dirs] = "${PKGDATA_DIR} ${TOPDIR}"
 do_populate_sdk[depends] += "${@' '.join([x + ':do_populate_sysroot' for x in 
d.getVar('SDK_DEPENDS').split()])}  ${@d.getVarFlag('do_rootfs', 'depends', 
False)}"
 do_populate_sdk[rdepends] = "${@' '.join([x + 
':do_package_write_${IMAGE_PKGTYPE} ' + x + ':do_packagedata' for x in 
d.getVar('SDK_RDEPENDS').split()])}"
 do_populate_sdk[recrdeptask] += "do_packagedata do_package_write_rpm 
do_package_write_ipk do_package_write_deb"
+do_populate_sdk[file-checksums] += "${POSTINST_INTERCEPT_CHECKSUMS}"
 addtask populate_sdk
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 2/5] oe.package_manager: support loading intercepts from multiple paths

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

- if POSTINST_INTERCEPTS is set, use the listed intercept files, or
- if POSTINST_INTERCEPTS_PATH is set, load from the listed paths, or
- if POSTINST_INTERCEPTS_DIR is set, load from it (for compatibility), or
- load from ${COREBASE}/meta/postinst-intercepts

Signed-off-by: Christopher Larson 
---
 meta/lib/oe/package_manager.py | 19 +--
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
index 5ac729455e9..7f9cbba7084 100644
--- a/meta/lib/oe/package_manager.py
+++ b/meta/lib/oe/package_manager.py
@@ -336,17 +336,24 @@ class PackageManager(object, metaclass=ABCMeta):
 
 def _initialize_intercepts(self):
 bb.note("Initializing intercept dir for %s" % self.target_rootfs)
-postinst_intercepts_dir = self.d.getVar("POSTINST_INTERCEPTS_DIR")
-if not postinst_intercepts_dir:
-postinst_intercepts_dir = 
self.d.expand("${COREBASE}/scripts/postinst-intercepts")
 # As there might be more than one instance of PackageManager operating 
at the same time
 # we need to isolate the intercept_scripts directories from each other,
 # hence the ugly hash digest in dir name.
-self.intercepts_dir = os.path.join(self.d.getVar('WORKDIR'),
-  "intercept_scripts-%s" 
%(hashlib.sha256(self.target_rootfs.encode()).hexdigest()) )
+self.intercepts_dir = os.path.join(self.d.getVar('WORKDIR'), 
"intercept_scripts-%s" %
+   
(hashlib.sha256(self.target_rootfs.encode()).hexdigest()))
 
+postinst_intercepts = (self.d.getVar("POSTINST_INTERCEPTS") or 
"").split()
+if not postinst_intercepts:
+postinst_intercepts_path = 
self.d.getVar("POSTINST_INTERCEPTS_PATH")
+if not postinst_intercepts_path:
+postinst_intercepts_path = 
self.d.getVar("POSTINST_INTERCEPTS_DIR") or 
self.d.expand("${COREBASE}/scripts/postinst-intercepts")
+postinst_intercepts = oe.path.which_wild('*', 
postinst_intercepts_path)
+
+bb.debug(1, 'Collected intercepts:\n%s' % ''.join('  %s\n' % i for i 
in postinst_intercepts))
 bb.utils.remove(self.intercepts_dir, True)
-shutil.copytree(postinst_intercepts_dir, self.intercepts_dir)
+bb.utils.mkdirhier(self.intercepts_dir)
+for intercept in postinst_intercepts:
+bb.utils.copyfile(intercept, os.path.join(self.intercepts_dir, 
os.path.basename(intercept)))
 
 @abstractmethod
 def _handle_intercept_failure(self, failed_script):
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] recipetool: add 'edit' subcommand

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

This edits the recipe and any bbappends for the specified target.

Signed-off-by: Christopher Larson 
---
 scripts/lib/recipetool/edit.py | 54 ++
 1 file changed, 54 insertions(+)
 create mode 100644 scripts/lib/recipetool/edit.py

diff --git a/scripts/lib/recipetool/edit.py b/scripts/lib/recipetool/edit.py
new file mode 100644
index 000..c4789a9994c
--- /dev/null
+++ b/scripts/lib/recipetool/edit.py
@@ -0,0 +1,54 @@
+# Recipe creation tool - edit plugin
+#
+# This sub-command edits the recipe and appends for the specified target
+#
+# Example: recipetool edit busybox
+#
+# Copyright (C) 2018 Mentor Graphics Corporation
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License version 2 as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+import argparse
+import errno
+import logging
+import os
+import re
+import subprocess
+import sys
+import scriptutils
+
+
+logger = logging.getLogger('recipetool')
+tinfoil = None
+
+
+def tinfoil_init(instance):
+global tinfoil
+tinfoil = instance
+
+
+def edit(args):
+import oe.recipeutils
+
+recipe_path = tinfoil.get_recipe_file(args.target)
+appends = tinfoil.get_file_appends(recipe_path)
+
+return scriptutils.run_editor([recipe_path] + appends, logger)
+
+
+def register_commands(subparsers):
+parser = subparsers.add_parser('edit',
+   help='Edit the recipe and appends for the 
specified target. This obeys $VISUAL if set, otherwise $EDITOR, otherwise vi.')
+parser.add_argument('target', help='Target recipe/provide to edit')
+parser.set_defaults(func=edit, parserecipes=True)
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 4/5] image.bbclass: inherit and use bbpath-intercepts

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Signed-off-by: Christopher Larson 
---
 meta/classes/image.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index 2247b305da8..29bd716f8d3 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -191,7 +191,7 @@ LINGUAS_INSTALL ?= "${@" ".join(map(lambda s: 
"locale-base-%s" % s, d.getVar('IM
 # aren't yet available.
 PSEUDO_PASSWD = "${IMAGE_ROOTFS}:${STAGING_DIR_NATIVE}"
 
-inherit rootfs-postcommands
+inherit rootfs-postcommands bbpath-intercepts
 
 PACKAGE_EXCLUDE ??= ""
 PACKAGE_EXCLUDE[type] = "list"
@@ -262,6 +262,7 @@ fakeroot python do_rootfs () {
 do_rootfs[dirs] = "${TOPDIR}"
 do_rootfs[cleandirs] += "${S} ${IMGDEPLOYDIR}"
 do_rootfs[umask] = "022"
+do_rootfs[file-checksums] += "${POSTINST_INTERCEPT_CHECKSUMS}"
 addtask rootfs after do_prepare_recipe_sysroot
 
 fakeroot python do_image () {
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 0/5] Pull postinst-intercepts from BBPATH

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

These are the bits to  allow us to pull intercepts from BBPATH. This is kept
as a separate class, as it's needed by both image construction and sdk
construction, the latter to support meta-toolchain & similar recipes.

The following changes since commit 83c9405df5748744ef673ac8757bb89d7050ad8d:

  alsa-tools: rewrite packaging (2018-06-15 11:18:15 +0100)

are available in the git repository at:

  g...@github.com:kergoth/openembedded-core.git bbpath-intercepts

for you to fetch changes up to 4bb748f4c9b177722a13a1f09dcd86b4595752ea:

  populate_sdk_base.bbclass: inherit and use bbpath-intercepts (2018-06-16 
02:05:39 +0500)


Christopher Larson (5):
  oe.path: add which_wild function
  oe.package_manager: support loading intercepts from multiple paths
  bbpath-intercepts.bbclass: add class
  image.bbclass: inherit and use bbpath-intercepts
  populate_sdk_base.bbclass: inherit and use bbpath-intercepts

 meta/classes/bbpath-intercepts.bbclass | 23 +++
 meta/classes/image.bbclass |  3 ++-
 meta/classes/populate_sdk_base.bbclass |  3 ++-
 meta/lib/oe/package_manager.py | 19 +--
 meta/lib/oe/path.py| 34 ++
 5 files changed, 74 insertions(+), 8 deletions(-)
 create mode 100644 meta/classes/bbpath-intercepts.bbclass

-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 1/2] rng-tools: drop unnecessary INHIBIT_UPDATERCD_BBCLASS

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

This was added as the recipe didn't inherit systemd, but it does inherit
systemd now.

Signed-off-by: Christopher Larson 
---
 meta/recipes-support/rng-tools/rng-tools_5.bb | 7 ---
 1 file changed, 7 deletions(-)

diff --git a/meta/recipes-support/rng-tools/rng-tools_5.bb 
b/meta/recipes-support/rng-tools/rng-tools_5.bb
index b3c9fd97455..24d545dc3fa 100644
--- a/meta/recipes-support/rng-tools/rng-tools_5.bb
+++ b/meta/recipes-support/rng-tools/rng-tools_5.bb
@@ -16,13 +16,6 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/gkernel/${BP}.tar.gz \
 SRC_URI[md5sum] = "6726cdc6fae1f5122463f24ae980dd68"
 SRC_URI[sha256sum] = 
"60a102b6603bbcce2da341470cad42eeaa9564a16b4490e7867026ca11a3078e"
 
-# As the recipe doesn't inherit systemd.bbclass, we need to set this variable
-# manually to avoid unnecessary postinst/preinst generated.
-python () {
-if not bb.utils.contains('DISTRO_FEATURES', 'sysvinit', True, False, d):
-d.setVar("INHIBIT_UPDATERCD_BBCLASS", "1")
-}
-
 inherit autotools update-rc.d systemd
 
 PACKAGECONFIG = "libgcrypt"
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 6/6] perf: drop the rdep on man

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

It's not common practice to make -doc rdep on man when man pages are
included, and doing so prevents the recipe from building for non-gplv3

Signed-off-by: Christopher Larson 
---
 meta/recipes-kernel/perf/perf.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta/recipes-kernel/perf/perf.bb b/meta/recipes-kernel/perf/perf.bb
index 51f5597eb92..5e710f61834 100644
--- a/meta/recipes-kernel/perf/perf.bb
+++ b/meta/recipes-kernel/perf/perf.bb
@@ -239,7 +239,6 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
 PACKAGES =+ "${PN}-archive ${PN}-tests ${PN}-perl ${PN}-python"
 
 RDEPENDS_${PN} += "elfutils bash"
-RDEPENDS_${PN}-doc += "man"
 RDEPENDS_${PN}-archive =+ "bash"
 RDEPENDS_${PN}-python =+ "bash python python-modules 
${@bb.utils.contains('PACKAGECONFIG', 'audit', 'audit-python', '', d)}"
 RDEPENDS_${PN}-perl =+ "bash perl perl-modules"
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 0/6] A few misc fixes from meta-mentor-staging

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

The following changes since commit 83c9405df5748744ef673ac8757bb89d7050ad8d:

  alsa-tools: rewrite packaging (2018-06-15 11:18:15 +0100)

are available in the git repository at:

  g...@github.com:kergoth/openembedded-core.git misc-fixes

for you to fetch changes up to 5ce7b09e00b4967d2c0efaad58049cf871fab57a:

  perf: drop the rdep on man (2018-06-16 02:04:35 +0500)


Christopher Larson (6):
  oe.scriptutils.run_editor: ditch the error-prone argument quoting
  pcmciautils: depend on bison-native
  console-tools: add missing flex/bison deps
  ovmf: set PARALLEL_MAKE for target as well
  dmidecode: correct docdir
  perf: drop the rdep on man

 meta/recipes-bsp/pcmciautils/pcmciautils.inc   |  2 +-
 .../console-tools/console-tools_0.3.2.bb   |  1 +
 meta/recipes-core/ovmf/ovmf_git.bb |  2 +-
 meta/recipes-devtools/dmidecode/dmidecode_3.1.bb   |  4 
 meta/recipes-kernel/perf/perf.bb   |  1 -
 scripts/lib/scriptutils.py | 24 +++---
 6 files changed, 19 insertions(+), 15 deletions(-)

-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 2/2] rng-tools: start earlier in the boot process

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Entropy is needed earlier in the boot process in some cases, for example
connman can require it, and rgd doesn't require much, so move it earlier
in the boot process, 03 for sysvinit, and before sysinit for systemd.

Signed-off-by: Christopher Larson 
---
 meta/recipes-support/rng-tools/rng-tools/rngd.service | 3 +++
 meta/recipes-support/rng-tools/rng-tools_5.bb | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-support/rng-tools/rng-tools/rngd.service 
b/meta/recipes-support/rng-tools/rng-tools/rngd.service
index b94ad50209c..cb8102442d6 100644
--- a/meta/recipes-support/rng-tools/rng-tools/rngd.service
+++ b/meta/recipes-support/rng-tools/rng-tools/rngd.service
@@ -1,5 +1,8 @@
 [Unit]
 Description=Hardware RNG Entropy Gatherer Daemon
+DefaultDependencies=no
+After=systemd-udev-settle.service
+Before=sysinit.target
 
 [Service]
 ExecStart=@SBINDIR@/rngd -f -r /dev/urandom
diff --git a/meta/recipes-support/rng-tools/rng-tools_5.bb 
b/meta/recipes-support/rng-tools/rng-tools_5.bb
index 24d545dc3fa..6765667a59b 100644
--- a/meta/recipes-support/rng-tools/rng-tools_5.bb
+++ b/meta/recipes-support/rng-tools/rng-tools_5.bb
@@ -43,6 +43,6 @@ do_install_append() {
 }
 
 INITSCRIPT_NAME = "rng-tools"
-INITSCRIPT_PARAMS = "start 30 2 3 4 5 . stop 30 0 6 1 ."
+INITSCRIPT_PARAMS = "start 03 2 3 4 5 . stop 30 0 6 1 ."
 
 SYSTEMD_SERVICE_${PN} = "rngd.service"
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 1/5] oe.path: add which_wild function

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

This is a function much like shutil.which or bb.utils.which, retaining
shutil.which-like function semantics, bb.utils.which's support for
returning available candidates for signatures, and most importantly,
supports wildcards, returning only the first occurrance of each found
pathname in the search path.

Signed-off-by: Christopher Larson 
---
 meta/lib/oe/path.py | 34 ++
 1 file changed, 34 insertions(+)

diff --git a/meta/lib/oe/path.py b/meta/lib/oe/path.py
index 76c58fa7601..be02218c31d 100644
--- a/meta/lib/oe/path.py
+++ b/meta/lib/oe/path.py
@@ -259,3 +259,37 @@ def is_path_parent(possible_parent, *paths):
 if not path_abs.startswith(possible_parent_abs):
 return False
 return True
+
+def which_wild(pathname, path=None, mode=os.F_OK, *, reverse=False, 
candidates=False):
+"""Search a search path for pathname, supporting wildcards.
+
+Return all paths in the specific search path matching the wildcard pattern
+in pathname, returning only the first encountered for each file. If
+candidates is True, information on all potential candidate paths are
+included.
+"""
+paths = (path or os.environ.get('PATH', os.defpath)).split(':')
+if reverse:
+paths.reverse()
+
+seen, files = set(), []
+for index, element in enumerate(paths):
+if not os.path.isabs(element):
+element = os.path.abspath(element)
+
+candidate = os.path.join(element, pathname)
+globbed = glob.glob(candidate)
+if globbed:
+for found_path in sorted(globbed):
+if not os.access(found_path, mode):
+continue
+rel = os.path.relpath(found_path, element)
+if rel not in seen:
+seen.add(rel)
+if candidates:
+files.append((found_path, [os.path.join(p, rel) for p 
in paths[:index+1]]))
+else:
+files.append(found_path)
+
+return files
+
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 5/6] dmidecode: correct docdir

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Without this, the package clutters up the root of /usr/share/doc.

Signed-off-by: Christopher Larson 
---
 meta/recipes-devtools/dmidecode/dmidecode_3.1.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-devtools/dmidecode/dmidecode_3.1.bb 
b/meta/recipes-devtools/dmidecode/dmidecode_3.1.bb
index f83281e2359..b423fda6d24 100644
--- a/meta/recipes-devtools/dmidecode/dmidecode_3.1.bb
+++ b/meta/recipes-devtools/dmidecode/dmidecode_3.1.bb
@@ -9,6 +9,10 @@ COMPATIBLE_HOST = 
"(i.86|x86_64|aarch64|arm|powerpc|powerpc64).*-linux"
 
 EXTRA_OEMAKE = "-e MAKEFLAGS="
 
+# The upstream buildsystem uses 'docdir' as the path where it puts AUTHORS,
+# README, etc, but we don't want those in the root of our docdir.
+docdir .= "/${BPN}"
+
 do_install() {
oe_runmake DESTDIR="${D}" install
 }
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 0/2] rng-tools: start earlier in the boot process

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Entropy is needed earlier in the boot process in some cases, for example
connman can require it, and rgd doesn't require much, so move it earlier
in the boot process, 03 for sysvinit, and before sysinit for systemd.

The following changes since commit 83c9405df5748744ef673ac8757bb89d7050ad8d:

  alsa-tools: rewrite packaging (2018-06-15 11:18:15 +0100)

are available in the git repository at:

  g...@github.com:kergoth/openembedded-core.git rng-tools-early

for you to fetch changes up to abb55c262e6dab31c0e77bf800e83cdba155fc13:

  rng-tools: start earlier in the boot process (2018-06-16 02:05:09 +0500)


Christopher Larson (2):
  rng-tools: drop unnecessary INHIBIT_UPDATERCD_BBCLASS
  rng-tools: start earlier in the boot process

 meta/recipes-support/rng-tools/rng-tools/rngd.service | 3 +++
 meta/recipes-support/rng-tools/rng-tools_5.bb | 9 +
 2 files changed, 4 insertions(+), 8 deletions(-)

-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 4/6] ovmf: set PARALLEL_MAKE for target as well

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

This can fail for target, not just native.

Signed-off-by: Christopher Larson 
---
 meta/recipes-core/ovmf/ovmf_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/ovmf/ovmf_git.bb 
b/meta/recipes-core/ovmf/ovmf_git.bb
index 212530acbf9..e57fa097274 100644
--- a/meta/recipes-core/ovmf/ovmf_git.bb
+++ b/meta/recipes-core/ovmf/ovmf_git.bb
@@ -39,7 +39,7 @@ SRC_URI[openssl.sha256sum] = 
"57be8618979d80c910728cfc99369bf97b2a1abd8f366ab6eb
 
 inherit deploy
 
-PARALLEL_MAKE_class-native = ""
+PARALLEL_MAKE = ""
 
 S = "${WORKDIR}/git"
 
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 3/6] console-tools: add missing flex/bison deps

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Signed-off-by: Christopher Larson 
---
 meta/recipes-core/console-tools/console-tools_0.3.2.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/console-tools/console-tools_0.3.2.bb 
b/meta/recipes-core/console-tools/console-tools_0.3.2.bb
index 6b9c82834cb..ba44fbe60f9 100644
--- a/meta/recipes-core/console-tools/console-tools_0.3.2.bb
+++ b/meta/recipes-core/console-tools/console-tools_0.3.2.bb
@@ -3,6 +3,7 @@ DESCRIPTION = "Provides tools that enable the set-up and 
manipulation of the lin
 SECTION = "base"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING.kbd;md5=9b2d91511d3d80d4d20ac6e6b0137fe9"
+DEPENDS = "flex-native bison-native"
 PR = "r8"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/lct/console-tools-${PV}.tar.gz \
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 2/6] pcmciautils: depend on bison-native

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

pcmciautils needs yacc as well as lex.

Signed-off-by: Christopher Larson 
---
 meta/recipes-bsp/pcmciautils/pcmciautils.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/pcmciautils/pcmciautils.inc 
b/meta/recipes-bsp/pcmciautils/pcmciautils.inc
index 05249805098..26c6d75d4b7 100644
--- a/meta/recipes-bsp/pcmciautils/pcmciautils.inc
+++ b/meta/recipes-bsp/pcmciautils/pcmciautils.inc
@@ -5,7 +5,7 @@ SECTION = "kernel/userland"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"
 
-DEPENDS = "udev sysfsutils flex-native"
+DEPENDS = "udev sysfsutils flex-native bison-native"
 RDEPENDS_${PN} = "udev module-init-tools"
 
 SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/pcmcia/${BP}.tar.xz"
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 1/6] oe.scriptutils.run_editor: ditch the error-prone argument quoting

2018-06-21 Thread Christopher Larson
From: Christopher Larson 

Rather than trying to construct a string by quoting the files in an
error-prone way, parse $EDITOR to pass a list to subprocess rather than
a string.

Signed-off-by: Christopher Larson 
---
 scripts/lib/scriptutils.py | 24 
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/scripts/lib/scriptutils.py b/scripts/lib/scriptutils.py
index 85b1c949bf5..470f76995f0 100644
--- a/scripts/lib/scriptutils.py
+++ b/scripts/lib/scriptutils.py
@@ -15,16 +15,17 @@
 # with this program; if not, write to the Free Software Foundation, Inc.,
 # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
-import sys
-import os
-import logging
-import glob
 import argparse
-import subprocess
-import tempfile
-import shutil
+import glob
+import logging
+import os
 import random
+import shlex
+import shutil
 import string
+import subprocess
+import sys
+import tempfile
 
 def logger_create(name, stream=None):
 logger = logging.getLogger(name)
@@ -214,15 +215,14 @@ def fetch_url(tinfoil, srcuri, srcrev, destdir, logger, 
preserve_tmp=False, mirr
 
 def run_editor(fn, logger=None):
 if isinstance(fn, str):
-params = '"%s"' % fn
+files = [fn]
 else:
-params = ''
-for fnitem in fn:
-params += ' "%s"' % fnitem
+files = fn
 
 editor = os.getenv('VISUAL', os.getenv('EDITOR', 'vi'))
 try:
-return subprocess.check_call('%s %s' % (editor, params), shell=True)
+print(shlex.split(editor) + files)
+return subprocess.check_call(shlex.split(editor) + files)
 except subprocess.CalledProcessError as exc:
 logger.error("Execution of '%s' failed: %s" % (editor, exc))
 return 1
-- 
2.11.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/4] recipes-extended/psmisc: Don't exclude autopoint for RISCV

2018-06-21 Thread Alistair Francis
On Thu, Jun 14, 2018 at 1:15 PM, Alexander Kanavin
 wrote:
> Don't exclude why? Needs to be in recipe comment.

Apparently this is no longer required. I'll remove it.

Alistair

>
> Alex
>
> 2018-06-14 22:45 GMT+03:00 Alistair Francis :
>> Signed-off-by: Alistair Francis 
>> ---
>>  meta/recipes-extended/psmisc/psmisc.inc | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-extended/psmisc/psmisc.inc 
>> b/meta/recipes-extended/psmisc/psmisc.inc
>> index 82ef947709..73b4c56f69 100644
>> --- a/meta/recipes-extended/psmisc/psmisc.inc
>> +++ b/meta/recipes-extended/psmisc/psmisc.inc
>> @@ -19,7 +19,9 @@ inherit autotools gettext
>>  # Upstream has a custom autogen.sh which invokes po/update-potfiles as they
>>  # don't ship a po/POTFILES.in (which is silly).  Without that file gettext
>>  # doesn't believe po/ is a gettext directory and won't generate po/Makefile.
>> -EXTRA_AUTORECONF_remove = "--exclude=autopoint"
>> +AUTORECONF_REMOVE_ARGS = "--exclude=autopoint"
>> +AUTORECONF_REMOVE_ARGS_riscv64 = ""
>> +EXTRA_AUTORECONF_remove = "${AUTORECONF_REMOVE_ARGS}"
>>  do_configure_prepend() {
>>  ( cd ${S} && po/update-potfiles )
>>  }
>> --
>> 2.17.1
>>
>> --
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] systemd: quote SYSTEMD_SERVICE in prerm and postinst

2018-06-21 Thread Damien Riegel
Systemd mount configuration file must have a name that match the mount
point directory they control. So for instance, if a mount file contains

[Mount]
...
Where=/mnt/my-data

The file must be named `mnt-my\x2ddata.mount`, or systemd will refuse to
honour it.

If this config file contains an [Install] section, it will silently fail
because the unit file is not quoted when systemctl is called. To fix
that, quote `${SYSTEMD_SERVICE}` in prerm and postinst hooks.

Signed-off-by: Damien Riegel 
---
 meta/classes/systemd.bbclass | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/classes/systemd.bbclass b/meta/classes/systemd.bbclass
index 1b134322fb..589e73c855 100644
--- a/meta/classes/systemd.bbclass
+++ b/meta/classes/systemd.bbclass
@@ -34,10 +34,10 @@ if type systemctl >/dev/null 2>/dev/null; then
systemctl daemon-reload
fi
 
-   systemctl $OPTS ${SYSTEMD_AUTO_ENABLE} ${SYSTEMD_SERVICE}
+   systemctl $OPTS ${SYSTEMD_AUTO_ENABLE} "${SYSTEMD_SERVICE}"
 
if [ -z "$D" -a "${SYSTEMD_AUTO_ENABLE}" = "enable" ]; then
-   systemctl --no-block restart ${SYSTEMD_SERVICE}
+   systemctl --no-block restart "${SYSTEMD_SERVICE}"
fi
 fi
 }
@@ -51,10 +51,10 @@ fi
 
 if type systemctl >/dev/null 2>/dev/null; then
if [ -z "$D" ]; then
-   systemctl stop ${SYSTEMD_SERVICE}
+   systemctl stop "${SYSTEMD_SERVICE}"
fi
 
-   systemctl $OPTS disable ${SYSTEMD_SERVICE}
+   systemctl $OPTS disable "${SYSTEMD_SERVICE}"
 fi
 }
 
-- 
2.17.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wic: allow bitbake variables in kickstarter files

2018-06-21 Thread Martin Hundebøll

On 2018-03-22 14:44, Rasmus Villemoes wrote:

image_types_wic.bbclass has a mechanism for doing variable substitution
on .wks files by simply letting the input file be called
.wks.in. However, that doesn't allow using variables in files included
via the include directive.

This adds (somewhat naive) support for variable substitution in all
files parsed by wic. The user should add all required variables to
WICVARS to get them exported appropriately.

Signed-off-by: Rasmus Villemoes 


ping³
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2] harfbuzz: upgrade 1.7.5 -> 1.8.1

2018-06-21 Thread Burton, Ross
On 21 June 2018 at 18:48, Andre McCurdy  wrote:
>> +FILES_${PN}-dev += "${libdir}/cmake/harfbuzz/harfbuzz-config.cmake"
>
> Perhaps we should move the cmake.bbclass packaging rule for these
> files into the default packaging rules set by bitbake.conf?

I was considering that too.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [oe-core][PATCH 1/1] postinst-intercepts: do not execute any variant of delay_to_first_boot

2018-06-21 Thread Joe Slater
As of commit 2c5c6e3ff we create multilib variants of intercept
hooks but we did not account for delay_to_first_boot variants.
This was covered up until commit a335e7867, but will now cause
an error.

Signed-off-by: Joe Slater 
---
 meta/lib/oe/package_manager.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
index 20a41d5..9aa5847 100644
--- a/meta/lib/oe/package_manager.py
+++ b/meta/lib/oe/package_manager.py
@@ -382,7 +382,8 @@ class PackageManager(object, metaclass=ABCMeta):
 if script == "postinst_intercept" or not os.access(script_full, 
os.X_OK):
 continue
 
-if script == "delay_to_first_boot":
+# we do not want to run any multilib variant of this
+if script.startswith("delay_to_first_boot"):
 self._postpone_to_first_boot(script_full)
 continue
 
-- 
2.7.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2] harfbuzz: upgrade 1.7.5 -> 1.8.1

2018-06-21 Thread Andre McCurdy
On Thu, Jun 21, 2018 at 1:34 AM, Maxin B. John  wrote:
> Signed-off-by: Maxin B. John 
> ---
>  .../harfbuzz/{harfbuzz_1.7.5.bb => harfbuzz_1.8.1.bb}| 5 
> +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>  rename meta/recipes-graphics/harfbuzz/{harfbuzz_1.7.5.bb => 
> harfbuzz_1.8.1.bb} (84%)
>
> diff --git a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb 
> b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> similarity index 84%
> rename from meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
> rename to meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> index 0a9b029..a42bf8c 100644
> --- a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
> +++ b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> @@ -12,8 +12,8 @@ DEPENDS = "glib-2.0 cairo fontconfig freetype"
>
>  SRC_URI = 
> "http://www.freedesktop.org/software/harfbuzz/release/${BP}.tar.bz2;
>
> -SRC_URI[md5sum] = "1466ab51fc5aaa6af4065936939cec62"
> -SRC_URI[sha256sum] = 
> "84574e1b1f65ca694cb8fb6905309665c0368af18a312357f8ff886ee2f29563"
> +SRC_URI[md5sum] = "97cb89c6e2a914c05c8b7e858703885f"
> +SRC_URI[sha256sum] = 
> "fbed6392ddb085e45e6090a9f389f72926d0e355f4b0a2ef51d35cf21686df45"
>
>  inherit autotools pkgconfig lib_package gtk-doc
>
> @@ -35,5 +35,6 @@ FILES_${PN}-icu-dev = "${libdir}/libharfbuzz-icu.la \
> ${libdir}/libharfbuzz-icu.so \
> ${libdir}/pkgconfig/harfbuzz-icu.pc \
>  "
> +FILES_${PN}-dev += "${libdir}/cmake/harfbuzz/harfbuzz-config.cmake"

Perhaps we should move the cmake.bbclass packaging rule for these
files into the default packaging rules set by bitbake.conf?

>
>  BBCLASSEXTEND = "native"
> --
> 2.4.0
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2] harfbuzz: upgrade 1.7.5 -> 1.8.1

2018-06-21 Thread Burton, Ross
Getting a buildhistory warnings that suggests packaging went wrong:

packages/corei7-64-poky-linux/harfbuzz/harfbuzz-bin: PKG changed from
libharfbuzz-bin to harfbuzz-bin [default] - may indicate debian
renaming failure
  * PV changed from "1.7.5" to "1.8.1"
packages/corei7-64-poky-linux/harfbuzz/harfbuzz: FILELIST: removed
"/usr/lib/libharfbuzz.so.0.10705.0", added
"/usr/lib/libharfbuzz-subset.so.0.10800.1
/usr/lib/libharfbuzz.so.0.10800.1 /usr/lib/libharfbuzz-subset.so.0"

The problem is that harfbuzz-subset is in PN so it doesn't know what
to call the package.  Either pull subset out into a separate package
(which sounds wrong considering the name) or set LEAD_SONAME.

Ross

On 21 June 2018 at 09:34, Maxin B. John  wrote:
> Signed-off-by: Maxin B. John 
> ---
>  .../harfbuzz/{harfbuzz_1.7.5.bb => harfbuzz_1.8.1.bb}| 5 
> +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>  rename meta/recipes-graphics/harfbuzz/{harfbuzz_1.7.5.bb => 
> harfbuzz_1.8.1.bb} (84%)
>
> diff --git a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb 
> b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> similarity index 84%
> rename from meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
> rename to meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> index 0a9b029..a42bf8c 100644
> --- a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
> +++ b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> @@ -12,8 +12,8 @@ DEPENDS = "glib-2.0 cairo fontconfig freetype"
>
>  SRC_URI = 
> "http://www.freedesktop.org/software/harfbuzz/release/${BP}.tar.bz2;
>
> -SRC_URI[md5sum] = "1466ab51fc5aaa6af4065936939cec62"
> -SRC_URI[sha256sum] = 
> "84574e1b1f65ca694cb8fb6905309665c0368af18a312357f8ff886ee2f29563"
> +SRC_URI[md5sum] = "97cb89c6e2a914c05c8b7e858703885f"
> +SRC_URI[sha256sum] = 
> "fbed6392ddb085e45e6090a9f389f72926d0e355f4b0a2ef51d35cf21686df45"
>
>  inherit autotools pkgconfig lib_package gtk-doc
>
> @@ -35,5 +35,6 @@ FILES_${PN}-icu-dev = "${libdir}/libharfbuzz-icu.la \
> ${libdir}/libharfbuzz-icu.so \
> ${libdir}/pkgconfig/harfbuzz-icu.pc \
>  "
> +FILES_${PN}-dev += "${libdir}/cmake/harfbuzz/harfbuzz-config.cmake"
>
>  BBCLASSEXTEND = "native"
> --
> 2.4.0
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] oeqa: add Git mirror test

2018-06-21 Thread Ross Burton
Add an automated test to exercise that fetching a git: recipe...
1) works over git without mirrors
2) fails without git connectivity or mirrors
3) works without git connectivity but with a mirror

(2) is done by setting GIT_PROXY_COMMAND to 'false' which should break any git
network operations.

Signed-off-by: Ross Burton 
---
 meta/lib/oeqa/selftest/cases/fetch.py | 49 +++
 1 file changed, 49 insertions(+)
 create mode 100644 meta/lib/oeqa/selftest/cases/fetch.py

diff --git a/meta/lib/oeqa/selftest/cases/fetch.py 
b/meta/lib/oeqa/selftest/cases/fetch.py
new file mode 100644
index 000..4acc8cdcc8e
--- /dev/null
+++ b/meta/lib/oeqa/selftest/cases/fetch.py
@@ -0,0 +1,49 @@
+import oe.path
+from oeqa.selftest.case import OESelftestTestCase
+from oeqa.utils.commands import bitbake
+from oeqa.core.decorator.oeid import OETestID
+
+class Fetch(OESelftestTestCase):
+@OETestID(1058)
+def test_git_mirrors(self):
+"""
+Verify that the git fetcher will fall back to the HTTP mirrors. The
+recipe needs to be one that we have on the Yocto Project source mirror
+and is hosted in git.
+"""
+
+# TODO: mktempd instead of hardcoding
+dldir = os.path.join(self.builddir, "download-git-mirrors")
+self.track_for_cleanup(dldir)
+
+# No mirrors, should use git to fetch successfully
+features = """
+DL_DIR = "%s"
+MIRRORS_forcevariable = ""
+PREMIRRORS_forcevariable = ""
+""" % dldir
+self.write_config(features)
+oe.path.remove(dldir, recurse=True)
+bitbake("dbus-wait -c fetch -f")
+
+# No mirrors and broken git, should fail
+features = """
+DL_DIR = "%s"
+GIT_PROXY_COMMAND = "false"
+MIRRORS_forcevariable = ""
+PREMIRRORS_forcevariable = ""
+""" % dldir
+self.write_config(features)
+oe.path.remove(dldir, recurse=True)
+with self.assertRaises(AssertionError):
+bitbake("dbus-wait -c fetch -f")
+
+# Broken git but a specific mirror
+features = """
+DL_DIR = "%s"
+GIT_PROXY_COMMAND = "false"
+MIRRORS_forcevariable = "git://.*/.* 
http://downloads.yoctoproject.org/mirror/sources/;
+""" % dldir
+self.write_config(features)
+oe.path.remove(dldir, recurse=True)
+bitbake("dbus-wait -c fetch -f")
-- 
2.11.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] patchtest.sh: optionally specify start branch

2018-06-21 Thread Trevor Woerner
ping?

On Tue, May 29, 2018 at 9:38 AM, Trevor Woerner  wrote:

> Allow the user to optionally specify a starting branch, otherwise assume
> master.
>
> Signed-off-by: Trevor Woerner 
> ---
>  scripts/contrib/patchtest.sh | 19 ++-
>  1 file changed, 14 insertions(+), 5 deletions(-)
>
> diff --git a/scripts/contrib/patchtest.sh b/scripts/contrib/patchtest.sh
> index 7fe56e..2a4aea1d28 100755
> --- a/scripts/contrib/patchtest.sh
> +++ b/scripts/contrib/patchtest.sh
> @@ -2,7 +2,7 @@
>  # ex:ts=4:sw=4:sts=4:et
>  # -*- tab-width: 4; c-basic-offset: 4; indent-tabs-mode: nil -*-
>  #
> -# patchtest: Run patchtest on commits starting at master
> +# patchtest: Run patchtest on commits starting at specified branch, or
> master
>  #
>  # Copyright (c) 2017, Intel Corporation.
>  # All rights reserved.
> @@ -29,8 +29,9 @@ pokydir=''
>  usage() {
>  CMD=$(basename $0)
>  cat < -Usage: $CMD [-h] [-p pokydir]
> -  -p pokydir  Defaults to current directory
> +Usage: $CMD [-h] [-p pokydir] [-b startbranch]
> +  -p pokydirDefaults to current directory
> +  -b startbranchDefaults to master
>  EOM
>  >&2
>  exit 1
> @@ -46,11 +47,14 @@ function clone() {
>  fi
>  }
>
> -while getopts ":p:h" opt; do
> +while getopts ":p:hb:" opt; do
>  case $opt in
> p)
> pokydir=$OPTARG
> ;;
> +   b)
> +   branch=$OPTARG
> +   ;;
> h)
> usage
> ;;
> @@ -73,6 +77,11 @@ if [ -z "$pokydir" ]; then
>  pokydir="$CDIR"
>  fi
>
> +# default branch to master if the user did not specify otherwise
> +if [ -z "$branch" ]; then
> +branch="master"
> +fi
> +
>  PTENV="$PWD/patchtest"
>  PT="$PTENV/patchtest"
>  PTOE="$PTENV/patchtest-oe"
> @@ -99,7 +108,7 @@ pip install -r $PTOE/requirements.txt --quiet
>  PATH="$PT:$PT/scripts:$PATH"
>
>  # loop through parent to HEAD and execute patchtest on each commit
> -for commit in $(git rev-list master..HEAD --reverse)
> +for commit in $(git rev-list ${branch}..HEAD --reverse)
>  do
>  shortlog="$(git log "$commit^1..$commit" --pretty='%h: %aN: %cd: %s')"
>  log="$(git format-patch "$commit^1..$commit" --stdout | patchtest -
> -r $pokydir -s $PTOE/tests --base-commit $commit^1 --json 2>/dev/null |
> create-summary --fail --only-results)"
> --
> 2.17.0.582.gccdcbd54c
>
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8

2018-06-21 Thread Otavio Salvador
On Fri, Jun 15, 2018 at 6:57 AM, Nicolas Dechesne
 wrote:
> On Fri, Jun 15, 2018 at 5:22 AM, Randy Li  wrote:
>> There are some addtional instructions apart from bare armv8,
>> also there is armv8.1, armv8.2.
>>
>> Most the processor would support crc, except X-gene 1.
>>
>> Signed-off-by: Randy Li 
>> ---
>>  meta/conf/machine/include/arm/arch-armv8.inc  |  1 -
>>  meta/conf/machine/include/arm/arch-armv8a.inc | 28 
>> +++
>>  2 files changed, 28 insertions(+), 1 deletion(-)
>>  delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
>
> My previous comment still applies here. You can't just delete this
> file. It's used by many BSP. and it's even used in oe-core itself:
>
> layers/openembedded-core/meta/conf/machine/include/tune-thunderx.inc:require
> conf/machine/include/arm/arch-armv8.inc
> layers/openembedded-core/meta/conf/machine/qemuarm64.conf:require
> conf/machine/include/arm/arch-armv8.inc

I think the rename is fine (of course, considering it handles the
rename inside whole oe-core).

Breaking existing BSP in new release is fine. People needs to handle
it when upgrading the branches.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH V2 1/1] linux-yocto: add ptest support

2018-06-21 Thread Bruce Ashfield
My general comment still stands from v1, That I don't see a lot of
value in creating a
whole set of custom tests and wrappers just for ptest use.

i.e. ltp already has a lot of these covered. kprobes, as do the
built-in self tests.

I want to be sure that we aren't creating custom wrappers that will
need to be maintained
for a long time.

Comments/documentation in the script don't exist, so it is difficult
to look at this and
understand what it is actually trying to test. A header block and
comments before each
test are required.

On Wed, Jun 13, 2018 at 10:49 PM, Dengke Du  wrote:
>
> Signed-off-by: Dengke Du 
> ---
>  meta/recipes-kernel/linux/linux-yocto/run-ptest | 138 
> 
>  meta/recipes-kernel/linux/linux-yocto_4.14.bb   |   9 ++
>  2 files changed, 147 insertions(+)
>  create mode 100644 meta/recipes-kernel/linux/linux-yocto/run-ptest
>
> diff --git a/meta/recipes-kernel/linux/linux-yocto/run-ptest 
> b/meta/recipes-kernel/linux/linux-yocto/run-ptest
> new file mode 100644
> index 000..6db4d93
> --- /dev/null
> +++ b/meta/recipes-kernel/linux/linux-yocto/run-ptest
> @@ -0,0 +1,138 @@
> +#!/bin/bash
> +depmod
> +touch kernel.log
> +
> +#dma-example bytestream-example inttype-example record-example
> +list1=("dma-example" "bytestream-example" "inttype-example" "record-example")
> +for i in "${list1[@]}"
> +do
> +  dmesg -c
> +  modprobe "$i"

All of the modprobes really should be checking for failure, I realize you are
doing some lsmod calls and checking those, but that isn't always the case
in this script.

Cheers,

Bruce

> +  result=""
> +  IFS="-" read -ra array <<< "$i"
> +  len=${#array[@]}
> +  if [ $len -eq 2 ];then
> +result="${array[0]}_${array[1]}"
> +  elif [ $len -eq 3 ];then
> +result="${array[0]}_${array[1]}_${array[2]}"
> +  fi
> +  lsmod | grep -q "$result"
> +  if [ $? -eq 0 ];then
> +dmesg | grep "test passed"
> +if [ $? -eq 0 ];then
> +  echo "$i: PASS" >> kernel.log
> +fi
> +  else
> +echo "$i: FAILED" >> kernel.log
> +  fi
> +  rmmod "$i"
> +done
> +
> +#kobject-example kset-example
> +list2=("kobject-example" "kset-example")
> +for i in "${list2[@]}"
> +do
> +  dmesg -c
> +  modprobe "$i"
> +  result=""
> +  IFS="-" read -ra array <<< "$i"
> +  len=${#array[@]}
> +  if [ $len -eq 2 ];then
> +result="${array[0]}_${array[1]}"
> +  elif [ $len -eq 3 ];then
> +result="${array[0]}_${array[1]}_${array[2]}"
> +  fi
> +  basedir="/sys/kernel/${result}"
> +  echo "$basedir"
> +  if [ -e ${basedir}/bar -a -e ${basedir}/baz -a -e ${basedir}/foo ];then
> +echo "$i: PASS" >> kernel.log
> +  else
> +echo "$i: FAILED" >> kernel.log
> +  fi
> +  rmmod "$i"
> +done
> +
> +#trace-events-sample
> +list3="trace-events-sample"
> +result=""
> +IFS="-" read -ra array <<< "$list3"
> +len=${#array[@]}
> +if [ $len -eq 2 ];then
> +  result="${array[0]}_${array[1]}"
> +elif [ $len -eq 3 ];then
> +  result="${array[0]}_${array[1]}_${array[2]}"
> +fi
> +modprobe "$list3"
> +lsmod | grep "$result"
> +if [ $? -eq 0 ];then
> +  if [ -e "/sys/kernel/debug/tracing/events/sample-trace" ];then
> +echo 1 > /sys/kernel/debug/tracing/events/sample-trace/enable
> +sleep 5
> +ret=`cat /sys/kernel/debug/tracing/trace | grep hello | head -n1 | cut 
> -d':' -f2`
> +if [ "$ret" = " foo_bar" ];then
> +  echo "$list3: PASS"  >> kernel.log
> +else
> +  echo "$list3: FAILED-" >> kernel.log
> +fi
> +  else
> +echo "$list3: FAILED--" >> kernel.log
> +  fi
> +else
> +  echo "$list3: FAILED---" >> kernel.log
> +fi
> +rmmod "$list3"
> +
> +#trace-printk
> +list4="trace-printk"
> +modprobe "$list4"
> +lsmod | grep "trace_printk"
> +if [ $? -eq 0 ];then
> +  ret=`cat /sys/kernel/debug/tracing/trace | grep trace_printk | head -n1 | 
> cut -d':' -f2`
> +  if [ "$ret" = " trace_printk_irq_work" ];then
> +echo "$list4: PASS" >> kernel.log
> +rmmod "$list4"
> +  else
> +echo "$list4: FAILED" >> kernel.log
> +  fi
> +else
> +  echo "$list4: FAILED" >> kernel.log
> +fi
> +rmmod "$list4"
> +
> +#kprobe_example
> +list5="kprobe_example"
> +dmesg -c
> +modprobe "$list5"
> +lsmod | grep "$list5"
> +if [ $? -eq 0 ];then
> +  dmesg | grep "_do_fork"
> +  if [ $? -eq 0 ];then
> +echo "$list5: PASS" >> kernel.log
> +  else
> +echo "$list5: FAILED" >> kernel.log
> +  fi
> +else
> +  echo "$list5: FAILED" >> kernel.log
> +fi
> +rmmod "$list5"
> +
> +#kretprobe_example
> +list6="kretprobe_example"
> +dmesg -c
> +modprobe "$list6"
> +lsmod | grep "$list6"
> +if [ $? -eq 0 ];then
> +  dmesg | grep "_do_fork returned"
> +  if [ $? -eq 0 ];then
> +echo "$list6: PASS" >> kernel.log
> +  else
> +echo "$list6: FAILED" >> kernel.log
> +  fi
> +else
> +  echo "$list6: FAILED" >> kernel.log
> +fi
> +rmmod "$list6"
> +
> +echo "#result#"
> +cat kernel.log
> +rm kernel.log
> diff --git a/meta/recipes-kernel/linux/linux-yocto_4.14.bb 
> b/meta/recipes-kernel/linux/linux-yocto_4.14.bb
> index 0449213..9650ee2 100644
> --- 

Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-06-21 Thread Martin Jansa
It might be DEBUG_BUILD enabled in my builds, let me check.

On Thu, Jun 21, 2018 at 1:20 PM Burton, Ross  wrote:

> Master builds fine for me, but interestingly I don't see that in my
> mtd-utils unpack.  Do you have patches?
>
> Ross
>
> On 21 June 2018 at 12:10, Martin Jansa  wrote:
> > Does this upgrade fix the build failure with gcc8?
> >
> > With current master I'm seeing:
> > super0.c:238:32: error: 'snprintf' output may be truncated before the
> last
> > format character [-Werror=format-truncation=]
> >
> > On Thu, Jun 21, 2018 at 12:33 PM Burton, Ross 
> wrote:
> >>
> >> Sorry, no it doesn't.  I've just picked this in ross/mut but a
> >> follow-up patch to remove that mystery patch (can't see it being used
> >> in image generation anywhere) would be good.
> >>
> >> Ross
> >>
> >> On 20 June 2018 at 00:53, Denys Dmytriyenko  wrote:
> >> > On Tue, May 22, 2018 at 01:16:34PM +0100, Burton, Ross wrote:
> >> >> On 22 May 2018 at 08:49, Andrea Adami 
> wrote:
> >> >> > I am just wondering about this patch:we carry it since 2009  [1]
> and
> >> >> > is still marked as pending.
> >> >> > Why was it necessary to ignore part of the filesystem? Can you
> >> >> > remember?
> >> >>
> >> >> Make that at least 2004.  This commit in oe-classic touches it:
> >> >>
> >> >> commit c70605ebdca81412a93b4d867de7931543193f26
> >> >> Author: Chris Larson 
> >> >> Date:   Thu Dec 23 21:24:57 2004 +
> >> >>
> >> >> Update mtd to a current snapshot, change the mkfs.jffs2 -i
> cmdline
> >> >> option to
> >> >> -I, and rename mtd and mtd-native to mtd-utils and
> >> >> mtd-utils-native,
> >> >> respectively.
> >> >>
> >> >> Prior history is lost to the git/bk migration unless anyone has a
> >> >> BitKeeper clone to mine...
> >> >
> >> > So, does this prevent the version upgrade from being merged?
> >> >
> >> > --
> >> > Denys
> >> --
> >> ___
> >> Openembedded-core mailing list
> >> Openembedded-core@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-06-21 Thread Burton, Ross
Master builds fine for me, but interestingly I don't see that in my
mtd-utils unpack.  Do you have patches?

Ross

On 21 June 2018 at 12:10, Martin Jansa  wrote:
> Does this upgrade fix the build failure with gcc8?
>
> With current master I'm seeing:
> super0.c:238:32: error: 'snprintf' output may be truncated before the last
> format character [-Werror=format-truncation=]
>
> On Thu, Jun 21, 2018 at 12:33 PM Burton, Ross  wrote:
>>
>> Sorry, no it doesn't.  I've just picked this in ross/mut but a
>> follow-up patch to remove that mystery patch (can't see it being used
>> in image generation anywhere) would be good.
>>
>> Ross
>>
>> On 20 June 2018 at 00:53, Denys Dmytriyenko  wrote:
>> > On Tue, May 22, 2018 at 01:16:34PM +0100, Burton, Ross wrote:
>> >> On 22 May 2018 at 08:49, Andrea Adami  wrote:
>> >> > I am just wondering about this patch:we carry it since 2009  [1] and
>> >> > is still marked as pending.
>> >> > Why was it necessary to ignore part of the filesystem? Can you
>> >> > remember?
>> >>
>> >> Make that at least 2004.  This commit in oe-classic touches it:
>> >>
>> >> commit c70605ebdca81412a93b4d867de7931543193f26
>> >> Author: Chris Larson 
>> >> Date:   Thu Dec 23 21:24:57 2004 +
>> >>
>> >> Update mtd to a current snapshot, change the mkfs.jffs2 -i cmdline
>> >> option to
>> >> -I, and rename mtd and mtd-native to mtd-utils and
>> >> mtd-utils-native,
>> >> respectively.
>> >>
>> >> Prior history is lost to the git/bk migration unless anyone has a
>> >> BitKeeper clone to mine...
>> >
>> > So, does this prevent the version upgrade from being merged?
>> >
>> > --
>> > Denys
>> --
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-06-21 Thread Martin Jansa
Does this upgrade fix the build failure with gcc8?

With current master I'm seeing:
super0.c:238:32: error: 'snprintf' output may be truncated before the last
format character [-Werror=format-truncation=]

On Thu, Jun 21, 2018 at 12:33 PM Burton, Ross  wrote:

> Sorry, no it doesn't.  I've just picked this in ross/mut but a
> follow-up patch to remove that mystery patch (can't see it being used
> in image generation anywhere) would be good.
>
> Ross
>
> On 20 June 2018 at 00:53, Denys Dmytriyenko  wrote:
> > On Tue, May 22, 2018 at 01:16:34PM +0100, Burton, Ross wrote:
> >> On 22 May 2018 at 08:49, Andrea Adami  wrote:
> >> > I am just wondering about this patch:we carry it since 2009  [1] and
> >> > is still marked as pending.
> >> > Why was it necessary to ignore part of the filesystem? Can you
> remember?
> >>
> >> Make that at least 2004.  This commit in oe-classic touches it:
> >>
> >> commit c70605ebdca81412a93b4d867de7931543193f26
> >> Author: Chris Larson 
> >> Date:   Thu Dec 23 21:24:57 2004 +
> >>
> >> Update mtd to a current snapshot, change the mkfs.jffs2 -i cmdline
> option to
> >> -I, and rename mtd and mtd-native to mtd-utils and mtd-utils-native,
> >> respectively.
> >>
> >> Prior history is lost to the git/bk migration unless anyone has a
> >> BitKeeper clone to mine...
> >
> > So, does this prevent the version upgrade from being merged?
> >
> > --
> > Denys
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] cryptodev: Fix build errors with v4.17+

2018-06-21 Thread He Zhe
Backport from upstream to update internal syscall function usage.
https://github.com/cryptodev-linux/cryptodev-linux
f60aa08c63fc02780554a0a12180a478ca27d49f

Signed-off-by: He Zhe 
---
 .../cryptodev/cryptodev-module_1.9.bb  |  1 +
 .../0001-ioctl.c-Fix-build-with-linux-4.17.patch   | 43 ++
 2 files changed, 44 insertions(+)
 create mode 100644 
meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.17.patch

diff --git a/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb 
b/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
index ed6d0ecae97..6052650c955 100644
--- a/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
+++ b/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
@@ -10,6 +10,7 @@ DEPENDS += "cryptodev-linux"
 SRC_URI += " \
 file://0001-Disable-installing-header-file-provided-by-another-p.patch \
 file://0001-ioctl.c-Fix-build-with-linux-4.13.patch \
+file://0001-ioctl.c-Fix-build-with-linux-4.17.patch \
 "
 
 EXTRA_OEMAKE='KERNEL_DIR="${STAGING_KERNEL_DIR}" PREFIX="${D}"'
diff --git 
a/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.17.patch
 
b/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.17.patch
new file mode 100644
index 000..5881d1c4eec
--- /dev/null
+++ 
b/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.17.patch
@@ -0,0 +1,43 @@
+From f60aa08c63fc02780554a0a12180a478ca27d49f Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Horia=20Geant=C4=83?= 
+Date: Wed, 23 May 2018 18:43:39 +0300
+Subject: [PATCH] ioctl.c: Fix build with linux 4.17
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Since kernel 4.17-rc1, sys_* syscalls can no longer be called directly:
+819671ff849b ("syscalls: define and explain goal to not call syscalls in the 
kernel")
+
+Since cryptodev uses sys_close() - and this has been removed in commit:
+2ca2a09d6215 ("fs: add ksys_close() wrapper; remove in-kernel calls to 
sys_close()")
+cryptodev has to be updated to use the ksys_close() wrapper.
+
+Signed-off-by: Horia Geantă 
+
+Upstream-Status: Backport
+
+Signed-off-by: He Zhe 
+---
+ ioctl.c | 4 
+ 1 file changed, 4 insertions(+)
+
+diff --git a/ioctl.c b/ioctl.c
+index d831b0c..2571034 100644
+--- a/ioctl.c
 b/ioctl.c
+@@ -828,7 +828,11 @@ cryptodev_ioctl(struct file *filp, unsigned int cmd, 
unsigned long arg_)
+   fd = clonefd(filp);
+   ret = put_user(fd, p);
+   if (unlikely(ret)) {
++#if (LINUX_VERSION_CODE < KERNEL_VERSION(4, 17, 0))
+   sys_close(fd);
++#else
++  ksys_close(fd);
++#endif
+   return ret;
+   }
+   return ret;
+-- 
+2.7.4
+
-- 
2.11.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-06-21 Thread Burton, Ross
Sorry, no it doesn't.  I've just picked this in ross/mut but a
follow-up patch to remove that mystery patch (can't see it being used
in image generation anywhere) would be good.

Ross

On 20 June 2018 at 00:53, Denys Dmytriyenko  wrote:
> On Tue, May 22, 2018 at 01:16:34PM +0100, Burton, Ross wrote:
>> On 22 May 2018 at 08:49, Andrea Adami  wrote:
>> > I am just wondering about this patch:we carry it since 2009  [1] and
>> > is still marked as pending.
>> > Why was it necessary to ignore part of the filesystem? Can you remember?
>>
>> Make that at least 2004.  This commit in oe-classic touches it:
>>
>> commit c70605ebdca81412a93b4d867de7931543193f26
>> Author: Chris Larson 
>> Date:   Thu Dec 23 21:24:57 2004 +
>>
>> Update mtd to a current snapshot, change the mkfs.jffs2 -i cmdline 
>> option to
>> -I, and rename mtd and mtd-native to mtd-utils and mtd-utils-native,
>> respectively.
>>
>> Prior history is lost to the git/bk migration unless anyone has a
>> BitKeeper clone to mine...
>
> So, does this prevent the version upgrade from being merged?
>
> --
> Denys
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2] piglit: upgrade to latest revision

2018-06-21 Thread Maxin B. John
Signed-off-by: Maxin B. John 
---
 meta/recipes-graphics/piglit/piglit_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/piglit/piglit_git.bb 
b/meta/recipes-graphics/piglit/piglit_git.bb
index b936ad8..c186ca9 100644
--- a/meta/recipes-graphics/piglit/piglit_git.bb
+++ b/meta/recipes-graphics/piglit/piglit_git.bb
@@ -8,8 +8,8 @@ SRC_URI = "git://anongit.freedesktop.org/piglit \
"
 UPSTREAM_CHECK_COMMITS = "1"
 
-# From 2018-02-26
-SRCREV = "4ce0887e2f7f848d2be2e435a2d0f3c80e44ea3b"
+# From 2018-06-18
+SRCREV = "431bcc74ff54521ed1c498af9b13e629f949bda6"
 # (when PV goes above 1.0 remove the trailing r)
 PV = "1.0+gitr${SRCPV}"
 
-- 
2.4.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/5] piglit: upgrade to latest revision

2018-06-21 Thread Maxin B. John
Hi Khem,

On Wed, Jun 20, 2018 at 10:19:52AM -0700, Khem Raj wrote:
> On Wed, Jun 20, 2018 at 7:07 AM Maxin B. John  wrote:
> >
> > Signed-off-by: Maxin B. John 
> > ---
> >  meta/recipes-graphics/piglit/piglit_git.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-graphics/piglit/piglit_git.bb 
> > b/meta/recipes-graphics/piglit/piglit_git.bb
> > index b936ad8..1a3e888 100644
> > --- a/meta/recipes-graphics/piglit/piglit_git.bb
> > +++ b/meta/recipes-graphics/piglit/piglit_git.bb
> > @@ -9,7 +9,7 @@ SRC_URI = "git://anongit.freedesktop.org/piglit \
> >  UPSTREAM_CHECK_COMMITS = "1"
> >
> >  # From 2018-02-26
> > -SRCREV = "4ce0887e2f7f848d2be2e435a2d0f3c80e44ea3b"
> > +SRCREV = "3aa8c1cfe1a9ddae40c60d9dc76fe8dce47b15cc"
> 
> Can you update the comment about the date as well

Will do that. Thanks !

> >  # (when PV goes above 1.0 remove the trailing r)
> >  PV = "1.0+gitr${SRCPV}"
> >
> > --
> > 2.4.0
> >

Best Regards,
Maxin
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2] harfbuzz: upgrade 1.7.5 -> 1.8.1

2018-06-21 Thread Maxin B. John
Signed-off-by: Maxin B. John 
---
 .../harfbuzz/{harfbuzz_1.7.5.bb => harfbuzz_1.8.1.bb}| 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
 rename meta/recipes-graphics/harfbuzz/{harfbuzz_1.7.5.bb => harfbuzz_1.8.1.bb} 
(84%)

diff --git a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb 
b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
similarity index 84%
rename from meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
rename to meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
index 0a9b029..a42bf8c 100644
--- a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
+++ b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
@@ -12,8 +12,8 @@ DEPENDS = "glib-2.0 cairo fontconfig freetype"
 
 SRC_URI = "http://www.freedesktop.org/software/harfbuzz/release/${BP}.tar.bz2;
 
-SRC_URI[md5sum] = "1466ab51fc5aaa6af4065936939cec62"
-SRC_URI[sha256sum] = 
"84574e1b1f65ca694cb8fb6905309665c0368af18a312357f8ff886ee2f29563"
+SRC_URI[md5sum] = "97cb89c6e2a914c05c8b7e858703885f"
+SRC_URI[sha256sum] = 
"fbed6392ddb085e45e6090a9f389f72926d0e355f4b0a2ef51d35cf21686df45"
 
 inherit autotools pkgconfig lib_package gtk-doc
 
@@ -35,5 +35,6 @@ FILES_${PN}-icu-dev = "${libdir}/libharfbuzz-icu.la \
${libdir}/libharfbuzz-icu.so \
${libdir}/pkgconfig/harfbuzz-icu.pc \
 "
+FILES_${PN}-dev += "${libdir}/cmake/harfbuzz/harfbuzz-config.cmake"
 
 BBCLASSEXTEND = "native"
-- 
2.4.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 5/5] harfbuzz: upgrade 1.7.5 -> 1.8.1

2018-06-21 Thread Maxin B. John
Hi Khem, Ross,

On Wed, Jun 20, 2018 at 07:07:58PM +0100, Burton, Ross wrote:
> Yes, those should be in the -dev package.

I will update it and send a v2 soon. 

> Ross

Best Regards,
Maxin

> On 20 June 2018 at 18:18, Khem Raj  wrote:
> > On Wed, Jun 20, 2018 at 7:07 AM Maxin B. John  wrote:
> >>
> >> Signed-off-by: Maxin B. John 
> >> ---
> >>  .../harfbuzz/{harfbuzz_1.7.5.bb => harfbuzz_1.8.1.bb} | 8 
> >> ++--
> >>  1 file changed, 6 insertions(+), 2 deletions(-)
> >>  rename meta/recipes-graphics/harfbuzz/{harfbuzz_1.7.5.bb => 
> >> harfbuzz_1.8.1.bb} (82%)
> >>
> >> diff --git a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb 
> >> b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> >> similarity index 82%
> >> rename from meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
> >> rename to meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> >> index 0a9b029..f8b0056 100644
> >> --- a/meta/recipes-graphics/harfbuzz/harfbuzz_1.7.5.bb
> >> +++ b/meta/recipes-graphics/harfbuzz/harfbuzz_1.8.1.bb
> >> @@ -12,8 +12,8 @@ DEPENDS = "glib-2.0 cairo fontconfig freetype"
> >>
> >>  SRC_URI = 
> >> "http://www.freedesktop.org/software/harfbuzz/release/${BP}.tar.bz2;
> >>
> >> -SRC_URI[md5sum] = "1466ab51fc5aaa6af4065936939cec62"
> >> -SRC_URI[sha256sum] = 
> >> "84574e1b1f65ca694cb8fb6905309665c0368af18a312357f8ff886ee2f29563"
> >> +SRC_URI[md5sum] = "97cb89c6e2a914c05c8b7e858703885f"
> >> +SRC_URI[sha256sum] = 
> >> "fbed6392ddb085e45e6090a9f389f72926d0e355f4b0a2ef51d35cf21686df45"
> >>
> >>  inherit autotools pkgconfig lib_package gtk-doc
> >>
> >> @@ -27,6 +27,10 @@ EXTRA_OECONF = " \
> >>  --with-glib \
> >>  --without-graphite2 \
> >>  "
> >> +do_install_append() {
> >> +# Remove unnecessary cmake files
> >> +rm -rf ${D}${libdir}/cmake
> >> +}
> >
> > perhaps they should go into -dev package instead.
> >
> >>
> >>  PACKAGES =+ "${PN}-icu ${PN}-icu-dev"
> >>
> >> --
> >> 2.4.0
> >>
> >> --
> >> ___
> >> Openembedded-core mailing list
> >> Openembedded-core@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> > --
> > ___
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH V2 1/1] linux-yocto: add ptest support

2018-06-21 Thread lei yang



On 2018年06月14日 10:49, Dengke Du wrote:

Signed-off-by: Dengke Du 
---
  meta/recipes-kernel/linux/linux-yocto/run-ptest | 138 
  meta/recipes-kernel/linux/linux-yocto_4.14.bb   |   9 ++
  2 files changed, 147 insertions(+)
  create mode 100644 meta/recipes-kernel/linux/linux-yocto/run-ptest

diff --git a/meta/recipes-kernel/linux/linux-yocto/run-ptest 
b/meta/recipes-kernel/linux/linux-yocto/run-ptest
new file mode 100644
index 000..6db4d93
--- /dev/null
+++ b/meta/recipes-kernel/linux/linux-yocto/run-ptest
@@ -0,0 +1,138 @@
+#!/bin/bash
+depmod
+touch kernel.log
+
+#dma-example bytestream-example inttype-example record-example
+list1=("dma-example" "bytestream-example" "inttype-example" "record-example")
+for i in "${list1[@]}"
+do
+  dmesg -c
+  modprobe "$i"
+  result=""
+  IFS="-" read -ra array <<< "$i"
+  len=${#array[@]}
+  if [ $len -eq 2 ];then
+result="${array[0]}_${array[1]}"
+  elif [ $len -eq 3 ];then
+result="${array[0]}_${array[1]}_${array[2]}"
+  fi


if you want to substitute "-" to "_" here, using bash pattern 
substitution will be much easier

# i="xx-yy-zz-ff"
# echo ${i//-/_}
xx_yy_zz_ff




+  lsmod | grep -q "$result"
+  if [ $? -eq 0 ];then
+dmesg | grep "test passed"
+if [ $? -eq 0 ];then
+  echo "$i: PASS" >> kernel.log
+fi
+  else
+echo "$i: FAILED" >> kernel.log
+  fi


most ptest (but not all) result format are following the same rule

"[PASS|SKIP|FAIL]" : testname

look at here 
https://www.yoctoproject.org/docs/2.5/dev-manual/dev-manual.html#testing-packages-with-ptest


PS: unified test output matters for LAVA testparse patten.



+  rmmod "$i"
+done
+
+#kobject-example kset-example
+list2=("kobject-example" "kset-example")
+for i in "${list2[@]}"
+do
+  dmesg -c
+  modprobe "$i"
+  result=""
+  IFS="-" read -ra array <<< "$i"
+  len=${#array[@]}
+  if [ $len -eq 2 ];then
+result="${array[0]}_${array[1]}"
+  elif [ $len -eq 3 ];then
+result="${array[0]}_${array[1]}_${array[2]}"
+  fi
+  basedir="/sys/kernel/${result}"
+  echo "$basedir"
+  if [ -e ${basedir}/bar -a -e ${basedir}/baz -a -e ${basedir}/foo ];then
+echo "$i: PASS" >> kernel.log
+  else
+echo "$i: FAILED" >> kernel.log
+  fi
+  rmmod "$i"
+done
+
+#trace-events-sample
+list3="trace-events-sample"
+result=""
+IFS="-" read -ra array <<< "$list3"
+len=${#array[@]}
+if [ $len -eq 2 ];then
+  result="${array[0]}_${array[1]}"
+elif [ $len -eq 3 ];then
+  result="${array[0]}_${array[1]}_${array[2]}"
+fi
+modprobe "$list3"
+lsmod | grep "$result"
+if [ $? -eq 0 ];then
+  if [ -e "/sys/kernel/debug/tracing/events/sample-trace" ];then
+echo 1 > /sys/kernel/debug/tracing/events/sample-trace/enable
+sleep 5
+ret=`cat /sys/kernel/debug/tracing/trace | grep hello | head -n1 | cut 
-d':' -f2`
+if [ "$ret" = " foo_bar" ];then
+  echo "$list3: PASS"  >> kernel.log
+else
+  echo "$list3: FAILED-" >> kernel.log
+fi
+  else
+echo "$list3: FAILED--" >> kernel.log
+  fi
+else
+  echo "$list3: FAILED---" >> kernel.log
+fi
+rmmod "$list3"
+
+#trace-printk
+list4="trace-printk"
+modprobe "$list4"
+lsmod | grep "trace_printk"
+if [ $? -eq 0 ];then
+  ret=`cat /sys/kernel/debug/tracing/trace | grep trace_printk | head -n1 | 
cut -d':' -f2`
+  if [ "$ret" = " trace_printk_irq_work" ];then
+echo "$list4: PASS" >> kernel.log
+rmmod "$list4"
+  else
+echo "$list4: FAILED" >> kernel.log
+  fi
+else
+  echo "$list4: FAILED" >> kernel.log
+fi
+rmmod "$list4"
+
+#kprobe_example
+list5="kprobe_example"
+dmesg -c
+modprobe "$list5"
+lsmod | grep "$list5"
+if [ $? -eq 0 ];then
+  dmesg | grep "_do_fork"
+  if [ $? -eq 0 ];then
+echo "$list5: PASS" >> kernel.log
+  else
+echo "$list5: FAILED" >> kernel.log
+  fi
+else
+  echo "$list5: FAILED" >> kernel.log
+fi
+rmmod "$list5"
+
+#kretprobe_example
+list6="kretprobe_example"
+dmesg -c
+modprobe "$list6"
+lsmod | grep "$list6"
+if [ $? -eq 0 ];then
+  dmesg | grep "_do_fork returned"
+  if [ $? -eq 0 ];then
+echo "$list6: PASS" >> kernel.log
+  else
+echo "$list6: FAILED" >> kernel.log
+  fi
+else
+  echo "$list6: FAILED" >> kernel.log
+fi
+rmmod "$list6"
+
+echo "#result#"
+cat kernel.log
+rm kernel.log
diff --git a/meta/recipes-kernel/linux/linux-yocto_4.14.bb 
b/meta/recipes-kernel/linux/linux-yocto_4.14.bb
index 0449213..9650ee2 100644
--- a/meta/recipes-kernel/linux/linux-yocto_4.14.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_4.14.bb


only enable ptest for linux-yocto_4.14 is not good.

we can still have one copy of run-ptest for different kernel version, 
but we need to make it works for different kernel.


To be clear, for example, linux-yocto-4.14, you might have 10 test 
cases, while in linux-yocto-4.15, you might have 12 test cases


so run-ptest might be like

a) run whatever test case we have, and the first thing is the check 
whether the test case exists,  for example, if directory 
"samples/trace_printk" exists, run it, if not , do nothing