Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Richard Purdie
On Mon, 2020-03-23 at 16:25 -0700, Khem Raj wrote:
> Distros which want to enable LTO can utilize this file, it only
> covers
> packages from OE-Core, other layers should include there own
> exclusion
> list for recipe which dont work with LTO
> 
> Signed-off-by: Khem Raj 
> ---
>  meta/conf/distro/include/lto.inc | 23 +++
>  1 file changed, 23 insertions(+)
>  create mode 100644 meta/conf/distro/include/lto.inc
> 
> diff --git a/meta/conf/distro/include/lto.inc
> b/meta/conf/distro/include/lto.inc
> new file mode 100644
> index 00..0cedc8deca
> --- /dev/null
> +++ b/meta/conf/distro/include/lto.inc
> @@ -0,0 +1,23 @@
> +LTO_pn-glibc = ""
> +LTO_pn-gcc-runtime = ""
> +LTO_pn-libgcc-initial = ""
> +LTO_pn-libgcc = ""
> +LTO_pn-libpam = ""
> +LTO_pn-elfutils = ""
> +LTO_pn-perl = ""
> +LTO_pn-busybox = ""
> +LTO_pn-libxcrypt = ""
> +LTO_pn-curl = ""
> +LTO_pn-libcap = ""
> +LTO_pn-python3 = ""
> +LTO_pn-libproxy = ""
> +LTO_pn-libbsd = ""
> +
> +LTO ?= "-flto"
> +
> +SELECTED_OPTIMIZATION_append_class-target = " ${LTO}"
> +TARGET_LDFLAGS_append_class-target = " ${LTO}"
> +
> +SELECTED_OPTIMIZATION[vardeps] += "LTO"
> +
> +PACKAGE_DEBUG_SPLIT_STYLE = "debug-without-src"

Why do we need to specify the debug split style here?

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136628): 
https://lists.openembedded.org/g/openembedded-core/message/136628
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.27

2020-03-24 Thread Khem Raj
Hi Bruce

This results in kernel-selftest failure regression

errors.yoctoproject.org/Errors/Build/100385/

and also same errors as reported here
https://bugs.launchpad.net/ubuntu-kernel-tests/+bug/1867596

I have fixed them via three patches that are here

https://git.openembedded.org/openembedded-core-contrib/commit/?h=yoe/mut=2c15bc35809365db8c586df7c383ed01b00e6618

Please cherry-pick these 3 patches as well on this pull while you are here.

On Sat, Mar 21, 2020 at 6:18 PM Bruce Ashfield  wrote:
>
> From: Bruce Ashfield 
>
> Updating linux-yocto/5.4 to the latest korg -stable release that comprises
> the following commits:
>
> 585e0cc08069 Linux 5.4.27
> 20ba40d262b0 ipv4: ensure rcu_read_lock() in cipso_v4_error()
> 81e0dac3780a ARM: 8961/2: Fix Kbuild issue caused by per-task stack 
> protector GCC plugin
> 1dc67950e2d1 HID: add ALWAYS_POLL quirk to lenovo pixart mouse
> 5d961582ea88 HID: google: add moonball USB id
> ae119b7e1247 mm: slub: add missing TID bump in kmem_cache_alloc_bulk()
> 47b6d269dbbd ARM: 8958/1: rename missed uaccess .fixup section
> 1a9e78628d44 ARM: 8957/1: VDSO: Match ARMv8 timer in cntvct_functional()
> 881876bee657 net: qrtr: fix len of skb_put_padto in qrtr_node_enqueue
> 235fb892d846 blk-mq: insert flush request to the front of dispatch queue
> dbce8292217a jbd2: fix data races at struct journal_head
> 99746e92fa4c sfc: fix timestamp reconstruction at 16-bit rollover points
> e2b8b042c98b net: rmnet: fix packet forwarding in rmnet bridge mode
> 34c550ebb219 net: rmnet: fix bridge mode bugs
> 8eccc6d06ea1 net: rmnet: use upper/lower device infrastructure
> 835bbd892683 net: rmnet: do not allow to change mux id if mux id is 
> duplicated
> 5ce5e95d72f1 net: rmnet: remove rcu_read_lock in 
> rmnet_force_unassociate_device()
> 002b4bd9b51d net: rmnet: fix suspicious RCU usage
> 8a027eee62f7 net: rmnet: fix NULL pointer dereference in 
> rmnet_changelink()
> 006cd2426a05 net: rmnet: fix NULL pointer dereference in rmnet_newlink()
> cfc7f5fe5c61 hinic: fix a bug of rss configuration
> e6f52d5ce172 hinic: fix a bug of setting hw_ioctxt
> 65340b196811 hinic: fix a irq affinity bug
> 1c7dde0a9bfc net: phy: mscc: fix firmware paths
> 4d6ee3123d65 slip: not call free_netdev before rtnl_unlock in slip_open
> d1984c92f072 signal: avoid double atomic counter increments for user 
> accounting
> df8e98b00941 kbuild: add dt_binding_check to PHONY in a correct place
> fd1f29f2a824 kbuild: add dtbs_check to PHONY
> bbf3acbb8a77 drm/amdgpu: fix memory leak during TDR test(v2)
> 74c77d6a4ea9 blk-mq: insert passthrough request into hctx->dispatch 
> directly
> 74886a6d9de8 net: ll_temac: Handle DMA halt condition caused by buffer 
> underrun
> c2cffaea4fde net: ll_temac: Fix RX buffer descriptor handling on 
> GFP_ATOMIC pressure
> 6e3b69543b5e net: ll_temac: Add more error handling of dma_map_single() 
> calls
> 0c050d655475 net: ll_temac: Fix race condition causing TX hang
> a342bb7b9902 mac80211: rx: avoid RCU list traversal under mutex
> 5d92d834aa27 net: ks8851-ml: Fix IRQ handling and locking
> d939f3389f9b net: usb: qmi_wwan: restore mtu min/max values after raw_ip 
> switch
> 6c663120ecdb scsi: libfc: free response frame from GPN_ID
> 079e9ad79410 cfg80211: check reg_rule for NULL in handle_channel_custom()
> a6f7e69cfafd tracing: Fix number printing bug in print_synth_event()
> c35aa36fec91 selftests/rseq: Fix out-of-tree compilation
> dd0d2d6ab6a2 HID: hid-bigbenff: fix race condition for scheduled work 
> during removal
> ccb747d805d3 HID: hid-bigbenff: call hid_hw_stop() in case of error
> 1bc271dd397d HID: hid-bigbenff: fix general protection fault caused by 
> double kfree
> 190e929e6cba HID: i2c-hid: add Trekstor Surfbook E11B to descriptor 
> override
> 87bd74b77531 ACPI: watchdog: Set default timeout in probe
> 65b44497414b HID: apple: Add support for recent firmware on Magic 
> Keyboards
> 20eed7692749 ACPI: watchdog: Allow disabling WDAT at boot
> f3c478104a9a drm/amdgpu: Fix TLB invalidation request when using semaphore
> f6fccb0a0e59 netfilter: xt_hashlimit: unregister proc file before 
> releasing mutex
> 536942bf70b5 netfilter: hashlimit: do not use indirect calls during gc
> 257edc6db943 Linux 5.4.26
> 881f9469cc9f net/smc: cancel event worker during device removal
> dbaba2a4db6e net/smc: check for valid ib_client_data
> aa38bea47fd2 ipv6: restrict IPV6_ADDRFORM operation
> 62fd4e348df2 iommu/amd: Fix IOMMU AVIC not properly update the is_run bit 
> in IRTE
> 8753b0cad128 i2c: acpi: put device when verifying client fails
> 03d524d70ecf iommu/vt-d: Ignore devices with out-of-spec domain number
> 9493a6361d81 iommu/vt-d: Fix the wrong printing in RHSA parsing
> 2d106e75da45 netfilter: nft_chain_nat: inet family is missing module 
> 

Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Adrian Bunk
On Mon, Mar 23, 2020 at 04:25:29PM -0700, Khem Raj wrote:
>...
> +LTO_pn-python3 = ""
>...

Building Python with LTO works by passing --with-lto to configure.

cu
Adrian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136637): 
https://lists.openembedded.org/g/openembedded-core/message/136637
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Adrian Bunk
On Mon, Mar 23, 2020 at 04:25:29PM -0700, Khem Raj wrote:
> Distros which want to enable LTO can utilize this file, it only covers
> packages from OE-Core, other layers should include there own exclusion
> list for recipe which dont work with LTO
>...

More and more upstreams already default to LTO in their build system.

What is the recommended way to get the correct configure options to 
disable LTO in all such recipes?

The option is obviously recipe-specific, but there has to be some
global define the recipe can check for disable/default/enable.

cu
Adrian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136639): 
https://lists.openembedded.org/g/openembedded-core/message/136639
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] python3 and util-linux dependency loop

2020-03-24 Thread Diego
Hi list,

I'm trying to enable the pylibmount PACKAGECONFIG of util-linux.
Unfortunately this seems to uncover a dependency loop:
python3 depends on util-linux:
https://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/meta/recipes-devtools/python/python3_3.8.2.bb#n69
while util-linux with pylibmount enabled depends on python3:
https://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/meta/recipes-core/util-linux/util-linux.inc#n123

What do you think is the best approach to solve this dependency loop?

Thank you,
Diego Rondini
Sr. Embedded Engineer

Kynetics
www.kynetics.com
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136629): 
https://lists.openembedded.org/g/openembedded-core/message/136629
Mute This Topic: https://lists.openembedded.org/mt/72514587/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Adrian Bunk
On Mon, Mar 23, 2020 at 04:25:29PM -0700, Khem Raj wrote:
> Distros which want to enable LTO can utilize this file, it only covers
> packages from OE-Core, other layers should include there own exclusion
> list for recipe which dont work with LTO
>...
> +++ b/meta/conf/distro/include/lto.inc
> @@ -0,0 +1,23 @@
> +LTO_pn-glibc = ""
> +LTO_pn-gcc-runtime = ""
> +LTO_pn-libgcc-initial = ""
> +LTO_pn-libgcc = ""
> +LTO_pn-libpam = ""
> +LTO_pn-elfutils = ""
> +LTO_pn-perl = ""
> +LTO_pn-busybox = ""
> +LTO_pn-libxcrypt = ""
> +LTO_pn-curl = ""
> +LTO_pn-libcap = ""
> +LTO_pn-python3 = ""
> +LTO_pn-libproxy = ""
> +LTO_pn-libbsd = ""
>...

It would be better to have these in the individual recipes,
it is much easier to understand what is going on when all
recipe-specific information is in the recipe.

Otherwise this will end as an implementation of workarounds that
might not even be needed anymore, for a previous example with
several workarounds that were no longer needed see
https://git.openembedded.org/openembedded-core/commit/?id=8bbb17d2dcafa9b4e26941a55932f4be2782e1d1

cu
Adrian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136630): 
https://lists.openembedded.org/g/openembedded-core/message/136630
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 1/5] buildstats-plot.sh: fix invoking buildstats.sh

2020-03-24 Thread Luis Martins
Fix the internal call to buildstats.sh by also providing
the buildstats folder location in the system, which might
differ from the default location.

Upstream-Status: Pending

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 898834e5ac..089621c9f8 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -101,7 +101,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -H -s "$STATS" -H > $OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -H > $OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136632): 
https://lists.openembedded.org/g/openembedded-core/message/136632
Mute This Topic: https://lists.openembedded.org/mt/72514839/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 4/5] buildstats*.sh: add recipe parameter to scripts

2020-03-24 Thread Luis Martins
Add option to filter the stats results for a single recipe, allowing
to easilly visuality the task resource consumption per recipe.
This is specially useful when debugging the performance of
the overall bitbake build system.

Upstream-Status: Pending

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh | 9 +++--
 scripts/contrib/bb-perf/buildstats.sh  | 9 +++--
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index f26312f355..320c4910a1 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -39,6 +39,7 @@ set -o errexit
 
 BS_DIR="tmp/buildstats"
 N=10
+RECIPE=""
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="utime"
 ACCUMULATE=""
@@ -53,6 +54,7 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 (default: "$BS_DIR")
   -n N  Top N recipes to display. Ignored if -S is present
 (default: "$N")
+  -r recipe The recipe mask to be searched
   -t tasks  The tasks to be computed
 (default: "$TASKS")
   -s stats  The stats to be matched. If more that one stat, units
@@ -69,7 +71,7 @@ EOM
 }
 
 # Parse and validate arguments
-while getopts "b:n:t:s:o:aSh" OPT; do
+while getopts "b:n:r:t:s:o:aSh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -77,6 +79,9 @@ while getopts "b:n:t:s:o:aSh" OPT; do
n)
N="$OPTARG"
;;
+r)
+RECIPE="-r $OPTARG"
+;;
 t)
 TASKS="$OPTARG"
 ;;
@@ -112,7 +117,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" $ACCUMULATE -H > 
$OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" $RECIPE $ACCUMULATE -H 
> $OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
diff --git a/scripts/contrib/bb-perf/buildstats.sh 
b/scripts/contrib/bb-perf/buildstats.sh
index 26bfaca86b..2f017efeb8 100755
--- a/scripts/contrib/bb-perf/buildstats.sh
+++ b/scripts/contrib/bb-perf/buildstats.sh
@@ -36,6 +36,7 @@ Child rusage ru_majflt:Child rusage ru_inblock:Child rusage 
ru_oublock:Child rus
 Child rusage ru_nivcsw"
 
 BS_DIR="tmp/buildstats"
+RECIPE=""
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="$TIME"
 ACCUMULATE=""
@@ -47,6 +48,7 @@ cat <-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136635): 
https://lists.openembedded.org/g/openembedded-core/message/136635
Mute This Topic: https://lists.openembedded.org/mt/72514842/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 5/5] buildstats*.sh: fix spacing indentation in the files

2020-03-24 Thread Luis Martins
Fix indentation in buildstats.sh and buildstats-plot.sh
to convert every line to 4 spaces indentation.

Upstream-Status: Pending

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh |  56 -
 scripts/contrib/bb-perf/buildstats.sh  | 131 +++--
 2 files changed, 94 insertions(+), 93 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 320c4910a1..80fd30960c 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -72,40 +72,40 @@ EOM
 
 # Parse and validate arguments
 while getopts "b:n:r:t:s:o:aSh" OPT; do
-   case $OPT in
-   b)
-   BS_DIR="$OPTARG"
-   ;;
-   n)
-   N="$OPTARG"
-   ;;
+case $OPT in
+b)
+BS_DIR="$OPTARG"
+;;
+n)
+N="$OPTARG"
+;;
 r)
 RECIPE="-r $OPTARG"
 ;;
 t)
 TASKS="$OPTARG"
 ;;
-   s)
-   STATS="$OPTARG"
-   ;;
-   a)
+s)
+STATS="$OPTARG"
+;;
+a)
 ACCUMULATE="-a"
 ;;
-   S)
-   SUM="y"
-   ;;
-   o)
-   OUTDATA_FILE="$OPTARG"
-   ;;
-   h)
-   usage
-   exit 0
-   ;;
-   *)
-   usage
-   exit 1
-   ;;
-   esac
+S)
+SUM="y"
+;;
+o)
+OUTDATA_FILE="$OPTARG"
+;;
+h)
+usage
+exit 0
+;;
+*)
+usage
+exit 1
+;;
+esac
 done
 
 # Get number of stats
@@ -141,8 +141,8 @@ else
 declare -a sumargs
 j=0
 for i in `seq $nstats`; do
-   sumargs[j]=sum; j=$(( $j + 1 ))
-   sumargs[j]=`expr 3 + $i - 1`;  j=$(( $j + 1 ))
+sumargs[j]=sum; j=$(( $j + 1 ))
+sumargs[j]=`expr 3 + $i - 1`;  j=$(( $j + 1 ))
 done
 
 # Do the processing with datamash
diff --git a/scripts/contrib/bb-perf/buildstats.sh 
b/scripts/contrib/bb-perf/buildstats.sh
index 2f017efeb8..e45cfc146d 100755
--- a/scripts/contrib/bb-perf/buildstats.sh
+++ b/scripts/contrib/bb-perf/buildstats.sh
@@ -66,62 +66,63 @@ EOM
 
 # Parse and validate arguments
 while getopts "b:r:t:s:aHh" OPT; do
-   case $OPT in
-   b)
-   BS_DIR="$OPTARG"
-   ;;
-   r)
-   RECIPE="$OPTARG"
-   ;;
-   t)
-   TASKS="$OPTARG"
-   ;;
-   s)
-   STATS="$OPTARG"
-   ;;
-   a)
+case $OPT in
+b)
+BS_DIR="$OPTARG"
+;;
+r)
+RECIPE="$OPTARG"
+;;
+t)
+TASKS="$OPTARG"
+;;
+s)
+STATS="$OPTARG"
+;;
+a)
 ACCUMULATE="y"
 ;;
-   H)
-   HEADER="y"
-   ;;
-   h)
-   usage
-   exit 0
-   ;;
-   *)
-   usage
-   exit 1
-   ;;
-   esac
+H)
+HEADER="y"
+;;
+h)
+usage
+exit 0
+;;
+*)
+usage
+exit 1
+;;
+esac
 done
 
 # Ensure the buildstats folder exists
 if [ ! -d "$BS_DIR" ]; then
-   echo "ERROR: $BS_DIR does not exist"
-   usage
-   exit 1
+echo "ERROR: $BS_DIR does not exist"
+usage
+exit 1
 fi
 
 stats=""
 IFS=":"
 for stat in ${STATS}; do
-   case $stat in
-   TIME)
-   stats="${stats}:${TIME}"
-   ;;
-   IO)
-   stats="${stats}:${IO}"
-   ;;
-   RUSAGE)
-   stats="${stats}:${RUSAGE}"
-   ;;
-   CHILD_RUSAGE)
-   stats="${stats}:${CHILD_RUSAGE}"
-   ;;
-   *)
-   stats="${STATS}"
-   esac
+case $stat in
+TIME)
+stats="${stats}:${TIME}"
+;;
+IO)
+stats="${stats}:${IO}"
+;;
+RUSAGE)
+stats="${stats}:${RUSAGE}"
+;;
+CHILD_RUSAGE)
+stats="${stats}:${CHILD_RUSAGE}"
+;;
+*)
+stats="${STATS}"
+;;
+esac
 done
 
 # remove possible colon at the beginning
@@ -140,27 +141,27 @@ for task in ${TASKS}; do
 task="do_${task}"
 for file in $(find ${BS_DIR} -type f -path *${RECIPE}*/${task} | awk 
'BEGIN{ ORS=""; OFS=":" } { print $0,"" }'); do
 recipe="$(basename $(dirname $file))"
-   times=""
-   for stat in ${stats}; do
-   [ -z "$stat" ] && { echo "empty stats"; }
-   time=$(sed -n -e "s/^\($stat\): \\(.*\\)/\\2/p" $file)
-   # in case the stat is not present, set the value as NA
-   [ -z "$time" ] && { time="NA"; }
-   # Append it to times
-   if [ -z "$times" ]; then
-   times="${time}"
-   else
-   

[OE-core] [PATCH 3/5] buildstats*.sh: add accumulate parameter to scripts

2020-03-24 Thread Luis Martins
Add option to accumulate stats values per recipe, allowing
to sum related values such as memory (main process + childs).
This is specially useful when debugging the performance of
the overall bitbake build system.

Upstream-Status: Pending

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh |  9 +++--
 scripts/contrib/bb-perf/buildstats.sh  | 23 --
 2 files changed, 28 insertions(+), 4 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 1d22e60d73..f26312f355 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -41,6 +41,7 @@ BS_DIR="tmp/buildstats"
 N=10
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="utime"
+ACCUMULATE=""
 SUM=""
 OUTDATA_FILE="$PWD/buildstats-plot.out"
 
@@ -59,6 +60,7 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 (see buildstats.sh -h for all options) or any other defined
 (build)stat separated by colons, i.e. stime:utime
 (default: "$STATS")
+  -aAccumulate all stats values for found recipes
   -SSum values for a particular stat for found recipes
   -oOutput data file.
 (default: "$OUTDATA_FILE")
@@ -67,7 +69,7 @@ EOM
 }
 
 # Parse and validate arguments
-while getopts "b:n:t:s:o:Sh" OPT; do
+while getopts "b:n:t:s:o:aSh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -81,6 +83,9 @@ while getopts "b:n:t:s:o:Sh" OPT; do
s)
STATS="$OPTARG"
;;
+   a)
+ACCUMULATE="-a"
+;;
S)
SUM="y"
;;
@@ -107,7 +112,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" -H > $OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" $ACCUMULATE -H > 
$OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
diff --git a/scripts/contrib/bb-perf/buildstats.sh 
b/scripts/contrib/bb-perf/buildstats.sh
index e9ec2d476a..26bfaca86b 100755
--- a/scripts/contrib/bb-perf/buildstats.sh
+++ b/scripts/contrib/bb-perf/buildstats.sh
@@ -38,6 +38,7 @@ Child rusage ru_nivcsw"
 BS_DIR="tmp/buildstats"
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="$TIME"
+ACCUMULATE=""
 HEADER="" # No header by default
 
 function usage {
@@ -56,12 +57,13 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 IO=$IO
 RUSAGE=$RUSAGE
 CHILD_RUSAGE=$CHILD_RUSAGE
+  -aAccumulate all stats values for found recipes
   -hDisplay this help message
 EOM
 }
 
 # Parse and validate arguments
-while getopts "b:t:s:Hh" OPT; do
+while getopts "b:t:s:aHh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -72,6 +74,9 @@ while getopts "b:t:s:Hh" OPT; do
s)
STATS="$OPTARG"
;;
+   a)
+ACCUMULATE="y"
+;;
H)
HEADER="y"
;;
@@ -118,7 +123,13 @@ done
 stats="$(echo "$stats" | sed -e 's/^://1')"
 
 # Provide a header if required by the user
-[ -n "$HEADER" ] && { echo "task:recipe:$stats"; }
+if [ -n "$HEADER" ] ; then
+if [ -n "$ACCUMULATE" ]; then
+echo "task:recipe:accumulated(${stats//:/;})"
+else
+echo "task:recipe:$stats"
+fi
+fi
 
 for task in ${TASKS}; do
 task="do_${task}"
@@ -137,6 +148,14 @@ for task in ${TASKS}; do
times="${times} ${time}"
fi
done
+if [ -n "$ACCUMULATE" ]; then
+IFS=' '; valuesarray=(${times}); IFS=':'
+times=0
+for value in "${valuesarray[@]}"; do
+[ "$value" == "NA" ] && { echo "ERROR: stat is not present."; 
usage; exit 1; }
+times=$(( $times + $value ))
+done
+fi
 echo "${task} ${recipe} ${times}"
 done
 done
-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136634): 
https://lists.openembedded.org/g/openembedded-core/message/136634
Mute This Topic: https://lists.openembedded.org/mt/72514841/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 2/5] buildstats-plot.sh: filter by task

2020-03-24 Thread Luis Martins
Extend buildstats-plot.sh script to also accept the name
of the tasks as parameter.
This value will be passed directly to buildstats.sh is
already provides this option.

Upstream-Status: Pending

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh | 10 --
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 089621c9f8..1d22e60d73 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -39,6 +39,7 @@ set -o errexit
 
 BS_DIR="tmp/buildstats"
 N=10
+TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="utime"
 SUM=""
 OUTDATA_FILE="$PWD/buildstats-plot.out"
@@ -51,6 +52,8 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 (default: "$BS_DIR")
   -n N  Top N recipes to display. Ignored if -S is present
 (default: "$N")
+  -t tasks  The tasks to be computed
+(default: "$TASKS")
   -s stats  The stats to be matched. If more that one stat, units
 should be the same because data is plot as histogram.
 (see buildstats.sh -h for all options) or any other defined
@@ -64,7 +67,7 @@ EOM
 }
 
 # Parse and validate arguments
-while getopts "b:n:s:o:Sh" OPT; do
+while getopts "b:n:t:s:o:Sh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -72,6 +75,9 @@ while getopts "b:n:s:o:Sh" OPT; do
n)
N="$OPTARG"
;;
+t)
+TASKS="$OPTARG"
+;;
s)
STATS="$OPTARG"
;;
@@ -101,7 +107,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -H > $OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" -H > $OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136633): 
https://lists.openembedded.org/g/openembedded-core/message/136633
Mute This Topic: https://lists.openembedded.org/mt/72514840/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 0/5] improvements to buildstats scripts

2020-03-24 Thread Luis Martins
This patches add some new functionalities to the buildstats
scripts to allow easier debugging of the bitbake performance.
Some new features like accumulating the stats values allow
to compare the total amount of memory used by a single task
(main process + child processes) and other features allow
convenient analysis of task performance per recipe.

Luis Martins (5):
  buildstats-plot.sh: fix invoking buildstats.sh
  buildstats-plot.sh: filter by task
  buildstats*.sh: add accumulate parameter to scripts
  buildstats*.sh: add recipe parameter to scripts
  buildstats*.sh: fix spacing indentation in the files

 scripts/contrib/bb-perf/buildstats-plot.sh |  74 ++-
 scripts/contrib/bb-perf/buildstats.sh  | 139 -
 2 files changed, 127 insertions(+), 86 deletions(-)

-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136631): 
https://lists.openembedded.org/g/openembedded-core/message/136631
Mute This Topic: https://lists.openembedded.org/mt/72514838/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 13:27 +0200, Adrian Bunk wrote:
> On Mon, Mar 23, 2020 at 04:25:29PM -0700, Khem Raj wrote:
> > Distros which want to enable LTO can utilize this file, it only
> > covers
> > packages from OE-Core, other layers should include there own
> > exclusion
> > list for recipe which dont work with LTO
> > ...
> > +++ b/meta/conf/distro/include/lto.inc
> > @@ -0,0 +1,23 @@
> > +LTO_pn-glibc = ""
> > +LTO_pn-gcc-runtime = ""
> > +LTO_pn-libgcc-initial = ""
> > +LTO_pn-libgcc = ""
> > +LTO_pn-libpam = ""
> > +LTO_pn-elfutils = ""
> > +LTO_pn-perl = ""
> > +LTO_pn-busybox = ""
> > +LTO_pn-libxcrypt = ""
> > +LTO_pn-curl = ""
> > +LTO_pn-libcap = ""
> > +LTO_pn-python3 = ""
> > +LTO_pn-libproxy = ""
> > +LTO_pn-libbsd = ""
> > ...
> 
> It would be better to have these in the individual recipes,
> it is much easier to understand what is going on when all
> recipe-specific information is in the recipe.

I disagree. These files do let someone working on specific enablement
see the status collectively and allow people to opt in/out of a given
development feature.

At some point in the future that can tip into making more sense in a
recipe but its a staged process and things often make sense to start
like this.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136638): 
https://lists.openembedded.org/g/openembedded-core/message/136638
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Khem Raj
On Tue, Mar 24, 2020 at 4:37 AM Adrian Bunk  wrote:
>
> On Mon, Mar 23, 2020 at 04:25:29PM -0700, Khem Raj wrote:
> >...
> > +LTO_pn-python3 = ""
> >...
>
> Building Python with LTO works by passing --with-lto to configure.

I think per recipe knobs I did not work on yet but perhaps worth
adding them in next release
or may be as packageconfig dependent on lto distro feature
>
> cu
> Adrian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136643): 
https://lists.openembedded.org/g/openembedded-core/message/136643
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Khem Raj
On Tue, Mar 24, 2020 at 1:37 AM Richard Purdie
 wrote:
>
> On Mon, 2020-03-23 at 16:25 -0700, Khem Raj wrote:
> > Distros which want to enable LTO can utilize this file, it only
> > covers
> > packages from OE-Core, other layers should include there own
> > exclusion
> > list for recipe which dont work with LTO
> >
> > Signed-off-by: Khem Raj 
> > ---
> >  meta/conf/distro/include/lto.inc | 23 +++
> >  1 file changed, 23 insertions(+)
> >  create mode 100644 meta/conf/distro/include/lto.inc
> >
> > diff --git a/meta/conf/distro/include/lto.inc
> > b/meta/conf/distro/include/lto.inc
> > new file mode 100644
> > index 00..0cedc8deca
> > --- /dev/null
> > +++ b/meta/conf/distro/include/lto.inc
> > @@ -0,0 +1,23 @@
> > +LTO_pn-glibc = ""
> > +LTO_pn-gcc-runtime = ""
> > +LTO_pn-libgcc-initial = ""
> > +LTO_pn-libgcc = ""
> > +LTO_pn-libpam = ""
> > +LTO_pn-elfutils = ""
> > +LTO_pn-perl = ""
> > +LTO_pn-busybox = ""
> > +LTO_pn-libxcrypt = ""
> > +LTO_pn-curl = ""
> > +LTO_pn-libcap = ""
> > +LTO_pn-python3 = ""
> > +LTO_pn-libproxy = ""
> > +LTO_pn-libbsd = ""
> > +
> > +LTO ?= "-flto"
> > +
> > +SELECTED_OPTIMIZATION_append_class-target = " ${LTO}"
> > +TARGET_LDFLAGS_append_class-target = " ${LTO}"
> > +
> > +SELECTED_OPTIMIZATION[vardeps] += "LTO"
> > +
> > +PACKAGE_DEBUG_SPLIT_STYLE = "debug-without-src"
>
> Why do we need to specify the debug split style here?
>

debug file generator does  not generate debugsrc packages with lto i
have not investigates in
detail but its due to the fact that .o files are not normal elf objs

> Cheers,
>
> Richard
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136642): 
https://lists.openembedded.org/g/openembedded-core/message/136642
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.27

2020-03-24 Thread Khem Raj
On Tue, Mar 24, 2020 at 5:40 AM Bruce Ashfield  wrote:
>
> On Tue, Mar 24, 2020 at 2:35 AM Khem Raj  wrote:
> >
> > Hi Bruce
> >
> > This results in kernel-selftest failure regression
> >
> > errors.yoctoproject.org/Errors/Build/100385/
> >
> > and also same errors as reported here
> > https://bugs.launchpad.net/ubuntu-kernel-tests/+bug/1867596
> >
> > I have fixed them via three patches that are here
> >
> > https://git.openembedded.org/openembedded-core-contrib/commit/?h=yoe/mut=2c15bc35809365db8c586df7c383ed01b00e6618
> >
> > Please cherry-pick these 3 patches as well on this pull while you are here.
> >
>
> Only one of the patches has any upstream information.
>
> Are the reverts also being done in mainline ? Or is a different fix ?
>

these are all results of half cherry picks into stable. all works fine in master
they should be reverted in stable but I have not sent them to lkml

> What about the one that does have an upstream status, has it been
> submitted to -stable ?
>
> Bruce
>
> > On Sat, Mar 21, 2020 at 6:18 PM Bruce Ashfield  
> > wrote:
> > >
> > > From: Bruce Ashfield 
> > >
> > > Updating linux-yocto/5.4 to the latest korg -stable release that comprises
> > > the following commits:
> > >
> > > 585e0cc08069 Linux 5.4.27
> > > 20ba40d262b0 ipv4: ensure rcu_read_lock() in cipso_v4_error()
> > > 81e0dac3780a ARM: 8961/2: Fix Kbuild issue caused by per-task stack 
> > > protector GCC plugin
> > > 1dc67950e2d1 HID: add ALWAYS_POLL quirk to lenovo pixart mouse
> > > 5d961582ea88 HID: google: add moonball USB id
> > > ae119b7e1247 mm: slub: add missing TID bump in kmem_cache_alloc_bulk()
> > > 47b6d269dbbd ARM: 8958/1: rename missed uaccess .fixup section
> > > 1a9e78628d44 ARM: 8957/1: VDSO: Match ARMv8 timer in 
> > > cntvct_functional()
> > > 881876bee657 net: qrtr: fix len of skb_put_padto in qrtr_node_enqueue
> > > 235fb892d846 blk-mq: insert flush request to the front of dispatch 
> > > queue
> > > dbce8292217a jbd2: fix data races at struct journal_head
> > > 99746e92fa4c sfc: fix timestamp reconstruction at 16-bit rollover 
> > > points
> > > e2b8b042c98b net: rmnet: fix packet forwarding in rmnet bridge mode
> > > 34c550ebb219 net: rmnet: fix bridge mode bugs
> > > 8eccc6d06ea1 net: rmnet: use upper/lower device infrastructure
> > > 835bbd892683 net: rmnet: do not allow to change mux id if mux id is 
> > > duplicated
> > > 5ce5e95d72f1 net: rmnet: remove rcu_read_lock in 
> > > rmnet_force_unassociate_device()
> > > 002b4bd9b51d net: rmnet: fix suspicious RCU usage
> > > 8a027eee62f7 net: rmnet: fix NULL pointer dereference in 
> > > rmnet_changelink()
> > > 006cd2426a05 net: rmnet: fix NULL pointer dereference in 
> > > rmnet_newlink()
> > > cfc7f5fe5c61 hinic: fix a bug of rss configuration
> > > e6f52d5ce172 hinic: fix a bug of setting hw_ioctxt
> > > 65340b196811 hinic: fix a irq affinity bug
> > > 1c7dde0a9bfc net: phy: mscc: fix firmware paths
> > > 4d6ee3123d65 slip: not call free_netdev before rtnl_unlock in 
> > > slip_open
> > > d1984c92f072 signal: avoid double atomic counter increments for user 
> > > accounting
> > > df8e98b00941 kbuild: add dt_binding_check to PHONY in a correct place
> > > fd1f29f2a824 kbuild: add dtbs_check to PHONY
> > > bbf3acbb8a77 drm/amdgpu: fix memory leak during TDR test(v2)
> > > 74c77d6a4ea9 blk-mq: insert passthrough request into hctx->dispatch 
> > > directly
> > > 74886a6d9de8 net: ll_temac: Handle DMA halt condition caused by 
> > > buffer underrun
> > > c2cffaea4fde net: ll_temac: Fix RX buffer descriptor handling on 
> > > GFP_ATOMIC pressure
> > > 6e3b69543b5e net: ll_temac: Add more error handling of 
> > > dma_map_single() calls
> > > 0c050d655475 net: ll_temac: Fix race condition causing TX hang
> > > a342bb7b9902 mac80211: rx: avoid RCU list traversal under mutex
> > > 5d92d834aa27 net: ks8851-ml: Fix IRQ handling and locking
> > > d939f3389f9b net: usb: qmi_wwan: restore mtu min/max values after 
> > > raw_ip switch
> > > 6c663120ecdb scsi: libfc: free response frame from GPN_ID
> > > 079e9ad79410 cfg80211: check reg_rule for NULL in 
> > > handle_channel_custom()
> > > a6f7e69cfafd tracing: Fix number printing bug in print_synth_event()
> > > c35aa36fec91 selftests/rseq: Fix out-of-tree compilation
> > > dd0d2d6ab6a2 HID: hid-bigbenff: fix race condition for scheduled work 
> > > during removal
> > > ccb747d805d3 HID: hid-bigbenff: call hid_hw_stop() in case of error
> > > 1bc271dd397d HID: hid-bigbenff: fix general protection fault caused 
> > > by double kfree
> > > 190e929e6cba HID: i2c-hid: add Trekstor Surfbook E11B to descriptor 
> > > override
> > > 87bd74b77531 ACPI: watchdog: Set default timeout in probe
> > > 65b44497414b HID: apple: Add support for recent firmware on Magic 
> > > Keyboards
> > > 20eed7692749 ACPI: watchdog: Allow 

Re: [OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.27

2020-03-24 Thread Bruce Ashfield
On Tue, Mar 24, 2020 at 9:42 AM Khem Raj  wrote:
>
> On Tue, Mar 24, 2020 at 5:40 AM Bruce Ashfield  
> wrote:
> >
> > On Tue, Mar 24, 2020 at 2:35 AM Khem Raj  wrote:
> > >
> > > Hi Bruce
> > >
> > > This results in kernel-selftest failure regression
> > >
> > > errors.yoctoproject.org/Errors/Build/100385/
> > >
> > > and also same errors as reported here
> > > https://bugs.launchpad.net/ubuntu-kernel-tests/+bug/1867596
> > >
> > > I have fixed them via three patches that are here
> > >
> > > https://git.openembedded.org/openembedded-core-contrib/commit/?h=yoe/mut=2c15bc35809365db8c586df7c383ed01b00e6618
> > >
> > > Please cherry-pick these 3 patches as well on this pull while you are 
> > > here.
> > >
> >
> > Only one of the patches has any upstream information.
> >
> > Are the reverts also being done in mainline ? Or is a different fix ?
> >
>
> these are all results of half cherry picks into stable. all works fine in 
> master
> they should be reverted in stable but I have not sent them to lkml

Aha. Thanks, that helps me understand what was going on.

I'll amend the logs slightly and yank them into my tree.

Cheers,

Bruce

>
> > What about the one that does have an upstream status, has it been
> > submitted to -stable ?
> >
> > Bruce
> >
> > > On Sat, Mar 21, 2020 at 6:18 PM Bruce Ashfield  
> > > wrote:
> > > >
> > > > From: Bruce Ashfield 
> > > >
> > > > Updating linux-yocto/5.4 to the latest korg -stable release that 
> > > > comprises
> > > > the following commits:
> > > >
> > > > 585e0cc08069 Linux 5.4.27
> > > > 20ba40d262b0 ipv4: ensure rcu_read_lock() in cipso_v4_error()
> > > > 81e0dac3780a ARM: 8961/2: Fix Kbuild issue caused by per-task stack 
> > > > protector GCC plugin
> > > > 1dc67950e2d1 HID: add ALWAYS_POLL quirk to lenovo pixart mouse
> > > > 5d961582ea88 HID: google: add moonball USB id
> > > > ae119b7e1247 mm: slub: add missing TID bump in 
> > > > kmem_cache_alloc_bulk()
> > > > 47b6d269dbbd ARM: 8958/1: rename missed uaccess .fixup section
> > > > 1a9e78628d44 ARM: 8957/1: VDSO: Match ARMv8 timer in 
> > > > cntvct_functional()
> > > > 881876bee657 net: qrtr: fix len of skb_put_padto in 
> > > > qrtr_node_enqueue
> > > > 235fb892d846 blk-mq: insert flush request to the front of dispatch 
> > > > queue
> > > > dbce8292217a jbd2: fix data races at struct journal_head
> > > > 99746e92fa4c sfc: fix timestamp reconstruction at 16-bit rollover 
> > > > points
> > > > e2b8b042c98b net: rmnet: fix packet forwarding in rmnet bridge mode
> > > > 34c550ebb219 net: rmnet: fix bridge mode bugs
> > > > 8eccc6d06ea1 net: rmnet: use upper/lower device infrastructure
> > > > 835bbd892683 net: rmnet: do not allow to change mux id if mux id is 
> > > > duplicated
> > > > 5ce5e95d72f1 net: rmnet: remove rcu_read_lock in 
> > > > rmnet_force_unassociate_device()
> > > > 002b4bd9b51d net: rmnet: fix suspicious RCU usage
> > > > 8a027eee62f7 net: rmnet: fix NULL pointer dereference in 
> > > > rmnet_changelink()
> > > > 006cd2426a05 net: rmnet: fix NULL pointer dereference in 
> > > > rmnet_newlink()
> > > > cfc7f5fe5c61 hinic: fix a bug of rss configuration
> > > > e6f52d5ce172 hinic: fix a bug of setting hw_ioctxt
> > > > 65340b196811 hinic: fix a irq affinity bug
> > > > 1c7dde0a9bfc net: phy: mscc: fix firmware paths
> > > > 4d6ee3123d65 slip: not call free_netdev before rtnl_unlock in 
> > > > slip_open
> > > > d1984c92f072 signal: avoid double atomic counter increments for 
> > > > user accounting
> > > > df8e98b00941 kbuild: add dt_binding_check to PHONY in a correct 
> > > > place
> > > > fd1f29f2a824 kbuild: add dtbs_check to PHONY
> > > > bbf3acbb8a77 drm/amdgpu: fix memory leak during TDR test(v2)
> > > > 74c77d6a4ea9 blk-mq: insert passthrough request into hctx->dispatch 
> > > > directly
> > > > 74886a6d9de8 net: ll_temac: Handle DMA halt condition caused by 
> > > > buffer underrun
> > > > c2cffaea4fde net: ll_temac: Fix RX buffer descriptor handling on 
> > > > GFP_ATOMIC pressure
> > > > 6e3b69543b5e net: ll_temac: Add more error handling of 
> > > > dma_map_single() calls
> > > > 0c050d655475 net: ll_temac: Fix race condition causing TX hang
> > > > a342bb7b9902 mac80211: rx: avoid RCU list traversal under mutex
> > > > 5d92d834aa27 net: ks8851-ml: Fix IRQ handling and locking
> > > > d939f3389f9b net: usb: qmi_wwan: restore mtu min/max values after 
> > > > raw_ip switch
> > > > 6c663120ecdb scsi: libfc: free response frame from GPN_ID
> > > > 079e9ad79410 cfg80211: check reg_rule for NULL in 
> > > > handle_channel_custom()
> > > > a6f7e69cfafd tracing: Fix number printing bug in print_synth_event()
> > > > c35aa36fec91 selftests/rseq: Fix out-of-tree compilation
> > > > dd0d2d6ab6a2 HID: hid-bigbenff: fix race condition for scheduled 
> > > > work during removal
> > > > ccb747d805d3 HID: hid-bigbenff: call 

Re: [OE-core] [PATCH] lto.inc: Add global LTO distro policy file

2020-03-24 Thread Khem Raj
On Tue, Mar 24, 2020 at 4:50 AM Richard Purdie
 wrote:
>
> On Tue, 2020-03-24 at 13:27 +0200, Adrian Bunk wrote:
> > On Mon, Mar 23, 2020 at 04:25:29PM -0700, Khem Raj wrote:
> > > Distros which want to enable LTO can utilize this file, it only
> > > covers
> > > packages from OE-Core, other layers should include there own
> > > exclusion
> > > list for recipe which dont work with LTO
> > > ...
> > > +++ b/meta/conf/distro/include/lto.inc
> > > @@ -0,0 +1,23 @@
> > > +LTO_pn-glibc = ""
> > > +LTO_pn-gcc-runtime = ""
> > > +LTO_pn-libgcc-initial = ""
> > > +LTO_pn-libgcc = ""
> > > +LTO_pn-libpam = ""
> > > +LTO_pn-elfutils = ""
> > > +LTO_pn-perl = ""
> > > +LTO_pn-busybox = ""
> > > +LTO_pn-libxcrypt = ""
> > > +LTO_pn-curl = ""
> > > +LTO_pn-libcap = ""
> > > +LTO_pn-python3 = ""
> > > +LTO_pn-libproxy = ""
> > > +LTO_pn-libbsd = ""
> > > ...
> >
> > It would be better to have these in the individual recipes,
> > it is much easier to understand what is going on when all
> > recipe-specific information is in the recipe.
>
> I disagree. These files do let someone working on specific enablement
> see the status collectively and allow people to opt in/out of a given
> development feature.
>
> At some point in the future that can tip into making more sense in a
> recipe but its a staged process and things often make sense to start
> like this.

yeah. Maybe there should be a distro feature knob this inc file should
add which can be used by recipes
for an exception like adding configure options
>
> Cheers,
>
> Richard
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136644): 
https://lists.openembedded.org/g/openembedded-core/message/136644
Mute This Topic: https://lists.openembedded.org/mt/72506212/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.27

2020-03-24 Thread Bruce Ashfield
On Tue, Mar 24, 2020 at 2:35 AM Khem Raj  wrote:
>
> Hi Bruce
>
> This results in kernel-selftest failure regression
>
> errors.yoctoproject.org/Errors/Build/100385/
>
> and also same errors as reported here
> https://bugs.launchpad.net/ubuntu-kernel-tests/+bug/1867596
>
> I have fixed them via three patches that are here
>
> https://git.openembedded.org/openembedded-core-contrib/commit/?h=yoe/mut=2c15bc35809365db8c586df7c383ed01b00e6618
>
> Please cherry-pick these 3 patches as well on this pull while you are here.
>

Only one of the patches has any upstream information.

Are the reverts also being done in mainline ? Or is a different fix ?

What about the one that does have an upstream status, has it been
submitted to -stable ?

Bruce

> On Sat, Mar 21, 2020 at 6:18 PM Bruce Ashfield  
> wrote:
> >
> > From: Bruce Ashfield 
> >
> > Updating linux-yocto/5.4 to the latest korg -stable release that comprises
> > the following commits:
> >
> > 585e0cc08069 Linux 5.4.27
> > 20ba40d262b0 ipv4: ensure rcu_read_lock() in cipso_v4_error()
> > 81e0dac3780a ARM: 8961/2: Fix Kbuild issue caused by per-task stack 
> > protector GCC plugin
> > 1dc67950e2d1 HID: add ALWAYS_POLL quirk to lenovo pixart mouse
> > 5d961582ea88 HID: google: add moonball USB id
> > ae119b7e1247 mm: slub: add missing TID bump in kmem_cache_alloc_bulk()
> > 47b6d269dbbd ARM: 8958/1: rename missed uaccess .fixup section
> > 1a9e78628d44 ARM: 8957/1: VDSO: Match ARMv8 timer in cntvct_functional()
> > 881876bee657 net: qrtr: fix len of skb_put_padto in qrtr_node_enqueue
> > 235fb892d846 blk-mq: insert flush request to the front of dispatch queue
> > dbce8292217a jbd2: fix data races at struct journal_head
> > 99746e92fa4c sfc: fix timestamp reconstruction at 16-bit rollover points
> > e2b8b042c98b net: rmnet: fix packet forwarding in rmnet bridge mode
> > 34c550ebb219 net: rmnet: fix bridge mode bugs
> > 8eccc6d06ea1 net: rmnet: use upper/lower device infrastructure
> > 835bbd892683 net: rmnet: do not allow to change mux id if mux id is 
> > duplicated
> > 5ce5e95d72f1 net: rmnet: remove rcu_read_lock in 
> > rmnet_force_unassociate_device()
> > 002b4bd9b51d net: rmnet: fix suspicious RCU usage
> > 8a027eee62f7 net: rmnet: fix NULL pointer dereference in 
> > rmnet_changelink()
> > 006cd2426a05 net: rmnet: fix NULL pointer dereference in rmnet_newlink()
> > cfc7f5fe5c61 hinic: fix a bug of rss configuration
> > e6f52d5ce172 hinic: fix a bug of setting hw_ioctxt
> > 65340b196811 hinic: fix a irq affinity bug
> > 1c7dde0a9bfc net: phy: mscc: fix firmware paths
> > 4d6ee3123d65 slip: not call free_netdev before rtnl_unlock in slip_open
> > d1984c92f072 signal: avoid double atomic counter increments for user 
> > accounting
> > df8e98b00941 kbuild: add dt_binding_check to PHONY in a correct place
> > fd1f29f2a824 kbuild: add dtbs_check to PHONY
> > bbf3acbb8a77 drm/amdgpu: fix memory leak during TDR test(v2)
> > 74c77d6a4ea9 blk-mq: insert passthrough request into hctx->dispatch 
> > directly
> > 74886a6d9de8 net: ll_temac: Handle DMA halt condition caused by buffer 
> > underrun
> > c2cffaea4fde net: ll_temac: Fix RX buffer descriptor handling on 
> > GFP_ATOMIC pressure
> > 6e3b69543b5e net: ll_temac: Add more error handling of dma_map_single() 
> > calls
> > 0c050d655475 net: ll_temac: Fix race condition causing TX hang
> > a342bb7b9902 mac80211: rx: avoid RCU list traversal under mutex
> > 5d92d834aa27 net: ks8851-ml: Fix IRQ handling and locking
> > d939f3389f9b net: usb: qmi_wwan: restore mtu min/max values after 
> > raw_ip switch
> > 6c663120ecdb scsi: libfc: free response frame from GPN_ID
> > 079e9ad79410 cfg80211: check reg_rule for NULL in 
> > handle_channel_custom()
> > a6f7e69cfafd tracing: Fix number printing bug in print_synth_event()
> > c35aa36fec91 selftests/rseq: Fix out-of-tree compilation
> > dd0d2d6ab6a2 HID: hid-bigbenff: fix race condition for scheduled work 
> > during removal
> > ccb747d805d3 HID: hid-bigbenff: call hid_hw_stop() in case of error
> > 1bc271dd397d HID: hid-bigbenff: fix general protection fault caused by 
> > double kfree
> > 190e929e6cba HID: i2c-hid: add Trekstor Surfbook E11B to descriptor 
> > override
> > 87bd74b77531 ACPI: watchdog: Set default timeout in probe
> > 65b44497414b HID: apple: Add support for recent firmware on Magic 
> > Keyboards
> > 20eed7692749 ACPI: watchdog: Allow disabling WDAT at boot
> > f3c478104a9a drm/amdgpu: Fix TLB invalidation request when using 
> > semaphore
> > f6fccb0a0e59 netfilter: xt_hashlimit: unregister proc file before 
> > releasing mutex
> > 536942bf70b5 netfilter: hashlimit: do not use indirect calls during gc
> > 257edc6db943 Linux 5.4.26
> > 881f9469cc9f net/smc: cancel event worker during device removal
> > dbaba2a4db6e net/smc: check for valid 

Re: [OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.27

2020-03-24 Thread Khem Raj
On Tue, Mar 24, 2020 at 6:50 AM Bruce Ashfield  wrote:
>
> On Tue, Mar 24, 2020 at 9:42 AM Khem Raj  wrote:
> >
> > On Tue, Mar 24, 2020 at 5:40 AM Bruce Ashfield  
> > wrote:
> > >
> > > On Tue, Mar 24, 2020 at 2:35 AM Khem Raj  wrote:
> > > >
> > > > Hi Bruce
> > > >
> > > > This results in kernel-selftest failure regression
> > > >
> > > > errors.yoctoproject.org/Errors/Build/100385/
> > > >
> > > > and also same errors as reported here
> > > > https://bugs.launchpad.net/ubuntu-kernel-tests/+bug/1867596
> > > >
> > > > I have fixed them via three patches that are here
> > > >
> > > > https://git.openembedded.org/openembedded-core-contrib/commit/?h=yoe/mut=2c15bc35809365db8c586df7c383ed01b00e6618
> > > >
> > > > Please cherry-pick these 3 patches as well on this pull while you are 
> > > > here.
> > > >
> > >
> > > Only one of the patches has any upstream information.
> > >
> > > Are the reverts also being done in mainline ? Or is a different fix ?
> > >
> >
> > these are all results of half cherry picks into stable. all works fine in 
> > master
> > they should be reverted in stable but I have not sent them to lkml
>
> Aha. Thanks, that helps me understand what was going on.
>
> I'll amend the logs slightly and yank them into my tree.
>


thanks


> Cheers,
>
> Bruce
>
> >
> > > What about the one that does have an upstream status, has it been
> > > submitted to -stable ?
> > >
> > > Bruce
> > >
> > > > On Sat, Mar 21, 2020 at 6:18 PM Bruce Ashfield 
> > > >  wrote:
> > > > >
> > > > > From: Bruce Ashfield 
> > > > >
> > > > > Updating linux-yocto/5.4 to the latest korg -stable release that 
> > > > > comprises
> > > > > the following commits:
> > > > >
> > > > > 585e0cc08069 Linux 5.4.27
> > > > > 20ba40d262b0 ipv4: ensure rcu_read_lock() in cipso_v4_error()
> > > > > 81e0dac3780a ARM: 8961/2: Fix Kbuild issue caused by per-task 
> > > > > stack protector GCC plugin
> > > > > 1dc67950e2d1 HID: add ALWAYS_POLL quirk to lenovo pixart mouse
> > > > > 5d961582ea88 HID: google: add moonball USB id
> > > > > ae119b7e1247 mm: slub: add missing TID bump in 
> > > > > kmem_cache_alloc_bulk()
> > > > > 47b6d269dbbd ARM: 8958/1: rename missed uaccess .fixup section
> > > > > 1a9e78628d44 ARM: 8957/1: VDSO: Match ARMv8 timer in 
> > > > > cntvct_functional()
> > > > > 881876bee657 net: qrtr: fix len of skb_put_padto in 
> > > > > qrtr_node_enqueue
> > > > > 235fb892d846 blk-mq: insert flush request to the front of 
> > > > > dispatch queue
> > > > > dbce8292217a jbd2: fix data races at struct journal_head
> > > > > 99746e92fa4c sfc: fix timestamp reconstruction at 16-bit rollover 
> > > > > points
> > > > > e2b8b042c98b net: rmnet: fix packet forwarding in rmnet bridge 
> > > > > mode
> > > > > 34c550ebb219 net: rmnet: fix bridge mode bugs
> > > > > 8eccc6d06ea1 net: rmnet: use upper/lower device infrastructure
> > > > > 835bbd892683 net: rmnet: do not allow to change mux id if mux id 
> > > > > is duplicated
> > > > > 5ce5e95d72f1 net: rmnet: remove rcu_read_lock in 
> > > > > rmnet_force_unassociate_device()
> > > > > 002b4bd9b51d net: rmnet: fix suspicious RCU usage
> > > > > 8a027eee62f7 net: rmnet: fix NULL pointer dereference in 
> > > > > rmnet_changelink()
> > > > > 006cd2426a05 net: rmnet: fix NULL pointer dereference in 
> > > > > rmnet_newlink()
> > > > > cfc7f5fe5c61 hinic: fix a bug of rss configuration
> > > > > e6f52d5ce172 hinic: fix a bug of setting hw_ioctxt
> > > > > 65340b196811 hinic: fix a irq affinity bug
> > > > > 1c7dde0a9bfc net: phy: mscc: fix firmware paths
> > > > > 4d6ee3123d65 slip: not call free_netdev before rtnl_unlock in 
> > > > > slip_open
> > > > > d1984c92f072 signal: avoid double atomic counter increments for 
> > > > > user accounting
> > > > > df8e98b00941 kbuild: add dt_binding_check to PHONY in a correct 
> > > > > place
> > > > > fd1f29f2a824 kbuild: add dtbs_check to PHONY
> > > > > bbf3acbb8a77 drm/amdgpu: fix memory leak during TDR test(v2)
> > > > > 74c77d6a4ea9 blk-mq: insert passthrough request into 
> > > > > hctx->dispatch directly
> > > > > 74886a6d9de8 net: ll_temac: Handle DMA halt condition caused by 
> > > > > buffer underrun
> > > > > c2cffaea4fde net: ll_temac: Fix RX buffer descriptor handling on 
> > > > > GFP_ATOMIC pressure
> > > > > 6e3b69543b5e net: ll_temac: Add more error handling of 
> > > > > dma_map_single() calls
> > > > > 0c050d655475 net: ll_temac: Fix race condition causing TX hang
> > > > > a342bb7b9902 mac80211: rx: avoid RCU list traversal under mutex
> > > > > 5d92d834aa27 net: ks8851-ml: Fix IRQ handling and locking
> > > > > d939f3389f9b net: usb: qmi_wwan: restore mtu min/max values after 
> > > > > raw_ip switch
> > > > > 6c663120ecdb scsi: libfc: free response frame from GPN_ID
> > > > > 079e9ad79410 cfg80211: check reg_rule for NULL in 
> > > > > handle_channel_custom()
> > > 

[OE-core] [PATCH 2/2] gtk+3: turn explicit disabling of colord to PACKAGECONFIG

2020-03-24 Thread Ross Burton
From: Ross Burton 

Signed-off-by: Ross Burton 
---
 meta/recipes-gnome/gtk+/gtk+3.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-gnome/gtk+/gtk+3.inc 
b/meta/recipes-gnome/gtk+/gtk+3.inc
index 12fa83550a0..5d1f59ea1ac 100644
--- a/meta/recipes-gnome/gtk+/gtk+3.inc
+++ b/meta/recipes-gnome/gtk+/gtk+3.inc
@@ -37,7 +37,6 @@ EXTRA_OECONF += " \
  --disable-glibtest \
  --disable-xinerama \
  --enable-modules \
- --disable-colord \
  ${@bb.utils.contains("DISTRO_FEATURES", "x11", "", 
"--disable-gtk-doc", d)} \
  "
 
@@ -55,6 +54,7 @@ PACKAGECONFIG[glx] = "--enable-glx,--disable-glx,,libgl"
 PACKAGECONFIG[wayland] = 
"--enable-wayland-backend,--disable-wayland-backend,wayland wayland-protocols 
libxkbcommon virtual/egl virtual/libgles2 wayland-native"
 PACKAGECONFIG[cups] = "--enable-cups,--disable-cups,cups"
 PACKAGECONFIG[cloudprint] = "--enable-cloudprint,--disable-cloudprint,rest 
json-glib"
+PACKAGECONFIG[colord] = "--enable-colord,--disable-colord,colord"
 
 prepare_gtk_scripts() {
 mv ${D}${bindir}/gtk-update-icon-cache 
${D}${bindir}/gtk-update-icon-cache-3.0
-- 
2.25.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136648): 
https://lists.openembedded.org/g/openembedded-core/message/136648
Mute This Topic: https://lists.openembedded.org/mt/72517375/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 1/2] gtk+3: add cloudprint PACKAGECONFIG

2020-03-24 Thread Ross Burton
From: Ross Burton 

Signed-off-by: Ross Burton 
---
 meta/recipes-gnome/gtk+/gtk+3.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-gnome/gtk+/gtk+3.inc 
b/meta/recipes-gnome/gtk+/gtk+3.inc
index e1f1e0db314..12fa83550a0 100644
--- a/meta/recipes-gnome/gtk+/gtk+3.inc
+++ b/meta/recipes-gnome/gtk+/gtk+3.inc
@@ -54,6 +54,7 @@ PACKAGECONFIG[opengl] = 
"--enable-opengl,--disable-opengl,libepoxy"
 PACKAGECONFIG[glx] = "--enable-glx,--disable-glx,,libgl"
 PACKAGECONFIG[wayland] = 
"--enable-wayland-backend,--disable-wayland-backend,wayland wayland-protocols 
libxkbcommon virtual/egl virtual/libgles2 wayland-native"
 PACKAGECONFIG[cups] = "--enable-cups,--disable-cups,cups"
+PACKAGECONFIG[cloudprint] = "--enable-cloudprint,--disable-cloudprint,rest 
json-glib"
 
 prepare_gtk_scripts() {
 mv ${D}${bindir}/gtk-update-icon-cache 
${D}${bindir}/gtk-update-icon-cache-3.0
-- 
2.25.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136647): 
https://lists.openembedded.org/g/openembedded-core/message/136647
Mute This Topic: https://lists.openembedded.org/mt/72517374/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] Yocto Project Status WW12'20

2020-03-24 Thread Stephen Jolley
Current Dev Position: YP 3.1 M4 - Stabilization

Next Deadline: YP 3.1 M4 build date  3/30/2020

 

Next Team Meetings:

*   Bug Triage meeting Thursday Mar. 26th  at 7:30am PDT (
 https://zoom.us/j/454367603)
*   Monthly Project Meeting Tuesday Apr. 7th  at 8am PDT (
 https://zoom.us/j/990892712)
*   Weekly Engineering Sync Tuesday Mar. 24th  at 8am PDT (
 https://zoom.us/j/990892712)
*   Twitch -  See http://www.twitch.tv/letoatreidesthe2nd

 

Key Status/Updates:

*   M3 is through QA and due to be released.
*   The first M4 rc candidate is due to be built on Monday.
*   YP 3.1 has been announced as an LTS release:

https://www.yoctoproject.org/yocto-project-long-term-support-announced/
*   We've uncovered significant issues with the datastore connector
approach in tinfoil. There is a proposed fix which effectively rewrites it
which is not something undertaken lightly at this point in release but
reluctantly, is probably necessary.
*   The focus is now on stabilizing and bug fixing for the final release

 

YP 3.1 Milestone Dates:

*   YP 3.1 M3 being reviewed for release
*   YP 3.1 M4 build date  3/30/2020
*   YP 3.1 M4 release date  4/24/2020

 

Planned upcoming dot releases:

*   YP 3.0.3 build date  5/4/2020
*   YP 3.0.3 release date 5/15/2020
*   YP 2.7.4 build date  5/18/2020
*   YP 2.7.4 release date 5/29/2020

 

Tracking Metrics:

*   WDD 2681 (last week 2698) (

https://wiki.yoctoproject.org/charts/combo.html)
*   Poky Patch Metrics  

*   Total patches found: 1350 (last week 1351)
*   Patches in the Pending State: 535 (40%) [last week 534 (40%)]

 

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

*Cell:(208) 244-4460

* Email:  sjolley.yp...@gmail.com
 

 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136655): 
https://lists.openembedded.org/g/openembedded-core/message/136655
Mute This Topic: https://lists.openembedded.org/mt/72518725/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH v2 2/5] buildstats-plot.sh: filter by task

2020-03-24 Thread Luis Martins
Extend buildstats-plot.sh script to also accept the name
of the tasks as parameter.
This value will be passed directly to buildstats.sh is
already provides this option.

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh | 10 --
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 089621c9f8..1d22e60d73 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -39,6 +39,7 @@ set -o errexit
 
 BS_DIR="tmp/buildstats"
 N=10
+TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="utime"
 SUM=""
 OUTDATA_FILE="$PWD/buildstats-plot.out"
@@ -51,6 +52,8 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 (default: "$BS_DIR")
   -n N  Top N recipes to display. Ignored if -S is present
 (default: "$N")
+  -t tasks  The tasks to be computed
+(default: "$TASKS")
   -s stats  The stats to be matched. If more that one stat, units
 should be the same because data is plot as histogram.
 (see buildstats.sh -h for all options) or any other defined
@@ -64,7 +67,7 @@ EOM
 }
 
 # Parse and validate arguments
-while getopts "b:n:s:o:Sh" OPT; do
+while getopts "b:n:t:s:o:Sh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -72,6 +75,9 @@ while getopts "b:n:s:o:Sh" OPT; do
n)
N="$OPTARG"
;;
+t)
+TASKS="$OPTARG"
+;;
s)
STATS="$OPTARG"
;;
@@ -101,7 +107,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -H > $OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" -H > $OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136651): 
https://lists.openembedded.org/g/openembedded-core/message/136651
Mute This Topic: https://lists.openembedded.org/mt/72518490/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH v2 5/5] buildstats*.sh: fix spacing indentation in the files

2020-03-24 Thread Luis Martins
Fix indentation in buildstats.sh and buildstats-plot.sh
to convert every line to 4 spaces indentation.

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh |  56 -
 scripts/contrib/bb-perf/buildstats.sh  | 131 +++--
 2 files changed, 94 insertions(+), 93 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 320c4910a1..80fd30960c 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -72,40 +72,40 @@ EOM
 
 # Parse and validate arguments
 while getopts "b:n:r:t:s:o:aSh" OPT; do
-   case $OPT in
-   b)
-   BS_DIR="$OPTARG"
-   ;;
-   n)
-   N="$OPTARG"
-   ;;
+case $OPT in
+b)
+BS_DIR="$OPTARG"
+;;
+n)
+N="$OPTARG"
+;;
 r)
 RECIPE="-r $OPTARG"
 ;;
 t)
 TASKS="$OPTARG"
 ;;
-   s)
-   STATS="$OPTARG"
-   ;;
-   a)
+s)
+STATS="$OPTARG"
+;;
+a)
 ACCUMULATE="-a"
 ;;
-   S)
-   SUM="y"
-   ;;
-   o)
-   OUTDATA_FILE="$OPTARG"
-   ;;
-   h)
-   usage
-   exit 0
-   ;;
-   *)
-   usage
-   exit 1
-   ;;
-   esac
+S)
+SUM="y"
+;;
+o)
+OUTDATA_FILE="$OPTARG"
+;;
+h)
+usage
+exit 0
+;;
+*)
+usage
+exit 1
+;;
+esac
 done
 
 # Get number of stats
@@ -141,8 +141,8 @@ else
 declare -a sumargs
 j=0
 for i in `seq $nstats`; do
-   sumargs[j]=sum; j=$(( $j + 1 ))
-   sumargs[j]=`expr 3 + $i - 1`;  j=$(( $j + 1 ))
+sumargs[j]=sum; j=$(( $j + 1 ))
+sumargs[j]=`expr 3 + $i - 1`;  j=$(( $j + 1 ))
 done
 
 # Do the processing with datamash
diff --git a/scripts/contrib/bb-perf/buildstats.sh 
b/scripts/contrib/bb-perf/buildstats.sh
index 2f017efeb8..e45cfc146d 100755
--- a/scripts/contrib/bb-perf/buildstats.sh
+++ b/scripts/contrib/bb-perf/buildstats.sh
@@ -66,62 +66,63 @@ EOM
 
 # Parse and validate arguments
 while getopts "b:r:t:s:aHh" OPT; do
-   case $OPT in
-   b)
-   BS_DIR="$OPTARG"
-   ;;
-   r)
-   RECIPE="$OPTARG"
-   ;;
-   t)
-   TASKS="$OPTARG"
-   ;;
-   s)
-   STATS="$OPTARG"
-   ;;
-   a)
+case $OPT in
+b)
+BS_DIR="$OPTARG"
+;;
+r)
+RECIPE="$OPTARG"
+;;
+t)
+TASKS="$OPTARG"
+;;
+s)
+STATS="$OPTARG"
+;;
+a)
 ACCUMULATE="y"
 ;;
-   H)
-   HEADER="y"
-   ;;
-   h)
-   usage
-   exit 0
-   ;;
-   *)
-   usage
-   exit 1
-   ;;
-   esac
+H)
+HEADER="y"
+;;
+h)
+usage
+exit 0
+;;
+*)
+usage
+exit 1
+;;
+esac
 done
 
 # Ensure the buildstats folder exists
 if [ ! -d "$BS_DIR" ]; then
-   echo "ERROR: $BS_DIR does not exist"
-   usage
-   exit 1
+echo "ERROR: $BS_DIR does not exist"
+usage
+exit 1
 fi
 
 stats=""
 IFS=":"
 for stat in ${STATS}; do
-   case $stat in
-   TIME)
-   stats="${stats}:${TIME}"
-   ;;
-   IO)
-   stats="${stats}:${IO}"
-   ;;
-   RUSAGE)
-   stats="${stats}:${RUSAGE}"
-   ;;
-   CHILD_RUSAGE)
-   stats="${stats}:${CHILD_RUSAGE}"
-   ;;
-   *)
-   stats="${STATS}"
-   esac
+case $stat in
+TIME)
+stats="${stats}:${TIME}"
+;;
+IO)
+stats="${stats}:${IO}"
+;;
+RUSAGE)
+stats="${stats}:${RUSAGE}"
+;;
+CHILD_RUSAGE)
+stats="${stats}:${CHILD_RUSAGE}"
+;;
+*)
+stats="${STATS}"
+;;
+esac
 done
 
 # remove possible colon at the beginning
@@ -140,27 +141,27 @@ for task in ${TASKS}; do
 task="do_${task}"
 for file in $(find ${BS_DIR} -type f -path *${RECIPE}*/${task} | awk 
'BEGIN{ ORS=""; OFS=":" } { print $0,"" }'); do
 recipe="$(basename $(dirname $file))"
-   times=""
-   for stat in ${stats}; do
-   [ -z "$stat" ] && { echo "empty stats"; }
-   time=$(sed -n -e "s/^\($stat\): \\(.*\\)/\\2/p" $file)
-   # in case the stat is not present, set the value as NA
-   [ -z "$time" ] && { time="NA"; }
-   # Append it to times
-   if [ -z "$times" ]; then
-   times="${time}"
-   else
-   

[OE-core] [PATCH v2 3/5] buildstats*.sh: add accumulate parameter to scripts

2020-03-24 Thread Luis Martins
Add option to accumulate stats values per recipe, allowing
to sum related values such as memory (main process + childs).
This is specially useful when debugging the performance of
the overall bitbake build system.

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh |  9 +++--
 scripts/contrib/bb-perf/buildstats.sh  | 23 --
 2 files changed, 28 insertions(+), 4 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 1d22e60d73..f26312f355 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -41,6 +41,7 @@ BS_DIR="tmp/buildstats"
 N=10
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="utime"
+ACCUMULATE=""
 SUM=""
 OUTDATA_FILE="$PWD/buildstats-plot.out"
 
@@ -59,6 +60,7 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 (see buildstats.sh -h for all options) or any other defined
 (build)stat separated by colons, i.e. stime:utime
 (default: "$STATS")
+  -aAccumulate all stats values for found recipes
   -SSum values for a particular stat for found recipes
   -oOutput data file.
 (default: "$OUTDATA_FILE")
@@ -67,7 +69,7 @@ EOM
 }
 
 # Parse and validate arguments
-while getopts "b:n:t:s:o:Sh" OPT; do
+while getopts "b:n:t:s:o:aSh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -81,6 +83,9 @@ while getopts "b:n:t:s:o:Sh" OPT; do
s)
STATS="$OPTARG"
;;
+   a)
+ACCUMULATE="-a"
+;;
S)
SUM="y"
;;
@@ -107,7 +112,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" -H > $OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" $ACCUMULATE -H > 
$OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
diff --git a/scripts/contrib/bb-perf/buildstats.sh 
b/scripts/contrib/bb-perf/buildstats.sh
index e9ec2d476a..26bfaca86b 100755
--- a/scripts/contrib/bb-perf/buildstats.sh
+++ b/scripts/contrib/bb-perf/buildstats.sh
@@ -38,6 +38,7 @@ Child rusage ru_nivcsw"
 BS_DIR="tmp/buildstats"
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="$TIME"
+ACCUMULATE=""
 HEADER="" # No header by default
 
 function usage {
@@ -56,12 +57,13 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 IO=$IO
 RUSAGE=$RUSAGE
 CHILD_RUSAGE=$CHILD_RUSAGE
+  -aAccumulate all stats values for found recipes
   -hDisplay this help message
 EOM
 }
 
 # Parse and validate arguments
-while getopts "b:t:s:Hh" OPT; do
+while getopts "b:t:s:aHh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -72,6 +74,9 @@ while getopts "b:t:s:Hh" OPT; do
s)
STATS="$OPTARG"
;;
+   a)
+ACCUMULATE="y"
+;;
H)
HEADER="y"
;;
@@ -118,7 +123,13 @@ done
 stats="$(echo "$stats" | sed -e 's/^://1')"
 
 # Provide a header if required by the user
-[ -n "$HEADER" ] && { echo "task:recipe:$stats"; }
+if [ -n "$HEADER" ] ; then
+if [ -n "$ACCUMULATE" ]; then
+echo "task:recipe:accumulated(${stats//:/;})"
+else
+echo "task:recipe:$stats"
+fi
+fi
 
 for task in ${TASKS}; do
 task="do_${task}"
@@ -137,6 +148,14 @@ for task in ${TASKS}; do
times="${times} ${time}"
fi
done
+if [ -n "$ACCUMULATE" ]; then
+IFS=' '; valuesarray=(${times}); IFS=':'
+times=0
+for value in "${valuesarray[@]}"; do
+[ "$value" == "NA" ] && { echo "ERROR: stat is not present."; 
usage; exit 1; }
+times=$(( $times + $value ))
+done
+fi
 echo "${task} ${recipe} ${times}"
 done
 done
-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136652): 
https://lists.openembedded.org/g/openembedded-core/message/136652
Mute This Topic: https://lists.openembedded.org/mt/72518492/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH v2 4/5] buildstats*.sh: add recipe parameter to scripts

2020-03-24 Thread Luis Martins
Add option to filter the stats results for a single recipe, allowing
to easilly visuality the task resource consumption per recipe.
This is specially useful when debugging the performance of
the overall bitbake build system.

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh | 9 +++--
 scripts/contrib/bb-perf/buildstats.sh  | 9 +++--
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index f26312f355..320c4910a1 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -39,6 +39,7 @@ set -o errexit
 
 BS_DIR="tmp/buildstats"
 N=10
+RECIPE=""
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="utime"
 ACCUMULATE=""
@@ -53,6 +54,7 @@ Usage: $CMD [-b buildstats_dir] [-t do_task]
 (default: "$BS_DIR")
   -n N  Top N recipes to display. Ignored if -S is present
 (default: "$N")
+  -r recipe The recipe mask to be searched
   -t tasks  The tasks to be computed
 (default: "$TASKS")
   -s stats  The stats to be matched. If more that one stat, units
@@ -69,7 +71,7 @@ EOM
 }
 
 # Parse and validate arguments
-while getopts "b:n:t:s:o:aSh" OPT; do
+while getopts "b:n:r:t:s:o:aSh" OPT; do
case $OPT in
b)
BS_DIR="$OPTARG"
@@ -77,6 +79,9 @@ while getopts "b:n:t:s:o:aSh" OPT; do
n)
N="$OPTARG"
;;
+r)
+RECIPE="-r $OPTARG"
+;;
 t)
 TASKS="$OPTARG"
 ;;
@@ -112,7 +117,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" $ACCUMULATE -H > 
$OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -t "$TASKS" $RECIPE $ACCUMULATE -H 
> $OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
diff --git a/scripts/contrib/bb-perf/buildstats.sh 
b/scripts/contrib/bb-perf/buildstats.sh
index 26bfaca86b..2f017efeb8 100755
--- a/scripts/contrib/bb-perf/buildstats.sh
+++ b/scripts/contrib/bb-perf/buildstats.sh
@@ -36,6 +36,7 @@ Child rusage ru_majflt:Child rusage ru_inblock:Child rusage 
ru_oublock:Child rus
 Child rusage ru_nivcsw"
 
 BS_DIR="tmp/buildstats"
+RECIPE=""
 
TASKS="compile:configure:fetch:install:patch:populate_lic:populate_sysroot:unpack"
 STATS="$TIME"
 ACCUMULATE=""
@@ -47,6 +48,7 @@ cat <-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136653): 
https://lists.openembedded.org/g/openembedded-core/message/136653
Mute This Topic: https://lists.openembedded.org/mt/72518493/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH v2 1/5] buildstats-plot.sh: fix invoking buildstats.sh

2020-03-24 Thread Luis Martins
Fix the internal call to buildstats.sh by also providing
the buildstats folder location in the system, which might
differ from the default location.

Signed-off-by: Luis Martins 
---
 scripts/contrib/bb-perf/buildstats-plot.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/contrib/bb-perf/buildstats-plot.sh 
b/scripts/contrib/bb-perf/buildstats-plot.sh
index 898834e5ac..089621c9f8 100755
--- a/scripts/contrib/bb-perf/buildstats-plot.sh
+++ b/scripts/contrib/bb-perf/buildstats-plot.sh
@@ -101,7 +101,7 @@ CD=$(dirname $0)
 
 # Parse buildstats recipes to produce a single table
 OUTBUILDSTATS="$PWD/buildstats.log"
-$CD/buildstats.sh -H -s "$STATS" -H > $OUTBUILDSTATS
+$CD/buildstats.sh -b "$BS_DIR" -s "$STATS" -H > $OUTBUILDSTATS
 
 # Get headers
 HEADERS=$(cat $OUTBUILDSTATS | sed -n -e '1s/ /-/g' -e '1s/:/ /gp')
-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136650): 
https://lists.openembedded.org/g/openembedded-core/message/136650
Mute This Topic: https://lists.openembedded.org/mt/72518489/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH v2 0/5] improvements to buildstats scripts

2020-03-24 Thread Luis Martins
This patches add some new functionalities to the buildstats
scripts to allow easier debugging of the bitbake performance.
Some new features like accumulating the stats values allow
to compare the total amount of memory used by a single task
(main process + child processes) and other features allow
convenient analysis of task performance per recipe.

changes from v1
* removed unecessary "Upstream-Status: Pending" line

Luis Martins (5):
  buildstats-plot.sh: fix invoking buildstats.sh
  buildstats-plot.sh: filter by task
  buildstats*.sh: add accumulate parameter to scripts
  buildstats*.sh: add recipe parameter to scripts
  buildstats*.sh: fix spacing indentation in the files

 scripts/contrib/bb-perf/buildstats-plot.sh |  74 ++-
 scripts/contrib/bb-perf/buildstats.sh  | 139 -
 2 files changed, 127 insertions(+), 86 deletions(-)

-- 
2.25.2

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136649): 
https://lists.openembedded.org/g/openembedded-core/message/136649
Mute This Topic: https://lists.openembedded.org/mt/72518487/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH] qemu.inc: Always disable compile time git updating

2020-03-24 Thread Mark Hatle
If the SRC_URI is switched to be git based, QEMU will automatically update
itself at compile time for select git submodules.  This by passes the
bitbake git fetcher.  These modules are always present in the release
tarballs, so only are problematic when used with git based SRC_URIs.

These switches will have no effect on a tarball based SRC_URI.

Signed-off-by: Mark Hatle 
---
 meta/recipes-devtools/qemu/qemu.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-devtools/qemu/qemu.inc 
b/meta/recipes-devtools/qemu/qemu.inc
index 7cf436783d..a6205b06c9 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -82,6 +82,8 @@ EXTRA_OECONF = " \
 --disable-strip \
 --disable-werror \
 --extra-cflags='${CFLAGS}' \
+--with-git=/bin/false \
+--disable-git-update \
 ${PACKAGECONFIG_CONFARGS} \
 "
 
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136656): 
https://lists.openembedded.org/g/openembedded-core/message/136656
Mute This Topic: https://lists.openembedded.org/mt/72521664/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Tom Hochstein
The security flags were missing from the SDK toolchain
because they were added specifically to class-target.
Add them to class-cross-canadian as well.

Signed-off-by: Tom Hochstein 
---
 meta/conf/distro/include/security_flags.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/conf/distro/include/security_flags.inc 
b/meta/conf/distro/include/security_flags.inc
index aaf04e9e59..568d03693c 100644
--- a/meta/conf/distro/include/security_flags.inc
+++ b/meta/conf/distro/include/security_flags.inc
@@ -57,6 +57,8 @@ SECURITY_STRINGFORMAT_pn-gcc = ""
 
 TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
 TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
+TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
+TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"
 
 SECURITY_STACK_PROTECTOR_pn-gcc-runtime = ""
 SECURITY_STACK_PROTECTOR_pn-glibc = ""
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136657): 
https://lists.openembedded.org/g/openembedded-core/message/136657
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Antoine Manache
I've already sent this patch few months ago but not taken into account yet.
AM

>From : Antoine Manache 
Date: mer. 20 nov. 2019 at 16:44
Subject: [OE-core] [PATCH v2] sdk: environment script doesn't set security
options in CC and LDFLAGS variables
To: 

When building a SDK from a DISTRO with security flags enabled, options
added to
CC and LDFLAGS are not replicated in the SDK environment script.
This could lead to some situations where an application compiled with
the SDK and having some security weaknesses correctly runs on target but
crashes once integrated to the core image built with the full Yocto
stack.

Signed-off-by: Antoine Manache 
---
 meta/conf/distro/include/security_flags.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/conf/distro/include/security_flags.inc
b/meta/conf/distro/include/security_flags.inc
index 620978a8ed..329482bfa3 100644
--- a/meta/conf/distro/include/security_flags.inc
+++ b/meta/conf/distro/include/security_flags.inc
@@ -56,7 +56,9 @@ SECURITY_STRINGFORMAT_pn-busybox = ""
 SECURITY_STRINGFORMAT_pn-gcc = ""

 TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
+TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
 TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
+TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"

 SECURITY_STACK_PROTECTOR_pn-gcc-runtime = ""
 SECURITY_STACK_PROTECTOR_pn-glibc = ""

--
2.11.0

Le mar. 24 mars 2020 à 18:03, Tom Hochstein  a
écrit :

> The security flags were missing from the SDK toolchain
> because they were added specifically to class-target.
> Add them to class-cross-canadian as well.
>
> Signed-off-by: Tom Hochstein 
> ---
>  meta/conf/distro/include/security_flags.inc | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/meta/conf/distro/include/security_flags.inc
> b/meta/conf/distro/include/security_flags.inc
> index aaf04e9e59..568d03693c 100644
> --- a/meta/conf/distro/include/security_flags.inc
> +++ b/meta/conf/distro/include/security_flags.inc
> @@ -57,6 +57,8 @@ SECURITY_STRINGFORMAT_pn-gcc = ""
>
>  TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
>  TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
> +TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
> +TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"
>
>  SECURITY_STACK_PROTECTOR_pn-gcc-runtime = ""
>  SECURITY_STACK_PROTECTOR_pn-glibc = ""
> --
> 2.17.1
>
> 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136658): 
https://lists.openembedded.org/g/openembedded-core/message/136658
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Denys Dmytriyenko
On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> Instead of disabling unconditionally, add PACKAGECONFIG for
> qt5. Imported from meta-qt5 layer.
> 
> Signed-off-by: Anuj Mittal 
> ---
>  .../gstreamer/gstreamer1.0-plugins-good_1.16.2.bb  | 7 
> +--
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git 
> a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb 
> b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> index f3e0cee..8b34111 100644
> --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = 
> "-Dpng=enabled,-Dpng=disabled,libpng"
>  PACKAGECONFIG[libv4l2]= 
> "-Dv4l2-libv4l2=enabled,-Dv4l2-libv4l2=disabled,v4l-utils"
>  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-Dmpg123=disabled,mpg123"
>  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-Dpulse=disabled,pulseaudio"
> +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-Dqt5=disabled,qtbase 
> qtdeclarative qtbase-native"

Does it make oe-core depend on meta-qt5 layer?

What is the policy here for having inter-layer dependencies in DEPENDS vs. 
PACKAGECONFIG without explicitly listing them in LAYERDEPENDS?

-- 
Denys


>  PACKAGECONFIG[soup]   = "-Dsoup=enabled,-Dsoup=disabled,libsoup-2.4"
>  PACKAGECONFIG[speex]  = "-Dspeex=enabled,-Dspeex=disabled,speex"
>  PACKAGECONFIG[taglib] = "-Dtaglib=enabled,-Dtaglib=disabled,taglib"
> @@ -50,11 +51,6 @@ PACKAGECONFIG[vpx]= 
> "-Dvpx=enabled,-Dvpx=disabled,libvpx"
>  PACKAGECONFIG[wavpack]= "-Dwavpack=enabled,-Dwavpack=disabled,wavpack"
>  PACKAGECONFIG[x11]= 
> "${X11ENABLEOPTS},${X11DISABLEOPTS},${X11DEPENDS}"
>  
> -# qt5 support is disabled, because it is not present in OE core, and 
> requires more work than
> -# just adding a packageconfig (it requires access to moc, uic, rcc, and 
> qmake paths).
> -# This is better done in a separate qt5 layer (which then should add a "qt5" 
> packageconfig
> -# in a gstreamer1.0-plugins-good bbappend).
> -
>  EXTRA_OEMESON += " \
>  -Daalib=disabled \
>  -Ddirectsound=disabled \
> @@ -64,7 +60,6 @@ EXTRA_OEMESON += " \
>  -Doss4=disabled \
>  -Dosxaudio=disabled \
>  -Dosxvideo=disabled \
> --Dqt5=disabled \
>  -Dshout2=disabled \
>  -Dtwolame=disabled \
>  -Dwaveform=disabled \
> -- 
> 2.7.4
> 
> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136659): 
https://lists.openembedded.org/g/openembedded-core/message/136659
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Denys Dmytriyenko
On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> > > Instead of disabling unconditionally, add PACKAGECONFIG for
> > > qt5. Imported from meta-qt5 layer.
> > > 
> > > Signed-off-by: Anuj Mittal 
> > > ---
> > >  .../gstreamer/gstreamer1.0-plugins-
> > > good_1.16.2.bb  | 7 +--
> > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > 
> > > diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > plugins-good_1.16.2.bb b/meta/recipes-
> > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > index f3e0cee..8b34111 100644
> > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > good_1.16.2.bb
> > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > good_1.16.2.bb
> > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-Dpng=enabled,-
> > > Dpng=disabled,libpng"
> > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-Dv4l2-
> > > libv4l2=disabled,v4l-utils"
> > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > Dmpg123=disabled,mpg123"
> > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > Dpulse=disabled,pulseaudio"
> > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-Dqt5=disabled,qtbase
> > > qtdeclarative qtbase-native"
> > 
> > Does it make oe-core depend on meta-qt5 layer?
> > 
> > What is the policy here for having inter-layer dependencies in
> > DEPENDS vs. PACKAGECONFIG without explicitly listing them in
> > LAYERDEPENDS?
> 
> We have a clear policy that listing such options is ok as long as its
> not enabled by default.

Richard,

This is good, thank you for confirming. Do you have a link where this is 
clearly explained? Thanks again.

-- 
Denys
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136662): 
https://lists.openembedded.org/g/openembedded-core/message/136662
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 14:12 -0400, Denys Dmytriyenko wrote:
> On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> > On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> > > > Instead of disabling unconditionally, add PACKAGECONFIG for
> > > > qt5. Imported from meta-qt5 layer.
> > > > 
> > > > Signed-off-by: Anuj Mittal 
> > > > ---
> > > >  .../gstreamer/gstreamer1.0-plugins-
> > > > good_1.16.2.bb  | 7 +--
> > > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > > 
> > > > diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > plugins-good_1.16.2.bb b/meta/recipes-
> > > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > > index f3e0cee..8b34111 100644
> > > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > > good_1.16.2.bb
> > > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > > good_1.16.2.bb
> > > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-Dpng=enabled,-
> > > > Dpng=disabled,libpng"
> > > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-Dv4l2-
> > > > libv4l2=disabled,v4l-utils"
> > > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > > Dmpg123=disabled,mpg123"
> > > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > > Dpulse=disabled,pulseaudio"
> > > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-
> > > > Dqt5=disabled,qtbase
> > > > qtdeclarative qtbase-native"
> > > 
> > > Does it make oe-core depend on meta-qt5 layer?
> > > 
> > > What is the policy here for having inter-layer dependencies in
> > > DEPENDS vs. PACKAGECONFIG without explicitly listing them in
> > > LAYERDEPENDS?
> > 
> > We have a clear policy that listing such options is ok as long as
> > its
> > not enabled by default.
> 
> Richard,
> 
> This is good, thank you for confirming. Do you have a link where this
> is clearly explained? Thanks again.

Its come up many times over the years and we've been consistent about
it. I don't have a link but there has been mailing list discussion and
perhaps TSC discussion long ago, I simply don't remember. There are
many examples of that in OE-Core.

The alternative is forcing more into bbappends which doesn't seem to
gain anything (PACKAGECONFIG itself could be changed from distro config
for example).

Its also easy to catch bad defaults since the recipes would no longer
parse due to missing dependencies which we'd quickly spot.

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136663): 
https://lists.openembedded.org/g/openembedded-core/message/136663
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Denys Dmytriyenko
On Tue, Mar 24, 2020 at 07:29:42PM +, Richard Purdie wrote:
> On Tue, 2020-03-24 at 14:43 -0400, Denys Dmytriyenko wrote:
> > On Tue, Mar 24, 2020 at 02:33:19PM -0400, Denys Dmytriyenko wrote:
> > > On Tue, Mar 24, 2020 at 06:15:51PM +, Richard Purdie wrote:
> > > > On Tue, 2020-03-24 at 14:12 -0400, Denys Dmytriyenko wrote:
> > > > > On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> > > > > > On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > > > > > > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal
> > > > > > > wrote:
> > > > > > > > Instead of disabling unconditionally, add PACKAGECONFIG
> > > > > > > > for
> > > > > > > > qt5. Imported from meta-qt5 layer.
> > > > > > > > 
> > > > > > > > Signed-off-by: Anuj Mittal 
> > > > > > > > ---
> > > > > > > >  .../gstreamer/gstreamer1.0-plugins-
> > > > > > > > good_1.16.2.bb  | 7 +--
> > > > > > > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > > > > > > 
> > > > > > > > diff --git a/meta/recipes-
> > > > > > > > multimedia/gstreamer/gstreamer1.0-
> > > > > > > > plugins-good_1.16.2.bb b/meta/recipes-
> > > > > > > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > > > > > > index f3e0cee..8b34111 100644
> > > > > > > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > > > plugins-
> > > > > > > > good_1.16.2.bb
> > > > > > > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > > > plugins-
> > > > > > > > good_1.16.2.bb
> > > > > > > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-
> > > > > > > > Dpng=enabled,-
> > > > > > > > Dpng=disabled,libpng"
> > > > > > > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-
> > > > > > > > Dv4l2-
> > > > > > > > libv4l2=disabled,v4l-utils"
> > > > > > > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > > > > > > Dmpg123=disabled,mpg123"
> > > > > > > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > > > > > > Dpulse=disabled,pulseaudio"
> > > > > > > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-
> > > > > > > > Dqt5=disabled,qtbase
> > > > > > > > qtdeclarative qtbase-native"
> > > > > > > 
> > > > > > > Does it make oe-core depend on meta-qt5 layer?
> > > > > > > 
> > > > > > > What is the policy here for having inter-layer dependencies
> > > > > > > in
> > > > > > > DEPENDS vs. PACKAGECONFIG without explicitly listing them
> > > > > > > in
> > > > > > > LAYERDEPENDS?
> > > > > > 
> > > > > > We have a clear policy that listing such options is ok as
> > > > > > long as
> > > > > > its
> > > > > > not enabled by default.
> > > > > 
> > > > > Richard,
> > > > > 
> > > > > This is good, thank you for confirming. Do you have a link
> > > > > where this
> > > > > is clearly explained? Thanks again.
> > > > 
> > > > Its come up many times over the years and we've been consistent
> > > > about
> > > > it. I don't have a link but there has been mailing list
> > > > discussion and
> > > > perhaps TSC discussion long ago, I simply don't remember. There
> > > > are
> > > > many examples of that in OE-Core.
> > > > 
> > > > The alternative is forcing more into bbappends which doesn't seem
> > > > to
> > > > gain anything (PACKAGECONFIG itself could be changed from distro
> > > > config
> > > > for example).
> > > > 
> > > > Its also easy to catch bad defaults since the recipes would no
> > > > longer
> > > > parse due to missing dependencies which we'd quickly spot.
> > > 
> > > Richard,
> > > 
> > > Yes, yes, I understanf all that from a technical perspective and
> > > agree with 
> > > it, but I was thinking about a more clear definition of this policy
> > > to be able 
> > > to point anyone having such questions in the future...
> > 
> > BTW, I don't remember discussing this at YP TSC... Maybe OE TSC
> > covered this in the past?
> 
> Metadata architecture decision would be the OE TSC and "TSC discussion
> long ago" means something from several years ago.

Thought so - were there any meeting minutes from back then? Sorry for being so 
persistent, just trying to find some paper trail... :)

-- 
Denys
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136671): 
https://lists.openembedded.org/g/openembedded-core/message/136671
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> > Instead of disabling unconditionally, add PACKAGECONFIG for
> > qt5. Imported from meta-qt5 layer.
> > 
> > Signed-off-by: Anuj Mittal 
> > ---
> >  .../gstreamer/gstreamer1.0-plugins-
> > good_1.16.2.bb  | 7 +--
> >  1 file changed, 1 insertion(+), 6 deletions(-)
> > 
> > diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > plugins-good_1.16.2.bb b/meta/recipes-
> > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > index f3e0cee..8b34111 100644
> > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > good_1.16.2.bb
> > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > good_1.16.2.bb
> > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-Dpng=enabled,-
> > Dpng=disabled,libpng"
> >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-Dv4l2-
> > libv4l2=disabled,v4l-utils"
> >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > Dmpg123=disabled,mpg123"
> >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > Dpulse=disabled,pulseaudio"
> > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-Dqt5=disabled,qtbase
> > qtdeclarative qtbase-native"
> 
> Does it make oe-core depend on meta-qt5 layer?
> 
> What is the policy here for having inter-layer dependencies in
> DEPENDS vs. PACKAGECONFIG without explicitly listing them in
> LAYERDEPENDS?

We have a clear policy that listing such options is ok as long as its
not enabled by default.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136661): 
https://lists.openembedded.org/g/openembedded-core/message/136661
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 14:33 -0400, Denys Dmytriyenko wrote:
> On Tue, Mar 24, 2020 at 06:15:51PM +, Richard Purdie wrote:
> > On Tue, 2020-03-24 at 14:12 -0400, Denys Dmytriyenko wrote:
> > > On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> > > > On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > > > > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> > > > > > Instead of disabling unconditionally, add PACKAGECONFIG for
> > > > > > qt5. Imported from meta-qt5 layer.
> > > > > > 
> > > > > > Signed-off-by: Anuj Mittal 
> > > > > > ---
> > > > > >  .../gstreamer/gstreamer1.0-plugins-
> > > > > > good_1.16.2.bb  | 7 +--
> > > > > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > > > > 
> > > > > > diff --git a/meta/recipes-
> > > > > > multimedia/gstreamer/gstreamer1.0-
> > > > > > plugins-good_1.16.2.bb b/meta/recipes-
> > > > > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > > > > index f3e0cee..8b34111 100644
> > > > > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > plugins-
> > > > > > good_1.16.2.bb
> > > > > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > plugins-
> > > > > > good_1.16.2.bb
> > > > > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-
> > > > > > Dpng=enabled,-
> > > > > > Dpng=disabled,libpng"
> > > > > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-
> > > > > > Dv4l2-
> > > > > > libv4l2=disabled,v4l-utils"
> > > > > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > > > > Dmpg123=disabled,mpg123"
> > > > > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > > > > Dpulse=disabled,pulseaudio"
> > > > > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-
> > > > > > Dqt5=disabled,qtbase
> > > > > > qtdeclarative qtbase-native"
> > > > > 
> > > > > Does it make oe-core depend on meta-qt5 layer?
> > > > > 
> > > > > What is the policy here for having inter-layer dependencies
> > > > > in
> > > > > DEPENDS vs. PACKAGECONFIG without explicitly listing them in
> > > > > LAYERDEPENDS?
> > > > 
> > > > We have a clear policy that listing such options is ok as long
> > > > as
> > > > its
> > > > not enabled by default.
> > > 
> > > Richard,
> > > 
> > > This is good, thank you for confirming. Do you have a link where
> > > this
> > > is clearly explained? Thanks again.
> > 
> > Its come up many times over the years and we've been consistent
> > about
> > it. I don't have a link but there has been mailing list discussion
> > and
> > perhaps TSC discussion long ago, I simply don't remember. There are
> > many examples of that in OE-Core.
> > 
> > The alternative is forcing more into bbappends which doesn't seem
> > to
> > gain anything (PACKAGECONFIG itself could be changed from distro
> > config
> > for example).
> > 
> > Its also easy to catch bad defaults since the recipes would no
> > longer
> > parse due to missing dependencies which we'd quickly spot.
> 
> Yes, yes, I understanf all that from a technical perspective and
> agree with it, but I was thinking about a more clear definition of
> this policy to be able to point anyone having such questions in the
> future...

Sure, that sounds great. This doesn't happen without someone doing it
though.

I have not looked at the wiki to see if we did already document it. I
guess you already searched there before asking the question. If we
don't have a policy documented, you can edit the wiki and add something
?

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136667): 
https://lists.openembedded.org/g/openembedded-core/message/136667
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 14:43 -0400, Denys Dmytriyenko wrote:
> On Tue, Mar 24, 2020 at 02:33:19PM -0400, Denys Dmytriyenko wrote:
> > On Tue, Mar 24, 2020 at 06:15:51PM +, Richard Purdie wrote:
> > > On Tue, 2020-03-24 at 14:12 -0400, Denys Dmytriyenko wrote:
> > > > On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> > > > > On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > > > > > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal
> > > > > > wrote:
> > > > > > > Instead of disabling unconditionally, add PACKAGECONFIG
> > > > > > > for
> > > > > > > qt5. Imported from meta-qt5 layer.
> > > > > > > 
> > > > > > > Signed-off-by: Anuj Mittal 
> > > > > > > ---
> > > > > > >  .../gstreamer/gstreamer1.0-plugins-
> > > > > > > good_1.16.2.bb  | 7 +--
> > > > > > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > > > > > 
> > > > > > > diff --git a/meta/recipes-
> > > > > > > multimedia/gstreamer/gstreamer1.0-
> > > > > > > plugins-good_1.16.2.bb b/meta/recipes-
> > > > > > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > > > > > index f3e0cee..8b34111 100644
> > > > > > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > > plugins-
> > > > > > > good_1.16.2.bb
> > > > > > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > > plugins-
> > > > > > > good_1.16.2.bb
> > > > > > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-
> > > > > > > Dpng=enabled,-
> > > > > > > Dpng=disabled,libpng"
> > > > > > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-
> > > > > > > Dv4l2-
> > > > > > > libv4l2=disabled,v4l-utils"
> > > > > > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > > > > > Dmpg123=disabled,mpg123"
> > > > > > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > > > > > Dpulse=disabled,pulseaudio"
> > > > > > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-
> > > > > > > Dqt5=disabled,qtbase
> > > > > > > qtdeclarative qtbase-native"
> > > > > > 
> > > > > > Does it make oe-core depend on meta-qt5 layer?
> > > > > > 
> > > > > > What is the policy here for having inter-layer dependencies
> > > > > > in
> > > > > > DEPENDS vs. PACKAGECONFIG without explicitly listing them
> > > > > > in
> > > > > > LAYERDEPENDS?
> > > > > 
> > > > > We have a clear policy that listing such options is ok as
> > > > > long as
> > > > > its
> > > > > not enabled by default.
> > > > 
> > > > Richard,
> > > > 
> > > > This is good, thank you for confirming. Do you have a link
> > > > where this
> > > > is clearly explained? Thanks again.
> > > 
> > > Its come up many times over the years and we've been consistent
> > > about
> > > it. I don't have a link but there has been mailing list
> > > discussion and
> > > perhaps TSC discussion long ago, I simply don't remember. There
> > > are
> > > many examples of that in OE-Core.
> > > 
> > > The alternative is forcing more into bbappends which doesn't seem
> > > to
> > > gain anything (PACKAGECONFIG itself could be changed from distro
> > > config
> > > for example).
> > > 
> > > Its also easy to catch bad defaults since the recipes would no
> > > longer
> > > parse due to missing dependencies which we'd quickly spot.
> > 
> > Richard,
> > 
> > Yes, yes, I understanf all that from a technical perspective and
> > agree with 
> > it, but I was thinking about a more clear definition of this policy
> > to be able 
> > to point anyone having such questions in the future...
> 
> BTW, I don't remember discussing this at YP TSC... Maybe OE TSC
> covered this in the past?

Metadata architecture decision would be the OE TSC and "TSC discussion
long ago" means something from several years ago.

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136670): 
https://lists.openembedded.org/g/openembedded-core/message/136670
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 15:32 -0400, Denys Dmytriyenko wrote:
> On Tue, Mar 24, 2020 at 07:29:42PM +, Richard Purdie wrote:
> > On Tue, 2020-03-24 at 14:43 -0400, Denys Dmytriyenko wrote:
> > > On Tue, Mar 24, 2020 at 02:33:19PM -0400, Denys Dmytriyenko
> > > wrote:
> > > > Richard,
> > > > 
> > > > Yes, yes, I understanf all that from a technical perspective
> > > > and
> > > > agree with 
> > > > it, but I was thinking about a more clear definition of this
> > > > policy
> > > > to be able 
> > > > to point anyone having such questions in the future...
> > > 
> > > BTW, I don't remember discussing this at YP TSC... Maybe OE TSC
> > > covered this in the past?
> > 
> > Metadata architecture decision would be the OE TSC and "TSC
> > discussion
> > long ago" means something from several years ago.
> 
> Thought so - were there any meeting minutes from back then? Sorry for
> being so 
> persistent, just trying to find some paper trail... :)

Yes, there is a record. You can find it quite easily. I don't need to
do this for you...

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136672): 
https://lists.openembedded.org/g/openembedded-core/message/136672
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 2/2] ptest: further bump RAM in qemu

2020-03-24 Thread Alexander Kanavin
Particularly gdk-pixbuf tests were hitting the 2M limit with an image size
that is just under that.

Signed-off-by: Alexander Kanavin 
---
 meta/recipes-sato/images/core-image-sato-ptest-fast.bb | 2 +-
 meta/recipes-sato/images/core-image-sato-sdk-ptest.bb  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-sato/images/core-image-sato-ptest-fast.bb 
b/meta/recipes-sato/images/core-image-sato-ptest-fast.bb
index 3641217306..29afb91d77 100644
--- a/meta/recipes-sato/images/core-image-sato-ptest-fast.bb
+++ b/meta/recipes-sato/images/core-image-sato-ptest-fast.bb
@@ -13,7 +13,7 @@ IMAGE_OVERHEAD_FACTOR = "1.0"
 IMAGE_ROOTFS_EXTRA_SPACE = "1524288"
 
 # ptests need more memory than standard to avoid the OOM killer
-QB_MEM = "-m 1024"
+QB_MEM = "-m 2560"
 
 # Sadly at the moment the fast set of ptests is not robust enough and 
sporadically fails in a couple of places
 PTEST_EXPECT_FAILURE = "1"
diff --git a/meta/recipes-sato/images/core-image-sato-sdk-ptest.bb 
b/meta/recipes-sato/images/core-image-sato-sdk-ptest.bb
index bf749acd79..220c427387 100644
--- a/meta/recipes-sato/images/core-image-sato-sdk-ptest.bb
+++ b/meta/recipes-sato/images/core-image-sato-sdk-ptest.bb
@@ -17,7 +17,7 @@ IMAGE_ROOTFS_EXTRA_SPACE = "1124288"
 
 # ptests need more memory than standard to avoid the OOM killer
 # also lttng-tools needs /tmp that has at least 1G
-QB_MEM = "-m 2048"
+QB_MEM = "-m 2560"
 
 # Sadly at the moment the full set of ptests is not robust enough and 
sporadically fails in random places
 PTEST_EXPECT_FAILURE = "1"
-- 
2.25.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136675): 
https://lists.openembedded.org/g/openembedded-core/message/136675
Mute This Topic: https://lists.openembedded.org/mt/72528132/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 1/2] quilt: do not test patch-wrapper as it is not enabled

2020-03-24 Thread Alexander Kanavin
Upstream only enables the test if --with-patch-wrapper
was supplied to ./configure, and so should we.

Signed-off-by: Alexander Kanavin 
---
 meta/recipes-devtools/quilt/quilt.inc | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-devtools/quilt/quilt.inc 
b/meta/recipes-devtools/quilt/quilt.inc
index d6d06c049c..d2db079e0b 100644
--- a/meta/recipes-devtools/quilt/quilt.inc
+++ b/meta/recipes-devtools/quilt/quilt.inc
@@ -71,6 +71,9 @@ do_install_ptest() {
tar -c --exclude=mail.test --exclude=delete.test test/ | ( cd 
${D}${PTEST_PATH} && tar -xf - && chmod 777 test)
cp ${WORKDIR}/Makefile ${D}${PTEST_PATH}
cp ${WORKDIR}/test.sh ${D}${PTEST_PATH}
+
+   # This is specifically excluded when --with-patch-wrapper is absent 
during ./configure
+   rm ${D}${PTEST_PATH}/test/patch-wrapper.test
 }
 
 PACKAGES += "guards guards-doc"
-- 
2.25.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136674): 
https://lists.openembedded.org/g/openembedded-core/message/136674
Mute This Topic: https://lists.openembedded.org/mt/72528131/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.27

2020-03-24 Thread Bruce Ashfield
On Tue, Mar 24, 2020 at 9:50 AM Bruce Ashfield via
Lists.Openembedded.Org
 wrote:
>
> On Tue, Mar 24, 2020 at 9:42 AM Khem Raj  wrote:
> >
> > On Tue, Mar 24, 2020 at 5:40 AM Bruce Ashfield  
> > wrote:
> > >
> > > On Tue, Mar 24, 2020 at 2:35 AM Khem Raj  wrote:
> > > >
> > > > Hi Bruce
> > > >
> > > > This results in kernel-selftest failure regression
> > > >
> > > > errors.yoctoproject.org/Errors/Build/100385/
> > > >
> > > > and also same errors as reported here
> > > > https://bugs.launchpad.net/ubuntu-kernel-tests/+bug/1867596
> > > >
> > > > I have fixed them via three patches that are here
> > > >
> > > > https://git.openembedded.org/openembedded-core-contrib/commit/?h=yoe/mut=2c15bc35809365db8c586df7c383ed01b00e6618
> > > >
> > > > Please cherry-pick these 3 patches as well on this pull while you are 
> > > > here.
> > > >
> > >
> > > Only one of the patches has any upstream information.
> > >
> > > Are the reverts also being done in mainline ? Or is a different fix ?
> > >
> >
> > these are all results of half cherry picks into stable. all works fine in 
> > master
> > they should be reverted in stable but I have not sent them to lkml
>
> Aha. Thanks, that helps me understand what was going on.
>
> I'll amend the logs slightly and yank them into my tree.
>

These are now queued. I'll launch an AB test and send the queue once it passes.

Bruce

> Cheers,
>
> Bruce
>
> >
> > > What about the one that does have an upstream status, has it been
> > > submitted to -stable ?
> > >
> > > Bruce
> > >
> > > > On Sat, Mar 21, 2020 at 6:18 PM Bruce Ashfield 
> > > >  wrote:
> > > > >
> > > > > From: Bruce Ashfield 
> > > > >
> > > > > Updating linux-yocto/5.4 to the latest korg -stable release that 
> > > > > comprises
> > > > > the following commits:
> > > > >
> > > > > 585e0cc08069 Linux 5.4.27
> > > > > 20ba40d262b0 ipv4: ensure rcu_read_lock() in cipso_v4_error()
> > > > > 81e0dac3780a ARM: 8961/2: Fix Kbuild issue caused by per-task 
> > > > > stack protector GCC plugin
> > > > > 1dc67950e2d1 HID: add ALWAYS_POLL quirk to lenovo pixart mouse
> > > > > 5d961582ea88 HID: google: add moonball USB id
> > > > > ae119b7e1247 mm: slub: add missing TID bump in 
> > > > > kmem_cache_alloc_bulk()
> > > > > 47b6d269dbbd ARM: 8958/1: rename missed uaccess .fixup section
> > > > > 1a9e78628d44 ARM: 8957/1: VDSO: Match ARMv8 timer in 
> > > > > cntvct_functional()
> > > > > 881876bee657 net: qrtr: fix len of skb_put_padto in 
> > > > > qrtr_node_enqueue
> > > > > 235fb892d846 blk-mq: insert flush request to the front of 
> > > > > dispatch queue
> > > > > dbce8292217a jbd2: fix data races at struct journal_head
> > > > > 99746e92fa4c sfc: fix timestamp reconstruction at 16-bit rollover 
> > > > > points
> > > > > e2b8b042c98b net: rmnet: fix packet forwarding in rmnet bridge 
> > > > > mode
> > > > > 34c550ebb219 net: rmnet: fix bridge mode bugs
> > > > > 8eccc6d06ea1 net: rmnet: use upper/lower device infrastructure
> > > > > 835bbd892683 net: rmnet: do not allow to change mux id if mux id 
> > > > > is duplicated
> > > > > 5ce5e95d72f1 net: rmnet: remove rcu_read_lock in 
> > > > > rmnet_force_unassociate_device()
> > > > > 002b4bd9b51d net: rmnet: fix suspicious RCU usage
> > > > > 8a027eee62f7 net: rmnet: fix NULL pointer dereference in 
> > > > > rmnet_changelink()
> > > > > 006cd2426a05 net: rmnet: fix NULL pointer dereference in 
> > > > > rmnet_newlink()
> > > > > cfc7f5fe5c61 hinic: fix a bug of rss configuration
> > > > > e6f52d5ce172 hinic: fix a bug of setting hw_ioctxt
> > > > > 65340b196811 hinic: fix a irq affinity bug
> > > > > 1c7dde0a9bfc net: phy: mscc: fix firmware paths
> > > > > 4d6ee3123d65 slip: not call free_netdev before rtnl_unlock in 
> > > > > slip_open
> > > > > d1984c92f072 signal: avoid double atomic counter increments for 
> > > > > user accounting
> > > > > df8e98b00941 kbuild: add dt_binding_check to PHONY in a correct 
> > > > > place
> > > > > fd1f29f2a824 kbuild: add dtbs_check to PHONY
> > > > > bbf3acbb8a77 drm/amdgpu: fix memory leak during TDR test(v2)
> > > > > 74c77d6a4ea9 blk-mq: insert passthrough request into 
> > > > > hctx->dispatch directly
> > > > > 74886a6d9de8 net: ll_temac: Handle DMA halt condition caused by 
> > > > > buffer underrun
> > > > > c2cffaea4fde net: ll_temac: Fix RX buffer descriptor handling on 
> > > > > GFP_ATOMIC pressure
> > > > > 6e3b69543b5e net: ll_temac: Add more error handling of 
> > > > > dma_map_single() calls
> > > > > 0c050d655475 net: ll_temac: Fix race condition causing TX hang
> > > > > a342bb7b9902 mac80211: rx: avoid RCU list traversal under mutex
> > > > > 5d92d834aa27 net: ks8851-ml: Fix IRQ handling and locking
> > > > > d939f3389f9b net: usb: qmi_wwan: restore mtu min/max values after 
> > > > > raw_ip switch
> > > > > 6c663120ecdb scsi: libfc: free response frame from GPN_ID

Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Antoine Manache
Absolutely.
I would add an example of issue I've seen in a software project.
Someone developing a new application and using the image-based SDK that I
provided got the right functional behaviour on target.
Once this application was integrated into the core image of our Yocto
project (new recipe), the same tests on target failed (application crash).
The root cause was : the two binaries produced were different because the
effective GCC options at compile time were different between both methods
(security flags not enabled when compiling with the SDK).
Antoine

Le mar. 24 mars 2020 à 19:34, Tom Hochstein  a
écrit :

>
>
> > -Original Message-
> > From: Richard Purdie 
> > Sent: Tuesday, March 24, 2020 1:03 PM
> > To: Antoine Manache ; Tom Hochstein <
> tom.hochst...@nxp.com>
> > Cc: openembedded-core@lists.openembedded.org
> > Subject: Re: [OE-core][PATCH] security_flags.inc: fix flags missing from
> SDK toolchain
> >
> > On Tue, 2020-03-24 at 18:25 +0100, Antoine Manache wrote:
> > > I've already sent this patch few months ago but not taken into
> > > account yet.
> >
> > Can someone explain more about the issue here please?
> >
> > Shouldn't these flags be added to the SDK toolchain environment files
> > rather than coded into the compiler? Why aren't they being coded in? or
> > is gcc being misconfigured?
>
> The recipe meta-environment inherits cross-canadian and toolchain-scripts.
> The latter generates the SDK toolchain environment file using
> TARGET_CC_ARCH and TARGET_LDFLAGS like this:
>
> echo 'export CC="${TARGET_PREFIX}gcc ${TARGET_CC_ARCH}
> --sysroot=$SDKTARGETSYSROOT"' >> $script
> echo 'export LDFLAGS="${TARGET_LDFLAGS}"' >> $script
>
>
> https://github.com/openembedded/openembedded-core/blob/master/meta/classes/toolchain-scripts.bbclass#L82
>
> With class-cross-canadian as the applicable override, updating the
> security flags for class-target does nothing for the SDK toolchain, hence
> the proposed fix:
>
> TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
> +TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
>  TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
> +TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"
>
> Tom
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136668): 
https://lists.openembedded.org/g/openembedded-core/message/136668
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 18:25 +0100, Antoine Manache wrote:
> I've already sent this patch few months ago but not taken into
> account yet.

Can someone explain more about the issue here please?

Shouldn't these flags be added to the SDK toolchain environment files
rather than coded into the compiler? Why aren't they being coded in? or
is gcc being misconfigured?

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136660): 
https://lists.openembedded.org/g/openembedded-core/message/136660
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Denys Dmytriyenko
On Tue, Mar 24, 2020 at 02:33:19PM -0400, Denys Dmytriyenko wrote:
> On Tue, Mar 24, 2020 at 06:15:51PM +, Richard Purdie wrote:
> > On Tue, 2020-03-24 at 14:12 -0400, Denys Dmytriyenko wrote:
> > > On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> > > > On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > > > > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> > > > > > Instead of disabling unconditionally, add PACKAGECONFIG for
> > > > > > qt5. Imported from meta-qt5 layer.
> > > > > > 
> > > > > > Signed-off-by: Anuj Mittal 
> > > > > > ---
> > > > > >  .../gstreamer/gstreamer1.0-plugins-
> > > > > > good_1.16.2.bb  | 7 +--
> > > > > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > > > > 
> > > > > > diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > > plugins-good_1.16.2.bb b/meta/recipes-
> > > > > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > > > > index f3e0cee..8b34111 100644
> > > > > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > > > > good_1.16.2.bb
> > > > > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > > > > good_1.16.2.bb
> > > > > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-Dpng=enabled,-
> > > > > > Dpng=disabled,libpng"
> > > > > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-Dv4l2-
> > > > > > libv4l2=disabled,v4l-utils"
> > > > > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > > > > Dmpg123=disabled,mpg123"
> > > > > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > > > > Dpulse=disabled,pulseaudio"
> > > > > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-
> > > > > > Dqt5=disabled,qtbase
> > > > > > qtdeclarative qtbase-native"
> > > > > 
> > > > > Does it make oe-core depend on meta-qt5 layer?
> > > > > 
> > > > > What is the policy here for having inter-layer dependencies in
> > > > > DEPENDS vs. PACKAGECONFIG without explicitly listing them in
> > > > > LAYERDEPENDS?
> > > > 
> > > > We have a clear policy that listing such options is ok as long as
> > > > its
> > > > not enabled by default.
> > > 
> > > Richard,
> > > 
> > > This is good, thank you for confirming. Do you have a link where this
> > > is clearly explained? Thanks again.
> > 
> > Its come up many times over the years and we've been consistent about
> > it. I don't have a link but there has been mailing list discussion and
> > perhaps TSC discussion long ago, I simply don't remember. There are
> > many examples of that in OE-Core.
> > 
> > The alternative is forcing more into bbappends which doesn't seem to
> > gain anything (PACKAGECONFIG itself could be changed from distro config
> > for example).
> > 
> > Its also easy to catch bad defaults since the recipes would no longer
> > parse due to missing dependencies which we'd quickly spot.
> 
> Richard,
> 
> Yes, yes, I understanf all that from a technical perspective and agree with 
> it, but I was thinking about a more clear definition of this policy to be 
> able 
> to point anyone having such questions in the future...

BTW, I don't remember discussing this at YP TSC... Maybe OE TSC covered this 
in the past?

-- 
Denys
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#13): 
https://lists.openembedded.org/g/openembedded-core/message/13
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Tom Hochstein


> -Original Message-
> From: Richard Purdie 
> Sent: Tuesday, March 24, 2020 1:03 PM
> To: Antoine Manache ; Tom Hochstein 
> 
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK 
> toolchain
> 
> On Tue, 2020-03-24 at 18:25 +0100, Antoine Manache wrote:
> > I've already sent this patch few months ago but not taken into
> > account yet.
> 
> Can someone explain more about the issue here please?
> 
> Shouldn't these flags be added to the SDK toolchain environment files
> rather than coded into the compiler? Why aren't they being coded in? or
> is gcc being misconfigured?

The recipe meta-environment inherits cross-canadian and toolchain-scripts. The 
latter generates the SDK toolchain environment file using TARGET_CC_ARCH and 
TARGET_LDFLAGS like this:

echo 'export CC="${TARGET_PREFIX}gcc ${TARGET_CC_ARCH} 
--sysroot=$SDKTARGETSYSROOT"' >> $script
echo 'export LDFLAGS="${TARGET_LDFLAGS}"' >> $script

https://github.com/openembedded/openembedded-core/blob/master/meta/classes/toolchain-scripts.bbclass#L82

With class-cross-canadian as the applicable override, updating the security 
flags for class-target does nothing for the SDK toolchain, hence the proposed 
fix:

TARGET_CC_ARCH_append_class-target = " ${SECURITY_CFLAGS}"
+TARGET_CC_ARCH_append_class-cross-canadian = " ${SECURITY_CFLAGS}"
 TARGET_LDFLAGS_append_class-target = " ${SECURITY_LDFLAGS}"
+TARGET_LDFLAGS_append_class-cross-canadian = " ${SECURITY_LDFLAGS}"

Tom
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136665): 
https://lists.openembedded.org/g/openembedded-core/message/136665
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Denys Dmytriyenko
On Tue, Mar 24, 2020 at 06:15:51PM +, Richard Purdie wrote:
> On Tue, 2020-03-24 at 14:12 -0400, Denys Dmytriyenko wrote:
> > On Tue, Mar 24, 2020 at 06:07:44PM +, Richard Purdie wrote:
> > > On Tue, 2020-03-24 at 13:58 -0400, Denys Dmytriyenko wrote:
> > > > On Fri, Feb 21, 2020 at 10:41:58AM +0800, Anuj Mittal wrote:
> > > > > Instead of disabling unconditionally, add PACKAGECONFIG for
> > > > > qt5. Imported from meta-qt5 layer.
> > > > > 
> > > > > Signed-off-by: Anuj Mittal 
> > > > > ---
> > > > >  .../gstreamer/gstreamer1.0-plugins-
> > > > > good_1.16.2.bb  | 7 +--
> > > > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > > > 
> > > > > diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
> > > > > plugins-good_1.16.2.bb b/meta/recipes-
> > > > > multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.2.bb
> > > > > index f3e0cee..8b34111 100644
> > > > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > > > good_1.16.2.bb
> > > > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
> > > > > good_1.16.2.bb
> > > > > @@ -42,6 +42,7 @@ PACKAGECONFIG[libpng] = "-Dpng=enabled,-
> > > > > Dpng=disabled,libpng"
> > > > >  PACKAGECONFIG[libv4l2]= "-Dv4l2-libv4l2=enabled,-Dv4l2-
> > > > > libv4l2=disabled,v4l-utils"
> > > > >  PACKAGECONFIG[mpg123] = "-Dmpg123=enabled,-
> > > > > Dmpg123=disabled,mpg123"
> > > > >  PACKAGECONFIG[pulseaudio] = "-Dpulse=enabled,-
> > > > > Dpulse=disabled,pulseaudio"
> > > > > +PACKAGECONFIG[qt5]= "-Dqt5=enabled,-
> > > > > Dqt5=disabled,qtbase
> > > > > qtdeclarative qtbase-native"
> > > > 
> > > > Does it make oe-core depend on meta-qt5 layer?
> > > > 
> > > > What is the policy here for having inter-layer dependencies in
> > > > DEPENDS vs. PACKAGECONFIG without explicitly listing them in
> > > > LAYERDEPENDS?
> > > 
> > > We have a clear policy that listing such options is ok as long as
> > > its
> > > not enabled by default.
> > 
> > Richard,
> > 
> > This is good, thank you for confirming. Do you have a link where this
> > is clearly explained? Thanks again.
> 
> Its come up many times over the years and we've been consistent about
> it. I don't have a link but there has been mailing list discussion and
> perhaps TSC discussion long ago, I simply don't remember. There are
> many examples of that in OE-Core.
> 
> The alternative is forcing more into bbappends which doesn't seem to
> gain anything (PACKAGECONFIG itself could be changed from distro config
> for example).
> 
> Its also easy to catch bad defaults since the recipes would no longer
> parse due to missing dependencies which we'd quickly spot.

Richard,

Yes, yes, I understanf all that from a technical perspective and agree with 
it, but I was thinking about a more clear definition of this policy to be able 
to point anyone having such questions in the future...

-- 
Denys
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136664): 
https://lists.openembedded.org/g/openembedded-core/message/136664
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH] bitbake: gitsm: download submodules

2020-03-24 Thread Adrian Freihofer
On Mon, 2020-03-23 at 17:30 +, Paul Barker wrote:
> On Thu, 5 Mar 2020 22:56:15 +
> "Freihofer, Adrian"  wrote:
> 
> > On Thu, 2020-03-05 at 09:42 +, Paul Barker wrote:
> > > On Wed, 4 Mar 2020 12:30:10 +
> > > "Freihofer, Adrian"  wrote:
> > >   
> > > > -Original Message-
> > > > From: Paul Barker 
> > > > To: "Freihofer, Adrian" 
> > > > Cc: openembedded-core@lists.openembedded.org <
> > > > openembedded-core@lists.openembedded.org>
> > > > Subject: Re: [OE-core] [PATCH] bitbake: gitsm: download submodules
> > > > Date: Wed, 04 Mar 2020 09:59:44 +
> > > > 
> > > > On Wed, 4 Mar 2020 08:12:27 +
> > > > "Freihofer, Adrian"  wrote:
> > > >   
> > > > > The unpack function failed because the submodules were not
> > > > > downloaded.
> > > > > Calling download before unpack for each submodule solves this
> > > > > issue.
> > > > > 
> > > > > Signed-off-by: Adrian Freihofer 
> > > > > ---
> > > > >  bitbake/lib/bb/fetch2/gitsm.py | 1 +
> > > > >  1 file changed, 1 insertion(+)
> > > > > 
> > > > > diff --git a/bitbake/lib/bb/fetch2/gitsm.py
> > > > > b/bitbake/lib/bb/fetch2/gitsm.py
> > > > > index c622771d21..3715e9824f 100644
> > > > > --- a/bitbake/lib/bb/fetch2/gitsm.py
> > > > > +++ b/bitbake/lib/bb/fetch2/gitsm.py
> > > > > @@ -184,6 +184,7 @@ class GitSM(Git):
> > > > >  
> > > > >  try:
> > > > >  newfetch = Fetch([url], d, cache=False)
> > > > > +newfetch.download()
> > > > >  newfetch.unpack(root=os.path.dirname(os.path.joi
> > > > > n(re
> > > > > po
> > > > > _conf, 'modules', module)))
> > > > >  except Exception as e:
> > > > >  logger.error('gitsm: submodule unpack failed: %s
> > > > > %s'
> > > > > %
> > > > > (type(e).__name__, str(e)))
> > > > 
> > > > You shouldn't be trying to download submodules in the do_unpack
> > > > step.
> > > > If
> > > > they're missing at this stage it probably indicates a fetch issue.
> > > > 
> > > > Basically true, but the information about which submodules need to
> > > > be
> > > > downloaded is not available before the top level repo has been
> > > > unpacked. I guess the solution must be something like:
> > > > 
> > > > fetch top-level
> > > > unpack top-level
> > > > foreach submodule:
> > > >   fetch submodule
> > > >   unpack submodule
> > > > 
> > > > What's the exact error that you're seeing?
> > > > 
> > > > Don't remeber exactly. But when I debugged the flow, it became
> > > > obvious,
> > > > that the submodules are not downloaded.
> > > > 
> > > > This could be related to the issue I saw when the fetcher uses git
> > > > shallow
> > > > tarballs from a mirror - if that's the case I'm planning to get
> > > > that
> > > > fixed
> > > > this weekend.
> > > > 
> > > > Yes, the problem occurs with git shallows. The patch solves it with
> > > > and
> > > > without shallows for us.
> > > > 
> > > > Also, your email client seems to have chewed the patch up. It's
> > > > best to
> > > > send
> > > > patches using `git send-email` only.
> > > > 
> > > > Sorry, we switched to a new e-mail solution. I have to adapt my
> > > > setup.  
> > > 
> > > The problem with your patch is that it would require network access
> > > during
> > > do_unpack which then breaks our ability to do offline builds.  
> > Fetching in the unpack task is probably really wrong, I agree.
> > However, it also works on our CI infrastructure which runs without
> > Internet connectivity.
> > > Do you only see this issue when fetching from git shallow mirror
> > > tarballs?  
> > 
> > On our CI which works with gitshallow archives, fetching did not work
> > at all. On machines with Internet access we also had problems, but I did 
> > not completely analyze what happened. The ovmf package from poky was 
> > causing the troubles in our case.
> > 
> > > If so, the mirror tarballs need to be extracted during do_fetch but
> > > not placed in the usual 'git2' directory (as future fetches can't
> > > cope with shallow clones in that directory).  
> > 
> > Do you already have an idea how this could be implemented?
> 
> Sorry for the late response, I meant to let you know when my fix for this had
> gone in to bitbake but I ended up away due to illness for a couple of weeks.
> Commit 3987db95 "fetch2/gitsm: Unpack shallow mirror tarballs" should resolve
> your issue, please let me know if it doesn't.

Thank you Paul. We picked your commit during the last days. It looks
much better than mine and it works perfect.

Regards,
Adrian
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136673): 
https://lists.openembedded.org/g/openembedded-core/message/136673
Mute This Topic: https://lists.openembedded.org/mt/72395274/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Tom Hochstein


> -Original Message-
> From: Richard Purdie 
> Sent: Tuesday, March 24, 2020 5:58 PM
> To: Antoine Manache ; Tom Hochstein 
> 
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK 
> toolchain
> 
> I've therefore taken Tom's version with some tweaks to the commit
> message.

Thanks. Can we get this on zeus? Do I need to cherry-pick and submit the patch?

Tom

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136685): 
https://lists.openembedded.org/g/openembedded-core/message/136685
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 18:12 -0400, Denys Dmytriyenko wrote:
> On Tue, Mar 24, 2020 at 07:57:27PM +, Richard Purdie wrote:
> > On Tue, 2020-03-24 at 15:32 -0400, Denys Dmytriyenko wrote:
> > > On Tue, Mar 24, 2020 at 07:29:42PM +, Richard Purdie wrote:
> > > > On Tue, 2020-03-24 at 14:43 -0400, Denys Dmytriyenko wrote:
> > > > > On Tue, Mar 24, 2020 at 02:33:19PM -0400, Denys Dmytriyenko
> > > > > wrote:
> > > > > > Richard,
> > > > > > 
> > > > > > Yes, yes, I understanf all that from a technical
> > > > > > perspective
> > > > > > and
> > > > > > agree with 
> > > > > > it, but I was thinking about a more clear definition of
> > > > > > this
> > > > > > policy
> > > > > > to be able 
> > > > > > to point anyone having such questions in the future...
> > > > > 
> > > > > BTW, I don't remember discussing this at YP TSC... Maybe OE
> > > > > TSC
> > > > > covered this in the past?
> > > > 
> > > > Metadata architecture decision would be the OE TSC and "TSC
> > > > discussion
> > > > long ago" means something from several years ago.
> > > 
> > > Thought so - were there any meeting minutes from back then? Sorry
> > > for
> > > being so 
> > > persistent, just trying to find some paper trail... :)
> > 
> > Yes, there is a record. You can find it quite easily. I don't need
> > to
> > do this for you...
> 
> All I could find is OE TSC minutes from 7 May 2013 referring to some
> related 
> list discussion:
> 
> https://lists.openembedded.org/g/tsc/message/369?p=,,,20,0,0,0::Created,,PACKAGECONFIG,20,2,0,72188707
> 
> (9:47:21 AM) RP: bluelightning: I think this was to PACKAGECONFIG
> more recipes
> (9:47:28 AM) bluelightning: ah ok
> (9:47:36 AM) RP: bluelightning: remove the need to bbappend, have
> people set mode config options
> (9:47:51 AM) bluelightning: and PACKAGECONFIG to enable deps on
> things
> outside OE-Core as well presumably
> (9:48:01 AM) bluelightning: (as discussed on the ml a few weeks ago)
> (9:48:06 AM) fray: yup

Well found. There is clearly more context behind that even back then.

> I spent last couple hours digging through my archives of
> openembedded-core and 
> openembedded-devel lists around that time (Spring 2013), but besides
> general move to enable PACKAGECONFIG in recipes, I didn't see
> anything specific about crossing layer boundaries by PACKAGECONFIG
> dependencies, unfortunately. And it was a wild time - all the energy,
> extra activity, conflicts, flamewars - that definitely brought up
> some memories re-reading those discussions again... :)

:)

I'm sure this has been discussed, I just don't remember where/when
unfortunately.

I do believe the conclusion is right though and I believe there is
enough general acceptance we can document it as such. If anyone
objects, the TSC can discuss it but I'm not seeing that being requested
unless there is more context to this discussion than I'm seeing.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136677): 
https://lists.openembedded.org/g/openembedded-core/message/136677
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 0/5] linux-yocto: consolidated pull request

2020-03-24 Thread Bruce Ashfield
From: Bruce Ashfield 

Hi all,

Here's the queued bug fixes for linux-yocto. It has khem's identified fixes
for the kernel selftest issues with the latest -stable udpates, as well as
some BSP fixes and config changes.

I ran this through the AB and everything came back green.

I've done this in multiple commits, so we'll have granularity to look
into individual fixes if we have trouble.

Cheers,

Bruce

The following changes since commit d5db9d12642e03a1d5cca5f1520dad5fcc9524c3:

  archiver.py: fix typo of "ARCHIVER_MIRROR_EXCLUDE" in comment (2020-03-24 
16:39:40 +)

are available in the Git repository at:

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

Bruce Ashfield (5):
  linux-yocto/5.4: Revert "platform/x86: wmi: Destroy on cleanup rather
than unregister"
  linux-yocto/5.4: fix kernel selftests
  linux-yocto/5.4: KVM: LAPIC: Mark hrtimer for period or oneshot mode
to expire in hard interrupt context
  linux-yocto/5.4: ARM: 8918/2: only build return_address() if needed
  linux-yocto/5.4: configuration tweaks

 .../linux/linux-yocto-rt_5.4.bb   |  4 ++--
 .../linux/linux-yocto-tiny_5.4.bb |  6 +++---
 meta/recipes-kernel/linux/linux-yocto_5.4.bb  | 20 +--
 3 files changed, 15 insertions(+), 15 deletions(-)

-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136679): 
https://lists.openembedded.org/g/openembedded-core/message/136679
Mute This Topic: https://lists.openembedded.org/mt/72529850/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 3/5] linux-yocto/5.4: KVM: LAPIC: Mark hrtimer for period or oneshot mode to expire in hard interrupt context

2020-03-24 Thread Bruce Ashfield
From: Bruce Ashfield 

Integrating the following commits to linux-yocto/5.4:

9065d188f874 KVM: LAPIC: Mark hrtimer for period or oneshot mode to expire 
in hard interrupt context

Signed-off-by: Bruce Ashfield 
---
 .../recipes-kernel/linux/linux-yocto-rt_5.4.bb |  2 +-
 .../linux/linux-yocto-tiny_5.4.bb  |  4 ++--
 meta/recipes-kernel/linux/linux-yocto_5.4.bb   | 18 +-
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
index 9d237ee32f..2eb1e846e8 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
@@ -11,7 +11,7 @@ python () {
 raise bb.parse.SkipRecipe("Set PREFERRED_PROVIDER_virtual/kernel to 
linux-yocto-rt to enable it")
 }
 
-SRCREV_machine ?= "a069c47171c8b2510401dba1922b49b294c19b0e"
+SRCREV_machine ?= "87d86ed4bfc78baee66933982c2962309fca4ffe"
 SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
index 02beca0097..18ff195af3 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
@@ -15,8 +15,8 @@ DEPENDS += "openssl-native util-linux-native"
 KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
-SRCREV_machine_qemuarm ?= "84a0aabc9db1eba2fbfcbd80b3c538cd73f0b03c"
-SRCREV_machine ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemuarm ?= "b6bb0cf2d13aca397c216c2323985c87ac98919a"
+SRCREV_machine ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
 SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
index 8bad042528..8db3d1c974 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
@@ -12,15 +12,15 @@ KBRANCH_qemux86  ?= "v5.4/standard/base"
 KBRANCH_qemux86-64 ?= "v5.4/standard/base"
 KBRANCH_qemumips64 ?= "v5.4/standard/mti-malta64"
 
-SRCREV_machine_qemuarm ?= "f4bf0307957258b20048e704ff3683437d7b0373"
-SRCREV_machine_qemuarm64 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
-SRCREV_machine_qemumips ?= "06bdb6af393b84e2c1a6f34b67bf4af36b9331e3"
-SRCREV_machine_qemuppc ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
-SRCREV_machine_qemuriscv64 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
-SRCREV_machine_qemux86 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
-SRCREV_machine_qemux86-64 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
-SRCREV_machine_qemumips64 ?= "958ee61d50f664b413f368a7573918a18a00d4ac"
-SRCREV_machine ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemuarm ?= "9f04047a4f31450342960f04571194122e890461"
+SRCREV_machine_qemuarm64 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemumips ?= "f6f24e8e404d1debf07cb5793a183ba8e759d09b"
+SRCREV_machine_qemuppc ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemuriscv64 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemux86 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemux86-64 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemumips64 ?= "6f93582af92ab433d43c2a67a7ed871e5c229e55"
+SRCREV_machine ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
 SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 # remap qemuarm to qemuarma15 for the 5.4 kernel
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136684): 
https://lists.openembedded.org/g/openembedded-core/message/136684
Mute This Topic: https://lists.openembedded.org/mt/72529860/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 2/5] linux-yocto/5.4: fix kernel selftests

2020-03-24 Thread Bruce Ashfield
From: Bruce Ashfield 

Khem reported that kernel selftests are failing after some of the recent
-stable udpates, and identified the followig commits to solve the
issues:

4cd12df48b83 selftest/bpf: Use CHECK macro instead of RET_IF
84591c1cb409 Revert "selftests/bpf: Fix perf_buffer test on systems w/ 
offline CPUs"
844b959f96bd Revert "selftests/bpf: Skip perf hw events test if the setup 
disabled it"

Signed-off-by: Bruce Ashfield 
---
 .../linux/linux-yocto-rt_5.4.bb   |  4 ++--
 .../linux/linux-yocto-tiny_5.4.bb |  6 +++---
 meta/recipes-kernel/linux/linux-yocto_5.4.bb  | 20 +--
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
index 5bdd2f629a..9d237ee32f 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
@@ -11,8 +11,8 @@ python () {
 raise bb.parse.SkipRecipe("Set PREFERRED_PROVIDER_virtual/kernel to 
linux-yocto-rt to enable it")
 }
 
-SRCREV_machine ?= "b6eb513c50e6de3746287c4f17058a81d1be613c"
-SRCREV_meta ?= "ccf845f650255696bcd41c4004abf8adba212d5d"
+SRCREV_machine ?= "a069c47171c8b2510401dba1922b49b294c19b0e"
+SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-5.4;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
index 0850725383..02beca0097 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
@@ -15,9 +15,9 @@ DEPENDS += "openssl-native util-linux-native"
 KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
-SRCREV_machine_qemuarm ?= "8f0a54b42e11bee3bf552ae6a04f7ec2267f4dd3"
-SRCREV_machine ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_meta ?= "ccf845f650255696bcd41c4004abf8adba212d5d"
+SRCREV_machine_qemuarm ?= "84a0aabc9db1eba2fbfcbd80b3c538cd73f0b03c"
+SRCREV_machine ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
index 98e4accc74..8bad042528 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
@@ -12,16 +12,16 @@ KBRANCH_qemux86  ?= "v5.4/standard/base"
 KBRANCH_qemux86-64 ?= "v5.4/standard/base"
 KBRANCH_qemumips64 ?= "v5.4/standard/mti-malta64"
 
-SRCREV_machine_qemuarm ?= "e0b9db4dd28cfccc6b0cffd2da6cde7c8d760bdd"
-SRCREV_machine_qemuarm64 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_machine_qemumips ?= "84c43062449817e61dd8e331f49821c65aa23967"
-SRCREV_machine_qemuppc ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_machine_qemuriscv64 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_machine_qemux86 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_machine_qemux86-64 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_machine_qemumips64 ?= "08ac7d212e539ed2d17c6714d97b019e48e1fe1c"
-SRCREV_machine ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
-SRCREV_meta ?= "ccf845f650255696bcd41c4004abf8adba212d5d"
+SRCREV_machine_qemuarm ?= "f4bf0307957258b20048e704ff3683437d7b0373"
+SRCREV_machine_qemuarm64 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemumips ?= "06bdb6af393b84e2c1a6f34b67bf4af36b9331e3"
+SRCREV_machine_qemuppc ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemuriscv64 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemux86 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemux86-64 ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_machine_qemumips64 ?= "958ee61d50f664b413f368a7573918a18a00d4ac"
+SRCREV_machine ?= "4cd12df48b83cef9cc7d6b80b128afbf68746718"
+SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 # remap qemuarm to qemuarma15 for the 5.4 kernel
 # KMACHINE_qemuarm ?= "qemuarma15"
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136681): 
https://lists.openembedded.org/g/openembedded-core/message/136681
Mute This Topic: https://lists.openembedded.org/mt/72529854/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 5/5] linux-yocto/5.4: configuration tweaks

2020-03-24 Thread Bruce Ashfield
From: Bruce Ashfield 

Integrating the following configuration fixes:

   bee554e595e bsp/ti-am65x: remove duplicate config option CONFIG_NETDEVICES
   32405486b46 bsp/ti-am335x: drop PRUSS invalid cfg option
   d40bf6ad14a xilinx-zynqmp: add cpuidle support for zcu102
   533cfd5af00 bsp: delete obsolete kernel option CONFIG_MTD_M25P80

Signed-off-by: Bruce Ashfield 
---
 meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb   | 2 +-
 meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb | 2 +-
 meta/recipes-kernel/linux/linux-yocto_5.4.bb  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
index 7037d980d2..9e74ec17af 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
@@ -12,7 +12,7 @@ python () {
 }
 
 SRCREV_machine ?= "79037ae58e6b0dfd0c63d4a0e131f1bd5efe7e53"
-SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
+SRCREV_meta ?= "bee554e595e49c963900d1c78c01ab2d041382e7"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-5.4;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
index 2a621622eb..ba5e668d73 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
@@ -17,7 +17,7 @@ KCONF_BSP_AUDIT_LEVEL = "2"
 
 SRCREV_machine_qemuarm ?= "5c2d35eeb4be0e8bae4cf5ee0733e41ff1262ff3"
 SRCREV_machine ?= "03142acba06e8b33888410a518358a626dceb720"
-SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
+SRCREV_meta ?= "bee554e595e49c963900d1c78c01ab2d041382e7"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
index 0f1cd07945..25e933d683 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
@@ -21,7 +21,7 @@ SRCREV_machine_qemux86 ?= 
"03142acba06e8b33888410a518358a626dceb720"
 SRCREV_machine_qemux86-64 ?= "03142acba06e8b33888410a518358a626dceb720"
 SRCREV_machine_qemumips64 ?= "d3e850da830241c424d557a6a072527e09e784ab"
 SRCREV_machine ?= "03142acba06e8b33888410a518358a626dceb720"
-SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
+SRCREV_meta ?= "bee554e595e49c963900d1c78c01ab2d041382e7"
 
 # remap qemuarm to qemuarma15 for the 5.4 kernel
 # KMACHINE_qemuarm ?= "qemuarma15"
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136683): 
https://lists.openembedded.org/g/openembedded-core/message/136683
Mute This Topic: https://lists.openembedded.org/mt/72529856/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 1/5] linux-yocto/5.4: Revert "platform/x86: wmi: Destroy on cleanup rather than unregister"

2020-03-24 Thread Bruce Ashfield
From: Bruce Ashfield 

We had reverted this commit in v5.2, and it is still required in 5.4.

The following config commit is also included in this change:

  bsp/ti-am335x: drop some invalid cfg options

Signed-off-by: Bruce Ashfield 
---
 .../linux/linux-yocto-rt_5.4.bb   |  4 ++--
 .../linux/linux-yocto-tiny_5.4.bb |  6 +++---
 meta/recipes-kernel/linux/linux-yocto_5.4.bb  | 20 +--
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
index 446eae2be6..5bdd2f629a 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
@@ -11,8 +11,8 @@ python () {
 raise bb.parse.SkipRecipe("Set PREFERRED_PROVIDER_virtual/kernel to 
linux-yocto-rt to enable it")
 }
 
-SRCREV_machine ?= "2659f7a7212cd80e2b69b2e5e8ceec8bcffec6aa"
-SRCREV_meta ?= "9fe365b9395a05831177ec309e9857962b2496a9"
+SRCREV_machine ?= "b6eb513c50e6de3746287c4f17058a81d1be613c"
+SRCREV_meta ?= "ccf845f650255696bcd41c4004abf8adba212d5d"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \

git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-5.4;destsuffix=${KMETA}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
index 653f33f9cb..0850725383 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
@@ -15,9 +15,9 @@ DEPENDS += "openssl-native util-linux-native"
 KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
-SRCREV_machine_qemuarm ?= "c90bb8009bf65f13d120f8e4989763f85133762e"
-SRCREV_machine ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_meta ?= "9fe365b9395a05831177ec309e9857962b2496a9"
+SRCREV_machine_qemuarm ?= "8f0a54b42e11bee3bf552ae6a04f7ec2267f4dd3"
+SRCREV_machine ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_meta ?= "ccf845f650255696bcd41c4004abf8adba212d5d"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
 
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
index d3fed81c56..98e4accc74 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
@@ -12,16 +12,16 @@ KBRANCH_qemux86  ?= "v5.4/standard/base"
 KBRANCH_qemux86-64 ?= "v5.4/standard/base"
 KBRANCH_qemumips64 ?= "v5.4/standard/mti-malta64"
 
-SRCREV_machine_qemuarm ?= "e28b25e558cdc53d13f67df8b249bb770f3dcfba"
-SRCREV_machine_qemuarm64 ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_machine_qemumips ?= "0e1fe5b6bb2a34ed05dcc69fadec6581cbb0f9f5"
-SRCREV_machine_qemuppc ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_machine_qemuriscv64 ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_machine_qemux86 ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_machine_qemux86-64 ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_machine_qemumips64 ?= "ac5e243548907567d8285492c9fa0cbcf7d35d8a"
-SRCREV_machine ?= "20eb96b378204dcc435b193c5d692bdf561beff0"
-SRCREV_meta ?= "9fe365b9395a05831177ec309e9857962b2496a9"
+SRCREV_machine_qemuarm ?= "e0b9db4dd28cfccc6b0cffd2da6cde7c8d760bdd"
+SRCREV_machine_qemuarm64 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_machine_qemumips ?= "84c43062449817e61dd8e331f49821c65aa23967"
+SRCREV_machine_qemuppc ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_machine_qemuriscv64 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_machine_qemux86 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_machine_qemux86-64 ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_machine_qemumips64 ?= "08ac7d212e539ed2d17c6714d97b019e48e1fe1c"
+SRCREV_machine ?= "45760a7e1301894b3ffd4d1bd76684c99fbff6f7"
+SRCREV_meta ?= "ccf845f650255696bcd41c4004abf8adba212d5d"
 
 # remap qemuarm to qemuarma15 for the 5.4 kernel
 # KMACHINE_qemuarm ?= "qemuarma15"
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136680): 
https://lists.openembedded.org/g/openembedded-core/message/136680
Mute This Topic: https://lists.openembedded.org/mt/72529852/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH 4/5] linux-yocto/5.4: ARM: 8918/2: only build return_address() if needed

2020-03-24 Thread Bruce Ashfield
From: Bruce Ashfield 

Integrating the following commit(s) to linux-yocto/5.4:

03142acba06e ARM: 8918/2: only build return_address() if needed

Signed-off-by: Bruce Ashfield 
---
 .../recipes-kernel/linux/linux-yocto-rt_5.4.bb |  2 +-
 .../linux/linux-yocto-tiny_5.4.bb  |  4 ++--
 meta/recipes-kernel/linux/linux-yocto_5.4.bb   | 18 +-
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
index 2eb1e846e8..7037d980d2 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.4.bb
@@ -11,7 +11,7 @@ python () {
 raise bb.parse.SkipRecipe("Set PREFERRED_PROVIDER_virtual/kernel to 
linux-yocto-rt to enable it")
 }
 
-SRCREV_machine ?= "87d86ed4bfc78baee66933982c2962309fca4ffe"
+SRCREV_machine ?= "79037ae58e6b0dfd0c63d4a0e131f1bd5efe7e53"
 SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \
diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
index 18ff195af3..2a621622eb 100644
--- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.4.bb
@@ -15,8 +15,8 @@ DEPENDS += "openssl-native util-linux-native"
 KMETA = "kernel-meta"
 KCONF_BSP_AUDIT_LEVEL = "2"
 
-SRCREV_machine_qemuarm ?= "b6bb0cf2d13aca397c216c2323985c87ac98919a"
-SRCREV_machine ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemuarm ?= "5c2d35eeb4be0e8bae4cf5ee0733e41ff1262ff3"
+SRCREV_machine ?= "03142acba06e8b33888410a518358a626dceb720"
 SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 PV = "${LINUX_VERSION}+git${SRCPV}"
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.4.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
index 8db3d1c974..0f1cd07945 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.4.bb
@@ -12,15 +12,15 @@ KBRANCH_qemux86  ?= "v5.4/standard/base"
 KBRANCH_qemux86-64 ?= "v5.4/standard/base"
 KBRANCH_qemumips64 ?= "v5.4/standard/mti-malta64"
 
-SRCREV_machine_qemuarm ?= "9f04047a4f31450342960f04571194122e890461"
-SRCREV_machine_qemuarm64 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
-SRCREV_machine_qemumips ?= "f6f24e8e404d1debf07cb5793a183ba8e759d09b"
-SRCREV_machine_qemuppc ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
-SRCREV_machine_qemuriscv64 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
-SRCREV_machine_qemux86 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
-SRCREV_machine_qemux86-64 ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
-SRCREV_machine_qemumips64 ?= "6f93582af92ab433d43c2a67a7ed871e5c229e55"
-SRCREV_machine ?= "9065d188f87410b7c862e44ff3143648c1ad2295"
+SRCREV_machine_qemuarm ?= "ab849248b45403c7d6f1fb8e8f57840cc2880618"
+SRCREV_machine_qemuarm64 ?= "03142acba06e8b33888410a518358a626dceb720"
+SRCREV_machine_qemumips ?= "967a1708cbbfe1b524dc28f04fa5e0d79a270bf5"
+SRCREV_machine_qemuppc ?= "03142acba06e8b33888410a518358a626dceb720"
+SRCREV_machine_qemuriscv64 ?= "03142acba06e8b33888410a518358a626dceb720"
+SRCREV_machine_qemux86 ?= "03142acba06e8b33888410a518358a626dceb720"
+SRCREV_machine_qemux86-64 ?= "03142acba06e8b33888410a518358a626dceb720"
+SRCREV_machine_qemumips64 ?= "d3e850da830241c424d557a6a072527e09e784ab"
+SRCREV_machine ?= "03142acba06e8b33888410a518358a626dceb720"
 SRCREV_meta ?= "d40bf6ad14acc6d84d5c9a8e2c794ce37ffa16ff"
 
 # remap qemuarm to qemuarma15 for the 5.4 kernel
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136682): 
https://lists.openembedded.org/g/openembedded-core/message/136682
Mute This Topic: https://lists.openembedded.org/mt/72529855/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH] mesa: upgrade 20.0.1 -> 20.0.2

2020-03-24 Thread Wang Mingyu
Signed-off-by: Wang Mingyu 
---
 .../mesa/{mesa-gl_20.0.1.bb => mesa-gl_20.0.2.bb} | 0
 meta/recipes-graphics/mesa/{mesa_20.0.1.bb => mesa_20.0.2.bb} | 4 ++--
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-graphics/mesa/{mesa-gl_20.0.1.bb => mesa-gl_20.0.2.bb} 
(100%)
 rename meta/recipes-graphics/mesa/{mesa_20.0.1.bb => mesa_20.0.2.bb} (88%)

diff --git a/meta/recipes-graphics/mesa/mesa-gl_20.0.1.bb 
b/meta/recipes-graphics/mesa/mesa-gl_20.0.2.bb
similarity index 100%
rename from meta/recipes-graphics/mesa/mesa-gl_20.0.1.bb
rename to meta/recipes-graphics/mesa/mesa-gl_20.0.2.bb
diff --git a/meta/recipes-graphics/mesa/mesa_20.0.1.bb 
b/meta/recipes-graphics/mesa/mesa_20.0.2.bb
similarity index 88%
rename from meta/recipes-graphics/mesa/mesa_20.0.1.bb
rename to meta/recipes-graphics/mesa/mesa_20.0.2.bb
index e18c00b051..56ee28c870 100644
--- a/meta/recipes-graphics/mesa/mesa_20.0.1.bb
+++ b/meta/recipes-graphics/mesa/mesa_20.0.2.bb
@@ -9,8 +9,8 @@ SRC_URI = 
"https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
file://0001-meson-misdetects-64bit-atomics-on-mips-clang.patch \
"
 
-SRC_URI[md5sum] = "428ac511e4b05647a00b3778f1279da7"
-SRC_URI[sha256sum] = 
"6153ba3f8cb0524bbfc08e4db76b408126b2d1be8f789dffe28d1a0461eedde4"
+SRC_URI[md5sum] = "31ca59cd92c38cb8e1a58b62b01c3e08"
+SRC_URI[sha256sum] = 
"aa54f1cb669550606aab8ceb475105d15aeb814fca5a778ce70d0fd10e98e86f"
 
 UPSTREAM_CHECK_GITTAGREGEX = "mesa-(?P\d+(\.\d+)+)"
 
-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136688): 
https://lists.openembedded.org/g/openembedded-core/message/136688
Mute This Topic: https://lists.openembedded.org/mt/72532782/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH] gsettings-desktop-schemas: upgrade 3.34.0 -> 3.36.0

2020-03-24 Thread Wang Mingyu
Signed-off-by: Wang Mingyu 
---
 ...-schemas_3.34.0.bb => gsettings-desktop-schemas_3.36.0.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename 
meta/recipes-gnome/gsettings-desktop-schemas/{gsettings-desktop-schemas_3.34.0.bb
 => gsettings-desktop-schemas_3.36.0.bb} (80%)

diff --git 
a/meta/recipes-gnome/gsettings-desktop-schemas/gsettings-desktop-schemas_3.34.0.bb
 
b/meta/recipes-gnome/gsettings-desktop-schemas/gsettings-desktop-schemas_3.36.0.bb
similarity index 80%
rename from 
meta/recipes-gnome/gsettings-desktop-schemas/gsettings-desktop-schemas_3.34.0.bb
rename to 
meta/recipes-gnome/gsettings-desktop-schemas/gsettings-desktop-schemas_3.36.0.bb
index eea578bb6b..a68175dc84 100644
--- 
a/meta/recipes-gnome/gsettings-desktop-schemas/gsettings-desktop-schemas_3.34.0.bb
+++ 
b/meta/recipes-gnome/gsettings-desktop-schemas/gsettings-desktop-schemas_3.36.0.bb
@@ -13,6 +13,6 @@ GNOMEBASEBUILDCLASS = "meson"
 
 inherit gnomebase gsettings gobject-introspection gettext 
upstream-version-is-even
 
-SRC_URI[archive.md5sum] = "9759ef53fb2e53fc8d19190e58f2c332"
-SRC_URI[archive.sha256sum] = 
"288b04260f7040b0e004a8d59c773cfb4e32df4f1b4a0f9d705c51afccc95ead"
+SRC_URI[archive.md5sum] = "697db9e6b2ae058f8a0d0b023e32ceac"
+SRC_URI[archive.sha256sum] = 
"764ab683286536324533a58d4e95fc57f81adaba7d880dd0ebbbced63e960ea6"
 SRC_URI += "file://0001-Do-not-skip-gir-installation-for-cross-compiling.patch"
-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136687): 
https://lists.openembedded.org/g/openembedded-core/message/136687
Mute This Topic: https://lists.openembedded.org/mt/72532779/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


[OE-core] [PATCH] gnupg: upgrade 2.2.19 -> 2.2.20

2020-03-24 Thread Wang Mingyu
Signed-off-by: Wang Mingyu 
---
 .../gnupg/{gnupg_2.2.19.bb => gnupg_2.2.20.bb}| 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-support/gnupg/{gnupg_2.2.19.bb => gnupg_2.2.20.bb} (95%)

diff --git a/meta/recipes-support/gnupg/gnupg_2.2.19.bb 
b/meta/recipes-support/gnupg/gnupg_2.2.20.bb
similarity index 95%
rename from meta/recipes-support/gnupg/gnupg_2.2.19.bb
rename to meta/recipes-support/gnupg/gnupg_2.2.20.bb
index c07bff13be..f754573c88 100644
--- a/meta/recipes-support/gnupg/gnupg_2.2.19.bb
+++ b/meta/recipes-support/gnupg/gnupg_2.2.20.bb
@@ -20,8 +20,8 @@ SRC_URI_append_class-native = " 
file://0001-configure.ac-use-a-custom-value-for-
 file://relocate.patch"
 SRC_URI_append_class-nativesdk = " file://relocate.patch"
 
-SRC_URI[md5sum] = "cb3b373d08ba078c325299945a7f2818"
-SRC_URI[sha256sum] = 
"242554c0e06f3a83c420b052f750b65ead711cc3fb5e7274fcdbb4e9dec0"
+SRC_URI[md5sum] = "4ff88920cf52b35db0dedaee87bdbbb1"
+SRC_URI[sha256sum] = 
"04a7c9d48b74c399168ee8270e548588ddbe52218c337703d7f06373d326ca30"
 
 EXTRA_OECONF = "--disable-ldap \
--disable-ccid-driver \
-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136686): 
https://lists.openembedded.org/g/openembedded-core/message/136686
Mute This Topic: https://lists.openembedded.org/mt/72532778/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core] [PATCH 2/2] gstreamer1.0-plugins-good: add qt5 PACKAGECONFIG

2020-03-24 Thread Denys Dmytriyenko
On Tue, Mar 24, 2020 at 07:57:27PM +, Richard Purdie wrote:
> On Tue, 2020-03-24 at 15:32 -0400, Denys Dmytriyenko wrote:
> > On Tue, Mar 24, 2020 at 07:29:42PM +, Richard Purdie wrote:
> > > On Tue, 2020-03-24 at 14:43 -0400, Denys Dmytriyenko wrote:
> > > > On Tue, Mar 24, 2020 at 02:33:19PM -0400, Denys Dmytriyenko
> > > > wrote:
> > > > > Richard,
> > > > > 
> > > > > Yes, yes, I understanf all that from a technical perspective
> > > > > and
> > > > > agree with 
> > > > > it, but I was thinking about a more clear definition of this
> > > > > policy
> > > > > to be able 
> > > > > to point anyone having such questions in the future...
> > > > 
> > > > BTW, I don't remember discussing this at YP TSC... Maybe OE TSC
> > > > covered this in the past?
> > > 
> > > Metadata architecture decision would be the OE TSC and "TSC
> > > discussion
> > > long ago" means something from several years ago.
> > 
> > Thought so - were there any meeting minutes from back then? Sorry for
> > being so 
> > persistent, just trying to find some paper trail... :)
> 
> Yes, there is a record. You can find it quite easily. I don't need to
> do this for you...

All I could find is OE TSC minutes from 7 May 2013 referring to some related 
list discussion:

https://lists.openembedded.org/g/tsc/message/369?p=,,,20,0,0,0::Created,,PACKAGECONFIG,20,2,0,72188707

(9:47:21 AM) RP: bluelightning: I think this was to PACKAGECONFIG more recipes
(9:47:28 AM) bluelightning: ah ok
(9:47:36 AM) RP: bluelightning: remove the need to bbappend, have
people set mode config options
(9:47:51 AM) bluelightning: and PACKAGECONFIG to enable deps on things
outside OE-Core as well presumably
(9:48:01 AM) bluelightning: (as discussed on the ml a few weeks ago)
(9:48:06 AM) fray: yup

I spent last couple hours digging through my archives of openembedded-core and 
openembedded-devel lists around that time (Spring 2013), but besides general 
move to enable PACKAGECONFIG in recipes, I didn't see anything specific about 
crossing layer boundaries by PACKAGECONFIG dependencies, unfortunately. And it 
was a wild time - all the energy, extra activity, conflicts, flamewars - that 
definitely brought up some memories re-reading those discussions again... :)


BTW, all the links on this page are now dead, since they point to the old 
mailman/pipermail archives:
https://www.openembedded.org/wiki/TSC

Michael,

Is there any way to correct the links in bulk or preserve the old archives, so 
the old links would still work?

-- 
Denys
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136676): 
https://lists.openembedded.org/g/openembedded-core/message/136676
Mute This Topic: https://lists.openembedded.org/mt/72394353/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [OE-core][PATCH] security_flags.inc: fix flags missing from SDK toolchain

2020-03-24 Thread Richard Purdie
On Tue, 2020-03-24 at 20:06 +0100, Antoine Manache wrote:
> Absolutely.
> I would add an example of issue I've seen in a software project.
> Someone developing a new application and using the image-based SDK
> that I provided got the right functional behaviour on target.
> Once this application was integrated into the core image of our Yocto
> project (new recipe), the same tests on target failed (application
> crash).
> The root cause was : the two binaries produced were different because
> the effective GCC options at compile time were different between both
> methods (security flags not enabled when compiling with the SDK).
> Antoine

Thanks. Antonine's emails/patches are coming through as html and hence
are mangled and I can't apply them which is probably where things got
lost previously.

I've therefore taken Tom's version with some tweaks to the commit
message.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#136678): 
https://lists.openembedded.org/g/openembedded-core/message/136678
Mute This Topic: https://lists.openembedded.org/mt/72521987/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-