4.14.90-stable review
Subsystems (4)
Greg Kroah-Hartman
maintainercover 4.14.90-stable review
This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gz
or in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.14.y
and the diffstat can be found below.
thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:
Greg Kroah-Hartman <[email protected]>
Linux 4.14.90-rc1
Nicolas Schichan <[email protected]>
bpf, arm: fix emit_ldx_r and emit_mov_i using TMP_REG_1
Trent Piepho <[email protected]>
rtc: snvs: Add timeouts to avoid kernel lockups
Israel Rukshin <[email protected]>
nvmet-rdma: fix response use after free
Masahiro Yamada <[email protected]>
i2c: uniphier-f: fix violation of tLOW requirement for Fast-mode
Masahiro Yamada <[email protected]>
i2c: uniphier: fix violation of tLOW requirement for Fast-mode
Hans de Goede <[email protected]>
i2c: scmi: Fix probe error on devices with an empty SMB0001 ACPI device node
Adamski, Krzysztof (Nokia - PL/Wroclaw) <[email protected]>
i2c: axxia: properly handle master timeout
Ido Schimmel <[email protected]>
mlxsw: spectrum_switchdev: Fix VLAN device deletion via ioctl
Stefan Hajnoczi <[email protected]>
vhost/vsock: fix reset orphans race with close timeout
Steve French <[email protected]>
cifs: In Kconfig CONFIG_CIFS_POSIX needs depends on legacy (insecure cifs)
Sam Bobroff <[email protected]>
drm/ast: Fix connector leak during driver unload
Nicolas Saenz Julienne <[email protected]>
ethernet: fman: fix wrong of_node_put() in probe function
Vladimir Murzin <[email protected]>
ARM: 8815/1: V7M: align v7m_dma_inv_range() with v7 counterpart
Chris Cole <[email protected]>
ARM: 8814/1: mm: improve/fix ARM v7_dma_inv_range() unaligned address handling
Saeed Mahameed <[email protected]>
net/mlx4_en: Fix build break when CONFIG_INET is off
Anderson Luiz Alves <[email protected]>
mv88e6060: disable hardware level MAC learning
Juha-Matti Tilli <[email protected]>
libata: whitelist all SAMSUNG MZ7KM* solid-state disks
Tony Lindgren <[email protected]>
Input: omap-keypad - fix keyboard debounce configuration
Teika Kazura <[email protected]>
Input: synaptics - enable SMBus for HP 15-ay000
Dan Carpenter <[email protected]>
clk: mmp: Off by one in mmp_clk_add()
Dan Carpenter <[email protected]>
clk: mvebu: Off by one bugs in cp110_of_clk_get()
Wen Yang <[email protected]>
drm/msm: Fix error return checking
Yangtao Li <[email protected]>
ide: pmac: add of_node_put()
Yangtao Li <[email protected]>
drivers/tty: add missing of_node_put()
Yangtao Li <[email protected]>
drivers/sbus/char: add of_node_put()
Yangtao Li <[email protected]>
sbus: char: add of_node_put()
Trond Myklebust <[email protected]>
SUNRPC: Fix a potential race in xprt_connect()
Dave Kleikamp <[email protected]>
nfs: don't dirty kernel pages read by direct-io
David Miller <[email protected]>
bpf: Fix verifier log string check for bad alignment.
Toni Peltonen <[email protected]>
bonding: fix 802.3ad state sent to partner when unbinding slave
Jose Abreu <[email protected]>
ARC: io.h: Implement reads{x}()/writes{x}()Sean Paul <[email protected]>
drm/msm: Grab a vblank reference when waiting for commit_done
YiFei Zhu <[email protected]>
x86/earlyprintk/efi: Fix infinite loop on some screen widths
Cathy Avery <[email protected]>
scsi: vmw_pscsi: Rearrange code to avoid multiple calls to free_irq during unload
Fred Herard <[email protected]>
scsi: libiscsi: Fix NULL pointer dereference in iscsi_eh_session_reset
Vitaly Kuznetsov <[email protected]>
Input: hyper-v - fix wakeup from suspend-to-idle
Alexey Khoroshilov <[email protected]>
mac80211_hwsim: fix module init error paths for netlink
Steven Rostedt (VMware) <[email protected]>
locking/qspinlock: Fix build for anonymous union in older GCC compilers
Michael J. Ruhl <[email protected]>
IB/hfi1: Remove race conditions in user_sdma send path
Ilan Peer <[email protected]>
mac80211: Fix condition validating WMM IE
Emmanuel Grumbach <[email protected]>
mac80211: don't WARN on bad WMM parameters from buggy APs
Jozsef Kadlecsik <[email protected]>
netfilter: ipset: Fix wraparound in hash:*net* types
Jens Axboe <[email protected]>
elevator: lookup mq vs non-mq elevators
Peter Zijlstra <[email protected]>
locking/qspinlock, x86: Provide liveness guarantee
Will Deacon <[email protected]>
locking/qspinlock/x86: Increase _Q_PENDING_LOOPS upper bound
Peter Zijlstra <[email protected]>
locking/qspinlock: Re-order code
Will Deacon <[email protected]>
locking/qspinlock: Kill cmpxchg() loop when claiming lock from head of queue
Will Deacon <[email protected]>
locking/qspinlock: Remove duplicate clear_pending() function from PV code
Will Deacon <[email protected]>
locking/qspinlock: Remove unbounded cmpxchg() loop from locking slowpath
Will Deacon <[email protected]>
locking/qspinlock: Merge 'struct __qspinlock' into 'struct qspinlock'
Will Deacon <[email protected]>
locking/qspinlock: Bound spinning on pending->locked transition in slowpath
Will Deacon <[email protected]>
locking/qspinlock: Ensure node is initialised before updating prev->next
Paul E. McKenney <[email protected]>
locking: Remove smp_read_barrier_depends() from queued_spin_lock_slowpath()
Masahiro Yamada <[email protected]>
x86/build: Fix compiler support check for CONFIG_RETPOLINE
Junwei Zhang <[email protected]>
drm/amdgpu: update SMC firmware image for polaris10 variants
Chris Wilson <[email protected]>
drm/i915/execlists: Apply a full mb before execution for Braswell
Brian Norris <[email protected]>
Revert "drm/rockchip: Allow driver to be shutdown on reboot/kexec"
Lyude Paul <[email protected]>
drm/nouveau/kms: Fix memory leak in nv50_mstm_del()
Radu Rendec <[email protected]>
powerpc/msi: Fix NULL pointer access in teardown code
Steven Rostedt (VMware) <[email protected]>
tracing: Fix memory leak of instance function hash filters
Steven Rostedt (VMware) <[email protected]>
tracing: Fix memory leak in set_trigger_filter()
Mike Snitzer <[email protected]>
dm cache metadata: verify cache has blocks in blocks_are_clean_separate_dirty()
Mike Snitzer <[email protected]>
dm thin: send event about thin-pool state change _after_ making it
Lubomir Rintel <[email protected]>
ARM: mmp/mmp2: fix cpu_is_mmp2() on mmp2-dt
Chad Austin <[email protected]>
fuse: continue to send FUSE_RELEASEDIR when FUSE_OPEN returns ENOSYS
Alek Du <[email protected]>
mmc: sdhci: fix the timeout check window for clock and reset
Aaro Koskinen <[email protected]>
MMC: OMAP: fix broken MMC on OMAP15XX/OMAP5910/OMAP310
Robin Murphy <[email protected]>
arm64: dma-mapping: Fix FORCE_CONTIGUOUS buffer clearing
Andrea Arcangeli <[email protected]>
userfaultfd: check VM_MAYWRITE was set after verifying the uffd is registered
Jeff Moyer <[email protected]>
aio: fix spectre gadget in lookup_ioctx
Chen-Yu Tsai <[email protected]>
pinctrl: sunxi: a83t: Fix IRQ offset typo for PH11
Ingo Molnar <[email protected]>
timer/debug: Change /proc/timer_list from 0444 to 0400
-------------
Diffstat:
Makefile | 4 +- arch/arc/include/asm/io.h | 72 ++++++++ arch/arm/mach-mmp/cputype.h | 6 +- arch/arm/mm/cache-v7.S | 8 +- arch/arm/mm/cache-v7m.S | 14 +- arch/arm/net/bpf_jit_32.c | 2 +- arch/arm64/mm/dma-mapping.c | 2 +- arch/powerpc/kernel/msi.c | 7 +- arch/x86/Makefile | 10 +- arch/x86/include/asm/qspinlock.h | 25 ++- arch/x86/include/asm/qspinlock_paravirt.h | 3 +- arch/x86/platform/efi/early_printk.c | 2 +- block/elevator.c | 44 +++-- drivers/ata/libata-core.c | 1 + drivers/clk/mmp/clk.c | 2 +- drivers/clk/mvebu/cp110-system-controller.c | 4 +- drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 3 +- drivers/gpu/drm/ast/ast_fb.c | 1 + drivers/gpu/drm/i915/intel_lrc.c | 7 +- drivers/gpu/drm/msm/msm_atomic.c | 5 + drivers/gpu/drm/msm/msm_iommu.c | 2 +- drivers/gpu/drm/nouveau/nv50_display.c | 1 + drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 6 - drivers/hid/hid-hyperv.c | 2 +- drivers/i2c/busses/i2c-axxia.c | 40 +++-- drivers/i2c/busses/i2c-scmi.c | 10 +- drivers/i2c/busses/i2c-uniphier-f.c | 19 +- drivers/i2c/busses/i2c-uniphier.c | 8 +- drivers/ide/pmac.c | 1 + drivers/infiniband/hw/hfi1/user_sdma.c | 24 ++- drivers/infiniband/hw/hfi1/user_sdma.h | 9 +- drivers/input/keyboard/omap4-keypad.c | 18 +- drivers/input/mouse/synaptics.c | 1 + drivers/input/serio/hyperv-keyboard.c | 2 +- drivers/md/dm-cache-metadata.c | 4 + drivers/md/dm-thin.c | 68 +++---- drivers/mmc/host/omap.c | 11 +- drivers/mmc/host/sdhci.c | 18 +- drivers/net/bonding/bond_3ad.c | 3 + drivers/net/dsa/mv88e6060.c | 10 +- drivers/net/ethernet/freescale/fman/fman.c | 5 +- drivers/net/ethernet/mellanox/mlx4/Kconfig | 2 +- .../ethernet/mellanox/mlxsw/spectrum_switchdev.c | 10 +- drivers/net/wireless/mac80211_hwsim.c | 12 +- drivers/nvme/target/rdma.c | 3 +- drivers/pinctrl/sunxi/pinctrl-sun8i-a83t.c | 2 +- drivers/rtc/rtc-snvs.c | 105 +++++++---- drivers/sbus/char/display7seg.c | 1 + drivers/sbus/char/envctrl.c | 2 + drivers/scsi/libiscsi.c | 4 +- drivers/scsi/vmw_pvscsi.c | 4 +- drivers/tty/serial/suncore.c | 1 + drivers/vhost/vsock.c | 22 ++- fs/aio.c | 2 + fs/cifs/Kconfig | 2 +- fs/fuse/dir.c | 2 +- fs/fuse/file.c | 21 +-- fs/fuse/fuse_i.h | 2 +- fs/nfs/direct.c | 9 +- fs/userfaultfd.c | 3 +- include/asm-generic/qspinlock_types.h | 32 +++- kernel/locking/qspinlock.c | 195 ++++++++++----------- kernel/locking/qspinlock_paravirt.h | 42 ++--- kernel/time/timer_list.c | 2 +- kernel/trace/ftrace.c | 1 + kernel/trace/trace_events_trigger.c | 6 +- net/mac80211/mlme.c | 3 +- net/netfilter/ipset/ip_set_hash_ipportnet.c | 26 +-- net/netfilter/ipset/ip_set_hash_net.c | 9 +- net/netfilter/ipset/ip_set_hash_netiface.c | 9 +- net/netfilter/ipset/ip_set_hash_netnet.c | 28 +-- net/netfilter/ipset/ip_set_hash_netport.c | 19 +- net/netfilter/ipset/ip_set_hash_netportnet.c | 35 ++-- net/sunrpc/xprt.c | 11 +- tools/testing/selftests/bpf/test_verifier.c | 2 +- 75 files changed, 692 insertions(+), 421 deletions(-)
Greg Kroah-Hartman
maintainerPATCH 1/72 timer/debug: Change /proc/timer_list from 0444 to 0400
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 8e7df2b5b7f245c9bd11064712db5cb69044a362 ]
While it uses %pK, there's still few reasons to read this file as non-root.
--- kernel/time/timer_list.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
kernel/time/timer_list.c +1 -1
@@ -389,7 +389,7 @@ static int __init init_timer_list_procfs {struct proc_dir_entry *pe; - pe =proc_create ("timer_list" ,0444 ,NULL , &timer_list_fops); + pe =proc_create ("timer_list" ,0400 ,NULL , &timer_list_fops);if (!pe)return -ENOMEM ;return 0 ;
Greg Kroah-Hartman
maintainerPATCH 2/72 pinctrl: sunxi: a83t: Fix IRQ offset typo for PH11
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Chen-Yu Tsai <[email protected]>
commit 478b6767ad26ab86d9ecc341027dd09a87b1f997 upstream.
Pin PH11 is used on various A83T board to detect a change in the OTG port's ID pin, as in when an OTG host cable is plugged in.
The incorrect offset meant the gpiochip/irqchip was activating the wrong pin for interrupts.
--- drivers/pinctrl/sunxi/pinctrl-sun8i-a83t.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pinctrl/sunxi/pinctrl-sun8i-a83t.c +1 -1
@@ -568,7 +568,7 @@ static const struct sunxi_desc_pin sun8iSUNXI_PIN (SUNXI_PINCTRL_PIN (H ,11 ),SUNXI_FUNCTION (0x0 ,"gpio_in" ),SUNXI_FUNCTION (0x1 ,"gpio_out" ), - SUNXI_FUNCTION_IRQ_BANK(0x6, 2, 1)), /* PH_EINT11 */ + SUNXI_FUNCTION_IRQ_BANK(0x6, 2, 11)), /* PH_EINT11 */ };static const struct sunxi_pinctrl_desc sun8i_a83t_pinctrl_data = {
Greg Kroah-Hartman
maintainerPATCH 3/72 aio: fix spectre gadget in lookup_ioctx
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jeff Moyer <[email protected]>
commit a538e3ff9dabcdf6c3f477a373c629213d1c3066 upstream.
Matthew pointed out that the ioctx_table is susceptible to spectre v1, because the index can be controlled by an attacker. The below patch should mitigate the attack for all of the aio system calls.
--- fs/aio.c | 2 ++ 1 file changed, 2 insertions(+)
fs/aio.c +2 -0
@@ -43,6 +43,7 @@#include <asm/kmap_types.h> #include <linux/uaccess.h> +#include <linux/nospec.h> #include "internal.h"
@@ -1084,6 +1085,7 @@ static struct kioctx *lookup_ioctx(unsigif (!table || id >= table->nr) goto out; + id =array_index_nospec (id, table->nr); ctx =rcu_dereference (table->table[id]);if (ctx && ctx->user_id == ctx_id) {if (percpu_ref_tryget_live (&ctx->users))
Greg Kroah-Hartman
maintainerPATCH 4/72 userfaultfd: check VM_MAYWRITE was set after verifying the uffd is registered
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andrea Arcangeli <[email protected]>
commit 01e881f5a1fca4677e82733061868c6d6ea05ca7 upstream.
Calling UFFDIO_UNREGISTER on virtual ranges not yet registered in uffd could trigger an harmless false positive WARN_ON. Check the vma is already registered before checking VM_MAYWRITE to shut off the false positive warning.
--- fs/userfaultfd.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
fs/userfaultfd.c +2 -1
@@ -1567,7 +1567,6 @@ static int userfaultfd_unregister(structcond_resched ();BUG_ON (!vma_can_userfault (vma)); -WARN_ON (!(vma->vm_flags &VM_MAYWRITE ));/* * Nothing to do: this vma is already registered into this
@@ -1576,6 +1575,8 @@ static int userfaultfd_unregister(structif (!vma->vm_userfaultfd_ctx.ctx) goto skip; +WARN_ON (!(vma->vm_flags &VM_MAYWRITE )); +if (vma->vm_start > start) start = vma->vm_start; vma_end =min (end, vma->vm_end);
Greg Kroah-Hartman
maintainerPATCH 5/72 arm64: dma-mapping: Fix FORCE_CONTIGUOUS buffer clearing
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Robin Murphy <[email protected]>
commit 3238c359acee4ab57f15abb5a82b8ab38a661ee7 upstream.
We need to invalidate the caches *before* clearing the buffer via the non-cacheable alias, else in the worst case __dma_flush_area() may write back dirty lines over the top of our nice new zeros.
--- arch/arm64/mm/dma-mapping.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm64/mm/dma-mapping.c +1 -1
@@ -633,9 +633,9 @@ static void *__iommu_alloc_attrs(struct prot, __builtin_return_address(0));if (addr) { -memset (addr,0 , size);if (!coherent)__dma_flush_area (page_to_virt (page), iosize); +memset (addr,0 , size); }else {iommu_dma_unmap_page (dev, *handle, iosize,0 , attrs); dma_release_from_contiguous(dev, page,
Greg Kroah-Hartman
maintainerPATCH 6/72 MMC: OMAP: fix broken MMC on OMAP15XX/OMAP5910/OMAP310
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Aaro Koskinen <[email protected]>
commit e8cde625bfe8a714a856e1366bcbb259d7346095 upstream.
Since v2.6.22 or so there has been reports [1] about OMAP MMC being broken on OMAP15XX based hardware (OMAP5910 and OMAP310). The breakage seems to have been caused by commit 46a6730e3ff9 ("mmc-omap: Fix omap to use MMC_POWER_ON") that changed clock enabling to be done on MMC_POWER_ON. This can happen multiple times in a row, and on 15XX the hardware doesn't seem to like it and the MMC just stops responding. Fix by memorizing the power mode and do the init only when necessary.
Before the patch (on Palm TE):
mmc0: new SD card at address b368
mmcblk0: mmc0:b368 SDC 977 MiB
mmci-omap mmci-omap.0: command timeout (CMD18) mmci-omap mmci-omap.0: command timeout (CMD13) mmci-omap mmci-omap.0: command timeout (CMD13) mmci-omap mmci-omap.0: command timeout (CMD12) [x 6] mmci-omap mmci-omap.0: command timeout (CMD13) [x 6]
mmcblk0: error -110 requesting status
mmci-omap mmci-omap.0: command timeout (CMD8) mmci-omap mmci-omap.0: command timeout (CMD18) mmci-omap mmci-omap.0: command timeout (CMD13) mmci-omap mmci-omap.0: command timeout (CMD13) mmci-omap mmci-omap.0: command timeout (CMD12) [x 6] mmci-omap mmci-omap.0: command timeout (CMD13) [x 6]
mmcblk0: error -110 requesting status
mmcblk0: recovery failed!
print_req_error: I/O error, dev mmcblk0, sector 0 Buffer I/O error on dev mmcblk0, logical block 0, async page read
mmcblk0: unable to read partition table
After the patch:
mmc0: new SD card at address b368 mmcblk0: mmc0:b368 SDC 977 MiB mmcblk0: p1
The patch is based on a fix and analysis done by Ladislav Michl.
Tested on OMAP15XX/OMAP310 (Palm TE), OMAP1710 (Nokia 770) and OMAP2420 (Nokia N810).
[1] https://marc.info/?t=123175197000003&r=1&w=2
--- drivers/mmc/host/omap.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-)
drivers/mmc/host/omap.c +9 -2
@@ -104,6 +104,7 @@ struct mmc_omap_slot {unsigned int vdd;u16 saved_con;u16 bus_mode; +u16 power_mode;unsigned int fclk_freq;struct tasklet_struct cover_tasklet;
@@ -1157,7 +1158,7 @@ static void mmc_omap_set_ios(struct mmc_struct mmc_omap_slot *slot =mmc_priv (mmc);struct mmc_omap_host *host = slot->host;int i, dsor; -int clk_enabled; +int clk_enabled, init_stream;mmc_omap_select_slot (slot,0 );
@@ -1167,6 +1168,7 @@ static void mmc_omap_set_ios(struct mmc_ slot->vdd = ios->vdd; clk_enabled =0 ; + init_stream =0 ;switch (ios->power_mode) {case MMC_POWER_OFF :mmc_omap_set_power (slot,0 , ios->vdd);
@@ -1174,13 +1176,17 @@ static void mmc_omap_set_ios(struct mmc_case MMC_POWER_UP :/* Cannot touch dsor yet, just power up MMC */ mmc_omap_set_power (slot,1 , ios->vdd); + slot->power_mode = ios->power_mode; goto exit;case MMC_POWER_ON :mmc_omap_fclk_enable (host,1 ); clk_enabled =1 ; dsor |=1 <<11 ; +if (slot->power_mode !=MMC_POWER_ON ) + init_stream =1 ;break ; } + slot->power_mode = ios->power_mode;if (slot->bus_mode != ios->bus_mode) {if (slot->pdata->set_bus_mode !=NULL )
@@ -1196,7 +1202,7 @@ static void mmc_omap_set_ios(struct mmc_for (i =0 ; i <2 ; i++)OMAP_MMC_WRITE (host,CON , dsor); slot->saved_con = dsor; -if (ios->power_mode ==MMC_POWER_ON ) { +if (init_stream) {/* worst case at 400kHz, 80 cycles makes 200 microsecs */ int usecs =250 ;
@@ -1234,6 +1240,7 @@ static int mmc_omap_new_slot(struct mmc_ slot->host = host; slot->mmc = mmc; slot->id = id; + slot->power_mode =MMC_POWER_UNDEFINED ; slot->pdata = &host->pdata->slots[id]; host->slots[id] = slot;
Greg Kroah-Hartman
maintainerPATCH 7/72 mmc: sdhci: fix the timeout check window for clock and reset
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Alek Du <[email protected]>
commit b704441e38f645dcfba1348ca3cc1ba43d1a9f31 upstream.
We observed some premature timeouts on a virtualization platform, the log is like this:
case 1:
[159525.255629] mmc1: Internal clock never stabilised. [159525.255818] mmc1: sdhci: ============ SDHCI REGISTER DUMP =========== [159525.256049] mmc1: sdhci: Sys addr: 0x00000000 | Version: 0x00001002 ... [159525.257205] mmc1: sdhci: Wake-up: 0x00000000 | Clock: 0x0000fa03
Quoted 1 line
Quoted 1 line
From the clock control register dump, we are pretty sure the clock was
stablized.
case 2:
[ 914.550127] mmc1: Reset 0x2 never completed. [ 914.550321] mmc1: sdhci: ============ SDHCI REGISTER DUMP =========== [ 914.550608] mmc1: sdhci: Sys addr: 0x00000010 | Version: 0x00001002
After checking the sdhci code, we found the timeout check actually has a little window that the CPU can be scheduled out and when it comes back, the original time set or check is not valid.
--- drivers/mmc/host/sdhci.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-)
drivers/mmc/host/sdhci.c +13 -5
@@ -193,8 +193,12 @@ void sdhci_reset(struct sdhci_host *host timeout =ktime_add_ms (ktime_get (),100 );/* hw clears the bit when it's done */ -while (sdhci_readb (host,SDHCI_SOFTWARE_RESET ) & mask) { -if (ktime_after (ktime_get (), timeout)) { +while (1 ) { +bool timedout =ktime_after (ktime_get (), timeout); + +if (!(sdhci_readb (host,SDHCI_SOFTWARE_RESET ) & mask)) +break ; +if (timedout) {pr_err ("%s: Reset 0x%x never completed.\n" ,mmc_hostname (host->mmc), (int )mask);sdhci_dumpregs (host);
@@ -1417,9 +1421,13 @@ void sdhci_enable_clk(struct sdhci_host/* Wait max 20 ms */ timeout =ktime_add_ms (ktime_get (),20 ); -while (!((clk =sdhci_readw (host,SDHCI_CLOCK_CONTROL )) - &SDHCI_CLOCK_INT_STABLE )) { -if (ktime_after (ktime_get (), timeout)) { +while (1 ) { +bool timedout =ktime_after (ktime_get (), timeout); + + clk =sdhci_readw (host,SDHCI_CLOCK_CONTROL ); +if (clk &SDHCI_CLOCK_INT_STABLE ) +break ; +if (timedout) {pr_err ("%s: Internal clock never stabilised.\n" ,mmc_hostname (host->mmc));sdhci_dumpregs (host);
Greg Kroah-Hartman
maintainerPATCH 8/72 fuse: continue to send FUSE_RELEASEDIR when FUSE_OPEN returns ENOSYS
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Chad Austin <[email protected]>
commit 2e64ff154ce6ce9a8dc0f9556463916efa6ff460 upstream.
When FUSE_OPEN returns ENOSYS, the no_open bit is set on the connection.
Because the FUSE_RELEASE and FUSE_RELEASEDIR paths share code, this incorrectly caused the FUSE_RELEASEDIR request to be dropped and never sent to userspace.
Pass an isdir bool to distinguish between FUSE_RELEASE and FUSE_RELEASEDIR inside of fuse_file_put.
--- fs/fuse/dir.c | 2 +- fs/fuse/file.c | 21 +++++++++++---------- fs/fuse/fuse_i.h | 2 +- 3 files changed, 13 insertions(+), 12 deletions(-)
fs/fuse/dir.c +1 -1
@@ -1424,7 +1424,7 @@ static int fuse_dir_open(struct inode *istatic int fuse_dir_release (struct inode *inode,struct file *file) { -fuse_release_common (file,FUSE_RELEASEDIR ); +fuse_release_common (file, true);return 0 ; }
fs/fuse/file.c +11 -10
@@ -86,12 +86,12 @@ static void fuse_release_end(struct fuseiput (req->misc.release.inode); } -static void fuse_file_put (struct fuse_file *ff,bool sync) +static void fuse_file_put (struct fuse_file *ff,bool sync,bool isdir) {if (refcount_dec_and_test (&ff->count)) {struct fuse_req *req = ff->reserved_req; -if (ff->fc->no_open) { +if (ff->fc->no_open && !isdir) {/* * Drop the release request when client does not * implement 'open'
@@ -244,10 +244,11 @@ static void fuse_prepare_release(struct req->in.args[0 ].value = inarg; } -void fuse_release_common (struct file *file,int opcode) +void fuse_release_common (struct file *file,bool isdir) {struct fuse_file *ff = file->private_data;struct fuse_req *req = ff->reserved_req; +int opcode = isdir ?FUSE_RELEASEDIR :FUSE_RELEASE ;fuse_prepare_release (ff, file->f_flags, opcode);
@@ -269,7 +270,7 @@ void fuse_release_common(struct file *fi* synchronous RELEASE is allowed (and desirable) in this case * because the server can be trusted not to screw up. */ -fuse_file_put (ff, ff->fc->destroy_req !=NULL ); +fuse_file_put (ff, ff->fc->destroy_req !=NULL , isdir); }static int fuse_open (struct inode *inode,struct file *file)
@@ -285,7 +286,7 @@ static int fuse_release(struct inode *inif (fc->writeback_cache)write_inode_now (inode,1 ); -fuse_release_common (file,FUSE_RELEASE ); +fuse_release_common (file, false);/* return value is ignored by VFS */ return 0 ;
@@ -299,7 +300,7 @@ void fuse_sync_release(struct fuse_file* iput(NULL) is a no-op and since the refcount is 1 and everything's * synchronous, we are fine with not doing igrab() here" */ -fuse_file_put (ff, true); +fuse_file_put (ff, true, false); }EXPORT_SYMBOL_GPL (fuse_sync_release);
@@ -804,7 +805,7 @@ static void fuse_readpages_end(struct fuput_page (page); }if (req->ff) -fuse_file_put (req->ff, false); +fuse_file_put (req->ff, false, false); }static void fuse_send_readpages (struct fuse_req *req,struct file *file)
@@ -1458,7 +1459,7 @@ static void fuse_writepage_free(struct f__free_page (req->pages[i]);if (req->ff) -fuse_file_put (req->ff, false); +fuse_file_put (req->ff, false, false); }static void fuse_writepage_finish (struct fuse_conn *fc,struct fuse_req *req)
@@ -1615,7 +1616,7 @@ int fuse_write_inode(struct inode *inode ff =__fuse_write_file_get (fc, fi); err =fuse_flush_times (inode, ff);if (ff) -fuse_file_put (ff,0 ); +fuse_file_put (ff, false, false);return err; }
@@ -1929,7 +1930,7 @@ static int fuse_writepages(struct addres err =0 ; }if (data.ff) -fuse_file_put (data.ff, false); +fuse_file_put (data.ff, false, false);kfree (data.orig_pages); out:
fs/fuse/fuse_i.h +1 -1
@@ -739,7 +739,7 @@ void fuse_sync_release(struct fuse_file/** * Send RELEASE or RELEASEDIR request */ -void fuse_release_common (struct file *file,int opcode); +void fuse_release_common (struct file *file,bool isdir);/** * Send FSYNC or FSYNCDIR request
Greg Kroah-Hartman
maintainerPATCH 9/72 ARM: mmp/mmp2: fix cpu_is_mmp2() on mmp2-dt
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Lubomir Rintel <[email protected]>
commit 76f4e2c3b6a560cdd7a75b87df543e04d05a9e5f upstream.
cpu_is_mmp2() was equivalent to cpu_is_pj4(), wouldn't be correct for multiplatform kernels. Fix it by also considering mmp_chip_id, as is done for cpu_is_pxa168() and cpu_is_pxa910() above.
Moreover, it is only available with CONFIG_CPU_MMP2 and thus doesn't work on DT-based MMP2 machines. Enable it on CONFIG_MACH_MMP2_DT too.
Note: CONFIG_CPU_MMP2 is only used for machines that use board files instead of DT. It should perhaps be renamed. I'm not doing it now, because I don't have a better idea.
--- arch/arm/mach-mmp/cputype.h | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
arch/arm/mach-mmp/cputype.h +4 -2
@@ -44,10 +44,12 @@ static inline int cpu_is_pxa910(void)#define cpu_is_pxa910 () (0)#endif -#ifdef CONFIG_CPU_MMP2 +#if defined(CONFIG_CPU_MMP2 ) || defined(CONFIG_MACH_MMP2_DT )static inline int cpu_is_mmp2 (void ) { -return (((read_cpuid_id () >>8 ) &0xff ) ==0x58 ); +return (((read_cpuid_id () >>8 ) &0xff ) ==0x58 ) && + (((mmp_chip_id &0xfff ) ==0x410 ) || + ((mmp_chip_id &0xfff ) ==0x610 )); }#else #define cpu_is_mmp2 () (0)
Greg Kroah-Hartman
maintainerPATCH 10/72 dm thin: send event about thin-pool state change _after_ making it
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mike Snitzer <[email protected]>
commit f6c367585d0d851349d3a9e607c43e5bea993fa1 upstream.
Sending a DM event before a thin-pool state change is about to happen is a bug. It wasn't realized until it became clear that userspace response to the event raced with the actual state change that the event was meant to notify about.
Fix this by first updating internal thin-pool state to reflect what the DM event is being issued about. This fixes a long-standing racey/buggy userspace device-mapper-test-suite 'resize_io' test that would get an event but not find the state it was looking for -- so it would just go on to hang because no other events caused the test to reevaluate the thin-pool's state.
--- drivers/md/dm-thin.c | 68 ++++++++++++++++++++++++++------------------------- 1 file changed, 35 insertions(+), 33 deletions(-)
drivers/md/dm-thin.c +35 -33
@@ -195,7 +195,7 @@ static void throttle_unlock(struct throtstruct dm_thin_new_mapping ;/* -* The pool runs in 4 modes. Ordered in degraded order for comparisons. +* The pool runs in various modes. Ordered in degraded order for comparisons. */ enum pool_mode {PM_WRITE ,/* metadata may be changed */
@@ -281,9 +281,38 @@ struct pool {struct dm_bio_prison_cell **cell_sort_array; }; -static enum pool_mode get_pool_mode (struct pool *pool);static void metadata_operation_failed (struct pool *pool,const char *op,int r); +static enum pool_mode get_pool_mode (struct pool *pool) +{ +return pool->pf.mode; +} + +static void notify_of_pool_mode_change (struct pool *pool) +{ +const char *descs[] = { +"write" , +"out-of-data-space" , +"read-only" , +"read-only" , +"fail" + }; +const char *extra_desc =NULL ; +enum pool_mode mode =get_pool_mode (pool); + +if (mode ==PM_OUT_OF_DATA_SPACE ) { +if (!pool->pf.error_if_no_space) + extra_desc =" (queue IO)" ; +else + extra_desc =" (error IO)" ; + } + +dm_table_event (pool->ti->table); +DMINFO ("%s: switching pool to %s%s mode" , +dm_device_name (pool->pool_md), + descs[(int )mode], extra_desc ? :"" ); +} +/* * Target context for a pool. */
@@ -2362,8 +2391,6 @@ static void do_waker(struct work_structqueue_delayed_work (pool->wq, &pool->waker,COMMIT_PERIOD ); } -static void notify_of_pool_mode_change_to_oods (struct pool *pool); -/* * We're holding onto IO to allow userland time to react. After the * timeout either the pool will have been resized (and thus back in
@@ -2376,7 +2403,7 @@ static void do_no_space_timeout(struct wif (get_pool_mode (pool) ==PM_OUT_OF_DATA_SPACE && !pool->pf.error_if_no_space) { pool->pf.error_if_no_space = true; -notify_of_pool_mode_change_to_oods (pool); +notify_of_pool_mode_change (pool);error_retry_list_with_code (pool,BLK_STS_NOSPC ); } }
@@ -2444,26 +2471,6 @@ static void noflush_work(struct thin_c */*----------------------------------------------------------------*/ -static enum pool_mode get_pool_mode (struct pool *pool) -{ -return pool->pf.mode; -} - -static void notify_of_pool_mode_change (struct pool *pool,const char *new_mode) -{ -dm_table_event (pool->ti->table); -DMINFO ("%s: switching pool to %s mode" , -dm_device_name (pool->pool_md), new_mode); -} - -static void notify_of_pool_mode_change_to_oods (struct pool *pool) -{ -if (!pool->pf.error_if_no_space) -notify_of_pool_mode_change (pool,"out-of-data-space (queue IO)" ); -else -notify_of_pool_mode_change (pool,"out-of-data-space (error IO)" ); -} -static bool passdown_enabled (struct pool_c *pt) {return pt->adjusted_pf.discard_passdown;
@@ -2512,8 +2519,6 @@ static void set_pool_mode(struct pool *pswitch (new_mode) {case PM_FAIL : -if (old_mode != new_mode) -notify_of_pool_mode_change (pool,"failure" );dm_pool_metadata_read_only (pool->pmd); pool->process_bio = process_bio_fail; pool->process_discard = process_bio_fail;
@@ -2527,8 +2532,6 @@ static void set_pool_mode(struct pool *pcase PM_OUT_OF_METADATA_SPACE :case PM_READ_ONLY : -if (!is_read_only_pool_mode (old_mode)) -notify_of_pool_mode_change (pool,"read-only" );dm_pool_metadata_read_only (pool->pmd); pool->process_bio = process_bio_read_only; pool->process_discard = process_bio_success;
@@ -2549,8 +2552,6 @@ static void set_pool_mode(struct pool *p* alarming rate. Adjust your low water mark if you're * frequently seeing this mode. */ -if (old_mode != new_mode) -notify_of_pool_mode_change_to_oods (pool); pool->out_of_data_space = true; pool->process_bio = process_bio_read_only; pool->process_discard = process_discard_bio;
@@ -2563,8 +2564,6 @@ static void set_pool_mode(struct pool *pbreak ;case PM_WRITE : -if (old_mode != new_mode) -notify_of_pool_mode_change (pool,"write" );if (old_mode ==PM_OUT_OF_DATA_SPACE )cancel_delayed_work_sync (&pool->no_space_timeout); pool->out_of_data_space = false;
@@ -2584,6 +2583,9 @@ static void set_pool_mode(struct pool *p* doesn't cause an unexpected mode transition on resume. */ pt->adjusted_pf.mode = new_mode; + +if (old_mode != new_mode) +notify_of_pool_mode_change (pool); }static void abort_transaction (struct pool *pool)
Greg Kroah-Hartman
maintainerPATCH 11/72 dm cache metadata: verify cache has blocks in blocks_are_clean_separate_dirty()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mike Snitzer <[email protected]>
commit 687cf4412a343a63928a5c9d91bdc0f522939d43 upstream.
Otherwise dm_bitset_cursor_begin() return -ENODATA. Other calls to dm_bitset_cursor_begin() have similar negative checks.
Fixes inability to create a cache in passthrough mode (even though doing so makes no sense).
--- drivers/md/dm-cache-metadata.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/md/dm-cache-metadata.c +4 -0
@@ -929,6 +929,10 @@ static int blocks_are_clean_separate_dirbool dirty_flag; *result = true; +if (from_cblock (cmd->cache_blocks) ==0 ) +/* Nothing to do */ +return 0 ; + r =dm_bitset_cursor_begin (&cmd->dirty_info, cmd->dirty_root,from_cblock (cmd->cache_blocks), &cmd->dirty_cursor);if (r) {
Greg Kroah-Hartman
maintainerPATCH 12/72 tracing: Fix memory leak in set_trigger_filter()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Steven Rostedt (VMware) <[email protected]>
commit 3cec638b3d793b7cacdec5b8072364b41caeb0e1 upstream.
When create_event_filter() fails in set_trigger_filter(), the filter may still be allocated and needs to be freed. The caller expects the data->filter to be updated with the new filter, even if the new filter failed (we could add an error message by setting set_str parameter of create_event_filter(), but that's another update).
But because the error would just exit, filter was left hanging and nothing could free it.
Found by kmemleak detector.
--- kernel/trace/trace_events_trigger.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
kernel/trace/trace_events_trigger.c +4 -2
@@ -744,8 +744,10 @@ int set_trigger_filter(char *filter_str,/* The filter is for the 'trigger' event, not the triggered event */ ret =create_event_filter (file->event_call, filter_str, false, &filter); -if (ret) - goto out; +/* +* If create_event_filter() fails, filter still needs to be freed. +* Which the calling code will do with data->filter. +*/ assign: tmp =rcu_access_pointer (data->filter);
Greg Kroah-Hartman
maintainerPATCH 13/72 tracing: Fix memory leak of instance function hash filters
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Steven Rostedt (VMware) <[email protected]>
commit 2840f84f74035e5a535959d5f17269c69fa6edc5 upstream.
The following commands will cause a memory leak:
# cd /sys/kernel/tracing # mkdir instances/foo # echo schedule > instance/foo/set_ftrace_filter # rmdir instances/foo
The reason is that the hashes that hold the filters to set_ftrace_filter and set_ftrace_notrace are not freed if they contain any data on the instance and the instance is removed.
Found by kmemleak detector.
--- kernel/trace/ftrace.c | 1 + 1 file changed, 1 insertion(+)
kernel/trace/ftrace.c +1 -0
@@ -5534,6 +5534,7 @@ void ftrace_destroy_filter_files(structif (ops->flags &FTRACE_OPS_FL_ENABLED )ftrace_shutdown (ops,0 ); ops->flags |=FTRACE_OPS_FL_DELETED ; +ftrace_free_filter (ops);mutex_unlock (&ftrace_lock); }
Greg Kroah-Hartman
maintainerPATCH 14/72 powerpc/msi: Fix NULL pointer access in teardown code
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Radu Rendec <[email protected]>
commit 78e7b15e17ac175e7eed9e21c6f92d03d3b0a6fa upstream.
The arch_teardown_msi_irqs() function assumes that controller ops pointers were already checked in arch_setup_msi_irqs(), but this assumption is wrong: arch_teardown_msi_irqs() can be called even when arch_setup_msi_irqs() returns an error (-ENOSYS).
This can happen in the following scenario:
- msi_capability_init() calls pci_msi_setup_msi_irqs()
- pci_msi_setup_msi_irqs() returns -ENOSYS
- msi_capability_init() notices the error and calls free_msi_irqs()
- free_msi_irqs() calls pci_msi_teardown_msi_irqs()
This is easier to see when CONFIG_PCI_MSI_IRQ_DOMAIN is not set and pci_msi_setup_msi_irqs() and pci_msi_teardown_msi_irqs() are just aliases to arch_setup_msi_irqs() and arch_teardown_msi_irqs().
The call to free_msi_irqs() upon pci_msi_setup_msi_irqs() failure seems legit, as it does additional cleanup; e.g. list_del(&entry->list) and kfree(entry) inside free_msi_irqs() do happen (MSI descriptors are allocated before pci_msi_setup_msi_irqs() is called and need to be cleaned up if that fails).
--- arch/powerpc/kernel/msi.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
arch/powerpc/kernel/msi.c +6 -1
@@ -34,5 +34,10 @@ void arch_teardown_msi_irqs(struct pci_d {struct pci_controller *phb =pci_bus_to_host (dev->bus); - phb->controller_ops.teardown_msi_irqs (dev); +/* +* We can be called even when arch_setup_msi_irqs() returns -ENOSYS, +* so check the pointer again. +*/ +if (phb->controller_ops.teardown_msi_irqs) + phb->controller_ops.teardown_msi_irqs (dev); }
Greg Kroah-Hartman
maintainerPATCH 15/72 drm/nouveau/kms: Fix memory leak in nv50_mstm_del()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Lyude Paul <[email protected]>
commit 24199c5436f267399afed0c4f1f57663c0408f57 upstream.
Noticed this while working on redoing the reference counting scheme in the DP MST helpers. Nouveau doesn't attempt to call drm_dp_mst_topology_mgr_destroy() at all, which leaves it leaking all of the resources for drm_dp_mst_topology_mgr and it's children mstbs+ports.
--- drivers/gpu/drm/nouveau/nv50_display.c | 1 + 1 file changed, 1 insertion(+)
drivers/gpu/drm/nouveau/nv50_display.c +1 -0
@@ -3378,6 +3378,7 @@ nv50_mstm_del(struct nv50_mstm **pmstm) {struct nv50_mstm *mstm = *pmstm;if (mstm) { +drm_dp_mst_topology_mgr_destroy (&mstm->mgr);kfree (*pmstm); *pmstm =NULL ; }
Greg Kroah-Hartman
maintainerPATCH 16/72 Revert "drm/rockchip: Allow driver to be shutdown on reboot/kexec"
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Brian Norris <[email protected]>
commit 63238173b2faf3d6b85a416f1c69af6c7be2413f upstream.
This reverts commit 7f3ef5dedb146e3d5063b6845781ad1bb59b92b5.
It causes new warnings [1] on shutdown when running the Google Kevin or Scarlet (RK3399) boards under Chrome OS. Presumably our usage of DRM is different than what Marc and Heiko test.
We're looking at a different approach (e.g., [2]) to replace this, but IMO the revert should be taken first, as it already propagated to -stable.
[1] Report here:
http://lkml.kernel.org/lkml/[email protected]
WARNING: CPU: 4 PID: 2035 at drivers/gpu/drm/drm_mode_config.c:477 drm_mode_config_cleanup+0x1c4/0x294 ... Call trace: drm_mode_config_cleanup+0x1c4/0x294 rockchip_drm_unbind+0x4c/0x8c component_master_del+0x88/0xb8 rockchip_drm_platform_remove+0x2c/0x44 rockchip_drm_platform_shutdown+0x20/0x2c platform_drv_shutdown+0x2c/0x38 device_shutdown+0x164/0x1b8 kernel_restart_prepare+0x40/0x48 kernel_restart+0x20/0x68 ... Memory manager not clean during takedown. WARNING: CPU: 4 PID: 2035 at drivers/gpu/drm/drm_mm.c:950 drm_mm_takedown+0x34/0x44 ... drm_mm_takedown+0x34/0x44 rockchip_drm_unbind+0x64/0x8c component_master_del+0x88/0xb8 rockchip_drm_platform_remove+0x2c/0x44 rockchip_drm_platform_shutdown+0x20/0x2c platform_drv_shutdown+0x2c/0x38 device_shutdown+0x164/0x1b8 kernel_restart_prepare+0x40/0x48 kernel_restart+0x20/0x68
...
[2] https://patchwork.kernel.org/patch/10556151/
https://www.spinics.net/lists/linux-rockchip/msg21342.html
[PATCH] drm/rockchip: shutdown drm subsystem on shutdown
--- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 6 ------ 1 file changed, 6 deletions(-)
drivers/gpu/drm/rockchip/rockchip_drm_drv.c +0 -6
@@ -425,11 +425,6 @@ static int rockchip_drm_platform_remove(return 0 ; } -static void rockchip_drm_platform_shutdown (struct platform_device *pdev) -{ -rockchip_drm_platform_remove (pdev); -} -static const struct of_device_id rockchip_drm_dt_ids[] = { { .compatible ="rockchip,display-subsystem" , }, {/* sentinel */ },
@@ -439,7 +434,6 @@ MODULE_DEVICE_TABLE(of, rockchip_drm_dt_static struct platform_driver rockchip_drm_platform_driver = { .probe = rockchip_drm_platform_probe, .remove = rockchip_drm_platform_remove, - .shutdown = rockchip_drm_platform_shutdown, .driver = { .name ="rockchip-drm" , .of_match_table = rockchip_drm_dt_ids,
Greg Kroah-Hartman
maintainerPATCH 17/72 drm/i915/execlists: Apply a full mb before execution for Braswell
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Chris Wilson <[email protected]>
commit cf66b8a0ba142fbd1bf10ac8f3ae92d1b0cb7b8f upstream.
Braswell is really picky about having our writes posted to memory before we execute or else the GPU may see stale values. A wmb() is insufficient as it only ensures the writes are visible to other cores, we need a full mb() to ensure the writes are in memory and visible to the GPU.
The most frequent failure in flushing before execution is that we see stale PTE values and execute the wrong pages.
References: 987abd5c62f9 ("drm/i915/execlists: Force write serialisation into context image vs execution")
--- drivers/gpu/drm/i915/intel_lrc.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
drivers/gpu/drm/i915/intel_lrc.c +6 -1
@@ -343,8 +343,13 @@ static u64 execlists_update_context(stru* may not be visible to the HW prior to the completion of the UC * register write and that we may begin execution from the context * before its image is complete leading to invalid PD chasing. +* +* Furthermore, Braswell, at least, wants a full mb to be sure that +* the writes are coherent in memory (visible to the GPU) prior to +* execution, and not just visible to other CPUs (as is the result of +* wmb). */ -wmb (); +mb ();return ce->lrc_desc; }
Greg Kroah-Hartman
maintainerPATCH 18/72 drm/amdgpu: update SMC firmware image for polaris10 variants
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Junwei Zhang <[email protected]>
commit d55d8be0747c96db28a1d08fc24d22ccd9b448ac upstream.
Some new variants require different firmwares.
--- drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c +2 -1
@@ -723,7 +723,8 @@ static int amdgpu_cgs_get_firmware_info( (adev->pdev->revision ==0xe7 ) || (adev->pdev->revision == 0xef))) || ((adev->pdev->device ==0x6fdf ) && - (adev->pdev->revision == 0xef))) { + ((adev->pdev->revision ==0xef ) || + (adev->pdev->revision == 0xff)))) { info->is_kicker = true;strcpy (fw_name,"amdgpu/polaris10_k_smc.bin" ); } else
Greg Kroah-Hartman
maintainerPATCH 19/72 x86/build: Fix compiler support check for CONFIG_RETPOLINE
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Masahiro Yamada <[email protected]>
commit 25896d073d8a0403b07e6dec56f58e6c33678207 upstream.
It is troublesome to add a diagnostic like this to the Makefile parse stage because the top-level Makefile could be parsed with a stale include/config/auto.conf.
Once you are hit by the error about non-retpoline compiler, the compilation still breaks even after disabling CONFIG_RETPOLINE.
The easiest fix is to move this check to the "archprepare" like this commit did:
829fe4aa9ac1 ("x86: Allow generating user-space headers without a compiler")
--- arch/x86/Makefile | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-)
arch/x86/Makefile +7 -3
@@ -241,9 +241,6 @@ KBUILD_CFLAGS += -fno-asynchronous-unwin # Avoid indirect branches in kernel to deal with Spectre ifdef CONFIG_RETPOLINE -ifeq ($(RETPOLINE_CFLAGS),) - $(error You are building kernel with non-retpoline compiler, please update your compiler.) -endif KBUILD_CFLAGS += $(RETPOLINE_CFLAGS) endif
@@ -260,6 +257,13 @@ archprepare: ifeq ($(CONFIG_KEXEC_FILE),y) $(Q)$(MAKE) $(build)=arch/x86/purgatory arch/x86/purgatory/kexec-purgatory.c endif +ifdef CONFIG_RETPOLINE +ifeq ($(RETPOLINE_CFLAGS),) + @echo "You are building kernel with non-retpoline compiler." >&2 + @echo "Please update your compiler." >&2 + @false +endif +endif ### # Kernel objects
Greg Kroah-Hartman
maintainerPATCH 20/72 locking: Remove smp_read_barrier_depends() from queued_spin_lock_slowpath()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 548095dea63ffc016d39c35b32c628d033638aca upstream.
Queued spinlocks are not used by DEC Alpha, and furthermore operations such as READ_ONCE() and release/relaxed RMW atomics are being changed to imply smp_read_barrier_depends(). This commit therefore removes the now-redundant smp_read_barrier_depends() from queued_spin_lock_slowpath(), and adjusts the comments accordingly.
--- kernel/locking/qspinlock.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-)
kernel/locking/qspinlock.c +5 -7
@@ -170,7 +170,7 @@ static __always_inline void clear_pending_set_locked(struct qspinlock *lock)* @tail : The new queue tail code word * Return: The previous queue tail code word * -* xchg(lock, tail) +* xchg(lock, tail), which heads an address dependency * * p,*,* -> n,*,* ; prev = xchg(lock, node) */
@@ -417,13 +417,11 @@ queue:if (old & _Q_TAIL_MASK) { prev =decode_tail (old);/* -* The above xchg_tail() is also a load of @lock which generates, -* through decode_tail(), a pointer. -* -* The address dependency matches the RELEASE of xchg_tail() -* such that the access to @prev must happen after. +* The above xchg_tail() is also a load of @lock which +* generates, through decode_tail(), a pointer. The address +* dependency matches the RELEASE of xchg_tail() such that +* the subsequent access to @prev happens after. */ -smp_read_barrier_depends ();WRITE_ONCE (prev->next, node);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 21/72 locking/qspinlock: Ensure node is initialised before updating prev->next
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 95bcade33a8af38755c9b0636e36a36ad3789fe6 upstream.
When a locker ends up queuing on the qspinlock locking slowpath, we initialise the relevant mcs node and publish it indirectly by updating the tail portion of the lock word using xchg_tail. If we find that there was a pre-existing locker in the queue, we subsequently update their ->next field to point at our node so that we are notified when it's our turn to take the lock.
This can be roughly illustrated as follows:
/* Initialise the fields in node and encode a pointer to node in tail */ tail = initialise_node(node);
/*
* Exchange tail into the lockword using an atomic read-modify-write
* operation with release semantics
*/
old = xchg_tail(lock, tail);
/* If there was a pre-existing waiter ... */
if (old & _Q_TAIL_MASK) {
prev = decode_tail(old);
smp_read_barrier_depends();
/* ... then update their ->next field to point to node.
WRITE_ONCE(prev->next, node);}
The conditional update of prev->next therefore relies on the address dependency from the result of xchg_tail ensuring order against the prior initialisation of node. However, since the release semantics of the xchg_tail operation apply only to the write portion of the RmW, then this ordering is not guaranteed and it is possible for the CPU to return old before the writes to node have been published, consequently allowing us to point prev->next to an uninitialised node.
This patch fixes the problem by making the update of prev->next a RELEASE operation, which also removes the reliance on dependency ordering.
--- kernel/locking/qspinlock.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-)
kernel/locking/qspinlock.c +7 -6
@@ -416,14 +416,15 @@ queue:*/ if (old & _Q_TAIL_MASK) { prev =decode_tail (old); +/* -* The above xchg_tail() is also a load of @lock which -* generates, through decode_tail(), a pointer. The address -* dependency matches the RELEASE of xchg_tail() such that -* the subsequent access to @prev happens after. +* We must ensure that the stores to @node are observed before +* the write to prev->next. The address dependency from +* xchg_tail is not sufficient to ensure this because the read +* component of xchg_tail is unordered with respect to the +* initialisation of @node. */ - -WRITE_ONCE (prev->next, node); +smp_store_release (&prev->next, node);pv_wait_node (node, prev);arch_mcs_spin_lock_contended (&node->locked);
Signature · 1 line
2.19.1
Re: [PATCH 4.14 21/72] locking/qspinlock: Ensure node is initialised before updating prev->next
Hi Greg,
Quoted 6 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit
95bcade33a8af38755c9b0636e36a36ad3789fe6upstream.
It seems 9d4646d14d51 ("locking/qspinlock: Elide back-to-back RELEASE operations with smp_wmb()") is improving the fix. Is it worth to take this also?
Signature · 2 lines
Regards Sudip
Greg Kroah-Hartman
maintainerRe: [PATCH 4.14 21/72] locking/qspinlock: Ensure node is initialised before updating prev->next
Quoted 7 lines · Sudip Mukherjee
Hi Greg,
Quoted 3 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
Quoted 7 lines
Quoted 3 lines
------------------
commit
95bcade33a8af38755c9b0636e36a36ad3789fe6upstream.It seems
9d4646d14d51("locking/qspinlock: Elide back-to-back RELEASE operations withsmp_wmb()") is improving the fix. Is it worth to take this also?
If someone could test that this really does help things out, I would be glad to take it :)
thanks,
greg k-h
Re: [PATCH 4.14 21/72] locking/qspinlock: Ensure node is initialised before updating prev->next
Quoted 11 lines · Greg Kroah-Hartman
Quoted 9 lines · Sudip Mukherjee
Hi Greg,
Quoted 5 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
Quoted 8 lines
Quoted 5 lines
Quoted 1 line
commit
95bcade33a8af38755c9b0636e36a36ad3789fe6upstream.It seems
9d4646d14d51("locking/qspinlock: Elide back-to-back RELEASE operations withsmp_wmb()") is improving the fix. Is it worth to take this also?If someone could test that this really does help things out, I would be glad to take it :)
I can test if Will shows me how to test the improvement, but it might be easier for him to test than showing me :)
Signature · 2 lines
Regards Sudip
Greg Kroah-Hartman
maintainerPATCH 22/72 locking/qspinlock: Bound spinning on pending->locked transition in slowpath
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 6512276d97b160d90b53285bd06f7f201459a7e3 upstream.
If a locker taking the qspinlock slowpath reads a lock value indicating that only the pending bit is set, then it will spin whilst the concurrent pending->locked transition takes effect.
Unfortunately, there is no guarantee that such a transition will ever be observed since concurrent lockers could continuously set pending and hand over the lock amongst themselves, leading to starvation. Whilst this would probably resolve in practice, it means that it is not possible to prove liveness properties about the lock and means that lock acquisition time is unbounded.
Rather than removing the pending->locked spinning from the slowpath altogether (which has been shown to heavily penalise a 2-threaded locking stress test on x86), this patch replaces the explicit spinning with a call to atomic_cond_read_relaxed and allows the architecture to provide a bound on the number of spins. For architectures that can respond to changes in cacheline state in their smp_cond_load implementation, it should be sufficient to use the default bound of 1.
--- kernel/locking/qspinlock.c | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-)
kernel/locking/qspinlock.c +17 -3
@@ -76,6 +76,18 @@#define MAX_NODES 4#endif +/* +* The pending bit spinning loop count. +* This heuristic is used to limit the number of lockword accesses +* made by atomic_cond_read_relaxed when waiting for the lock to +* transition out of the "== _Q_PENDING_VAL" state. We don't spin +* indefinitely because there's no guarantee that we'll make forward +* progress. +*/ +#ifndef _Q_PENDING_LOOPS +#define _Q_PENDING_LOOPS 1 +#endif +/* * Per-CPU queue node structures; we can never have more than 4 nested * contexts: task, softirq, hardirq, nmi.
@@ -306,13 +318,15 @@ void queued_spin_lock_slowpath(struct qspinlock *lock, u32 val)return ;/* -* wait for in-progress pending->locked hand-overs +* Wait for in-progress pending->locked hand-overs with a bounded +* number of spins so that we guarantee forward progress. * * 0,1,0 -> 0,0,1 */ if (val == _Q_PENDING_VAL) { -while ((val =atomic_read (&lock->val)) == _Q_PENDING_VAL) -cpu_relax (); +int cnt = _Q_PENDING_LOOPS; + val =smp_cond_load_acquire (&lock->val.counter, + (VAL != _Q_PENDING_VAL) || !cnt--); }/*
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 23/72 locking/qspinlock: Merge struct __qspinlock into struct qspinlock
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 625e88be1f41b53cec55827c984e4a89ea8ee9f9 upstream.
'struct __qspinlock' provides a handy union of fields so that subcomponents of the lockword can be accessed by name, without having to manage shifts and masks explicitly and take endianness into account.
This is useful in qspinlock.h and also potentially in arch headers, so move the 'struct __qspinlock' into 'struct qspinlock' and kill the extra definition.
--- arch/x86/include/asm/qspinlock.h | 2 +- arch/x86/include/asm/qspinlock_paravirt.h | 3 +- include/asm-generic/qspinlock_types.h | 32 +++++++++++++++- kernel/locking/qspinlock.c | 46 ++--------------------- kernel/locking/qspinlock_paravirt.h | 34 ++++++----------- 5 files changed, 46 insertions(+), 71 deletions(-)
arch/x86/include/asm/qspinlock.h +1 -1
@@ -15,7 +15,7 @@*/ static inline void native_queued_spin_unlock (struct qspinlock *lock) { -smp_store_release ((u8 *)lock,0 ); +smp_store_release (&lock->locked,0 ); }#ifdef CONFIG_PARAVIRT_SPINLOCKS
arch/x86/include/asm/qspinlock_paravirt.h +1 -2
@@ -22,8 +22,7 @@ PV_CALLEE_SAVE_REGS_THUNK(__pv_queued_spin_unlock_slowpath);* * void __pv_queued_spin_unlock(struct qspinlock *lock) * { -* struct __qspinlock *l = (void *)lock; -* u8 lockval = cmpxchg(&l->locked, _Q_LOCKED_VAL, 0); +* u8 lockval = cmpxchg(&lock->locked, _Q_LOCKED_VAL, 0); * * if (likely(lockval == _Q_LOCKED_VAL)) * return;
include/asm-generic/qspinlock_types.h +30 -2
@@ -29,13 +29,41 @@#endif typedef struct qspinlock { -atomic_t val; +union { +atomic_t val; + +/* +* By using the whole 2nd least significant byte for the +* pending bit, we can allow better optimization of the lock +* acquisition for the pending bit holder. +*/ +#ifdef __LITTLE_ENDIAN +struct { +u8 locked; +u8 pending; + }; +struct { +u16 locked_pending; +u16 tail; + }; +#else +struct { +u16 tail; +u16 locked_pending; + }; +struct { +u8 reserved[2 ]; +u8 pending; +u8 locked; + }; +#endif + }; }arch_spinlock_t ;/* * Initializier */ -#define __ARCH_SPIN_LOCK_UNLOCKED { ATOMIC_INIT(0) } +#define __ARCH_SPIN_LOCK_UNLOCKED { .val = ATOMIC_INIT(0) }/* * Bitfields in the atomic value:
kernel/locking/qspinlock.c +3 -43
74 lines not on this page · raw message
kernel/locking/qspinlock_paravirt.h +11 -23
@@ -70,10 +70,8 @@ struct pv_node {#define queued_spin_trylock (l) pv_queued_spin_steal_lock(l)static inline bool pv_queued_spin_steal_lock (struct qspinlock *lock) { -struct __qspinlock *l = (void *)lock; -if (!(atomic_read (&lock->val) & _Q_LOCKED_PENDING_MASK) && - (cmpxchg_acquire (&l->locked,0 , _Q_LOCKED_VAL) ==0 )) { + (cmpxchg_acquire (&lock->locked,0 , _Q_LOCKED_VAL) ==0 )) {qstat_inc (qstat_pv_lock_stealing, true);return true; }
@@ -88,16 +86,12 @@ static inline bool pv_queued_spin_steal_lock(struct qspinlock *lock)#if _Q_PENDING_BITS ==8 static __always_inline void set_pending(struct qspinlock *lock) { -struct __qspinlock *l = (void *)lock; - -WRITE_ONCE (l->pending,1 ); +WRITE_ONCE (lock->pending,1 ); } static __always_inline void clear_pending(struct qspinlock *lock) { -struct __qspinlock *l = (void *)lock; - -WRITE_ONCE (l->pending,0 ); +WRITE_ONCE (lock->pending,0 ); }/*
@@ -107,10 +101,8 @@ static __always_inline void clear_pending(struct qspinlock *lock)*/ static __always_inline int trylock_clear_pending(struct qspinlock *lock) { -struct __qspinlock *l = (void *)lock; - -return !READ_ONCE (l->locked) && - (cmpxchg_acquire (&l->locked_pending, _Q_PENDING_VAL, +return !READ_ONCE (lock->locked) && + (cmpxchg_acquire (&lock->locked_pending, _Q_PENDING_VAL, _Q_LOCKED_VAL) == _Q_PENDING_VAL); }#else /* _Q_PENDING_BITS == 8 */
@@ -355,7 +347,6 @@ static void pv_wait_node(struct mcs_spinlock *node, struct mcs_spinlock *prev)static void pv_kick_node (struct qspinlock *lock,struct mcs_spinlock *node) {struct pv_node *pn = (struct pv_node *)node; -struct __qspinlock *l = (void *)lock;/* * If the vCPU is indeed halted, advance its state to match that of
@@ -384,7 +375,7 @@ static void pv_kick_node(struct qspinlock *lock, struct mcs_spinlock *node)* the hash table later on at unlock time, no atomic instruction is * needed. */ -WRITE_ONCE (l->locked, _Q_SLOW_VAL); +WRITE_ONCE (lock->locked, _Q_SLOW_VAL); (void )pv_hash (lock, pn); }
@@ -399,7 +390,6 @@ static u32 pv_wait_head_or_lock(struct qspinlock *lock, struct mcs_spinlock *node) { struct pv_node *pn = (struct pv_node *)node; -struct __qspinlock *l = (void *)lock;struct qspinlock **lp =NULL ;int waitcnt =0 ;int loop;
@@ -450,13 +440,13 @@ pv_wait_head_or_lock(struct qspinlock *lock, struct mcs_spinlock *node)* * Matches the smp_rmb() in __pv_queued_spin_unlock(). */ -if (xchg (&l->locked, _Q_SLOW_VAL) ==0 ) { +if (xchg (&lock->locked, _Q_SLOW_VAL) ==0 ) {/* * The lock was free and now we own the lock. * Change the lock value back to _Q_LOCKED_VAL * and unhash the table. */ -WRITE_ONCE (l->locked, _Q_LOCKED_VAL); +WRITE_ONCE (lock->locked, _Q_LOCKED_VAL);WRITE_ONCE (*lp,NULL ); goto gotlock; }
@@ -464,7 +454,7 @@ pv_wait_head_or_lock(struct qspinlock *lock, struct mcs_spinlock *node)WRITE_ONCE (pn->state, vcpu_hashed);qstat_inc (qstat_pv_wait_head, true);qstat_inc (qstat_pv_wait_again, waitcnt); -pv_wait (&l->locked, _Q_SLOW_VAL); +pv_wait (&lock->locked, _Q_SLOW_VAL);/* * Because of lock stealing, the queue head vCPU may not be
@@ -489,7 +479,6 @@ gotlock: __visible void__pv_queued_spin_unlock_slowpath (struct qspinlock *lock,u8 locked) { -struct __qspinlock *l = (void *)lock;struct pv_node *node;if (unlikely (locked != _Q_SLOW_VAL)) {
@@ -518,7 +507,7 @@ __pv_queued_spin_unlock_slowpath(struct qspinlock *lock, u8 locked)* Now that we have a reference to the (likely) blocked pv_node, * release the lock. */ -smp_store_release (&l->locked,0 ); +smp_store_release (&lock->locked,0 );/* * At this point the memory pointed at by lock can be freed/reused,
@@ -544,7 +533,6 @@ __pv_queued_spin_unlock_slowpath(struct qspinlock *lock, u8 locked)#ifndef __pv_queued_spin_unlock __visible void __pv_queued_spin_unlock(struct qspinlock *lock) { -struct __qspinlock *l = (void *)lock;u8 locked;/*
@@ -552,7 +540,7 @@ __visible void __pv_queued_spin_unlock(struct qspinlock *lock)* unhash. Otherwise it would be possible to have multiple @lock * entries, which would be BAD. */ - locked =cmpxchg_release (&l->locked, _Q_LOCKED_VAL,0 ); + locked =cmpxchg_release (&lock->locked, _Q_LOCKED_VAL,0 );if (likely (locked == _Q_LOCKED_VAL))return ;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 24/72 locking/qspinlock: Remove unbounded cmpxchg() loop from locking slowpath
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 59fb586b4a07b4e1a0ee577140ab4842ba451acd upstream.
The qspinlock locking slowpath utilises a "pending" bit as a simple form of an embedded test-and-set lock that can avoid the overhead of explicit queuing in cases where the lock is held but uncontended. This bit is managed using a cmpxchg() loop which tries to transition the uncontended lock word from (0,0,0) -> (0,0,1) or (0,0,1) -> (0,1,1).
Unfortunately, the cmpxchg() loop is unbounded and lockers can be starved indefinitely if the lock word is seen to oscillate between unlocked (0,0,0) and locked (0,0,1). This could happen if concurrent lockers are able to take the lock in the cmpxchg() loop without queuing and pass it around amongst themselves.
This patch fixes the problem by unconditionally setting _Q_PENDING_VAL using atomic_fetch_or, and then inspecting the old value to see whether we need to spin on the current lock owner, or whether we now effectively hold the lock. The tricky scenario is when concurrent lockers end up queuing on the lock and the lock becomes available, causing us to see a lockword of (n,0,0). With pending now set, simply queuing could lead to deadlock as the head of the queue may not have observed the pending flag being cleared. Conversely, if the head of the queue did observe pending being cleared, then it could transition the lock from (n,0,0) -> (0,0,1) meaning that any attempt to "undo" our setting of the pending bit could race with a concurrent locker trying to set it.
We handle this race by preserving the pending bit when taking the lock after reaching the head of the queue and leaving the tail entry intact if we saw pending set, because we know that the tail is going to be updated shortly.
--- kernel/locking/qspinlock.c | 102 ++++++++++++++++------------ kernel/locking/qspinlock_paravirt.h | 5 -- 2 files changed, 58 insertions(+), 49 deletions(-)
kernel/locking/qspinlock.c +58 -44
@@ -127,6 +127,17 @@ static inline __pure struct mcs_spinlock *decode_tail(u32 tail)#define _Q_LOCKED_PENDING_MASK (_Q_LOCKED_MASK | _Q_PENDING_MASK)#if _Q_PENDING_BITS ==8 +/** +* clear_pending - clear the pending bit. +* @lock: Pointer to queued spinlock structure +* +* *,1,* -> *,0,* +*/ +static __always_inline void clear_pending(struct qspinlock *lock) +{ +WRITE_ONCE (lock->pending,0 ); +} +/** * clear_pending_set_locked - take ownership and clear the pending bit. * @lock: Pointer to queued spinlock structure
@@ -162,6 +173,17 @@ static __always_inline u32 xchg_tail(struct qspinlock *lock, u32 tail)#else /* _Q_PENDING_BITS == 8 */ +/** +* clear_pending - clear the pending bit. +* @lock: Pointer to queued spinlock structure +* +* *,1,* -> *,0,* +*/ +static __always_inline void clear_pending(struct qspinlock *lock) +{ +atomic_andnot (_Q_PENDING_VAL, &lock->val); +} +/** * clear_pending_set_locked - take ownership and clear the pending bit. * @lock: Pointer to queued spinlock structure
@@ -266,7 +288,7 @@ static __always_inline u32 __pv_wait_head_or_lock(struct qspinlock *lock,void queued_spin_lock_slowpath (struct qspinlock *lock,u32 val) {struct mcs_spinlock *prev, *next, *node; -u32 new, old, tail; +u32 old, tail;int idx;BUILD_BUG_ON (CONFIG_NR_CPUS >= (1U << _Q_TAIL_CPU_BITS));
@@ -289,59 +311,51 @@ void queued_spin_lock_slowpath(struct qspinlock *lock, u32 val) (VAL != _Q_PENDING_VAL) || !cnt--); } +/* +* If we observe any contention; queue. +*/ +if (val & ~_Q_LOCKED_MASK) + goto queue; +/* * trylock || pending * * 0,0,0 -> 0,0,1 ; trylock * 0,0,1 -> 0,1,1 ; pending */ -for (;;) { + val =atomic_fetch_or_acquire (_Q_PENDING_VAL, &lock->val); +if (!(val & ~_Q_LOCKED_MASK)) {/* -* If we observe any contention; queue. +* We're pending, wait for the owner to go away. +* +* *,1,1 -> *,1,0 +* +* this wait loop must be a load-acquire such that we match the +* store-release that clears the locked bit and create lock +* sequentiality; this is because not all +* clear_pending_set_locked() implementations imply full +* barriers. */ -if (val & ~_Q_LOCKED_MASK) - goto queue; - - new = _Q_LOCKED_VAL; -if (val == new) - new |= _Q_PENDING_VAL; +if (val & _Q_LOCKED_MASK) { +smp_cond_load_acquire (&lock->val.counter, + !(VAL & _Q_LOCKED_MASK)); + }/* -* Acquire semantic is required here as the function may -* return immediately if the lock was free. +* take ownership and clear the pending bit. +* +* *,1,0 -> *,0,1 */ - old =atomic_cmpxchg_acquire (&lock->val, val, new); -if (old == val) -break ; - - val = old; - } - -/* -* we won the trylock -*/ -if (new == _Q_LOCKED_VAL) +clear_pending_set_locked (lock);return ; + }/* -* we're pending, wait for the owner to go away. -* -* *,1,1 -> *,1,0 -* -* this wait loop must be a load-acquire such that we match the -* store-release that clears the locked bit and create lock -* sequentiality; this is because not all clear_pending_set_locked() -* implementations imply full barriers. -*/ -smp_cond_load_acquire (&lock->val.counter, !(VAL & _Q_LOCKED_MASK)); - -/* -* take ownership and clear the pending bit. -* -* *,1,0 -> *,0,1 +* If pending was clear but there are waiters in the queue, then +* we need to undo our setting of pending before we queue ourselves. */ -clear_pending_set_locked (lock); -return ; +if (!(val & _Q_PENDING_MASK)) +clear_pending (lock);/* * End of pending bit optimistic spinning and beginning of MCS
@@ -445,15 +459,15 @@ locked:* claim the lock: * * n,0,0 -> 0,0,1 : lock, uncontended -* *,0,0 -> *,0,1 : lock, contended +* *,*,0 -> *,*,1 : lock, contended * -* If the queue head is the only one in the queue (lock value == tail), -* clear the tail code and grab the lock. Otherwise, we only need -* to grab the lock. +* If the queue head is the only one in the queue (lock value == tail) +* and nobody is pending, clear the tail code and grab the lock. +* Otherwise, we only need to grab the lock. */ for (;;) {/* In the PV case we might already have _Q_LOCKED_VAL set */ -if ((val & _Q_TAIL_MASK) != tail) { +if ((val & _Q_TAIL_MASK) != tail || (val & _Q_PENDING_MASK)) {set_locked (lock);break ; }
kernel/locking/qspinlock_paravirt.h +0 -5
@@ -89,11 +89,6 @@ static __always_inline void set_pending(struct qspinlock *lock)WRITE_ONCE (lock->pending,1 ); } -static __always_inline void clear_pending(struct qspinlock *lock) -{ -WRITE_ONCE (lock->pending,0 ); -} -/* * The pending bit check in pv_queued_spin_steal_lock() isn't a memory * barrier. Therefore, an atomic cmpxchg_acquire() is used to acquire the
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 25/72 locking/qspinlock: Remove duplicate clear_pending() function from PV code
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 3bea9adc96842b8a7345c7fb202c16ae9c8d5b25 upstream.
The native clear_pending() function is identical to the PV version, so the latter can simply be removed.
This fixes the build for systems with >= 16K CPUs using the PV lock implementation.
--- kernel/locking/qspinlock_paravirt.h | 5 ----- 1 file changed, 5 deletions(-)
kernel/locking/qspinlock_paravirt.h +0 -5
@@ -106,11 +106,6 @@ static __always_inline void set_pending(struct qspinlock *lock)atomic_or (_Q_PENDING_VAL, &lock->val); } -static __always_inline void clear_pending(struct qspinlock *lock) -{ -atomic_andnot (_Q_PENDING_VAL, &lock->val); -} - static __always_inline int trylock_clear_pending(struct qspinlock *lock) {int val =atomic_read (&lock->val);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 26/72 locking/qspinlock: Kill cmpxchg() loop when claiming lock from head of queue
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit c61da58d8a9ba9238250a548f00826eaf44af0f7 upstream.
When a queued locker reaches the head of the queue, it claims the lock by setting _Q_LOCKED_VAL in the lockword. If there isn't contention, it must also clear the tail as part of this operation so that subsequent lockers can avoid taking the slowpath altogether.
Currently this is expressed as a cmpxchg() loop that practically only runs up to two iterations. This is confusing to the reader and unhelpful to the compiler. Rewrite the cmpxchg() loop without the loop, so that a failed cmpxchg() implies that there is contention and we just need to write to _Q_LOCKED_VAL without considering the rest of the lockword.
--- kernel/locking/qspinlock.c | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-)
kernel/locking/qspinlock.c +8 -11
@@ -465,24 +465,21 @@ locked:* and nobody is pending, clear the tail code and grab the lock. * Otherwise, we only need to grab the lock. */ -for (;;) { -/* In the PV case we might already have _Q_LOCKED_VAL set */ -if ((val & _Q_TAIL_MASK) != tail || (val & _Q_PENDING_MASK)) { -set_locked (lock); -break ; - } + +/* In the PV case we might already have _Q_LOCKED_VAL set */ +if ((val & _Q_TAIL_MASK) == tail) {/* * The smp_cond_load_acquire() call above has provided the -* necessary acquire semantics required for locking. At most -* two iterations of this loop may be ran. +* necessary acquire semantics required for locking. */ old =atomic_cmpxchg_relaxed (&lock->val, val, _Q_LOCKED_VAL);if (old == val) - goto release;/* No contention */ - - val = old; + goto release;/* No contention */ } +/* Either somebody is queued behind us or _Q_PENDING_VAL is set */ +set_locked (lock); +/* * contended path; wait for next if not observed yet, release. */
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 27/72 locking/qspinlock: Re-order code
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 53bf57fab7321fb42b703056a4c80fc9d986d170 upstream.
Flip the branch condition after atomic_fetch_or_acquire(_Q_PENDING_VAL) such that we loose the indent. This also result in a more natural code flow IMO.
--- kernel/locking/qspinlock.c | 54 ++++++++++++++++++-------------------- 1 file changed, 26 insertions(+), 28 deletions(-)
kernel/locking/qspinlock.c +26 -28
@@ -324,38 +324,36 @@ void queued_spin_lock_slowpath(struct qspinlock *lock, u32 val)* 0,0,1 -> 0,1,1 ; pending */ val =atomic_fetch_or_acquire (_Q_PENDING_VAL, &lock->val); -if (!(val & ~_Q_LOCKED_MASK)) { -/* -* We're pending, wait for the owner to go away. -* -* *,1,1 -> *,1,0 -* -* this wait loop must be a load-acquire such that we match the -* store-release that clears the locked bit and create lock -* sequentiality; this is because not all -* clear_pending_set_locked() implementations imply full -* barriers. -*/ -if (val & _Q_LOCKED_MASK) { -smp_cond_load_acquire (&lock->val.counter, - !(VAL & _Q_LOCKED_MASK)); - } - -/* -* take ownership and clear the pending bit. -* -* *,1,0 -> *,0,1 -*/ -clear_pending_set_locked (lock); -return ; +/* +* If we observe any contention; undo and queue. +*/ +if (unlikely (val & ~_Q_LOCKED_MASK)) { +if (!(val & _Q_PENDING_MASK)) +clear_pending (lock); + goto queue; }/* -* If pending was clear but there are waiters in the queue, then -* we need to undo our setting of pending before we queue ourselves. +* We're pending, wait for the owner to go away. +* +* 0,1,1 -> 0,1,0 +* +* this wait loop must be a load-acquire such that we match the +* store-release that clears the locked bit and create lock +* sequentiality; this is because not all +* clear_pending_set_locked() implementations imply full +* barriers. +*/ +if (val & _Q_LOCKED_MASK) +smp_cond_load_acquire (&lock->val.counter, !(VAL & _Q_LOCKED_MASK)); + +/* +* take ownership and clear the pending bit. +* +* 0,1,0 -> 0,0,1 */ -if (!(val & _Q_PENDING_MASK)) -clear_pending (lock); +clear_pending_set_locked (lock); +return ;/* * End of pending bit optimistic spinning and beginning of MCS
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 28/72 locking/qspinlock/x86: Increase _Q_PENDING_LOOPS upper bound
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit b247be3fe89b6aba928bf80f4453d1c4ba8d2063 upstream.
On x86, atomic_cond_read_relaxed will busy-wait with a cpu_relax() loop, so it is desirable to increase the number of times we spin on the qspinlock lockword when it is found to be transitioning from pending to locked.
According to Waiman Long:
| Ideally, the spinning times should be at least a few times the typical | cacheline load time from memory which I think can be down to 100ns or | so for each cacheline load with the newest systems or up to several | hundreds ns for older systems.
which in his benchmarking corresponded to 512 iterations.
--- arch/x86/include/asm/qspinlock.h | 2 ++ 1 file changed, 2 insertions(+)
arch/x86/include/asm/qspinlock.h +2 -0
@@ -6,6 +6,8 @@#include <asm-generic/qspinlock_types.h> #include <asm/paravirt.h> +#define _Q_PENDING_LOOPS (1 << 9) +#define queued_spin_unlock queued_spin_unlock/** * queued_spin_unlock - release a queued spinlock
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 29/72 locking/qspinlock, x86: Provide liveness guarantee
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 7aa54be2976550f17c11a1c3e3630002dea39303 upstream.
On x86 we cannot do fetch_or() with a single instruction and thus end up using a cmpxchg loop, this reduces determinism. Replace the fetch_or() with a composite operation: tas-pending + load.
Using two instructions of course opens a window we previously did not have. Consider the scenario:
CPU0 CPU1 CPU2
1) lock
trylock -> (0,0,1)
2) lock
trylock /* fail */
3) unlock -> (0,0,0)
4) lock
trylock -> (0,0,1)
5) tas-pending -> (0,1,1)
load-val <- (0,1,0) from 3
6) clear-pending-set-locked -> (0,0,1)
FAIL: _2_ owners
where 5) is our new composite operation. When we consider each part of the qspinlock state as a separate variable (as we can when _Q_PENDING_BITS == 8) then the above is entirely possible, because tas-pending will only RmW the pending byte, so the later load is able to observe prior tail and lock state (but not earlier than its own trylock, which operates on the whole word, due to coherence).
To avoid this we need 2 things:
- the load must come after the tas-pending (obviously, otherwise it can trivially observe prior state).
- the tas-pending must be a full word RmW instruction, it cannot be an XCHGB for example, such that we cannot observe other state prior to setting pending.
On x86 we can realize this by using "LOCK BTS m32, r32" for tas-pending followed by a regular load.
Note that observing later state is not a problem:
- if we fail to observe a later unlock, we'll simply spin-wait for that store to become visible.
- if we observe a later xchg_tail(), there is no difference from that xchg_tail() having taken place before the tas-pending.
--- arch/x86/include/asm/qspinlock.h | 21 +++++++++++++++++++++ kernel/locking/qspinlock.c | 17 ++++++++++++++++- 2 files changed, 37 insertions(+), 1 deletion(-)
arch/x86/include/asm/qspinlock.h +21 -0
@@ -5,9 +5,30 @@#include <asm/cpufeature.h> #include <asm-generic/qspinlock_types.h> #include <asm/paravirt.h> +#include <asm/rmwcc.h> #define _Q_PENDING_LOOPS (1 << 9) +#define queued_fetch_set_pending_acquire queued_fetch_set_pending_acquire + +static __always_inline bool __queued_RMW_btsl(struct qspinlock *lock) +{ +GEN_BINARY_RMWcc (LOCK_PREFIX "btsl" , lock->val.counter, +"I" , _Q_PENDING_OFFSET,"%0" , c); +} + +static __always_inline u32 queued_fetch_set_pending_acquire(struct qspinlock *lock) +{ +u32 val =0 ; + +if (__queued_RMW_btsl (lock)) + val |= _Q_PENDING_VAL; + + val |=atomic_read (&lock->val) & ~_Q_PENDING_MASK; + +return val; +} +#define queued_spin_unlock queued_spin_unlock/** * queued_spin_unlock - release a queued spinlock
kernel/locking/qspinlock.c +16 -1
@@ -225,6 +225,20 @@ static __always_inline u32 xchg_tail(struct qspinlock *lock, u32 tail) }#endif /* _Q_PENDING_BITS == 8 */ +/** +* queued_fetch_set_pending_acquire - fetch the whole lock value and set pending +* @lock : Pointer to queued spinlock structure +* Return: The previous lock value +* +* *,*,* -> *,1,* +*/ +#ifndef queued_fetch_set_pending_acquire +static __always_inline u32 queued_fetch_set_pending_acquire(struct qspinlock *lock) +{ +return atomic_fetch_or_acquire (_Q_PENDING_VAL, &lock->val); +} +#endif +/** * set_locked - Set the lock bit and own the lock * @lock: Pointer to queued spinlock structure
@@ -323,7 +337,8 @@ void queued_spin_lock_slowpath(struct qspinlock *lock, u32 val)* 0,0,0 -> 0,0,1 ; trylock * 0,0,1 -> 0,1,1 ; pending */ - val =atomic_fetch_or_acquire (_Q_PENDING_VAL, &lock->val); + val =queued_fetch_set_pending_acquire (lock); +/* * If we observe any contention; undo and queue. */
Signature · 1 line
2.19.1
Re: [PATCH 4.14 29/72] locking/qspinlock, x86: Provide liveness guarantee
Hi Greg,
Quoted 6 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit
7aa54be2976550f17c11a1c3e3630002dea39303upstream.
Another upstream commit fixes this.b987ffc18fb3 ("x86/qspinlock: Fix compile error")
Signature · 2 lines
Regards Sudip
Greg Kroah-Hartman
maintainerRe: [PATCH 4.14 29/72] locking/qspinlock, x86: Provide liveness guarantee
Quoted 5 lines · Sudip Mukherjee
Hi Greg,
Quoted 1 line · Greg Kroah-Hartman
Quoted 8 lines
Quoted 5 lines
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit
7aa54be2976550f17c11a1c3e3630002dea39303upstream.Another upstream commit fixes this.
b987ffc18fb3("x86/qspinlock: Fix compile error")
Maybe, but that commit doesn't apply to any of these stable trees :(
Care to provide a backport?
thanks,
greg k-h
Re: [PATCH 4.14 29/72] locking/qspinlock, x86: Provide liveness guarantee
Quoted 10 lines · Greg Kroah-Hartman
Quoted 9 lines · Sudip Mukherjee
Hi Greg,
Quoted 5 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
Quoted 8 lines
Quoted 4 lines
Quoted 1 line
commit
7aa54be2976550f17c11a1c3e3630002dea39303upstream.Another upstream commit fixes this.
b987ffc18fb3("x86/qspinlock: Fix compile error")Maybe, but that commit doesn't apply to any of these stable trees :(
Care to provide a backport?
Attached now.
--
Regards
Sudip
Quoted 1 line
From
35c4feb16f5204faa40e9b0451cd86549819f375Mon Sep 17 00:00:00 2001
From: Peter Zijlstra <[email protected]>
Date: Fri, 2 Nov 2018 14:26:53 +0100
Subject: [PATCH] x86/qspinlock: Fix compile error
commit b987ffc18fb3b3b76b059aa9e372dbee26f7c4f2 upstream
With a compiler that has asm-goto but not asm-cc-output and CONFIG_PROFILE_ALL_BRANCHES=y we get a compiler error:
arch/x86/include/asm/rmwcc.h:23:17: error: jump into statement expression
Fix this by writing the if() as a boolean multiplication instead.
--- arch/x86/include/asm/qspinlock.h | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-)
arch/x86/include/asm/qspinlock.h +7 -3
@@ -19,10 +19,14 @@ static __always_inline bool __queued_RMW_btsl(struct qspinlock *lock) static __always_inline u32 queued_fetch_set_pending_acquire(struct qspinlock *lock) { -u32 val =0 ; +u32 val; -if (__queued_RMW_btsl (lock)) - val |= _Q_PENDING_VAL; +/* +* We can't use GEN_BINARY_RMWcc() inside an if() stmt because asm goto +* and CONFIG_PROFILE_ALL_BRANCHES=y results in a label inside a +* statement expression, which GCC doesn't like. +*/ + val =__queued_RMW_btsl (lock) * _Q_PENDING_VAL; val |=atomic_read (&lock->val) & ~_Q_PENDING_MASK;
Signature · 1 line
2.11.0
Re: [PATCH 4.14 29/72] locking/qspinlock, x86: Provide liveness guarantee
Quoted 13 lines · Sudip Mukherjee
Quoted 12 lines · Greg Kroah-Hartman
Quoted 11 lines · Sudip Mukherjee
Hi Greg,
Quoted 6 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit
7aa54be2976550f17c11a1c3e3630002dea39303upstream.
Quoted 8 lines
Quoted 6 lines
Quoted 2 lines
Another upstream commit fixes this.
b987ffc18fb3("x86/qspinlock: Fix compile error")Maybe, but that commit doesn't apply to any of these stable trees :(
Care to provide a backport?
Attached now.
Are you sure that it fails to compile without that patch? I have here Debian's gcc version 8.2.0 which probably isn't affected and I can compile kernel/locking/ in v4.19 + 4.14.
I'm asking because in my backport the GEN_BINARY_RMWcc macro is used like in all the other functions which use it - unlike like in the original commit where the macro is used directly in the if condition. So it might not be affected by the problem.
Therefore I would prefer not to try to patch something that isn't required.
Quoted 3 lines
--
Regards
Sudip
Sebastian
Re: [PATCH 4.14 29/72] locking/qspinlock, x86: Provide liveness guarantee
Quoted 24 lines · Sebastian Andrzej Siewior
Quoted 13 lines · Sudip Mukherjee
Quoted 12 lines · Greg Kroah-Hartman
Quoted 11 lines · Sudip Mukherjee
Hi Greg,
Quoted 6 lines · Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit
7aa54be2976550f17c11a1c3e3630002dea39303upstream.Quoted 8 lines
Quoted 6 lines
Quoted 2 lines
Another upstream commit fixes this.
b987ffc18fb3("x86/qspinlock: Fix compile error")Maybe, but that commit doesn't apply to any of these stable trees :(
Care to provide a backport?
Attached now.
Quoted 8 lines
Are you sure that it fails to compile without that patch? I have here Debian's gcc version 8.2.0 which probably isn't affected and I can compile
kernel/locking/in v4.19 + 4.14.I'm asking because in my backport the
GEN_BINARY_RMWccmacro is used like in all the other functions which use it - unlike like in the original commit where the macro is used directly in the if condition. So it might not be affected by the problem.
ofcourse.. and I overlooked that part. Sorry.
The original problem was using the "goto" inside if() and with your backported patch the problem should not exist.
Greg, please do not add it to your queue.
Signature · 2 lines
Regards Sudip
Greg Kroah-Hartman
maintainerPATCH 30/72 elevator: lookup mq vs non-mq elevators
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 2527d99789e248576ac8081530cd4fd88730f8c7 ]
If an IO scheduler is selected via elevator= and it doesn't match the driver in question wrt blk-mq support, then we fail to boot.
The elevator= parameter is deprecated and only supported for non-mq devices. Augment the elevator lookup API so that we pass in if we're looking for an mq capable scheduler or not, so that we only ever return a valid type for the queue in question.
--- block/elevator.c | 44 +++++++++++++++++++++----------------------- 1 file changed, 21 insertions(+), 23 deletions(-)
block/elevator.c +21 -23
@@ -83,12 +83,15 @@ bool elv_bio_merge_ok(struct request *rq, struct bio *bio) }EXPORT_SYMBOL (elv_bio_merge_ok); -static struct elevator_type *elevator_find (const char *name) +/* +* Return scheduler with name 'name' and with matching 'mq capability +*/ +static struct elevator_type *elevator_find (const char *name,bool mq) {struct elevator_type *e;list_for_each_entry (e, &elv_list, list) { -if (!strcmp (e->elevator_name, name)) +if (!strcmp (e->elevator_name, name) && (mq == e->uses_mq))return e; }
@@ -100,25 +103,25 @@ static void elevator_put(struct elevator_type *e)module_put (e->elevator_owner); } -static struct elevator_type *elevator_get (const char *name,bool try_loading) +static struct elevator_type *elevator_get (struct request_queue *q, +const char *name,bool try_loading) {struct elevator_type *e;spin_lock (&elv_list_lock); - e =elevator_find (name); + e =elevator_find (name, q->mq_ops !=NULL );if (!e && try_loading) {spin_unlock (&elv_list_lock);request_module ("%s-iosched" , name);spin_lock (&elv_list_lock); - e =elevator_find (name); + e =elevator_find (name, q->mq_ops !=NULL ); }if (e && !try_module_get (e->elevator_owner)) e =NULL ;spin_unlock (&elv_list_lock); -return e; }
@@ -144,8 +147,12 @@ void __init load_default_elevator_module(void)if (!chosen_elevator[0 ])return ; +/* +* Boot parameter is deprecated, we haven't supported that for MQ. +* Only look for non-mq schedulers from here. +*/ spin_lock (&elv_list_lock); - e =elevator_find (chosen_elevator); + e =elevator_find (chosen_elevator, false);spin_unlock (&elv_list_lock);if (!e)
@@ -202,7 +209,7 @@ int elevator_init(struct request_queue *q, char *name) q->boundary_rq =NULL ;if (name) { - e =elevator_get (name, true); + e =elevator_get (q, name, true);if (!e)return -EINVAL ; }
@@ -214,7 +221,7 @@ int elevator_init(struct request_queue *q, char *name)* allowed from async. */ if (!e && !q->mq_ops && *chosen_elevator) { - e =elevator_get (chosen_elevator, false); + e =elevator_get (q, chosen_elevator, false);if (!e)printk (KERN_ERR "I/O scheduler %s not found\n" , chosen_elevator);
@@ -229,17 +236,17 @@ int elevator_init(struct request_queue *q, char *name)*/ if (q->mq_ops) {if (q->nr_hw_queues ==1 ) - e =elevator_get ("mq-deadline" , false); + e =elevator_get (q,"mq-deadline" , false);if (!e)return 0 ; }else - e =elevator_get (CONFIG_DEFAULT_IOSCHED , false); + e =elevator_get (q,CONFIG_DEFAULT_IOSCHED , false);if (!e) {printk (KERN_ERR "Default I/O scheduler not found. " \"Using noop.\n" ); - e =elevator_get ("noop" , false); + e =elevator_get (q,"noop" , false); } }
@@ -905,7 +912,7 @@ int elv_register(struct elevator_type *e)/* register, don't allow duplicate names */ spin_lock (&elv_list_lock); -if (elevator_find (e->elevator_name)) { +if (elevator_find (e->elevator_name, e->uses_mq)) {spin_unlock (&elv_list_lock);if (e->icq_cache)kmem_cache_destroy (e->icq_cache);
@@ -1066,7 +1073,7 @@ static int __elevator_change(struct request_queue *q, const char *name)return elevator_switch (q,NULL );strlcpy (elevator_name, name,sizeof (elevator_name)); - e =elevator_get (strstrip (elevator_name), true); + e =elevator_get (q,strstrip (elevator_name), true);if (!e)return -EINVAL ;
@@ -1076,15 +1083,6 @@ static int __elevator_change(struct request_queue *q, const char *name)return 0 ; } -if (!e->uses_mq && q->mq_ops) { -elevator_put (e); -return -EINVAL ; - } -if (e->uses_mq && !q->mq_ops) { -elevator_put (e); -return -EINVAL ; - } -return elevator_switch (q, e); }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 31/72 netfilter: ipset: Fix wraparound in hash:*net* types
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 0b8d9073539e217f79ec1bff65eb205ac796723d ]
Fix wraparound bug which could lead to memory exhaustion when adding an x.x.x.x-255.255.255.255 range to any hash:*net* types.
Fixes Netfilter's bugzilla id #1212, reported by Thomas Schwark.
--- net/netfilter/ipset/ip_set_hash_ipportnet.c | 26 +++++++-------- net/netfilter/ipset/ip_set_hash_net.c | 9 +++-- net/netfilter/ipset/ip_set_hash_netiface.c | 9 +++-- net/netfilter/ipset/ip_set_hash_netnet.c | 28 ++++++++-------- net/netfilter/ipset/ip_set_hash_netport.c | 19 ++++++----- net/netfilter/ipset/ip_set_hash_netportnet.c | 35 ++++++++++---------- 6 files changed, 63 insertions(+), 63 deletions(-)
net/netfilter/ipset/ip_set_hash_ipportnet.c +13 -13
@@ -168,7 +168,7 @@ hash_ipportnet4_uadt(struct ip_set *set, struct nlattr *tb[],struct hash_ipportnet4_elem e = { .cidr =HOST_MASK -1 };struct ip_set_ext ext =IP_SET_INIT_UEXT (set);u32 ip =0 , ip_to =0 , p =0 , port, port_to; -u32 ip2_from =0 , ip2_to =0 , ip2_last, ip2; +u32 ip2_from =0 , ip2_to =0 , ip2;bool with_ports = false;u8 cidr;int ret;
@@ -269,22 +269,21 @@ hash_ipportnet4_uadt(struct ip_set *set, struct nlattr *tb[],ip_set_mask_from_to (ip2_from, ip2_to, e.cidr +1 ); } -if (retried) +if (retried) { ip =ntohl (h->next.ip); + p =ntohs (h->next.port); + ip2 =ntohl (h->next.ip2); + }else { + p = port; + ip2 = ip2_from; + }for (; ip <= ip_to; ip++) { e.ip =htonl (ip); - p = retried && ip ==ntohl (h->next.ip) ?ntohs (h->next.port) - : port;for (; p <= port_to; p++) { e.port =htons (p); - ip2 = retried && - ip ==ntohl (h->next.ip) && - p ==ntohs (h->next.port) - ?ntohl (h->next.ip2) : ip2_from; -while (ip2 <= ip2_to) { + do { e.ip2 = htonl(ip2); - ip2_last =ip_set_range_to_cidr (ip2, ip2_to, - &cidr); + ip2 = ip_set_range_to_cidr(ip2, ip2_to, &cidr); e.cidr = cidr - 1; ret = adtfn(set, &e, &ext, &ext, flags);
@@ -292,9 +291,10 @@ hash_ipportnet4_uadt(struct ip_set *set, struct nlattr *tb[],return ret; ret =0 ; - ip2 = ip2_last +1 ; - } + }while (ip2++ < ip2_to); + ip2 = ip2_from; } + p = port; }return ret; }
net/netfilter/ipset/ip_set_hash_net.c +4 -5
@@ -143,7 +143,7 @@ hash_net4_uadt(struct ip_set *set, struct nlattr *tb[],ipset_adtfn adtfn = set->variant->adt[adt];struct hash_net4_elem e = { .cidr =HOST_MASK };struct ip_set_ext ext =IP_SET_INIT_UEXT (set); -u32 ip =0 , ip_to =0 , last; +u32 ip =0 , ip_to =0 ;int ret;if (tb[IPSET_ATTR_LINENO ])
@@ -193,16 +193,15 @@ hash_net4_uadt(struct ip_set *set, struct nlattr *tb[], }if (retried) ip =ntohl (h->next.ip); -while (ip <= ip_to) { +do { e.ip =htonl (ip); - last =ip_set_range_to_cidr (ip, ip_to, &e.cidr); + ip =ip_set_range_to_cidr (ip, ip_to, &e.cidr); ret =adtfn (set, &e, &ext, &ext, flags);if (ret && !ip_set_eexist (ret, flags))return ret; ret =0 ; - ip = last +1 ; - } + }while (ip++ < ip_to);return ret; }
net/netfilter/ipset/ip_set_hash_netiface.c +4 -5
@@ -200,7 +200,7 @@ hash_netiface4_uadt(struct ip_set *set, struct nlattr *tb[],ipset_adtfn adtfn = set->variant->adt[adt];struct hash_netiface4_elem e = { .cidr =HOST_MASK , .elem =1 };struct ip_set_ext ext =IP_SET_INIT_UEXT (set); -u32 ip =0 , ip_to =0 , last; +u32 ip =0 , ip_to =0 ;int ret;if (tb[IPSET_ATTR_LINENO ])
@@ -255,17 +255,16 @@ hash_netiface4_uadt(struct ip_set *set, struct nlattr *tb[],if (retried) ip =ntohl (h->next.ip); -while (ip <= ip_to) { +do { e.ip =htonl (ip); - last =ip_set_range_to_cidr (ip, ip_to, &e.cidr); + ip =ip_set_range_to_cidr (ip, ip_to, &e.cidr); ret =adtfn (set, &e, &ext, &ext, flags);if (ret && !ip_set_eexist (ret, flags))return ret; ret =0 ; - ip = last +1 ; - } + }while (ip++ < ip_to);return ret; }
net/netfilter/ipset/ip_set_hash_netnet.c +14 -14
@@ -169,8 +169,8 @@ hash_netnet4_uadt(struct ip_set *set, struct nlattr *tb[],ipset_adtfn adtfn = set->variant->adt[adt];struct hash_netnet4_elem e = { };struct ip_set_ext ext =IP_SET_INIT_UEXT (set); -u32 ip =0 , ip_to =0 , last; -u32 ip2 =0 , ip2_from =0 , ip2_to =0 , last2; +u32 ip =0 , ip_to =0 ; +u32 ip2 =0 , ip2_from =0 , ip2_to =0 ;int ret;if (tb[IPSET_ATTR_LINENO ])
@@ -247,27 +247,27 @@ hash_netnet4_uadt(struct ip_set *set, struct nlattr *tb[],ip_set_mask_from_to (ip2_from, ip2_to, e.cidr[1 ]); } -if (retried) +if (retried) { ip =ntohl (h->next.ip[0 ]); + ip2 =ntohl (h->next.ip[1 ]); + }else { + ip2 = ip2_from; + } -while (ip <= ip_to) { +do { e.ip[0 ] =htonl (ip); - last =ip_set_range_to_cidr (ip, ip_to, &e.cidr[0 ]); - ip2 = (retried && - ip ==ntohl (h->next.ip[0 ])) ?ntohl (h->next.ip[1 ]) - : ip2_from; -while (ip2 <= ip2_to) { + ip =ip_set_range_to_cidr (ip, ip_to, &e.cidr[0 ]); +do { e.ip[1 ] =htonl (ip2); - last2 =ip_set_range_to_cidr (ip2, ip2_to, &e.cidr[1 ]); + ip2 =ip_set_range_to_cidr (ip2, ip2_to, &e.cidr[1 ]); ret =adtfn (set, &e, &ext, &ext, flags);if (ret && !ip_set_eexist (ret, flags))return ret; ret =0 ; - ip2 = last2 +1 ; - } - ip = last +1 ; - } + }while (ip2++ < ip2_to); + ip2 = ip2_from; + }while (ip++ < ip_to);return ret; }
net/netfilter/ipset/ip_set_hash_netport.c +10 -9
@@ -161,7 +161,7 @@ hash_netport4_uadt(struct ip_set *set, struct nlattr *tb[],ipset_adtfn adtfn = set->variant->adt[adt];struct hash_netport4_elem e = { .cidr =HOST_MASK -1 };struct ip_set_ext ext =IP_SET_INIT_UEXT (set); -u32 port, port_to, p =0 , ip =0 , ip_to =0 , last; +u32 port, port_to, p =0 , ip =0 , ip_to =0 ;bool with_ports = false;u8 cidr;int ret;
@@ -239,25 +239,26 @@ hash_netport4_uadt(struct ip_set *set, struct nlattr *tb[],ip_set_mask_from_to (ip, ip_to, e.cidr +1 ); } -if (retried) +if (retried) { ip =ntohl (h->next.ip); -while (ip <= ip_to) { + p =ntohs (h->next.port); + }else { + p = port; + } +do { e.ip =htonl (ip); - last =ip_set_range_to_cidr (ip, ip_to, &cidr); + ip =ip_set_range_to_cidr (ip, ip_to, &cidr); e.cidr = cidr -1 ; - p = retried && ip ==ntohl (h->next.ip) ?ntohs (h->next.port) - : port;for (; p <= port_to; p++) { e.port =htons (p); ret =adtfn (set, &e, &ext, &ext, flags); -if (ret && !ip_set_eexist (ret, flags))return ret; ret =0 ; } - ip = last +1 ; - } + p = port; + }while (ip++ < ip_to);return ret; }
net/netfilter/ipset/ip_set_hash_netportnet.c +18 -17
@@ -184,8 +184,8 @@ hash_netportnet4_uadt(struct ip_set *set, struct nlattr *tb[],ipset_adtfn adtfn = set->variant->adt[adt];struct hash_netportnet4_elem e = { };struct ip_set_ext ext =IP_SET_INIT_UEXT (set); -u32 ip =0 , ip_to =0 , ip_last, p =0 , port, port_to; -u32 ip2_from =0 , ip2_to =0 , ip2_last, ip2; +u32 ip =0 , ip_to =0 , p =0 , port, port_to; +u32 ip2_from =0 , ip2_to =0 , ip2;bool with_ports = false;int ret;
@@ -288,33 +288,34 @@ hash_netportnet4_uadt(struct ip_set *set, struct nlattr *tb[],ip_set_mask_from_to (ip2_from, ip2_to, e.cidr[1 ]); } -if (retried) +if (retried) { ip =ntohl (h->next.ip[0 ]); + p =ntohs (h->next.port); + ip2 =ntohl (h->next.ip[1 ]); + }else { + p = port; + ip2 = ip2_from; + } -while (ip <= ip_to) { +do { e.ip[0 ] =htonl (ip); - ip_last =ip_set_range_to_cidr (ip, ip_to, &e.cidr[0 ]); - p = retried && ip ==ntohl (h->next.ip[0 ]) ?ntohs (h->next.port) - : port; + ip =ip_set_range_to_cidr (ip, ip_to, &e.cidr[0 ]);for (; p <= port_to; p++) { e.port =htons (p); - ip2 = (retried && ip ==ntohl (h->next.ip[0 ]) && - p ==ntohs (h->next.port)) ?ntohl (h->next.ip[1 ]) - : ip2_from; -while (ip2 <= ip2_to) { +do { e.ip[1 ] =htonl (ip2); - ip2_last =ip_set_range_to_cidr (ip2, ip2_to, - &e.cidr[1 ]); + ip2 =ip_set_range_to_cidr (ip2, ip2_to, + &e.cidr[1 ]); ret =adtfn (set, &e, &ext, &ext, flags);if (ret && !ip_set_eexist (ret, flags))return ret; ret =0 ; - ip2 = ip2_last +1 ; - } + }while (ip2++ < ip2_to); + ip2 = ip2_from; } - ip = ip_last +1 ; - } + p = port; + }while (ip++ < ip_to);return ret; }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 32/72 mac80211: dont WARN on bad WMM parameters from buggy APs
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit c470bdc1aaf36669e04ba65faf1092b2d1c6cabe ]
Apparently, some APs are buggy enough to send a zeroed WMM IE. Don't WARN on this since this is not caused by a bug on the client's system.
This aligns the condition of the WARNING in drv_conf_tx with the validity check in ieee80211_sta_wmm_params. We will now pick the default values whenever we get a zeroed WMM IE.
This has been reported here:
https://bugzilla.kernel.org/show_bug.cgi?id=199161
--- net/mac80211/mlme.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
net/mac80211/mlme.c +2 -1
@@ -1861,7 +1861,8 @@ static bool ieee80211_sta_wmm_params(struct ieee80211_local *local, params[ac].acm = acm; params[ac].uapsd = uapsd; -if (params[ac].cw_min > params[ac].cw_max) { +if (params->cw_min ==0 || + params[ac].cw_min > params[ac].cw_max) {sdata_info (sdata,"AP has invalid WMM params (CWmin/max=%d/%d for ACI %d), using defaults\n" , params[ac].cw_min, params[ac].cw_max, aci);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 33/72 mac80211: Fix condition validating WMM IE
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 911a26484c33e10de6237228ca1d7293548e9f49 ]
Commit c470bdc1aaf3 ("mac80211: don't WARN on bad WMM parameters from buggy APs") handled cases where an AP reports a zeroed WMM IE. However, the condition that checks the validity accessed the wrong index in the ieee80211_tx_queue_params array, thus wrongly deducing that the parameters are invalid. Fix it.
--- net/mac80211/mlme.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/mac80211/mlme.c +1 -1
@@ -1861,7 +1861,7 @@ static bool ieee80211_sta_wmm_params(struct ieee80211_local *local, params[ac].acm = acm; params[ac].uapsd = uapsd; -if (params->cw_min ==0 || +if (params[ac].cw_min ==0 || params[ac].cw_min > params[ac].cw_max) { sdata_info(sdata, "AP has invalid WMM params (CWmin/max=%d/%d for ACI %d), using defaults\n",
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 34/72 IB/hfi1: Remove race conditions in user_sdma send path
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
commit 28a9a9e83ceae2cee25b9af9ad20d53aaa9ab951 upstream
Packet queue state is over used to determine SDMA descriptor availablitity and packet queue request state.
cpu 0 ret = user_sdma_send_pkts(req, pcount); cpu 0 if (atomic_read(&pq->n_reqs)) cpu 1 IRQ user_sdma_txreq_cb calls pq_update() (state to _INACTIVE)
cpu 0 xchg(&pq->state, SDMA_PKT_Q_ACTIVE);
At this point pq->n_reqs == 0 and pq->state is incorrectly SDMA_PKT_Q_ACTIVE. The close path will hang waiting for the state to return to _INACTIVE.
This can also change the state from _DEFERRED to _ACTIVE. However, this is a mostly benign race.
Remove the racy code path.
Use n_reqs to determine if a packet queue is active or not.
--- drivers/infiniband/hw/hfi1/user_sdma.c | 24 ++++++++++-------------- drivers/infiniband/hw/hfi1/user_sdma.h | 9 +++++---- 2 files changed, 15 insertions(+), 18 deletions(-)
drivers/infiniband/hw/hfi1/user_sdma.c +10 -14
@@ -187,7 +187,6 @@ int hfi1_user_sdma_alloc_queues(struct hfi1_ctxtdata *uctxt, pq->ctxt = uctxt->ctxt; pq->subctxt = fd->subctxt; pq->n_max_reqs = hfi1_sdma_comp_ring_size; - pq->state =SDMA_PKT_Q_INACTIVE ;atomic_set (&pq->n_reqs,0 );init_waitqueue_head (&pq->wait);atomic_set (&pq->n_locked,0 );
@@ -276,7 +275,7 @@ int hfi1_user_sdma_free_queues(struct hfi1_filedata *fd,/* Wait until all requests have been freed. */ wait_event_interruptible ( pq->wait, - (ACCESS_ONCE (pq->state) ==SDMA_PKT_Q_INACTIVE )); + !atomic_read (&pq->n_reqs));kfree (pq->reqs);kfree (pq->req_in_use);kmem_cache_destroy (pq->txreq_cache);
@@ -312,6 +311,13 @@ static u8 dlid_to_selector(u16 dlid)return mapping[hash]; } +/** +* hfi1_user_sdma_process_request() - Process and start a user sdma request +* @fd: valid file descriptor +* @iovec: array of io vectors to process +* @dim: overall iovec array size +* @count: number of io vector array entries processed +*/ int hfi1_user_sdma_process_request (struct hfi1_filedata *fd,struct iovec *iovec,unsigned long dim,unsigned long *count)
@@ -560,20 +566,12 @@ int hfi1_user_sdma_process_request(struct hfi1_filedata *fd, req->ahg_idx =sdma_ahg_alloc (req->sde);set_comp_state (pq, cq, info.comp_idx,QUEUED ,0 ); + pq->state =SDMA_PKT_Q_ACTIVE ;/* Send the first N packets in the request to buy us some time */ ret =user_sdma_send_pkts (req, pcount);if (unlikely (ret <0 && ret != -EBUSY )) goto free_req; -/* -* It is possible that the SDMA engine would have processed all the -* submitted packets by the time we get here. Therefore, only set -* packet queue state to ACTIVE if there are still uncompleted -* requests. -*/ -if (atomic_read (&pq->n_reqs)) -xchg (&pq->state,SDMA_PKT_Q_ACTIVE ); -/* * This is a somewhat blocking send implementation. * The driver will block the caller until all packets of the
@@ -1391,10 +1389,8 @@ static void user_sdma_txreq_cb(struct sdma_txreq *txreq, int status)static inline void pq_update (struct hfi1_user_sdma_pkt_q *pq) { -if (atomic_dec_and_test (&pq->n_reqs)) { -xchg (&pq->state,SDMA_PKT_Q_INACTIVE ); +if (atomic_dec_and_test (&pq->n_reqs))wake_up (&pq->wait); - } }static void user_sdma_free_request (struct user_sdma_request *req,bool unpin)
drivers/infiniband/hw/hfi1/user_sdma.h +5 -4
@@ -94,9 +94,10 @@#define TXREQ_FLAGS_REQ_ACK BIT(0)/* Set the ACK bit in the header */ #define TXREQ_FLAGS_REQ_DISABLE_SH BIT(1)/* Disable header suppression */ -#define SDMA_PKT_Q_INACTIVE BIT(0) -#define SDMA_PKT_Q_ACTIVE BIT(1) -#define SDMA_PKT_Q_DEFERRED BIT(2) +enum pkt_q_sdma_state { +SDMA_PKT_Q_ACTIVE , +SDMA_PKT_Q_DEFERRED , +};/* * Maximum retry attempts to submit a TX request
@@ -124,7 +125,7 @@ struct hfi1_user_sdma_pkt_q {struct user_sdma_request *reqs;unsigned long *req_in_use;struct iowait busy; -unsigned state; +enum pkt_q_sdma_state state;wait_queue_head_t wait;unsigned long unpinned;struct mmu_rb_handler *handler;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 35/72 locking/qspinlock: Fix build for anonymous union in older GCC compilers
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 6cc65be4f6f2a7186af8f3e09900787c7912dad2 ]
One of my tests compiles the kernel with gcc 4.5.3, and I hit the following build error:
include/linux/semaphore.h: In function 'sema_init':
include/linux/semaphore.h:35:17: error: unknown field 'val' specified in initializer include/linux/semaphore.h:35:17: warning: missing braces around initializer include/linux/semaphore.h:35:17: warning: (near initialization for '(anonymous).raw_lock.<anonymous>.val')
I bisected it down to:
625e88be1f41 ("locking/qspinlock: Merge 'struct __qspinlock' into 'struct qspinlock'")
... which makes qspinlock have an anonymous union, which makes initializing it special for older compilers. By adding strategic brackets, it makes the build happy again.
--- include/asm-generic/qspinlock_types.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
include/asm-generic/qspinlock_types.h +1 -1
@@ -63,7 +63,7 @@ typedef struct qspinlock {/* * Initializier */ -#define __ARCH_SPIN_LOCK_UNLOCKED { .val = ATOMIC_INIT(0) } +#define __ARCH_SPIN_LOCK_UNLOCKED { { .val = ATOMIC_INIT(0) } }/* * Bitfields in the atomic value:
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 36/72 mac80211_hwsim: fix module init error paths for netlink
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 05cc09de4c017663a217630682041066f2f9a5cd ]
There is no unregister netlink notifier and family on error paths in init_mac80211_hwsim(). Also there is an error path where hwsim_class is not destroyed.
Found by Linux Driver Verification project (linuxtesting.org).
--- drivers/net/wireless/mac80211_hwsim.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-)
drivers/net/wireless/mac80211_hwsim.c +7 -5
@@ -3472,16 +3472,16 @@ static int __init init_mac80211_hwsim(void)if (err) goto out_unregister_pernet; + err =hwsim_init_netlink (); +if (err) + goto out_unregister_driver; + hwsim_class =class_create (THIS_MODULE ,"mac80211_hwsim" );if (IS_ERR (hwsim_class)) { err =PTR_ERR (hwsim_class); - goto out_unregister_driver; + goto out_exit_netlink; } - err =hwsim_init_netlink (); -if (err <0 ) - goto out_unregister_driver; -for (i =0 ; i < radios; i++) {struct hwsim_new_radio_params param = {0 };
@@ -3587,6 +3587,8 @@ out_free_mon:free_netdev (hwsim_mon); out_free_radios:mac80211_hwsim_free (); +out_exit_netlink: +hwsim_exit_netlink (); out_unregister_driver:platform_driver_unregister (&mac80211_hwsim_driver); out_unregister_pernet:
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 37/72 Input: hyper-v - fix wakeup from suspend-to-idle
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 10f91c73cc41ceead210a905dbd196398e99c7d2 ]
It makes little sense but still possible to put Hyper-V guests into suspend-to-idle state. To wake them up two wakeup sources were registered in the past: hyperv-keyboard and hid-hyperv. However, since commit eed4d47efe95 ("ACPI / sleep: Ignore spurious SCI wakeups from suspend-to-idle") pm_wakeup_event() from these devices is ignored. Switch to pm_wakeup_hard_event() API as these devices are actually the only possible way to wakeup Hyper-V guests.
--- drivers/hid/hid-hyperv.c | 2 +- drivers/input/serio/hyperv-keyboard.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/hid/hid-hyperv.c +1 -1
@@ -309,7 +309,7 @@ static void mousevsc_on_receive(struct hv_device *device,hid_input_report (input_dev->hid_device,HID_INPUT_REPORT , input_dev->input_buf, len,1 ); -pm_wakeup_event (&input_dev->device->device,0 ); +pm_wakeup_hard_event (&input_dev->device->device);break ;default :
drivers/input/serio/hyperv-keyboard.c +1 -1
@@ -177,7 +177,7 @@ static void hv_kbd_on_receive(struct hv_device *hv_dev,* state because the Enter-UP can trigger a wakeup at once. */ if (!(info &IS_BREAK )) -pm_wakeup_event (&hv_dev->device,0 ); +pm_wakeup_hard_event (&hv_dev->device);break ;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 38/72 scsi: libiscsi: Fix NULL pointer dereference in iscsi_eh_session_reset
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 5db6dd14b31397e8cccaaddab2ff44ebec1acf25 ]
This commit addresses NULL pointer dereference in iscsi_eh_session_reset. Reference should not be made to session->leadconn when session->state is set to ISCSI_STATE_TERMINATE.
--- drivers/scsi/libiscsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/scsi/libiscsi.c +2 -2
@@ -2416,8 +2416,8 @@ int iscsi_eh_session_reset(struct scsi_cmnd *sc) failed:ISCSI_DBG_EH (session,"failing session reset: Could not log back into " -"%s, %s [age %d]\n" , session->targetname, - conn->persistent_address, session->age); +"%s [age %d]\n" , session->targetname, + session->age);spin_unlock_bh (&session->frwd_lock);mutex_unlock (&session->eh_mutex);return FAILED ;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 39/72 scsi: vmw_pscsi: Rearrange code to avoid multiple calls to free_irq during unload
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 02f425f811cefcc4d325d7a72272651e622dc97e ]
Currently pvscsi_remove calls free_irq more than once as pvscsi_release_resources and __pvscsi_shutdown both call pvscsi_shutdown_intr. This results in a 'Trying to free already-free IRQ' warning and stack trace. To solve the problem pvscsi_shutdown_intr has been moved out of pvscsi_release_resources.
--- drivers/scsi/vmw_pvscsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/scsi/vmw_pvscsi.c +2 -2
@@ -1202,8 +1202,6 @@ static void pvscsi_shutdown_intr(struct pvscsi_adapter *adapter)static void pvscsi_release_resources (struct pvscsi_adapter *adapter) { -pvscsi_shutdown_intr (adapter); -if (adapter->workqueue)destroy_workqueue (adapter->workqueue);
@@ -1535,6 +1533,7 @@ static int pvscsi_probe(struct pci_dev *pdev, const struct pci_device_id *id) out_reset_adapter:ll_adapter_reset (adapter); out_release_resources: +pvscsi_shutdown_intr (adapter);pvscsi_release_resources (adapter);scsi_host_put (host); out_disable_device:
@@ -1543,6 +1542,7 @@ out_disable_device:return error; out_release_resources_and_disable: +pvscsi_shutdown_intr (adapter);pvscsi_release_resources (adapter); goto out_disable_device; }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 40/72 x86/earlyprintk/efi: Fix infinite loop on some screen widths
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 79c2206d369b87b19ac29cb47601059b6bf5c291 ]
An affected screen resolution is 1366 x 768, which width is not divisible by 8, the default font width. On such screens, when longer lines are earlyprintk'ed, overflow-to-next-line can never trigger, due to the left-most x-coordinate of the next character always less than the screen width. Earlyprintk will infinite loop in trying to print the rest of the string but unable to, due to the line being full.
This patch makes the trigger consider the right-most x-coordinate, instead of left-most, as the value to compare against the screen width threshold.
--- arch/x86/platform/efi/early_printk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/x86/platform/efi/early_printk.c +1 -1
@@ -179,7 +179,7 @@ early_efi_write(struct console *con, const char *str, unsigned int num) num--; } -if (efi_x >= si->lfb_width) { +if (efi_x + font->width > si->lfb_width) { efi_x =0 ; efi_y += font->height; }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 41/72 drm/msm: Grab a vblank reference when waiting for commit_done
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 3b712e43e3876b42b38321ecf790a1f5fe59c834 ]
Similar to the atomic helpers, we should enable vblank while we're waiting for the commit to finish. DPU needs this, MDP5 seems to work fine without it.
--- drivers/gpu/drm/msm/msm_atomic.c | 5 +++++ 1 file changed, 5 insertions(+)
drivers/gpu/drm/msm/msm_atomic.c +5 -0
@@ -93,7 +93,12 @@ static void msm_atomic_wait_for_commit_done(struct drm_device *dev,if (!new_crtc_state->active)continue ; +if (drm_crtc_vblank_get (crtc)) +continue ; + kms->funcs->wait_for_crtc_commit_done (kms, crtc); + +drm_crtc_vblank_put (crtc); } }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 42/72 ARC: io.h: Implement reads{x}()/writes{x}()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 10d443431dc2bb733cf7add99b453e3fb9047a2e ]
Some ARC CPU's do not support unaligned loads/stores. Currently, generic implementation of reads{b/w/l}()/writes{b/w/l}() is being used with ARC. This can lead to misfunction of some drivers as generic functions do a plain dereference of a pointer that can be unaligned.
Let's use {get/put}_unaligned() helpers instead of plain dereference of pointer in order to fix. The helpers allow to get and store data from an unaligned address whilst preserving the CPU internal alignment. According to [1], the use of these helpers are costly in terms of performance so we added an initial check for a buffer already aligned so that the usage of the helpers can be avoided, when possible.
[1] Documentation/unaligned-memory-access.txt
--- arch/arc/include/asm/io.h | 72 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 72 insertions(+)
arch/arc/include/asm/io.h +72 -0
@@ -12,6 +12,7 @@#include <linux/types.h> #include <asm/byteorder.h> #include <asm/page.h> +#include <asm/unaligned.h> #ifdef CONFIG_ISA_ARCV2 #include <asm/barrier.h>
@@ -94,6 +95,42 @@ static inline u32 __raw_readl(const volatile void __iomem *addr)return w; } +/* +* {read,write}s{b,w,l}() repeatedly access the same IO address in +* native endianness in 8-, 16-, 32-bit chunks {into,from} memory, +* @count times +*/ +#define __raw_readsx (t,f) \ +static inline void __raw_reads##f(const volatile void __iomem *addr, \ + void *ptr, unsigned int count) \ +{ \ + bool is_aligned = ((unsigned long)ptr % ((t) / 8)) == 0; \ + u##t *buf = ptr; \ + \ + if (!count) \ + return; \ + \ +/* Some ARC CPU's don't support unaligned accesses */ \ + if (is_aligned) { \ + do { \ + u##t x = __raw_read##f(addr); \ + *buf++ = x; \ + } while (--count); \ + } else { \ + do { \ + u##t x = __raw_read##f(addr); \ + put_unaligned(x, buf++); \ + } while (--count); \ + } \ +} + +#define __raw_readsb __raw_readsb +__raw_readsx(8, b) +#define __raw_readsw __raw_readsw +__raw_readsx(16, w) +#define __raw_readsl __raw_readsl +__raw_readsx(32, l) + #define __raw_writeb __raw_writeb static inline void __raw_writeb(u8 b, volatile void __iomem *addr) {
@@ -126,6 +163,35 @@ static inline void __raw_writel(u32 w, volatile void __iomem *addr) } +#define __raw_writesx (t,f) \ +static inline void __raw_writes##f(volatile void __iomem *addr, \ + const void *ptr, unsigned int count) \ +{ \ + bool is_aligned = ((unsigned long)ptr % ((t) / 8)) == 0; \ + const u##t *buf = ptr; \ + \ + if (!count) \ + return; \ + \ +/* Some ARC CPU's don't support unaligned accesses */ \ + if (is_aligned) { \ + do { \ + __raw_write##f(*buf++, addr); \ + } while (--count); \ + } else { \ + do { \ + __raw_write##f(get_unaligned(buf++), addr); \ + } while (--count); \ + } \ +} + +#define __raw_writesb __raw_writesb +__raw_writesx(8, b) +#define __raw_writesw __raw_writesw +__raw_writesx(16, w) +#define __raw_writesl __raw_writesl +__raw_writesx(32, l) + /* * MMIO can also get buffered/optimized in micro-arch, so barriers needed * Based on ARM model for the typical use case
@@ -141,10 +207,16 @@ static inline void __raw_writel(u32 w, volatile void __iomem *addr)#define readb (c) ({ u8 __v = readb_relaxed(c); __iormb(); __v; })#define readw (c) ({ u16 __v = readw_relaxed(c); __iormb(); __v; })#define readl (c) ({ u32 __v = readl_relaxed(c); __iormb(); __v; }) +#define readsb (p,d,l) ({ __raw_readsb(p,d,l); __iormb(); }) +#define readsw (p,d,l) ({ __raw_readsw(p,d,l); __iormb(); }) +#define readsl (p,d,l) ({ __raw_readsl(p,d,l); __iormb(); })#define writeb (v,c) ({ __iowmb(); writeb_relaxed(v,c); })#define writew (v,c) ({ __iowmb(); writew_relaxed(v,c); })#define writel (v,c) ({ __iowmb(); writel_relaxed(v,c); }) +#define writesb (p,d,l) ({ __iowmb(); __raw_writesb(p,d,l); }) +#define writesw (p,d,l) ({ __iowmb(); __raw_writesw(p,d,l); }) +#define writesl (p,d,l) ({ __iowmb(); __raw_writesl(p,d,l); })/* * Relaxed API for drivers which can handle barrier ordering themselves
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 43/72 bonding: fix 802.3ad state sent to partner when unbinding slave
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 3b5b3a3331d141e8f2a7aaae3a94dfa1e61ecbe4 ]
Previously when unbinding a slave the 802.3ad implementation only told partner that the port is not suitable for aggregation by setting the port aggregation state from aggregatable to individual. This is not enough. If the physical layer still stays up and we only unbinded this port from the bond there is nothing in the aggregation status alone to prevent the partner from sending traffic towards us. To ensure that the partner doesn't consider this port at all anymore we should also disable collecting and distributing to signal that this actor is going away. Also clear AD_STATE_SYNCHRONIZATION to ensure partner exits collecting + distributing state.
I have tested this behaviour againts Arista EOS switches with mlx5 cards (physical link stays up even when interface is down) and simulated the same situation virtually Linux <-> Linux with two network namespaces running two veth device pairs. In both cases setting aggregation to individual doesn't alone prevent traffic from being to sent towards this port given that the link stays up in partners end. Partner still keeps it's end in collecting + distributing state and continues until timeout is reached. In most cases this means we are losing the traffic partner sends towards our port while we wait for timeout. This is most visible with slow periodic time (LACP rate slow).
Other open source implementations like Open VSwitch and libreswitch, and vendor implementations like Arista EOS, seem to disable collecting + distributing to when doing similar port disabling/detaching/removing change. With this patch kernel implementation would behave the same way and ensure partner doesn't consider our actor viable anymore.
--- drivers/net/bonding/bond_3ad.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/net/bonding/bond_3ad.c +3 -0
@@ -2086,6 +2086,9 @@ void bond_3ad_unbind_slave(struct slave *slave) aggregator->aggregator_identifier);/* Tell the partner that this port is not suitable for aggregation */ + port->actor_oper_port_state &= ~AD_STATE_SYNCHRONIZATION ; + port->actor_oper_port_state &= ~AD_STATE_COLLECTING ; + port->actor_oper_port_state &= ~AD_STATE_DISTRIBUTING ; port->actor_oper_port_state &= ~AD_STATE_AGGREGATION ;__update_lacpdu_from_port (port);ad_lacpdu_send (port);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 44/72 bpf: Fix verifier log string check for bad alignment.
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit c01ac66b38660f2b507ccd0b75d28e3002d56fbb ]
The message got changed a lot time ago.
This was responsible for 36 test case failures on sparc64.
--- tools/testing/selftests/bpf/test_verifier.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
tools/testing/selftests/bpf/test_verifier.c +1 -1
@@ -8070,7 +8070,7 @@ static void do_test_single(struct bpf_test *test, bool unpriv, reject_from_alignment = fd_prog <0 && (test->flags &F_NEEDS_EFFICIENT_UNALIGNED_ACCESS ) && -strstr (bpf_vlog,"Unknown alignment." ); +strstr (bpf_vlog,"misaligned" );#ifdef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS if (reject_from_alignment) { printf("FAIL\nFailed due to alignment despite having efficient unaligned access: '%s'!\n",
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 45/72 nfs: dont dirty kernel pages read by direct-io
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit ad3cba223ac02dc769c3bbe88efe277bbb457566 ]
When we use direct_IO with an NFS backing store, we can trigger a WARNING in __set_page_dirty(), as below, since we're dirtying the page unnecessarily in nfs_direct_read_completion().
To fix, replicate the logic in commit 53cbf3b157a0 ("fs: direct-io:
don't dirtying pages for ITER_BVEC/ITER_KVEC direct read").
Other filesystems that implement direct_IO handle this; most use blockdev_direct_IO(). ceph and cifs have similar logic.
mount 127.0.0.1:/export /nfs
dd if=/dev/zero of=/nfs/image bs=1M count=200
losetup --direct-io=on -f /nfs/image
mkfs.btrfs /dev/loop0
mount -t btrfs /dev/loop0 /mnt/
kernel: WARNING: CPU: 0 PID: 8067 at fs/buffer.c:580 __set_page_dirty+0xaf/0xd0 kernel: Modules linked in: loop(E) nfsv3(E) rpcsec_gss_krb5(E) nfsv4(E) dns_resolver(E) nfs(E) fscache(E) nfsd(E) auth_rpcgss(E) nfs_acl(E) lockd(E) grace(E) fuse(E) tun(E) ip6t_rpfilter(E) ipt_REJECT(E) nf_ kernel: snd_seq(E) snd_seq_device(E) snd_pcm(E) video(E) snd_timer(E) snd(E) soundcore(E) ip_tables(E) xfs(E) libcrc32c(E) sd_mod(E) sr_mod(E) cdrom(E) ata_generic(E) pata_acpi(E) crc32c_intel(E) ahci(E) li kernel: CPU: 0 PID: 8067 Comm: kworker/0:2 Tainted: G E 4.20.0-rc1.master.20181111.ol7.x86_64 #1 kernel: Hardware name: innotek GmbH VirtualBox/VirtualBox, BIOS VirtualBox 12/01/2006 kernel: Workqueue: nfsiod rpc_async_release [sunrpc] kernel: RIP: 0010:__set_page_dirty+0xaf/0xd0 kernel: Code: c3 48 8b 02 f6 c4 04 74 d4 48 89 df e8 ba 05 f7 ff 48 89 c6 eb cb 48 8b 43 08 a8 01 75 1f 48 89 d8 48 8b 00 a8 04 74 02 eb 87 <0f> 0b eb 83 48 83 e8 01 eb 9f 48 83 ea 01 0f 1f 00 eb 8b 48 83 e8 kernel: RSP: 0000:ffffc1c8825b7d78 EFLAGS: 00013046 kernel: RAX: 000fffffc0020089 RBX: fffff2b603308b80 RCX: 0000000000000001 kernel: RDX: 0000000000000001 RSI: ffff9d11478115c8 RDI: ffff9d11478115d0 kernel: RBP: ffffc1c8825b7da0 R08: 0000646f6973666e R09: 8080808080808080 kernel: R10: 0000000000000001 R11: 0000000000000000 R12: ffff9d11478115d0 kernel: R13: ffff9d11478115c8 R14: 0000000000003246 R15: 0000000000000001 kernel: FS: 0000000000000000(0000) GS:ffff9d115ba00000(0000) knlGS:0000000000000000 kernel: CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 kernel: CR2: 00007f408686f640 CR3: 0000000104d8e004 CR4: 00000000000606f0 kernel: DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 kernel: DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400 kernel: Call Trace: kernel: __set_page_dirty_buffers+0xb6/0x110 kernel: set_page_dirty+0x52/0xb0 kernel: nfs_direct_read_completion+0xc4/0x120 [nfs] kernel: nfs_pgio_release+0x10/0x20 [nfs] kernel: rpc_free_task+0x30/0x70 [sunrpc] kernel: rpc_async_release+0x12/0x20 [sunrpc] kernel: process_one_work+0x174/0x390 kernel: worker_thread+0x4f/0x3e0 kernel: kthread+0x102/0x140 kernel: ? drain_workqueue+0x130/0x130 kernel: ? kthread_stop+0x110/0x110 kernel: ret_from_fork+0x35/0x40 kernel: ---[ end trace 01341980905412c9 ]---
--- fs/nfs/direct.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
fs/nfs/direct.c +8 -1
@@ -98,8 +98,11 @@ struct nfs_direct_req {struct pnfs_ds_commit_info ds_cinfo;/* Storage for cinfo */ struct work_struct work;int flags; +/* for write */ #define NFS_ODIRECT_DO_COMMIT (1)/* an unstable reply was received */ #define NFS_ODIRECT_RESCHED_WRITES (2)/* write verification failed */ +/* for read */ +#define NFS_ODIRECT_SHOULD_DIRTY (3)/* dirty user-space page after read */ struct nfs_writeverf verf;/* unstable write verifier */ };
@@ -412,7 +415,8 @@ static void nfs_direct_read_completion(struct nfs_pgio_header *hdr)struct nfs_page *req =nfs_list_entry (hdr->pages.next);struct page *page = req->wb_page; -if (!PageCompound (page) && bytes < hdr->good_bytes) +if (!PageCompound (page) && bytes < hdr->good_bytes && + (dreq->flags ==NFS_ODIRECT_SHOULD_DIRTY ))set_page_dirty (page); bytes += req->wb_bytes;nfs_list_remove_request (req);
@@ -587,6 +591,9 @@ ssize_t nfs_file_direct_read(struct kiocb *iocb, struct iov_iter *iter)if (!is_sync_kiocb (iocb)) dreq->iocb = iocb; +if (iter_is_iovec (iter)) + dreq->flags =NFS_ODIRECT_SHOULD_DIRTY ; +nfs_start_io_direct (inode);NFS_I (inode)->read_io += count;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 46/72 SUNRPC: Fix a potential race in xprt_connect()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 0a9a4304f3614e25d9de9b63502ca633c01c0d70 ]
If an asynchronous connection attempt completes while another task is in xprt_connect(), then the call to rpc_sleep_on() could end up racing with the call to xprt_wake_pending_tasks().
So add a second test of the connection state after we've put the task to sleep and set the XPRT_CONNECTING flag, when we know that there can be no asynchronous connection attempts still in progress.
--- net/sunrpc/xprt.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-)
net/sunrpc/xprt.c +9 -2
@@ -780,8 +780,15 @@ void xprt_connect(struct rpc_task *task)return ;if (xprt_test_and_set_connecting (xprt))return ; - xprt->stat.connect_start = jiffies; - xprt->ops->connect (xprt, task); +/* Race breaker */ +if (!xprt_connected (xprt)) { + xprt->stat.connect_start = jiffies; + xprt->ops->connect (xprt, task); + }else { +xprt_clear_connecting (xprt); + task->tk_status =0 ; +rpc_wake_up_queued_task (&xprt->pending, task); + } }xprt_release_write (xprt, task); }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 47/72 sbus: char: add of_node_put()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 87d81a23e24f24ebe014891e8bdf3ff8785031e8 ]
use of_node_put() to release the refcount.
--- drivers/sbus/char/display7seg.c | 1 + 1 file changed, 1 insertion(+)
drivers/sbus/char/display7seg.c +1 -0
@@ -221,6 +221,7 @@ static int d7s_probe(struct platform_device *op)dev_set_drvdata (&op->dev, p); d7s_device = p; err =0 ; +of_node_put (opts); out:return err;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 48/72 drivers/sbus/char: add of_node_put()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 6bd520ab7cf69486ea81fd3cdfd2d5a390ad1100 ]
use of_node_put() to release the refcount.
--- drivers/sbus/char/envctrl.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/sbus/char/envctrl.c +2 -0
@@ -910,8 +910,10 @@ static void envctrl_init_i2c_child(struct device_node *dp,for (len =0 ; len <PCF8584_MAX_CHANNELS ; ++len) { pchild->mon_type[len] =ENVCTRL_NOMON ; } +of_node_put (root_node);return ; } +of_node_put (root_node); }/* Get the monitor channels. */
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 49/72 drivers/tty: add missing of_node_put()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit dac097c4546e4c5b16dd303a1e97c1d319c8ab3e ]
of_find_node_by_path() acquires a reference to the node returned by it and that reference needs to be dropped by its caller. This place is not doing this, so fix it.
--- drivers/tty/serial/suncore.c | 1 + 1 file changed, 1 insertion(+)
drivers/tty/serial/suncore.c +1 -0
@@ -111,6 +111,7 @@ void sunserial_console_termios(struct console *con, struct device_node *uart_dp) mode =of_get_property (dp, mode_prop,NULL );if (!mode) mode ="9600,8,n,1,-" ; +of_node_put (dp); } cflag =CREAD |HUPCL |CLOCAL ;
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 50/72 ide: pmac: add of_node_put()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit a51921c0db3fd26c4ed83dc0ec5d32988fa02aa5 ]
use of_node_put() to release the refcount.
--- drivers/ide/pmac.c | 1 + 1 file changed, 1 insertion(+)
drivers/ide/pmac.c +1 -0
@@ -920,6 +920,7 @@ static u8 pmac_ide_cable_detect(ide_hwif_t *hwif)struct device_node *root =of_find_node_by_path ("/" );const char *model =of_get_property (root,"model" ,NULL ); +of_node_put (root);/* Get cable type from device-tree. */ if (cable && !strncmp (cable,"80-" ,3 )) {/* Some drives fail to detect 80c cable in PowerBook */
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 51/72 drm/msm: Fix error return checking
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 098336deb946f37a70afc0979af388b615c378bf ]
The error checks on ret for a negative error return always fails because the return value of iommu_map_sg() is unsigned and can never be negative.
Detected with Coccinelle:
drivers/gpu/drm/msm/msm_iommu.c:69:9-12: WARNING: Unsigned expression compared with zero: ret < 0
--- drivers/gpu/drm/msm/msm_iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/gpu/drm/msm/msm_iommu.c +1 -1
@@ -66,7 +66,7 @@ static int msm_iommu_map(struct msm_mmu *mmu, uint64_t iova,// pm_runtime_get_sync(mmu->dev); ret =iommu_map_sg (iommu->domain, iova, sgt->sgl, sgt->nents, prot);// pm_runtime_put_sync(mmu->dev); -WARN_ON (ret <0 ); +WARN_ON (!ret);return (ret == len) ?0 : -EINVAL ; }
Signature · 8 lines
2.19.1 _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
Greg Kroah-Hartman
maintainerPATCH 52/72 clk: mvebu: Off by one bugs in cp110_of_clk_get()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit d9f5b7f5dd0fa74a89de5a7ac1e26366f211ccee ]
These > comparisons should be >= to prevent reading beyond the end of of the clk_data->hws[] buffer.
The clk_data->hws[] array is allocated in cp110_syscon_common_probe() when we do:
cp110_clk_data = devm_kzalloc(dev, sizeof(*cp110_clk_data) + sizeof(struct clk_hw *) * CP110_CLK_NUM, GFP_KERNEL);
As you can see, it has CP110_CLK_NUM elements which is equivalent to CP110_MAX_CORE_CLOCKS + CP110_MAX_GATABLE_CLOCKS.
--- drivers/clk/mvebu/cp110-system-controller.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/clk/mvebu/cp110-system-controller.c +2 -2
@@ -203,11 +203,11 @@ static struct clk_hw *cp110_of_clk_get(struct of_phandle_args *clkspec,unsigned int idx = clkspec->args[1 ];if (type ==CP110_CLK_TYPE_CORE ) { -if (idx >CP110_MAX_CORE_CLOCKS ) +if (idx >=CP110_MAX_CORE_CLOCKS )return ERR_PTR (-EINVAL );return clk_data->hws[idx]; }else if (type ==CP110_CLK_TYPE_GATABLE ) { -if (idx >CP110_MAX_GATABLE_CLOCKS ) +if (idx >=CP110_MAX_GATABLE_CLOCKS )return ERR_PTR (-EINVAL );return clk_data->hws[CP110_MAX_CORE_CLOCKS + idx]; }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 53/72 clk: mmp: Off by one in mmp_clk_add()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 2e85c57493e391b93445c1e0d530b36b95becc64 ]
The > comparison should be >= or we write one element beyond the end of the unit->clk_table[] array.
(The unit->clk_table[] array is allocated in the mmp_clk_init() function and it has unit->nr_clks elements).
--- drivers/clk/mmp/clk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clk/mmp/clk.c +1 -1
@@ -183,7 +183,7 @@ void mmp_clk_add(struct mmp_clk_unit *unit, unsigned int id,pr_err ("CLK %d has invalid pointer %p\n" , id, clk);return ; } -if (id > unit->nr_clks) { +if (id >= unit->nr_clks) {pr_err ("CLK %d is invalid\n" , id);return ; }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 54/72 Input: synaptics - enable SMBus for HP 15-ay000
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 5a6dab15f7a79817cab4af612ddd99eda793fce6 ]
SMBus works fine for the touchpad with id SYN3221, used in the HP 15-ay000 series,
This device has been reported in these messages in the "linux-input" mailing list:
* https://marc.info/?l=linux-input&m=152016683003369&w=2
* https://www.spinics.net/lists/linux-input/msg52525.html
--- drivers/input/mouse/synaptics.c | 1 + 1 file changed, 1 insertion(+)
drivers/input/mouse/synaptics.c +1 -0
@@ -178,6 +178,7 @@ static const char * const smbus_pnp_ids[] = {"LEN0096" ,/* X280 */ "LEN0097" ,/* X280 -> ALPS trackpoint */ "LEN200f" ,/* T450s */ +"SYN3221" ,/* HP 15-ay000 */ NULL };
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 55/72 Input: omap-keypad - fix keyboard debounce configuration
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 6c3516fed7b61a3527459ccfa67fab130d910610 ]
I noticed that the Android v3.0.8 kernel on droid4 is using different keypad values from the mainline kernel and does not have issues with keys occasionally being stuck until pressed again. Turns out there was an earlier patch posted to fix this as "Input: omap-keypad: errata i689:
Correct debounce time", but it was never reposted to fix use macros for timing calculations.
This updated version is using macros, and also fixes the use of the input clock rate to use 32768KiHz instead of 32000KiHz. And we want to use the known good Android kernel values of 3 and 6 instead of 2 and 6 in the earlier patch.
--- drivers/input/keyboard/omap4-keypad.c | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-)
drivers/input/keyboard/omap4-keypad.c +14 -4
@@ -60,8 +60,18 @@/* OMAP4 values */ #define OMAP4_VAL_IRQDISABLE 0x0 -#define OMAP4_VAL_DEBOUNCINGTIME 0x7 -#define OMAP4_VAL_PVT 0x7 + +/* +* Errata i689: If a key is released for a time shorter than debounce time, +* the keyboard will idle and never detect the key release. The workaround +* is to use at least a 12ms debounce time. See omap5432 TRM chapter +* "26.4.6.2 Keyboard Controller Timer" for more information. +*/ +#define OMAP4_KEYPAD_PTV_DIV_128 0x6 +#define OMAP4_KEYPAD_DEBOUNCINGTIME_MS (dbms, ptv) \ + ((((dbms) * 1000) / ((1 << ((ptv) + 1)) * (1000000 / 32768))) - 1) +#define OMAP4_VAL_DEBOUNCINGTIME_16MS \ + OMAP4_KEYPAD_DEBOUNCINGTIME_MS(16, OMAP4_KEYPAD_PTV_DIV_128)enum {KBD_REVISION_OMAP4 =0 ,
@@ -181,9 +191,9 @@ static int omap4_keypad_open(struct input_dev *input)kbd_writel (keypad_data,OMAP4_KBD_CTRL ,OMAP4_DEF_CTRL_NOSOFTMODE | - (OMAP4_VAL_PVT <<OMAP4_DEF_CTRL_PTV_SHIFT )); + (OMAP4_KEYPAD_PTV_DIV_128 <<OMAP4_DEF_CTRL_PTV_SHIFT ));kbd_writel (keypad_data,OMAP4_KBD_DEBOUNCINGTIME , -OMAP4_VAL_DEBOUNCINGTIME ); +OMAP4_VAL_DEBOUNCINGTIME_16MS );/* clear pending interrupts */ kbd_write_irqreg (keypad_data,OMAP4_KBD_IRQSTATUS ,kbd_read_irqreg (keypad_data,OMAP4_KBD_IRQSTATUS ));
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 56/72 libata: whitelist all SAMSUNG MZ7KM* solid-state disks
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit fd6f32f78645db32b6b95a42e45da2ddd6de0e67 ]
These devices support read zero after trim (RZAT), as they advertise to the OS. However, the OS doesn't believe the SSDs unless they are explicitly whitelisted.
--- drivers/ata/libata-core.c | 1 + 1 file changed, 1 insertion(+)
drivers/ata/libata-core.c +1 -0
@@ -4593,6 +4593,7 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = { { "SSD*INTEL*", NULL, ATA_HORKAGE_ZERO_AFTER_TRIM, }, { "Samsung*SSD*", NULL, ATA_HORKAGE_ZERO_AFTER_TRIM, }, { "SAMSUNG*SSD*", NULL, ATA_HORKAGE_ZERO_AFTER_TRIM, }, + { "SAMSUNG*MZ7KM*", NULL, ATA_HORKAGE_ZERO_AFTER_TRIM, }, { "ST[1248][0248]0[FH]*", NULL, ATA_HORKAGE_ZERO_AFTER_TRIM, },/*
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 57/72 mv88e6060: disable hardware level MAC learning
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit a74515604a7b171f2702bdcbd1e231225fb456d0 ]
Disable hardware level MAC learning because it breaks station roaming. When enabled it drops all frames that arrive from a MAC address that is on a different port at learning table.
--- drivers/net/dsa/mv88e6060.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-)
drivers/net/dsa/mv88e6060.c +3 -7
@@ -114,8 +114,7 @@ static int mv88e6060_switch_reset(struct dsa_switch *ds)/* Reset the switch. */ REG_WRITE (REG_GLOBAL ,GLOBAL_ATU_CONTROL ,GLOBAL_ATU_CONTROL_SWRESET | -GLOBAL_ATU_CONTROL_ATUSIZE_1024 | -GLOBAL_ATU_CONTROL_ATE_AGE_5MIN ); +GLOBAL_ATU_CONTROL_LEARNDIS );/* Wait up to one second for reset to complete. */ timeout = jiffies +1 *HZ ;
@@ -140,13 +139,10 @@ static int mv88e6060_setup_global(struct dsa_switch *ds)*/ REG_WRITE (REG_GLOBAL ,GLOBAL_CONTROL ,GLOBAL_CONTROL_MAX_FRAME_1536 ); -/* Enable automatic address learning, set the address -* database size to 1024 entries, and set the default aging -* time to 5 minutes. +/* Disable automatic address learning. */ REG_WRITE (REG_GLOBAL ,GLOBAL_ATU_CONTROL , -GLOBAL_ATU_CONTROL_ATUSIZE_1024 | -GLOBAL_ATU_CONTROL_ATE_AGE_5MIN ); +GLOBAL_ATU_CONTROL_LEARNDIS );return 0 ; }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 58/72 net/mlx4_en: Fix build break when CONFIG_INET is off
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 1b603f9e4313348608f256b564ed6e3d9e67f377 ]
MLX4_EN depends on NETDEVICES, ETHERNET and INET Kconfigs. Make sure they are listed in MLX4_EN Kconfig dependencies.
This fixes the following build break:
drivers/net/ethernet/mellanox/mlx4/en_rx.c:582:18: warning: ‘struct iphdr’ declared inside parameter list [enabled by default]
struct iphdr *iph)
^drivers/net/ethernet/mellanox/mlx4/en_rx.c:582:18: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default] drivers/net/ethernet/mellanox/mlx4/en_rx.c: In function ‘get_fixed_ipv4_csum’:drivers/net/ethernet/mellanox/mlx4/en_rx.c:586:20: error: dereferencing pointer to incomplete type_u8 ipproto = iph->protocol;
--- drivers/net/ethernet/mellanox/mlx4/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/mellanox/mlx4/Kconfig +1 -1
@@ -5,7 +5,7 @@ config MLX4_EN tristate "Mellanox Technologies 1/10/40Gbit Ethernet support" depends on MAY_USE_DEVLINK - depends on PCI + depends on PCI && NETDEVICES && ETHERNET && INET select MLX4_CORE imply PTP_1588_CLOCK ---help---
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 59/72 ARM: 8814/1: mm: improve/fix ARM v7_dma_inv_range() unaligned address handling
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit a1208f6a822ac29933e772ef1f637c5d67838da9 ]
This patch addresses possible memory corruption when v7_dma_inv_range(start_address, end_address) address parameters are not aligned to whole cache lines. This function issues "invalidate" cache management operations to all cache lines from start_address (inclusive) to end_address (exclusive). When start_address and/or end_address are not aligned, the start and/or end cache lines are first issued "clean & invalidate" operation. The assumption is this is done to ensure that any dirty data addresses outside the address range (but part of the first or last cache lines) are cleaned/flushed so that data is not lost, which could happen if just an invalidate is issued.
The problem is that these first/last partial cache lines are issued "clean & invalidate" and then "invalidate". This second "invalidate" is not required and worse can cause "lost" writes to addresses outside the address range but part of the cache line. If another component writes to its part of the cache line between the "clean & invalidate" and "invalidate" operations, the write can get lost. This fix is to remove the extra "invalidate" operation when unaligned addressed are used.
A kernel module is available that has a stress test to reproduce the issue and a unit test of the updated v7_dma_inv_range(). It can be downloaded from
http://ftp.sageembedded.com/outgoing/linux/cache-test-20181107.tgz.
v7_dma_inv_range() is call by dmac_[un]map_area(addr, len, direction) when the direction is DMA_FROM_DEVICE. One can (I believe) successfully argue that DMA from a device to main memory should use buffers aligned to cache line size, because the "clean & invalidate" might overwrite data that the device just wrote using DMA. But if a driver does use unaligned buffers, at least this fix will prevent memory corruption outside the buffer.
--- arch/arm/mm/cache-v7.S | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)
arch/arm/mm/cache-v7.S +5 -3
@@ -359,14 +359,16 @@ v7_dma_inv_range: ALT_UP(W(nop)) #endif mcrne p15, 0, r0, c7, c14, 1 @ clean & invalidate D / U line + addne r0, r0, r2 tst r1, r3 bic r1, r1, r3 mcrne p15, 0, r1, c7, c14, 1 @ clean & invalidate D / U line -1: - mcr p15, 0, r0, c7, c6, 1 @ invalidate D / U line - add r0, r0, r2 cmp r0, r1 +1: + mcrlo p15, 0, r0, c7, c6, 1 @ invalidate D / U line + addlo r0, r0, r2 + cmplo r0, r1 blo 1b dsb st ret lr
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 60/72 ARM: 8815/1: V7M: align v7m_dma_inv_range() with v7 counterpart
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 3d0358d0ba048c5afb1385787aaec8fa5ad78fcc ]
Chris has discovered and reported that v7_dma_inv_range() may corrupt memory if address range is not aligned to cache line size.
Since the whole cache-v7m.S was lifted form cache-v7.S the same observation applies to v7m_dma_inv_range(). So the fix just mirrors what has been done for v7 with a little specific of M-class.
--- arch/arm/mm/cache-v7m.S | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-)
arch/arm/mm/cache-v7m.S +9 -5
@@ -73,9 +73,11 @@ /* * dcimvac: Invalidate data cache line by MVA to PoC */ -.macro dcimvac, rt, tmp - v7m_cacheop \rt, \tmp, V7M_SCB_DCIMVAC +.irp c,,eq,ne,cs,cc,mi,pl,vs,vc,hi,ls,ge,lt,gt,le,hs,lo +.macro dcimvac\c, rt, tmp + v7m_cacheop \rt, \tmp, V7M_SCB_DCIMVAC, \c .endm +.endr /* * dccmvau: Clean data cache line by MVA to PoU
@@ -369,14 +371,16 @@ v7m_dma_inv_range: tst r0, r3 bic r0, r0, r3 dccimvacne r0, r3 + addne r0, r0, r2 subne r3, r2, #1 @ restore r3, corrupted by v7m's dccimvac tst r1, r3 bic r1, r1, r3 dccimvacne r1, r3 -1: - dcimvac r0, r3 - add r0, r0, r2 cmp r0, r1 +1: + dcimvaclo r0, r3 + addlo r0, r0, r2 + cmplo r0, r1 blo 1b dsb st ret lr
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 61/72 ethernet: fman: fix wrong of_node_put() in probe function
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit ecb239d96d369c23c33d41708646df646de669f4 ]
After getting a reference to the platform device's of_node the probe function ends up calling of_find_matching_node() using the node as an argument. The function takes care of decreasing the refcount on it. We are then incorrectly decreasing the refcount on that node again.
This patch removes the unwarranted call to of_node_put().
--- drivers/net/ethernet/freescale/fman/fman.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
drivers/net/ethernet/freescale/fman/fman.c +2 -3
@@ -2786,7 +2786,7 @@ static struct fman *read_dts_node(struct platform_device *of_dev)if (!muram_node) {dev_err (&of_dev->dev,"%s: could not find MURAM node\n" , __func__); - goto fman_node_put; + goto fman_free; } err = of_address_to_resource(muram_node, 0,
@@ -2795,11 +2795,10 @@ static struct fman *read_dts_node(struct platform_device *of_dev)of_node_put (muram_node);dev_err (&of_dev->dev,"%s: of_address_to_resource() = %d\n" , __func__, err); - goto fman_node_put; + goto fman_free; }of_node_put (muram_node); -of_node_put (fm_node); err =devm_request_irq (&of_dev->dev, irq, fman_irq,0 ,"fman" , fman);if (err <0 ) {
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 62/72 drm/ast: Fix connector leak during driver unload
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit e594a5e349ddbfdaca1951bb3f8d72f3f1660d73 ]
When unloading the ast driver, a warning message is printed by drm_mode_config_cleanup() because a reference is still held to one of the drm_connector structs.
Correct this by calling drm_crtc_force_disable_all() in ast_fbdev_destroy().
--- drivers/gpu/drm/ast/ast_fb.c | 1 + 1 file changed, 1 insertion(+)
drivers/gpu/drm/ast/ast_fb.c +1 -0
@@ -263,6 +263,7 @@ static void ast_fbdev_destroy(struct drm_device *dev, {struct ast_framebuffer *afb = &afbdev->afb; +drm_crtc_force_disable_all (dev);drm_fb_helper_unregister_fbi (&afbdev->helper);if (afb->obj) {
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 63/72 cifs: In Kconfig CONFIG_CIFS_POSIX needs depends on legacy (insecure cifs)
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 6e785302dad32228819d8066e5376acd15d0e6ba ]
Missing a dependency. Shouldn't show cifs posix extensions in Kconfig if CONFIG_CIFS_ALLOW_INSECURE_DIALECTS (ie SMB1 protocol) is disabled.
--- fs/cifs/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
fs/cifs/Kconfig +1 -1
@@ -121,7 +121,7 @@ config CIFS_XATTR config CIFS_POSIX bool "CIFS POSIX Extensions" - depends on CIFS_XATTR + depends on CIFS && CIFS_ALLOW_INSECURE_LEGACY && CIFS_XATTR help Enabling this option will cause the cifs client to attempt to negotiate a newer dialect with servers, such as Samba 3.0.5
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 64/72 vhost/vsock: fix reset orphans race with close timeout
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit c38f57da428b033f2721b611d84b1f40bde674a8 ]
If a local process has closed a connected socket and hasn't received a RST packet yet, then the socket remains in the table until a timeout expires.
When a vhost_vsock instance is released with the timeout still pending, the socket is never freed because vhost_vsock has already set the SOCK_DONE flag.
Check if the close timer is pending and let it close the socket. This prevents the race which can leak sockets.
--- drivers/vhost/vsock.c | 22 +++++++++++++++------- 1 file changed, 15 insertions(+), 7 deletions(-)
drivers/vhost/vsock.c +15 -7
@@ -561,13 +561,21 @@ static void vhost_vsock_reset_orphans(struct sock *sk)* executing. */ -if (!vhost_vsock_get (vsk->remote_addr.svm_cid)) { -sock_set_flag (sk,SOCK_DONE ); - vsk->peer_shutdown =SHUTDOWN_MASK ; - sk->sk_state =SS_UNCONNECTED ; - sk->sk_err =ECONNRESET ; - sk->sk_error_report (sk); - } +/* If the peer is still valid, no need to reset connection */ +if (vhost_vsock_get (vsk->remote_addr.svm_cid)) +return ; + +/* If the close timeout is pending, let it expire. This avoids races +* with the timeout callback. +*/ +if (vsk->close_work_scheduled) +return ; + +sock_set_flag (sk,SOCK_DONE ); + vsk->peer_shutdown =SHUTDOWN_MASK ; + sk->sk_state =SS_UNCONNECTED ; + sk->sk_err =ECONNRESET ; + sk->sk_error_report (sk); }static int vhost_vsock_dev_release (struct inode *inode,struct file *file)
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 65/72 mlxsw: spectrum_switchdev: Fix VLAN device deletion via ioctl
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 993107fea5eefdfdfde1ca38d3f01f0bebf76e77 ]
When deleting a VLAN device using an ioctl the netdev is unregistered before the VLAN filter is updated via ndo_vlan_rx_kill_vid(). It can lead to a use-after-free in mlxsw in case the VLAN device is deleted while being enslaved to a bridge.
The reason for the above is that when mlxsw receives the CHANGEUPPER event, it wrongly assumes that the VLAN device is no longer its upper and thus destroys the internal representation of the bridge port despite the reference count being non-zero.
Fix this by checking if the VLAN device is our upper using its real device. In net-next I'm going to remove this trick and instead make mlxsw completely agnostic to the order of the events.
--- .../net/ethernet/mellanox/mlxsw/spectrum_switchdev.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)
drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c +8 -2
@@ -295,7 +295,13 @@ static boolmlxsw_sp_bridge_port_should_destroy (const struct mlxsw_sp_bridge_port * bridge_port) { -struct mlxsw_sp *mlxsw_sp =mlxsw_sp_lower_get (bridge_port->dev); +struct net_device *dev = bridge_port->dev; +struct mlxsw_sp *mlxsw_sp; + +if (is_vlan_dev (dev)) + mlxsw_sp =mlxsw_sp_lower_get (vlan_dev_real_dev (dev)); +else + mlxsw_sp =mlxsw_sp_lower_get (dev);/* In case ports were pulled from out of a bridged LAG, then * it's possible the reference count isn't zero, yet the bridge
@@ -1646,7 +1652,7 @@ mlxsw_sp_bridge_8021d_port_leave(struct mlxsw_sp_bridge_device *bridge_device,u16 vid =vlan_dev_vlan_id (bridge_port->dev); mlxsw_sp_port_vlan =mlxsw_sp_port_vlan_find_by_vid (mlxsw_sp_port, vid); -if (WARN_ON (!mlxsw_sp_port_vlan)) +if (!mlxsw_sp_port_vlan)return ;mlxsw_sp_port_vlan_bridge_leave (mlxsw_sp_port_vlan);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 66/72 i2c: axxia: properly handle master timeout
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 6c7f25cae54b840302e4f1b371dbf318fbf09ab2 ]
According to Intel (R) Axxia TM Lionfish Communication Processor Peripheral Subsystem Hardware Reference Manual, the AXXIA I2C module have a programmable Master Wait Timer, which among others, checks the time between commands send in manual mode. When a timeout (25ms) passes, TSS bit is set in Master Interrupt Status register and a Stop command is issued by the hardware.
The axxia_i2c_xfer(), does not properly handle this situation, however. For each message a separate axxia_i2c_xfer_msg() is called and this function incorrectly assumes that any interrupt might happen only when waiting for completion. This is mostly correct but there is one exception - a master timeout can trigger if enough time has passed between individual transfers. It will, by definition, happen between transfers when the interrupts are disabled by the code. If that happens, the hardware issues Stop command.
The interrupt indicating timeout will not be triggered as soon as we enable them since the Master Interrupt Status is cleared when master mode is entered again (which happens before enabling irqs) meaning this error is lost and the transfer is continued even though the Stop was issued on the bus. The subsequent operations completes without error but a bogus value (0xFF in case of read) is read as the client device is confused because aborted transfer. No error is returned from master_xfer() making caller believe that a valid value was read.
To fix the problem, the TSS bit (indicating timeout) in Master Interrupt Status register is checked before each transfer. If it is set, there was a timeout before this transfer and (as described above) the hardware already issued Stop command so the transaction should be aborted thus -ETIMEOUT is returned from the master_xfer() callback. In order to be sure no timeout was issued we can't just read the status just before starting new transaction as there will always be a small window of time (few CPU cycles at best) where this might still happen. For this reason we have to temporally disable the timer before checking for TSS bit. Disabling it will, however, clear the TSS bit so in order to preserve that information, we have to read it in ISR so we have to ensure that the TSS interrupt is not masked between transfers of one transaction. There is no need to call bus recovery or controller reinitialization if that happens so it's skipped.
--- drivers/i2c/busses/i2c-axxia.c | 40 ++++++++++++++++++++++++---------- 1 file changed, 29 insertions(+), 11 deletions(-)
drivers/i2c/busses/i2c-axxia.c +29 -11
@@ -74,8 +74,7 @@ MST_STATUS_ND)#define MST_STATUS_ERR (MST_STATUS_NAK | \ MST_STATUS_AL | \ - MST_STATUS_IP | \ - MST_STATUS_TSS) + MST_STATUS_IP)#define MST_TX_BYTES_XFRD 0x50#define MST_RX_BYTES_XFRD 0x54#define SCL_HIGH_PERIOD 0x80
@@ -241,7 +240,7 @@ static int axxia_i2c_empty_rx_fifo(struct axxia_i2c_dev *idev)*/ if (c <=0 || c >I2C_SMBUS_BLOCK_MAX ) { idev->msg_err = -EPROTO ; -i2c_int_disable (idev, ~0 ); +i2c_int_disable (idev, ~MST_STATUS_TSS );complete (&idev->msg_complete);break ; }
@@ -299,14 +298,19 @@ static irqreturn_t axxia_i2c_isr(int irq, void *_dev)if (status &MST_STATUS_SCC ) {/* Stop completed */ -i2c_int_disable (idev, ~0 ); +i2c_int_disable (idev, ~MST_STATUS_TSS );complete (&idev->msg_complete); }else if (status &MST_STATUS_SNS ) {/* Transfer done */ -i2c_int_disable (idev, ~0 ); +i2c_int_disable (idev, ~MST_STATUS_TSS );if (i2c_m_rd (idev->msg) && idev->msg_xfrd < idev->msg->len)axxia_i2c_empty_rx_fifo (idev);complete (&idev->msg_complete); + }else if (status &MST_STATUS_TSS ) { +/* Transfer timeout */ + idev->msg_err = -ETIMEDOUT ; +i2c_int_disable (idev, ~MST_STATUS_TSS ); +complete (&idev->msg_complete); }else if (unlikely (status &MST_STATUS_ERR )) {/* Transfer error */ i2c_int_disable (idev, ~0 );
@@ -339,10 +343,10 @@ static int axxia_i2c_xfer_msg(struct axxia_i2c_dev *idev, struct i2c_msg *msg)u32 rx_xfer, tx_xfer;u32 addr_1, addr_2;unsigned long time_left; +unsigned int wt_value; idev->msg = msg; idev->msg_xfrd =0 ; - idev->msg_err =0 ;reinit_completion (&idev->msg_complete);if (i2c_m_ten (msg)) {
@@ -382,9 +386,18 @@ static int axxia_i2c_xfer_msg(struct axxia_i2c_dev *idev, struct i2c_msg *msg)else if (axxia_i2c_fill_tx_fifo (idev) !=0 ) int_mask |=MST_STATUS_TFL ; + wt_value =WT_VALUE (readl (idev->base +WAIT_TIMER_CONTROL )); +/* Disable wait timer temporarly */ +writel (wt_value, idev->base +WAIT_TIMER_CONTROL ); +/* Check if timeout error happened */ +if (idev->msg_err) + goto out; +/* Start manual mode */ writel (CMD_MANUAL , idev->base +MST_COMMAND ); +writel (WT_EN | wt_value, idev->base +WAIT_TIMER_CONTROL ); +i2c_int_enable (idev, int_mask); time_left = wait_for_completion_timeout(&idev->msg_complete,
@@ -395,13 +408,15 @@ static int axxia_i2c_xfer_msg(struct axxia_i2c_dev *idev, struct i2c_msg *msg)if (readl (idev->base +MST_COMMAND ) &CMD_BUSY )dev_warn (idev->dev,"busy after xfer\n" ); -if (time_left ==0 ) +if (time_left ==0 ) { idev->msg_err = -ETIMEDOUT ; - -if (idev->msg_err == -ETIMEDOUT )i2c_recover_bus (&idev->adapter); +axxia_i2c_init (idev); + } -if (unlikely (idev->msg_err) && idev->msg_err != -ENXIO ) +out: +if (unlikely (idev->msg_err) && idev->msg_err != -ENXIO && + idev->msg_err != -ETIMEDOUT )axxia_i2c_init (idev);return idev->msg_err;
@@ -409,7 +424,7 @@ static int axxia_i2c_xfer_msg(struct axxia_i2c_dev *idev, struct i2c_msg *msg)static int axxia_i2c_stop (struct axxia_i2c_dev *idev) { -u32 int_mask =MST_STATUS_ERR |MST_STATUS_SCC ; +u32 int_mask =MST_STATUS_ERR |MST_STATUS_SCC |MST_STATUS_TSS ;unsigned long time_left;reinit_completion (&idev->msg_complete);
@@ -436,6 +451,9 @@ axxia_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)int i;int ret =0 ; + idev->msg_err =0 ; +i2c_int_enable (idev,MST_STATUS_TSS ); +for (i =0 ; ret ==0 && i < num; ++i) ret =axxia_i2c_xfer_msg (idev, &msgs[i]);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 67/72 i2c: scmi: Fix probe error on devices with an empty SMB0001 ACPI device node
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 0544ee4b1ad574aec3b6379af5f5cdee42840971 ]
Some AMD based HP laptops have a SMB0001 ACPI device node which does not define any methods.
This leads to the following error in dmesg:
[ 5.222731] cmi: probe of SMB0001:00 failed with error -5
This commit makes acpi_smbus_cmi_add() return -ENODEV instead in this case silencing the error. In case of a failure of the i2c_add_adapter() call this commit now propagates the error from that call instead of -EIO.
--- drivers/i2c/busses/i2c-scmi.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-)
drivers/i2c/busses/i2c-scmi.c +7 -3
@@ -364,6 +364,7 @@ static int acpi_smbus_cmi_add(struct acpi_device *device) {struct acpi_smbus_cmi *smbus_cmi;const struct acpi_device_id *id; +int ret; smbus_cmi =kzalloc (sizeof (struct acpi_smbus_cmi ),GFP_KERNEL );if (!smbus_cmi)
@@ -385,8 +386,10 @@ static int acpi_smbus_cmi_add(struct acpi_device *device)acpi_walk_namespace (ACPI_TYPE_METHOD , smbus_cmi->handle,1 , acpi_smbus_cmi_query_methods,NULL , smbus_cmi,NULL ); -if (smbus_cmi->cap_info ==0 ) +if (smbus_cmi->cap_info ==0 ) { + ret = -ENODEV ; goto err; + } snprintf(smbus_cmi->adapter.name, sizeof(smbus_cmi->adapter.name), "SMBus CMI adapter %s",
@@ -397,7 +400,8 @@ static int acpi_smbus_cmi_add(struct acpi_device *device) smbus_cmi->adapter.class =I2C_CLASS_HWMON |I2C_CLASS_SPD ; smbus_cmi->adapter.dev.parent = &device->dev; -if (i2c_add_adapter (&smbus_cmi->adapter)) { + ret =i2c_add_adapter (&smbus_cmi->adapter); +if (ret) {dev_err (&device->dev,"Couldn't register adapter!\n" ); goto err; }
@@ -407,7 +411,7 @@ static int acpi_smbus_cmi_add(struct acpi_device *device) err:kfree (smbus_cmi); device->driver_data =NULL ; -return -EIO ; +return ret; }static int acpi_smbus_cmi_remove (struct acpi_device *device)
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 68/72 i2c: uniphier: fix violation of tLOW requirement for Fast-mode
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit 8469636ab5d8c77645b953746c10fda6983a8830 ]
Currently, the clock duty is set as tLOW/tHIGH = 1/1. For Fast-mode, tLOW is set to 1.25 us while the I2C spec requires tLOW >= 1.3 us.
tLOW/tHIGH = 5/4 would meet both Standard-mode and Fast-mode:
Standard-mode: tLOW = 5.56 us, tHIGH = 4.44 us
Fast-mode: tLOW = 1.39 us, tHIGH = 1.11 us
--- drivers/i2c/busses/i2c-uniphier.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
drivers/i2c/busses/i2c-uniphier.c +7 -1
@@ -320,7 +320,13 @@ static void uniphier_i2c_hw_init(struct uniphier_i2c_priv *priv)uniphier_i2c_reset (priv, true); -writel ((cyc /2 <<16 ) | cyc, priv->membase +UNIPHIER_I2C_CLK ); +/* +* Bit30-16: clock cycles of tLOW. +* Standard-mode: tLOW = 4.7 us, tHIGH = 4.0 us +* Fast-mode: tLOW = 1.3 us, tHIGH = 0.6 us +* "tLow/tHIGH = 5/4" meets both. +*/ +writel ((cyc *5 /9 <<16 ) | cyc, priv->membase +UNIPHIER_I2C_CLK );uniphier_i2c_reset (priv, false); }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 69/72 i2c: uniphier-f: fix violation of tLOW requirement for Fast-mode
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit ece27a337d42a3197935711997f2880f0957ed7e ]
Currently, the clock duty is set as tLOW/tHIGH = 1/1. For Fast-mode, tLOW is set to 1.25 us while the I2C spec requires tLOW >= 1.3 us.
tLOW/tHIGH = 5/4 would meet both Standard-mode and Fast-mode:
Standard-mode: tLOW = 5.56 us, tHIGH = 4.44 us
Fast-mode: tLOW = 1.39 us, tHIGH = 1.11 us
--- drivers/i2c/busses/i2c-uniphier-f.c | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-)
drivers/i2c/busses/i2c-uniphier-f.c +18 -1
@@ -470,9 +470,26 @@ static void uniphier_fi2c_hw_init(struct uniphier_fi2c_priv *priv)uniphier_fi2c_reset (priv); +/* +* Standard-mode: tLOW + tHIGH = 10 us +* Fast-mode: tLOW + tHIGH = 2.5 us +*/ writel (cyc, priv->membase +UNIPHIER_FI2C_CYC ); -writel (cyc /2 , priv->membase +UNIPHIER_FI2C_LCTL ); +/* +* Standard-mode: tLOW = 4.7 us, tHIGH = 4.0 us, tBUF = 4.7 us +* Fast-mode: tLOW = 1.3 us, tHIGH = 0.6 us, tBUF = 1.3 us +* "tLow/tHIGH = 5/4" meets both. +*/ +writel (cyc *5 /9 , priv->membase +UNIPHIER_FI2C_LCTL ); +/* +* Standard-mode: tHD;STA = 4.0 us, tSU;STA = 4.7 us, tSU;STO = 4.0 us +* Fast-mode: tHD;STA = 0.6 us, tSU;STA = 0.6 us, tSU;STO = 0.6 us +*/ writel (cyc /2 , priv->membase +UNIPHIER_FI2C_SSUT ); +/* +* Standard-mode: tSU;DAT = 250 ns +* Fast-mode: tSU;DAT = 100 ns +*/ writel (cyc /16 , priv->membase +UNIPHIER_FI2C_DSUT );uniphier_fi2c_prepare_operation (priv);
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 70/72 nvmet-rdma: fix response use after free
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit d7dcdf9d4e15189ecfda24cc87339a3425448d5c ]
nvmet_rdma_release_rsp() may free the response before using it at error flow.
--- drivers/nvme/target/rdma.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/nvme/target/rdma.c +2 -1
@@ -524,6 +524,7 @@ static void nvmet_rdma_send_done(struct ib_cq *cq, struct ib_wc *wc) {struct nvmet_rdma_rsp *rsp = container_of(wc->wr_cqe, struct nvmet_rdma_rsp, send_cqe); +struct nvmet_rdma_queue *queue = cq->cq_context;nvmet_rdma_release_rsp (rsp);
@@ -531,7 +532,7 @@ static void nvmet_rdma_send_done(struct ib_cq *cq, struct ib_wc *wc) wc->status != IB_WC_WR_FLUSH_ERR)) {pr_err ("SEND for CQE 0x%p failed with status %s (%d).\n" , wc->wr_cqe,ib_wc_status_msg (wc->status), wc->status); -nvmet_rdma_error_comp (rsp->queue); +nvmet_rdma_error_comp (queue); } }
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 71/72 rtc: snvs: Add timeouts to avoid kernel lockups
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
[ Upstream commit cd7f3a249dbed2858e6c2f30e5be7f1f7a709ee2 ]
In order to read correctly from asynchronously updated RTC registers, it's necessary to read repeatedly until their values do not change from read to read. It's also necessary to wait for three RTC clock ticks for certain operations. There are no timeouts in this code and these operations could possibly loop forever.
To avoid kernel hangs, put in timeouts.
The iMX7d can be configured to stop the SRTC on a tamper event, which will lockup the kernel inside this driver as described above.
These hangs can happen when running under qemu, which doesn't emulate the SNVS RTC, though currently the driver will refuse to load on qemu due to a timeout in the driver probe method.
It could also happen if the SRTC block where somehow placed into reset or the slow speed clock that drives the SRTC counter (but not the CPU) were to stop.
The symptoms on a two core iMX7d are a work queue hang on rtc_timer_do_work(), which eventually blocks a systemd fsnotify operation that triggers a work queue flush, causing systemd to hang and thus causing all services that should be started by systemd, like a console getty, to fail to start or stop.
Also optimize the wait code to wait less. It only needs to wait for the clock to advance three ticks, not to see it change three times.
--- drivers/rtc/rtc-snvs.c | 105 +++++++++++++++++++++++++++-------------- 1 file changed, 70 insertions(+), 35 deletions(-)
drivers/rtc/rtc-snvs.c +70 -35
@@ -47,49 +47,83 @@ struct snvs_rtc_data {struct clk *clk; }; +/* Read 64 bit timer register, which could be in inconsistent state */ +static u64 rtc_read_lpsrt (struct snvs_rtc_data *data) +{ +u32 msb, lsb; + +regmap_read (data->regmap, data->offset +SNVS_LPSRTCMR , &msb); +regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &lsb); +return (u64 )msb <<32 | lsb; +} + +/* Read the secure real time counter, taking care to deal with the cases of the +* counter updating while being read. +*/ static u32 rtc_read_lp_counter (struct snvs_rtc_data *data) {u64 read1, read2; -u32 val; +unsigned int timeout =100 ; +/* As expected, the registers might update between the read of the LSB +* reg and the MSB reg. It's also possible that one register might be +* in partially modified state as well. +*/ + read1 =rtc_read_lpsrt (data);do { -regmap_read (data->regmap, data->offset +SNVS_LPSRTCMR , &val); - read1 = val; - read1 <<=32 ; -regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &val); - read1 |= val; - -regmap_read (data->regmap, data->offset +SNVS_LPSRTCMR , &val); - read2 = val; - read2 <<=32 ; -regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &val); - read2 |= val; - }while (read1 != read2); + read2 = read1; + read1 =rtc_read_lpsrt (data); + }while (read1 != read2 && --timeout); +if (!timeout) +dev_err (&data->rtc->dev,"Timeout trying to get valid LPSRT Counter read\n" );/* Convert 47-bit counter to 32-bit raw second count */ return (u32) (read1 >>CNTR_TO_SECS_SH ); } -static void rtc_write_sync_lp (struct snvs_rtc_data *data) +/* Just read the lsb from the counter, dealing with inconsistent state */ +static int rtc_read_lp_counter_lsb (struct snvs_rtc_data *data,u32 *lsb) { -u32 count1, count2, count3; -int i; - -/* Wait for 3 CKIL cycles */ -for (i =0 ; i <3 ; i++) { -do { -regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &count1); -regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &count2); - }while (count1 != count2); - -/* Now wait until counter value changes */ -do { -do { -regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &count2); -regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &count3); - }while (count2 != count3); - }while (count3 == count1); +u32 count1, count2; +unsigned int timeout =100 ; + +regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &count1); +do { + count2 = count1; +regmap_read (data->regmap, data->offset +SNVS_LPSRTCLR , &count1); + }while (count1 != count2 && --timeout); +if (!timeout) { +dev_err (&data->rtc->dev,"Timeout trying to get valid LPSRT Counter read\n" ); +return -ETIMEDOUT ; } + + *lsb = count1; +return 0 ; +} + +static int rtc_write_sync_lp (struct snvs_rtc_data *data) +{ +u32 count1, count2; +u32 elapsed; +unsigned int timeout =1000 ; +int ret; + + ret =rtc_read_lp_counter_lsb (data, &count1); +if (ret) +return ret; + +/* Wait for 3 CKIL cycles, about 61.0-91.5 µs */ +do { + ret =rtc_read_lp_counter_lsb (data, &count2); +if (ret) +return ret; + elapsed = count2 - count1;/* wrap around _is_ handled! */ + }while (elapsed <3 && --timeout); +if (!timeout) { +dev_err (&data->rtc->dev,"Timeout waiting for LPSRT Counter to change\n" ); +return -ETIMEDOUT ; + } +return 0 ; }static int snvs_rtc_enable (struct snvs_rtc_data *data,bool enable)
@@ -173,9 +207,7 @@ static int snvs_rtc_alarm_irq_enable(struct device *dev, unsigned int enable) (SNVS_LPCR_LPTA_EN |SNVS_LPCR_LPWUI_EN ), enable ? (SNVS_LPCR_LPTA_EN | SNVS_LPCR_LPWUI_EN) : 0); -rtc_write_sync_lp (data); - -return 0 ; +return rtc_write_sync_lp (data); }static int snvs_rtc_set_alarm (struct device *dev,struct rtc_wkalrm *alrm)
@@ -183,11 +215,14 @@ static int snvs_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alrm)struct snvs_rtc_data *data =dev_get_drvdata (dev);struct rtc_time *alrm_tm = &alrm->time;unsigned long time; +int ret;rtc_tm_to_time (alrm_tm, &time);regmap_update_bits (data->regmap, data->offset +SNVS_LPCR ,SNVS_LPCR_LPTA_EN ,0 ); -rtc_write_sync_lp (data); + ret =rtc_write_sync_lp (data); +if (ret) +return ret;regmap_write (data->regmap, data->offset +SNVS_LPTAR , time);/* Clear alarm interrupt status bit */
Signature · 1 line
2.19.1
Greg Kroah-Hartman
maintainerPATCH 72/72 bpf, arm: fix emit_ldx_r and emit_mov_i using TMP_REG_1
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
emit_ldx_r() and emit_a32_mov_i() were both using TMP_REG_1 and clashing with each other. Using TMP_REG_2 in emit_ldx_r() fixes the issue.
--- arch/arm/net/bpf_jit_32.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/net/bpf_jit_32.c +1 -1
@@ -915,7 +915,7 @@ static inline void emit_str_r(const u8 dst, const u8 src, bool dstk,/* dst = *(size*)(src + off) */ static inline void emit_ldx_r (const u8 dst[],const u8 src,bool dstk,s32 off,struct jit_ctx *ctx,const u8 sz){ -const u8 *tmp = bpf2a32[TMP_REG_1 ]; +const u8 *tmp = bpf2a32[TMP_REG_2 ];const u8 *rd = dstk ? tmp : dst;u8 rm = src;s32 off_max;
Signature · 1 line
2.19.1
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
Build results:
total: 171 pass: 171 fail: 0 Qemu test results: total: 315 pass: 315 fail: 0
Details are available at https://kerneltests.org/builders/.
Guenter
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 9 lines
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.14.y
and the diffstat can be found below.thanks,
greg k-h
Compiled and booted on my test system. No dmesg regressions.
thanks,
-- Shuah
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.14.y
and the diffstat can be found below.thanks,
greg k-h
Results from Linaro’s test farm.
No regressions on arm64, arm, x86_64, and i386.
Summary
------------------------------------------------------------------------
kernel: 4.14.90-rc1
git repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git branch: linux-4.14.y
git commit: dafc7cbbb79e3fa3743ebcb0e0014b95905752b9
git describe: v4.14.89-73-gdafc7cbbb79e
Test details: https://qa-reports.linaro.org/lkft/linux-stable-rc-4.14-oe/build/v4.14.89-73-gdafc7cbbb79e
No regressions (compared to build v4.14.89)
No fixes (compared to build v4.14.89)
Ran 21645 total tests in the following environments and test suites.
Environments
--------------
- dragonboard-410c - arm64
- hi6220-hikey - arm64
- i386
- juno-r2 - arm64
- qemu_arm
- qemu_arm64
- qemu_i386
- qemu_x86_64
- x15 - arm
- x86_64
Test Suites
-----------
* boot
* install-android-platform-tools-r2600
* kselftest
* libhugetlbfs
* ltp-cap_bounds-tests
* ltp-containers-tests
* ltp-cpuhotplug-tests
* ltp-cve-tests
* ltp-fcntl-locktests-tests
* ltp-filecaps-tests
* ltp-fs-tests
* ltp-fs_bind-tests
* ltp-fs_perms_simple-tests
* ltp-fsx-tests
* ltp-hugetlb-tests
* ltp-ipc-tests
* ltp-math-tests
* ltp-nptl-tests
* ltp-pty-tests
* ltp-sched-tests
* ltp-securebits-tests
* ltp-syscalls-tests
* ltp-timers-tests
* spectre-meltdown-checker-test
* ltp-io-tests
* ltp-open-posix-tests
* kselftest-vsyscall-mode-native
* kselftest-vsyscall-mode-none
Signature · 2 lines
Linaro LKFT https://lkft.linaro.org
A message that is not in the archive (sent off-list or not yet received): CA+res+QQt7GEL0td1FraSJNnR2yOcDgXGahW+YyiuMSeWTshfw@mail.gmail.com
Quoted 21 lines
From: Greg Kroah-Hartman <[email protected]>
Date: 2018年12月20日周四 上午10:39
Subject: [PATCH 4.14 00/72] 4.14.90-stable review
To: <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gz
Quoted 8 lines
or in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
linux-4.14.y
and the diffstat can be found below.thanks,
greg k-h
Merged, basic functional tests, no regression found.
Thanks,
Signature · 19 lines
Jack Wang Linux Kernel Developer 1&1 IONOS Cloud GmbH | Greifswalder Str. 207 | 10405 Berlin | Germany Phone: +49 30 57700-8042 | Fax: +49 30 57700-8598 E-mail: [email protected] | Web: www.ionos.de Head Office: Berlin, Germany District Court Berlin Charlottenburg, Registration number: HRB 125506 B Executive Management: Christoph Steffens, Matthias Steinberg, Achim Weiss Member of United Internet This e-mail may contain confidential and/or privileged information. If you are not the intended recipient of this e-mail, you are hereby notified that saving, distribution or use of the content of this e-mail in any way is prohibited. If you have received this e-mail in error, please notify the sender and delete the e-mail.
Greg Kroah-Hartman
maintainerQuoted 26 lines · Jinpu Wang
Quoted 26 lines
From: Greg Kroah-Hartman <[email protected]>
Date: 2018年12月20日周四 上午10:39
Subject: [PATCH 4.14 00/72] 4.14.90-stable review
To: <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>, <[email protected]>This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
linux-4.14.y
and the diffstat can be found below.
Quoted 5 lines
Quoted 3 lines
thanks,
greg k-h
Merged, basic functional tests, no regression found.
Great, thanks for testing and letting me know.
greg k-h
Jon Hunter
maintainerQuoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.14.y
and the diffstat can be found below.thanks,
greg k-h
All tests passing for Tegra ...
Test results for stable-v4.14:
8 builds: 8 pass, 0 fail
16 boots: 16 pass, 0 fail
14 tests: 14 pass, 0 failLinux version: 4.14.90-rc1-gdafc7cb
Boards tested: tegra124-jetson-tk1, tegra20-ventana,
tegra210-p2371-2180, tegra30-cardhu-a04
Cheers
Jon
Signature · 1 line
nvpublic
Greg Kroah-Hartman
maintainerQuoted 22 lines · Jon Hunter
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.90 release. There are 72 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Dec 22 08:59:06 UTC 2018. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.90-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.14.y
and the diffstat can be found below.thanks,
greg k-h
All tests passing for Tegra ...
Quoted 9 lines
Test results for stable-v4.14:
8 builds: 8 pass, 0 fail 16 boots: 16 pass, 0 fail 14 tests: 14 pass, 0 failLinux version: 4.14.90-rc1-gdafc7cb
Boards tested: tegra124-jetson-tk1, tegra20-ventana,tegra210-p2371-2180, tegra30-cardhu-a04
Great, thanks for testing these and letting me know.
greg k-h