[OE-core] [PATCH] musl: Fix riscv64 CAS functions

2019-09-24 Thread Khem Raj
Signed-off-by: Khem Raj 
---
 ...and-specifiers-in-the-riscv64-CAS-ro.patch | 55 +++
 meta/recipes-core/musl/musl_git.bb|  1 +
 2 files changed, 56 insertions(+)
 create mode 100644 
meta/recipes-core/musl/musl/0001-correct-the-operand-specifiers-in-the-riscv64-CAS-ro.patch

diff --git 
a/meta/recipes-core/musl/musl/0001-correct-the-operand-specifiers-in-the-riscv64-CAS-ro.patch
 
b/meta/recipes-core/musl/musl/0001-correct-the-operand-specifiers-in-the-riscv64-CAS-ro.patch
new file mode 100644
index 00..09b71cb299
--- /dev/null
+++ 
b/meta/recipes-core/musl/musl/0001-correct-the-operand-specifiers-in-the-riscv64-CAS-ro.patch
@@ -0,0 +1,55 @@
+From 59f2954fcaacd9426827c69a729e2647cb9977e5 Mon Sep 17 00:00:00 2001
+From: Palmer Dabbelt 
+Date: Tue, 24 Sep 2019 20:30:15 -0700
+Subject: [PATCH] correct the operand specifiers in the riscv64 CAS routines
+
+The operand sepcifiers in a_cas and a_casp for riscv64 were incorrect:
+there's a backwards branch in the routine, so despite tmp being written
+at the end of the assembly fragment it cannot be allocated in one of the
+input registers because the input values may be needed for another trip
+around the loop.
+
+For code that follows the guarnteed forward progress requirements, he
+backwards branch is rarely taken: SiFive's hardware only fails a store
+conditional on execptional cases (ie, instruction cache misses inside
+the loop), and until recently a bug in QEMU allowed back-to-back
+store conditionals to succeed.  The bug has been fixed in the latest
+QEMU release, but it turns out that the fix caused this latent bug in
+musl to manifest.
+
+Full disclosure: I haven't actually even compiled musl.  I just guessed
+this would fix a bug introducted by the new QEMU behavior, Alistair
+(CC'd) actually checked it fixes the problem.  The rest is just
+conjecture.
+
+Upstream-Status: Submitted
+Signed-off-by: Khem Raj 
+---
+ arch/riscv64/atomic_arch.h | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/arch/riscv64/atomic_arch.h b/arch/riscv64/atomic_arch.h
+index c9765342..41ad4d04 100644
+--- a/arch/riscv64/atomic_arch.h
 b/arch/riscv64/atomic_arch.h
+@@ -14,7 +14,7 @@ static inline int a_cas(volatile int *p, int t, int s)
+   "   sc.w.aqrl %1, %4, (%2)\n"
+   "   bnez %1, 1b\n"
+   "1:"
+-  : "=&r"(old), "=r"(tmp)
++  : "=&r"(old), "=&r"(tmp)
+   : "r"(p), "r"(t), "r"(s)
+   : "memory");
+   return old;
+@@ -31,7 +31,7 @@ static inline void *a_cas_p(volatile void *p, void *t, void 
*s)
+   "   sc.d.aqrl %1, %4, (%2)\n"
+   "   bnez %1, 1b\n"
+   "1:"
+-  : "=&r"(old), "=r"(tmp)
++  : "=&r"(old), "=&r"(tmp)
+   : "r"(p), "r"(t), "r"(s)
+   : "memory");
+   return old;
+-- 
+2.23.0
+
diff --git a/meta/recipes-core/musl/musl_git.bb 
b/meta/recipes-core/musl/musl_git.bb
index 87453be07f..335d53d42f 100644
--- a/meta/recipes-core/musl/musl_git.bb
+++ b/meta/recipes-core/musl/musl_git.bb
@@ -15,6 +15,7 @@ PV = "${BASEVER}+git${SRCPV}"
 SRC_URI = "git://git.musl-libc.org/musl \
file://0001-Make-dynamic-linker-a-relative-symlink-to-libc.patch \

file://0002-ldso-Use-syslibdir-and-libdir-as-default-pathes-to-l.patch \
+   
file://0001-correct-the-operand-specifiers-in-the-riscv64-CAS-ro.patch \
   "
 
 S = "${WORKDIR}/git"
-- 
2.23.0

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


[OE-core] [PATCH v2 1/1] cases/bbtests.py: test_bitbake_g(): Check base-files rather than busybox

2019-09-24 Thread Robert Yang
It checked whether busybox is in task-depends.dot after run "bitbake -g
core-image-minimal", but busybox is not a must for core-image-minimal since it
is configurable:

VIRTUAL-RUNTIME_base-utils = ""
VIRTUAL-RUNTIME_base-utils-syslog = ""
VIRTUAL-RUNTIME_base-utils-hwclock  =  ""
VIRTUAL-RUNTIME_login_manager = "shadow"
VIRTUAL-RUNTIME_syslog = ""

So the case may fail when busybox is not present, check
base-files which is more reliable.

Signed-off-by: Robert Yang 
---
 meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py 
b/meta/lib/oeqa/selftest/cases/bbtests.py
index 8e59baf..9461c7e 100644
--- a/meta/lib/oeqa/selftest/cases/bbtests.py
+++ b/meta/lib/oeqa/selftest/cases/bbtests.py
@@ -118,11 +118,12 @@ class BitbakeTests(OESelftestTestCase):
 self.assertIn(task, result.output, msg="Couldn't find %s task.")
 
 def test_bitbake_g(self):
-result = bitbake('-g core-image-minimal')
+recipe = 'base-files'
+result = bitbake('-g %s' % recipe)
 for f in ['pn-buildlist', 'task-depends.dot']:
 self.addCleanup(os.remove, f)
 self.assertTrue('Task dependencies saved to \'task-depends.dot\'' in 
result.output, msg = "No task dependency \"task-depends.dot\" file was 
generated for the given task target. bitbake output: %s" % result.output)
-self.assertTrue('busybox' in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), msg = "No 
\"busybox\" dependency found in task-depends.dot file.")
+self.assertTrue(recipe in ftools.read_file(os.path.join(self.builddir, 
'task-depends.dot')), msg = "No \"%s\" dependency found in task-depends.dot 
file." % recipe)
 
 def test_image_manifest(self):
 bitbake('core-image-minimal')
-- 
2.7.4

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


[OE-core] [PATCH v2 0/1] cases/bbtests.py: test_bitbake_g(): Check base-files rather than busybox

2019-09-24 Thread Robert Yang
The following changes since commit 95ad5626296380358c8a502a3e04879dab653d78:

  build-appliance-image: Update to master head revision (2019-09-19 20:32:47 
+0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/test
  http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/test

Robert Yang (1):
  cases/bbtests.py: test_bitbake_g(): Check base-files rather than
busybox

 meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

-- 
2.7.4

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


Re: [OE-core] qemu 4.1 boot regression with riscv/musl

2019-09-24 Thread Khem Raj
On Tue, Sep 24, 2019 at 6:09 PM Alistair Francis 
wrote:

> I raised the regression on the QEMU mailing list while I was preparing
> the revert for QEMU and Palmer pointed out this diff for musl:
>
> diff --git a/arch/riscv64/atomic_arch.h b/arch/riscv64/atomic_arch.h
> index c9765342..41ad4d04 100644
> --- a/arch/riscv64/atomic_arch.h
> +++ b/arch/riscv64/atomic_arch.h
> @@ -14,7 +14,7 @@ static inline int a_cas(volatile int *p, int t, int s)
> "   sc.w.aqrl %1, %4, (%2)\n"
> "   bnez %1, 1b\n"
> "1:"
> -   : "=&r"(old), "=r"(tmp)
> +   : "=&r"(old), "=&r"(tmp)
> : "r"(p), "r"(t), "r"(s)
> : "memory");
> return old;
> @@ -31,7 +31,7 @@ static inline void *a_cas_p(volatile void *p, void
> *t, void *s)
> "   sc.d.aqrl %1, %4, (%2)\n"
> "   bnez %1, 1b\n"
> "1:"
> -   : "=&r"(old), "=r"(tmp)
> +   : "=&r"(old), "=&r"(tmp)
> : "r"(p), "r"(t), "r"(s)
> : "memory");
> return old;
>
> This seems to fix the problem for me. Hopefully Palmer will prepare a
> patch then we can just cherry pick that (from the list) on musl in OE
> core.
>
> I'm not going to send out the QEMU revert patch.
>

This is fine I think we were discussing atomics at musl IRC I will take
this patch to discussions

>
> Alistair
>
> On Mon, Sep 23, 2019 at 10:12 PM Khem Raj  wrote:
> >
> > Please send a patch
> >
> > On Mon, Sep 23, 2019 at 9:58 PM Alistair Francis 
> wrote:
> > >
> > > On Mon, Sep 23, 2019 at 7:11 PM Khem Raj  wrote:
> > > >
> > > > On Mon, Sep 23, 2019 at 6:39 PM Alistair Francis
> > > >  wrote:
> > > > >
> > > > > On Mon, 2019-09-23 at 17:59 -0700, Alistair Francis wrote:
> > > > > > On Mon, Sep 23, 2019 at 3:40 PM Khem Raj 
> wrote:
> > > > > > > Alistair,
> > > > > > >
> > > > > > > OE-core qemu upgrade to 4.1 has ended up with regressing
> musl/riscv
> > > > > > > port. I could pin point it to
> > > > > > >
> > > > > > >
> https://git.openembedded.org/openembedded-core/commit/?id=50a7dec95618080962e56fd347f505e691b7ad6f
> > > > > > >
> > > > > > > it works fine with 4.0, I havent looked further
> > > > > > >
> > > > > > > To reproduce
> > > > > > >
> > > > > > > build
> > > > > > > MACHINE=qemuriscv64 TCLIBC=musl  bitbake core-image-minimal
> > > > > > > TCLIBC=musl runqemu nographic
> > > > > >
> > > > > > Thanks, I can reproduce this and am bisecting QEMU now.
> > > > >
> > > > > This QEMU patch causes the failure. It reverts cleanly on master
> and
> > > > > that fixes the issue:
> > > > >
> > > > > RISC-V: Clear load reservations on context switch and SC
> > > > >
> > > > > This prevents a load reservation from being placed in one
> > > > > context/process,
> > > > > then being used in another, resulting in an SC succeeding
> incorrectly
> > > > > and
> > > > > breaking atomics.
> > > > >
> > > > > Signed-off-by: Joel Sing 
> > > > > Reviewed-by: Palmer Dabbelt 
> > > > > Reviewed-by: Richard Henderson 
> > > > > Signed-off-by: Palmer Dabbelt 
> > > > >
> > > > > I'll investigate a full solution tomorrow, in the mean time do you
> want
> > > > > to just revert the patch?
> > > >
> > > > yes lets use the revert for now, this will give you enough time to
> > > > investigate solution
> > > > since its riscv specific it will be low risk and we can even go to
> 3.0
> > > > release with it
> > >
> > > Do you want me to prepare a patch or do you want to do it?
> > >
> > > Alistair
> > >
> > > >
> > > > >
> > > > > Alistair
> > > > >
> > > > > >
> > > > > > Alistair
> > > > > >
> > > > > > > Thanks
> > > > > > > -Khem
> > > > > > > --
> > > > > > > ___
> > > > > > > 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 0/1] systemd: mark /etc as updated to avoid unnecessary services to run

2019-09-24 Thread Chen Qi
*** BLURB HERE ***
The following changes since commit 81f9e815d36848761a9dfa94b00ad998bb39a4a6:

  build-appliance-image: Update to master head revision (2019-09-19 20:32:53 
+0100)

are available in the git repository at:

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

Chen Qi (1):
  systemd: mark /etc as updated to avoid unnecessary services to run

 meta/classes/rootfs-postcommands.bbclass | 6 ++
 meta/recipes-core/systemd/systemd_243.bb | 6 --
 2 files changed, 6 insertions(+), 6 deletions(-)

-- 
1.9.1

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


Re: [OE-core] [PATCH 1/1] cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

2019-09-24 Thread Robert Yang



On 9/25/19 10:28 AM, Randy MacLeod wrote:

On 9/24/19 9:41 PM, Robert Yang wrote:

Hi Randy,

On 9/25/19 4:46 AM, Randy MacLeod wrote:

On 9/24/19 4:15 AM, Robert Yang wrote:

It checked whether busybox is in pn-buildlist and task-depends.dot after run
"bitbake -g core-image-minimal", but busybox is not a must for
core-image-minimal, 


You should explain why busybox is not a must since
usually busybox is the key package in c-i-m.

We have some images that deliberately exclude busybox and
I think this was found by using:
echo "BB_NO_NETWORK = \"0\"" >> conf/local.conf
echo "PNWHITELIST_LAYERS_remove = \"selftest\"" >> conf/local.conf
echo "SANITY_TESTED_DISTROS = \"\"" >> conf/local.conf
echo "DISTRO = \"wrlinux-graphics\"" >> conf/local.conf
echo "VIRTUAL-RUNTIME_no-busybox_forcevariable = \"\"" >> conf/local.conf
echo "PNWHITELIST_meta-layer0 += 'layerappendtest'" >> conf/local.conf

so mention that in general and VIRTUAL-RUNTIME_no-busybox_forcevariable
specifically.


Thanks for the explanation, I think that the maintainers know busybox
is not a must in oe-core, not matter for core-image-minimal or sato,
so I didn't add very detailed explanations.


commit logs are for maintainers, developers and users all
of varying ranges of familiarity with oe-core so
it's better to over-explain, without going over the top, IMO.






so the case may fail when busybox is not present, this case
is used for checking "bitbake -g", so check core-image-minimal itself is more
reliable.


It's true that busybox may not be present but we want to
test that some package is in the image, not just that the
task-depends.dot file exists and mentions c-i-m.
How about searching for 'base-files' ?


The base-files is my first thought, but it's not a must either, for testing
"bitbake -g recipe", I think that check the recipe itself is more reliable than
others.


If you had explained that in the commit log,
I wouldn't have replied, eh? ;-)

Sure base-files might not be part of the image but I
haven't seen an image that has done that. Your check will
always work so it really depends on the intent of the original
check.


Yes, make sense, I will send a V2 to check base-files.

// Robert



../Randy



// Robert



../Randy



Signed-off-by: Robert Yang 
---
  meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py 
b/meta/lib/oeqa/selftest/cases/bbtests.py

index 8e59baf..4103396 100644
--- a/meta/lib/oeqa/selftest/cases/bbtests.py
+++ b/meta/lib/oeqa/selftest/cases/bbtests.py
@@ -118,11 +118,12 @@ class BitbakeTests(OESelftestTestCase):
  self.assertIn(task, result.output, msg="Couldn't find %s task.")
  def test_bitbake_g(self):
-    result = bitbake('-g core-image-minimal')
+    recipe = 'core-image-minimal'
+    result = bitbake('-g %s' % recipe)
  for f in ['pn-buildlist', 'task-depends.dot']:
  self.addCleanup(os.remove, f)
  self.assertTrue('Task dependencies saved to \'task-depends.dot\'' 
in result.output, msg = "No task dependency \"task-depends.dot\" file was 
generated for the given task target. bitbake output: %s" % result.output)
-    self.assertTrue('busybox' in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), msg = "No 
\"busybox\" dependency found in task-depends.dot file.")
+    self.assertTrue(recipe in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), msg = "No 
\"%s\" dependency found in task-depends.dot file." % recipe)

  def test_image_manifest(self):
  bitbake('core-image-minimal')








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


[OE-core] [PATCH 1/1] systemd: mark /etc as updated to avoid unnecessary services to run

2019-09-24 Thread Chen Qi
We have updated hwdb via postinstall, updated systemd users via
'systemd_create_users', so there's no need to update /etc. Mark it
as updated to avoid unnecessary services like systemd-hwdb-update.service
to run.

This would solve timeout problem on qemumips. So also remove the
timeout change for systemd-hwdb-update.service.

Signed-off-by: Chen Qi 
---
 meta/classes/rootfs-postcommands.bbclass | 6 ++
 meta/recipes-core/systemd/systemd_243.bb | 6 --
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/meta/classes/rootfs-postcommands.bbclass 
b/meta/classes/rootfs-postcommands.bbclass
index 88f75d0..9323e92 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -36,6 +36,8 @@ POSTINST_LOGFILE ?= "${localstatedir}/log/postinstall.log"
 # Set default target for systemd images
 SYSTEMD_DEFAULT_TARGET ?= '${@bb.utils.contains("IMAGE_FEATURES", "x11-base", 
"graphical.target", "multi-user.target", d)}'
 ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("DISTRO_FEATURES", 
"systemd", "set_systemd_default_target; systemd_create_users;", "", d)}'
+# Mark /etc as updated for systemd as we have done things in rootfs time
+ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("DISTRO_FEATURES", 
"systemd", "systemd_mark_etc_updated;", "", d)}'
 
 ROOTFS_POSTPROCESS_COMMAND += 'empty_var_volatile;'
 
@@ -86,6 +88,10 @@ systemd_create_users () {
done
 }
 
+systemd_mark_etc_updated () {
+   touch ${IMAGE_ROOTFS}${sysconfdir}/.updated
+}
+
 #
 # A hook function to support read-only-rootfs IMAGE_FEATURES
 #
diff --git a/meta/recipes-core/systemd/systemd_243.bb 
b/meta/recipes-core/systemd/systemd_243.bb
index f0e8c56..0148c37 100644
--- a/meta/recipes-core/systemd/systemd_243.bb
+++ b/meta/recipes-core/systemd/systemd_243.bb
@@ -298,12 +298,6 @@ do_install() {
install -Dm 0644 ${WORKDIR}/99-default.preset 
${D}${systemd_unitdir}/system-preset/99-default.preset
 }
 
-do_install_append () {
-   # Mips qemu is extremely slow, allow more time for the hwdb update
-   # This is a workaround until 
https://github.com/systemd/systemd/issues/13581 is resolved
-   sed -i -e s#TimeoutSec=90s#TimeoutSec=180s# 
${D}${systemd_unitdir}/system/systemd-hwdb-update.service
-}
-
 python populate_packages_prepend (){
 systemdlibdir = d.getVar("rootlibdir")
 do_split_packages(d, systemdlibdir, '^lib(.*)\.so\.*', 'lib%s', 'Systemd 
%s library', extra_depends='', allow_links=True)
-- 
1.9.1

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


Re: [OE-core] [PATCH 1/1] cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

2019-09-24 Thread Randy MacLeod

On 9/24/19 9:41 PM, Robert Yang wrote:

Hi Randy,

On 9/25/19 4:46 AM, Randy MacLeod wrote:

On 9/24/19 4:15 AM, Robert Yang wrote:
It checked whether busybox is in pn-buildlist and task-depends.dot 
after run

"bitbake -g core-image-minimal", but busybox is not a must for
core-image-minimal, 


You should explain why busybox is not a must since
usually busybox is the key package in c-i-m.

We have some images that deliberately exclude busybox and
I think this was found by using:
echo "BB_NO_NETWORK = \"0\"" >> conf/local.conf
echo "PNWHITELIST_LAYERS_remove = \"selftest\"" >> conf/local.conf
echo "SANITY_TESTED_DISTROS = \"\"" >> conf/local.conf
echo "DISTRO = \"wrlinux-graphics\"" >> conf/local.conf
echo "VIRTUAL-RUNTIME_no-busybox_forcevariable = \"\"" >> conf/local.conf
echo "PNWHITELIST_meta-layer0 += 'layerappendtest'" >> conf/local.conf

so mention that in general and VIRTUAL-RUNTIME_no-busybox_forcevariable
specifically.


Thanks for the explanation, I think that the maintainers know busybox
is not a must in oe-core, not matter for core-image-minimal or sato,
so I didn't add very detailed explanations.


commit logs are for maintainers, developers and users all
of varying ranges of familiarity with oe-core so
it's better to over-explain, without going over the top, IMO.






so the case may fail when busybox is not present, this case
is used for checking "bitbake -g", so check core-image-minimal itself 
is more

reliable.


It's true that busybox may not be present but we want to
test that some package is in the image, not just that the
task-depends.dot file exists and mentions c-i-m.
How about searching for 'base-files' ?


The base-files is my first thought, but it's not a must either, for testing
"bitbake -g recipe", I think that check the recipe itself is more 
reliable than

others.


If you had explained that in the commit log,
I wouldn't have replied, eh? ;-)

Sure base-files might not be part of the image but I
haven't seen an image that has done that. Your check will
always work so it really depends on the intent of the original
check.

../Randy



// Robert



../Randy



Signed-off-by: Robert Yang 
---
  meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py 
b/meta/lib/oeqa/selftest/cases/bbtests.py

index 8e59baf..4103396 100644
--- a/meta/lib/oeqa/selftest/cases/bbtests.py
+++ b/meta/lib/oeqa/selftest/cases/bbtests.py
@@ -118,11 +118,12 @@ class BitbakeTests(OESelftestTestCase):
  self.assertIn(task, result.output, msg="Couldn't find 
%s task.")

  def test_bitbake_g(self):
-    result = bitbake('-g core-image-minimal')
+    recipe = 'core-image-minimal'
+    result = bitbake('-g %s' % recipe)
  for f in ['pn-buildlist', 'task-depends.dot']:
  self.addCleanup(os.remove, f)
  self.assertTrue('Task dependencies saved to 
\'task-depends.dot\'' in result.output, msg = "No task dependency 
\"task-depends.dot\" file was generated for the given task target. 
bitbake output: %s" % result.output)
-    self.assertTrue('busybox' in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), 
msg = "No \"busybox\" dependency found in task-depends.dot file.")
+    self.assertTrue(recipe in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), 
msg = "No \"%s\" dependency found in task-depends.dot file." % recipe)

  def test_image_manifest(self):
  bitbake('core-image-minimal')







--
# Randy MacLeod
# Wind River Linux
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/1] cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

2019-09-24 Thread Robert Yang

Hi Randy,

On 9/25/19 4:46 AM, Randy MacLeod wrote:

On 9/24/19 4:15 AM, Robert Yang wrote:

It checked whether busybox is in pn-buildlist and task-depends.dot after run
"bitbake -g core-image-minimal", but busybox is not a must for
core-image-minimal, 


You should explain why busybox is not a must since
usually busybox is the key package in c-i-m.

We have some images that deliberately exclude busybox and
I think this was found by using:
echo "BB_NO_NETWORK = \"0\"" >> conf/local.conf
echo "PNWHITELIST_LAYERS_remove = \"selftest\"" >> conf/local.conf
echo "SANITY_TESTED_DISTROS = \"\"" >> conf/local.conf
echo "DISTRO = \"wrlinux-graphics\"" >> conf/local.conf
echo "VIRTUAL-RUNTIME_no-busybox_forcevariable = \"\"" >> conf/local.conf
echo "PNWHITELIST_meta-layer0 += 'layerappendtest'" >> conf/local.conf

so mention that in general and VIRTUAL-RUNTIME_no-busybox_forcevariable
specifically.


Thanks for the explanation, I think that the maintainers know busybox
is not a must in oe-core, not matter for core-image-minimal or sato,
so I didn't add very detailed explanations.




so the case may fail when busybox is not present, this case
is used for checking "bitbake -g", so check core-image-minimal itself is more
reliable.


It's true that busybox may not be present but we want to
test that some package is in the image, not just that the
task-depends.dot file exists and mentions c-i-m.
How about searching for 'base-files' ?


The base-files is my first thought, but it's not a must either, for testing
"bitbake -g recipe", I think that check the recipe itself is more reliable than
others.

// Robert



../Randy



Signed-off-by: Robert Yang 
---
  meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py 
b/meta/lib/oeqa/selftest/cases/bbtests.py

index 8e59baf..4103396 100644
--- a/meta/lib/oeqa/selftest/cases/bbtests.py
+++ b/meta/lib/oeqa/selftest/cases/bbtests.py
@@ -118,11 +118,12 @@ class BitbakeTests(OESelftestTestCase):
  self.assertIn(task, result.output, msg="Couldn't find %s task.")
  def test_bitbake_g(self):
-    result = bitbake('-g core-image-minimal')
+    recipe = 'core-image-minimal'
+    result = bitbake('-g %s' % recipe)
  for f in ['pn-buildlist', 'task-depends.dot']:
  self.addCleanup(os.remove, f)
  self.assertTrue('Task dependencies saved to \'task-depends.dot\'' in 
result.output, msg = "No task dependency \"task-depends.dot\" file was 
generated for the given task target. bitbake output: %s" % result.output)
-    self.assertTrue('busybox' in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), msg = "No 
\"busybox\" dependency found in task-depends.dot file.")
+    self.assertTrue(recipe in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), msg = "No 
\"%s\" dependency found in task-depends.dot file." % recipe)

  def test_image_manifest(self):
  bitbake('core-image-minimal')





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


Re: [OE-core] Yocto Project Status WW39’19

2019-09-24 Thread ChenQi

On 09/24/2019 10:50 PM, Stephen K Jolley wrote:


Current Dev Position: YP 2.8 M4 Feature Freeze

Next Deadline: YP 3.0 Final Release 25th Oct


SWAT Team Rotation:

 *

SWAT lead is currently: Amanda

 *

SWAT team rotation: Amanda -> Chen on Sept. 27, 2019

 *

SWAT team rotation: Chen -> Armin on Oct. 4, 2019

 *

https://wiki.yoctoproject.org/wiki/Yocto_Build_Failure_Swat_Team




Hi Armin and Anju,

I want to switch the rotation with you two.
Because Oct. 1st is our National Day. We'll have a one-week holiday to 
celebrate it.


I've updated 
https://wiki.yoctoproject.org/wiki/Yocto_Build_Failure_Swat_Team#Members 
for now.

If you have any concern, please let me know.
Thanks in advance.

Best Regards,
Chen Qi




Next Team Meetings:

 *

Bug Triage meeting Thursday Sept 26th at 7:30am PDT
(https://zoom.us/j/454367603)

 *

Monthly Project Meeting Tuesday Oct. 1st at 8am PDT
(https://zoom.us/j/990892712) 

 *

Weekly Engineering Sync Tuesday Sept. 24th at 8am PDT
(https://zoom.us/j/990892712) 

 *

Twitch - Next event is Tuesday Octt. 8th at 8am PDT
(https://www.twitch.tv/yocto_project)


Key Status/Updates:

 *

We’re now in feature freeze for 3.0 and working on bug fixing for
final release

 *

We have built M3 and this is due from QA on Thursday

 *

The main unresolved issue for 3.0 is with the hash equivalence
server causing problems with the eSDK and this issue is being
investigated but can’t be reproduced as yet.

 *

We also want to find a better solution to the systemd timeout
problem before final release.

 *

A significant performance problem has been found on the
autobuilder where some builds are scaling in time badly as the
sstate cache grows, taking 12 hours or more in some cases.
Unfortunately nobody seems motivated to help work on this kind of
issue.

 *

The plan for M4 is to resolve the above issues and any issues
raised from M3 QA, then proceed with the main release rc builds.
The first M4 build is scheduled for next week.

 *

If anyone has any status items for the project they’d like to add
to the weekly reports, please email Richard+Stephen.


Planned Releases for YP 3.0 {2.8}:

 *

M3 Release 6th Sept

 *

M4 Cutoff 30th Sept - this will be YP 3.0.

 *

YP 3.0 {2.8 (M4)} Final Release 25th Oct


Planned upcoming dot releases:

 *

YP 2.7.2 (Warrior) is planned for after YP 3.0 release.

 *

YP 2.6.4 (Thud) is planned for after YP 2.7.2  release.


Tracking Metrics:

 *

WDD 2505 (last week
2449)(https://wiki.yoctoproject.org/charts/combo.html)

 *

Poky Patch Metrics

 o

Total patches found: 1427 (last week 1424)

 o

Patches in the Pending State: 587 (41%) [last week 587 (41%)]


Key Status Links for YP:

https://wiki.yoctoproject.org/wiki/Yocto_Project_v2.8_Status

https://wiki.yoctoproject.org/wiki/Yocto_2.8_Schedule

https://wiki.yoctoproject.org/wiki/Yocto_2.8_Features


The Yocto Project’s technical governance is through its Technical 
Steering Committee, more information is available at:


https://wiki.yoctoproject.org/wiki/TSC


The Status reports are now stored on the wiki at: 
https://wiki.yoctoproject.org/wiki/Weekly_Status



[If anyone has suggestions for other information you’d like to see on 
this weekly status update, let us know!]



--

Thanks,

*/Stephen K. Jolley/*

*Yocto Project Program Manager*

*7867 SW Bayberry Dr., Beaverton, OR 97007*

(*Cell*: (208) 244-4460

* *Email*:_stephen.k.jol...@intel.com _





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


Re: [OE-core] qemu 4.1 boot regression with riscv/musl

2019-09-24 Thread Alistair Francis
I raised the regression on the QEMU mailing list while I was preparing
the revert for QEMU and Palmer pointed out this diff for musl:

diff --git a/arch/riscv64/atomic_arch.h b/arch/riscv64/atomic_arch.h
index c9765342..41ad4d04 100644
--- a/arch/riscv64/atomic_arch.h
+++ b/arch/riscv64/atomic_arch.h
@@ -14,7 +14,7 @@ static inline int a_cas(volatile int *p, int t, int s)
"   sc.w.aqrl %1, %4, (%2)\n"
"   bnez %1, 1b\n"
"1:"
-   : "=&r"(old), "=r"(tmp)
+   : "=&r"(old), "=&r"(tmp)
: "r"(p), "r"(t), "r"(s)
: "memory");
return old;
@@ -31,7 +31,7 @@ static inline void *a_cas_p(volatile void *p, void
*t, void *s)
"   sc.d.aqrl %1, %4, (%2)\n"
"   bnez %1, 1b\n"
"1:"
-   : "=&r"(old), "=r"(tmp)
+   : "=&r"(old), "=&r"(tmp)
: "r"(p), "r"(t), "r"(s)
: "memory");
return old;

This seems to fix the problem for me. Hopefully Palmer will prepare a
patch then we can just cherry pick that (from the list) on musl in OE
core.

I'm not going to send out the QEMU revert patch.

Alistair

On Mon, Sep 23, 2019 at 10:12 PM Khem Raj  wrote:
>
> Please send a patch
>
> On Mon, Sep 23, 2019 at 9:58 PM Alistair Francis  wrote:
> >
> > On Mon, Sep 23, 2019 at 7:11 PM Khem Raj  wrote:
> > >
> > > On Mon, Sep 23, 2019 at 6:39 PM Alistair Francis
> > >  wrote:
> > > >
> > > > On Mon, 2019-09-23 at 17:59 -0700, Alistair Francis wrote:
> > > > > On Mon, Sep 23, 2019 at 3:40 PM Khem Raj  wrote:
> > > > > > Alistair,
> > > > > >
> > > > > > OE-core qemu upgrade to 4.1 has ended up with regressing musl/riscv
> > > > > > port. I could pin point it to
> > > > > >
> > > > > > https://git.openembedded.org/openembedded-core/commit/?id=50a7dec95618080962e56fd347f505e691b7ad6f
> > > > > >
> > > > > > it works fine with 4.0, I havent looked further
> > > > > >
> > > > > > To reproduce
> > > > > >
> > > > > > build
> > > > > > MACHINE=qemuriscv64 TCLIBC=musl  bitbake core-image-minimal
> > > > > > TCLIBC=musl runqemu nographic
> > > > >
> > > > > Thanks, I can reproduce this and am bisecting QEMU now.
> > > >
> > > > This QEMU patch causes the failure. It reverts cleanly on master and
> > > > that fixes the issue:
> > > >
> > > > RISC-V: Clear load reservations on context switch and SC
> > > >
> > > > This prevents a load reservation from being placed in one
> > > > context/process,
> > > > then being used in another, resulting in an SC succeeding incorrectly
> > > > and
> > > > breaking atomics.
> > > >
> > > > Signed-off-by: Joel Sing 
> > > > Reviewed-by: Palmer Dabbelt 
> > > > Reviewed-by: Richard Henderson 
> > > > Signed-off-by: Palmer Dabbelt 
> > > >
> > > > I'll investigate a full solution tomorrow, in the mean time do you want
> > > > to just revert the patch?
> > >
> > > yes lets use the revert for now, this will give you enough time to
> > > investigate solution
> > > since its riscv specific it will be low risk and we can even go to 3.0
> > > release with it
> >
> > Do you want me to prepare a patch or do you want to do it?
> >
> > Alistair
> >
> > >
> > > >
> > > > Alistair
> > > >
> > > > >
> > > > > Alistair
> > > > >
> > > > > > Thanks
> > > > > > -Khem
> > > > > > --
> > > > > > ___
> > > > > > 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] [warrior][ 25/50] boost: Fix build and enable context and coroutines on aarch64

2019-09-24 Thread akuster808



On 9/23/19 8:37 PM, Andre McCurdy wrote:
> On Mon, Sep 23, 2019 at 7:52 PM Armin Kuster  wrote:
>> From: "Bedel, Alban" 
>>
>> Like for ARM bjam need some hints about the ABI to properly build on
>> aarch64. While at it also enable context and coroutine as these are
>> supported on aarch64.
> What's the symptom when this is mis-detected ?
>
> I'm guessing big endian 32bit ARM (ie the _armeb over-ride) probably
> needs the same fix too... but it's not clear how to test or confirm
> that.

You need to ask the author of the patch. This is in master and a
backport request was sent
https://patchwork.openembedded.org/patch/163995/

with no community feedback. This seemed like a fix to an issue.

- armin
>> Signed-off-by: Alban Bedel 
>> Signed-off-by: Armin Kuster 
>> ---
>>  meta/recipes-support/boost/boost.inc | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-support/boost/boost.inc 
>> b/meta/recipes-support/boost/boost.inc
>> index 9be3717..c2e2cbb 100644
>> --- a/meta/recipes-support/boost/boost.inc
>> +++ b/meta/recipes-support/boost/boost.inc
>> @@ -33,6 +33,7 @@ BOOST_LIBS_append_x86 = " context coroutine"
>>  BOOST_LIBS_append_x86-64 = " context coroutine"
>>  BOOST_LIBS_append_powerpc = " context coroutine"
>>  BOOST_LIBS_append_arm = " context coroutine"
>> +BOOST_LIBS_append_aarch64 = " context coroutine"
>>  # need consistent settings for native builds (x86 override not applied for 
>> native)
>>  BOOST_LIBS_remove_class-native = " context coroutine"
>>  # does not compile
>> @@ -151,6 +152,7 @@ BJAM_OPTS_append_x86-x32 = " abi=x32 address-model=64"
>>
>>  # cross compiling for arm fails to detect abi, so provide some help
>>  BJAM_OPTS_append_arm = " abi=aapcs architecture=arm"
>> +BJAM_OPTS_append_aarch64 = " abi=aapcs address-model=64 architecture=arm"
>>
>>  do_configure() {
>> cp -f ${S}/boost/config/platform/linux.hpp 
>> ${S}/boost/config/platform/linux-gnueabi.hpp
>> --
>> 2.7.4
>>
>> --
>> ___
>> 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] [warrior][PATCH] kernel-uboot: compress arm64 kernels

2019-09-24 Thread akuster808


On 9/24/19 12:23 AM, Bedel, Alban wrote:
> On Tue, 2019-09-03 at 09:41 +, Bedel, Alban wrote:
>> On Wed, 2019-07-31 at 13:53 +, Bedel, Alban wrote:
>>> AArch64 images are not self-decompressing, thus usually much
>>> larger.
>>> Boot times can be reduced by compressing them in FIT and uImages.
>>>
>>> This commit is a backport of commit a725d188b5 (kernel-uboot:
>>> compress
>>> arm64 kernels) and commit 60bc7e180e (kernel-uboot: remove useless
>>> special casing of arm64 Image) from master. Both commit were melted
>>> into one to avoid some useless churn.
>> Was this patch overlooked, or is there a reason it is not considered
>> in
>> the next round of update for warrior? Without this patch kernel
>> images
>> are too large to fit in the flash of the system I'm using.
>> Furthermore
>> it is not trivial to fix this in my own layer.
> Please, I really like to get an answer here. I'm fine if there is a
> reason why this patch is not considered for warrior, but just getting
> ignored is very frustrating.

This appears to be a performance enhancement which does not fall into
the criteria for back porting to a stable branch.

- armin
>
> Alban
>




signature.asc
Description: OpenPGP digital signature
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] ✗ patchtest: failure for rpm: make rpm work in toolchain. (rev4)

2019-09-24 Thread Patchwork
== Series Details ==

Series: rpm: make rpm work in toolchain. (rev4)
Revision: 4
URL   : https://patchwork.openembedded.org/series/19789/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Errors in your Python code were encountered [test_pylint] 
  Suggested fixCorrect the lines introduced by your patch
  Output   Please, fix the listed issues:
   meta/recipes-devtools/rpm/files/rpm-setup.py does not exist



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

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


[OE-core] [PATCH v4] rpm: make rpm work in toolchain.

2019-09-24 Thread Zheng Ruoqin
We need to configure rpm to use package architecture from yocto build system.

Install rpmrc and rpm/platform to ${SDKTARGETSYSROOT} because config file in 
host-sysroot as /opt/poky/2.7+snapshot/sysroots/x86_64-pokysdk-linux will be 
covered by another ARCH which result in previous config settings inefficacy.

To resolve it, put config file in target-sysroot like 
/opt/poky/2.7+snapshot/sysroots/core2-64-poky-linux. As each ARCH has its own 
target-sysroot, config file will not be covered.

Signed-off-by: Zheng Ruoqin 
---
 meta/recipes-devtools/rpm/files/rpm-setup.py | 27 
 meta/recipes-devtools/rpm/rpm_4.14.2.1.bb| 19 ++
 2 files changed, 46 insertions(+)
 create mode 100644 meta/recipes-devtools/rpm/files/rpm-setup.py

diff --git a/meta/recipes-devtools/rpm/files/rpm-setup.py 
b/meta/recipes-devtools/rpm/files/rpm-setup.py
new file mode 100644
index 00..b3e8a1198c
--- /dev/null
+++ b/meta/recipes-devtools/rpm/files/rpm-setup.py
@@ -0,0 +1,27 @@
+#!/usr/bin/env python3
+
+import os
+import sys
+import shutil
+
+try:
+native_sysroot = os.environ['OECORE_NATIVE_SYSROOT']
+sdktarget_sysroot = os.environ['SDKTARGETSYSROOT']
+except KeyError:
+print("Not in environment setup, bailing")
+sys.exit(1)
+
+target_etc_dir = os.path.join(sdktarget_sysroot, 'etc/rpm')
+
+if not os.path.exists(target_etc_dir):
+os.makedirs(target_etc_dir)
+
+template_file = os.path.join(native_sysroot, 'usr/share/rpm/rpmrc')
+cross_file = os.path.join(sdktarget_sysroot, 'etc/rpmrc')
+shutil.copy(template_file, cross_file)
+
+template_file = os.path.join(native_sysroot, 'usr/share/rpm/platform')
+cross_file = os.path.join(sdktarget_sysroot, 'etc/rpm/platform')
+shutil.copy(template_file, cross_file)
+
+
diff --git a/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb 
b/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb
index c37330eb4c..e1d1951d74 100644
--- a/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb
+++ b/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb
@@ -44,6 +44,9 @@ SRC_URI = 
"git://github.com/rpm-software-management/rpm;branch=rpm-4.14.x \

file://0001-mono-find-provides-requires-do-not-use-monodis-from-.patch \
"
 
+SRC_URI_append_class-nativesdk = "file://rpm-setup.py \
+ "
+
 PE = "1"
 SRCREV = "4a9440006398646583f0d9ae1837dad2875013aa"
 
@@ -113,6 +116,20 @@ do_install_append_class-nativesdk() {
 done
 
 rm -rf ${D}/var
+install -d ${D}${datadir}/rpm
+
+cat >${D}/${datadir}/rpm/rpmrc <${D}/${datadir}/rpm/platform 

[OE-core] [PATCH 4/4] linux-yocto/5.2: update to v5.2.16

2019-09-24 Thread bruce . ashfield
From: Bruce Ashfield 

Updating linux-yocto/5.2 to the latest korg -stable release that comprises
the following commits:

1e2ba4a74fa7 Linux 5.2.16
2508f833875c x86/build: Add -Wnoaddress-of-packed-member to 
REALMODE_CFLAGS, to silence GCC9 build warning
8803770b85d3 rsi: fix a double free bug in rsi_91x_deinit()
d8bcd38cb933 platform/x86: pcengines-apuv2: use KEY_RESTART for front button
f7a15c7068f0 platform/x86: pmc_atom: Add CB4063 Beckhoff Automation board 
to critclk_systems DMI table
657d4ca80d64 KVM: SVM: Fix detection of AMD Errata 1096
54d782e7c865 kvm: nVMX: Remove unnecessary sync_roots from handle_invept
00ab9eeb0314 modules: always page-align module section allocations
ec93d0db91ce modules: fix compile error if don't have strict module rwx
0a199213f586 modules: fix BUG when load module with rodata=n
63e3de7299ba iio: adc: stm32-dfsdm: fix data type
bb9dee6edb5a iio: adc: stm32-dfsdm: fix output resolution
83ad11d6d7fa Revert "Bluetooth: btusb: driver to enable the usb-wakeup 
feature"
7c218ebc7899 mm/z3fold.c: fix lock/unlock imbalance in z3fold_page_isolate
1894d857ce86 mm/z3fold.c: remove z3fold_migration trylock
ead8b4f3b2f9 drm/mediatek: mtk_drm_drv.c: Add of_node_put() before goto
009b31923398 drm: panel-orientation-quirks: Add extra quirk table entry for 
GPD MicroPC
6dbe4b80180c firmware: ti_sci: Always request response from firmware
71f0dafe61ce crypto: talitos - HMAC SNOOP NO AFEU mode requires SW icv 
checking.
456945703690 crypto: talitos - Do not modify req->cryptlen on decryption.
dcefbf43b003 crypto: talitos - fix ECB algs ivsize
291d9ef44614 crypto: talitos - check data blocksize in ablkcipher.
86545794d2e5 crypto: talitos - fix CTR alg blocksize
808a33cd37dd crypto: talitos - check AES key size
0a7737c960e3 driver core: Fix use-after-free and double free on glue 
directory
6502d1acbe63 ubifs: Correctly use tnc_next() in search_dh_cookie()
85e398e6aa01 PCI: Always allow probing with driver_override
f5a7b0f01b23 mtd: rawnand: mtk: Fix wrongly assigned OOB buffer pointer 
issue
7b20eb930bd0 clk: rockchip: Don't yell about bad mmc phases when getting
4111a1f2142e mt76: mt7615: Use after free in mt7615_mcu_set_bcn()
195d154b8a71 mt76: Fix a signedness bug in mt7615_add_interface()
76d64d6d9ed8 clk: Simplify debugfs printing and add a newline
268311c39a7f clk: Fix debugfs clk_possible_parents for clks without parent 
string names
a9258bc6e0b3 drm/meson: Add support for XBGR & ABGR formats
5924bb61ac94 x86/ima: check EFI SetupMode too
c5699cc857a0 x86/boot: Use efi_setup_data for searching RSDP on kexec-ed 
kernels
2fd4aaaf615d kernel/module: Fix mem leak in module_add_modinfo_attrs
a9ce57d57155 powerpc: Add barrier_nospec to raw_copy_in_user()
6876bd0c34c8 x86/purgatory: Change compiler flags from -mcmodel=kernel to 
-mcmodel=large to fix kexec relocation errors
cc0feeebbb3f KVM: nVMX: handle page fault in vmread
9e8659bdca66 KVM: x86/mmu: Reintroduce fast invalidate/zap for flushing 
memslot
db68c0c83a76 KVM: x86: work around leak of uninitialized stack contents
17cd7cf4b355 KVM: s390: Do not leak kernel stack data in the 
KVM_S390_INTERRUPT ioctl
0c3ffd500445 KVM: s390: kvm_s390_vm_start_migration: check dirty_bitmap 
before using it as target for memset()
71661b871973 regulator: twl: voltage lists for vdd1/2 on twl4030
803c7bbc0d5d genirq: Prevent NULL pointer dereference in resend_irqs()
041f865f676b mt76: mt76x0e: disable 5GHz band for MT7630E
fbf07d827d82 Revert "rt2800: enable TX_PIN_CFG_LNA_PE_ bits per band"
868921b1f3d5 ixgbe: Prevent u8 wrapping of ITR value to something less than 
10us
c42a7b87b8d7 ixgbe: fix double clean of Tx descriptors with xdp
076566aafd94 ipc: fix sparc64 ipc() wrapper
b2c92cb1d141 ipc: fix semtimedop for generic 32-bit architectures
b68833b65bf0 drm/i915: Restore relaxed padding (OCL_OOB_SUPPRES_ENABLE) for 
skl+
e0e11df7f249 drm/i915: Limit MST to <= 8bpc once again
8eb5a78c20e8 drm/lima: fix lima_gem_wait() return value
46d1c96a1a5e mmc: tmio: Fixup runtime PM management during remove
f5e9862fedd0 mmc: tmio: Fixup runtime PM management during probe
e5b9762c20a3 Revert "mmc: sdhci: Remove unneeded quirk2 flag of O2 SD host 
controller"
0699a10fe9ed Revert "mmc: bcm2835: Terminate timeout work synchronously"
c554749c3f7e cgroup: freezer: fix frozen state inheritance
4fa875f06c43 Btrfs: fix assertion failure during fsync and use of stale 
transaction
09234426db8f gpio: fix line flag validation in lineevent_create
94cd945b2d24 gpio: fix line flag validation in linehandle_create
c54f5198f527 gpio: mockup: add missing single_release()
06477d8e4425 gpiolib: acpi: Add gpiolib_acpi_run_edge_events_on_boot option 
and blacklist
3c0bf14d186b net: sock_map, fix missing ulp check in sock ha

[OE-core] [PATCH 1/4] linux-yocto/5.2: update to v5.2.13

2019-09-24 Thread bruce . ashfield
From: Bruce Ashfield 

Integrating the korg -stable updates that comprise the following
commits:

   218ca2e5affe Linux 5.2.13
   4c634717d778 Revert "Input: elantech - enable SMBus on new (2018+) systems"
   140839fe4e71 Linux 5.2.12
   5566d1c61b2f Revert "ASoC: Fail card instantiation if DAI format setup fails"
   4d8966020827 hsr: switch ->dellink() to ->ndo_uninit()
   072c9337f3a0 hsr: fix a NULL pointer deref in hsr_dev_xmit()
   08523d5a2f4a hsr: implement dellink to clean up resources
   a282179be747 bpf: fix use after free in prog symbol exposure
   0d5014b80e5c x86/ptrace: fix up botched merge of spectrev1 fix
   3af8db6a99f6 drm/i915/dp: Fix DSC enable code to use cpu_transcoder instead 
of encoder->type
   b6980646c9ab drm/i915: Do not create a new max_bpc prop for MST connectors
   79d5d7317ce3 iwlwifi: pcie: handle switching killer Qu B0 NICs to C0
   3146a6de3d58 iwlwifi: pcie: don't switch FW to qnj when ax201 is detected
   9e7e685012f2 iwlwifi: pcie: add support for qu c-step devices
   c013312ef7a8 iwlwifi: change 0x02F0 fw from qu to quz
   feee62efa7ad iwlwifi: add new cards for 9000 and 2 series
   0fdbd727e12a iwlwifi: add new cards for 22000 and change wrong structs
   805363e2dfa5 iwlwifi: add new cards for 22000 and fix struct name
   e27fc3445751 mmc: sdhci-sprd: add get_ro hook function
   9ad0348f708d mmc: sdhci-sprd: Implement the get_max_timeout_count() interface
   d100666e3cfc mmc: sdhci-sprd: clear the UHS-I modes read from registers
   b018fcb92250 mac80211: Correctly set noencrypt for PAE frames
   08392de0ccc7 mac80211: Don't memset RXCB prior to PAE intercept
   4e118994cea8 cfg80211: Fix Extended Key ID key install checks
   b14f5ba7490e mac80211: fix possible sta leak
   3cd42050aac6 Revert "cfg80211: fix processing world regdomain when non 
modular"
   b6a0d1f9bf08 mm: memcontrol: fix percpu vmstats and vmevents flush
   bba5bcb0cc0e mm, memcg: partially revert "mm/memcontrol.c: keep local VM 
counters in sync with the hierarchical ones"
   8706ffe20444 mms: sdhci-sprd: add SDHCI_QUIRK_BROKEN_CARD_DETECTION
   d7c7531ac9a3 mt76: mt76x0u: do not reset radio on resume
   839e96131567 SUNRPC: Don't handle errors if the bind/connect succeeded
   90ff6dd4ced6 crypto: ccp - Ignore unconfigured CCP device on suspend/resume
   3c3c233738df VMCI: Release resource if the work is already queued
   6992ae83866a bus: hisi_lpc: Add .remove method to avoid driver unbind crash
   0a6caa4e3f21 bus: hisi_lpc: Unregister logical PIO range to avoid potential 
use-after-free
   3e3bf9dfdaf4 i2c: piix4: Fix port selection for AMD Family 16h Model 30h
   712029329a58 drm/i915: Call dma_set_max_seg_size() in i915_driver_hw_probe()
   0573f44d42b1 drm/i915: Don't deballoon unused ggtt drm_mm_node in linux guest
   f78e0d8172f3 drm/amdgpu: fix GFXOFF on Picasso and Raven2
   4c7ee7bd4f7e drm/amdgpu: Add APTX quirk for Dell Latitude 5495
   71817079db2f lib: logic_pio: Add logic_pio_unregister_range()
   0a6e75686543 lib: logic_pio: Avoid possible overlap for unregistering regions
   94cfb33438e0 lib: logic_pio: Fix RCU usage
   e0e3650991cd Revert "NFSv4/flexfiles: Abort I/O early if the layout segment 
was invalidated"
   666a848c6797 NFS: Ensure O_DIRECT reports an error if the bytes read/written 
is 0
   3fd824dc16de NFSv4/pnfs: Fix a page lock leak in nfs_pageio_resend()
   d3a2180a5720 lkdtm/bugs: fix build error in lkdtm_EXHAUST_STACK
   61c6f2ba fsi: scom: Don't abort operations for minor errors
   d886915cd91b typec: tcpm: fix a typo in the comparison of pdo_max_voltage
   82b520267b49 intel_th: pci: Add Tiger Lake support
   1c01033684f3 intel_th: pci: Add support for another Lewisburg PCH
   d36c706f854c stm class: Fix a double free of stm_source_device
   ba3e88a664c8 mmc: sdhci-sprd: add SDHCI_QUIRK2_PRESET_VALUE_BROKEN
   77162751167c mmc: sdhci-sprd: fixed incorrect clock divider
   a94076eed72e mmc: core: Fix init of SD cards reporting an invalid VDD range
   b8dd05ea2e74 mmc: sdhci-cadence: enable v4_mode to fix ADMA 64-bit addressing
   096179da39b6 mmc: sdhci-of-at91: add quirk for broken HS200
   cb82fa2d124a Revert "mmc: sdhci-tegra: drop ->get_ro() implementation"
   31f4eb72692c mei: me: add Tiger Lake point LP device ID
   871c867ac423 KVM: arm/arm64: vgic-v2: Handle SGI bits in GICD_I{S,C}PENDR0 
as WI
   ea93e118ef2e KVM: arm/arm64: vgic: Fix potential deadlock when ap_list is 
long
   d69ebe85e169 KVM: PPC: Book3S: Fix incorrect guest-to-user-translation error 
handling
   3698cd854f43 tools/power turbostat: Fix caller parameter of get_tdp_amd()
   2d26f7f401e7 USB: storage: ums-realtek: Whitelist auto-delink support
   35f1b0a62c36 USB: storage: ums-realtek: Update module parameter description 
for auto_delink_en
   960ff077477e usb: host: xhci: rcar: Fix typo in compatible string matching
   6ca46f45c208 usb: host: ohci: fix a race condition between shutdown and irq
   73830b83ece9 usb: chipidea: udc: don't do hardware access if gadget has 
stopped
   0aa

[OE-core] [PATCH 3/4] linux-yocto/5.2: update to v5.2.14

2019-09-24 Thread bruce . ashfield
From: Bruce Ashfield 

Updating to the latest korg -stable for 5.2, which comprises the
following changes:

   997fee5473ce Linux 5.2.14
   0ccc5c227f88 Revert "mmc: core: do not retry CMD6 in __mmc_switch()"
   317a66e24b46 x86/boot: Preserve boot_params.secure_boot from sanitizing
   1a7fd193e9d8 Revert "x86/apic: Include the LDR when clearing out APIC 
registers"
   ae96cf9e1e31 libceph: allow ceph_buffer_put() to receive a NULL ceph_buffer
   8d50b82ea79b x86/boot/compressed/64: Fix missing initialization in 
find_trampoline_placement()
   852a051ef22b KVM: arm/arm64: VGIC: Properly initialise private IRQ affinity
   44dfa46aaf7c gpio: Fix irqchip initialization order
   157ec0a3f834 RDMA/bnxt_re: Fix stack-out-of-bounds in 
bnxt_qplib_rcfw_send_message
   cf9ec2e9056c afs: use correct afs_call_type in yfs_fs_store_opaque_acl2
   3f6685835889 afs: Fix possible oops in afs_lookup trace event
   c231241d83b9 afs: Fix leak in afs_lookup_cell_rcu()
   7f134d569e1a KVM: arm/arm64: Only skip MMIO insn once
   39c62cdaa2d4 ceph: fix buffer free while holding i_ceph_lock in fill_inode()
   c4e28be29a45 ceph: fix buffer free while holding i_ceph_lock in 
__ceph_build_xattrs_blob()
   f41cd559f1f3 ceph: fix buffer free while holding i_ceph_lock in 
__ceph_setxattr()
   3ebab463e9f4 drm/amdgpu: prevent memory leaks in AMDGPU_CS ioctl
   23da8e9ef69b selftests/kvm: make platform_info_test pass on AMD
   666a41848d72 selftests: kvm: fix state save/load on processors without XSAVE
   dd53d830bb30 infiniband: hfi1: fix memory leaks
   bcb3211656fa infiniband: hfi1: fix a memory leak bug
   1e93149659cc IB/mlx4: Fix memory leaks
   fc38afc04343 RDMA/cma: fix null-ptr-deref Read in cma_cleanup
   d61a256fece2 nvme: Fix cntlid validation when not using NVMEoF
   e1031b6c8100 nvme-multipath: fix possible I/O hang when paths are updated
   6950d5b64a15 Tools: hv: kvp: eliminate 'may be used uninitialized' warning
   195b5aa923ec Input: hyperv-keyboard: Use in-place iterator API in the 
channel callback
   31b60e5c604c scsi: lpfc: Mitigate high memory pre-allocation by SCSI-MQ
   aa4bc3a35253 x86/boot/compressed/64: Fix boot on machines with broken E820 
table
   3091859b0b5e HID: cp2112: prevent sleeping function called from invalid 
context
   e8fd4f47b224 HID: intel-ish-hid: ipc: add EHL device id
   d58500c6fcb4 kprobes: Fix potential deadlock in kprobe_optimizer()
   9ad89d579c01 sched/core: Schedule new worker even if PI-blocked
   086ddc5e7172 ravb: Fix use-after-free ravb_tstamp_skb
   37f12b895446 wimax/i2400m: fix a memory leak bug
   461f5b613b4d net: cavium: fix driver name
   1915dd1826be ibmvnic: Unmap DMA address of TX descriptor buffers after use
   895a07a5a03a net: kalmia: fix memory leaks
   241a735f5f68 cx82310_eth: fix a memory leak bug
   e8f8411a8def vfs: fix page locking deadlocks when deduping files
   a4234e27ed05 lan78xx: Fix memory leaks
   96ece5798677 clk: Fix potential NULL dereference in clk_fetch_parent_index()
   a6cb8dd3664e clk: Fix falling back to legacy parent string matching
   7da16490e5a5 net: myri10ge: fix memory leaks
   0f1c537cf09c liquidio: add cleanup in octeon_setup_iq()
   101743ca0284 selftests: kvm: fix vmx_set_nested_state_test
   5aac82ac9c7c selftests: kvm: provide common function to enable eVMCS
   926a6e9efce9 selftests: kvm: do not try running the VM in 
vmx_set_nested_state_test
   bf31a46ead59 cxgb4: fix a memory leak bug
   46bf670f4455 scsi: target: tcmu: avoid use-after-free after command timeout
   8e639df33729 scsi: qla2xxx: Fix gnl.l memory leak on adapter init failure
   6fe444e60cda drm/mediatek: set DMA max segment size
   1e12afb4c60a drm/mediatek: use correct device to import PRIME buffers
   2b4a29f04533 netfilter: nft_flow_offload: skip tcp rst and fin packets
   1dcb0d4eaf85 gpio: Fix build error of function redefinition
   f6bd80bc6223 ibmveth: Convert multicast list size for little-endian system
   d45c33d890bc s390/qeth: serialize cmd reply with concurrent timeout
   b12691c24ea9 Bluetooth: hci_qca: Send VS pre shutdown command.
   113d5ca74866 Bluetooth: btqca: Add a short delay before downloading the NVM
   ab0f749707ea net: tc35815: Explicitly check NET_IP_ALIGN is not zero in 
tc35815_rx
   37a0be861375 hv_netvsc: Fix a warning of suspicious RCU usage
   936315c0c94e ixgbe: fix possible deadlock in ixgbe_service_task()
   1dc961de7733 tools: bpftool: fix error message (prog -> object)
   ea3c243ce799 netfilter: nf_flow_table: teardown flow timeout race
   96a517d0ba5f netfilter: nf_flow_table: conntrack picks up expired flows
   586f00143091 netfilter: nf_tables: use-after-free in failing rule with bound 
set
   830b5c3760ff net: tundra: tsi108: use spin_lock_irqsave instead of 
spin_lock_irq in IRQ context
   60a4f2b2b6b8 clk: samsung: exynos542x: Move MSCL subsystem clocks to its 
sub-CMU
   c65a2b20a934 clk: samsung: exynos5800: Move MAU subsystem clocks to MAU 
sub-CMU
   f7bd5e9fe13f clk: samsung: Change signature of exynos5_subcmus_init()

[OE-core] [PATCH 2/4] linux-yocto/4.19: update to v4.19.72

2019-09-24 Thread bruce . ashfield
From: Bruce Ashfield 

Updating to the latest -korg -stable relase. A summary of the
changes follows:

   ee809c7e0895 Linux 4.19.72
   991467a47cf2 Revert "x86/apic: Include the LDR when clearing out APIC 
registers"
   0f134f6e722c libceph: allow ceph_buffer_put() to receive a NULL ceph_buffer
   5049632bc958 x86/boot/compressed/64: Fix missing initialization in 
find_trampoline_placement()
   b8727dff55f0 KVM: arm/arm64: VGIC: Properly initialise private IRQ affinity
   1a31b0d0ddfa afs: Fix leak in afs_lookup_cell_rcu()
   111d36b6fb7e KVM: arm/arm64: Only skip MMIO insn once
   b84817d96e0f ceph: fix buffer free while holding i_ceph_lock in fill_inode()
   5cd1e3552f0e ceph: fix buffer free while holding i_ceph_lock in 
__ceph_build_xattrs_blob()
   dfb8712c7acc ceph: fix buffer free while holding i_ceph_lock in 
__ceph_setxattr()
   ddb55cc39c70 selftests/kvm: make platform_info_test pass on AMD
   6cb9f8d60f8f selftests: kvm: fix state save/load on processors without XSAVE
   08c2052815e3 infiniband: hfi1: fix memory leaks
   d1b7f3252d56 infiniband: hfi1: fix a memory leak bug
   adb87027b5ff IB/mlx4: Fix memory leaks
   5e416b11b4a9 nvme-multipath: fix possible I/O hang when paths are updated
   5bbebceec61d Tools: hv: kvp: eliminate 'may be used uninitialized' warning
   1d42e40fe3c5 Input: hyperv-keyboard: Use in-place iterator API in the 
channel callback
   e02aac3edb46 x86/boot/compressed/64: Fix boot on machines with broken E820 
table
   05d611c4ffb7 HID: cp2112: prevent sleeping function called from invalid 
context
   5e1d50a3eafe kprobes: Fix potential deadlock in kprobe_optimizer()
   a5839b6b3a79 ravb: Fix use-after-free ravb_tstamp_skb
   54eac3997ee4 wimax/i2400m: fix a memory leak bug
   7506e8c4bec8 net: cavium: fix driver name
   ea78dc8b5e66 ibmvnic: Unmap DMA address of TX descriptor buffers after use
   4fcb9b3f263e net: kalmia: fix memory leaks
   1ad45d0f69d2 cx82310_eth: fix a memory leak bug
   ac3cc25f380b vfs: fix page locking deadlocks when deduping files
   9ea1feadf5e0 lan78xx: Fix memory leaks
   375ab446ec76 net: myri10ge: fix memory leaks
   f13b7ec5e1c5 liquidio: add cleanup in octeon_setup_iq()
   2c3dd20f852a cxgb4: fix a memory leak bug
   b8cd0b7b09ed scsi: target: tcmu: avoid use-after-free after command timeout
   6c9a1e17d063 scsi: qla2xxx: Fix gnl.l memory leak on adapter init failure
   b3496367d9cb drm/mediatek: set DMA max segment size
   c9f595453bc3 drm/mediatek: use correct device to import PRIME buffers
   a54fa5dff8cd netfilter: nft_flow_offload: skip tcp rst and fin packets
   6052090280b8 gpio: Fix build error of function redefinition
   cc8aa6155611 ibmveth: Convert multicast list size for little-endian system
   32e912b91b5d Bluetooth: btqca: Add a short delay before downloading the NVM
   7b7a11549dc1 net: tc35815: Explicitly check NET_IP_ALIGN is not zero in 
tc35815_rx
   752832f2e8c9 hv_netvsc: Fix a warning of suspicious RCU usage
   463d87bc13ff tools: bpftool: fix error message (prog -> object)
   5776970fb4ec netfilter: nf_tables: use-after-free in failing rule with bound 
set
   d22ed7b72acf net: tundra: tsi108: use spin_lock_irqsave instead of 
spin_lock_irq in IRQ context
   3ddda4f3114b spi: bcm2835aux: fix corruptions for longer spi transfers
   fe49c3de7372 spi: bcm2835aux: remove dangerous uncontrolled read of fifo
   a4a9ee79036b spi: bcm2835aux: unifying code between polling and interrupt 
driven code
   ee271ead3b61 x86/boot: Preserve boot_params.secure_boot from sanitizing
   9484203d254d net/rds: Fix info leak in rds6_inc_info_copy()
   5977bc19ce7f tcp: remove empty skb from write queue in error cases
   6f3126379879 tcp: inherit timestamp on mtu probe
   6f8348f63706 net: stmmac: dwmac-rk: Don't fail if phy regulator is absent
   38166934f89c net_sched: fix a NULL pointer deref in ipt action
   5ff0ab0c668b net: sched: act_sample: fix psample group handling on overwrite
   6a2bd8261630 net: fix skb use after free in netpoll
   8a5d27eaba68 mld: fix memory leak in mld_del_delrec()
   e7d2672c66e4 Linux 4.19.71
   72168ae78629 Revert "Input: elantech - enable SMBus on new (2018+) systems"
   0fed55c248d9 Linux 4.19.70
   9854d089f790 Revert "ASoC: Fail card instantiation if DAI format setup fails"
   e064466cb614 mt76: mt76x0u: do not reset radio on resume
   b307f99dca5a x86/ptrace: fix up botched merge of spectrev1 fix
   3b26fa9e3ec3 i2c: piix4: Fix port selection for AMD Family 16h Model 30h
   4f4be79c9ee7 NFS: Ensure O_DIRECT reports an error if the bytes read/written 
is 0
   b5891b624b9a NFS: Pass error information to the pgio error cleanup routine
   812de6dee596 NFSv4/pnfs: Fix a page lock leak in nfs_pageio_resend()
   57c491fd844d NFS: Clean up list moves of struct nfs_page
   79f1b33c53a0 KVM: arm/arm64: vgic-v2: Handle SGI bits in GICD_I{S,C}PENDR0 
as WI
   ab8ecc278dc8 KVM: arm/arm64: vgic: Fix potential deadlock when ap_list is 
long
   db1841a2dd4c KVM: PPC: Book3S: Fix incorrect guest-to-user-translation er

[OE-core] [PATCH 0/4] linux-yocto: -stable updates

2019-09-24 Thread bruce . ashfield
From: Bruce Ashfield 

Richard,

Here's the latest round of -stable updates. I haven't found any issues
with my build and boot tests with these .. but as you know, we sometimes
do find corner cases.

I've left all my 5.2 -stable bumps as separate commits, since they
represent a set of SRCREVs that I've tested, and that should be preserved
in the history in case we have issues.

Cheers,

Bruce

The following changes since commit 81f9e815d36848761a9dfa94b00ad998bb39a4a6:

  build-appliance-image: Update to master head revision (2019-09-19 20:32:53 
+0100)

are available in the Git repository at:

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

Bruce Ashfield (4):
  linux-yocto/5.2: update to v5.2.13
  linux-yocto/4.19: update to v4.19.72
  linux-yocto/5.2: update to v5.2.14
  linux-yocto/5.2: update to v5.2.16

 .../linux/linux-yocto-rt_4.19.bb  |  6 ++---
 .../linux/linux-yocto-rt_5.2.bb   |  6 ++---
 .../linux/linux-yocto-tiny_4.19.bb|  8 +++
 .../linux/linux-yocto-tiny_5.2.bb |  8 +++
 meta/recipes-kernel/linux/linux-yocto_4.19.bb | 20 -
 meta/recipes-kernel/linux/linux-yocto_5.2.bb  | 22 +--
 6 files changed, 35 insertions(+), 35 deletions(-)

-- 
2.19.1

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


Re: [OE-core] [PATCH 1/1] cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

2019-09-24 Thread Randy MacLeod

On 9/24/19 4:15 AM, Robert Yang wrote:

It checked whether busybox is in pn-buildlist and task-depends.dot after run
"bitbake -g core-image-minimal", but busybox is not a must for
core-image-minimal, 


You should explain why busybox is not a must since
usually busybox is the key package in c-i-m.

We have some images that deliberately exclude busybox and
I think this was found by using:
echo "BB_NO_NETWORK = \"0\"" >> conf/local.conf
echo "PNWHITELIST_LAYERS_remove = \"selftest\"" >> conf/local.conf
echo "SANITY_TESTED_DISTROS = \"\"" >> conf/local.conf
echo "DISTRO = \"wrlinux-graphics\"" >> conf/local.conf
echo "VIRTUAL-RUNTIME_no-busybox_forcevariable = \"\"" >> conf/local.conf
echo "PNWHITELIST_meta-layer0 += 'layerappendtest'" >> conf/local.conf

so mention that in general and VIRTUAL-RUNTIME_no-busybox_forcevariable
specifically.


so the case may fail when busybox is not present, this case
is used for checking "bitbake -g", so check core-image-minimal itself is more
reliable.


It's true that busybox may not be present but we want to
test that some package is in the image, not just that the
task-depends.dot file exists and mentions c-i-m.
How about searching for 'base-files' ?

../Randy



Signed-off-by: Robert Yang 
---
  meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py 
b/meta/lib/oeqa/selftest/cases/bbtests.py
index 8e59baf..4103396 100644
--- a/meta/lib/oeqa/selftest/cases/bbtests.py
+++ b/meta/lib/oeqa/selftest/cases/bbtests.py
@@ -118,11 +118,12 @@ class BitbakeTests(OESelftestTestCase):
  self.assertIn(task, result.output, msg="Couldn't find %s task.")
  
  def test_bitbake_g(self):

-result = bitbake('-g core-image-minimal')
+recipe = 'core-image-minimal'
+result = bitbake('-g %s' % recipe)
  for f in ['pn-buildlist', 'task-depends.dot']:
  self.addCleanup(os.remove, f)
  self.assertTrue('Task dependencies saved to \'task-depends.dot\'' in result.output, msg = 
"No task dependency \"task-depends.dot\" file was generated for the given task 
target. bitbake output: %s" % result.output)
-self.assertTrue('busybox' in ftools.read_file(os.path.join(self.builddir, 
'task-depends.dot')), msg = "No \"busybox\" dependency found in task-depends.dot 
file.")
+self.assertTrue(recipe in ftools.read_file(os.path.join(self.builddir, 
'task-depends.dot')), msg = "No \"%s\" dependency found in task-depends.dot 
file." % recipe)
  
  def test_image_manifest(self):

  bitbake('core-image-minimal')




--
# Randy MacLeod
# Wind River Linux
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] Keeping multiconfig config files in sync

2019-09-24 Thread chris.laplante--- via Openembedded-core
Hello,

Thanks to Richard and others recent hard work, multiconfig is poised to become 
much more practical in YP 3.0.

One thing I'm not clear on, however, is how it will work in a team environment. 
If I have recipes with multiconfig dependencies, then I must ensure that anyone 
else using those recipes has the exact same set of multiconfig config files 
defined in their build/conf/multiconfig/ directory. Or at least, compatible 
multiconfigs with matching names.

Since these files are under build/conf/, it's not immediately clear how one 
might version control these. Unless I'm mistaken and there is a better way, as 
a workaround I will probably put the multiconfigs in a layer and instruct 
developers to symlink build/conf/multiconfig to there.

Any insight?

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


[OE-core] [PATCH] opkg: remove redundant systemd inherit

2019-09-24 Thread Ross Burton
The service file was removed in oe-core 23dcf7ea but the inherit was not.

Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/opkg/opkg_0.4.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg/opkg_0.4.1.bb 
b/meta/recipes-devtools/opkg/opkg_0.4.1.bb
index 62708628994..1fda6e3bdac 100644
--- a/meta/recipes-devtools/opkg/opkg_0.4.1.bb
+++ b/meta/recipes-devtools/opkg/opkg_0.4.1.bb
@@ -25,7 +25,7 @@ SRC_URI[sha256sum] = 
"45ac1e037d3877f635d883f8a555e172883a25d3eeb7986c75890fdd31
 # PTEST_PATH ?= "${libdir}/${BPN}/ptest"
 PACKAGES =+ "libopkg"
 
-inherit autotools pkgconfig systemd ptest
+inherit autotools pkgconfig ptest
 
 target_localstatedir := "${localstatedir}"
 OPKGLIBDIR ??= "${target_localstatedir}/lib"
-- 
2.20.1

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


Re: [OE-core] [PATCH] nativesdk-dnf: ensure installed systemd files are shipped

2019-09-24 Thread Ross Burton

On 24/09/2019 15:01, Trevor Gamblin wrote:

From: Trevor Gamblin 

"bitbake nativesdk-dnf" throws a QA warning if the contents of
${base_libdir}/systemd/system/ are not shipped, so add them.


Looks like a bug in systemd.bbclass, not this recipe.  For example if I 
enable nativesdk for acpid:


ERROR: nativesdk-acpid-2.0.32-r0 do_package: QA Issue: nativesdk-acpid: 
Files/directories were installed but not shipped in any package:

  /opt/poky/2.7+snapshot/sysroots/x86_64-pokysdk-linux/lib
  /opt/poky/2.7+snapshot/sysroots/x86_64-pokysdk-linux/lib/systemd
  /opt/poky/2.7+snapshot/sysroots/x86_64-pokysdk-linux/lib/systemd/system

/opt/poky/2.7+snapshot/sysroots/x86_64-pokysdk-linux/lib/systemd/system/acpid.service

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


Re: [OE-core] [warrior][PATCH] kernel-uboot: compress arm64 kernels

2019-09-24 Thread Randy MacLeod

On 9/24/19 3:23 AM, Bedel, Alban wrote:

On Tue, 2019-09-03 at 09:41 +, Bedel, Alban wrote:

On Wed, 2019-07-31 at 13:53 +, Bedel, Alban wrote:

AArch64 images are not self-decompressing, thus usually much
larger.
Boot times can be reduced by compressing them in FIT and uImages.

This commit is a backport of commit a725d188b5 (kernel-uboot:
compress
arm64 kernels) and commit 60bc7e180e (kernel-uboot: remove useless
special casing of arm64 Image) from master. Both commit were melted
into one to avoid some useless churn.


Was this patch overlooked, or is there a reason it is not considered
in
the next round of update for warrior? Without this patch kernel
images
are too large to fit in the flash of the system I'm using.
Furthermore
it is not trivial to fix this in my own layer.


Please, I really like to get an answer here. I'm fine if there is a
reason why this patch is not considered for warrior, but just getting
ignored is very frustrating.


Add Armin, who maintains warrior.
It seems like he just missed the thread.

There was a 50 patch update posted for warrior last night
but I don't see this patch mentioned.

../Randy


Alban





--
# Randy MacLeod
# Wind River Linux
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] Yocto Project Status WW39’19

2019-09-24 Thread Stephen K Jolley
Current Dev Position: YP 2.8 M4 Feature Freeze

Next Deadline: YP 3.0 Final Release 25th Oct

SWAT Team Rotation:

   -

   SWAT lead is currently: Amanda
   -

   SWAT team rotation: Amanda -> Chen on Sept. 27, 2019
   -

   SWAT team rotation: Chen -> Armin on Oct. 4, 2019
   -

   https://wiki.yoctoproject.org/wiki/Yocto_Build_Failure_Swat_Team


Next Team Meetings:

   -

   Bug Triage meeting Thursday Sept 26th at 7:30am PDT (
   https://zoom.us/j/454367603)
   -

   Monthly Project Meeting Tuesday Oct. 1st at 8am PDT (
   https://zoom.us/j/990892712) 
   -

   Weekly Engineering Sync Tuesday Sept. 24th at 8am PDT (
   https://zoom.us/j/990892712) 
   -

   Twitch - Next event is Tuesday Octt. 8th at 8am PDT (
   https://www.twitch.tv/yocto_project)


Key Status/Updates:

   -

   We’re now in feature freeze for 3.0 and working on bug fixing for final
   release
   -

   We have built M3 and this is due from QA on Thursday
   -

   The main unresolved issue for 3.0 is with the hash equivalence server
   causing problems with the eSDK and this issue is being investigated but
   can’t be reproduced as yet.
   -

   We also want to find a better solution to the systemd timeout problem
   before final release.
   -

   A significant performance problem has been found on the autobuilder
   where some builds are scaling in time badly as the sstate cache grows,
   taking 12 hours or more in some cases. Unfortunately nobody seems motivated
   to help work on this kind of issue.
   -

   The plan for M4 is to resolve the above issues and any issues raised
   from M3 QA, then proceed with the main release rc builds. The first M4
   build is scheduled for next week.
   -

   If anyone has any status items for the project they’d like to add to the
   weekly reports, please email Richard+Stephen.


Planned Releases for YP 3.0 {2.8}:

   -

   M3 Release 6th Sept
   -

   M4 Cutoff 30th Sept - this will be YP 3.0.
   -

   YP 3.0 {2.8 (M4)} Final Release 25th Oct


Planned upcoming dot releases:

   -

   YP 2.7.2 (Warrior) is planned for after YP 3.0 release.
   -

   YP 2.6.4 (Thud) is planned for after YP 2.7.2  release.


Tracking Metrics:

   -

   WDD 2505 (last week 2449) (
   https://wiki.yoctoproject.org/charts/combo.html)
   -

   Poky Patch Metrics
   -

  Total patches found: 1427 (last week 1424)
  -

  Patches in the Pending State: 587 (41%) [last week 587 (41%)]


Key Status Links for YP:

https://wiki.yoctoproject.org/wiki/Yocto_Project_v2.8_Status

https://wiki.yoctoproject.org/wiki/Yocto_2.8_Schedule

https://wiki.yoctoproject.org/wiki/Yocto_2.8_Features

The Yocto Project’s technical governance is through its Technical Steering
Committee, more information is available at:

https://wiki.yoctoproject.org/wiki/TSC

The Status reports are now stored on the wiki at:
https://wiki.yoctoproject.org/wiki/Weekly_Status

[If anyone has suggestions for other information you’d like to see on this
weekly status update, let us know!]

-- 

Thanks,



*Stephen K. Jolley*

*Yocto Project Program Manager*

*7867 SW Bayberry Dr., Beaverton, OR 97007*

(*Cell*:(208) 244-4460

* *Email*: *stephen.k.jol...@intel.com
*
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] nativesdk-dnf: ensure installed systemd files are shipped

2019-09-24 Thread Trevor Gamblin
From: Trevor Gamblin 

"bitbake nativesdk-dnf" throws a QA warning if the contents of
${base_libdir}/systemd/system/ are not shipped, so add them.

Warning log:

WARNING: nativesdk-dnf-4.2.2-r0 do_package: QA Issue: nativesdk-dnf: 
Files/directories were installed but not shipped in any package:
  /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib
  /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd
  /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic.timer
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-makecache.timer
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic.service
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-makecache.service
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-notifyonly.timer
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-download.service
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-download.timer
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-notifyonly.service
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-install.timer
  
/usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-install.service
Please set FILES such that these items are packaged. Alternatively if they 
are unneeded, avoid installing them or delete them within do_install.
nativesdk-dnf: 13 installed and not shipped files. [installed-vs-shipped]

Signed-off-by: Trevor Gamblin 
---
 meta/recipes-devtools/dnf/dnf_4.2.2.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/dnf/dnf_4.2.2.bb 
b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
index c3332c..7ea454a8f6 100644
--- a/meta/recipes-devtools/dnf/dnf_4.2.2.bb
+++ b/meta/recipes-devtools/dnf/dnf_4.2.2.bb
@@ -74,6 +74,9 @@ do_install_append_class-nativesdk() {
 RPM_NO_CHROOT_FOR_SCRIPTS=1 
 }
 
+# Ensure installed systemd files are shipped when building nativesdk-dnf
+FILES_${PN} += "${base_libdir}/systemd/system/*"
+
 SYSTEMD_SERVICE_${PN} = "dnf-makecache.service dnf-makecache.timer \
  dnf-automatic.service dnf-automatic.timer \
  dnf-automatic-download.service 
dnf-automatic-download.timer \
-- 
2.21.0

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


[OE-core] [thud][PATCH] kernel-devsrc: check for localversion files in the kernel source tree

2019-09-24 Thread Christian Thaler
localversion files are ignored. This might lead to a bad version magic when
building out-of-tree modules via SDK.
(Backport from master 
https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-kernel/linux/kernel-devsrc.bb?id=59fcee90de0cbb5b6b8333ab2b0e36214b174e52)

Signed-off-by: Christian Thaler 
---
 meta/recipes-kernel/linux/kernel-devsrc.bb | 9 +
 1 file changed, 9 insertions(+)

diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb 
b/meta/recipes-kernel/linux/kernel-devsrc.bb
index 361ad21..ec5cf09 100644
--- a/meta/recipes-kernel/linux/kernel-devsrc.bb
+++ b/meta/recipes-kernel/linux/kernel-devsrc.bb
@@ -62,6 +62,12 @@ do_install() {
cd ${S}
cp --parents $(find  -type f -name "Makefile*" -o -name "Kconfig*") 
$kerneldir/build
cp --parents $(find  -type f -name "Build" -o -name "Build.include") 
$kerneldir/build
+
+   # Copy localversion file if any to keep correct version magic after
+   # modules_prepare.
+   if [ -f *localversion* ]; then
+   cp *localversion* $kerneldir/build
+   fi
 )
 
 # then drop all but the needed Makefiles/Kconfig files
@@ -213,6 +219,9 @@ do_install() {
 
 # required to build scripts/selinux/genheaders/genheaders
 cp -a --parents security/selinux/include/* $kerneldir/build/
+
+   # copy any localversion files
+   cp -a localversion* $kerneldir/build/ 2>/dev/null || :
 )
 
 # Make sure the Makefile and version.h have a matching timestamp so that
-- 
2.7.4

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


[OE-core] [sumo][PATCH v2 4/4] cross-localedef-native: Fix build for hosts with recent glibc

2019-09-24 Thread Andreas Müller
Fixes:
| argp-help.c:(.text+0x1f51): undefined reference to `_IO_fwide'
| collect2: error: ld returned 1 exit status
| make: *** [Makefile:60: localedef] Error 1

Signed-off-by: Andreas Müller 
---
 .../glibc/cross-localedef-native_2.27.bb  |  1 +
 ...ll-_IO_fwide-if-_LIBC-is-not-defined.patch | 78 +++
 2 files changed, 79 insertions(+)
 create mode 100644 
meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch

diff --git a/meta/recipes-core/glibc/cross-localedef-native_2.27.bb 
b/meta/recipes-core/glibc/cross-localedef-native_2.27.bb
index 5e92eb71d55..a999d35201a 100644
--- a/meta/recipes-core/glibc/cross-localedef-native_2.27.bb
+++ b/meta/recipes-core/glibc/cross-localedef-native_2.27.bb
@@ -35,6 +35,7 @@ SRC_URI = "${GLIBC_GIT_URI};branch=${SRCBRANCH};name=glibc \
file://0021-eglibc-Install-PIC-archives.patch \

file://0022-eglibc-Forward-port-cross-locale-generation-support.patch \
file://0023-Define-DUMMY_LOCALE_T-if-not-defined.patch \
+   
file://0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch \
file://archive-path.patch \
 "
 # Makes for a rather long rev (22 characters), but...
diff --git 
a/meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
 
b/meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
new file mode 100644
index 000..edeb27da0c1
--- /dev/null
+++ 
b/meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
@@ -0,0 +1,78 @@
+From 3a67e81d7527363a96af095a5af03b6201b82e9d Mon Sep 17 00:00:00 2001
+From: Charles-Antoine Couret 
+Date: Thu, 29 Nov 2018 17:56:55 +
+Subject: [PATCH] argp: do not call _IO_fwide() if _LIBC is not defined
+
+_IO_fwide() is defined in libio.h file. This file is included only
+when _LIBC is defined.
+
+So, in case of compilation of these files without _LIBC definition,
+the compilation failed due to this unknown function.
+
+Now this function is called when libio.h file is included.
+
+(Change merged from gnulib.  Tested on x86_64.)
+
+   * argp/argp-fmtstream.c (__argp_fmtstream_update): Use [_LIBC]
+   conditional on calls to _IO_fwide and putwc_unlocked.  (Merge from
+   gnulib.)
+   * argp/argp-help.c (__argp_failure): Likewise.
+   
+Upstream-Status: Backport [1]
+
+[1] 
https://sourceware.org/git/?p=glibc.git;a=commit;h=3a67e81d7527363a96af095a5af03b6201b82e9d
+
+Signed-off-by: Andreas Müller 
+---
+ ChangeLog | 7 +++
+ argp/argp-fmtstream.c | 4 
+ argp/argp-help.c  | 2 ++
+ 3 files changed, 13 insertions(+)
+
+diff --git a/argp/argp-fmtstream.c b/argp/argp-fmtstream.c
+index e43a0c7cf1..e9e4c0e5cc 100644
+--- a/argp/argp-fmtstream.c
 b/argp/argp-fmtstream.c
+@@ -149,9 +149,11 @@ __argp_fmtstream_update (argp_fmtstream_t fs)
+ size_t i;
+ for (i = 0; i < pad; i++)
+   {
++#ifdef _LIBC
+ if (_IO_fwide (fs->stream, 0) > 0)
+   putwc_unlocked (L' ', fs->stream);
+ else
++#endif
+   putc_unlocked (' ', fs->stream);
+   }
+   }
+@@ -312,9 +314,11 @@ __argp_fmtstream_update (argp_fmtstream_t fs)
+ *nl++ = ' ';
+ else
+   for (i = 0; i < fs->wmargin; ++i)
++#ifdef _LIBC
+ if (_IO_fwide (fs->stream, 0) > 0)
+   putwc_unlocked (L' ', fs->stream);
+ else
++#endif
+   putc_unlocked (' ', fs->stream);
+ 
+ /* Copy the tail of the original buffer into the current buffer
+diff --git a/argp/argp-help.c b/argp/argp-help.c
+index 2b6b0775d6..a17260378c 100644
+--- a/argp/argp-help.c
 b/argp/argp-help.c
+@@ -1873,9 +1873,11 @@ __argp_failure (const struct argp_state *state, int 
status, int errnum,
+ #endif
+   }
+ 
++#ifdef _LIBC
+ if (_IO_fwide (stream, 0) > 0)
+   putwc_unlocked (L'\n', stream);
+ else
++#endif
+   putc_unlocked ('\n', stream);
+ 
+ #if _LIBC || (HAVE_FLOCKFILE && HAVE_FUNLOCKFILE)
+-- 
+2.21.0
+
-- 
2.21.0

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


[OE-core] [sumo][PATCH v2 3/4] glib-2.0: Add patch from upstream to fix build with host gcc9

2019-09-24 Thread Andreas Müller
Signed-off-by: Andreas Müller 
---
 ...11-gdbus-Avoid-printing-null-strings.patch | 77 +++
 meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb |  1 +
 2 files changed, 78 insertions(+)
 create mode 100644 
meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch

diff --git 
a/meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
 
b/meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
new file mode 100644
index 000..19085d0b789
--- /dev/null
+++ 
b/meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
@@ -0,0 +1,77 @@
+From 566e1d61a500267c7849ad0b2552feec9c9a29a6 Mon Sep 17 00:00:00 2001
+From: Ernestas Kulik 
+Date: Tue, 29 Jan 2019 09:50:46 +0100
+Subject: [PATCH] gdbus: Avoid printing null strings
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This mostly affects the 2.56 branch, but, given that GCC 9 is being
+stricter about passing null string pointers to printf-like functions, it
+might make sense to proactively fix such calls.
+
+gdbusauth.c: In function '_g_dbus_auth_run_server':
+gdbusauth.c:1302:11: error: '%s' directive argument is null
+[-Werror=format-overflow=]
+ 1302 |   debug_print ("SERVER: WaitingForBegin, read '%s'",
+ line);
+   |
+
+gdbusmessage.c: In function ‘g_dbus_message_to_blob’:
+gdbusmessage.c:2730:30: error: ‘%s’ directive argument is null 
[-Werror=format-overflow=]
+ 2730 |   tupled_signature_str = g_strdup_printf ("(%s)", signature_str);
+  |
+
+Upstream-Status: Accepted
+
+[1] 
https://gitlab.gnome.org/GNOME/glib/commit/566e1d61a500267c7849ad0b2552feec9c9a29a6
+
+Signed-off-by: Andreas Müller 
+---
+ gio/gdbusauth.c| 2 +-
+ gio/gdbusmessage.c | 5 ++---
+ 2 files changed, 3 insertions(+), 4 deletions(-)
+
+diff --git a/gio/gdbusauth.c b/gio/gdbusauth.c
+index 1f8ea8057..752ec23fc 100644
+--- a/gio/gdbusauth.c
 b/gio/gdbusauth.c
+@@ -1272,9 +1272,9 @@ _g_dbus_auth_run_server (GDBusAuth  *auth,
+ &line_length,
+ cancellable,
+ error);
+-  debug_print ("SERVER: WaitingForBegin, read '%s'", line);
+   if (line == NULL)
+ goto out;
++  debug_print ("SERVER: WaitingForBegin, read '%s'", line);
+   if (g_strcmp0 (line, "BEGIN") == 0)
+ {
+   /* YAY, done! */
+diff --git a/gio/gdbusmessage.c b/gio/gdbusmessage.c
+index 3221b925d..3a1a1f9e9 100644
+--- a/gio/gdbusmessage.c
 b/gio/gdbusmessage.c
+@@ -2731,7 +2731,6 @@ g_dbus_message_to_blob (GDBusMessage  *message,
+   if (message->body != NULL)
+ {
+   gchar *tupled_signature_str;
+-  tupled_signature_str = g_strdup_printf ("(%s)", signature_str);
+   if (signature == NULL)
+ {
+   g_set_error (error,
+@@ -2739,10 +2738,10 @@ g_dbus_message_to_blob (GDBusMessage  *message,
+G_IO_ERROR_INVALID_ARGUMENT,
+_("Message body has signature “%s” but there is no 
signature header"),
+signature_str);
+-  g_free (tupled_signature_str);
+   goto out;
+ }
+-  else if (g_strcmp0 (tupled_signature_str, g_variant_get_type_string 
(message->body)) != 0)
++  tupled_signature_str = g_strdup_printf ("(%s)", signature_str);
++  if (g_strcmp0 (tupled_signature_str, g_variant_get_type_string 
(message->body)) != 0)
+ {
+   g_set_error (error,
+G_IO_ERROR,
+-- 
+2.21.0
+
diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb 
b/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb
index faf594569d6..35fd86819bf 100644
--- a/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb
+++ b/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb
@@ -16,6 +16,7 @@ SRC_URI = "${GNOME_MIRROR}/glib/${SHRT_VER}/glib-${PV}.tar.xz 
\
file://0001-Do-not-ignore-return-value-of-write.patch \
file://0001-Test-for-pthread_getname_np-before-using-it.patch \
file://0010-Do-not-hardcode-python-path-into-various-tools.patch \
+   file://0011-gdbus-Avoid-printing-null-strings.patch \
"
 
 SRC_URI_append_class-native = " file://relocate-modules.patch"
-- 
2.21.0

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


[OE-core] [sumo][PATCH v2 0/4] Fix build with host gcc9 / latest glibc

2019-09-24 Thread Andreas Müller
Build errors popped up with Fedora 30

V1 -> V2:
Corrected Upstream-Status

Andreas Müller (3):
  elfutils(-native): Add backported patch to fix build with host gcc9
  glib-2.0: Add patch from upstream to fix build with host gcc9
  cross-localedef-native: Fix build for hosts with recent glibc

Khem Raj (1):
  bison: Fix build break with glibc 2.28

 ...11-gdbus-Avoid-printing-null-strings.patch | 77 ++
 meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb |  1 +
 .../glibc/cross-localedef-native_2.27.bb  |  1 +
 ...ll-_IO_fwide-if-_LIBC-is-not-defined.patch | 78 +++
 .../recipes-devtools/bison/bison/gnulib.patch | 21 +
 meta/recipes-devtools/bison/bison_3.0.4.bb|  1 +
 .../elfutils/elfutils_0.170.bb|  1 +
 ...-fsize-and-msize-with-const-attribut.patch | 42 ++
 8 files changed, 222 insertions(+)
 create mode 100644 
meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
 create mode 100644 
meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
 create mode 100644 meta/recipes-devtools/bison/bison/gnulib.patch
 create mode 100644 
meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch

-- 
2.21.0

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


[OE-core] [sumo][PATCH v2 2/4] bison: Fix build break with glibc 2.28

2019-09-24 Thread Andreas Müller
From: Khem Raj 

(From OE-Core rev: 741415c58b356576428c26efd081212eb989)

Signed-off-by: Khem Raj 
Signed-off-by: Richard Purdie 
---
 .../recipes-devtools/bison/bison/gnulib.patch | 21 +++
 meta/recipes-devtools/bison/bison_3.0.4.bb|  1 +
 2 files changed, 22 insertions(+)
 create mode 100644 meta/recipes-devtools/bison/bison/gnulib.patch

diff --git a/meta/recipes-devtools/bison/bison/gnulib.patch 
b/meta/recipes-devtools/bison/bison/gnulib.patch
new file mode 100644
index 000..7eaf0ce0fa9
--- /dev/null
+++ b/meta/recipes-devtools/bison/bison/gnulib.patch
@@ -0,0 +1,21 @@
+Fix gnulib issues found with glibc 2.28 libio.h removal
+
+see
+https://lists.gnu.org/r/bug-gnulib/2018-03/msg0.html
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+
+Index: bison-3.0.4/lib/fseterr.c
+===
+--- bison-3.0.4.orig/lib/fseterr.c
 bison-3.0.4/lib/fseterr.c
+@@ -29,7 +29,7 @@ fseterr (FILE *fp)
+   /* Most systems provide FILE as a struct and the necessary bitmask in
+  , because they need it for implementing getc() and putc() as
+  fast macros.  */
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   fp->_flags |= _IO_ERR_SEEN;
+ #elif defined __sferror || defined __DragonFly__ /* FreeBSD, NetBSD, OpenBSD, 
DragonFly, Mac OS X, Cygwin */
+   fp_->_flags |= __SERR;
diff --git a/meta/recipes-devtools/bison/bison_3.0.4.bb 
b/meta/recipes-devtools/bison/bison_3.0.4.bb
index 58728f5021f..cc155f0fbe3 100644
--- a/meta/recipes-devtools/bison/bison_3.0.4.bb
+++ b/meta/recipes-devtools/bison/bison_3.0.4.bb
@@ -14,6 +14,7 @@ SRC_URI = "${GNU_MIRROR}/bison/bison-${PV}.tar.xz \
file://dont-depend-on-help2man.patch.patch \
file://0001-src-local.mk-fix-parallel-issue.patch \
file://add-with-bisonlocaledir.patch \
+   file://gnulib.patch \
 "
 
 # No point in hardcoding path to m4, just use PATH
-- 
2.21.0

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


[OE-core] [sumo][PATCH v2 1/4] elfutils(-native): Add backported patch to fix build with host gcc9

2019-09-24 Thread Andreas Müller
Signed-off-by: Andreas Müller 
---
 .../elfutils/elfutils_0.170.bb|  1 +
 ...-fsize-and-msize-with-const-attribut.patch | 42 +++
 2 files changed, 43 insertions(+)
 create mode 100644 
meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch

diff --git a/meta/recipes-devtools/elfutils/elfutils_0.170.bb 
b/meta/recipes-devtools/elfutils/elfutils_0.170.bb
index 14ac22ee643..48f14286203 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.170.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.170.bb
@@ -25,6 +25,7 @@ SRC_URI = 
"https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
file://debian/0003-Add-mips-n64-relocation-format-hack.patch \
file://0001-Use-fallthrough-attribute.patch \

file://0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch \
+   
file://0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch \
"
 SRC_URI_append_libc-musl = " 
file://0008-build-Provide-alternatives-for-glibc-assumptions-hel.patch"
 SRC_URI[md5sum] = "03599aee98c9b726c7a732a2dd0245d5"
diff --git 
a/meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch
 
b/meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch
new file mode 100644
index 000..b5f8fd417aa
--- /dev/null
+++ 
b/meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch
@@ -0,0 +1,42 @@
+From a01938d584b91e747167bb4b3f30ec300c4d6e43 Mon Sep 17 00:00:00 2001
+From: Mark Wielaard 
+Date: Fri, 16 Nov 2018 09:27:00 +0100
+Subject: [PATCH] libelf: Mark both fsize and msize with const attribute.
+
+GCC9 -Wmissing-attributes pointed out that although we alias the fsize
+and msize functions only fsize was marked as const. Fix by also marking
+the msize definition as const.
+
+https://sourceware.org/bugzilla/show_bug.cgi?id=23884
+
+Signed-off-by: Mark Wielaard 
+
+Upstream-Status: Backport [1]
+
+[1] 
https://sourceware.org/git/?p=elfutils.git;a=commit;h=a01938d584b91e747167bb4b3f30ec300c4d6e43
+
+Signed-off-by: Andreas Müller 
+---
+ libelf/ChangeLog | 5 +
+ libelf/libelfP.h | 4 ++--
+ 2 files changed, 7 insertions(+), 2 deletions(-)
+
+diff --git a/libelf/libelfP.h b/libelf/libelfP.h
+index fa6d55d8..9f3e8e9d 100644
+--- a/libelf/libelfP.h
 b/libelf/libelfP.h
+@@ -461,9 +461,9 @@ extern Elf_Type __libelf_data_type (Elf *elf, int sh_type, 
GElf_Xword align)
+These functions cannot be marked internal since they are aliases
+of the export elfXX_fsize functions.*/
+ extern size_t __elf32_msize (Elf_Type __type, size_t __count,
+-   unsigned int __version);
++   unsigned int __version) __const_attribute__;
+ extern size_t __elf64_msize (Elf_Type __type, size_t __count,
+-   unsigned int __version);
++   unsigned int __version) __const_attribute__;
+ 
+ 
+ /* Create Elf descriptor from memory image.  */
+-- 
+2.21.0
+
-- 
2.21.0

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


[OE-core] ✗ patchtest: failure for Fix build with host gcc9 / latest glibc

2019-09-24 Thread Patchwork
== Series Details ==

Series: Fix build with host gcc9 / latest glibc
Revision: 1
URL   : https://patchwork.openembedded.org/series/20136/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Upstream-Status is in incorrect format 
[test_upstream_status_presence_format] 
  Suggested fixFix Upstream-Status format in 
0011-gdbus-Avoid-printing-null-strings.patch
  Current  Upstream-Status: Applied [1]
  Standard format  Upstream-Status: 
  Valid status Pending, Accepted, Backport, Denied, Inappropriate [reason], 
Submitted [where]



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

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


Re: [OE-core] [PATCH V2] mdadm: fix do_package failed when changed local.conf but not cleaned

2019-09-24 Thread Ross Burton

On 24/09/2019 07:00, changqing...@windriver.com wrote:

From: Changqing Li 

reproduce steps:
1. add DISTRO_FEATURE_append = 'usrmerge' in local.conf
2. bitbake mdadm --success
3. remove DISTRO_FEATURE_append = 'usrmerge' from local.conf
4. bitbake mdadm  -- failed when do_package

it is not proper to change source Makefile during do_install by sed,
change to add patch for it.

[YOCTO #13493]

Signed-off-by: Changqing Li 
---
  .../mdadm/files/0001-mdadm-support-usrmerge.patch  | 46 ++
  meta/recipes-extended/mdadm/mdadm_4.1.bb   |  7 ++--
  2 files changed, 50 insertions(+), 3 deletions(-)
  create mode 100644 
meta/recipes-extended/mdadm/files/0001-mdadm-support-usrmerge.patch

diff --git 
a/meta/recipes-extended/mdadm/files/0001-mdadm-support-usrmerge.patch 
b/meta/recipes-extended/mdadm/files/0001-mdadm-support-usrmerge.patch
new file mode 100644
index 000..f1aa119
--- /dev/null
+++ b/meta/recipes-extended/mdadm/files/0001-mdadm-support-usrmerge.patch
@@ -0,0 +1,46 @@
+From 2c7ccea05d7cf9dde48d735faa511d1b06c14878 Mon Sep 17 00:00:00 2001
+From: Changqing Li 
+Date: Thu, 5 Sep 2019 18:03:11 +0800
+Subject: [PATCH] mdadm: support usrmerge
+
+Upstream-Status: Inappropriate[oe-specific]
+
+Signed-off-by: Changqing Li 
+---
+ Makefile | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/Makefile b/Makefile
+index 4839001..466e960 100644
+--- a/Makefile
 b/Makefile
+@@ -88,7 +88,7 @@ MAP_PATH = $(MAP_DIR)/$(MAP_FILE)
+ MDMON_DIR = $(RUN_DIR)
+ # place for autoreplace cookies
+ FAILED_SLOTS_DIR = $(RUN_DIR)/failed-slots
+-SYSTEMD_DIR=/lib/systemd/system
++SYSTEMD_DIR=${SYSTEMD_UNITDIR}/system
+ LIB_DIR=/usr/libexec/mdadm
+
+ COROSYNC:=$(shell [ -f $(SYSROOT)/usr/include/corosync/cmap.h ] || echo 
-DNO_COROSYNC)
+@@ -120,7 +120,7 @@ LDLIBS=-ldl
+
+ INSTALL = /usr/bin/install
+ DESTDIR =
+-BINDIR  = /sbin
++BINDIR  = ${BASE_SBINDIR}
+ MANDIR  = /usr/share/man
+ MAN4DIR = $(MANDIR)/man4
+ MAN5DIR = $(MANDIR)/man5
+@@ -128,7 +128,7 @@ MAN8DIR = $(MANDIR)/man8
+
+ UDEVDIR := $(shell $(PKG_CONFIG) --variable=udevdir udev 2>/dev/null)
+ ifndef UDEVDIR
+- UDEVDIR = /lib/udev
++ UDEVDIR = ${NONARCH_BASE_LIBDIR}/udev
+ endif
+
+ ifeq (,$(findstring s,$(MAKEFLAGS)))
+--
+2.7.4
+
@@ -43,13 +44,13 @@ CFLAGS_append_powerpc64 = ' -D__SANE_USERSPACE_TYPES__'
  CFLAGS_append_mipsarchn64 = ' -D__SANE_USERSPACE_TYPES__'
  CFLAGS_append_mipsarchn32 = ' -D__SANE_USERSPACE_TYPES__'
  
-EXTRA_OEMAKE = 'CHECK_RUN_DIR=0 CXFLAGS="${CFLAGS}"'

+EXTRA_OEMAKE = 'CHECK_RUN_DIR=0 CXFLAGS="${CFLAGS}" 
BASE_SBINDIR="${base_sbindir}" \
+NONARCH_BASE_LIBDIR="${nonarch_base_libdir}" \
+SYSTEMD_UNITDIR="${systemd_unitdir}"'


Can't you just pass BINDIR=${base_sbindir} MANDIR=${docdir}/man 
UDEVDIR=${nonarch_base_libdir}/udev etc, instead of patching the makefile?


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


Re: [OE-core] [PATCH 1/1] expect: Fix buffer overflow error when build in long path

2019-09-24 Thread Ross Burton

On 24/09/2019 08:34, Robert Yang wrote:

+Upstream-Status: Pending [Upstream seems dead]


The web site says to use SourceForge, but that's wrong.  The code is 
(sort of) alive in Fossil at https://core.tcl-lang.org/expect/home so 
please do submit the patch to either their SF or the Fossil instance.


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


[OE-core] [sumo][PATCH 1/4] elfutils(-native): Add backported patch to fix build with host gcc9

2019-09-24 Thread Andreas Müller
Signed-off-by: Andreas Müller 
---
 .../elfutils/elfutils_0.170.bb|  1 +
 ...-fsize-and-msize-with-const-attribut.patch | 42 +++
 2 files changed, 43 insertions(+)
 create mode 100644 
meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch

diff --git a/meta/recipes-devtools/elfutils/elfutils_0.170.bb 
b/meta/recipes-devtools/elfutils/elfutils_0.170.bb
index 14ac22ee643..48f14286203 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.170.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.170.bb
@@ -25,6 +25,7 @@ SRC_URI = 
"https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
file://debian/0003-Add-mips-n64-relocation-format-hack.patch \
file://0001-Use-fallthrough-attribute.patch \

file://0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch \
+   
file://0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch \
"
 SRC_URI_append_libc-musl = " 
file://0008-build-Provide-alternatives-for-glibc-assumptions-hel.patch"
 SRC_URI[md5sum] = "03599aee98c9b726c7a732a2dd0245d5"
diff --git 
a/meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch
 
b/meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch
new file mode 100644
index 000..b5f8fd417aa
--- /dev/null
+++ 
b/meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch
@@ -0,0 +1,42 @@
+From a01938d584b91e747167bb4b3f30ec300c4d6e43 Mon Sep 17 00:00:00 2001
+From: Mark Wielaard 
+Date: Fri, 16 Nov 2018 09:27:00 +0100
+Subject: [PATCH] libelf: Mark both fsize and msize with const attribute.
+
+GCC9 -Wmissing-attributes pointed out that although we alias the fsize
+and msize functions only fsize was marked as const. Fix by also marking
+the msize definition as const.
+
+https://sourceware.org/bugzilla/show_bug.cgi?id=23884
+
+Signed-off-by: Mark Wielaard 
+
+Upstream-Status: Backport [1]
+
+[1] 
https://sourceware.org/git/?p=elfutils.git;a=commit;h=a01938d584b91e747167bb4b3f30ec300c4d6e43
+
+Signed-off-by: Andreas Müller 
+---
+ libelf/ChangeLog | 5 +
+ libelf/libelfP.h | 4 ++--
+ 2 files changed, 7 insertions(+), 2 deletions(-)
+
+diff --git a/libelf/libelfP.h b/libelf/libelfP.h
+index fa6d55d8..9f3e8e9d 100644
+--- a/libelf/libelfP.h
 b/libelf/libelfP.h
+@@ -461,9 +461,9 @@ extern Elf_Type __libelf_data_type (Elf *elf, int sh_type, 
GElf_Xword align)
+These functions cannot be marked internal since they are aliases
+of the export elfXX_fsize functions.*/
+ extern size_t __elf32_msize (Elf_Type __type, size_t __count,
+-   unsigned int __version);
++   unsigned int __version) __const_attribute__;
+ extern size_t __elf64_msize (Elf_Type __type, size_t __count,
+-   unsigned int __version);
++   unsigned int __version) __const_attribute__;
+ 
+ 
+ /* Create Elf descriptor from memory image.  */
+-- 
+2.21.0
+
-- 
2.21.0

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


[OE-core] [sumo][PATCH 4/4] cross-localedef-native: Fix build for hosts with recent glibc

2019-09-24 Thread Andreas Müller
Fixes:
| argp-help.c:(.text+0x1f51): undefined reference to `_IO_fwide'
| collect2: error: ld returned 1 exit status
| make: *** [Makefile:60: localedef] Error 1

Signed-off-by: Andreas Müller 
---
 .../glibc/cross-localedef-native_2.27.bb  |  1 +
 ...ll-_IO_fwide-if-_LIBC-is-not-defined.patch | 78 +++
 2 files changed, 79 insertions(+)
 create mode 100644 
meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch

diff --git a/meta/recipes-core/glibc/cross-localedef-native_2.27.bb 
b/meta/recipes-core/glibc/cross-localedef-native_2.27.bb
index 5e92eb71d55..a999d35201a 100644
--- a/meta/recipes-core/glibc/cross-localedef-native_2.27.bb
+++ b/meta/recipes-core/glibc/cross-localedef-native_2.27.bb
@@ -35,6 +35,7 @@ SRC_URI = "${GLIBC_GIT_URI};branch=${SRCBRANCH};name=glibc \
file://0021-eglibc-Install-PIC-archives.patch \

file://0022-eglibc-Forward-port-cross-locale-generation-support.patch \
file://0023-Define-DUMMY_LOCALE_T-if-not-defined.patch \
+   
file://0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch \
file://archive-path.patch \
 "
 # Makes for a rather long rev (22 characters), but...
diff --git 
a/meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
 
b/meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
new file mode 100644
index 000..edeb27da0c1
--- /dev/null
+++ 
b/meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
@@ -0,0 +1,78 @@
+From 3a67e81d7527363a96af095a5af03b6201b82e9d Mon Sep 17 00:00:00 2001
+From: Charles-Antoine Couret 
+Date: Thu, 29 Nov 2018 17:56:55 +
+Subject: [PATCH] argp: do not call _IO_fwide() if _LIBC is not defined
+
+_IO_fwide() is defined in libio.h file. This file is included only
+when _LIBC is defined.
+
+So, in case of compilation of these files without _LIBC definition,
+the compilation failed due to this unknown function.
+
+Now this function is called when libio.h file is included.
+
+(Change merged from gnulib.  Tested on x86_64.)
+
+   * argp/argp-fmtstream.c (__argp_fmtstream_update): Use [_LIBC]
+   conditional on calls to _IO_fwide and putwc_unlocked.  (Merge from
+   gnulib.)
+   * argp/argp-help.c (__argp_failure): Likewise.
+   
+Upstream-Status: Backport [1]
+
+[1] 
https://sourceware.org/git/?p=glibc.git;a=commit;h=3a67e81d7527363a96af095a5af03b6201b82e9d
+
+Signed-off-by: Andreas Müller 
+---
+ ChangeLog | 7 +++
+ argp/argp-fmtstream.c | 4 
+ argp/argp-help.c  | 2 ++
+ 3 files changed, 13 insertions(+)
+
+diff --git a/argp/argp-fmtstream.c b/argp/argp-fmtstream.c
+index e43a0c7cf1..e9e4c0e5cc 100644
+--- a/argp/argp-fmtstream.c
 b/argp/argp-fmtstream.c
+@@ -149,9 +149,11 @@ __argp_fmtstream_update (argp_fmtstream_t fs)
+ size_t i;
+ for (i = 0; i < pad; i++)
+   {
++#ifdef _LIBC
+ if (_IO_fwide (fs->stream, 0) > 0)
+   putwc_unlocked (L' ', fs->stream);
+ else
++#endif
+   putc_unlocked (' ', fs->stream);
+   }
+   }
+@@ -312,9 +314,11 @@ __argp_fmtstream_update (argp_fmtstream_t fs)
+ *nl++ = ' ';
+ else
+   for (i = 0; i < fs->wmargin; ++i)
++#ifdef _LIBC
+ if (_IO_fwide (fs->stream, 0) > 0)
+   putwc_unlocked (L' ', fs->stream);
+ else
++#endif
+   putc_unlocked (' ', fs->stream);
+ 
+ /* Copy the tail of the original buffer into the current buffer
+diff --git a/argp/argp-help.c b/argp/argp-help.c
+index 2b6b0775d6..a17260378c 100644
+--- a/argp/argp-help.c
 b/argp/argp-help.c
+@@ -1873,9 +1873,11 @@ __argp_failure (const struct argp_state *state, int 
status, int errnum,
+ #endif
+   }
+ 
++#ifdef _LIBC
+ if (_IO_fwide (stream, 0) > 0)
+   putwc_unlocked (L'\n', stream);
+ else
++#endif
+   putc_unlocked ('\n', stream);
+ 
+ #if _LIBC || (HAVE_FLOCKFILE && HAVE_FUNLOCKFILE)
+-- 
+2.21.0
+
-- 
2.21.0

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


[OE-core] [sumo][PATCH 3/4] glib-2.0: Add patch from upstream to fix build with host gcc9

2019-09-24 Thread Andreas Müller
Signed-off-by: Andreas Müller 
---
 ...11-gdbus-Avoid-printing-null-strings.patch | 77 +++
 meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb |  1 +
 2 files changed, 78 insertions(+)
 create mode 100644 
meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch

diff --git 
a/meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
 
b/meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
new file mode 100644
index 000..043879c85e0
--- /dev/null
+++ 
b/meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
@@ -0,0 +1,77 @@
+From 566e1d61a500267c7849ad0b2552feec9c9a29a6 Mon Sep 17 00:00:00 2001
+From: Ernestas Kulik 
+Date: Tue, 29 Jan 2019 09:50:46 +0100
+Subject: [PATCH] gdbus: Avoid printing null strings
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This mostly affects the 2.56 branch, but, given that GCC 9 is being
+stricter about passing null string pointers to printf-like functions, it
+might make sense to proactively fix such calls.
+
+gdbusauth.c: In function '_g_dbus_auth_run_server':
+gdbusauth.c:1302:11: error: '%s' directive argument is null
+[-Werror=format-overflow=]
+ 1302 |   debug_print ("SERVER: WaitingForBegin, read '%s'",
+ line);
+   |
+
+gdbusmessage.c: In function ‘g_dbus_message_to_blob’:
+gdbusmessage.c:2730:30: error: ‘%s’ directive argument is null 
[-Werror=format-overflow=]
+ 2730 |   tupled_signature_str = g_strdup_printf ("(%s)", signature_str);
+  |
+
+Upstream-Status: Applied [1]
+
+[1] 
https://gitlab.gnome.org/GNOME/glib/commit/566e1d61a500267c7849ad0b2552feec9c9a29a6
+
+Signed-off-by: Andreas Müller 
+---
+ gio/gdbusauth.c| 2 +-
+ gio/gdbusmessage.c | 5 ++---
+ 2 files changed, 3 insertions(+), 4 deletions(-)
+
+diff --git a/gio/gdbusauth.c b/gio/gdbusauth.c
+index 1f8ea8057..752ec23fc 100644
+--- a/gio/gdbusauth.c
 b/gio/gdbusauth.c
+@@ -1272,9 +1272,9 @@ _g_dbus_auth_run_server (GDBusAuth  *auth,
+ &line_length,
+ cancellable,
+ error);
+-  debug_print ("SERVER: WaitingForBegin, read '%s'", line);
+   if (line == NULL)
+ goto out;
++  debug_print ("SERVER: WaitingForBegin, read '%s'", line);
+   if (g_strcmp0 (line, "BEGIN") == 0)
+ {
+   /* YAY, done! */
+diff --git a/gio/gdbusmessage.c b/gio/gdbusmessage.c
+index 3221b925d..3a1a1f9e9 100644
+--- a/gio/gdbusmessage.c
 b/gio/gdbusmessage.c
+@@ -2731,7 +2731,6 @@ g_dbus_message_to_blob (GDBusMessage  *message,
+   if (message->body != NULL)
+ {
+   gchar *tupled_signature_str;
+-  tupled_signature_str = g_strdup_printf ("(%s)", signature_str);
+   if (signature == NULL)
+ {
+   g_set_error (error,
+@@ -2739,10 +2738,10 @@ g_dbus_message_to_blob (GDBusMessage  *message,
+G_IO_ERROR_INVALID_ARGUMENT,
+_("Message body has signature “%s” but there is no 
signature header"),
+signature_str);
+-  g_free (tupled_signature_str);
+   goto out;
+ }
+-  else if (g_strcmp0 (tupled_signature_str, g_variant_get_type_string 
(message->body)) != 0)
++  tupled_signature_str = g_strdup_printf ("(%s)", signature_str);
++  if (g_strcmp0 (tupled_signature_str, g_variant_get_type_string 
(message->body)) != 0)
+ {
+   g_set_error (error,
+G_IO_ERROR,
+-- 
+2.21.0
+
diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb 
b/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb
index faf594569d6..35fd86819bf 100644
--- a/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb
+++ b/meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb
@@ -16,6 +16,7 @@ SRC_URI = "${GNOME_MIRROR}/glib/${SHRT_VER}/glib-${PV}.tar.xz 
\
file://0001-Do-not-ignore-return-value-of-write.patch \
file://0001-Test-for-pthread_getname_np-before-using-it.patch \
file://0010-Do-not-hardcode-python-path-into-various-tools.patch \
+   file://0011-gdbus-Avoid-printing-null-strings.patch \
"
 
 SRC_URI_append_class-native = " file://relocate-modules.patch"
-- 
2.21.0

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


[OE-core] [sumo][PATCH 2/4] bison: Fix build break with glibc 2.28

2019-09-24 Thread Andreas Müller
From: Khem Raj 

(From OE-Core rev: 741415c58b356576428c26efd081212eb989)

Signed-off-by: Khem Raj 
Signed-off-by: Richard Purdie 
---
 .../recipes-devtools/bison/bison/gnulib.patch | 21 +++
 meta/recipes-devtools/bison/bison_3.0.4.bb|  1 +
 2 files changed, 22 insertions(+)
 create mode 100644 meta/recipes-devtools/bison/bison/gnulib.patch

diff --git a/meta/recipes-devtools/bison/bison/gnulib.patch 
b/meta/recipes-devtools/bison/bison/gnulib.patch
new file mode 100644
index 000..7eaf0ce0fa9
--- /dev/null
+++ b/meta/recipes-devtools/bison/bison/gnulib.patch
@@ -0,0 +1,21 @@
+Fix gnulib issues found with glibc 2.28 libio.h removal
+
+see
+https://lists.gnu.org/r/bug-gnulib/2018-03/msg0.html
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+
+Index: bison-3.0.4/lib/fseterr.c
+===
+--- bison-3.0.4.orig/lib/fseterr.c
 bison-3.0.4/lib/fseterr.c
+@@ -29,7 +29,7 @@ fseterr (FILE *fp)
+   /* Most systems provide FILE as a struct and the necessary bitmask in
+  , because they need it for implementing getc() and putc() as
+  fast macros.  */
+-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, 
Haiku, Linux libc5 */
++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, 
Linux libc5 */
+   fp->_flags |= _IO_ERR_SEEN;
+ #elif defined __sferror || defined __DragonFly__ /* FreeBSD, NetBSD, OpenBSD, 
DragonFly, Mac OS X, Cygwin */
+   fp_->_flags |= __SERR;
diff --git a/meta/recipes-devtools/bison/bison_3.0.4.bb 
b/meta/recipes-devtools/bison/bison_3.0.4.bb
index 58728f5021f..cc155f0fbe3 100644
--- a/meta/recipes-devtools/bison/bison_3.0.4.bb
+++ b/meta/recipes-devtools/bison/bison_3.0.4.bb
@@ -14,6 +14,7 @@ SRC_URI = "${GNU_MIRROR}/bison/bison-${PV}.tar.xz \
file://dont-depend-on-help2man.patch.patch \
file://0001-src-local.mk-fix-parallel-issue.patch \
file://add-with-bisonlocaledir.patch \
+   file://gnulib.patch \
 "
 
 # No point in hardcoding path to m4, just use PATH
-- 
2.21.0

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


[OE-core] [sumo][PATCH 0/4] Fix build with host gcc9 / latest glibc

2019-09-24 Thread Andreas Müller
Build errors popped up with Fedora 30

Andreas Müller (3):
  elfutils(-native): Add backported patch to fix build with host gcc9
  glib-2.0: Add patch from upstream to fix build with host gcc9
  cross-localedef-native: Fix build for hosts with recent glibc

Khem Raj (1):
  bison: Fix build break with glibc 2.28

 ...11-gdbus-Avoid-printing-null-strings.patch | 77 ++
 meta/recipes-core/glib-2.0/glib-2.0_2.54.3.bb |  1 +
 .../glibc/cross-localedef-native_2.27.bb  |  1 +
 ...ll-_IO_fwide-if-_LIBC-is-not-defined.patch | 78 +++
 .../recipes-devtools/bison/bison/gnulib.patch | 21 +
 meta/recipes-devtools/bison/bison_3.0.4.bb|  1 +
 .../elfutils/elfutils_0.170.bb|  1 +
 ...-fsize-and-msize-with-const-attribut.patch | 42 ++
 8 files changed, 222 insertions(+)
 create mode 100644 
meta/recipes-core/glib-2.0/glib-2.0/0011-gdbus-Avoid-printing-null-strings.patch
 create mode 100644 
meta/recipes-core/glibc/glibc/0032-argp-do-not-call-_IO_fwide-if-_LIBC-is-not-defined.patch
 create mode 100644 meta/recipes-devtools/bison/bison/gnulib.patch
 create mode 100644 
meta/recipes-devtools/elfutils/files/0009-libelf-Mark-both-fsize-and-msize-with-const-attribut.patch

-- 
2.21.0

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


Re: [OE-core] [RFC][PATCH 0/3] Move rust from meta-rust to oe-core

2019-09-24 Thread Adrian Bunk
On Mon, Sep 23, 2019 at 10:45:05PM -0400, Randy MacLeod wrote:
> I moved the Rust support from meta-rust to oe-core.
> 
> As you'd expect, it's still building rust-hello-world for all qemus 
> (except riscv64 which may never have worked). 
> Rust-hello-world runs for me on qemu:
>x86-64, arm, arm64
> It may run on other machines but that's what I've tested so far.
> 
> We need a list of meta-rust issues that should be resolved 
> in order to get the code accepted in oe-core. I opened an issue 
> to track that:
>https://github.com/meta-rust/meta-rust/issues/251
> but there haven't been any comments yet. I'd rather track
> issues in github than in the YP Bugzilla:
>   https://bugzilla.yoctoproject.org/show_bug.cgi?id=12675
> 
> Here, in no particular order, are some of
> the defects that I believe need to be fixed:
> 
> 1. build system is assumed to be linux-x86_64
>https://github.com/meta-rust/meta-rust/issues/23
> 
> 2. Make rust-native build in parallel (it's very slow!)
>(meta-rust issue/PR to be created by Randy)
> 
> 3. Document developer workflows such as cargo-bitbake.

Is there consensus on how libraries should be handled in general for 
various ecosystems?

Precedent with C, Python and Perl are recipes for each library,
but vendoring seems to be a popular option now.

There are also related topics like how to provide security support
for vendored libraries - this is the main reason why vendored libraries
shipped by upstream are rarely used by distributions in the C ecosystem.

> 4. Build offline ( BB_NO_NETWORK = '1' )
> 
> 5. make rust-hello-world install for qemuriscv64
>https://github.com/meta-rust/meta-rust/issues/252

LLVM in meta-rust is too old for RISC-V.

Using an own LLVM recipe might makes sense for an external layer, but in 
oe-core the llvm recipe that is already there should be used instead of 
another copy of LLVM.

> 6. ... build rustc and cargo for target (nice to have, only).
>https://github.com/meta-rust/meta-rust/issues/81
> 
> 7. Bootstrap from C as an option?
>See: https://bugzilla.yoctoproject.org/show_bug.cgi?id=12675#c2
> 
> 8. Update librsvg to latest (2.46) which uses rust.

2.44 built for me a few months ago with meta-rust.

> What have I missed?
>...

9. Ensure that there are no non-optional dependencies on the target
   librsvg since users will build for targets not supported by Rust.
   The one in webkitgtk seems to be optional or obsolete.

10. Review the whole contents of the layer.
There are several things where I hope there are better solutions,
and I expect that what will land in oe-core will look quite 
different from the current meta-rust.
See item 5 above for an example.

> ../Randy
>...

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

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


[OE-core] oe-core warrior backport request

2019-09-24 Thread Paul Barker
Hi,

Could we get the following patches backported to the warrior branch? We're 
carrying them locally to enable builds on our infrastructure where DEPLOY_DIR 
and WORKDIR are placed on different devices.

f064a581ba license_image: Use new oe.path.copyhardlink() helper
5437efa16f oe.path: Add copyhardlink() helper function

Thanks,

-- 
Paul Barker
Managing Director & Principal Engineer
Beta Five Ltd
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 1/1] archiver.bbclass: Add new mirror archiver mode

2019-09-24 Thread Paul Barker
We define a new method of populating a source mirror using the archiver
bbclass instead of simply copying the contents of the downloads
directory. This allows the archiver features such as copyleft license
filtering and recipe type filtering to be used when preparing a source
mirror.

This new archiver mode is selected by setting `ARCHIVE_MODE[src]` to
'mirror'.

The source mirror mode can either be 'split' (default) or 'combined',
controlled by `ARCHIVER_MODE[mirror]`. Additionally, sources can be
excluded as needed by setting `ARCHIVER_MIRROR_EXCLUDE` to a list of
URI prefixes. These options are described in more detail in the new
entries in the header of archiver.bbclass.

Signed-off-by: Paul Barker 
---
 meta/classes/archiver.bbclass | 136 +-
 1 file changed, 117 insertions(+), 19 deletions(-)

diff --git a/meta/classes/archiver.bbclass b/meta/classes/archiver.bbclass
index 093e2d95af..e54f57fafe 100644
--- a/meta/classes/archiver.bbclass
+++ b/meta/classes/archiver.bbclass
@@ -2,25 +2,42 @@
 # -*- tab-width: 4; c-basic-offset: 4; indent-tabs-mode: nil -*-
 #
 # This bbclass is used for creating archive for:
-# 1) original (or unpacked) source: ARCHIVER_MODE[src] = "original"
-# 2) patched source: ARCHIVER_MODE[src] = "patched" (default)
-# 3) configured source: ARCHIVER_MODE[src] = "configured"
-# 4) The patches between do_unpack and do_patch:
-#ARCHIVER_MODE[diff] = "1"
-#And you can set the one that you'd like to exclude from the diff:
-#ARCHIVER_MODE[diff-exclude] ?= ".pc autom4te.cache patches"
-# 5) The environment data, similar to 'bitbake -e recipe':
-#ARCHIVER_MODE[dumpdata] = "1"
-# 6) The recipe (.bb and .inc): ARCHIVER_MODE[recipe] = "1"
-# 7) Whether output the .src.rpm package:
-#ARCHIVER_MODE[srpm] = "1"
-# 8) Filter the license, the recipe whose license in
-#COPYLEFT_LICENSE_INCLUDE will be included, and in
-#COPYLEFT_LICENSE_EXCLUDE will be excluded.
-#COPYLEFT_LICENSE_INCLUDE = 'GPL* LGPL*'
-#COPYLEFT_LICENSE_EXCLUDE = 'CLOSED Proprietary'
-# 9) The recipe type that will be archived:
-#COPYLEFT_RECIPE_TYPES = 'target'
+#  1) original (or unpacked) source: ARCHIVER_MODE[src] = "original"
+#  2) patched source: ARCHIVER_MODE[src] = "patched" (default)
+#  3) configured source: ARCHIVER_MODE[src] = "configured"
+#  4) source mirror: ARCHIVE_MODE[src] = "mirror"
+#  5) The patches between do_unpack and do_patch:
+# ARCHIVER_MODE[diff] = "1"
+# And you can set the one that you'd like to exclude from the diff:
+# ARCHIVER_MODE[diff-exclude] ?= ".pc autom4te.cache patches"
+#  6) The environment data, similar to 'bitbake -e recipe':
+# ARCHIVER_MODE[dumpdata] = "1"
+#  7) The recipe (.bb and .inc): ARCHIVER_MODE[recipe] = "1"
+#  8) Whether output the .src.rpm package:
+# ARCHIVER_MODE[srpm] = "1"
+#  9) Filter the license, the recipe whose license in
+# COPYLEFT_LICENSE_INCLUDE will be included, and in
+# COPYLEFT_LICENSE_EXCLUDE will be excluded.
+# COPYLEFT_LICENSE_INCLUDE = 'GPL* LGPL*'
+# COPYLEFT_LICENSE_EXCLUDE = 'CLOSED Proprietary'
+# 10) The recipe type that will be archived:
+# COPYLEFT_RECIPE_TYPES = 'target'
+# 11) The source mirror mode:
+# ARCHIVER_MODE[mirror] = "split" (default): Sources are split into
+# per-recipe directories in a similar way to other archiver modes.
+# Post-processing may be required to produce a single mirror directory.
+# This does however allow inspection of duplicate sources and more
+# intelligent handling.
+# ARCHIVER_MODE[mirror] = "combined": All sources are placed into a single
+# directory suitable for direct use as a mirror. Duplicate sources are
+# ignored.
+# 12) Source mirror exclusions:
+# ARCHIVE_MIRROR_EXCLUDE is a list of prefixes to exclude from the mirror.
+# This may be used for sources which you are already publishing yourself
+# (e.g. if the URI starts with 'https://mysite.com/' and your mirror is
+# going to be published to the same site). It may also be used to exclude
+# local files (with the prefix 'file://') if these will be provided as part
+# of an archive of the layers themselves.
 #
 
 # Create archive for all the recipe types
@@ -33,6 +50,7 @@ ARCHIVER_MODE[diff] ?= "0"
 ARCHIVER_MODE[diff-exclude] ?= ".pc autom4te.cache patches"
 ARCHIVER_MODE[dumpdata] ?= "0"
 ARCHIVER_MODE[recipe] ?= "0"
+ARCHIVER_MODE[mirror] ?= "split"
 
 DEPLOY_DIR_SRC ?= "${DEPLOY_DIR}/sources"
 ARCHIVER_TOPDIR ?= "${WORKDIR}/deploy-sources"
@@ -41,6 +59,10 @@ ARCHIVER_RPMTOPDIR ?= "${WORKDIR}/deploy-sources-rpm"
 ARCHIVER_RPMOUTDIR = "${ARCHIVER_RPMTOPDIR}/${TARGET_SYS}/${PF}/"
 ARCHIVER_WORKDIR = "${WORKDIR}/archiver-work/"
 
+# When producing a combined mirror directory, allow duplicates for the case
+# where multiple recipes use the same SRC_URI.
+ARCHIVER_COMBINED_MIRRORDIR = "${ARCHIVER_TOPDIR}/mirror"
+SSTATE_DUPWHITELIST += "${DEPLOY_DIR_SRC}/mirror"
 
 do_dumpdata[dirs] = "${

[OE-core] [PATCH 0/1] Add new mirror archiver mode

2019-09-24 Thread Paul Barker
This patch allows the creation of a full source mirror using the archiver
bbclass. Using the archiver allows us to make use of copyleft license
filtering, recipe type filtering and other features that are not available if
we just copy the contents of the downloads directory itself to create a
mirror. We also get the advantage of sstate caching and we don't have to
throw away the entire downloads directory before building to keep sources for
other builds out of our mirror.

The resulting mirror may be published and used as an entry in PREMIRRORS or
MIRRORS for a subsequent build. The functionality is explained in more detail
in the patch commit message and the additions to the documentation comments.

We've been using this patch in our Oryx Linux distro for a few months now to
confirm that it works as expected. All feedback is welcome, especially from
people who have worked on the archiver bbclass before.

Mirror creation was tested with the following additions to local.conf:

INHERIT += "archiver"
BB_GENERATE_MIRROR_TARBALLS = "1"
ARCHIVER_MODE[src] = "mirror"
ARCHIVER_MODE[mirror] = "combined"
ARCHIVER_MIRROR_EXCLUDE = "file://"
COPYLEFT_LICENSE_INCLUDE = "*"

The resulting mirror directory was moved out of the tmpdir and a rebuild was
tested with the following additions to local.conf:

BB_NO_NETWORK = "1"
INHERIT += "own-mirrors"
SOURCE_MIRROR_URL = "file://${TOPDIR}/mirror"

Paul Barker (1):
  archiver.bbclass: Add new mirror archiver mode

 meta/classes/archiver.bbclass | 136 +-
 1 file changed, 117 insertions(+), 19 deletions(-)

-- 
2.23.0

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


[OE-core] [PATCH 0/1] cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

2019-09-24 Thread Robert Yang
The following changes since commit 95ad5626296380358c8a502a3e04879dab653d78:

  build-appliance-image: Update to master head revision (2019-09-19 20:32:47 
+0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/test
  http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/test

Robert Yang (1):
  cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

 meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

-- 
2.7.4

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


[OE-core] [PATCH 1/1] cases/bbtests.py: test_bitbake_g(): Check core-image-minimal itself

2019-09-24 Thread Robert Yang
It checked whether busybox is in pn-buildlist and task-depends.dot after run
"bitbake -g core-image-minimal", but busybox is not a must for
core-image-minimal, so the case may fail when busybox is not present, this case
is used for checking "bitbake -g", so check core-image-minimal itself is more
reliable.

Signed-off-by: Robert Yang 
---
 meta/lib/oeqa/selftest/cases/bbtests.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py 
b/meta/lib/oeqa/selftest/cases/bbtests.py
index 8e59baf..4103396 100644
--- a/meta/lib/oeqa/selftest/cases/bbtests.py
+++ b/meta/lib/oeqa/selftest/cases/bbtests.py
@@ -118,11 +118,12 @@ class BitbakeTests(OESelftestTestCase):
 self.assertIn(task, result.output, msg="Couldn't find %s task.")
 
 def test_bitbake_g(self):
-result = bitbake('-g core-image-minimal')
+recipe = 'core-image-minimal'
+result = bitbake('-g %s' % recipe)
 for f in ['pn-buildlist', 'task-depends.dot']:
 self.addCleanup(os.remove, f)
 self.assertTrue('Task dependencies saved to \'task-depends.dot\'' in 
result.output, msg = "No task dependency \"task-depends.dot\" file was 
generated for the given task target. bitbake output: %s" % result.output)
-self.assertTrue('busybox' in 
ftools.read_file(os.path.join(self.builddir, 'task-depends.dot')), msg = "No 
\"busybox\" dependency found in task-depends.dot file.")
+self.assertTrue(recipe in ftools.read_file(os.path.join(self.builddir, 
'task-depends.dot')), msg = "No \"%s\" dependency found in task-depends.dot 
file." % recipe)
 
 def test_image_manifest(self):
 bitbake('core-image-minimal')
-- 
2.7.4

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


[OE-core] [PATCH 0/1] expect: Fix buffer overflow error when build in long path

2019-09-24 Thread Robert Yang
The following changes since commit 95ad5626296380358c8a502a3e04879dab653d78:

  build-appliance-image: Update to master head revision (2019-09-19 20:32:47 
+0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/expect
  http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/expect

Robert Yang (1):
  expect: Fix buffer overflow error when build in long path

 ...0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch | 55 ++
 meta/recipes-devtools/expect/expect_5.45.4.bb  |  1 +
 2 files changed, 56 insertions(+)
 create mode 100644 
meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch

-- 
2.7.4

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


[OE-core] [PATCH 1/1] expect: Fix buffer overflow error when build in long path

2019-09-24 Thread Robert Yang
Fixed when built the project in a long path (len(TMPDIR) > 200):
$ bitbake dejagnu-native
[snip]
checking Tcl version 8.5 or greater... *** buffer overflow detected ***:
TOPDIR/tmp-glibc/work/x86_64-linux/dejagnu-native/1.6.2-r0/recipe-sysroot-native/usr/bin/expect
 terminated
[snip]

Signed-off-by: Robert Yang 
---
 ...0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch | 55 ++
 meta/recipes-devtools/expect/expect_5.45.4.bb  |  1 +
 2 files changed, 56 insertions(+)
 create mode 100644 
meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch

diff --git 
a/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch
 
b/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch
new file mode 100644
index 000..37512fb
--- /dev/null
+++ 
b/meta/recipes-devtools/expect/expect/0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch
@@ -0,0 +1,55 @@
+From 1407fcad6f1dac0a4efe8041660bf6139c1cd16a Mon Sep 17 00:00:00 2001
+From: Robert Yang 
+Date: Tue, 24 Sep 2019 13:40:10 +0800
+Subject: [PATCH] exp_main_sub.c: Use PATH_MAX for path
+
+If expect was built from a long path whose length > 200, then it couldn't run:
+$ expect -c 'puts yes'
+*** buffer overflow detected ***: expect terminated
+Aborted (core dumped)
+
+Use PATH_MAX to fix the problem.
+
+Upstream-Status: Pending [Upstream seems dead]
+
+Signed-off-by: Robert Yang 
+---
+ exp_main_sub.c | 8 ++--
+ 1 file changed, 6 insertions(+), 2 deletions(-)
+
+diff --git a/exp_main_sub.c b/exp_main_sub.c
+index fcfaa6e..bf6c4be 100644
+--- a/exp_main_sub.c
 b/exp_main_sub.c
+@@ -48,6 +48,10 @@ char exp_version[] = PACKAGE_VERSION;
+ #define NEED_TCL_MAJOR7
+ #define NEED_TCL_MINOR5
+ 
++#ifndef PATH_MAX
++#define PATH_MAX 4096
++#endif
++
+ char *exp_argv0 = "this program"; /* default program name */
+ void (*exp_app_exit)() = 0;
+ void (*exp_event_exit)() = 0;
+@@ -901,7 +905,7 @@ int sys_rc;
+   int rc;
+ 
+   if (sys_rc) {
+-  char file[200];
++  char file[PATH_MAX];
+   int fd;
+ 
+   sprintf(file,"%s/expect.rc",SCRIPTDIR);
+@@ -917,7 +921,7 @@ int sys_rc;
+   }
+   }
+   if (my_rc) {
+-  char file[200];
++  char file[PATH_MAX];
+   char *home;
+   int fd;
+   char *getenv();
+-- 
+2.7.4
+
diff --git a/meta/recipes-devtools/expect/expect_5.45.4.bb 
b/meta/recipes-devtools/expect/expect_5.45.4.bb
index 96eacd9..dcd252f 100644
--- a/meta/recipes-devtools/expect/expect_5.45.4.bb
+++ b/meta/recipes-devtools/expect/expect_5.45.4.bb
@@ -25,6 +25,7 @@ SRC_URI = 
"${SOURCEFORGE_MIRROR}/expect/Expect/${PV}/${BPN}${PV}.tar.gz \

file://0001-expect-install-scripts-without-using-the-fixline1-tc.patch \
file://0001-Resolve-string-formatting-issues.patch \

file://0001-expect-Fix-segfaults-if-Tcl-is-built-with-stubs-and-.patch \
+   file://0001-exp_main_sub.c-Use-PATH_MAX-for-path.patch \
   "
 SRC_URI[md5sum] = "00fce8de158422f5ccd2666512329bd2"
 SRC_URI[sha256sum] = 
"49a7da83b0bdd9f46d04a04deec19c7767bb9a323e40c4781f89caf760b92c34"
-- 
2.7.4

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


Re: [OE-core] [warrior][PATCH] kernel-uboot: compress arm64 kernels

2019-09-24 Thread Bedel, Alban
On Tue, 2019-09-03 at 09:41 +, Bedel, Alban wrote:
> On Wed, 2019-07-31 at 13:53 +, Bedel, Alban wrote:
> > AArch64 images are not self-decompressing, thus usually much
> > larger.
> > Boot times can be reduced by compressing them in FIT and uImages.
> > 
> > This commit is a backport of commit a725d188b5 (kernel-uboot:
> > compress
> > arm64 kernels) and commit 60bc7e180e (kernel-uboot: remove useless
> > special casing of arm64 Image) from master. Both commit were melted
> > into one to avoid some useless churn.
> 
> Was this patch overlooked, or is there a reason it is not considered
> in
> the next round of update for warrior? Without this patch kernel
> images
> are too large to fit in the flash of the system I'm using.
> Furthermore
> it is not trivial to fix this in my own layer.

Please, I really like to get an answer here. I'm fine if there is a
reason why this patch is not considered for warrior, but just getting
ignored is very frustrating.

Alban



signature.asc
Description: This is a digitally signed message part
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [Thud][ 15/24] dhcp: fix issue with new bind changes

2019-09-24 Thread Adrian Bunk
In master this patch was reverted due to a bug that caused a regression.

Please drop this change, and use the upstream fix
0001-master-Added-includes-of-new-BIND9-compatibility-hea.patch
instead.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

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