* [PATCH 0/4] kernel-yocto: consolidated pull request
@ 2016-12-02 21:09 Bruce Ashfield
2016-12-02 21:09 ` [PATCH 1/4] linux-yocto/4.8: update to v4.8.10 Bruce Ashfield
` (4 more replies)
0 siblings, 5 replies; 15+ messages in thread
From: Bruce Ashfield @ 2016-12-02 21:09 UTC (permalink / raw)
To: richard.purdie; +Cc: openembedded-core
Hi all,
This pull request is mainly to fix a couple of bugs that were reported
on the mailing list recently, but it also includes some kernel version
updates that I *think* I sent previously.
kern-tools: fix processing for no branch meta-data
kernel-yocto: exit on non-zero return code
Those two changes complete the error handling in the tools to display
a (more) helpful message to the user, and also to fix an important
use case where meta-data is applied to a kernel without new branches
being created.
I test built this against the rpi, and all is well.
Cheers,
Bruce
The following changes since commit 9e63f81c78e284c9b325fe04a1b59e61c7ad8a1a:
bitbake: ast: remove BBVERSIONS support (2016-11-30 15:48:10 +0000)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib zedd/kernel
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=zedd/kernel
Bruce Ashfield (4):
linux-yocto/4.8: update to v4.8.10
linux-yocto-dev: update to 4.9-rcX
kern-tools: fix processing for no branch meta-data
kernel-yocto: exit on non-zero return code
meta/classes/kernel-yocto.bbclass | 1 -
.../kern-tools/kern-tools-native_git.bb | 2 +-
meta/recipes-kernel/linux/linux-yocto-dev.bb | 2 +-
meta/recipes-kernel/linux/linux-yocto-rt_4.8.bb | 6 +++---
meta/recipes-kernel/linux/linux-yocto-tiny_4.8.bb | 6 +++---
meta/recipes-kernel/linux/linux-yocto_4.8.bb | 20 ++++++++++----------
6 files changed, 18 insertions(+), 19 deletions(-)
--
2.5.0
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 1/4] linux-yocto/4.8: update to v4.8.10 2016-12-02 21:09 [PATCH 0/4] kernel-yocto: consolidated pull request Bruce Ashfield @ 2016-12-02 21:09 ` Bruce Ashfield 2016-12-02 21:09 ` [PATCH 2/4] linux-yocto-dev: update to 4.9-rcX Bruce Ashfield ` (3 subsequent siblings) 4 siblings, 0 replies; 15+ messages in thread From: Bruce Ashfield @ 2016-12-02 21:09 UTC (permalink / raw) To: richard.purdie; +Cc: openembedded-core Integrating the 4.8.9 and 4.8.10 -stable updates. The commit log is as follows: cf5ae2989a32 Linux 4.8.10 5cd8f6788ff3 usb: gadget: f_fs: stop sleeping in ffs_func_eps_disable e2458382c792 usb: gadget: f_fs: edit epfile->ep under lock e34a0f1c53b5 sparc64: Delete now unused user copy fixup functions. af97481a6f5b sparc64: Delete now unused user copy assembler helpers. ac663c54f40b sparc64: Convert U3copy_{from,to}_user to accurate exception reporting. d91bb7a87e26 sparc64: Convert NG2copy_{from,to}_user to accurate exception reporting. a15859f9d839 sparc64: Convert NGcopy_{from,to}_user to accurate exception reporting. bb522726d311 sparc64: Convert NG4copy_{from,to}_user to accurate exception reporting. b0580eadc19f sparc64: Convert U1copy_{from,to}_user to accurate exception reporting. 50e927483ccf sparc64: Convert GENcopy_{from,to}_user to accurate exception reporting. 620ec41010d1 sparc64: Convert copy_in_user to accurate exception reporting. bf4d0da8e800 sparc64: Prepare to move to more saner user copy exception handling. bbbab9f59ea7 sparc64: Delete __ret_efault. 81a91edbb91a sparc64: Handle extremely large kernel TLB range flushes more gracefully. 7f8a50eb38d3 sparc64: Fix illegal relative branches in hypervisor patched TLB cross-call code. f7ef55af2f1b sparc64: Fix instruction count in comment for __hypervisor_flush_tlb_pending. 2a28ab3d4148 sparc64: Fix illegal relative branches in hypervisor patched TLB code. f4fb552a033e sparc64: Handle extremely large kernel TSB range flushes sanely. 51915c6d9070 sparc: Handle negative offsets in arch_jump_label_transform da6fe239ceff spi: spidev_test: fix build with musl libc 4ea98e573d65 net: stmmac: Fix lack of link transition for fixed PHYs 150b491b1b88 sctp: change sk state only when it has assocs in sctp_shutdown 5235fcfa6cf8 bnx2: Wait for in-flight DMA to complete at probe stage 6523ff2e27fe Revert "bnx2: Reset device during driver initialization" 224fb8cbefb2 mlxsw: spectrum_router: Correctly dump neighbour activity 9092bbd64bd9 mlxsw: spectrum: Fix refcount bug on span entries 5712922773b5 Revert "include/uapi/linux/atm_zatm.h: include linux/time.h" 2b5f22e4f7fd tcp: take care of truncations done by sk_filter() 22a78d4c7f43 ipv4: use new_gw for redirect neigh lookup bccb4093d464 net: __skb_flow_dissect() must cap its return value a1632e969a55 net: icmp_route_lookup should use rt dev to determine L3 domain 9885f474d92b sock: fix sendmmsg for partial sendmsg b78ba0a0f231 fib_trie: Correct /proc/net/route off by one error 92fd1c1f2fd2 net: icmp6_send should use dst dev to determine L3 domain 09ee09498bca bpf: fix htab map destruction when extra reserve is in use de289ad2e575 sctp: assign assoc_id earlier in __sctp_connect 76b5fee5cfa0 ipv6: dccp: add missing bind_conflict to dccp_ipv6_mapped 84d9c612bb7a ipv6: dccp: fix out of bound access in dccp_v6_err() ba93cf7d2118 dccp: fix out of bound access in dccp_v4_err() 378a61101374 dccp: do not send reset to already closed sockets 72b03e549b95 dccp: do not release listeners too soon b3523a0773ed tcp: fix return value for partial writes 1f49cc6fa91c ipv4: allow local fragmentation in ip_finish_output_gso() 842a858fa048 tcp: fix potential memory corruption fc3b825f2c81 ip6_tunnel: Clear IP6CB in ip6tunnel_xmit() f5f4b71d5632 bgmac: stop clearing DMA receive control register right after it is set 0c7f764d2c6a net: mangle zero checksum in skb_checksum_help() ac22a3ba0796 net: clear sk_err_soft in sk_clone_lock() 5b078dc6fb64 dctcp: avoid bogus doubling of cwnd after loss 876577321657 Linux 4.8.9 07d00beb1e04 netfilter: fix namespace handling in nf_log_proc_dostring 8ef009e09c13 drm/i915: Fix mismatched INIT power domain disabling during suspend 88a45e5d2c0d drm/amdgpu: fix a vm_flush fence leak 25ed6e4b0b65 drm/amdgpu: fix fence slab teardown de5e9aa77a3c NFSv4.1: work around -Wmaybe-uninitialized warning 18c801047a18 libceph: fix legacy layout decode with pool 0 53c1792b94da memcg: prevent memcg caches to be both OFF_SLAB & OBJFREELIST_SLAB 02e1ee6b3e1c mmc: mxs: Initialize the spinlock prior to using it ce0702e35aeb pinctrl: iproc: Fix iProc and NSP GPIO support 320244ac9eb6 ASoC: sun4i-codec: return error code instead of NULL when create_card fails 2140d4fd9277 ASoC: Intel: Skylake: Always acquire runtime pm ref on unload 5037fdbc62c2 gpio: of: fix GPIO drivers with multiple gpio_chip for a single node 7a9239fd0480 gpio/mvebu: Use irq_domain_add_linear 6de98e87effb batman-adv: Modify neigh_list only with rcu-list functions a3f000ce7b44 ACPI/PCI: pci_link: Include PIRQ_PENALTY_PCI_USING for ISA IRQs 6c76dd0c7066 ACPI/PCI: pci_link: penalize SCI correctly 86c711665c84 ACPI/PCI/IRQ: assign ISA IRQ directly during early boot stages ad185d9251e1 ACPI / APEI: Fix incorrect return value of ghes_proc() b55ebc89ab1d mmc: sdhci-msm: Fix error return code in sdhci_msm_probe() 85284c0850f9 i40e: fix call of ndo_dflt_bridge_getlink() 1242c9dfab0c hwrng: core - Don't use a stack buffer in add_early_randomness() c1a2ada73dac lib/genalloc.c: start search from start of chunk 06bb5ebedbb4 s390/dumpstack: restore reliable indicator for call traces 1ef1bd02ad23 rtc: pcf2123: Add missing error code assignment before test 4baabb72e9dd clk: samsung: clk-exynos-audss: Fix module autoload 3bbdbd8aa3c8 x86/build: Fix build with older GCC versions f5eadc27a60c Revert "clocksource/drivers/timer_sun5i: Replace code by clocksource_mmio_init" 645a6b823739 nvme: Delete created IO queues on reset 07c4cbe01341 svcrdma: Tail iovec leaves an orphaned DMA mapping 4131e00a436e svcrdma: Skip put_page() when send_reply() fails 755ab7aa1466 mei: bus: fix received data size check in NFC fixup d1b564536c6a perf top: Fix refreshing hierarchy entries on TUI 6ac4e06b717f Input: synaptics-rmi4 - fix error handling in I2C transport driver d3716f1b3e4b Input: synaptics-rmi4 - fix error handling in SPI transport driver 66503ec38f34 watchdog: core: Fix devres_alloc() allocation size c5e9e5cc8cd1 agp/intel: Flush chipset writes after updating a single PTE 813617a4c8dc iommu/vt-d: Fix dead-locks in disable_dmar_iommu() path b6ef0b142208 iommu/amd: Free domain id when free a domain of struct dma_ops_domain 2ef38255b588 iommu/io-pgtable-arm: Check for v7s-incapable systems d3d9428d7133 xprtrdma: Fix DMAR failure in frwr_op_map() after reconnect 31c749bee3de xprtrdma: use complete() instead complete_all() 67080e2785a3 drm/amd: fix scheduler fence teardown order v2 b0da5ab2ffb5 drm/amdgpu: fix sched fence slab teardown 920a85ba4306 tty/serial: at91: fix hardware handshake on Atmel platforms 9d76a886eb2f drm/amdgpu: fix crash in acp_hw_fini 6e652d18d73d drm/amdgpu: disable runtime pm in certain cases 8c8fdc683295 drm/i915/dp: Extend BDW DP audio workaround to GEN9 platforms db8e005bf2ba drm/i915/dp: BDW cdclk fix for DP audio f50b7450a8e4 drm/i915: Respect alternate_ddc_pin for all DDI ports e1b24f6a0b3c drm/radeon: disable runtime pm in certain cases eb13abb0e515 KVM: arm/arm64: vgic: Prevent access to invalid SPIs 2850fad5acb0 scsi: scsi_dh_alua: Fix a reference counting bug 5fac70d772a4 scsi: scsi_dh_alua: fix missing kref_put() in alua_rtpg_work() f29bcd11a170 scsi: mpt3sas: Fix for block device of raid exists even after deleting raid disk b2040deabb07 scsi: qla2xxx: Fix scsi scan hang triggered if adapter fails during init 1281b9683f96 iio: st_sensors: fix scale configuration for h3lis331dl 4dfb6d1dd11d iio: orientation: hid-sensor-rotation: Add PM function (fix non working driver) 341c5534edad iio: hid-sensors: Increase the precision of scale to fix wrong reading interpretation. b9d031f354da cdc-acm: fix uninitialized variable c480880cd12c clk: qoriq: Don't allow CPU clocks higher than starting value ee27fd32c499 toshiba-wmi: Fix loading the driver on non Toshiba laptops f713523a234c drbd: Fix kernel_sendmsg() usage - potential NULL deref 31da266e4b9a usb: gadget: u_ether: remove interrupt throttling 264e4131a167 USB: cdc-acm: fix TIOCMIWAIT c12c24eeaf77 usb: dwc3: Fix error handling for core init 6b95417150b7 staging: nvec: remove managed resource from PS2 driver 82239ab105a5 Revert "staging: nvec: ps2: change serio type to passthrough" 529789866abe drivers: staging: nvec: remove bogus reset command for PS/2 interface dea774aac054 staging: comedi: ni_tio: fix buggy ni_tio_clock_period_ps() return value 5d510185f99a staging: sm750fb: Fix bugs introduced by early commits 45983d678b70 staging: iio: ad5933: avoid uninitialized variable in error case 5289e59246dd mmc: mmc: Use 500ms as the default generic CMD6 timeout ce4dfe7d927e mmc: sdhci: Fix unexpected data interrupt handling bde8d3f73375 mmc: sdhci: Fix CMD line reset interfering with ongoing data transfer 10d24701f3f4 cpupower: Correct return type of cpu_power_is_cpu_online() in cpufreq-set f062e738c19b pinctrl: cherryview: Prevent possible interrupt storm on resume 1d99fe3317e3 pinctrl: cherryview: Serialize register access in suspend/resume f5ad96462615 arc: Implement arch-specific dma_map_ops.mmap 004e7c97f181 PCI: Don't attempt to claim shadow copies of ROM 7fac0361f94e ARC: timer: rtc: implement read loop in "C" vs. inline asm 4058116db4d0 s390/hypfs: Use get_free_page() instead of kmalloc to ensure page alignment 2f3e0b82ebe9 coredump: fix unfreezable coredumping task d6ee4f47e2e7 mm/hugetlb: fix huge page reservation leak in private mapping error paths e87bf4f558f1 mm: hwpoison: fix thp split handling in memory_failure() 67c79e166d46 swapfile: fix memory corruption via malformed swapfile c87739e5d029 shmem: fix pageflags after swapping DMA32 object 5b5243b606ec mm, frontswap: make sure allocated frontswap map is assigned 2e594273d4c5 dib0700: fix nec repeat handling 9964230320d5 ASoC: cs4270: fix DAPM stream name mismatch 9386a722d250 ALSA: info: Limit the proc text input size c3ea1b15b71e ALSA: info: Return error for invalid read/write Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> --- meta/recipes-kernel/linux/linux-yocto-rt_4.8.bb | 6 +++--- meta/recipes-kernel/linux/linux-yocto-tiny_4.8.bb | 6 +++--- meta/recipes-kernel/linux/linux-yocto_4.8.bb | 20 ++++++++++---------- 3 files changed, 16 insertions(+), 16 deletions(-) diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.8.bb b/meta/recipes-kernel/linux/linux-yocto-rt_4.8.bb index 0996fa044e73..3d9ff393fdd0 100644 --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.8.bb +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.8.bb @@ -11,13 +11,13 @@ python () { raise bb.parse.SkipPackage("Set PREFERRED_PROVIDER_virtual/kernel to linux-yocto-rt to enable it") } -SRCREV_machine ?= "c4a91e62ea517274fde5ebb46916010eac5159f6" -SRCREV_meta ?= "1eac0b697a507e2dfeeb06294c887425170711b9" +SRCREV_machine ?= "4b6a2814a1ac2a917a8334748b3168315e678041" +SRCREV_meta ?= "1dc615a67779fdfd36548fd48e54bd19b6e6209e" SRC_URI = "git://git.yoctoproject.org/linux-yocto-4.8.git;branch=${KBRANCH};name=machine \ git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-4.8;destsuffix=${KMETA}" -LINUX_VERSION ?= "4.8.8" +LINUX_VERSION ?= "4.8.10" PV = "${LINUX_VERSION}+git${SRCPV}" diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_4.8.bb b/meta/recipes-kernel/linux/linux-yocto-tiny_4.8.bb index 120f268b96d1..870a3487f3d3 100644 --- a/meta/recipes-kernel/linux/linux-yocto-tiny_4.8.bb +++ b/meta/recipes-kernel/linux/linux-yocto-tiny_4.8.bb @@ -4,13 +4,13 @@ KCONFIG_MODE = "--allnoconfig" require recipes-kernel/linux/linux-yocto.inc -LINUX_VERSION ?= "4.8.8" +LINUX_VERSION ?= "4.8.10" KMETA = "kernel-meta" KCONF_BSP_AUDIT_LEVEL = "2" -SRCREV_machine ?= "2e7e9f1e599eb255d038ede09f1765a1bb1378cf" -SRCREV_meta ?= "1eac0b697a507e2dfeeb06294c887425170711b9" +SRCREV_machine ?= "96ea072413fb386598a62c77ed202a6de08085f6" +SRCREV_meta ?= "1dc615a67779fdfd36548fd48e54bd19b6e6209e" PV = "${LINUX_VERSION}+git${SRCPV}" diff --git a/meta/recipes-kernel/linux/linux-yocto_4.8.bb b/meta/recipes-kernel/linux/linux-yocto_4.8.bb index f9457051aa90..a4dd9adc7162 100644 --- a/meta/recipes-kernel/linux/linux-yocto_4.8.bb +++ b/meta/recipes-kernel/linux/linux-yocto_4.8.bb @@ -11,20 +11,20 @@ KBRANCH_qemux86 ?= "standard/base" KBRANCH_qemux86-64 ?= "standard/base" KBRANCH_qemumips64 ?= "standard/mti-malta64" -SRCREV_machine_qemuarm ?= "800a8564447b6ec75500b7cd6ebce78d652a466e" -SRCREV_machine_qemuarm64 ?= "2e7e9f1e599eb255d038ede09f1765a1bb1378cf" -SRCREV_machine_qemumips ?= "e01c5adddc9e48bb2b3ae74ae5f12f356a6519dd" -SRCREV_machine_qemuppc ?= "2e7e9f1e599eb255d038ede09f1765a1bb1378cf" -SRCREV_machine_qemux86 ?= "6745463c22f59717df34f819359ea2955611fd03" -SRCREV_machine_qemux86-64 ?= "6745463c22f59717df34f819359ea2955611fd03" -SRCREV_machine_qemumips64 ?= "b9217c2fa88d138f4082c730fb3b450f7c56da81" -SRCREV_machine ?= "6745463c22f59717df34f819359ea2955611fd03" -SRCREV_meta ?= "1eac0b697a507e2dfeeb06294c887425170711b9" +SRCREV_machine_qemuarm ?= "a0241e756033146b308e94e262f0c434d4d5dcdc" +SRCREV_machine_qemuarm64 ?= "72e32211033cff873824d944470d06f590f66a85" +SRCREV_machine_qemumips ?= "b17c74d6df96711a98c48515b6c171dfe492a4a6" +SRCREV_machine_qemuppc ?= "15c034ae478c08219aaefaf8a9b57ee2939715a9" +SRCREV_machine_qemux86 ?= "f5d46e21688b7463c942d9d27789d88d580f566b" +SRCREV_machine_qemux86-64 ?= "f5d46e21688b7463c942d9d27789d88d580f566b" +SRCREV_machine_qemumips64 ?= "7688c6f57bc68c6b39f0bd7a20ea84419ebfaf50" +SRCREV_machine ?= "f5d46e21688b7463c942d9d27789d88d580f566b" +SRCREV_meta ?= "1dc615a67779fdfd36548fd48e54bd19b6e6209e" SRC_URI = "git://git.yoctoproject.org/linux-yocto-4.8.git;name=machine;branch=${KBRANCH}; \ git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-4.8;destsuffix=${KMETA}" -LINUX_VERSION ?= "4.8.8" +LINUX_VERSION ?= "4.8.10" PV = "${LINUX_VERSION}+git${SRCPV}" -- 2.5.0 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/4] linux-yocto-dev: update to 4.9-rcX 2016-12-02 21:09 [PATCH 0/4] kernel-yocto: consolidated pull request Bruce Ashfield 2016-12-02 21:09 ` [PATCH 1/4] linux-yocto/4.8: update to v4.8.10 Bruce Ashfield @ 2016-12-02 21:09 ` Bruce Ashfield 2016-12-02 21:09 ` [PATCH 3/4] kern-tools: fix processing for no branch meta-data Bruce Ashfield ` (2 subsequent siblings) 4 siblings, 0 replies; 15+ messages in thread From: Bruce Ashfield @ 2016-12-02 21:09 UTC (permalink / raw) To: richard.purdie; +Cc: openembedded-core Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> --- meta/recipes-kernel/linux/linux-yocto-dev.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-kernel/linux/linux-yocto-dev.bb b/meta/recipes-kernel/linux/linux-yocto-dev.bb index 9154bb7c3113..0cda553afdba 100644 --- a/meta/recipes-kernel/linux/linux-yocto-dev.bb +++ b/meta/recipes-kernel/linux/linux-yocto-dev.bb @@ -28,7 +28,7 @@ SRC_URI = "git://git.yoctoproject.org/linux-yocto-dev.git;branch=${KBRANCH};name SRCREV_machine ?= '${@oe.utils.conditional("PREFERRED_PROVIDER_virtual/kernel", "linux-yocto-dev", "${AUTOREV}", "29594404d7fe73cd80eaa4ee8c43dcc53970c60e", d)}' SRCREV_meta ?= '${@oe.utils.conditional("PREFERRED_PROVIDER_virtual/kernel", "linux-yocto-dev", "${AUTOREV}", "29594404d7fe73cd80eaa4ee8c43dcc53970c60e", d)}' -LINUX_VERSION ?= "4.8-rc+" +LINUX_VERSION ?= "4.9-rc+" LINUX_VERSION_EXTENSION ?= "-yoctodev-${LINUX_KERNEL_TYPE}" PV = "${LINUX_VERSION}+git${SRCPV}" -- 2.5.0 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-02 21:09 [PATCH 0/4] kernel-yocto: consolidated pull request Bruce Ashfield 2016-12-02 21:09 ` [PATCH 1/4] linux-yocto/4.8: update to v4.8.10 Bruce Ashfield 2016-12-02 21:09 ` [PATCH 2/4] linux-yocto-dev: update to 4.9-rcX Bruce Ashfield @ 2016-12-02 21:09 ` Bruce Ashfield 2016-12-06 21:14 ` Patrick Ohly 2016-12-02 21:09 ` [PATCH 4/4] kernel-yocto: exit on non-zero return code Bruce Ashfield 2016-12-06 19:32 ` [PATCH 0/4] kernel-yocto: consolidated pull request Trevor Woerner 4 siblings, 1 reply; 15+ messages in thread From: Bruce Ashfield @ 2016-12-02 21:09 UTC (permalink / raw) To: richard.purdie; +Cc: openembedded-core Lernel meta-data that has patches, but no branches, can trigger an error due to no branch specific patch queue. This error then cascades to more issues since the tools are using a named file in /tmp to store and display error messages to the user. We fix both issues though the following kern tools tweaks: commit bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c Author: Bruce Ashfield <bruce.ashfield@windriver.com> Date: Fri Dec 2 13:09:40 2016 -0500 scc: use mktemp for consolidated output capture To provide useful error messages the tools dump pre-processed files and messages to a temporary file. If multiple users are doing builds, this means they either race, or can have permissions issues. By creating the temporary file via mktemp, we avoid both issues. (We also make sure to clean these up on exit, or /tmp will get polluted quickly). commit a287da4bfe0b4acb8f2b0627bd8e7abd1a1dde26 Author: Bruce Ashfield <bruce.ashfield@windriver.com> Date: Fri Dec 2 13:08:08 2016 -0500 patch: do not assume a branch specific patch queue is needed When processing input files per-branch and global patch queues are generated. If the meta-data has not created any branches in the repo, no branch specific queue is required. The tools assumed that one is always valid, and hence would throw a non-zero exit code and stop processing. By testing for a named per-branch queue, we avoid this issue. Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> --- meta/recipes-kernel/kern-tools/kern-tools-native_git.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb b/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb index 88b1aee729c9..0f8a786176f9 100644 --- a/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb +++ b/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb @@ -4,7 +4,7 @@ LIC_FILES_CHKSUM = "file://git/tools/kgit;beginline=5;endline=9;md5=a6c2fa8aef1b DEPENDS = "git-native" -SRCREV = "b36f6f9a5695f2084b83823393e13ca42284bed9" +SRCREV = "bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c" PR = "r12" PV = "0.2+git${SRCPV}" -- 2.5.0 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-02 21:09 ` [PATCH 3/4] kern-tools: fix processing for no branch meta-data Bruce Ashfield @ 2016-12-06 21:14 ` Patrick Ohly 2016-12-06 23:02 ` Paul Barker 0 siblings, 1 reply; 15+ messages in thread From: Patrick Ohly @ 2016-12-06 21:14 UTC (permalink / raw) To: Bruce Ashfield, Mikko Ylinen; +Cc: openembedded-core On Fri, 2016-12-02 at 16:09 -0500, Bruce Ashfield wrote: > Lernel meta-data that has patches, but no branches, can trigger an > error due to no branch specific patch queue. > > This error then cascades to more issues since the tools are using > a named file in /tmp to store and display error messages to the > user. > > We fix both issues though the following kern tools tweaks: > > commit bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c > Author: Bruce Ashfield <bruce.ashfield@windriver.com> > Date: Fri Dec 2 13:09:40 2016 -0500 > > scc: use mktemp for consolidated output capture > > To provide useful error messages the tools dump pre-processed > files and messages to a temporary file. If multiple users are > doing builds, this means they either race, or can have permissions > issues. > > By creating the temporary file via mktemp, we avoid both issues. > (We also make sure to clean these up on exit, or /tmp will get > polluted quickly). > > commit a287da4bfe0b4acb8f2b0627bd8e7abd1a1dde26 > Author: Bruce Ashfield <bruce.ashfield@windriver.com> > Date: Fri Dec 2 13:08:08 2016 -0500 > > patch: do not assume a branch specific patch queue is needed > > When processing input files per-branch and global patch queues are > generated. If the meta-data has not created any branches in the > repo, no branch specific queue is required. > > The tools assumed that one is always valid, and hence would throw a > non-zero exit code and stop processing. > > By testing for a named per-branch queue, we avoid this issue. Ostro OS runs into the problem while trying to use current OE-core master: .../patch.cmd: line 29: : No such file or directory | ERROR: Function failed: do_kernel_metadata (log file is located ...) This commit here fixed it for me. I see that it is already in Ross' mut2 branch, so hopefully that'll land in master soon. -- Best Regards, Patrick Ohly The content of this message is my personal opinion only and although I am an employee of Intel, the statements I make here in no way represent Intel's position on the issue, nor am I authorized to speak on behalf of Intel on this matter. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-06 21:14 ` Patrick Ohly @ 2016-12-06 23:02 ` Paul Barker 2016-12-07 15:11 ` Trevor Woerner 0 siblings, 1 reply; 15+ messages in thread From: Paul Barker @ 2016-12-06 23:02 UTC (permalink / raw) To: Patrick Ohly; +Cc: Bruce Ashfield, openembedded-core On Tue, 06 Dec 2016 22:14:41 +0100 Patrick Ohly <patrick.ohly@intel.com> wrote: > On Fri, 2016-12-02 at 16:09 -0500, Bruce Ashfield wrote: > > Lernel meta-data that has patches, but no branches, can trigger an > > error due to no branch specific patch queue. > > > > This error then cascades to more issues since the tools are using > > a named file in /tmp to store and display error messages to the > > user. > > > > We fix both issues though the following kern tools tweaks: > > > > commit bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c > > Author: Bruce Ashfield <bruce.ashfield@windriver.com> > > Date: Fri Dec 2 13:09:40 2016 -0500 > > > > scc: use mktemp for consolidated output capture > > > > To provide useful error messages the tools dump pre-processed > > files and messages to a temporary file. If multiple users are > > doing builds, this means they either race, or can have permissions > > issues. > > > > By creating the temporary file via mktemp, we avoid both issues. > > (We also make sure to clean these up on exit, or /tmp will get > > polluted quickly). > > > > commit a287da4bfe0b4acb8f2b0627bd8e7abd1a1dde26 > > Author: Bruce Ashfield <bruce.ashfield@windriver.com> > > Date: Fri Dec 2 13:08:08 2016 -0500 > > > > patch: do not assume a branch specific patch queue is needed > > > > When processing input files per-branch and global patch queues are > > generated. If the meta-data has not created any branches in the > > repo, no branch specific queue is required. > > > > The tools assumed that one is always valid, and hence would throw a > > non-zero exit code and stop processing. > > > > By testing for a named per-branch queue, we avoid this issue. > > Ostro OS runs into the problem while trying to use current OE-core > master: > > .../patch.cmd: line 29: : No such file or directory > > | ERROR: Function failed: do_kernel_metadata (log file is located ...) > > This commit here fixed it for me. I see that it is already in Ross' mut2 > branch, so hopefully that'll land in master soon. > Ditto for meta-raspberrypi, the kernel doesn't build with current oe-core master as discussed on the yocto@ list. This patch is needed to fix things. Thanks, Paul Barker ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-06 23:02 ` Paul Barker @ 2016-12-07 15:11 ` Trevor Woerner 2016-12-07 15:14 ` Bruce Ashfield 2016-12-07 16:18 ` Bruce Ashfield 0 siblings, 2 replies; 15+ messages in thread From: Trevor Woerner @ 2016-12-07 15:11 UTC (permalink / raw) To: Paul Barker Cc: Bruce Ashfield, Patches and discussions about the oe-core layer This patch does fix the raspberrypi issue, and it has been pushed, thankfully. But I'm still seeing hard-coded filenames in /tmp that are messing up the ability for more than one person to use a given build machine. E.g.: /tmp/patch.standard.arm-versatile-926ejs.queue /tmp/patch.standard.mti-malta32.queue /tmp/patch.standard.mti-malta64.queue /tmp/patch.standard.preempt-rt.base.queue /tmp/patch.standard.preempt-rt.queue /tmp/patch.standard.queue /tmp/patch.unused.queue I think the malta ones are used for qemumips64, which is still failing for this reason (multiple users) On Tue, Dec 6, 2016 at 6:02 PM, Paul Barker <paul@paulbarker.me.uk> wrote: > On Tue, 06 Dec 2016 22:14:41 +0100 > Patrick Ohly <patrick.ohly@intel.com> wrote: > >> On Fri, 2016-12-02 at 16:09 -0500, Bruce Ashfield wrote: >> > Lernel meta-data that has patches, but no branches, can trigger an >> > error due to no branch specific patch queue. >> > >> > This error then cascades to more issues since the tools are using >> > a named file in /tmp to store and display error messages to the >> > user. >> > >> > We fix both issues though the following kern tools tweaks: >> > >> > commit bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c >> > Author: Bruce Ashfield <bruce.ashfield@windriver.com> >> > Date: Fri Dec 2 13:09:40 2016 -0500 >> > >> > scc: use mktemp for consolidated output capture >> > >> > To provide useful error messages the tools dump pre-processed >> > files and messages to a temporary file. If multiple users are >> > doing builds, this means they either race, or can have permissions >> > issues. >> > >> > By creating the temporary file via mktemp, we avoid both issues. >> > (We also make sure to clean these up on exit, or /tmp will get >> > polluted quickly). >> > >> > commit a287da4bfe0b4acb8f2b0627bd8e7abd1a1dde26 >> > Author: Bruce Ashfield <bruce.ashfield@windriver.com> >> > Date: Fri Dec 2 13:08:08 2016 -0500 >> > >> > patch: do not assume a branch specific patch queue is needed >> > >> > When processing input files per-branch and global patch queues are >> > generated. If the meta-data has not created any branches in the >> > repo, no branch specific queue is required. >> > >> > The tools assumed that one is always valid, and hence would throw a >> > non-zero exit code and stop processing. >> > >> > By testing for a named per-branch queue, we avoid this issue. >> >> Ostro OS runs into the problem while trying to use current OE-core >> master: >> >> .../patch.cmd: line 29: : No such file or directory >> >> | ERROR: Function failed: do_kernel_metadata (log file is located ...) >> >> This commit here fixed it for me. I see that it is already in Ross' mut2 >> branch, so hopefully that'll land in master soon. >> > > Ditto for meta-raspberrypi, the kernel doesn't build with current > oe-core master as discussed on the yocto@ list. This patch is needed > to fix things. > > Thanks, > Paul Barker > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-07 15:11 ` Trevor Woerner @ 2016-12-07 15:14 ` Bruce Ashfield 2016-12-07 16:18 ` Bruce Ashfield 1 sibling, 0 replies; 15+ messages in thread From: Bruce Ashfield @ 2016-12-07 15:14 UTC (permalink / raw) To: Trevor Woerner, Paul Barker Cc: Patches and discussions about the oe-core layer On 2016-12-07 10:11 AM, Trevor Woerner wrote: > This patch does fix the raspberrypi issue, and it has been pushed, thankfully. > > But I'm still seeing hard-coded filenames in /tmp that are messing up > the ability for more than one person to use a given build machine. > E.g.: > > /tmp/patch.standard.arm-versatile-926ejs.queue > /tmp/patch.standard.mti-malta32.queue > /tmp/patch.standard.mti-malta64.queue > /tmp/patch.standard.preempt-rt.base.queue > /tmp/patch.standard.preempt-rt.queue > /tmp/patch.standard.queue > /tmp/patch.unused.queue > > I think the malta ones are used for qemumips64, which is still failing > for this reason (multiple users) Those are never supposed to go to /tmp/, they are targeted to ${S} of the kernel build. It must be a side effect of the change I just did where the branch isn't defined. I'll whip up another patch. Bruce > > On Tue, Dec 6, 2016 at 6:02 PM, Paul Barker <paul@paulbarker.me.uk> wrote: >> On Tue, 06 Dec 2016 22:14:41 +0100 >> Patrick Ohly <patrick.ohly@intel.com> wrote: >> >>> On Fri, 2016-12-02 at 16:09 -0500, Bruce Ashfield wrote: >>>> Lernel meta-data that has patches, but no branches, can trigger an >>>> error due to no branch specific patch queue. >>>> >>>> This error then cascades to more issues since the tools are using >>>> a named file in /tmp to store and display error messages to the >>>> user. >>>> >>>> We fix both issues though the following kern tools tweaks: >>>> >>>> commit bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c >>>> Author: Bruce Ashfield <bruce.ashfield@windriver.com> >>>> Date: Fri Dec 2 13:09:40 2016 -0500 >>>> >>>> scc: use mktemp for consolidated output capture >>>> >>>> To provide useful error messages the tools dump pre-processed >>>> files and messages to a temporary file. If multiple users are >>>> doing builds, this means they either race, or can have permissions >>>> issues. >>>> >>>> By creating the temporary file via mktemp, we avoid both issues. >>>> (We also make sure to clean these up on exit, or /tmp will get >>>> polluted quickly). >>>> >>>> commit a287da4bfe0b4acb8f2b0627bd8e7abd1a1dde26 >>>> Author: Bruce Ashfield <bruce.ashfield@windriver.com> >>>> Date: Fri Dec 2 13:08:08 2016 -0500 >>>> >>>> patch: do not assume a branch specific patch queue is needed >>>> >>>> When processing input files per-branch and global patch queues are >>>> generated. If the meta-data has not created any branches in the >>>> repo, no branch specific queue is required. >>>> >>>> The tools assumed that one is always valid, and hence would throw a >>>> non-zero exit code and stop processing. >>>> >>>> By testing for a named per-branch queue, we avoid this issue. >>> >>> Ostro OS runs into the problem while trying to use current OE-core >>> master: >>> >>> .../patch.cmd: line 29: : No such file or directory >>> >>> | ERROR: Function failed: do_kernel_metadata (log file is located ...) >>> >>> This commit here fixed it for me. I see that it is already in Ross' mut2 >>> branch, so hopefully that'll land in master soon. >>> >> >> Ditto for meta-raspberrypi, the kernel doesn't build with current >> oe-core master as discussed on the yocto@ list. This patch is needed >> to fix things. >> >> Thanks, >> Paul Barker >> -- >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-07 15:11 ` Trevor Woerner 2016-12-07 15:14 ` Bruce Ashfield @ 2016-12-07 16:18 ` Bruce Ashfield 2016-12-07 18:05 ` Trevor Woerner 1 sibling, 1 reply; 15+ messages in thread From: Bruce Ashfield @ 2016-12-07 16:18 UTC (permalink / raw) To: Trevor Woerner, Paul Barker Cc: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 3683 bytes --] On 2016-12-07 10:11 AM, Trevor Woerner wrote: > This patch does fix the raspberrypi issue, and it has been pushed, thankfully. > > But I'm still seeing hard-coded filenames in /tmp that are messing up > the ability for more than one person to use a given build machine. > E.g.: > > /tmp/patch.standard.arm-versatile-926ejs.queue > /tmp/patch.standard.mti-malta32.queue > /tmp/patch.standard.mti-malta64.queue > /tmp/patch.standard.preempt-rt.base.queue > /tmp/patch.standard.preempt-rt.queue > /tmp/patch.standard.queue > /tmp/patch.unused.queue > > I think the malta ones are used for qemumips64, which is still failing > for this reason (multiple users) I tracked this down to some of the sub commands using the dirname of the global patch queue .. and since that is in /tmp/, they are following along. With the attached patch, I see nothing else that is named in /tmp/ If you have the cycles, can you give it a try and let me know ? Bruce > > On Tue, Dec 6, 2016 at 6:02 PM, Paul Barker <paul@paulbarker.me.uk> wrote: >> On Tue, 06 Dec 2016 22:14:41 +0100 >> Patrick Ohly <patrick.ohly@intel.com> wrote: >> >>> On Fri, 2016-12-02 at 16:09 -0500, Bruce Ashfield wrote: >>>> Lernel meta-data that has patches, but no branches, can trigger an >>>> error due to no branch specific patch queue. >>>> >>>> This error then cascades to more issues since the tools are using >>>> a named file in /tmp to store and display error messages to the >>>> user. >>>> >>>> We fix both issues though the following kern tools tweaks: >>>> >>>> commit bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c >>>> Author: Bruce Ashfield <bruce.ashfield@windriver.com> >>>> Date: Fri Dec 2 13:09:40 2016 -0500 >>>> >>>> scc: use mktemp for consolidated output capture >>>> >>>> To provide useful error messages the tools dump pre-processed >>>> files and messages to a temporary file. If multiple users are >>>> doing builds, this means they either race, or can have permissions >>>> issues. >>>> >>>> By creating the temporary file via mktemp, we avoid both issues. >>>> (We also make sure to clean these up on exit, or /tmp will get >>>> polluted quickly). >>>> >>>> commit a287da4bfe0b4acb8f2b0627bd8e7abd1a1dde26 >>>> Author: Bruce Ashfield <bruce.ashfield@windriver.com> >>>> Date: Fri Dec 2 13:08:08 2016 -0500 >>>> >>>> patch: do not assume a branch specific patch queue is needed >>>> >>>> When processing input files per-branch and global patch queues are >>>> generated. If the meta-data has not created any branches in the >>>> repo, no branch specific queue is required. >>>> >>>> The tools assumed that one is always valid, and hence would throw a >>>> non-zero exit code and stop processing. >>>> >>>> By testing for a named per-branch queue, we avoid this issue. >>> >>> Ostro OS runs into the problem while trying to use current OE-core >>> master: >>> >>> .../patch.cmd: line 29: : No such file or directory >>> >>> | ERROR: Function failed: do_kernel_metadata (log file is located ...) >>> >>> This commit here fixed it for me. I see that it is already in Ross' mut2 >>> branch, so hopefully that'll land in master soon. >>> >> >> Ditto for meta-raspberrypi, the kernel doesn't build with current >> oe-core master as discussed on the yocto@ list. This patch is needed >> to fix things. >> >> Thanks, >> Paul Barker >> -- >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core [-- Attachment #2: 0001-kern-tools-ensure-that-no-shared-directories-are-use.patch --] [-- Type: text/x-patch, Size: 1287 bytes --] From 707978ff47533c705eac42619988793f07a66b80 Mon Sep 17 00:00:00 2001 From: Bruce Ashfield <bruce.ashfield@windriver.com> Date: Wed, 7 Dec 2016 11:14:34 -0500 Subject: [PATCH] kern-tools: ensure that no shared directories are used We need to avoid using shared/common directories for any files that are part of specific build, since permissions issues in multi user environments will cause issues. Integrating the following commit to solve the issue: scc: move unused patch queue under output dir Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> --- meta/recipes-kernel/kern-tools/kern-tools-native_git.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb b/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb index 0f8a786176f9..3ca6ca22ef20 100644 --- a/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb +++ b/meta/recipes-kernel/kern-tools/kern-tools-native_git.bb @@ -4,7 +4,7 @@ LIC_FILES_CHKSUM = "file://git/tools/kgit;beginline=5;endline=9;md5=a6c2fa8aef1b DEPENDS = "git-native" -SRCREV = "bd9e1d6c9b0a34ff3e19a06999aaf57ffadfd04c" +SRCREV = "626ceac135fa66277c2fa53197be33cc9d4d7614" PR = "r12" PV = "0.2+git${SRCPV}" -- 2.5.0 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-07 16:18 ` Bruce Ashfield @ 2016-12-07 18:05 ` Trevor Woerner 2016-12-07 23:50 ` Trevor Woerner 0 siblings, 1 reply; 15+ messages in thread From: Trevor Woerner @ 2016-12-07 18:05 UTC (permalink / raw) To: Bruce Ashfield; +Cc: Patches and discussions about the oe-core layer On Wed, Dec 7, 2016 at 11:18 AM, Bruce Ashfield <bruce.ashfield@windriver.com> wrote: > With the attached patch, I see nothing else that is named in /tmp/ > > If you have the cycles, can you give it a try and let me know ? Yes, I'm giving it a whirl right now. Thanks! ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-07 18:05 ` Trevor Woerner @ 2016-12-07 23:50 ` Trevor Woerner 2016-12-08 3:23 ` Bruce Ashfield 2016-12-08 15:27 ` Bruce Ashfield 0 siblings, 2 replies; 15+ messages in thread From: Trevor Woerner @ 2016-12-07 23:50 UTC (permalink / raw) To: Bruce Ashfield; +Cc: Patches and discussions about the oe-core layer On Wed, Dec 7, 2016 at 1:05 PM, Trevor Woerner <twoerner@gmail.com> wrote: > On Wed, Dec 7, 2016 at 11:18 AM, Bruce Ashfield > <bruce.ashfield@windriver.com> wrote: >> With the attached patch, I see nothing else that is named in /tmp/ >> >> If you have the cycles, can you give it a try and let me know ? > > Yes, I'm giving it a whirl right now. Thanks! That patch looks good, how soon can it land? ;-) I'm sure there's still something in the build that is creating temporary files in /tmp. I'm not sure which process is doing it, I have no reason to suspect it's the kernel tools (in fact I think it happens too early in the build to be the kernel tools), and the tmp files that are created are named with temporary names (e.g. /tmp/tmp.2wavbhTlDU) so they shouldn't interfere with multiple users building on the same machine. Each build appears to create two such temp files that aren't cleaned up at the end of the build. Not a big deal. I don't think it's something new that was introduced recently (but I'd have to do more investigation to verify). ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-07 23:50 ` Trevor Woerner @ 2016-12-08 3:23 ` Bruce Ashfield 2016-12-08 15:27 ` Bruce Ashfield 1 sibling, 0 replies; 15+ messages in thread From: Bruce Ashfield @ 2016-12-08 3:23 UTC (permalink / raw) To: Trevor Woerner; +Cc: Patches and discussions about the oe-core layer On 2016-12-07 6:50 PM, Trevor Woerner wrote: > On Wed, Dec 7, 2016 at 1:05 PM, Trevor Woerner <twoerner@gmail.com> wrote: >> On Wed, Dec 7, 2016 at 11:18 AM, Bruce Ashfield >> <bruce.ashfield@windriver.com> wrote: >>> With the attached patch, I see nothing else that is named in /tmp/ >>> >>> If you have the cycles, can you give it a try and let me know ? >> >> Yes, I'm giving it a whirl right now. Thanks! > > That patch looks good, how soon can it land? ;-) For my part, I'll send it first thing tomorrow along with some version bumps to the kernel. > > I'm sure there's still something in the build that is creating > temporary files in /tmp. I'm not sure which process is doing it, I > have no reason to suspect it's the kernel tools (in fact I think it > happens too early in the build to be the kernel tools), and the tmp > files that are created are named with temporary names (e.g. > /tmp/tmp.2wavbhTlDU) so they shouldn't interfere with multiple users > building on the same machine. Each build appears to create two such > temp files that aren't cleaned up at the end of the build. Not a big > deal. I don't think it's something new that was introduced recently > (but I'd have to do more investigation to verify). I noticed one mktemp file leaking when I was fixing the bug at hand today. I made a note to loop back and have a look for where an early exit is skipping clean up. If it is the tools, I'll fix it along with the patch I did today. Bruce > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] kern-tools: fix processing for no branch meta-data 2016-12-07 23:50 ` Trevor Woerner 2016-12-08 3:23 ` Bruce Ashfield @ 2016-12-08 15:27 ` Bruce Ashfield 1 sibling, 0 replies; 15+ messages in thread From: Bruce Ashfield @ 2016-12-08 15:27 UTC (permalink / raw) To: Trevor Woerner; +Cc: Patches and discussions about the oe-core layer On 2016-12-07 06:50 PM, Trevor Woerner wrote: > On Wed, Dec 7, 2016 at 1:05 PM, Trevor Woerner <twoerner@gmail.com> wrote: >> On Wed, Dec 7, 2016 at 11:18 AM, Bruce Ashfield >> <bruce.ashfield@windriver.com> wrote: >>> With the attached patch, I see nothing else that is named in /tmp/ >>> >>> If you have the cycles, can you give it a try and let me know ? >> >> Yes, I'm giving it a whirl right now. Thanks! > > That patch looks good, how soon can it land? ;-) > > I'm sure there's still something in the build that is creating > temporary files in /tmp. I'm not sure which process is doing it, I > have no reason to suspect it's the kernel tools (in fact I think it > happens too early in the build to be the kernel tools), and the tmp > files that are created are named with temporary names (e.g. > /tmp/tmp.2wavbhTlDU) so they shouldn't interfere with multiple users > building on the same machine. Each build appears to create two such > temp files that aren't cleaned up at the end of the build. Not a big > deal. I don't think it's something new that was introduced recently > (but I'd have to do more investigation to verify). I did find one mktemp file that wasn't being cleaned up. I fixed it and updated my commit to include that fix. It'll all be out later today with my kernel version bumps + CVE series. Bruce > ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 4/4] kernel-yocto: exit on non-zero return code 2016-12-02 21:09 [PATCH 0/4] kernel-yocto: consolidated pull request Bruce Ashfield ` (2 preceding siblings ...) 2016-12-02 21:09 ` [PATCH 3/4] kern-tools: fix processing for no branch meta-data Bruce Ashfield @ 2016-12-02 21:09 ` Bruce Ashfield 2016-12-06 19:32 ` [PATCH 0/4] kernel-yocto: consolidated pull request Trevor Woerner 4 siblings, 0 replies; 15+ messages in thread From: Bruce Ashfield @ 2016-12-02 21:09 UTC (permalink / raw) To: richard.purdie; +Cc: openembedded-core Historically the processing of kernel meta data contained some commands that exited with a non-zero return code. Special processing was required to properly deal with their exit. That is no longer true, and instead of handling all return codes and doing an explicit 'exit' call, we can remove set -e from the routine and have all errors be trapped and stop processing. Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> --- meta/classes/kernel-yocto.bbclass | 1 - 1 file changed, 1 deletion(-) diff --git a/meta/classes/kernel-yocto.bbclass b/meta/classes/kernel-yocto.bbclass index 6160a29ecb76..7c32c9664610 100644 --- a/meta/classes/kernel-yocto.bbclass +++ b/meta/classes/kernel-yocto.bbclass @@ -61,7 +61,6 @@ def get_machine_branch(d, default): return default do_kernel_metadata() { - set +e cd ${S} export KMETA=${KMETA} -- 2.5.0 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 0/4] kernel-yocto: consolidated pull request 2016-12-02 21:09 [PATCH 0/4] kernel-yocto: consolidated pull request Bruce Ashfield ` (3 preceding siblings ...) 2016-12-02 21:09 ` [PATCH 4/4] kernel-yocto: exit on non-zero return code Bruce Ashfield @ 2016-12-06 19:32 ` Trevor Woerner 4 siblings, 0 replies; 15+ messages in thread From: Trevor Woerner @ 2016-12-06 19:32 UTC (permalink / raw) To: Bruce Ashfield; +Cc: openembedded-core On Fri 2016-12-02 @ 04:09:21 PM, Bruce Ashfield wrote: > This pull request is mainly to fix a couple of bugs that were reported > on the mailing list recently, but it also includes some kernel version > updates that I *think* I sent previously. Thanks Bruce, I've tested these and they look good. ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2016-12-08 15:27 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-12-02 21:09 [PATCH 0/4] kernel-yocto: consolidated pull request Bruce Ashfield 2016-12-02 21:09 ` [PATCH 1/4] linux-yocto/4.8: update to v4.8.10 Bruce Ashfield 2016-12-02 21:09 ` [PATCH 2/4] linux-yocto-dev: update to 4.9-rcX Bruce Ashfield 2016-12-02 21:09 ` [PATCH 3/4] kern-tools: fix processing for no branch meta-data Bruce Ashfield 2016-12-06 21:14 ` Patrick Ohly 2016-12-06 23:02 ` Paul Barker 2016-12-07 15:11 ` Trevor Woerner 2016-12-07 15:14 ` Bruce Ashfield 2016-12-07 16:18 ` Bruce Ashfield 2016-12-07 18:05 ` Trevor Woerner 2016-12-07 23:50 ` Trevor Woerner 2016-12-08 3:23 ` Bruce Ashfield 2016-12-08 15:27 ` Bruce Ashfield 2016-12-02 21:09 ` [PATCH 4/4] kernel-yocto: exit on non-zero return code Bruce Ashfield 2016-12-06 19:32 ` [PATCH 0/4] kernel-yocto: consolidated pull request Trevor Woerner
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.