4.14.31-stable review
Subsystems (8)
Greg Kroah-Hartman
maintainercover 4.14.31-stable review
This is the start of the stable review cycle for the 4.14.31 release. There are 101 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 Thu Mar 29 16:27:21 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.31-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.31-rc1
Daniel Borkmann <[email protected]>
bpf, x64: increase number of passes
Chenbo Feng <[email protected]>
bpf: skip unnecessary capability check
Daniel Borkmann <[email protected]>
kbuild: disable clang's default use of -fmerge-all-constants
Dave Hansen <[email protected]>
x86/pkeys/selftests: Rename 'si_pkey' to 'siginfo_pkey'
Lu Baolu <[email protected]>
usb: xhci: Fix potential memory leak in xhci_disable_slot()
Lu Baolu <[email protected]>
usb: xhci: Disable slot even when virt-dev is null
Nadav Amit <[email protected]>
staging: lustre: ptlrpc: kfree used instead of kvfree
Liam Mark <[email protected]>
staging: android: ion: Zero CMA allocated memory
Lorenzo Bianconi <[email protected]>
iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
Linus Walleij <[email protected]>
iio: ABI: Fix name of timestamp sysfs file
Kan Liang <[email protected]>
perf/x86/intel/uncore: Fix multi-domain PCI CHA enumeration bug on Skylake servers
Dan Carpenter <[email protected]>
perf/x86/intel: Don't accidentally clear high bits in bdw_limit_period()
Song Liu <[email protected]>
perf/core: Fix ctx_event_type in ctx_resched()
Ilya Pronin <[email protected]>
perf stat: Fix CVS output format for non-supported counters
Kan Liang <[email protected]>
perf/x86/intel/uncore: Fix Skylake UPI event format
Chris Wilson <[email protected]>
drm/syncobj: Stop reusing the same struct file for all syncobj -> fd
H.J. Lu <[email protected]>
x86/boot/64: Verify alignment of the LOAD segment
H.J. Lu <[email protected]>
x86/build/64: Force the linker to use 2MB page size
Linus Torvalds <[email protected]>
kvm/x86: fix icebp instruction handling
Thomas Gleixner <[email protected]>
posix-timers: Protect posix clock array access against speculation
Waiman Long <[email protected]>
x86/efi: Free efi_pgd with free_pages()
Boris Ostrovsky <[email protected]>
x86/vsyscall/64: Use proper accessor to update P4D entry
Andy Lutomirski <[email protected]>
selftests/x86/ptrace_syscall: Fix for yet more glibc interference
Andy Lutomirski <[email protected]>
x86/entry/64: Don't use IST entry for #BP stack
Linus Torvalds <[email protected]>
tty: vt: fix up tabstops properly
Andri Yngvason <[email protected]>
can: cc770: Fix use after free in cc770_tx_interrupt()
Andri Yngvason <[email protected]>
can: cc770: Fix queue stall & dropped RTR reply
Andri Yngvason <[email protected]>
can: cc770: Fix stalls on rt-linux, remove redundant IRQ ack
Marek Vasut <[email protected]>
can: ifi: Check core revision upon probe
Marek Vasut <[email protected]>
can: ifi: Repair the error handling
Stephane Grosjean <[email protected]>
can: peak/pcie_fd: remove useless code when interface starts
Stephane Grosjean <[email protected]>
can: peak/pcie_fd: fix echo_skb is occupied! bug
Dan Carpenter <[email protected]>
staging: ncpfs: memory corruption in ncp_read_kernel()
Jagdish Gediya <[email protected]>
mtd: nand: fsl_ifc: Read ECCSTAT0 and ECCSTAT1 registers for IFC 2.0
Jagdish Gediya <[email protected]>
mtd: nand: fsl_ifc: Fix eccstat array overflow for IFC ver >= 2.0.0
Jagdish Gediya <[email protected]>
mtd: nand: fsl_ifc: Fix nand waitfunc return value
OuYang ZhiZhong <[email protected]>
mtdchar: fix usage of mtd_ooblayout_ecc()
Masami Hiramatsu <[email protected]>
tracing: probeevent: Fix to support minus offset from symbol
Larry Finger <[email protected]>
rtlwifi: rtl8723be: Fix loss of signal
Arend Van Spriel <[email protected]>
brcmfmac: fix P2P_DEVICE ethernet address generation
Vishal Verma <[email protected]>
libnvdimm, {btt, blk}: do integrity setup before add_disk()
Takashi Iwai <[email protected]>
ACPI / watchdog: Fix off-by-one error at resource assignment
Dan Williams <[email protected]>
acpi, numa: fix pxm to online numa node associations
Andrey Ryabinin <[email protected]>
mm/vmscan: wake up flushers for legacy cgroups too
Greg Kroah-Hartman <[email protected]>
drm: udl: Properly check framebuffer mmap offsets
Daniel Stone <[email protected]>
drm: Reject getfb for multi-plane framebuffers
Michel Dänzer <[email protected]>
drm/radeon: Don't turn off DP sink when disconnected
Thomas Hellstrom <[email protected]>
drm/vmwgfx: Fix a destoy-while-held mutex problem.
Thomas Hellstrom <[email protected]>
drm/vmwgfx: Fix black screen and device errors when running without fbdev
Daniel Vacek <[email protected]>
Revert "mm: page_alloc: skip over regions of invalid pfns where possible"
Kirill A. Shutemov <[email protected]>
mm/shmem: do not wait for lock_page() in shmem_unused_huge_shrink()
Kirill A. Shutemov <[email protected]>
mm/thp: do not wait for lock_page() in deferred_split_scan()
Kirill A. Shutemov <[email protected]>
mm/khugepaged.c: convert VM_BUG_ON() to collapse fail
Toshi Kani <[email protected]>
x86/mm: implement free pmd/pte page interfaces
Toshi Kani <[email protected]>
mm/vmalloc: add interfaces to free unmapped page table
Arnd Bergmann <[email protected]>
h8300: remove extraneous __BIG_ENDIAN definition
Mike Kravetz <[email protected]>
hugetlbfs: check for pgoff value overflow
Jeff Layton <[email protected]>
nfsd: remove blocked locks on client teardown
Tejun Heo <[email protected]>
cgroup: fix rule checking for threaded mode switching
Hans de Goede <[email protected]>
libata: Modify quirks for MX100 to limit NCQ_TRIM quirk to MU01 version
Hans de Goede <[email protected]>
libata: Make Crucial BX100 500GB LPM quirk apply to all firmware versions
Hans de Goede <[email protected]>
libata: Apply NOLPM quirk to Crucial M500 480 and 960GB SSDs
Ju Hyung Park <[email protected]>
libata: Enable queued TRIM for Samsung SSD 860
Kai-Heng Feng <[email protected]>
libata: disable LPM for Crucial BX100 SSD 500GB drive
Hans de Goede <[email protected]>
libata: Apply NOLPM quirk to Crucial MX100 512GB SSDs
Eric Biggers <[email protected]>
libata: don't try to pass through NCQ commands to non-NCQ devices
Eric Biggers <[email protected]>
libata: remove WARN() for DMA or PIO command without data
Eric Biggers <[email protected]>
libata: fix length validation of ATAPI-relayed SCSI commands
Takashi Iwai <[email protected]>
Bluetooth: btusb: Fix quirk for Atheros 1525/QCA6174
Kai-Heng Feng <[email protected]>
Bluetooth: btusb: Add Dell OptiPlex 3060 to btusb_needs_reset_resume_table
Hans de Goede <[email protected]>
Bluetooth: btusb: Remove Yoga 920 from the btusb_needs_reset_resume_table
Krzysztof Kozlowski <[email protected]>
pinctrl: samsung: Validate alias coming from DT
Michael Kelley <[email protected]>
Drivers: hv: vmbus: Fix ring buffer signaling
Leon Romanovsky <[email protected]>
RDMA/mlx5: Fix crash while accessing garbage pointer and freed memory
Chen-Yu Tsai <[email protected]>
clk: sunxi-ng: a31: Fix CLK_OUT_* clock ops
Boris Brezillon <[email protected]>
clk: bcm2835: Protect sections updating shared registers
Boris Brezillon <[email protected]>
clk: bcm2835: Fix ana->maskX definitions
Tetsuo Handa <[email protected]>
lockdep: fix fs_reclaim warning
Hans de Goede <[email protected]>
ahci: Add PCI-id for the Highpoint Rocketraid 644L card
Hans de Goede <[email protected]>
PCI: Add function 1 DMA alias quirk for Highpoint RocketRAID 644L
Evgeniy Didin <[email protected]>
mmc: dw_mmc: fix falling from idmac to PIO mode when dw_mci_reset occurs
Jaehoon Chung <[email protected]>
mmc: dw_mmc: exynos: fix the suspend/resume issue for exynos5433
Evgeniy Didin <[email protected]>
mmc: dw_mmc: Fix the DTO/CTO timeout overflow calculation for 32-bit systems
Bastian Stender <[email protected]>
mmc: block: fix updating ext_csd caches on ioctl call
Dirk Behme <[email protected]>
mmc: core: Disable HPI for certain Micron (Numonyx) eMMC cards
Adrian Hunter <[email protected]>
mmc: core: Fix tracepoint print of blk_addr and blksz
Takashi Iwai <[email protected]>
ALSA: hda/realtek - Always immediately update mute LED with pin VREF
Kailang Yang <[email protected]>
ALSA: hda/realtek - Fix Dell headset Mic can't record
Kailang Yang <[email protected]>
ALSA: hda/realtek - Fix speaker no sound after system resume
Takashi Iwai <[email protected]>
ALSA: hda - Force polling mode on CFL for fixing codec communication
Takashi Iwai <[email protected]>
ALSA: aloop: Fix access to not-yet-ready substream via cable
Takashi Iwai <[email protected]>
ALSA: aloop: Sync stale timer before release
Kirill Marinushkin <[email protected]>
ALSA: usb-audio: Fix parsing descriptor of UAC2 processing unit
Dan Carpenter <[email protected]>
iio: adc: meson-saradc: unlock on error in meson_sar_adc_lock()
Michael Nosthoff <[email protected]>
iio: st_pressure: st_accel: pass correct platform data to init
Richard Lai <[email protected]>
iio: chemical: ccs811: Corrected firmware boot/application mode transition
Mathias Kresin <[email protected]>
MIPS: lantiq: ase: Enable MFD_SYSCON
Mathias Kresin <[email protected]>
MIPS: lantiq: Enable AHB Bus for USB
Mathias Kresin <[email protected]>
MIPS: lantiq: Fix Danube USB clock
NeilBrown <[email protected]>
MIPS: ralink: Fix booting on MT7621
NeilBrown <[email protected]>
MIPS: ralink: Remove ralink_halt()
-------------
Diffstat:
Documentation/ABI/testing/sysfs-bus-iio | 2 +- Makefile | 13 ++- arch/arm64/mm/mmu.c | 10 +++ arch/h8300/include/asm/byteorder.h | 1 - arch/mips/lantiq/Kconfig | 2 + arch/mips/lantiq/xway/sysctrl.c | 6 +- arch/mips/ralink/mt7621.c | 42 ++++----- arch/mips/ralink/reset.c | 7 -- arch/x86/Makefile | 9 ++ arch/x86/boot/compressed/misc.c | 4 + arch/x86/entry/entry_64.S | 2 +- arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- arch/x86/events/intel/core.c | 2 +- arch/x86/events/intel/uncore_snbep.c | 33 +++---- arch/x86/include/asm/vmx.h | 1 + arch/x86/kernel/idt.c | 2 - arch/x86/kernel/traps.c | 15 ++-- arch/x86/kvm/vmx.c | 9 +- arch/x86/mm/pgtable.c | 48 ++++++++++ arch/x86/net/bpf_jit_comp.c | 3 +- arch/x86/platform/efi/efi_64.c | 2 +- drivers/acpi/acpi_watchdog.c | 4 +- drivers/acpi/numa.c | 10 ++- drivers/ata/ahci.c | 4 +- drivers/ata/libata-core.c | 26 +++++- drivers/ata/libata-scsi.c | 10 ++- drivers/bluetooth/btusb.c | 8 +- drivers/clk/bcm/clk-bcm2835.c | 12 ++- drivers/clk/sunxi-ng/ccu-sun6i-a31.c | 6 +- drivers/gpu/drm/drm_framebuffer.c | 7 ++ drivers/gpu/drm/drm_syncobj.c | 81 +++++++---------- drivers/gpu/drm/radeon/radeon_connectors.c | 31 +++---- drivers/gpu/drm/udl/udl_fb.c | 9 +- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 13 +++ drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 1 + drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 39 ++++++-- drivers/gpu/drm/vmwgfx/vmwgfx_kms.h | 13 ++- drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | 5 +- drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c | 5 +- drivers/hv/ring_buffer.c | 52 +++++++---- drivers/iio/accel/st_accel_core.c | 2 +- drivers/iio/adc/meson_saradc.c | 4 +- drivers/iio/chemical/ccs811.c | 3 + drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 2 +- drivers/iio/pressure/st_pressure_core.c | 2 +- drivers/infiniband/hw/mlx5/mr.c | 12 ++- drivers/mmc/core/block.c | 19 ++++ drivers/mmc/core/card.h | 1 + drivers/mmc/core/quirks.h | 6 ++ drivers/mmc/host/dw_mmc-exynos.c | 8 +- drivers/mmc/host/dw_mmc.c | 15 ++-- drivers/mtd/mtdchar.c | 4 +- drivers/mtd/nand/fsl_ifc_nand.c | 32 +++---- drivers/net/can/cc770/cc770.c | 100 +++++++++++++-------- drivers/net/can/cc770/cc770.h | 2 + drivers/net/can/ifi_canfd/ifi_canfd.c | 75 ++++++++++------ drivers/net/can/peak_canfd/peak_canfd.c | 25 ++---- drivers/net/can/peak_canfd/peak_pciefd_main.c | 8 +- .../net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 24 +++-- .../net/wireless/realtek/rtlwifi/rtl8723be/hw.c | 3 +- drivers/nvdimm/blk.c | 3 +- drivers/nvdimm/btt.c | 3 +- drivers/pci/quirks.c | 2 + drivers/pinctrl/samsung/pinctrl-exynos-arm.c | 56 ++++++++++-- drivers/pinctrl/samsung/pinctrl-exynos-arm64.c | 14 ++- drivers/pinctrl/samsung/pinctrl-s3c24xx.c | 28 +++++- drivers/pinctrl/samsung/pinctrl-s3c64xx.c | 7 +- drivers/pinctrl/samsung/pinctrl-samsung.c | 61 ++++++++----- drivers/pinctrl/samsung/pinctrl-samsung.h | 40 +++++---- drivers/staging/android/ion/ion_cma_heap.c | 17 ++++ drivers/staging/lustre/lustre/ptlrpc/sec.c | 2 +- drivers/tty/vt/vt.c | 8 +- drivers/usb/host/xhci-hub.c | 5 +- drivers/usb/host/xhci.c | 34 ++----- drivers/usb/host/xhci.h | 3 +- drivers/watchdog/wdat_wdt.c | 2 +- fs/hugetlbfs/inode.c | 17 +++- fs/ncpfs/ncplib_kernel.c | 4 + fs/nfsd/nfs4state.c | 62 +++++++++---- include/asm-generic/pgtable.h | 10 +++ include/drm/drm_syncobj.h | 5 -- include/linux/fsl_ifc.h | 6 +- include/linux/memblock.h | 1 - include/trace/events/mmc.h | 4 +- include/uapi/linux/usb/audio.h | 4 +- kernel/bpf/syscall.c | 2 +- kernel/cgroup/cgroup.c | 10 +++ kernel/events/core.c | 4 +- kernel/time/posix-timers.c | 11 ++- kernel/trace/trace_kprobe.c | 4 +- kernel/trace/trace_probe.c | 8 +- kernel/trace/trace_probe.h | 2 +- lib/ioremap.c | 6 +- mm/huge_memory.c | 4 +- mm/hugetlb.c | 7 ++ mm/khugepaged.c | 7 +- mm/memblock.c | 28 ------ mm/page_alloc.c | 13 +-- mm/shmem.c | 31 ++++--- mm/vmscan.c | 31 +++---- sound/drivers/aloop.c | 17 +++- sound/pci/hda/hda_intel.c | 5 ++ sound/pci/hda/patch_realtek.c | 20 ++++- tools/perf/builtin-stat.c | 2 +- tools/testing/selftests/x86/protection_keys.c | 10 +-- tools/testing/selftests/x86/ptrace_syscall.c | 8 +- 106 files changed, 979 insertions(+), 547 deletions(-)
Greg Kroah-Hartman
maintainerPATCH 1/101 MIPS: ralink: Remove ralink_halt()
4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: NeilBrown <[email protected]>
commit 891731f6a5dbe508d12443175a7e166a2fba616a upstream.
ralink_halt() does nothing that machine_halt() doesn't already do, so it adds no value.
It actually causes incorrect behaviour due to the "unreachable()" at the end. This tells the compiler that the end of the function will never be reached, which isn't true. The compiler responds by not adding a 'return' instruction, so control simply moves on to whatever bytes come afterwards in memory. In my tested, that was the ralink_restart() function. This means that an attempt to 'halt' the machine would actually cause a reboot.
So remove ralink_halt() so that a 'halt' really does halt.
Patchwork: https://patchwork.linux-mips.org/patch/18851/
--- arch/mips/ralink/reset.c | 7 ------- 1 file changed, 7 deletions(-)
arch/mips/ralink/reset.c +0 -7
@@ -96,16 +96,9 @@ static void ralink_restart(char *commandunreachable (); } -static void ralink_halt (void ) -{ -local_irq_disable (); -unreachable (); -} -static int __init mips_reboot_setup (void ) { _machine_restart = ralink_restart; - _machine_halt = ralink_halt;return 0 ; }
Greg Kroah-Hartman
maintainerPATCH 2/101 MIPS: ralink: Fix booting on MT7621
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: NeilBrown <[email protected]>
commit a63d706ea719190a79a6c769e898f70680044d3e upstream.
Since commit 3af5a67c86a3 ("MIPS: Fix early CM probing") the MT7621 has not been able to boot.
This commit caused mips_cm_probe() to be called before mt7621.c::proc_soc_init().
prom_soc_init() has a comment explaining that mips_cm_probe() "wipes out the bootloader config" and means that configuration registers are no longer available. It has some code to re-enable this config.
Before this re-enable code is run, the sysc register cannot be read, so when SYSC_REG_CHIP_NAME0 is read, a garbage value is returned and panic() is called.
If we move the config-repair code to the top of prom_soc_init(), the registers can be read and boot can proceed.
Very occasionally, the first register read after the reconfiguration returns garbage, so add a call to __sync().
Patchwork: https://patchwork.linux-mips.org/patch/18859/
--- arch/mips/ralink/mt7621.c | 42 ++++++++++++++++++++++-------------------- 1 file changed, 22 insertions(+), 20 deletions(-)
arch/mips/ralink/mt7621.c +22 -20
@@ -170,6 +170,28 @@ void prom_soc_init(struct ralink_soc_infu32 n1;u32 rev; +/* Early detection of CMP support */ +mips_cm_probe (); +mips_cpc_probe (); + +if (mips_cps_numiocu (0 )) { +/* +* mips_cm_probe() wipes out bootloader +* config for CM regions and we have to configure them +* again. This SoC cannot talk to pamlbus devices +* witout proper iocu region set up. +* +* FIXME: it would be better to do this with values +* from DT, but we need this very early because +* without this we cannot talk to pretty much anything +* including serial. +*/ +write_gcr_reg0_base (MT7621_PALMBUS_BASE ); +write_gcr_reg0_mask (~MT7621_PALMBUS_SIZE | + CM_GCR_REGn_MASK_CMTGT_IOCU0); +__sync (); + } + n0 =__raw_readl (sysc +SYSC_REG_CHIP_NAME0 ); n1 =__raw_readl (sysc +SYSC_REG_CHIP_NAME1 );
@@ -194,26 +216,6 @@ void prom_soc_init(struct ralink_soc_inf rt2880_pinmux_data = mt7621_pinmux_data; -/* Early detection of CMP support */ -mips_cm_probe (); -mips_cpc_probe (); - -if (mips_cps_numiocu (0 )) { -/* -* mips_cm_probe() wipes out bootloader -* config for CM regions and we have to configure them -* again. This SoC cannot talk to pamlbus devices -* witout proper iocu region set up. -* -* FIXME: it would be better to do this with values -* from DT, but we need this very early because -* without this we cannot talk to pretty much anything -* including serial. -*/ -write_gcr_reg0_base (MT7621_PALMBUS_BASE ); -write_gcr_reg0_mask (~MT7621_PALMBUS_SIZE | - CM_GCR_REGn_MASK_CMTGT_IOCU0); - }if (!register_cps_smp_ops ())return ;
Greg Kroah-Hartman
maintainerPATCH 3/101 MIPS: lantiq: Fix Danube USB clock
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mathias Kresin <[email protected]>
commit 214cbc14734958fe533916fdb4194f5983ad4bc4 upstream.
On Danube the USB0 controller registers are at 1e101000 and the USB0 PHY register is at 1f203018 similar to all other lantiq SoCs. Activate the USB controller gating clock thorough the USB controller driver and not the PHY.
This fixes a problem introduced in a previous commit.
Patchwork: https://patchwork.linux-mips.org/patch/18816/
--- arch/mips/lantiq/xway/sysctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/mips/lantiq/xway/sysctrl.c +1 -1
@@ -562,7 +562,7 @@ void __init ltq_soc_init(void) }else {clkdev_add_static (ltq_danube_cpu_hz (),ltq_danube_fpi_hz (),ltq_danube_fpi_hz (),ltq_danube_pp32_hz ()); -clkdev_add_pmu ("1f203018.usb2-phy" ,"ctrl" ,1 ,0 ,PMU_USB0 ); +clkdev_add_pmu ("1e101000.usb" ,"otg" ,1 ,0 ,PMU_USB0 );clkdev_add_pmu ("1f203018.usb2-phy" ,"phy" ,1 ,0 ,PMU_USB0_P );clkdev_add_pmu ("1e103000.sdio" ,NULL ,1 ,0 ,PMU_SDIO );clkdev_add_pmu ("1e103100.deu" ,NULL ,1 ,0 ,PMU_DEU );
Greg Kroah-Hartman
maintainerPATCH 4/101 MIPS: lantiq: Enable AHB Bus for USB
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mathias Kresin <[email protected]>
commit 3223a5a7d3a606dcb7d9190a788b9544a45441ee upstream.
On Danube and AR9 the USB core is connected though a AHB bus to the main system cross bar, hence we need to enable the gating clock of the AHB Bus as well to make the USB controller work.
Patchwork: https://patchwork.linux-mips.org/patch/18814/
--- arch/mips/lantiq/xway/sysctrl.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
arch/mips/lantiq/xway/sysctrl.c +3 -3
@@ -551,9 +551,9 @@ void __init ltq_soc_init(void)clkdev_add_static (ltq_ar9_cpu_hz (),ltq_ar9_fpi_hz (),ltq_ar9_fpi_hz (),CLOCK_250M );clkdev_add_pmu ("1f203018.usb2-phy" ,"phy" ,1 ,0 ,PMU_USB0_P ); -clkdev_add_pmu ("1e101000.usb" ,"otg" ,1 ,0 ,PMU_USB0 ); +clkdev_add_pmu ("1e101000.usb" ,"otg" ,1 ,0 ,PMU_USB0 |PMU_AHBM );clkdev_add_pmu ("1f203034.usb2-phy" ,"phy" ,1 ,0 ,PMU_USB1_P ); -clkdev_add_pmu ("1e106000.usb" ,"otg" ,1 ,0 ,PMU_USB1 ); +clkdev_add_pmu ("1e106000.usb" ,"otg" ,1 ,0 ,PMU_USB1 |PMU_AHBM );clkdev_add_pmu ("1e180000.etop" ,"switch" ,1 ,0 ,PMU_SWITCH );clkdev_add_pmu ("1e103000.sdio" ,NULL ,1 ,0 ,PMU_SDIO );clkdev_add_pmu ("1e103100.deu" ,NULL ,1 ,0 ,PMU_DEU );
@@ -562,7 +562,7 @@ void __init ltq_soc_init(void) }else {clkdev_add_static (ltq_danube_cpu_hz (),ltq_danube_fpi_hz (),ltq_danube_fpi_hz (),ltq_danube_pp32_hz ()); -clkdev_add_pmu ("1e101000.usb" ,"otg" ,1 ,0 ,PMU_USB0 ); +clkdev_add_pmu ("1e101000.usb" ,"otg" ,1 ,0 ,PMU_USB0 |PMU_AHBM );clkdev_add_pmu ("1f203018.usb2-phy" ,"phy" ,1 ,0 ,PMU_USB0_P );clkdev_add_pmu ("1e103000.sdio" ,NULL ,1 ,0 ,PMU_SDIO );clkdev_add_pmu ("1e103100.deu" ,NULL ,1 ,0 ,PMU_DEU );
Greg Kroah-Hartman
maintainerPATCH 5/101 MIPS: lantiq: ase: Enable MFD_SYSCON
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mathias Kresin <[email protected]>
commit a821328c2f3003b908880792d71b2781b44fa53c upstream.
Enable syscon to use it for the RCU MFD on Amazon SE as well.
The Amazon SE also has similar reset controller system as Danube and XWAY and use their drivers mostly. As these drivers now need syscon also activate the syscon subsystem for for Amazon SE.
Patchwork: https://patchwork.linux-mips.org/patch/18817/
--- arch/mips/lantiq/Kconfig | 2 ++ 1 file changed, 2 insertions(+)
arch/mips/lantiq/Kconfig +2 -0
@@ -13,6 +13,8 @@ choice config SOC_AMAZON_SE bool "Amazon SE" select SOC_TYPE_XWAY + select MFD_SYSCON + select MFD_CORE config SOC_XWAY bool "XWAY"
Greg Kroah-Hartman
maintainerPATCH 6/101 iio: chemical: ccs811: Corrected firmware boot/application mode transition
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Richard Lai <[email protected]>
commit b91e146c38b003c899710ede6d05fc824675e386 upstream.
CCS811 has different I2C register maps in boot and application mode. When CCS811 is in boot mode, register APP_START (0xF4) is used to transit the firmware state from boot to application mode. However, APP_START is not a valid register location when CCS811 is in application mode (refer to "CCS811 Bootloader Register Map" and "CCS811 Application Register Map" in CCS811 datasheet). The driver should not attempt to perform a write to APP_START while CCS811 is in application mode, as this is not a valid or documented register location.
When prob function is being called, the driver assumes the CCS811 sensor is in boot mode, and attempts to perform a write to APP_START. Although CCS811 powers-up in boot mode, it may have already been transited to application mode by previous instances, e.g. unload and reload device driver by the system, or explicitly by user. Depending on the system design, CCS811 sensor may be permanently connected to system power source rather than power controlled by GPIO, hence it is possible that the sensor is never power reset, thus the firmware could be in either boot or application mode at any given time when driver prob function is being called.
This patch checks the STATUS register before attempting to send a write to APP_START. Only if the firmware is not in application mode and has valid firmware application loaded, then it will continue to start transiting the firmware boot to application mode.
--- drivers/iio/chemical/ccs811.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/iio/chemical/ccs811.c +3 -0
@@ -128,6 +128,9 @@ static int ccs811_start_sensor_applicatiif (ret <0 )return ret; +if ((ret &CCS811_STATUS_FW_MODE_APPLICATION )) +return 0 ; +if ((ret &CCS811_STATUS_APP_VALID_MASK ) !=CCS811_STATUS_APP_VALID_LOADED )return -EIO ;
Greg Kroah-Hartman
maintainerPATCH 7/101 iio: st_pressure: st_accel: pass correct platform data to init
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Michael Nosthoff <[email protected]>
commit 8b438686a001db64c21782d04ef68111e53c45d9 upstream.
Commit 7383d44b added a pointer pdata which get set to the default platform_data when non was defined in the device. But it did not pass this pointer to the st_sensors_init_sensor call but still used the maybe uninitialized platform_data from dev.
This breaks initialization when no platform_data is given and the optional st,drdy-int-pin devicetree option is not set.
This commit fixes this.
--- drivers/iio/accel/st_accel_core.c | 2 +- drivers/iio/pressure/st_pressure_core.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/iio/accel/st_accel_core.c +1 -1
@@ -794,7 +794,7 @@ int st_accel_common_probe(struct iio_devif (!pdata) pdata = (struct st_sensors_platform_data *)&default_accel_pdata; - err =st_sensors_init_sensor (indio_dev, adata->dev->platform_data); + err =st_sensors_init_sensor (indio_dev, pdata);if (err <0 ) goto st_accel_power_off;
drivers/iio/pressure/st_pressure_core.c +1 -1
@@ -617,7 +617,7 @@ int st_press_common_probe(struct iio_devif (!pdata && press_data->sensor_settings->drdy_irq.addr) pdata = (struct st_sensors_platform_data *)&default_press_pdata; - err =st_sensors_init_sensor (indio_dev, press_data->dev->platform_data); + err =st_sensors_init_sensor (indio_dev, pdata);if (err <0 ) goto st_press_power_off;
Greg Kroah-Hartman
maintainerPATCH 8/101 iio: adc: meson-saradc: unlock on error in meson_sar_adc_lock()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dan Carpenter <[email protected]>
commit 3c3e4b3a708a9d6451052e348981f37d2b3e92b0 upstream.
The meson_sar_adc_lock() function is not supposed to hold the "indio_dev->mlock" on the error path.
--- drivers/iio/adc/meson_saradc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/iio/adc/meson_saradc.c +3 -1
@@ -462,8 +462,10 @@ static int meson_sar_adc_lock(struct iioregmap_read (priv->regmap,MESON_SAR_ADC_DELAY , &val); }while (val &MESON_SAR_ADC_DELAY_BL30_BUSY && timeout--); -if (timeout <0 ) +if (timeout <0 ) { +mutex_unlock (&indio_dev->mlock);return -ETIMEDOUT ; + } }return 0 ;
Greg Kroah-Hartman
maintainerPATCH 9/101 ALSA: usb-audio: Fix parsing descriptor of UAC2 processing unit
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kirill Marinushkin <[email protected]>
commit a6618f4aedb2b60932d766bd82ae7ce866e842aa upstream.
Currently, the offsets in the UAC2 processing unit descriptor are calculated incorrectly. It causes an issue when connecting the device which provides such a feature:
~~~~
[84126.724420] usb 1-1.3.1: invalid Processing Unit descriptor (id 18)
~~~~
After this patch is applied, the UAC2 processing unit inits w/o this error.
--- include/uapi/linux/usb/audio.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
include/uapi/linux/usb/audio.h +2 -2
@@ -370,7 +370,7 @@ static inline __u8 uac_processing_unit_b {return (protocol ==UAC_VERSION_1 ) ? desc->baSourceID[desc->bNrInPins +4 ] : - desc->baSourceID[desc->bNrInPins +6 ]; +2 ;/* in UAC2, this value is constant */ } static inline __u8 *uac_processing_unit_bmControls(struct uac_processing_unit_descriptor *desc,
@@ -378,7 +378,7 @@ static inline __u8 *uac_processing_unit_ {return (protocol ==UAC_VERSION_1 ) ? &desc->baSourceID[desc->bNrInPins +5 ] : - &desc->baSourceID[desc->bNrInPins +7 ]; + &desc->baSourceID[desc->bNrInPins +6 ]; } static inline __u8 uac_processing_unit_iProcessing(struct uac_processing_unit_descriptor *desc,
Greg Kroah-Hartman
maintainerPATCH 10/101 ALSA: aloop: Sync stale timer before release
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit 67a01afaf3d34893cf7d2ea19b34555d6abb7cb0 upstream.
The aloop driver tries to stop the pending timer via timer_del() in the trigger callback and in the close callback. The former is correct, as it's an atomic operation, while the latter expects that the timer gets really removed and proceeds the resource releases after that. But timer_del() doesn't synchronize, hence the running timer may still access the released resources.
A similar situation can be also seen in the prepare callback after trigger(STOP) where the prepare tries to re-initialize the things while a timer is still running.
The problems like the above are seen indirectly in some syzkaller reports (although it's not 100% clear whether this is the only cause, as the race condition is quite narrow and not always easy to trigger).
For addressing these issues, this patch adds the explicit alls of timer_del_sync() in some places, so that the pending timer is properly killed / synced.
--- sound/drivers/aloop.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
sound/drivers/aloop.c +8 -1
@@ -192,6 +192,11 @@ static inline void loopback_timer_stop(s dpcm->timer.expires =0 ; } +static inline void loopback_timer_stop_sync (struct loopback_pcm *dpcm) +{ +del_timer_sync (&dpcm->timer); +} +#define CABLE_VALID_PLAYBACK (1 << SNDRV_PCM_STREAM_PLAYBACK)#define CABLE_VALID_CAPTURE (1 << SNDRV_PCM_STREAM_CAPTURE)#define CABLE_VALID_BOTH (CABLE_VALID_PLAYBACK|CABLE_VALID_CAPTURE)
@@ -326,6 +331,8 @@ static int loopback_prepare(struct snd_pstruct loopback_cable *cable = dpcm->cable;int bps, salign; +loopback_timer_stop_sync (dpcm); + salign = (snd_pcm_format_width (runtime->format) * runtime->channels) /8 ; bps = salign * runtime->rate;
@@ -745,7 +752,7 @@ static int loopback_close(struct snd_pcmstruct loopback *loopback = substream->private_data;struct loopback_pcm *dpcm = substream->runtime->private_data; -loopback_timer_stop (dpcm); +loopback_timer_stop_sync (dpcm);mutex_lock (&loopback->cable_lock);free_cable (substream);mutex_unlock (&loopback->cable_lock);
Greg Kroah-Hartman
maintainerPATCH 11/101 ALSA: aloop: Fix access to not-yet-ready substream via cable
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit 8e6b1a72a75bb5067ccb6b56d8ca4aa3a300a64e upstream.
In loopback_open() and loopback_close(), we assign and release the substream object to the corresponding cable in a racy way. It's neither locked nor done in the right position. The open callback assigns the substream before its preparation finishes, hence the other side of the cable may pick it up, which may lead to the invalid memory access.
This patch addresses these: move the assignment to the end of the open callback, and wrap with cable->lock for avoiding concurrent accesses.
--- sound/drivers/aloop.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
sound/drivers/aloop.c +7 -1
@@ -666,7 +666,9 @@ static void free_cable(struct snd_pcm_sureturn ;if (cable->streams[!substream->stream]) {/* other stream is still alive */ +spin_lock_irq (&cable->lock); cable->streams[substream->stream] =NULL ; +spin_unlock_irq (&cable->lock); }else {/* free the cable */ loopback->cables[substream->number][dev] =NULL ;
@@ -706,7 +708,6 @@ static int loopback_open(struct snd_pcm_ loopback->cables[substream->number][dev] = cable; } dpcm->cable = cable; - cable->streams[substream->stream] = dpcm;snd_pcm_hw_constraint_integer (runtime,SNDRV_PCM_HW_PARAM_PERIODS );
@@ -738,6 +739,11 @@ static int loopback_open(struct snd_pcm_ runtime->hw = loopback_pcm_hardware; else runtime->hw = cable->hw; + +spin_lock_irq (&cable->lock); + cable->streams[substream->stream] = dpcm; +spin_unlock_irq (&cable->lock); + unlock:if (err <0 ) {free_cable (substream);
Greg Kroah-Hartman
maintainerPATCH 12/101 ALSA: hda - Force polling mode on CFL for fixing codec communication
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit a8d7bde23e7130686b76624b099f3e22dd38aef7 upstream.
We've observed too long probe time with Coffee Lake (CFL) machines, and the likely cause is some communication problem between the HD-audio controller and the codec chips. While the controller expects an IRQ wakeup for each codec response, it seems sometimes missing, and it takes one second for the controller driver to time out and read the response in the polling mode.
Although we aren't sure about the real culprit yet, in this patch, we put a workaround by forcing the polling mode as default for CFL machines; the polling mode itself isn't too heavy, and much better than other workarounds initially suggested (e.g. disabling power-save), at least.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=199007
--- sound/pci/hda/hda_intel.c | 5 +++++ 1 file changed, 5 insertions(+)
sound/pci/hda/hda_intel.c +5 -0
@@ -375,6 +375,7 @@ enum { ((pci)->device == 0x160c))#define IS_BXT (pci) ((pci)->vendor == 0x8086 && (pci)->device == 0x5a98) +#define IS_CFL (pci) ((pci)->vendor == 0x8086 && (pci)->device == 0xa348)static char *driver_short_names[] = { [AZX_DRIVER_ICH ] ="HDA Intel" ,
@@ -1744,6 +1745,10 @@ static int azx_create(struct snd_card *celse chip->bdl_pos_adj = bdl_pos_adj[dev]; +/* Workaround for a communication error on CFL (bko#199007) */ +if (IS_CFL (pci)) + chip->polling_mode =1 ; + err =azx_bus_init (chip, model[dev], &pci_hda_io_ops);if (err <0 ) {kfree (hda);
Greg Kroah-Hartman
maintainerPATCH 13/101 ALSA: hda/realtek - Fix speaker no sound after system resume
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kailang Yang <[email protected]>
commit 88d42b2b45d7208cc872c2c9dec0b1ae6c6008d7 upstream.
It will have a chance speaker no sound after system resume. To toggle NID 0x53 index 0x2 bit 15 will solve this issue. This usage will also suitable with ALC256.
--- sound/pci/hda/patch_realtek.c | 4 ++++ 1 file changed, 4 insertions(+)
sound/pci/hda/patch_realtek.c +4 -0
@@ -3107,6 +3107,8 @@ static void alc256_init(struct hda_codecalc_update_coef_idx (codec,0x46 ,3 <<12 ,0 );alc_update_coefex_idx (codec,0x57 ,0x04 ,0x0007 ,0x4 );/* Hight power */ +alc_update_coefex_idx (codec,0x53 ,0x02 ,0x8000 ,1 <<15 );/* Clear bit */ +alc_update_coefex_idx (codec,0x53 ,0x02 ,0x8000 ,0 <<15 ); }static void alc256_shutup (struct hda_codec *codec)
@@ -7004,6 +7006,8 @@ static int patch_alc269(struct hda_codecbreak ;case 0x10ec0257 : spec->codec_variant =ALC269_TYPE_ALC257 ; + spec->shutup = alc256_shutup; + spec->init_hook = alc256_init; spec->gen.mixer_nid =0 ;break ;case 0x10ec0215 :
Greg Kroah-Hartman
maintainerPATCH 14/101 ALSA: hda/realtek - Fix Dell headset Mic cant record
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kailang Yang <[email protected]>
commit f0ba9d699e5ca2bcd07f70185d18720c4f1b597c upstream.
This platform was hardware fixed type for CTIA type for headset port. Assigned 0x19 verb will fix can't record issue.
--- sound/pci/hda/patch_realtek.c | 10 ++++++++++ 1 file changed, 10 insertions(+)
sound/pci/hda/patch_realtek.c +10 -0
@@ -5354,6 +5354,7 @@ enum {ALC274_FIXUP_DELL_AIO_LINEOUT_VERB ,ALC298_FIXUP_TPT470_DOCK ,ALC255_FIXUP_DUMMY_LINEOUT_VERB , + ALC255_FIXUP_DELL_HEADSET_MIC, }; static const struct hda_fixup alc269_fixups[] = {
@@ -6214,6 +6215,13 @@ static const struct hda_fixup alc269_fix .chained = true, .chain_id =ALC255_FIXUP_DELL1_MIC_NO_PRESENCE }, + [ALC255_FIXUP_DELL_HEADSET_MIC] = { + .type = HDA_FIXUP_PINS, + .v.pins = (const struct hda_pintbl []) { + {0x19 ,0x01a1913c },/* use as headset mic, without its own jack detect */ + { } + }, + }, };static const struct snd_pci_quirk alc269_fixup_tbl[] = {
@@ -6268,6 +6276,8 @@ static const struct snd_pci_quirk alc269SND_PCI_QUIRK (0x1028 ,0x082a ,"Dell XPS 13 9360" ,ALC256_FIXUP_DELL_XPS_13_HEADPHONE_NOISE ),SND_PCI_QUIRK (0x1028 ,0x084b ,"Dell" ,ALC274_FIXUP_DELL_AIO_LINEOUT_VERB ),SND_PCI_QUIRK (0x1028 ,0x084e ,"Dell" ,ALC274_FIXUP_DELL_AIO_LINEOUT_VERB ), +SND_PCI_QUIRK (0x1028 ,0x0871 ,"Dell Precision 3630" ,ALC255_FIXUP_DELL_HEADSET_MIC ), +SND_PCI_QUIRK (0x1028 ,0x0872 ,"Dell Precision 3630" ,ALC255_FIXUP_DELL_HEADSET_MIC ),SND_PCI_QUIRK (0x1028 ,0x0873 ,"Dell Precision 3930" ,ALC255_FIXUP_DUMMY_LINEOUT_VERB ),SND_PCI_QUIRK (0x1028 ,0x164a ,"Dell" ,ALC293_FIXUP_DELL1_MIC_NO_PRESENCE ), SND_PCI_QUIRK(0x1028, 0x164b, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE),
Greg Kroah-Hartman
maintainerPATCH 15/101 ALSA: hda/realtek - Always immediately update mute LED with pin VREF
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit e40bdb03d3cd7da66bd0bc1e40cbcfb49351265c upstream.
Some HP laptops have a mute mute LED controlled by a pin VREF. The Realtek codec driver updates the VREF via vmaster hook by calling snd_hda_set_pin_ctl_cache().
This works fine as long as the driver is running in a normal mode. However, when the VREF change happens during the codec being in runtime PM suspend, the regmap access will skip and postpone the actual register change. This ends up with the unchanged LED status until the next runtime PM resume even if you change the Master mute switch. (Interestingly, the machine keeps the LED status even after the codec goes into D3 -- but it's another story.)
For improving this usability, let the driver temporarily powering up / down only during the pin VREF change. This can be achieved easily by wrapping the call with snd_hda_power_up_pm() / *_down_pm().
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=199073
--- sound/pci/hda/patch_realtek.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
sound/pci/hda/patch_realtek.c +5 -1
@@ -3488,8 +3488,12 @@ static void alc269_fixup_mic_mute_hook(v pinval =snd_hda_codec_get_pin_target (codec, spec->mute_led_nid); pinval &= ~AC_PINCTL_VREFEN ; pinval |= enabled ?AC_PINCTL_VREF_HIZ :AC_PINCTL_VREF_80 ; -if (spec->mute_led_nid) +if (spec->mute_led_nid) { +/* temporarily power up/down for setting VREF */ +snd_hda_power_up_pm (codec);snd_hda_set_pin_ctl_cache (codec, spec->mute_led_nid, pinval); +snd_hda_power_down_pm (codec); + } }/* Make sure the led works even in runtime suspend */
Greg Kroah-Hartman
maintainerPATCH 16/101 mmc: core: Fix tracepoint print of blk_addr and blksz
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Adrian Hunter <[email protected]>
commit c658dc58c7eaa8569ceb0edd1ddbdfda84fe8aa5 upstream.
Swap the positions of blk_addr and blksz in the tracepoint print arguments so that they match the print format.
--- include/trace/events/mmc.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
include/trace/events/mmc.h +2 -2
@@ -86,8 +86,8 @@ TRACE_EVENT(mmc_request_start, __entry->stop_flags, __entry->stop_retries, __entry->sbc_opcode, __entry->sbc_arg, __entry->sbc_flags, __entry->sbc_retries, - __entry->blocks, __entry->blk_addr, - __entry->blksz, __entry->data_flags, __entry->tag, + __entry->blocks, __entry->blksz, + __entry->blk_addr, __entry->data_flags, __entry->tag, __entry->can_retune, __entry->doing_retune, __entry->retune_now, __entry->need_retune, __entry->hold_retune, __entry->retune_period)
Greg Kroah-Hartman
maintainerPATCH 17/101 mmc: core: Disable HPI for certain Micron (Numonyx) eMMC cards
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dirk Behme <[email protected]>
commit dbe7dc6b9b28f5b012b0bedc372aa0c52521f3e4 upstream.
Certain Micron eMMC v4.5 cards might get broken when HPI feature is used and hence this patch disables the HPI feature for such buggy cards.
In U-Boot, these cards are reported as
Manufacturer: Micron (ID: 0xFE)
OEM: 0x4E
Name: MMC32G
Revision: 19 (0x13)
Serial: 959241022 Manufact. date: 8/2015 (0x82) CRC: 0x00 Tran Speed: 52000000
Rd Block Len: 512
MMC version 4.5
High Capacity: Yes
Capacity: 29.1 GiB
Boot Partition Size: 16 MiB
Bus Width: 8-bit
According to JEDEC JEP106 manufacturer 0xFE is Numonyx, which was bought by Micron.
--- drivers/mmc/core/card.h | 1 + drivers/mmc/core/quirks.h | 6 ++++++ 2 files changed, 7 insertions(+)
drivers/mmc/core/card.h +1 -0
@@ -82,6 +82,7 @@ struct mmc_fixup {#define CID_MANFID_APACER 0x27#define CID_MANFID_KINGSTON 0x70#define CID_MANFID_HYNIX 0x90 +#define CID_MANFID_NUMONYX 0xFE#define END_FIXUP { NULL }
drivers/mmc/core/quirks.h +6 -0
@@ -109,6 +109,12 @@ static const struct mmc_fixup mmc_ext_cs*/ MMC_FIXUP_EXT_CSD_REV (CID_NAME_ANY ,CID_MANFID_HYNIX ,0x014a , add_quirk,MMC_QUIRK_BROKEN_HPI ,5 ), +/* +* Certain Micron (Numonyx) eMMC 4.5 cards might get broken when HPI +* feature is used so disable the HPI feature for such buggy cards. +*/ +MMC_FIXUP_EXT_CSD_REV (CID_NAME_ANY ,CID_MANFID_NUMONYX , +0x014e , add_quirk,MMC_QUIRK_BROKEN_HPI ,6 ), END_FIXUP };
Greg Kroah-Hartman
maintainerPATCH 18/101 mmc: block: fix updating ext_csd caches on ioctl call
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Bastian Stender <[email protected]>
commit e74ef2194b41ba5e511fab29fe5ff00e72d2f42a upstream.
PARTITION_CONFIG is cached in mmc_card->ext_csd.part_config and the currently active partition in mmc_blk_data->part_curr. These caches do not always reflect changes if the ioctl call modifies the PARTITION_CONFIG registers, e.g. by changing BOOT_PARTITION_ENABLE.
Write the PARTITION_CONFIG value extracted from the ioctl call to the cache and update the currently active partition accordingly. This ensures that the user space cannot change the values behind the kernel's back. The next call to mmc_blk_part_switch() will operate on the data set by the ioctl and reflect the changes appropriately.
--- drivers/mmc/core/block.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)
drivers/mmc/core/block.c +19 -0
@@ -65,6 +65,7 @@ MODULE_ALIAS("mmc:block");#define MMC_BLK_TIMEOUT_MS (10 * 60 * 1000)/* 10 minute timeout */ #define MMC_SANITIZE_REQ_TIMEOUT 240000#define MMC_EXTRACT_INDEX_FROM_ARG (x) ((x & 0x00FF0000) >> 16) +#define MMC_EXTRACT_VALUE_FROM_ARG (x) ((x & 0x0000FF00) >> 8)#define mmc_req_rel_wr (req) ((req->cmd_flags & REQ_FUA) && \ (rq_data_dir(req) == WRITE))
@@ -539,6 +540,24 @@ static int __mmc_blk_ioctl_cmd(struct mm }/* +* Make sure the cache of the PARTITION_CONFIG register and +* PARTITION_ACCESS bits is updated in case the ioctl ext_csd write +* changed it successfully. +*/ +if ((MMC_EXTRACT_INDEX_FROM_ARG (cmd.arg) ==EXT_CSD_PART_CONFIG ) && + (cmd.opcode ==MMC_SWITCH )) { +struct mmc_blk_data *main_md =dev_get_drvdata (&card->dev); +u8 value =MMC_EXTRACT_VALUE_FROM_ARG (cmd.arg); + +/* +* Update cache so the next mmc_blk_part_switch call operates +* on up-to-date data. +*/ + card->ext_csd.part_config = value; + main_md->part_curr = value &EXT_CSD_PART_CONFIG_ACC_MASK ; + } + +/* * According to the SD specs, some commands require a delay after * issuing the command. */
Greg Kroah-Hartman
maintainerPATCH 19/101 mmc: dw_mmc: Fix the DTO/CTO timeout overflow calculation for 32-bit systems
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Evgeniy Didin <[email protected]>
commit c7151602255a36ba07c84fe2baeef846fdb988b8 upstream.
The commit 9d9491a7da2a ("mmc: dw_mmc: Fix the DTO timeout calculation") and commit 4c2357f57dd5 ("mmc: dw_mmc: Fix the CTO timeout calculation") made changes, which cause multiply overflow for 32-bit systems. The broken timeout calculations leads to unexpected ETIMEDOUT errors and causes stacktrace splat (such as below) during normal data exchange with SD-card.
| Running : 4M-check-reassembly-tcp-cmykw2-rotatew2.out -v0 -w1 | - Info: Finished target initialization. | mmcblk0: error -110 transferring data, sector 320544, nr 2048, cmd | response 0x900, card status 0x0
DIV_ROUND_UP_ULL helps to escape usage of __udivdi3() from libgcc and so code gets compiled on all 32-bit platforms as opposed to usage of DIV_ROUND_UP when we may only compile stuff on a very few arches.
Lets cast this multiply to u64 type to prevent the overflow.
--- drivers/mmc/host/dw_mmc.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
drivers/mmc/host/dw_mmc.c +6 -3
@@ -413,7 +413,9 @@ static inline void dw_mci_set_cto(struct cto_div = (mci_readl (host,CLKDIV ) &0xff ) *2 ;if (cto_div ==0 ) cto_div =1 ; - cto_ms =DIV_ROUND_UP (MSEC_PER_SEC * cto_clks * cto_div, host->bus_hz); + + cto_ms =DIV_ROUND_UP_ULL ((u64 )MSEC_PER_SEC * cto_clks * cto_div, + host->bus_hz);/* add a bit spare time */ cto_ms +=10 ;
@@ -1947,8 +1949,9 @@ static void dw_mci_set_drto(struct dw_mc drto_div = (mci_readl (host,CLKDIV ) &0xff ) *2 ;if (drto_div ==0 ) drto_div =1 ; - drto_ms =DIV_ROUND_UP (MSEC_PER_SEC * drto_clks * drto_div, - host->bus_hz); + + drto_ms =DIV_ROUND_UP_ULL ((u64 )MSEC_PER_SEC * drto_clks * drto_div, + host->bus_hz);/* add a bit spare time */ drto_ms +=10 ;
Greg Kroah-Hartman
maintainerPATCH 20/101 mmc: dw_mmc: exynos: fix the suspend/resume issue for exynos5433
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jaehoon Chung <[email protected]>
commit e22842dd64bf86753d3f2b6ea474d73fc1e6ca24 upstream.
Before enabling the clock, dwmmc exynos driver is trying to access the register. Then the kernel panic can be occurred.
--- drivers/mmc/host/dw_mmc-exynos.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
drivers/mmc/host/dw_mmc-exynos.c +7 -1
@@ -165,9 +165,15 @@ static void dw_mci_exynos_set_clksel_timstatic int dw_mci_exynos_runtime_resume (struct device *dev) {struct dw_mci *host =dev_get_drvdata (dev); +int ret; + + ret =dw_mci_runtime_resume (dev); +if (ret) +return ret;dw_mci_exynos_config_smu (host); -return dw_mci_runtime_resume (dev); + +return ret; }/**
Greg Kroah-Hartman
maintainerPATCH 21/101 mmc: dw_mmc: fix falling from idmac to PIO mode when dw_mci_reset occurs
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Evgeniy Didin <[email protected]>
commit 47b7de2f6c18f75d1f2716efe752cba43f32a626 upstream.
It was found that in IDMAC mode after soft-reset driver switches to PIO mode.
That's what happens in case of DTO timeout overflow calculation failure:
1. soft-reset is called
2. driver restarts dma
3. descriptors states are checked, one of descriptor is owned by the IDMAC.
4. driver can't use DMA and then switches to PIO mode.
Failure was already fixed in:
https://www.spinics.net/lists/linux-mmc/msg48125.html.
Behaviour while soft-reset is not something we except or even want to happen. So we switch from dw_mci_idmac_reset to dw_mci_idmac_init, so descriptors are cleaned before starting dma.
And while at it explicitly zero des0 which otherwise might contain garbage as being allocated by dmam_alloc_coherent().
--- drivers/mmc/host/dw_mmc.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
drivers/mmc/host/dw_mmc.c +4 -2
@@ -564,6 +564,7 @@ static int dw_mci_idmac_init(struct dw_m (sizeof (struct idmac_desc_64addr ) * (i + 1))) >> 32;/* Initialize reserved and buffer size fields to "0" */ + p->des0 =0 ; p->des1 =0 ; p->des2 =0 ; p->des3 =0 ;
@@ -586,6 +587,7 @@ static int dw_mci_idmac_init(struct dw_m i++, p++) { p->des3 =cpu_to_le32 (host->sg_dma + (sizeof (struct idmac_desc ) * (i +1 ))); + p->des0 =0 ; p->des1 =0 ; }
@@ -1801,8 +1803,8 @@ static bool dw_mci_reset(struct dw_mci * }if (host->use_dma ==TRANS_MODE_IDMAC ) -/* It is also recommended that we reset and reprogram idmac */ -dw_mci_idmac_reset (host); +/* It is also required that we reinit idmac */ +dw_mci_idmac_init (host); ret = true;
Greg Kroah-Hartman
maintainerPATCH 22/101 PCI: Add function 1 DMA alias quirk for Highpoint RocketRAID 644L
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit 1903be8222b7c278ca897c129ce477c1dd6403a8 upstream.
The Highpoint RocketRAID 644L uses a Marvel 88SE9235 controller, as with other Marvel controllers this needs a function 1 DMA alias quirk.
Note the RocketRAID 642L uses the same Marvel 88SE9235 controller and already is listed with a function 1 DMA alias quirk.
--- drivers/pci/quirks.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/pci/quirks.c +2 -0
@@ -3899,6 +3899,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_M quirk_dma_func1_alias);DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_TTI ,0x0642 , quirk_dma_func1_alias); +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_TTI ,0x0645 , + quirk_dma_func1_alias);/* https://bugs.gentoo.org/show_bug.cgi?id=497630 */ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON, PCI_DEVICE_ID_JMICRON_JMB388_ESD,
Greg Kroah-Hartman
maintainerPATCH 23/101 ahci: Add PCI-id for the Highpoint Rocketraid 644L card
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit 28b2182dad43f6f8fcbd167539a26714fd12bd64 upstream.
Like the Highpoint Rocketraid 642L and cards using a Marvel 88SE9235 controller in general, this RAID card also supports AHCI mode and short of a custom driver, this is the only way to make it work under Linux.
Note that even though the card is called to 644L, it has a product-id of 0x0645.
--- drivers/ata/ahci.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/ata/ahci.c +3 -1
@@ -541,7 +541,9 @@ static const struct pci_device_id ahci_p .driver_data = board_ahci_yes_fbs }, { PCI_DEVICE(PCI_VENDOR_ID_MARVELL_EXT, 0x9230), .driver_data = board_ahci_yes_fbs }, - { PCI_DEVICE(PCI_VENDOR_ID_TTI, 0x0642), + { PCI_DEVICE(PCI_VENDOR_ID_TTI, 0x0642), /* highpoint rocketraid 642L */ + .driver_data = board_ahci_yes_fbs }, + { PCI_DEVICE(PCI_VENDOR_ID_TTI, 0x0645), /* highpoint rocketraid 644L */ .driver_data = board_ahci_yes_fbs },/* Promise */
Greg Kroah-Hartman
maintainerPATCH 24/101 lockdep: fix fs_reclaim warning
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Tetsuo Handa <[email protected]>
commit 2e517d681632326ed98399cb4dd99519efe3e32c upstream.
Dave Jones reported fs_reclaim lockdep warnings.
============================================
WARNING: possible recursive locking detected 4.15.0-rc9-backup-debug+ #1 Not tainted
--------------------------------------------
sshd/24800 is trying to acquire lock:
(fs_reclaim){+.+.}, at: [<0000000084f438c2>] fs_reclaim_acquire.part.102+0x5/0x30
but task is already holding lock:
(fs_reclaim){+.+.}, at: [<0000000084f438c2>] fs_reclaim_acquire.part.102+0x5/0x30
other info that might help us debug this:
Possible unsafe locking scenario:
CPU0
----
lock(fs_reclaim);
lock(fs_reclaim);*** DEADLOCK ***
May be due to missing lock nesting notation
2 locks held by sshd/24800:
#0: (sk_lock-AF_INET6){+.+.}, at: [<000000001a069652>] tcp_sendmsg+0x19/0x40 #1: (fs_reclaim){+.+.}, at: [<0000000084f438c2>] fs_reclaim_acquire.part.102+0x5/0x30
stack backtrace: CPU: 3 PID: 24800 Comm: sshd Not tainted 4.15.0-rc9-backup-debug+ #1 Call Trace: dump_stack+0xbc/0x13f __lock_acquire+0xa09/0x2040 lock_acquire+0x12e/0x350 fs_reclaim_acquire.part.102+0x29/0x30 kmem_cache_alloc+0x3d/0x2c0 alloc_extent_state+0xa7/0x410 __clear_extent_bit+0x3ea/0x570 try_release_extent_mapping+0x21a/0x260 __btrfs_releasepage+0xb0/0x1c0 btrfs_releasepage+0x161/0x170 try_to_release_page+0x162/0x1c0 shrink_page_list+0x1d5a/0x2fb0 shrink_inactive_list+0x451/0x940 shrink_node_memcg.constprop.88+0x4c9/0x5e0 shrink_node+0x12d/0x260 try_to_free_pages+0x418/0xaf0 __alloc_pages_slowpath+0x976/0x1790 __alloc_pages_nodemask+0x52c/0x5c0 new_slab+0x374/0x3f0 ___slab_alloc.constprop.81+0x47e/0x5a0 __slab_alloc.constprop.80+0x32/0x60 __kmalloc_track_caller+0x267/0x310 __kmalloc_reserve.isra.40+0x29/0x80 __alloc_skb+0xee/0x390 sk_stream_alloc_skb+0xb8/0x340 tcp_sendmsg_locked+0x8e6/0x1d30 tcp_sendmsg+0x27/0x40 inet_sendmsg+0xd0/0x310 sock_write_iter+0x17a/0x240 __vfs_write+0x2ab/0x380 vfs_write+0xfb/0x260 SyS_write+0xb6/0x140 do_syscall_64+0x1e5/0xc05 entry_SYSCALL64_slow_path+0x25/0x25
This warning is caused by commit d92a8cfcb37e ("locking/lockdep:
Rework FS_RECLAIM annotation") which replaced the use of lockdep_{set,clear}_current_reclaim_state() in __perform_reclaim() and lockdep_trace_alloc() in slab_pre_alloc_hook() with fs_reclaim_acquire()/ fs_reclaim_release().
Since __kmalloc_reserve() from __alloc_skb() adds __GFP_NOMEMALLOC | __GFP_NOWARN to gfp_mask, and all reclaim path simply propagates __GFP_NOMEMALLOC, fs_reclaim_acquire() in slab_pre_alloc_hook() is trying to grab the 'fake' lock again when __perform_reclaim() already grabbed the 'fake' lock.
The
/* this guy won't enter reclaim */
if ((current->flags & PF_MEMALLOC) && !(gfp_mask & __GFP_NOMEMALLOC))
return false;test which causes slab_pre_alloc_hook() to try to grab the 'fake' lock was added by commit cf40bd16fdad ("lockdep: annotate reclaim context (__GFP_NOFS)"). But that test is outdated because PF_MEMALLOC thread won't enter reclaim regardless of __GFP_NOMEMALLOC after commit 341ce06f69ab ("page allocator: calculate the alloc_flags for allocation only once") added the PF_MEMALLOC safeguard (
/* Avoid recursion of direct reclaim */
if (p->flags & PF_MEMALLOC)
goto nopage;
in __alloc_pages_slowpath()).
Thus, let's fix outdated test by removing __GFP_NOMEMALLOC test and allow __need_fs_reclaim() to return false.
--- mm/page_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
mm/page_alloc.c +1 -1
@@ -3532,7 +3532,7 @@ static bool __need_fs_reclaim(gfp_t gfp_return false;/* this guy won't enter reclaim */ -if ((current->flags &PF_MEMALLOC ) && !(gfp_mask & __GFP_NOMEMALLOC)) +if (current->flags &PF_MEMALLOC )return false;/* We're only interested __GFP_FS allocations for now */
Greg Kroah-Hartman
maintainerPATCH 25/101 clk: bcm2835: Fix ana->maskX definitions
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Boris Brezillon <[email protected]>
commit 49012d1bf5f78782d398adb984a080a88ba42965 upstream.
ana->maskX values are already '~'-ed in bcm2835_pll_set_rate(). Remove the '~' in the definition to fix ANA setup.
Note that this commit fixes a long standing bug preventing one from using an HDMI display if it's plugged after the FW has booted Linux. This is because PLLH is used by the HDMI encoder to generate the pixel clock.
--- drivers/clk/bcm/clk-bcm2835.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/clk/bcm/clk-bcm2835.c +4 -4
@@ -479,17 +479,17 @@ struct bcm2835_pll_ana_bits {static const struct bcm2835_pll_ana_bits bcm2835_ana_default = { .mask0 =0 , .set0 =0 , - .mask1 = (u32 )~(A2W_PLL_KI_MASK |A2W_PLL_KP_MASK ), + .mask1 =A2W_PLL_KI_MASK |A2W_PLL_KP_MASK , .set1 = (2 <<A2W_PLL_KI_SHIFT ) | (8 <<A2W_PLL_KP_SHIFT ), - .mask3 = (u32 )~A2W_PLL_KA_MASK , + .mask3 =A2W_PLL_KA_MASK , .set3 = (2 <<A2W_PLL_KA_SHIFT ), .fb_prediv_mask =BIT (14 ), };static const struct bcm2835_pll_ana_bits bcm2835_ana_pllh = { - .mask0 = (u32 )~(A2W_PLLH_KA_MASK |A2W_PLLH_KI_LOW_MASK ), + .mask0 =A2W_PLLH_KA_MASK |A2W_PLLH_KI_LOW_MASK , .set0 = (2 <<A2W_PLLH_KA_SHIFT ) | (2 <<A2W_PLLH_KI_LOW_SHIFT ), - .mask1 = (u32 )~(A2W_PLLH_KI_HIGH_MASK |A2W_PLLH_KP_MASK ), + .mask1 =A2W_PLLH_KI_HIGH_MASK |A2W_PLLH_KP_MASK , .set1 = (6 <<A2W_PLLH_KP_SHIFT ), .mask3 =0 , .set3 =0 ,
Greg Kroah-Hartman
maintainerPATCH 26/101 clk: bcm2835: Protect sections updating shared registers
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Boris Brezillon <[email protected]>
commit 7997f3b2df751aab0b8e60149b226a32966c41ac upstream.
CM_PLLx and A2W_XOSC_CTRL registers are accessed by different clock handlers and must be accessed with ->regs_lock held. Update the sections where this protection is missing.
--- drivers/clk/bcm/clk-bcm2835.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/clk/bcm/clk-bcm2835.c +4 -0
@@ -653,8 +653,10 @@ static int bcm2835_pll_on(struct clk_hw ~A2W_PLL_CTRL_PWRDN);/* Take the PLL out of reset. */ +spin_lock (&cprman->regs_lock);cprman_write (cprman, data->cm_ctrl_reg,cprman_read (cprman, data->cm_ctrl_reg) & ~CM_PLL_ANARST ); +spin_unlock (&cprman->regs_lock);/* Wait for the PLL to lock. */ timeout =ktime_add_ns (ktime_get (),LOCK_TIMEOUT_NS );
@@ -731,9 +733,11 @@ static int bcm2835_pll_set_rate(struct c }/* Unmask the reference clock from the oscillator. */ +spin_lock (&cprman->regs_lock);cprman_write (cprman,A2W_XOSC_CTRL ,cprman_read (cprman,A2W_XOSC_CTRL ) | data->reference_enable_mask); +spin_unlock (&cprman->regs_lock);if (do_ana_setup_first)bcm2835_pll_write_ana (cprman, data->ana_reg_base, ana);
Greg Kroah-Hartman
maintainerPATCH 27/101 clk: sunxi-ng: a31: Fix CLK_OUT_* clock ops
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 5682e268350f9eccdbb04006605c1b7068a7b323 upstream.
When support for the A31/A31s CCU was first added, the clock ops for the CLK_OUT_* clocks was set to the wrong type. The clocks are MP-type, but the ops was set for div (M) clocks. This went unnoticed until now. This was because while they are different clocks, their data structures aligned in a way that ccu_div_ops would access the second ccu_div_internal and ccu_mux_internal structures, which were valid, if not incorrect.
Furthermore, the use of these CLK_OUT_* was for feeding a precise 32.768 kHz clock signal to the WiFi chip. This was achievable by using the parent with the same clock rate and no divider. So the incorrect divider setting did not affect this usage.
Commit 946797aa3f08 ("clk: sunxi-ng: Support fixed post-dividers on MP style clocks") added a new field to the ccu_mp structure, which broke the aforementioned alignment. Now the system crashes as div_ops tries to look up a nonexistent table.
--- drivers/clk/sunxi-ng/ccu-sun6i-a31.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/clk/sunxi-ng/ccu-sun6i-a31.c +3 -3
@@ -750,7 +750,7 @@ static struct ccu_mp out_a_clk = { .features = CCU_FEATURE_FIXED_PREDIV, .hw.init =CLK_HW_INIT_PARENTS ("out-a" , clk_out_parents, - &ccu_div_ops, + &ccu_mp_ops, 0), }, };
@@ -771,7 +771,7 @@ static struct ccu_mp out_b_clk = { .features = CCU_FEATURE_FIXED_PREDIV, .hw.init =CLK_HW_INIT_PARENTS ("out-b" , clk_out_parents, - &ccu_div_ops, + &ccu_mp_ops, 0), }, };
@@ -792,7 +792,7 @@ static struct ccu_mp out_c_clk = { .features = CCU_FEATURE_FIXED_PREDIV, .hw.init =CLK_HW_INIT_PARENTS ("out-c" , clk_out_parents, - &ccu_div_ops, + &ccu_mp_ops, 0), }, };
Greg Kroah-Hartman
maintainerPATCH 28/101 RDMA/mlx5: Fix crash while accessing garbage pointer and freed memory
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Leon Romanovsky <[email protected]>
commit f3f134f5260ae9ee1f5a4d0a8cc625c6c77655b4 upstream.
The failure in rereg_mr flow caused to set garbage value (error value) into mr->umem pointer. This pointer is accessed at the release stage and it causes to the following crash.
There is not enough to simply change umem to point to NULL, because the MR struct is needed to be accessed during MR deregistration phase, so delay kfree too.
[ 6.237617] BUG: unable to handle kernel NULL pointer dereference a 0000000000000228 [ 6.238756] IP: ib_dereg_mr+0xd/0x30 [ 6.239264] PGD 80000000167eb067 P4D 80000000167eb067 PUD 167f9067 PMD 0 [ 6.240320] Oops: 0000 [#1] SMP PTI [ 6.240782] CPU: 0 PID: 367 Comm: dereg Not tainted 4.16.0-rc1-00029-gc198fafe0453 #183 [ 6.242120] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.7.5-0-ge51488c-20140602_164612-nilsson.home.kraxel.org 04/01/2014 [ 6.244504] RIP: 0010:ib_dereg_mr+0xd/0x30 [ 6.245253] RSP: 0018:ffffaf5d001d7d68 EFLAGS: 00010246 [ 6.246100] RAX: 0000000000000000 RBX: ffff95d4172daf00 RCX: 0000000000000000 [ 6.247414] RDX: 00000000ffffffff RSI: 0000000000000001 RDI: ffff95d41a317600 [ 6.248591] RBP: 0000000000000001 R08: 0000000000000000 R09: 0000000000000000 [ 6.249810] R10: ffff95d417033c10 R11: 0000000000000000 R12: ffff95d4172c3a80 [ 6.251121] R13: ffff95d4172c3720 R14: ffff95d4172c3a98 R15: 00000000ffffffff [ 6.252437] FS: 0000000000000000(0000) GS:ffff95d41fc00000(0000) knlGS:0000000000000000 [ 6.253887] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 6.254814] CR2: 0000000000000228 CR3: 00000000172b4000 CR4: 00000000000006b0 [ 6.255943] Call Trace: [ 6.256368] remove_commit_idr_uobject+0x1b/0x80 [ 6.257118] uverbs_cleanup_ucontext+0xe4/0x190 [ 6.257855] ib_uverbs_cleanup_ucontext.constprop.14+0x19/0x40 [ 6.258857] ib_uverbs_close+0x2a/0x100 [ 6.259494] __fput+0xca/0x1c0 [ 6.259938] task_work_run+0x84/0xa0 [ 6.260519] do_exit+0x312/0xb40 [ 6.261023] ? __do_page_fault+0x24d/0x490 [ 6.261707] do_group_exit+0x3a/0xa0 [ 6.262267] SyS_exit_group+0x10/0x10 [ 6.262802] do_syscall_64+0x75/0x180 [ 6.263391] entry_SYSCALL_64_after_hwframe+0x21/0x86 [ 6.264253] RIP: 0033:0x7f1b39c49488 [ 6.264827] RSP: 002b:00007ffe2de05b68 EFLAGS: 00000246 ORIG_RAX: 00000000000000e7 [ 6.266049] RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00007f1b39c49488 [ 6.267187] RDX: 0000000000000000 RSI: 000000000000003c RDI: 0000000000000000 [ 6.268377] RBP: 00007f1b39f258e0 R08: 00000000000000e7 R09: ffffffffffffff98 [ 6.269640] R10: 00007f1b3a147260 R11: 0000000000000246 R12: 00007f1b39f258e0 [ 6.270783] R13: 00007f1b39f2ac20 R14: 0000000000000000 R15: 0000000000000000 [ 6.271943] Code: 74 07 31 d2 e9 25 d8 6c 00 b8 da ff ff ff c3 0f 1f 44 00 00 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 44 00 00 48 8b 07 53 48 8b 5f 08 <48> 8b 80 28 02 00 00 e8 f7 d7 6c 00 85 c0 75 04 3e ff 4b 18 5b [ 6.274927] RIP: ib_dereg_mr+0xd/0x30 RSP: ffffaf5d001d7d68 [ 6.275760] CR2: 0000000000000228 [ 6.276200] ---[ end trace a35641f1c474bd20 ]---
--- drivers/infiniband/hw/mlx5/mr.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-)
drivers/infiniband/hw/mlx5/mr.c +8 -4
@@ -838,7 +838,8 @@ static int mr_umem_get(struct ib_pd *pd, *umem =ib_umem_get (pd->uobject->context, start, length, access_flags,0 ); err =PTR_ERR_OR_ZERO (*umem); -if (err <0 ) { +if (err) { + *umem =NULL ;mlx5_ib_err (dev,"umem get failed (%d)\n" , err);return err; }
@@ -1415,6 +1416,7 @@ int mlx5_ib_rereg_user_mr(struct ib_mr *if (err) {mlx5_ib_warn (dev,"Failed to rereg UMR\n" );ib_umem_release (mr->umem); + mr->umem =NULL ;clean_mr (dev, mr);return err; }
@@ -1498,14 +1500,11 @@ static int clean_mr(struct mlx5_ib_dev *u32 key = mr->mmkey.key; err =destroy_mkey (dev, mr); -kfree (mr);if (err) {mlx5_ib_warn (dev,"failed to destroy mkey 0x%x (%d)\n" , key, err);return err; } - }else { -mlx5_mr_cache_free (dev, mr); }return 0 ;
@@ -1548,6 +1547,11 @@ static int dereg_mr(struct mlx5_ib_dev *atomic_sub (npages, &dev->mdev->priv.reg_pages); } +if (!mr->allocated_from_cache) +kfree (mr); +else +mlx5_mr_cache_free (dev, mr); +return 0 ; }
Greg Kroah-Hartman
maintainerPATCH 29/101 Drivers: hv: vmbus: Fix ring buffer signaling
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Michael Kelley <[email protected]>
commit 655296c8bbeffcf020558c4455305d597a73bde1 upstream.
Fix bugs in signaling the Hyper-V host when freeing space in the host->guest ring buffer:
1. The interrupt_mask must not be used to determine whether to signal on the host->guest ring buffer
2. The ring buffer write_index must be read (via hv_get_bytes_to_write) *after* pending_send_sz is read in order to avoid a race condition
3. Comparisons with pending_send_sz must treat the "equals" case as not-enough-space
4. Don't signal if the pending_send_sz feature is not present. Older versions of Hyper-V that don't implement this feature will poll.
--- drivers/hv/ring_buffer.c | 54 +++++++++++++++++++++++++++++++---------------- 1 file changed, 36 insertions(+), 18 deletions(-)
drivers/hv/ring_buffer.c +36 -18
@@ -394,13 +394,24 @@ __hv_pkt_iter_next(struct vmbus_channel }EXPORT_SYMBOL_GPL (__hv_pkt_iter_next); +/* How many bytes were read in this iterator cycle */ +static u32 hv_pkt_iter_bytes_read (const struct hv_ring_buffer_info *rbi, +u32 start_read_index) +{ +if (rbi->priv_read_index >= start_read_index) +return rbi->priv_read_index - start_read_index; +else +return rbi->ring_datasize - start_read_index + + rbi->priv_read_index; +} +/* * Update host ring buffer after iterating over packets. */ void hv_pkt_iter_close (struct vmbus_channel *channel) {struct hv_ring_buffer_info *rbi = &channel->inbound; -u32 orig_write_sz =hv_get_bytes_to_write (rbi); +u32 curr_write_sz, pending_sz, bytes_read, start_read_index;/* * Make sure all reads are done before we update the read index since
@@ -408,8 +419,12 @@ void hv_pkt_iter_close(struct vmbus_chan* is updated. */ virt_rmb (); + start_read_index = rbi->ring_buffer->read_index; rbi->ring_buffer->read_index = rbi->priv_read_index; +if (!rbi->ring_buffer->feature_bits.feat_pending_send_sz) +return ; +/* * Issue a full memory barrier before making the signaling decision. * Here is the reason for having this barrier:
@@ -423,26 +438,29 @@ void hv_pkt_iter_close(struct vmbus_chan*/ virt_mb (); -/* If host has disabled notifications then skip */ -if (rbi->ring_buffer->interrupt_mask) + pending_sz =READ_ONCE (rbi->ring_buffer->pending_send_sz); +if (!pending_sz)return ; -if (rbi->ring_buffer->feature_bits.feat_pending_send_sz) { -u32 pending_sz =READ_ONCE (rbi->ring_buffer->pending_send_sz); +/* +* Ensure the read of write_index in hv_get_bytes_to_write() +* happens after the read of pending_send_sz. +*/ +virt_rmb (); + curr_write_sz =hv_get_bytes_to_write (rbi); + bytes_read =hv_pkt_iter_bytes_read (rbi, start_read_index); + +/* +* If there was space before we began iteration, +* then host was not blocked. +*/ -/* -* If there was space before we began iteration, -* then host was not blocked. Also handles case where -* pending_sz is zero then host has nothing pending -* and does not need to be signaled. -*/ -if (orig_write_sz > pending_sz) -return ; - -/* If pending write will not fit, don't give false hope. */ -if (hv_get_bytes_to_write (rbi) < pending_sz) -return ; - } +if (curr_write_sz - bytes_read > pending_sz) +return ; + +/* If pending write will not fit, don't give false hope. */ +if (curr_write_sz <= pending_sz) +return ;vmbus_setevent (channel); }
Greg Kroah-Hartman
maintainerPATCH 30/101 pinctrl: samsung: Validate alias coming from DT
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Krzysztof Kozlowski <[email protected]>
commit 93b0beae721b3344923b4b8317e9d83b542f4ca6 upstream.
Driver uses alias from Device Tree as an index of pin controller data array. In case of a wrong DTB or an out-of-tree DTB, the alias could be outside of this data array leading to out-of-bounds access.
Depending on binary and memory layout, this could be handled properly (showing error like "samsung-pinctrl 3860000.pinctrl: driver data not available") or could lead to exceptions.
--- drivers/pinctrl/samsung/pinctrl-exynos-arm.c | 56 +++++++++++++++++++--- drivers/pinctrl/samsung/pinctrl-exynos-arm64.c | 14 ++++- drivers/pinctrl/samsung/pinctrl-s3c24xx.c | 28 +++++++++-- drivers/pinctrl/samsung/pinctrl-s3c64xx.c | 7 ++ drivers/pinctrl/samsung/pinctrl-samsung.c | 61 +++++++++++++++---------- drivers/pinctrl/samsung/pinctrl-samsung.h | 40 ++++++++++------ 6 files changed, 154 insertions(+), 52 deletions(-)
drivers/pinctrl/samsung/pinctrl-exynos-arm.c +48 -8
@@ -129,7 +129,7 @@ static const struct samsung_pin_bank_dat EXYNOS_PIN_BANK_EINTW(8, 0xc60, "gph3", 0x0c), }; -const struct samsung_pin_ctrl s5pv210_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl s5pv210_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = s5pv210_pin_bank,
@@ -142,6 +142,11 @@ const struct samsung_pin_ctrl s5pv210_pi }, }; +const struct samsung_pinctrl_of_match_data s5pv210_of_data__initconst = { + .ctrl = s5pv210_pin_ctrl, + .num_ctrl =ARRAY_SIZE (s5pv210_pin_ctrl), +}; +/* Pad retention control code for accessing PMU regmap */ static atomic_t exynos_shared_retention_refcnt;
@@ -204,7 +209,7 @@ static const struct samsung_retention_da* Samsung pinctrl driver data for Exynos3250 SoC. Exynos3250 SoC includes * two gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos3250_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos3250_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos3250_pin_banks0,
@@ -225,6 +230,11 @@ const struct samsung_pin_ctrl exynos3250 }, }; +const struct samsung_pinctrl_of_match_data exynos3250_of_data__initconst = { + .ctrl = exynos3250_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos3250_pin_ctrl), +}; +/* pin banks of exynos4210 pin-controller 0 */ static const struct samsung_pin_bank_data exynos4210_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTG (8 ,0x000 ,"gpa0" ,0x00 ),
@@ -308,7 +318,7 @@ static const struct samsung_retention_da* Samsung pinctrl driver data for Exynos4210 SoC. Exynos4210 SoC includes * three gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos4210_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos4210_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos4210_pin_banks0,
@@ -334,6 +344,11 @@ const struct samsung_pin_ctrl exynos4210 }, }; +const struct samsung_pinctrl_of_match_data exynos4210_of_data__initconst = { + .ctrl = exynos4210_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos4210_pin_ctrl), +}; +/* pin banks of exynos4x12 pin-controller 0 */ static const struct samsung_pin_bank_data exynos4x12_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTG (8 ,0x000 ,"gpa0" ,0x00 ),
@@ -396,7 +411,7 @@ static const struct samsung_pin_bank_dat* Samsung pinctrl driver data for Exynos4x12 SoC. Exynos4x12 SoC includes * four gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos4x12_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos4x12_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos4x12_pin_banks0,
@@ -432,6 +447,11 @@ const struct samsung_pin_ctrl exynos4x12 }, }; +const struct samsung_pinctrl_of_match_data exynos4x12_of_data__initconst = { + .ctrl = exynos4x12_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos4x12_pin_ctrl), +}; +/* pin banks of exynos5250 pin-controller 0 */ static const struct samsung_pin_bank_data exynos5250_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTG (8 ,0x000 ,"gpa0" ,0x00 ),
@@ -492,7 +512,7 @@ static const struct samsung_pin_bank_dat* Samsung pinctrl driver data for Exynos5250 SoC. Exynos5250 SoC includes * four gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos5250_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos5250_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos5250_pin_banks0,
@@ -528,6 +548,11 @@ const struct samsung_pin_ctrl exynos5250 }, }; +const struct samsung_pinctrl_of_match_data exynos5250_of_data__initconst = { + .ctrl = exynos5250_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos5250_pin_ctrl), +}; +/* pin banks of exynos5260 pin-controller 0 */ static const struct samsung_pin_bank_data exynos5260_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTG (4 ,0x000 ,"gpa0" ,0x00 ),
@@ -572,7 +597,7 @@ static const struct samsung_pin_bank_dat* Samsung pinctrl driver data for Exynos5260 SoC. Exynos5260 SoC includes * three gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos5260_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos5260_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos5260_pin_banks0,
@@ -592,6 +617,11 @@ const struct samsung_pin_ctrl exynos5260 }, }; +const struct samsung_pinctrl_of_match_data exynos5260_of_data__initconst = { + .ctrl = exynos5260_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos5260_pin_ctrl), +}; +/* pin banks of exynos5410 pin-controller 0 */ static const struct samsung_pin_bank_data exynos5410_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTG (8 ,0x000 ,"gpa0" ,0x00 ),
@@ -662,7 +692,7 @@ static const struct samsung_pin_bank_dat* Samsung pinctrl driver data for Exynos5410 SoC. Exynos5410 SoC includes * four gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos5410_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos5410_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos5410_pin_banks0,
@@ -695,6 +725,11 @@ const struct samsung_pin_ctrl exynos5410 }, }; +const struct samsung_pinctrl_of_match_data exynos5410_of_data__initconst = { + .ctrl = exynos5410_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos5410_pin_ctrl), +}; +/* pin banks of exynos5420 pin-controller 0 */ static const struct samsung_pin_bank_data exynos5420_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTG (8 ,0x000 ,"gpy7" ,0x00 ),
@@ -779,7 +814,7 @@ static const struct samsung_retention_da* Samsung pinctrl driver data for Exynos5420 SoC. Exynos5420 SoC includes * four gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos5420_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos5420_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos5420_pin_banks0,
@@ -813,3 +848,8 @@ const struct samsung_pin_ctrl exynos5420 .retention_data = &exynos4_audio_retention_data, }, }; + +const struct samsung_pinctrl_of_match_data exynos5420_of_data__initconst = { + .ctrl = exynos5420_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos5420_pin_ctrl), +};
drivers/pinctrl/samsung/pinctrl-exynos-arm64.c +12 -2
@@ -180,7 +180,7 @@ static const struct samsung_retention_da* Samsung pinctrl driver data for Exynos5433 SoC. Exynos5433 SoC includes * ten gpio/pin-mux/pinconfig controllers. */ -const struct samsung_pin_ctrl exynos5433_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos5433_pin_ctrl[]__initconst = { {/* pin-controller instance 0 data */ .pin_banks = exynos5433_pin_banks0,
@@ -265,6 +265,11 @@ const struct samsung_pin_ctrl exynos5433 }, }; +const struct samsung_pinctrl_of_match_data exynos5433_of_data__initconst = { + .ctrl = exynos5433_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos5433_pin_ctrl), +}; +/* pin banks of exynos7 pin-controller - ALIVE */ static const struct samsung_pin_bank_data exynos7_pin_banks0[]__initconst = {EXYNOS_PIN_BANK_EINTW (8 ,0x000 ,"gpa0" ,0x00 ),
@@ -344,7 +349,7 @@ static const struct samsung_pin_bank_dat EXYNOS_PIN_BANK_EINTG(4, 0x020, "gpz1", 0x04), }; -const struct samsung_pin_ctrl exynos7_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl exynos7_pin_ctrl[]__initconst = { {/* pin-controller instance 0 Alive data */ .pin_banks = exynos7_pin_banks0,
@@ -397,3 +402,8 @@ const struct samsung_pin_ctrl exynos7_pi .eint_gpio_init = exynos_eint_gpio_init, }, }; + +const struct samsung_pinctrl_of_match_data exynos7_of_data__initconst = { + .ctrl = exynos7_pin_ctrl, + .num_ctrl =ARRAY_SIZE (exynos7_pin_ctrl), +};
drivers/pinctrl/samsung/pinctrl-s3c24xx.c +24 -4
@@ -570,7 +570,7 @@ static const struct samsung_pin_bank_dat PIN_BANK_2BIT(13, 0x080, "gpj"), }; -const struct samsung_pin_ctrl s3c2412_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl s3c2412_pin_ctrl[]__initconst = { { .pin_banks = s3c2412_pin_banks, .nr_banks =ARRAY_SIZE (s3c2412_pin_banks),
@@ -578,6 +578,11 @@ const struct samsung_pin_ctrl s3c2412_pi }, }; +const struct samsung_pinctrl_of_match_data s3c2412_of_data__initconst = { + .ctrl = s3c2412_pin_ctrl, + .num_ctrl =ARRAY_SIZE (s3c2412_pin_ctrl), +}; +static const struct samsung_pin_bank_data s3c2416_pin_banks[]__initconst = {PIN_BANK_A (27 ,0x000 ,"gpa" ),PIN_BANK_2BIT (11 ,0x010 ,"gpb" ),
@@ -592,7 +597,7 @@ static const struct samsung_pin_bank_dat PIN_BANK_2BIT(2, 0x100, "gpm"), }; -const struct samsung_pin_ctrl s3c2416_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl s3c2416_pin_ctrl[]__initconst = { { .pin_banks = s3c2416_pin_banks, .nr_banks =ARRAY_SIZE (s3c2416_pin_banks),
@@ -600,6 +605,11 @@ const struct samsung_pin_ctrl s3c2416_pi }, }; +const struct samsung_pinctrl_of_match_data s3c2416_of_data__initconst = { + .ctrl = s3c2416_pin_ctrl, + .num_ctrl =ARRAY_SIZE (s3c2416_pin_ctrl), +}; +static const struct samsung_pin_bank_data s3c2440_pin_banks[]__initconst = {PIN_BANK_A (25 ,0x000 ,"gpa" ),PIN_BANK_2BIT (11 ,0x010 ,"gpb" ),
@@ -612,7 +622,7 @@ static const struct samsung_pin_bank_dat PIN_BANK_2BIT(13, 0x0d0, "gpj"), }; -const struct samsung_pin_ctrl s3c2440_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl s3c2440_pin_ctrl[]__initconst = { { .pin_banks = s3c2440_pin_banks, .nr_banks =ARRAY_SIZE (s3c2440_pin_banks),
@@ -620,6 +630,11 @@ const struct samsung_pin_ctrl s3c2440_pi }, }; +const struct samsung_pinctrl_of_match_data s3c2440_of_data__initconst = { + .ctrl = s3c2440_pin_ctrl, + .num_ctrl =ARRAY_SIZE (s3c2440_pin_ctrl), +}; +static const struct samsung_pin_bank_data s3c2450_pin_banks[]__initconst = {PIN_BANK_A (28 ,0x000 ,"gpa" ),PIN_BANK_2BIT (11 ,0x010 ,"gpb" ),
@@ -635,10 +650,15 @@ static const struct samsung_pin_bank_dat PIN_BANK_2BIT(2, 0x100, "gpm"), }; -const struct samsung_pin_ctrl s3c2450_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl s3c2450_pin_ctrl[]__initconst = { { .pin_banks = s3c2450_pin_banks, .nr_banks =ARRAY_SIZE (s3c2450_pin_banks), .eint_wkup_init = s3c24xx_eint_init, }, }; + +const struct samsung_pinctrl_of_match_data s3c2450_of_data__initconst = { + .ctrl = s3c2450_pin_ctrl, + .num_ctrl =ARRAY_SIZE (s3c2450_pin_ctrl), +};
drivers/pinctrl/samsung/pinctrl-s3c64xx.c +6 -1
@@ -794,7 +794,7 @@ static const struct samsung_pin_bank_dat* Samsung pinctrl driver data for S3C64xx SoC. S3C64xx SoC includes * one gpio/pin-mux/pinconfig controller. */ -const struct samsung_pin_ctrl s3c64xx_pin_ctrl[]__initconst = { +static const struct samsung_pin_ctrl s3c64xx_pin_ctrl[]__initconst = { {/* pin-controller instance 1 data */ .pin_banks = s3c64xx_pin_banks0,
@@ -803,3 +803,8 @@ const struct samsung_pin_ctrl s3c64xx_pi .eint_wkup_init = s3c64xx_eint_eint0_init, }, }; + +const struct samsung_pinctrl_of_match_data s3c64xx_of_data__initconst = { + .ctrl = s3c64xx_pin_ctrl, + .num_ctrl =ARRAY_SIZE (s3c64xx_pin_ctrl), +};
drivers/pinctrl/samsung/pinctrl-samsung.c +39 -22
105 lines not on this page · raw message
drivers/pinctrl/samsung/pinctrl-samsung.h +25 -15
@@ -286,6 +286,16 @@ struct samsung_pinctrl_drv_data { };/** +* struct samsung_pinctrl_of_match_data: OF match device specific configuration data. +* @ctrl: array of pin controller data. +* @num_ctrl: size of array @ctrl. +*/ +struct samsung_pinctrl_of_match_data { +const struct samsung_pin_ctrl *ctrl; +unsigned int num_ctrl; +}; + +/** * struct samsung_pin_group: represent group of pins of a pinmux function. * @name: name of the pin group, used to lookup the group. * @pins: the pins included in this group.
@@ -313,20 +323,20 @@ struct samsung_pmx_func { };/* list of all exported SoC specific data */ -extern const struct samsung_pin_ctrl exynos3250_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos4210_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos4x12_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos5250_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos5260_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos5410_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos5420_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos5433_pin_ctrl[]; -extern const struct samsung_pin_ctrl exynos7_pin_ctrl[]; -extern const struct samsung_pin_ctrl s3c64xx_pin_ctrl[]; -extern const struct samsung_pin_ctrl s3c2412_pin_ctrl[]; -extern const struct samsung_pin_ctrl s3c2416_pin_ctrl[]; -extern const struct samsung_pin_ctrl s3c2440_pin_ctrl[]; -extern const struct samsung_pin_ctrl s3c2450_pin_ctrl[]; -extern const struct samsung_pin_ctrl s5pv210_pin_ctrl[]; +extern const struct samsung_pinctrl_of_match_data exynos3250_of_data; +extern const struct samsung_pinctrl_of_match_data exynos4210_of_data; +extern const struct samsung_pinctrl_of_match_data exynos4x12_of_data; +extern const struct samsung_pinctrl_of_match_data exynos5250_of_data; +extern const struct samsung_pinctrl_of_match_data exynos5260_of_data; +extern const struct samsung_pinctrl_of_match_data exynos5410_of_data; +extern const struct samsung_pinctrl_of_match_data exynos5420_of_data; +extern const struct samsung_pinctrl_of_match_data exynos5433_of_data; +extern const struct samsung_pinctrl_of_match_data exynos7_of_data; +extern const struct samsung_pinctrl_of_match_data s3c64xx_of_data; +extern const struct samsung_pinctrl_of_match_data s3c2412_of_data; +extern const struct samsung_pinctrl_of_match_data s3c2416_of_data; +extern const struct samsung_pinctrl_of_match_data s3c2440_of_data; +extern const struct samsung_pinctrl_of_match_data s3c2450_of_data; +extern const struct samsung_pinctrl_of_match_data s5pv210_of_data;#endif /* __PINCTRL_SAMSUNG_H */
Greg Kroah-Hartman
maintainerPATCH 31/101 Bluetooth: btusb: Remove Yoga 920 from the btusb_needs_reset_resume_table
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit f0e8c61110c2c85903b136ba070daf643a8b6842 upstream.
Commit 1fdb92697469 ("Bluetooth: btusb: Use DMI matching for QCA reset_resume quirking"), added the Lenovo Yoga 920 to the btusb_needs_reset_resume_table.
Testing has shown that this is a false positive and the problems where caused by issues with the initial fix: commit fd865802c66b ("Bluetooth:
btusb: fix QCA Rome suspend/resume"), which has already been reverted.
So the QCA Rome BT in the Yoga 920 does not need a reset-resume quirk at all and this commit removes it from the btusb_needs_reset_resume_table.
Note that after this commit the btusb_needs_reset_resume_table is now empty. It is kept around on purpose, since this whole series of commits started for a reason and there are actually broken platforms around, which need to be added to it.
--- drivers/bluetooth/btusb.c | 7 ------- 1 file changed, 7 deletions(-)
drivers/bluetooth/btusb.c +0 -7
@@ -387,13 +387,6 @@ static const struct usb_device_id blackl* the module itself. So we use a DMI list to match known broken platforms. */ static const struct dmi_system_id btusb_needs_reset_resume_table[] = { - { -/* Lenovo Yoga 920 (QCA Rome device 0cf3:e300) */ - .matches = { -DMI_MATCH (DMI_SYS_VENDOR ,"LENOVO" ), -DMI_MATCH (DMI_PRODUCT_VERSION ,"Lenovo YOGA 920" ), - }, - }, {} };
Greg Kroah-Hartman
maintainerPATCH 32/101 Bluetooth: btusb: Add Dell OptiPlex 3060 to btusb_needs_reset_resume_table
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kai-Heng Feng <[email protected]>
commit 0c6e526646c04ce31d4aaa280ed2237dd1cd774c upstream.
The issue can be reproduced before commit fd865802c66b ("Bluetooth:
btusb: fix QCA Rome suspend/resume") gets introduced, so the reset resume quirk is still needed for this system.
T: Bus=01 Lev=01 Prnt=01 Port=13 Cnt=01 Dev#= 4 Spd=12 MxCh= 0 D: Ver= 2.01 Cls=e0(wlcon) Sub=01 Prot=01 MxPS=64 #Cfgs= 1 P: Vendor=0cf3 ProdID=e007 Rev=00.01 C: #Ifs= 2 Cfg#= 1 Atr=e0 MxPwr=100mA I: If#= 0 Alt= 0 #EPs= 3 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb I: If#= 1 Alt= 0 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
--- drivers/bluetooth/btusb.c | 7 +++++++ 1 file changed, 7 insertions(+)
drivers/bluetooth/btusb.c +7 -0
@@ -387,6 +387,13 @@ static const struct usb_device_id blackl* the module itself. So we use a DMI list to match known broken platforms. */ static const struct dmi_system_id btusb_needs_reset_resume_table[] = { + { +/* Dell OptiPlex 3060 (QCA ROME device 0cf3:e007) */ + .matches = { +DMI_MATCH (DMI_SYS_VENDOR ,"Dell Inc." ), +DMI_MATCH (DMI_PRODUCT_NAME ,"OptiPlex 3060" ), + }, + }, {} };
Greg Kroah-Hartman
maintainerPATCH 33/101 Bluetooth: btusb: Fix quirk for Atheros 1525/QCA6174
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit f44cb4b19ed40b655c2d422c9021ab2c2625adb6 upstream.
The Atheros 1525/QCA6174 BT doesn't seem working properly on the recent kernels, as it tries to load a wrong firmware ar3k/AthrBT_0x00000200.dfu and it fails.
This seems to have been a problem for some time, and the known workaround is to apply BTUSB_QCA_ROM quirk instead of BTUSB_ATH3012.
The device in question is:
T: Bus=01 Lev=01 Prnt=01 Port=09 Cnt=03 Dev#= 4 Spd=12 MxCh= 0
D: Ver= 1.10 Cls=e0(wlcon) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
P: Vendor=0cf3 ProdID=3004 Rev= 0.01
C:* #Ifs= 2 Cfg#= 1 Atr=e0 MxPwr=100mA
I:* If#= 0 Alt= 0 #EPs= 3 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=81(I) Atr=03(Int.) MxPS= 16 Ivl=1ms
E: Ad=82(I) Atr=02(Bulk) MxPS= 64 Ivl=0ms
E: Ad=02(O) Atr=02(Bulk) MxPS= 64 Ivl=0ms
I:* If#= 1 Alt= 0 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=83(I) Atr=01(Isoc) MxPS= 0 Ivl=1ms
E: Ad=03(O) Atr=01(Isoc) MxPS= 0 Ivl=1ms
I: If#= 1 Alt= 1 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=83(I) Atr=01(Isoc) MxPS= 9 Ivl=1ms
E: Ad=03(O) Atr=01(Isoc) MxPS= 9 Ivl=1ms
I: If#= 1 Alt= 2 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=83(I) Atr=01(Isoc) MxPS= 17 Ivl=1ms
E: Ad=03(O) Atr=01(Isoc) MxPS= 17 Ivl=1ms
I: If#= 1 Alt= 3 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=83(I) Atr=01(Isoc) MxPS= 25 Ivl=1ms
E: Ad=03(O) Atr=01(Isoc) MxPS= 25 Ivl=1ms
I: If#= 1 Alt= 4 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=83(I) Atr=01(Isoc) MxPS= 33 Ivl=1ms
E: Ad=03(O) Atr=01(Isoc) MxPS= 33 Ivl=1ms
I: If#= 1 Alt= 5 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb
E: Ad=83(I) Atr=01(Isoc) MxPS= 49 Ivl=1ms
E: Ad=03(O) Atr=01(Isoc) MxPS= 49 Ivl=1ms
Bugzilla: http://bugzilla.opensuse.org/show_bug.cgi?id=1082504
--- drivers/bluetooth/btusb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/bluetooth/btusb.c +1 -1
@@ -235,7 +235,6 @@ static const struct usb_device_id blackl { USB_DEVICE(0x0930, 0x0227), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x0b05, 0x17d0), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x0cf3, 0x0036), .driver_info = BTUSB_ATH3012 }, - { USB_DEVICE(0x0cf3, 0x3004), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x0cf3, 0x3008), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x0cf3, 0x311d), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x0cf3, 0x311e), .driver_info = BTUSB_ATH3012 },
@@ -268,6 +267,7 @@ static const struct usb_device_id blackl { USB_DEVICE(0x0489, 0xe03c), .driver_info = BTUSB_ATH3012 }, /* QCA ROME chipset */ + { USB_DEVICE(0x0cf3, 0x3004), .driver_info = BTUSB_QCA_ROME }, { USB_DEVICE(0x0cf3, 0xe007), .driver_info = BTUSB_QCA_ROME }, { USB_DEVICE(0x0cf3, 0xe009), .driver_info = BTUSB_QCA_ROME }, { USB_DEVICE(0x0cf3, 0xe300), .driver_info = BTUSB_QCA_ROME },
Greg Kroah-Hartman
maintainerPATCH 34/101 libata: fix length validation of ATAPI-relayed SCSI commands
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Eric Biggers <[email protected]>
commit 058f58e235cbe03e923b30ea7c49995a46a8725f upstream.
syzkaller reported a crash in ata_bmdma_fill_sg() when writing to /dev/sg1. The immediate cause was that the ATA command's scatterlist was not DMA-mapped, which causes 'pi - 1' to underflow, resulting in a write to 'qc->ap->bmdma_prd[0xffffffff]'.
Strangely though, the flag ATA_QCFLAG_DMAMAP was set in qc->flags. The root cause is that when __ata_scsi_queuecmd() is preparing to relay a SCSI command to an ATAPI device, it doesn't correctly validate the CDB length before copying it into the 16-byte buffer 'cdb' in 'struct ata_queued_cmd'. Namely, it validates the fixed CDB length expected based on the SCSI opcode but not the actual CDB length, which can be larger due to the use of the SG_NEXT_CMD_LEN ioctl. Since 'flags' is the next member in ata_queued_cmd, a buffer overflow corrupts it.
Fix it by requiring that the actual CDB length be <= 16 (ATAPI_CDB_LEN).
[Really it seems the length should be required to be <= dev->cdb_len, but the current behavior seems to have been intentionally introduced by commit 607126c2a21c ("libata-scsi: be tolerant of 12-byte ATAPI commands in 16-byte CDBs") to work around a userspace bug in mplayer. Probably the workaround is no longer needed (mplayer was fixed in 2007), but continuing to allow lengths to up 16 appears harmless for now.]
Here's a reproducer that works in QEMU when /dev/sg1 refers to the CD-ROM drive that qemu-system-x86_64 creates by default:
#include <fcntl.h>
#include <sys/ioctl.h>
#include <unistd.h>
#define SG_NEXT_CMD_LEN 0x2283
int main()
{
char buf[53] = { [36] = 0x7e, [52] = 0x02 };
int fd = open("/dev/sg1", O_RDWR);
ioctl(fd, SG_NEXT_CMD_LEN, &(int){ 17 });
write(fd, buf, sizeof(buf));
}The crash was:
BUG: unable to handle kernel paging request at ffff8cb97db37ffc
IP: ata_bmdma_fill_sg drivers/ata/libata-sff.c:2623 [inline]
IP: ata_bmdma_qc_prep+0xa4/0xc0 drivers/ata/libata-sff.c:2727
PGD fb6c067 P4D fb6c067 PUD 0
Oops: 0002 [#1] SMP
CPU: 1 PID: 150 Comm: syz_ata_bmdma_q Not tainted 4.15.0-next-20180202 #99
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.11.0-20171110_100015-anatol 04/01/2014
[...]
Call Trace:
ata_qc_issue+0x100/0x1d0 drivers/ata/libata-core.c:5421
ata_scsi_translate+0xc9/0x1a0 drivers/ata/libata-scsi.c:2024
__ata_scsi_queuecmd drivers/ata/libata-scsi.c:4326 [inline]
ata_scsi_queuecmd+0x8c/0x210 drivers/ata/libata-scsi.c:4375
scsi_dispatch_cmd+0xa2/0xe0 drivers/scsi/scsi_lib.c:1727
scsi_request_fn+0x24c/0x530 drivers/scsi/scsi_lib.c:1865
__blk_run_queue_uncond block/blk-core.c:412 [inline]
__blk_run_queue+0x3a/0x60 block/blk-core.c:432
blk_execute_rq_nowait+0x93/0xc0 block/blk-exec.c:78
sg_common_write.isra.7+0x272/0x5a0 drivers/scsi/sg.c:806
sg_write+0x1ef/0x340 drivers/scsi/sg.c:677
__vfs_write+0x31/0x160 fs/read_write.c:480
vfs_write+0xa7/0x160 fs/read_write.c:544
SYSC_write fs/read_write.c:589 [inline]
SyS_write+0x4d/0xc0 fs/read_write.c:581
do_syscall_64+0x5e/0x110 arch/x86/entry/common.c:287
entry_SYSCALL_64_after_hwframe+0x21/0x86--- drivers/ata/libata-scsi.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/ata/libata-scsi.c +3 -1
@@ -4308,7 +4308,9 @@ static inline int __ata_scsi_queuecmd(stif (likely ((scsi_op !=ATA_16 ) || !atapi_passthru16)) {/* relay SCSI command to ATAPI device */ int len =COMMAND_SIZE (scsi_op); -if (unlikely (len > scmd->cmd_len || len > dev->cdb_len)) +if (unlikely (len > scmd->cmd_len || + len > dev->cdb_len || + scmd->cmd_len >ATAPI_CDB_LEN )) goto bad_cdb_len; xlat_func = atapi_xlat;
Greg Kroah-Hartman
maintainerPATCH 35/101 libata: remove WARN() for DMA or PIO command without data
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Eric Biggers <[email protected]>
commit 9173e5e80729c8434b8d27531527c5245f4a5594 upstream.
syzkaller hit a WARN() in ata_qc_issue() when writing to /dev/sg0. This happened because it issued a READ_6 command with no data buffer.
Just remove the WARN(), as it doesn't appear indicate a kernel bug. The expected behavior is to fail the command, which the code does.
Here's a reproducer that works in QEMU when /dev/sg0 refers to a disk of the default type ("82371SB PIIX3 IDE"):
#include <fcntl.h>
#include <unistd.h>
int main()
{
char buf[42] = { [36] = 0x8 /* READ_6 */ };
write(open("/dev/sg0", O_RDWR), buf, sizeof(buf));
}--- drivers/ata/libata-core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/ata/libata-core.c +1 -2
@@ -5391,8 +5391,7 @@ void ata_qc_issue(struct ata_queued_cmd* We guarantee to LLDs that they will have at least one * non-zero sg if the command is a data command. */ -if (WARN_ON_ONCE (ata_is_data (prot) && - (!qc->sg || !qc->n_elem || !qc->nbytes))) +if (ata_is_data (prot) && (!qc->sg || !qc->n_elem || !qc->nbytes)) goto sys_err; if (ata_is_dma(prot) || (ata_is_pio(prot) &&
Greg Kroah-Hartman
maintainerPATCH 36/101 libata: dont try to pass through NCQ commands to non-NCQ devices
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Eric Biggers <[email protected]>
commit 2c1ec6fda2d07044cda922ee25337cf5d4b429b3 upstream.
syzkaller hit a WARN() in ata_bmdma_qc_issue() when writing to /dev/sg0. This happened because it issued an ATA pass-through command (ATA_16) where the protocol field indicated that NCQ should be used -- but the device did not support NCQ.
We could just remove the WARN() from libata-sff.c, but the real problem seems to be that the SCSI -> ATA translation code passes through NCQ commands without verifying that the device actually supports NCQ.
Fix this by adding the appropriate check to ata_scsi_pass_thru().
Here's reproducer that works in QEMU when /dev/sg0 refers to a disk of the default type ("82371SB PIIX3 IDE"):
#include <fcntl.h>
#include <unistd.h>
int main()
{
char buf[53] = { 0 };
buf[36] = 0x85; /* ATA_16 */
buf[37] = (12 << 1); /* FPDMA */
buf[38] = 0x1; /* Has data */
buf[51] = 0xC8; /* ATA_CMD_READ */
write(open("/dev/sg0", O_RDWR), buf, sizeof(buf));
}--- drivers/ata/libata-scsi.c | 6 ++++++ 1 file changed, 6 insertions(+)
drivers/ata/libata-scsi.c +6 -0
@@ -3315,6 +3315,12 @@ static unsigned int ata_scsi_pass_thru(s goto invalid_fld; } +/* We may not issue NCQ commands to devices not supporting NCQ */ +if (ata_is_ncq (tf->protocol) && !ata_ncq_enabled (dev)) { + fp =1 ; + goto invalid_fld; + } +/* sanity check for pio multi commands */ if ((cdb[1 ] &0xe0 ) && !is_multi_taskfile (tf)) { fp =1 ;
Greg Kroah-Hartman
maintainerPATCH 37/101 libata: Apply NOLPM quirk to Crucial MX100 512GB SSDs
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit 9c7be59fc519af9081c46c48f06f2b8fadf55ad8 upstream.
Various people have reported the Crucial MX100 512GB model not working with LPM set to min_power. I've now received a report that it also does not work with the new med_power_with_dipm level.
It does work with medium_power, but that has no measurable power-savings and given the amount of people being bitten by the other levels not working, this commit just disables LPM altogether.
Note all reporters of this have either the 512GB model (max capacity), or are not specifying their SSD's size. So for now this quirk assumes this is a problem with the 512GB model only.
--- drivers/ata/libata-core.c | 5 +++++ 1 file changed, 5 insertions(+)
drivers/ata/libata-core.c +5 -0
@@ -4520,6 +4520,11 @@ static const struct ata_blacklist_entry { "PIONEER DVD-RW DVR-212D", NULL, ATA_HORKAGE_NOSETXFER }, { "PIONEER DVD-RW DVR-216D", NULL, ATA_HORKAGE_NOSETXFER }, +/* The 512GB version of the MX100 has both queued TRIM and LPM issues */ + { "Crucial_CT512MX100*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | +ATA_HORKAGE_ZERO_AFTER_TRIM | + ATA_HORKAGE_NOLPM, }, + /* devices that don't properly handle queued TRIM commands */ { "Micron_M500_*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | ATA_HORKAGE_ZERO_AFTER_TRIM, },
Greg Kroah-Hartman
maintainerPATCH 38/101 libata: disable LPM for Crucial BX100 SSD 500GB drive
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kai-Heng Feng <[email protected]>
commit b17e5729a630d8326a48ec34ef02e6b4464a6aef upstream.
After Laptop Mode Tools starts to use min_power for LPM, a user found out Crucial BX100 SSD can't get mounted.
Crucial BX100 SSD 500GB drive don't work well with min_power. This also happens to med_power_with_dipm.
So let's disable LPM for Crucial BX100 SSD 500GB drive.
--- drivers/ata/libata-core.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/ata/libata-core.c +3 -0
@@ -4520,6 +4520,9 @@ static const struct ata_blacklist_entry { "PIONEER DVD-RW DVR-212D", NULL, ATA_HORKAGE_NOSETXFER }, { "PIONEER DVD-RW DVR-216D", NULL, ATA_HORKAGE_NOSETXFER }, +/* Crucial BX100 SSD 500GB has broken LPM support */ + { "CT500BX100SSD1", "MU02", ATA_HORKAGE_NOLPM }, +/* The 512GB version of the MX100 has both queued TRIM and LPM issues */ { "Crucial_CT512MX100*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | ATA_HORKAGE_ZERO_AFTER_TRIM |
Greg Kroah-Hartman
maintainerPATCH 39/101 libata: Enable queued TRIM for Samsung SSD 860
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Ju Hyung Park <[email protected]>
commit ca6bfcb2f6d9deab3924bf901e73622a94900473 upstream.
Samsung explicitly states that queued TRIM is supported for Linux with 860 PRO and 860 EVO.
Make the previous blacklist to cover only 840 and 850 series.
--- drivers/ata/libata-core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/ata/libata-core.c +3 -1
@@ -4539,7 +4539,9 @@ static const struct ata_blacklist_entry ATA_HORKAGE_ZERO_AFTER_TRIM, }, {"Crucial_CT*MX100*" ,"MU01" ,ATA_HORKAGE_NO_NCQ_TRIM | ATA_HORKAGE_ZERO_AFTER_TRIM, }, - { "Samsung SSD 8*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | + { "Samsung SSD 840*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | + ATA_HORKAGE_ZERO_AFTER_TRIM, }, + { "Samsung SSD 850*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | ATA_HORKAGE_ZERO_AFTER_TRIM, }, { "FCCT*M500*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | ATA_HORKAGE_ZERO_AFTER_TRIM, },
Greg Kroah-Hartman
maintainerPATCH 40/101 libata: Apply NOLPM quirk to Crucial M500 480 and 960GB SSDs
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit 62ac3f7305470e3f52f159de448bc1a771717e88 upstream.
There have been reports of the Crucial M500 480GB model not working with LPM set to min_power / med_power_with_dipm level.
It has not been tested with medium_power, but that typically has no measurable power-savings.
Note the reporters Crucial_CT480M500SSD3 has a firmware version of MU03 and there is a MU05 update available, but that update does not mention any LPM fixes in its changelog, so the quirk matches all firmware versions.
In my experience the LPM problems with (older) Crucial SSDs seem to be limited to higher capacity versions of the SSDs (different firmware?), so this commit adds a NOLPM quirk for the 480 and 960GB versions of the M500, to avoid LPM causing issues with these SSDs.
--- drivers/ata/libata-core.c | 8 ++++++++ 1 file changed, 8 insertions(+)
drivers/ata/libata-core.c +8 -0
@@ -4528,6 +4528,14 @@ static const struct ata_blacklist_entry ATA_HORKAGE_ZERO_AFTER_TRIM | ATA_HORKAGE_NOLPM, }, +/* 480GB+ M500 SSDs have both queued TRIM and LPM issues */ + {"Crucial_CT480M500*" ,NULL ,ATA_HORKAGE_NO_NCQ_TRIM | +ATA_HORKAGE_ZERO_AFTER_TRIM | + ATA_HORKAGE_NOLPM, }, + { "Crucial_CT960M500*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | +ATA_HORKAGE_ZERO_AFTER_TRIM | + ATA_HORKAGE_NOLPM, }, + /* devices that don't properly handle queued TRIM commands */ { "Micron_M500_*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | ATA_HORKAGE_ZERO_AFTER_TRIM, },
Greg Kroah-Hartman
maintainerPATCH 41/101 libata: Make Crucial BX100 500GB LPM quirk apply to all firmware versions
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit 3bf7b5d6d017c27e0d3b160aafb35a8e7cfeda1f upstream.
Commit b17e5729a630 ("libata: disable LPM for Crucial BX100 SSD 500GB drive"), introduced a ATA_HORKAGE_NOLPM quirk for Crucial BX100 500GB SSDs but limited this to the MU02 firmware version, according to:
http://www.crucial.com/usa/en/support-ssd-firmware
MU02 is the last version, so there are no newer possibly fixed versions and if the MU02 version has broken LPM then the MU01 almost certainly also has broken LPM, so this commit changes the quirk to apply to all firmware versions.
--- drivers/ata/libata-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/ata/libata-core.c +1 -1
@@ -4521,7 +4521,7 @@ static const struct ata_blacklist_entry { "PIONEER DVD-RW DVR-216D", NULL, ATA_HORKAGE_NOSETXFER },/* Crucial BX100 SSD 500GB has broken LPM support */ - { "CT500BX100SSD1", "MU02", ATA_HORKAGE_NOLPM }, + { "CT500BX100SSD1", NULL, ATA_HORKAGE_NOLPM },/* The 512GB version of the MX100 has both queued TRIM and LPM issues */ { "Crucial_CT512MX100*", NULL, ATA_HORKAGE_NO_NCQ_TRIM |
Greg Kroah-Hartman
maintainerPATCH 42/101 libata: Modify quirks for MX100 to limit NCQ_TRIM quirk to MU01 version
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hans de Goede <[email protected]>
commit d418ff56b8f2d2b296daafa8da151fe27689b757 upstream.
When commit 9c7be59fc519af ("libata: Apply NOLPM quirk to Crucial MX100 512GB SSDs") was added it inherited the ATA_HORKAGE_NO_NCQ_TRIM quirk from the existing "Crucial_CT*MX100*" entry, but that entry sets model_rev to "MU01", where as the entry adding the NOLPM quirk sets it to NULL.
This means that after this commit we no apply the NO_NCQ_TRIM quirk to all "Crucial_CT512MX100*" SSDs even if they have the fixed "MU02" firmware. This commit splits the "Crucial_CT512MX100*" quirk into 2 quirks, one for the "MU01" firmware and one for all other firmware versions, so that we once again only apply the NO_NCQ_TRIM quirk to the "MU01" firmware version.
--- drivers/ata/libata-core.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
drivers/ata/libata-core.c +5 -2
@@ -4523,10 +4523,13 @@ static const struct ata_blacklist_entry/* Crucial BX100 SSD 500GB has broken LPM support */ { "CT500BX100SSD1", NULL, ATA_HORKAGE_NOLPM }, -/* The 512GB version of the MX100 has both queued TRIM and LPM issues */ - { "Crucial_CT512MX100*", NULL, ATA_HORKAGE_NO_NCQ_TRIM | +/* 512GB MX100 with MU01 firmware has both queued TRIM and LPM issues */ + { "Crucial_CT512MX100*", "MU01", ATA_HORKAGE_NO_NCQ_TRIM |ATA_HORKAGE_ZERO_AFTER_TRIM | ATA_HORKAGE_NOLPM, }, + /* 512GB MX100 with newer firmware has only LPM issues */ + { "Crucial_CT512MX100*", NULL, ATA_HORKAGE_ZERO_AFTER_TRIM | + ATA_HORKAGE_NOLPM, }, /* 480GB+ M500 SSDs have both queued TRIM and LPM issues */ { "Crucial_CT480M500*", NULL, ATA_HORKAGE_NO_NCQ_TRIM |
Greg Kroah-Hartman
maintainerPATCH 43/101 cgroup: fix rule checking for threaded mode switching
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Tejun Heo <[email protected]>
commit d1897c9538edafd4ae6bbd03cc075962ddde2c21 upstream.
A domain cgroup isn't allowed to be turned threaded if its subtree is populated or domain controllers are enabled. cgroup_enable_threaded() depended on cgroup_can_be_thread_root() test to enforce this rule. A parent which has populated domain descendants or have domain controllers enabled can't become a thread root, so the above rules are enforced automatically.
However, for the root cgroup which can host mixed domain and threaded children, cgroup_can_be_thread_root() doesn't check any of those conditions and thus first level cgroups ends up escaping those rules.
This patch fixes the bug by adding explicit checks for those rules in cgroup_enable_threaded().
--- kernel/cgroup/cgroup.c | 10 ++++++++++ 1 file changed, 10 insertions(+)
kernel/cgroup/cgroup.c +10 -0
@@ -3148,6 +3148,16 @@ static int cgroup_enable_threaded(structif (cgroup_is_threaded (cgrp))return 0 ; +/* +* If @cgroup is populated or has domain controllers enabled, it +* can't be switched. While the below cgroup_can_be_thread_root() +* test can catch the same conditions, that's only when @parent is +* not mixable, so let's check it explicitly. +*/ +if (cgroup_is_populated (cgrp) || + cgrp->subtree_control & ~cgrp_dfl_threaded_ss_mask) +return -EOPNOTSUPP ; +/* we're joining the parent's domain, ensure its validity */ if (!cgroup_is_valid_domain (dom_cgrp) || !cgroup_can_be_thread_root (dom_cgrp))
Greg Kroah-Hartman
maintainerPATCH 44/101 nfsd: remove blocked locks on client teardown
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jeff Layton <[email protected]>
commit 68ef3bc3166468678d5e1fdd216628c35bd1186f upstream.
We had some reports of panics in nfsd4_lm_notify, and that showed a nfs4_lockowner that had outlived its so_client.
Ensure that we walk any leftover lockowners after tearing down all of the stateids, and remove any blocked locks that they hold.
With this change, we also don't need to walk the nbl_lru on nfsd_net shutdown, as that will happen naturally when we tear down the clients.
--- fs/nfsd/nfs4state.c | 62 ++++++++++++++++++++++++++++++++++++---------------- 1 file changed, 43 insertions(+), 19 deletions(-)
fs/nfsd/nfs4state.c +43 -19
@@ -263,6 +263,35 @@ free_blocked_lock(struct nfsd4_blocked_lkfree (nbl); } +static void +remove_blocked_locks (struct nfs4_lockowner *lo) +{ +struct nfs4_client *clp = lo->lo_owner.so_client; +struct nfsd_net *nn =net_generic (clp->net, nfsd_net_id); +struct nfsd4_blocked_lock *nbl; +LIST_HEAD (reaplist); + +/* Dequeue all blocked locks */ +spin_lock (&nn->blocked_locks_lock); +while (!list_empty (&lo->lo_blocked)) { + nbl =list_first_entry (&lo->lo_blocked, + struct nfsd4_blocked_lock, + nbl_list); +list_del_init (&nbl->nbl_list); +list_move (&nbl->nbl_lru, &reaplist); + } +spin_unlock (&nn->blocked_locks_lock); + +/* Now free them */ +while (!list_empty (&reaplist)) { + nbl = list_first_entry(&reaplist, struct nfsd4_blocked_lock, + nbl_lru); +list_del_init (&nbl->nbl_lru); +posix_unblock_lock (&nbl->nbl_lock); +free_blocked_lock (nbl); + } +} +static int nfsd4_cb_notify_lock_done (struct nfsd4_callback *cb,struct rpc_task *task) {
@@ -1854,6 +1883,7 @@ static __be32 mark_client_expired_lockedstatic void __destroy_client (struct nfs4_client *clp) { +int i;struct nfs4_openowner *oo;struct nfs4_delegation *dp;struct list_head reaplist;
@@ -1883,6 +1913,16 @@ __destroy_client(struct nfs4_client *clpnfs4_get_stateowner (&oo->oo_owner);release_openowner (oo); } +for (i =0 ; i <OWNER_HASH_SIZE ; i++) { +struct nfs4_stateowner *so, *tmp; + +list_for_each_entry_safe (so, tmp, &clp->cl_ownerstr_hashtbl[i], + so_strhash) { +/* Should be no openowners at this point */ +WARN_ON_ONCE (so->so_is_open_owner); +remove_blocked_locks (lockowner (so)); + } + }nfsd4_return_all_client_layouts (clp);nfsd4_shutdown_callback (clp);if (clp->cl_cb_conn.cb_xprt)
@@ -6275,6 +6315,7 @@ nfsd4_release_lockowner(struct svc_rqst }spin_unlock (&clp->cl_lock);free_ol_stateid_reaplist (&reaplist); +remove_blocked_locks (lo);nfs4_put_stateowner (&lo->lo_owner);return status;
@@ -7060,6 +7101,8 @@ nfs4_state_destroy_net(struct net *net) } } +WARN_ON (!list_empty (&nn->blocked_locks_lru)); +for (i =0 ; i <CLIENT_HASH_SIZE ; i++) {while (!list_empty (&nn->unconf_id_hashtbl[i])) { clp = list_entry(nn->unconf_id_hashtbl[i].next, struct nfs4_client, cl_idhash);
@@ -7126,7 +7169,6 @@ nfs4_state_shutdown_net(struct net *net)struct nfs4_delegation *dp =NULL ;struct list_head *pos, *next, reaplist;struct nfsd_net *nn =net_generic (net, nfsd_net_id); -struct nfsd4_blocked_lock *nbl;cancel_delayed_work_sync (&nn->laundromat_work);locks_end_grace (&nn->nfsd4_manager);
@@ -7147,24 +7189,6 @@ nfs4_state_shutdown_net(struct net *net)nfs4_put_stid (&dp->dl_stid); } -BUG_ON (!list_empty (&reaplist)); -spin_lock (&nn->blocked_locks_lock); -while (!list_empty (&nn->blocked_locks_lru)) { - nbl =list_first_entry (&nn->blocked_locks_lru, - struct nfsd4_blocked_lock, nbl_lru); -list_move (&nbl->nbl_lru, &reaplist); -list_del_init (&nbl->nbl_list); - } -spin_unlock (&nn->blocked_locks_lock); - -while (!list_empty (&reaplist)) { - nbl =list_first_entry (&reaplist, - struct nfsd4_blocked_lock, nbl_lru); -list_del_init (&nbl->nbl_lru); -posix_unblock_lock (&nbl->nbl_lock); -free_blocked_lock (nbl); - } -nfsd4_client_tracking_exit (net);nfs4_state_destroy_net (net); }
Greg Kroah-Hartman
maintainerPATCH 45/101 hugetlbfs: check for pgoff value overflow
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mike Kravetz <[email protected]>
commit 63489f8e821144000e0bdca7e65a8d1cc23a7ee7 upstream.
A vma with vm_pgoff large enough to overflow a loff_t type when converted to a byte offset can be passed via the remap_file_pages system call. The hugetlbfs mmap routine uses the byte offset to calculate reservations and file size.
A sequence such as:
mmap(0x20a00000, 0x600000, 0, 0x66033, -1, 0); remap_file_pages(0x20a00000, 0x600000, 0, 0x20000000000000, 0);
will result in the following when task exits/file closed,
kernel BUG at mm/hugetlb.c:749!
Call Trace:
hugetlbfs_evict_inode+0x2f/0x40
evict+0xcb/0x190
__dentry_kill+0xcb/0x150
__fput+0x164/0x1e0
task_work_run+0x84/0xa0
exit_to_usermode_loop+0x7d/0x80
do_syscall_64+0x18b/0x190
entry_SYSCALL_64_after_hwframe+0x3d/0xa2The overflowed pgoff value causes hugetlbfs to try to set up a mapping with a negative range (end < start) that leaves invalid state which causes the BUG.
The previous overflow fix to this code was incomplete and did not take the remap_file_pages system call into account.
--- fs/hugetlbfs/inode.c | 17 ++++++++++++++--- mm/hugetlb.c | 7 +++++++ 2 files changed, 21 insertions(+), 3 deletions(-)
fs/hugetlbfs/inode.c +14 -3
@@ -118,6 +118,16 @@ static void huge_pagevec_release(structpagevec_reinit (pvec); } +/* +* Mask used when checking the page offset value passed in via system +* calls. This value will be converted to a loff_t which is signed. +* Therefore, we want to check the upper PAGE_SHIFT + 1 bits of the +* value. The extra bit (- 1 in the shift value) is to take the sign +* bit into account. +*/ +#define PGOFF_LOFFT_MAX \ + (((1UL << (PAGE_SHIFT + 1)) - 1) << (BITS_PER_LONG - (PAGE_SHIFT + 1))) +static int hugetlbfs_file_mmap (struct file *file,struct vm_area_struct *vma) {struct inode *inode =file_inode (file);
@@ -137,12 +147,13 @@ static int hugetlbfs_file_mmap(struct fi vma->vm_ops = &hugetlb_vm_ops;/* -* Offset passed to mmap (before page shift) could have been -* negative when represented as a (l)off_t. +* page based offset in vm_pgoff could be sufficiently large to +* overflow a (l)off_t when converted to byte offset. */ -if (((loff_t )vma->vm_pgoff <<PAGE_SHIFT ) <0 ) +if (vma->vm_pgoff &PGOFF_LOFFT_MAX )return -EINVAL ; +/* must be huge page aligned */ if (vma->vm_pgoff & (~huge_page_mask (h) >>PAGE_SHIFT ))return -EINVAL ;
mm/hugetlb.c +7 -0
@@ -18,6 +18,7 @@#include <linux/bootmem.h> #include <linux/sysfs.h> #include <linux/slab.h> +#include <linux/mmdebug.h> #include <linux/sched/signal.h> #include <linux/rmap.h> #include <linux/string_helpers.h>
@@ -4344,6 +4345,12 @@ int hugetlb_reserve_pages(struct inode *struct resv_map *resv_map;long gbl_reserve; +/* This should never happen */ +if (from > to) { +VM_WARN (1 ,"%s called with a negative range\n" , __func__); +return -EINVAL ; + } +/* * Only apply hugepage reservation if asked. At fault time, an * attempt will be made for VM_NORESERVE to allocate a page
Greg Kroah-Hartman
maintainerPATCH 46/101 h8300: remove extraneous __BIG_ENDIAN definition
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Arnd Bergmann <[email protected]>
commit 1705f7c534163594f8b05e060cb49fbea86ca70b upstream.
A bugfix I did earlier caused a build regression on h8300, which defines the __BIG_ENDIAN macro in a slightly different way than the generic code:
arch/h8300/include/asm/byteorder.h:5:0: warning: "__BIG_ENDIAN" redefined
We don't need to define it here, as the same macro is already provided by the linux/byteorder/big_endian.h, and that version does not conflict.
While this is a v4.16 regression, my earlier patch also got backported to the 4.14 and 4.15 stable kernels, so we need the fixup there as well.
--- arch/h8300/include/asm/byteorder.h | 1 - 1 file changed, 1 deletion(-)
arch/h8300/include/asm/byteorder.h +0 -1
@@ -2,7 +2,6 @@#ifndef __H8300_BYTEORDER_H__#define __H8300_BYTEORDER_H__ -#define __BIG_ENDIAN __ORDER_BIG_ENDIAN__#include <linux/byteorder/big_endian.h> #endif
Greg Kroah-Hartman
maintainerPATCH 47/101 mm/vmalloc: add interfaces to free unmapped page table
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Toshi Kani <[email protected]>
commit b6bdb7517c3d3f41f20e5c2948d6bc3f8897394e upstream.
On architectures with CONFIG_HAVE_ARCH_HUGE_VMAP set, ioremap() may create pud/pmd mappings. A kernel panic was observed on arm64 systems with Cortex-A75 in the following steps as described by Hanjun Guo.
1. ioremap a 4K size, valid page table will build,
2. iounmap it, pte0 will set to 0;
3. ioremap the same address with 2M size, pgd/pmd is unchanged,
then set the a new value for pmd;
4. pte0 is leaked;
5. CPU may meet exception because the old pmd is still in TLB, which will lead to kernel panic.
This panic is not reproducible on x86. INVLPG, called from iounmap, purges all levels of entries associated with purged address on x86. x86 still has memory leak.
The patch changes the ioremap path to free unmapped page table(s) since doing so in the unmap path has the following issues:
- The iounmap() path is shared with vunmap(). Since vmap() only supports pte mappings, making vunmap() to free a pte page is an overhead for regular vmap users as they do not need a pte page freed up.
- Checking if all entries in a pte page are cleared in the unmap path is racy, and serializing this check is expensive.
- The unmap path calls free_vmap_area_noflush() to do lazy TLB purges. Clearing a pud/pmd entry before the lazy TLB purges needs extra TLB purge.
Add two interfaces, pud_free_pmd_page() and pmd_free_pte_page(), which clear a given pud/pmd entry and free up a page for the lower level entries.
This patch implements their stub functions on x86 and arm64, which work as workaround.
--- arch/arm64/mm/mmu.c | 10 ++++++++++ arch/x86/mm/pgtable.c | 24 ++++++++++++++++++++++++ include/asm-generic/pgtable.h | 10 ++++++++++ lib/ioremap.c | 6 ++++-- 4 files changed, 48 insertions(+), 2 deletions(-)
arch/arm64/mm/mmu.c +10 -0
@@ -937,3 +937,13 @@ int pmd_clear_huge(pmd_t *pmd)pmd_clear (pmd);return 1 ; } + +int pud_free_pmd_page (pud_t *pud) +{ +return pud_none (*pud); +} + +int pmd_free_pte_page (pmd_t *pmd) +{ +return pmd_none (*pmd); +}
arch/x86/mm/pgtable.c +24 -0
@@ -702,4 +702,28 @@ int pmd_clear_huge(pmd_t *pmd) return 0; } + +/** +* pud_free_pmd_page - Clear pud entry and free pmd page. +* @pud: Pointer to a PUD. +* +* Context: The pud range has been unmaped and TLB purged. +* Return: 1 if clearing the entry succeeded. 0 otherwise. +*/ +int pud_free_pmd_page (pud_t *pud) +{ +return pud_none (*pud); +} + +/** +* pmd_free_pte_page - Clear pmd entry and free pte page. +* @pmd: Pointer to a PMD. +* +* Context: The pmd range has been unmaped and TLB purged. +* Return: 1 if clearing the entry succeeded. 0 otherwise. +*/ +int pmd_free_pte_page (pmd_t *pmd) +{ +return pmd_none (*pmd); +}#endif /* CONFIG_HAVE_ARCH_HUGE_VMAP */
include/asm-generic/pgtable.h +10 -0
@@ -976,6 +976,8 @@ int pud_set_huge(pud_t *pud, phys_addr_tint pmd_set_huge (pmd_t *pmd,phys_addr_t addr,pgprot_t prot);int pud_clear_huge (pud_t *pud);int pmd_clear_huge (pmd_t *pmd); +int pud_free_pmd_page (pud_t *pud); +int pmd_free_pte_page (pmd_t *pmd);#else /* !CONFIG_HAVE_ARCH_HUGE_VMAP */ static inline int p4d_set_huge (p4d_t *p4d,phys_addr_t addr,pgprot_t prot) {
@@ -1001,6 +1003,14 @@ static inline int pmd_clear_huge(pmd_t * {return 0 ; } +static inline int pud_free_pmd_page (pud_t *pud) +{ +return 0 ; +} +static inline int pmd_free_pte_page (pmd_t *pmd) +{ +return 0 ; +}#endif /* CONFIG_HAVE_ARCH_HUGE_VMAP */ #ifndef __HAVE_ARCH_FLUSH_PMD_TLB_RANGE
lib/ioremap.c +4 -2
@@ -91,7 +91,8 @@ static inline int ioremap_pmd_range(pud_if (ioremap_pmd_enabled () && ((next - addr) ==PMD_SIZE ) && -IS_ALIGNED (phys_addr + addr,PMD_SIZE )) { +IS_ALIGNED (phys_addr + addr,PMD_SIZE ) && +pmd_free_pte_page (pmd)) {if (pmd_set_huge (pmd, phys_addr + addr, prot))continue ; }
@@ -117,7 +118,8 @@ static inline int ioremap_pud_range(p4d_if (ioremap_pud_enabled () && ((next - addr) ==PUD_SIZE ) && -IS_ALIGNED (phys_addr + addr,PUD_SIZE )) { +IS_ALIGNED (phys_addr + addr,PUD_SIZE ) && +pud_free_pmd_page (pud)) {if (pud_set_huge (pud, phys_addr + addr, prot))continue ; }
Greg Kroah-Hartman
maintainerPATCH 48/101 x86/mm: implement free pmd/pte page interfaces
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Toshi Kani <[email protected]>
commit 28ee90fe6048fa7b7ceaeb8831c0e4e454a4cf89 upstream.
Implement pud_free_pmd_page() and pmd_free_pte_page() on x86, which clear a given pud/pmd entry and free up lower level page table(s).
The address range associated with the pud/pmd entry must have been purged by INVLPG.
--- arch/x86/mm/pgtable.c | 28 ++++++++++++++++++++++++++-- 1 file changed, 26 insertions(+), 2 deletions(-)
arch/x86/mm/pgtable.c +26 -2
@@ -712,7 +712,22 @@ int pmd_clear_huge(pmd_t *pmd)*/ int pud_free_pmd_page (pud_t *pud) { -return pud_none (*pud); +pmd_t *pmd; +int i; + +if (pud_none (*pud)) +return 1 ; + + pmd = (pmd_t *)pud_page_vaddr (*pud); + +for (i =0 ; i <PTRS_PER_PMD ; i++) +if (!pmd_free_pte_page (&pmd[i])) +return 0 ; + +pud_clear (pud); +free_page ((unsigned long )pmd); + +return 1 ; }/**
@@ -724,6 +739,15 @@ int pud_free_pmd_page(pud_t *pud)*/ int pmd_free_pte_page (pmd_t *pmd) { -return pmd_none (*pmd); +pte_t *pte; + +if (pmd_none (*pmd)) +return 1 ; + + pte = (pte_t *)pmd_page_vaddr (*pmd); +pmd_clear (pmd); +free_page ((unsigned long )pte); + +return 1 ; }#endif /* CONFIG_HAVE_ARCH_HUGE_VMAP */
Greg Kroah-Hartman
maintainerPATCH 49/101 mm/khugepaged.c: convert VM_BUG_ON() to collapse fail
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kirill A. Shutemov <[email protected]>
commit fece2029a9e65b9a990831afe2a2b83290cbbe26 upstream.
khugepaged is not yet able to convert PTE-mapped huge pages back to PMD mapped. We do not collapse such pages. See check khugepaged_scan_pmd().
But if between khugepaged_scan_pmd() and __collapse_huge_page_isolate() somebody managed to instantiate THP in the range and then split the PMD back to PTEs we would have a problem -- VM_BUG_ON_PAGE(PageCompound(page)) will get triggered.
It's possible since we drop mmap_sem during collapse to re-take for write.
Replace the VM_BUG_ON() with graceful collapse fail.
--- mm/khugepaged.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
mm/khugepaged.c +6 -1
@@ -530,7 +530,12 @@ static int __collapse_huge_page_isolate( goto out; } -VM_BUG_ON_PAGE (PageCompound (page), page); +/* TODO: teach khugepaged to collapse THP mapped with pte */ +if (PageCompound (page)) { + result =SCAN_PAGE_COMPOUND ; + goto out; + } +VM_BUG_ON_PAGE (!PageAnon (page), page);/*
Greg Kroah-Hartman
maintainerPATCH 50/101 mm/thp: do not wait for lock_page() in deferred_split_scan()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kirill A. Shutemov <[email protected]>
commit fa41b900c30b45fab03783724932dc30cd46a6be upstream.
deferred_split_scan() gets called from reclaim path. Waiting for page lock may lead to deadlock there.
Replace lock_page() with trylock_page() and skip the page if we failed to lock it. We will get to the page on the next scan.
--- mm/huge_memory.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
mm/huge_memory.c +3 -1
@@ -2739,11 +2739,13 @@ static unsigned long deferred_split_scanlist_for_each_safe (pos, next, &list) { page = list_entry((void *)pos, struct page, mapping); -lock_page (page); +if (!trylock_page (page)) + goto next;/* split_huge_page() removes page from list on success */ if (!split_huge_page (page)) split++;unlock_page (page); +next:put_page (page); }
Greg Kroah-Hartman
maintainerPATCH 51/101 mm/shmem: do not wait for lock_page() in shmem_unused_huge_shrink()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kirill A. Shutemov <[email protected]>
commit b3cd54b257ad95d344d121dc563d943ca39b0921 upstream.
shmem_unused_huge_shrink() gets called from reclaim path. Waiting for page lock may lead to deadlock there.
There was a bug report that may be attributed to this:
http://lkml.kernel.org/r/[email protected]
Replace lock_page() with trylock_page() and skip the page if we failed to lock it. We will get to the page on the next scan.
We can test for the PageTransHuge() outside the page lock as we only need protection against splitting the page under us. Holding pin oni the page is enough for this.
--- mm/shmem.c | 31 ++++++++++++++++++++----------- 1 file changed, 20 insertions(+), 11 deletions(-)
mm/shmem.c +20 -11
@@ -493,36 +493,45 @@ next: info = list_entry(pos, struct shmem_inode_info, shrinklist); inode = &info->vfs_inode; -if (nr_to_split && split >= nr_to_split) { -iput (inode); -continue ; - } +if (nr_to_split && split >= nr_to_split) + goto leave; - page =find_lock_page (inode->i_mapping, + page =find_get_page (inode->i_mapping, (inode->i_size &HPAGE_PMD_MASK ) >>PAGE_SHIFT );if (!page) goto drop; +/* No huge page at the end of the file: nothing to split */ if (!PageTransHuge (page)) { -unlock_page (page);put_page (page); goto drop; } +/* +* Leave the inode on the list if we failed to lock +* the page at this time. +* +* Waiting for the lock may lead to deadlock in the +* reclaim path. +*/ +if (!trylock_page (page)) { +put_page (page); + goto leave; + } + ret =split_huge_page (page);unlock_page (page);put_page (page); -if (ret) { -/* split failed: leave it on the list */ -iput (inode); -continue ; - } +/* If split failed leave the inode on the list */ +if (ret) + goto leave; split++; drop:list_del_init (&info->shrinklist); removed++; +leave:iput (inode); }
Greg Kroah-Hartman
maintainerPATCH 52/101 Revert "mm: page_alloc: skip over regions of invalid pfns where possible"
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Daniel Vacek <[email protected]>
commit f59f1caf72ba00d519c793c3deb32cd3be32edc2 upstream.
This reverts commit b92df1de5d28 ("mm: page_alloc: skip over regions of invalid pfns where possible"). The commit is meant to be a boot init speed up skipping the loop in memmap_init_zone() for invalid pfns.
But given some specific memory mapping on x86_64 (or more generally theoretically anywhere but on arm with CONFIG_HAVE_ARCH_PFN_VALID) the implementation also skips valid pfns which is plain wrong and causes 'kernel BUG at mm/page_alloc.c:1389!'
crash> log | grep -e BUG -e RIP -e Call.Trace -e move_freepages_block -e rmqueue -e freelist -A1 kernel BUG at mm/page_alloc.c:1389!
invalid opcode: 0000 [#1] SMP
--
RIP: 0010: move_freepages+0x15e/0x160
--
Call Trace:
move_freepages_block+0x73/0x80
__rmqueue+0x263/0x460
get_page_from_freelist+0x7e1/0x9e0
__alloc_pages_nodemask+0x176/0x420
-- crash> page_init_bug -v | grep RAM
<struct resource 0xffff88067fffd2f8> 1000 - 9bfff System RAM (620.00 KiB) <struct resource 0xffff88067fffd3a0> 100000 - 430bffff System RAM ( 1.05 GiB = 1071.75 MiB = 1097472.00 KiB) <struct resource 0xffff88067fffd410> 4b0c8000 - 4bf9cfff System RAM ( 14.83 MiB = 15188.00 KiB) <struct resource 0xffff88067fffd480> 4bfac000 - 646b1fff System RAM (391.02 MiB = 400408.00 KiB) <struct resource 0xffff88067fffd560> 7b788000 - 7b7fffff System RAM (480.00 KiB) <struct resource 0xffff88067fffd640> 100000000 - 67fffffff System RAM ( 22.00 GiB)
crash> page_init_bug | head -6
<struct resource 0xffff88067fffd560> 7b788000 - 7b7fffff System RAM (480.00 KiB)
<struct page 0xffffea0001ede200> 1fffff00000000 0 <struct pglist_data 0xffff88047ffd9000> 1 <struct zone 0xffff88047ffd9800> DMA32 4096 1048575
<struct page 0xffffea0001ede200> 505736 505344 <struct page 0xffffea0001ed8000> 505855 <struct page 0xffffea0001edffc0>
<struct page 0xffffea0001ed8000> 0 0 <struct pglist_data 0xffff88047ffd9000> 0 <struct zone 0xffff88047ffd9000> DMA 1 4095
<struct page 0xffffea0001edffc0> 1fffff00000400 0 <struct pglist_data 0xffff88047ffd9000> 1 <struct zone 0xffff88047ffd9800> DMA32 4096 1048575
BUG, zones differ!
crash> kmem -p 77fff000 78000000 7b5ff000 7b600000 7b787000 7b788000
PAGE PHYSICAL MAPPING INDEX CNT FLAGS ffffea0001e00000 78000000 0 0 0 0 ffffea0001ed7fc0 7b5ff000 0 0 0 0 ffffea0001ed8000 7b600000 0 0 0 0 <<<< ffffea0001ede1c0 7b787000 0 0 0 0 ffffea0001ede200 7b788000 0 0 1 1fffff00000000
--- include/linux/memblock.h | 1 - mm/memblock.c | 28 ---------------------------- mm/page_alloc.c | 11 +---------- 3 files changed, 1 insertion(+), 39 deletions(-)
include/linux/memblock.h +0 -1
@@ -187,7 +187,6 @@ int memblock_search_pfn_nid(unsigned lon unsigned long *end_pfn);void __next_mem_pfn_range (int *idx,int nid,unsigned long *out_start_pfn,unsigned long *out_end_pfn,int *out_nid); -unsigned long memblock_next_valid_pfn (unsigned long pfn,unsigned long max_pfn);/** * for_each_mem_pfn_range - early memory pfn range iterator
mm/memblock.c +0 -28
@@ -1101,34 +1101,6 @@ void __init_memblock __next_mem_pfn_rang *out_nid = r->nid; } -unsigned long __init_memblock memblock_next_valid_pfn (unsigned long pfn, -unsigned long max_pfn) -{ -struct memblock_type *type = &memblock.memory; -unsigned int right = type->cnt; -unsigned int mid, left =0 ; -phys_addr_t addr =PFN_PHYS (++pfn); - -do { - mid = (right + left) /2 ; - -if (addr < type->regions[mid].base) - right = mid; -else if (addr >= (type->regions[mid].base + - type->regions[mid].size)) - left = mid +1 ; -else { -/* addr is within the region, so pfn is valid */ -return pfn; - } - }while (left < right); - -if (right == type->cnt) -return -1UL ; -else -return PHYS_PFN (type->regions[right].base); -} -/** * memblock_set_node - set node ID on memblock regions * @base: base of area to set node ID for
mm/page_alloc.c +1 -10
@@ -5302,17 +5302,8 @@ void __meminit memmap_init_zone(unsignedif (context !=MEMMAP_EARLY ) goto not_early; -if (!early_pfn_valid (pfn)) { -#ifdef CONFIG_HAVE_MEMBLOCK_NODE_MAP -/* -* Skip to the pfn preceding the next valid one (or -* end_pfn), such that we hit a valid pfn (or end_pfn) -* on our next iteration of the loop. -*/ - pfn =memblock_next_valid_pfn (pfn, end_pfn) -1 ; -#endif +if (!early_pfn_valid (pfn))continue ; - }if (!early_pfn_in_nid (pfn, nid))continue ;if (!update_defer_init (pgdat, pfn, end_pfn, &nr_initialised))
Greg Kroah-Hartman
maintainerPATCH 53/101 drm/vmwgfx: Fix black screen and device errors when running without fbdev
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Thomas Hellstrom <[email protected]>
commit 140bcaa23a1c37b694910424075a15e009120dbe upstream.
When we are running without fbdev, transitioning from the login screen to X or gnome-shell/wayland will cause a vt switch and the driver will disable svga mode, losing all modesetting resources. However, the kms atomic state does not reflect that and may think that a crtc is still turned on, which will cause device errors when we try to bind an fb to the crtc, and the screen will remain black.
Fix this by turning off all kms resources before disabling svga mode.
--- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 13 +++++++++++++ drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 1 + drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 11 +++++++++++ drivers/gpu/drm/vmwgfx/vmwgfx_kms.h | 1 - 4 files changed, 25 insertions(+), 1 deletion(-)
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c +13 -0
@@ -1337,6 +1337,19 @@ static void __vmw_svga_disable(struct vm*/ void vmw_svga_disable (struct vmw_private *dev_priv) { +/* +* Disabling SVGA will turn off device modesetting capabilities, so +* notify KMS about that so that it doesn't cache atomic state that +* isn't valid anymore, for example crtcs turned on. +* Strictly we'd want to do this under the SVGA lock (or an SVGA mutex), +* but vmw_kms_lost_device() takes the reservation sem and thus we'll +* end up with lock order reversal. Thus, a master may actually perform +* a new modeset just after we call vmw_kms_lost_device() and race with +* vmw_svga_disable(), but that should at worst cause atomic KMS state +* to be inconsistent with the device, causing modesetting problems. +* +*/ +vmw_kms_lost_device (dev_priv->dev);ttm_write_lock (&dev_priv->reservation_sem, false);spin_lock (&dev_priv->svga_lock);if (dev_priv->bdev.man[TTM_PL_VRAM ].use_type) {
drivers/gpu/drm/vmwgfx/vmwgfx_drv.h +1 -0
@@ -938,6 +938,7 @@ int vmw_kms_present(struct vmw_private *int vmw_kms_update_layout_ioctl (struct drm_device *dev,void *data,struct drm_file *file_priv);void vmw_kms_legacy_hotspot_clear (struct vmw_private *dev_priv); +void vmw_kms_lost_device (struct drm_device *dev); int vmw_dumb_create(struct drm_file *file_priv, struct drm_device *dev,
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c +11 -0
@@ -2865,3 +2865,14 @@ int vmw_kms_set_config(struct drm_mode_sreturn drm_atomic_helper_set_config (set, ctx); } + + +/** +* vmw_kms_lost_device - Notify kms that modesetting capabilities will be lost +* +* @dev: Pointer to the drm device +*/ +void vmw_kms_lost_device (struct drm_device *dev) +{ +drm_atomic_helper_shutdown (dev); +}
drivers/gpu/drm/vmwgfx/vmwgfx_kms.h +0 -1
@@ -439,5 +439,4 @@ int vmw_kms_stdu_dma(struct vmw_privateint vmw_kms_set_config (struct drm_mode_set *set,struct drm_modeset_acquire_ctx *ctx); -#endif
Greg Kroah-Hartman
maintainerPATCH 54/101 drm/vmwgfx: Fix a destoy-while-held mutex problem.
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Thomas Hellstrom <[email protected]>
commit 73a88250b70954a8f27c2444e1c2411bba3c29d9 upstream.
When validating legacy surfaces, the backup bo might be destroyed at surface validate time. However, the kms resource validation code may have the bo reserved, so we will destroy a locked mutex. While there shouldn't be any other users of that mutex when it is destroyed, it causes a lock leak and thus throws a lockdep error.
Fix this by having the kms resource validation code hold a reference to the bo while we have it reserved. We do this by introducing a validation context which might come in handy when the kms code is extended to validate multiple resources or buffers.
--- drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 28 +++++++++++++++++++--------- drivers/gpu/drm/vmwgfx/vmwgfx_kms.h | 12 +++++++++--- drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | 5 +++-- drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c | 5 +++-- 4 files changed, 34 insertions(+), 16 deletions(-)
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c +19 -9
@@ -31,7 +31,6 @@#include <drm/drm_atomic_helper.h> #include <drm/drm_rect.h> -/* Might need a hrtimer here? */ #define VMWGFX_PRESENT_RATE ((HZ / 60 > 0) ? HZ / 60 : 1)
@@ -2531,9 +2530,12 @@ void vmw_kms_helper_buffer_finish(struct* Helper to be used if an error forces the caller to undo the actions of * vmw_kms_helper_resource_prepare. */ -void vmw_kms_helper_resource_revert (struct vmw_resource *res) +void vmw_kms_helper_resource_revert (struct vmw_validation_ctx *ctx) { -vmw_kms_helper_buffer_revert (res->backup); +struct vmw_resource *res = ctx->res; + +vmw_kms_helper_buffer_revert (ctx->buf); +vmw_dmabuf_unreference (&ctx->buf);vmw_resource_unreserve (res, false,NULL ,0 );mutex_unlock (&res->dev_priv->cmdbuf_mutex); }
@@ -2550,10 +2552,14 @@ void vmw_kms_helper_resource_revert(stru* interrupted by a signal. */ int vmw_kms_helper_resource_prepare (struct vmw_resource *res, -bool interruptible) +bool interruptible, +struct vmw_validation_ctx *ctx) {int ret =0 ; + ctx->buf =NULL ; + ctx->res = res; +if (interruptible) ret =mutex_lock_interruptible (&res->dev_priv->cmdbuf_mutex);else
@@ -2572,6 +2578,8 @@ int vmw_kms_helper_resource_prepare(stru res->dev_priv->has_mob);if (ret) goto out_unreserve; + + ctx->buf =vmw_dmabuf_reference (res->backup); } ret =vmw_resource_validate (res);if (ret)
@@ -2579,7 +2587,7 @@ int vmw_kms_helper_resource_prepare(strureturn 0 ; out_revert: -vmw_kms_helper_buffer_revert (res->backup); +vmw_kms_helper_buffer_revert (ctx->buf); out_unreserve:vmw_resource_unreserve (res, false,NULL ,0 ); out_unlock:
@@ -2595,11 +2603,13 @@ out_unlock:* @out_fence: Optional pointer to a fence pointer. If non-NULL, a * ref-counted fence pointer is returned here. */ -void vmw_kms_helper_resource_finish (struct vmw_resource *res, -struct vmw_fence_obj **out_fence) +void vmw_kms_helper_resource_finish (struct vmw_validation_ctx *ctx, +struct vmw_fence_obj **out_fence) { -if (res->backup || out_fence) -vmw_kms_helper_buffer_finish (res->dev_priv,NULL , res->backup, +struct vmw_resource *res = ctx->res; + +if (ctx->buf || out_fence) +vmw_kms_helper_buffer_finish (res->dev_priv,NULL , ctx->buf, out_fence,NULL );vmw_resource_unreserve (res, false,NULL ,0 );
drivers/gpu/drm/vmwgfx/vmwgfx_kms.h +9 -3
@@ -240,6 +240,11 @@ struct vmw_display_unit {int set_gui_y; }; +struct vmw_validation_ctx { +struct vmw_resource *res; +struct vmw_dma_buffer *buf; +}; +#define vmw_crtc_to_du (x) \ container_of(x, struct vmw_display_unit, crtc)#define vmw_connector_to_du (x) \
@@ -296,9 +301,10 @@ void vmw_kms_helper_buffer_finish(structstruct drm_vmw_fence_rep __user * user_fence_rep);int vmw_kms_helper_resource_prepare (struct vmw_resource *res, -bool interruptible); -void vmw_kms_helper_resource_revert (struct vmw_resource *res); -void vmw_kms_helper_resource_finish (struct vmw_resource *res, +bool interruptible, +struct vmw_validation_ctx *ctx); +void vmw_kms_helper_resource_revert (struct vmw_validation_ctx *ctx); +void vmw_kms_helper_resource_finish (struct vmw_validation_ctx *ctx,struct vmw_fence_obj **out_fence); int vmw_kms_readback(struct vmw_private *dev_priv, struct drm_file *file_priv,
drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c +3 -2
@@ -909,12 +909,13 @@ int vmw_kms_sou_do_surface_dirty(structstruct vmw_framebuffer_surface *vfbs =container_of (framebuffer,typeof (*vfbs), base);struct vmw_kms_sou_surface_dirty sdirty; +struct vmw_validation_ctx ctx;int ret;if (!srf) srf = &vfbs->surface->res; - ret =vmw_kms_helper_resource_prepare (srf, true); + ret =vmw_kms_helper_resource_prepare (srf, true, &ctx);if (ret)return ret;
@@ -933,7 +934,7 @@ int vmw_kms_sou_do_surface_dirty(struct ret =vmw_kms_helper_dirty (dev_priv, framebuffer, clips, vclips, dest_x, dest_y, num_clips, inc, &sdirty.base); -vmw_kms_helper_resource_finish (srf, out_fence); +vmw_kms_helper_resource_finish (&ctx, out_fence);return ret; }
drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c +3 -2
@@ -980,12 +980,13 @@ int vmw_kms_stdu_surface_dirty(struct vmstruct vmw_framebuffer_surface *vfbs =container_of (framebuffer,typeof (*vfbs), base);struct vmw_stdu_dirty sdirty; +struct vmw_validation_ctx ctx;int ret;if (!srf) srf = &vfbs->surface->res; - ret =vmw_kms_helper_resource_prepare (srf, true); + ret =vmw_kms_helper_resource_prepare (srf, true, &ctx);if (ret)return ret;
@@ -1008,7 +1009,7 @@ int vmw_kms_stdu_surface_dirty(struct vm dest_x, dest_y, num_clips, inc, &sdirty.base); out_finish: -vmw_kms_helper_resource_finish (srf, out_fence); +vmw_kms_helper_resource_finish (&ctx, out_fence);return ret; }
Greg Kroah-Hartman
maintainerPATCH 55/101 drm/radeon: Dont turn off DP sink when disconnected
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Michel Dänzer <[email protected]>
commit 2681bc79eeb640562c932007bfebbbdc55bf6a7d upstream.
Turning off the sink in this case causes various issues, because userspace expects it to stay on until it turns it off explicitly.
Instead, turn the sink off and back on when a display is connected again. This dance seems necessary for link training to work correctly.
Bugzilla: https://bugs.freedesktop.org/105308
--- drivers/gpu/drm/radeon/radeon_connectors.c | 29 +++++++++++------------------ 1 file changed, 11 insertions(+), 18 deletions(-)
drivers/gpu/drm/radeon/radeon_connectors.c +11 -18
@@ -90,25 +90,18 @@ void radeon_connector_hotplug(struct drm/* don't do anything if sink is not display port, i.e., * passive dp->(dvi|hdmi) adaptor */ -if (dig_connector->dp_sink_type ==CONNECTOR_OBJECT_ID_DISPLAYPORT ) { -int saved_dpms = connector->dpms; -/* Only turn off the display if it's physically disconnected */ -if (!radeon_hpd_sense (rdev, radeon_connector->hpd.hpd)) { -drm_helper_connector_dpms (connector,DRM_MODE_DPMS_OFF ); - }else if (radeon_dp_needs_link_train (radeon_connector)) { -/* Don't try to start link training before we -* have the dpcd */ -if (!radeon_dp_getdpcd (radeon_connector)) -return ; +if (dig_connector->dp_sink_type ==CONNECTOR_OBJECT_ID_DISPLAYPORT && +radeon_hpd_sense (rdev, radeon_connector->hpd.hpd) && +radeon_dp_needs_link_train (radeon_connector)) { +/* Don't start link training before we have the DPCD */ +if (!radeon_dp_getdpcd (radeon_connector)) +return ; -/* set it to OFF so that drm_helper_connector_dpms() -* won't return immediately since the current state -* is ON at this point. -*/ - connector->dpms =DRM_MODE_DPMS_OFF ; -drm_helper_connector_dpms (connector,DRM_MODE_DPMS_ON ); - } - connector->dpms = saved_dpms; +/* Turn the connector off and back on immediately, which +* will trigger link training +*/ +drm_helper_connector_dpms (connector,DRM_MODE_DPMS_OFF ); +drm_helper_connector_dpms (connector,DRM_MODE_DPMS_ON ); } } }
Greg Kroah-Hartman
maintainerPATCH 56/101 drm: Reject getfb for multi-plane framebuffers
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Daniel Stone <[email protected]>
commit b24791fe00f8b089d5b10cb7bcc4e1ae88b4831b upstream.
getfb can only return a single plane, so reject attempts to use it with multi-plane framebuffers.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105518
--- drivers/gpu/drm/drm_framebuffer.c | 7 +++++++ 1 file changed, 7 insertions(+)
drivers/gpu/drm/drm_framebuffer.c +7 -0
@@ -458,6 +458,12 @@ int drm_mode_getfb(struct drm_device *deif (!fb)return -ENOENT ; +/* Multi-planar framebuffers need getfb2. */ +if (fb->format->num_planes >1 ) { + ret = -EINVAL ; + goto out; + } + r->height = fb->height; r->width = fb->width; r->depth = fb->format->depth;
@@ -481,6 +487,7 @@ int drm_mode_getfb(struct drm_device *de ret = -ENODEV ; } +out:drm_framebuffer_put (fb);return ret;
Greg Kroah-Hartman
maintainerPATCH 57/101 drm: udl: Properly check framebuffer mmap offsets
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Greg Kroah-Hartman <[email protected]>
commit 3b82a4db8eaccce735dffd50b4d4e1578099b8e8 upstream.
The memmap options sent to the udl framebuffer driver were not being checked for all sets of possible crazy values. Fix this up by properly bounding the allowed values.
--- drivers/gpu/drm/udl/udl_fb.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)
drivers/gpu/drm/udl/udl_fb.c +7 -2
@@ -159,10 +159,15 @@ static int udl_fb_mmap(struct fb_info *i {unsigned long start = vma->vm_start;unsigned long size = vma->vm_end - vma->vm_start; -unsigned long offset = vma->vm_pgoff <<PAGE_SHIFT ; +unsigned long offset;unsigned long page, pos; -if (offset + size > info->fix.smem_len) +if (vma->vm_pgoff > (~0UL >>PAGE_SHIFT )) +return -EINVAL ; + + offset = vma->vm_pgoff <<PAGE_SHIFT ; + +if (offset > info->fix.smem_len || size > info->fix.smem_len - offset)return -EINVAL ; pos = (unsigned long )info->fix.smem_start + offset;
Greg Kroah-Hartman
maintainerPATCH 58/101 mm/vmscan: wake up flushers for legacy cgroups too
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andrey Ryabinin <[email protected]>
commit 1c610d5f93c709df56787f50b3576704ac271826 upstream.
Commit 726d061fbd36 ("mm: vmscan: kick flushers when we encounter dirty pages on the LRU") added flusher invocation to shrink_inactive_list() when many dirty pages on the LRU are encountered.
However, shrink_inactive_list() doesn't wake up flushers for legacy cgroup reclaim, so the next commit bbef938429f5 ("mm: vmscan: remove old flusher wakeup from direct reclaim path") removed the only source of flusher's wake up in legacy mem cgroup reclaim path.
This leads to premature OOM if there is too many dirty pages in cgroup:
# mkdir /sys/fs/cgroup/memory/test
# echo $$ > /sys/fs/cgroup/memory/test/tasks
# echo 50M > /sys/fs/cgroup/memory/test/memory.limit_in_bytes
# dd if=/dev/zero of=tmp_file bs=1M count=100
Killed dd invoked oom-killer: gfp_mask=0x14000c0(GFP_KERNEL), nodemask=(null), order=0, oom_score_adj=0
Call Trace:
dump_stack+0x46/0x65
dump_header+0x6b/0x2ac
oom_kill_process+0x21c/0x4a0
out_of_memory+0x2a5/0x4b0
mem_cgroup_out_of_memory+0x3b/0x60
mem_cgroup_oom_synchronize+0x2ed/0x330
pagefault_out_of_memory+0x24/0x54
__do_page_fault+0x521/0x540
page_fault+0x45/0x50
Task in /test killed as a result of limit of /test
memory: usage 51200kB, limit 51200kB, failcnt 73
memory+swap: usage 51200kB, limit 9007199254740988kB, failcnt 0
kmem: usage 296kB, limit 9007199254740988kB, failcnt 0
Memory cgroup stats for /test: cache:49632KB rss:1056KB rss_huge:0KB shmem:0KB
mapped_file:0KB dirty:49500KB writeback:0KB swap:0KB inactive_anon:0KB
active_anon:1168KB inactive_file:24760KB active_file:24960KB unevictable:0KB
Memory cgroup out of memory: Kill process 3861 (bash) score 88 or sacrifice child
Killed process 3876 (dd) total-vm:8484kB, anon-rss:1052kB, file-rss:1720kB, shmem-rss:0kB
oom_reaper: reaped process 3876 (dd), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kBWake up flushers in legacy cgroup reclaim too.
--- mm/vmscan.c | 31 ++++++++++++++++--------------- 1 file changed, 16 insertions(+), 15 deletions(-)
mm/vmscan.c +16 -15
@@ -1847,6 +1847,20 @@ shrink_inactive_list(unsigned long nr_toset_bit (PGDAT_WRITEBACK , &pgdat->flags);/* +* If dirty pages are scanned that are not queued for IO, it +* implies that flushers are not doing their job. This can +* happen when memory pressure pushes dirty pages to the end of +* the LRU before the dirty limits are breached and the dirty +* data has expired. It can also happen when the proportion of +* dirty pages grows not through writes but through memory +* pressure reclaiming all the clean cache. And in some cases, +* the flushers simply cannot keep up with the allocation +* rate. Nudge the flusher threads in case they are asleep. +*/ +if (stat.nr_unqueued_dirty == nr_taken) +wakeup_flusher_threads (0 ,WB_REASON_VMSCAN ); + +/* * Legacy memcg will stall in page writeback so avoid forcibly * stalling here. */
@@ -1858,22 +1872,9 @@ shrink_inactive_list(unsigned long nr_toif (stat.nr_dirty && stat.nr_dirty == stat.nr_congested)set_bit (PGDAT_CONGESTED , &pgdat->flags); -/* -* If dirty pages are scanned that are not queued for IO, it -* implies that flushers are not doing their job. This can -* happen when memory pressure pushes dirty pages to the end of -* the LRU before the dirty limits are breached and the dirty -* data has expired. It can also happen when the proportion of -* dirty pages grows not through writes but through memory -* pressure reclaiming all the clean cache. And in some cases, -* the flushers simply cannot keep up with the allocation -* rate. Nudge the flusher threads in case they are asleep, but -* also allow kswapd to start writing pages during reclaim. -*/ -if (stat.nr_unqueued_dirty == nr_taken) { -wakeup_flusher_threads (0 ,WB_REASON_VMSCAN ); +/* Allow kswapd to start writing pages during reclaim. */ +if (stat.nr_unqueued_dirty == nr_taken)set_bit (PGDAT_DIRTY , &pgdat->flags); - }/* * If kswapd scans pages marked marked for immediate
Greg Kroah-Hartman
maintainerPATCH 59/101 acpi, numa: fix pxm to online numa node associations
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dan Williams <[email protected]>
commit dc9e0a9347e932e3fd3cd03e7ff241022ed6ea8a upstream.
Commit 99759869faf1 "acpi: Add acpi_map_pxm_to_online_node()" added support for mapping a given proximity to its nearest, by SLIT distance, online node. However, it sometimes returns unexpected results due to the fact that it switches from comparing the PXM node to the last node that was closer than the current max.
for_each_online_node(n) {
dist = node_distance(node, n);
if (dist < min_dist) {
min_dist = dist;
node = n; <---- from this point we're using the
wrong node for node_distance()--- drivers/acpi/numa.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
drivers/acpi/numa.c +6 -4
@@ -103,25 +103,27 @@ int acpi_map_pxm_to_node(int pxm)*/ int acpi_map_pxm_to_online_node (int pxm) { -int node, n, dist, min_dist; +int node, min_node; node =acpi_map_pxm_to_node (pxm);if (node ==NUMA_NO_NODE ) node =0 ; + min_node = node;if (!node_online (node)) { - min_dist =INT_MAX ; +int min_dist =INT_MAX , dist, n; +for_each_online_node (n) { dist =node_distance (node, n);if (dist < min_dist) { min_dist = dist; - node = n; + min_node = n; } } } -return node; +return min_node; }EXPORT_SYMBOL (acpi_map_pxm_to_online_node);
Greg Kroah-Hartman
maintainerPATCH 60/101 ACPI / watchdog: Fix off-by-one error at resource assignment
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit b1abf6fc49829d89660c961fafe3f90f3d843c55 upstream.
The resource allocation in WDAT watchdog has off-one-by error, it sets one byte more than the actual end address. This may eventually lead to unexpected resource conflicts.
--- drivers/acpi/acpi_watchdog.c | 4 ++-- drivers/watchdog/wdat_wdt.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-)
drivers/acpi/acpi_watchdog.c +2 -2
@@ -74,10 +74,10 @@ void __init acpi_watchdog_init(void) res.start = gas->address;if (gas->space_id ==ACPI_ADR_SPACE_SYSTEM_MEMORY ) { res.flags =IORESOURCE_MEM ; - res.end = res.start +ALIGN (gas->access_width,4 ); + res.end = res.start +ALIGN (gas->access_width,4 ) -1 ; }else if (gas->space_id ==ACPI_ADR_SPACE_SYSTEM_IO ) { res.flags =IORESOURCE_IO ; - res.end = res.start + gas->access_width; + res.end = res.start + gas->access_width -1 ; }else {pr_warn ("Unsupported address space: %u\n" , gas->space_id);
drivers/watchdog/wdat_wdt.c +1 -1
@@ -392,7 +392,7 @@ static int wdat_wdt_probe(struct platformemset (&r,0 ,sizeof (r)); r.start = gas->address; - r.end = r.start + gas->access_width; + r.end = r.start + gas->access_width -1 ;if (gas->space_id ==ACPI_ADR_SPACE_SYSTEM_MEMORY ) { r.flags =IORESOURCE_MEM ; }else if (gas->space_id ==ACPI_ADR_SPACE_SYSTEM_IO ) {
Greg Kroah-Hartman
maintainerPATCH 61/101 libnvdimm, {btt, blk}: do integrity setup before add_disk()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Vishal Verma <[email protected]>
commit 3ffb0ba9b567a8efb9a04ed3d1ec15ff333ada22 upstream.
Prior to 25520d55cdb6 ("block: Inline blk_integrity in struct gendisk") we needed to temporarily add a zero-capacity disk before registering for blk-integrity. But adding a zero-capacity disk caused the partition table scanning to bail early, and this resulted in partitions not coming up after a probe of the BTT or blk namespaces.
We can now register for integrity before the disk has been added, and this fixes the rescan problems.
--- drivers/nvdimm/blk.c | 3 +-- drivers/nvdimm/btt.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-)
drivers/nvdimm/blk.c +1 -2
@@ -278,8 +278,6 @@ static int nsblk_attach_disk(struct nd_n disk->queue = q; disk->flags =GENHD_FL_EXT_DEVT ;nvdimm_namespace_disk_name (&nsblk->common, disk->disk_name); -set_capacity (disk,0 ); -device_add_disk (dev, disk);if (devm_add_action_or_reset (dev, nd_blk_release_disk, disk))return -ENOMEM ;
@@ -292,6 +290,7 @@ static int nsblk_attach_disk(struct nd_n }set_capacity (disk, available_disk_size >>SECTOR_SHIFT ); +device_add_disk (dev, disk);revalidate_disk (disk);return 0 ; }
drivers/nvdimm/btt.c +1 -2
@@ -1542,8 +1542,6 @@ static int btt_blk_init(struct btt *btt)queue_flag_set_unlocked (QUEUE_FLAG_NONROT , btt->btt_queue); btt->btt_queue->queuedata = btt; -set_capacity (btt->btt_disk,0 ); -device_add_disk (&btt->nd_btt->dev, btt->btt_disk);if (btt_meta_size (btt)) {int rc =nd_integrity_init (btt->btt_disk,btt_meta_size (btt));
@@ -1555,6 +1553,7 @@ static int btt_blk_init(struct btt *btt) } }set_capacity (btt->btt_disk, btt->nlba * btt->sector_size >>9 ); +device_add_disk (&btt->nd_btt->dev, btt->btt_disk); btt->nd_btt->size = btt->nlba * (u64 )btt->sector_size;revalidate_disk (btt->btt_disk);
Greg Kroah-Hartman
maintainerPATCH 62/101 brcmfmac: fix P2P_DEVICE ethernet address generation
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Arend Van Spriel <[email protected]>
commit 455f3e76cfc0d893585a5f358b9ddbe9c1e1e53b upstream.
The firmware has a requirement that the P2P_DEVICE address should be different from the address of the primary interface. When not specified by user-space, the driver generates the MAC address for the P2P_DEVICE interface using the MAC address of the primary interface and setting the locally administered bit. However, the MAC address of the primary interface may already have that bit set causing the creation of the P2P_DEVICE interface to fail with -EBUSY. Fix this by using a random address instead to determine the P2P_DEVICE address.
--- drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 24 +++++++---------- 1 file changed, 11 insertions(+), 13 deletions(-)
drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c +11 -13
@@ -462,25 +462,23 @@ static int brcmf_p2p_set_firmware(struct* @dev_addr: optional device address. * * P2P needs mac addresses for P2P device and interface. If no device -* address it specified, these are derived from the primary net device, ie. -* the permanent ethernet address of the device. +* address it specified, these are derived from a random ethernet +* address. */ static void brcmf_p2p_generate_bss_mac (struct brcmf_p2p_info *p2p,u8 *dev_addr) { -struct brcmf_if *pri_ifp = p2p->bss_idx[P2PAPI_BSSCFG_PRIMARY ].vif->ifp; -bool local_admin = false; +bool random_addr = false; -if (!dev_addr ||is_zero_ether_addr (dev_addr)) { - dev_addr = pri_ifp->mac_addr; - local_admin = true; - } +if (!dev_addr ||is_zero_ether_addr (dev_addr)) + random_addr = true; -/* Generate the P2P Device Address. This consists of the device's -* primary MAC address with the locally administered bit set. +/* Generate the P2P Device Address obtaining a random ethernet +* address with the locally administered bit set. */ -memcpy (p2p->dev_addr, dev_addr,ETH_ALEN ); -if (local_admin) - p2p->dev_addr[0 ] |=0x02 ; +if (random_addr) +eth_random_addr (p2p->dev_addr); +else +memcpy (p2p->dev_addr, dev_addr,ETH_ALEN );/* Generate the P2P Interface Address. If the discovery and connection * BSSCFGs need to simultaneously co-exist, then this address must be
Greg Kroah-Hartman
maintainerPATCH 63/101 rtlwifi: rtl8723be: Fix loss of signal
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Larry Finger <[email protected]>
commit 78dc897b7ee67205423dbbc6b56be49fb18d15b5 upstream.
In commit c713fb071edc ("rtlwifi: rtl8821ae: Fix connection lost problem correctly") a problem in rtl8821ae that caused loss of signal was fixed. That same problem has now been reported for rtl8723be. Accordingly, the ASPM L1 latency has been increased from 0 to 7 to fix the instability.
--- drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c +2 -1
@@ -1123,7 +1123,8 @@ static void _rtl8723be_enable_aspm_back_/* Configuration Space offset 0x70f BIT7 is used to control L0S */ tmp8 =_rtl8723be_dbi_read (rtlpriv,0x70f ); -_rtl8723be_dbi_write (rtlpriv,0x70f , tmp8 |BIT (7 )); +_rtl8723be_dbi_write (rtlpriv,0x70f , tmp8 |BIT (7 ) | +ASPM_L1_LATENCY <<3 );/* Configuration Space offset 0x719 Bit3 is for L1 * BIT4 is for clock request
Greg Kroah-Hartman
maintainerPATCH 64/101 tracing: probeevent: Fix to support minus offset from symbol
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Masami Hiramatsu <[email protected]>
commit c5d343b6b7badd1f5fe0873eff2e8d63a193e732 upstream.
In Documentation/trace/kprobetrace.txt, it says
@SYM[+|-offs] : Fetch memory at SYM +|- offs (SYM should be a data symbol)
However, the parser doesn't parse minus offset correctly, since commit 2fba0c8867af ("tracing/kprobes: Fix probe offset to be unsigned") drops minus ("-") offset support for kprobe probe address usage.
This fixes the traceprobe_split_symbol_offset() to parse minus offset again with checking the offset range, and add a minus offset check in kprobe probe address usage.
--- kernel/trace/trace_kprobe.c | 4 ++-- kernel/trace/trace_probe.c | 8 +++----- kernel/trace/trace_probe.h | 2 +- 3 files changed, 6 insertions(+), 8 deletions(-)
kernel/trace/trace_kprobe.c +2 -2
@@ -635,7 +635,7 @@ static int create_trace_kprobe(int argc,char *symbol =NULL , *event =NULL , *group =NULL ;int maxactive =0 ;char *arg; -unsigned long offset =0 ; +long offset =0 ;void *addr =NULL ;char buf[MAX_EVENT_NAME_LEN ];
@@ -723,7 +723,7 @@ static int create_trace_kprobe(int argc, symbol = argv[1 ];/* TODO: support .init module functions */ ret =traceprobe_split_symbol_offset (symbol, &offset); -if (ret) { +if (ret || offset <0 || offset >UINT_MAX ) {pr_info ("Failed to parse either an address or a symbol.\n" );return ret; }
kernel/trace/trace_probe.c +3 -5
@@ -320,7 +320,7 @@ static fetch_func_t get_fetch_size_funct }/* Split symbol and offset. */ -int traceprobe_split_symbol_offset (char *symbol,unsigned long *offset) +int traceprobe_split_symbol_offset (char *symbol,long *offset) {char *tmp;int ret;
@@ -328,13 +328,11 @@ int traceprobe_split_symbol_offset(charif (!offset)return -EINVAL ; - tmp =strchr (symbol,'+' ); + tmp =strpbrk (symbol,"+-" );if (tmp) { -/* skip sign because kstrtoul doesn't accept '+' */ - ret =kstrtoul (tmp +1 ,0 , offset); + ret =kstrtol (tmp,0 , offset);if (ret)return ret; - *tmp ='\0' ; }else *offset =0 ;
kernel/trace/trace_probe.h +1 -1
@@ -354,7 +354,7 @@ extern int traceprobe_conflict_field_namextern void traceprobe_update_arg (struct probe_arg *arg);extern void traceprobe_free_probe_arg (struct probe_arg *arg); -extern int traceprobe_split_symbol_offset (char *symbol,unsigned long *offset); +extern int traceprobe_split_symbol_offset (char *symbol,long *offset); extern ssize_t traceprobe_probes_write(struct file *file, const char __user *buffer, size_t count, loff_t *ppos,
Greg Kroah-Hartman
maintainerPATCH 65/101 mtdchar: fix usage of mtd_ooblayout_ecc()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: OuYang ZhiZhong <[email protected]>
commit 6de564939e14327148e31ddcf769e34105176447 upstream.
Section was not properly computed. The value of OOB region definition is always ECC section 0 information in the OOB area, but we want to get all the ECC bytes information, so we should call mtd_ooblayout_ecc(mtd, section++, &oobregion) until it returns -ERANGE.
--- drivers/mtd/mtdchar.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/mtd/mtdchar.c +2 -2
@@ -487,7 +487,7 @@ static int shrink_ecclayout(struct mtd_ifor (i =0 ; i <MTD_MAX_ECCPOS_ENTRIES ;) {u32 eccpos; - ret =mtd_ooblayout_ecc (mtd, section, &oobregion); + ret =mtd_ooblayout_ecc (mtd, section++, &oobregion);if (ret <0 ) {if (ret != -ERANGE )return ret;
@@ -534,7 +534,7 @@ static int get_oobinfo(struct mtd_info *for (i =0 ; i <ARRAY_SIZE (to->eccpos);) {u32 eccpos; - ret =mtd_ooblayout_ecc (mtd, section, &oobregion); + ret =mtd_ooblayout_ecc (mtd, section++, &oobregion);if (ret <0 ) {if (ret != -ERANGE )return ret;
Greg Kroah-Hartman
maintainerPATCH 66/101 mtd: nand: fsl_ifc: Fix nand waitfunc return value
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jagdish Gediya <[email protected]>
commit fa8e6d58c5bc260f4369c6699683d69695daed0a upstream.
As per the IFC hardware manual, Most significant 2 bytes in nand_fsr register are the outcome of NAND READ STATUS command.
So status value need to be shifted and aligned as per the nand framework requirement.
--- drivers/mtd/nand/fsl_ifc_nand.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
drivers/mtd/nand/fsl_ifc_nand.c +3 -2
@@ -626,6 +626,7 @@ static int fsl_ifc_wait(struct mtd_infostruct fsl_ifc_ctrl *ctrl = priv->ctrl;struct fsl_ifc_runtime __iomem *ifc = ctrl->rregs;u32 nand_fsr; +int status;/* Use READ_STATUS command, but wait for the device to be ready */ ifc_out32((IFC_FIR_OP_CW0 << IFC_NAND_FIR0_OP0_SHIFT) |
@@ -640,12 +641,12 @@ static int fsl_ifc_wait(struct mtd_infofsl_ifc_run_command (mtd); nand_fsr =ifc_in32 (&ifc->ifc_nand.nand_fsr); - + status = nand_fsr >>24 ;/* * The chip always seems to report that it is * write-protected, even when it is not. */ -return nand_fsr |NAND_STATUS_WP ; +return status |NAND_STATUS_WP ; }/*
Greg Kroah-Hartman
maintainerPATCH 67/101 mtd: nand: fsl_ifc: Fix eccstat array overflow for IFC ver >= 2.0.0
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jagdish Gediya <[email protected]>
commit 843c3a59997f18060848b8632607dd04781b52d1 upstream.
Number of ECC status registers i.e. (ECCSTATx) has been increased in IFC version 2.0.0 due to increase in SRAM size. This is causing eccstat array to over flow.
So, replace eccstat array with u32 variable to make it fail-safe and independent of number of ECC status registers or SRAM size.
--- drivers/mtd/nand/fsl_ifc_nand.c | 23 ++++++++++------------- 1 file changed, 10 insertions(+), 13 deletions(-)
drivers/mtd/nand/fsl_ifc_nand.c +10 -13
@@ -173,14 +173,9 @@ static void set_addr(struct mtd_info *mt/* returns nonzero if entire page is blank */ static int check_read_ecc (struct mtd_info *mtd,struct fsl_ifc_ctrl *ctrl, -u32 *eccstat,unsigned int bufnum) +u32 eccstat,unsigned int bufnum) { -u32 reg = eccstat[bufnum /4 ]; -int errors; - - errors = (reg >> ((3 - bufnum %4 ) *8 )) &15 ; - -return errors; +return (eccstat >> ((3 - bufnum %4 ) *8 )) &15 ; }/*
@@ -193,7 +188,7 @@ static void fsl_ifc_run_command(struct mstruct fsl_ifc_ctrl *ctrl = priv->ctrl;struct fsl_ifc_nand_ctrl *nctrl = ifc_nand_ctrl;struct fsl_ifc_runtime __iomem *ifc = ctrl->rregs; -u32 eccstat[4 ]; +u32 eccstat;int i;/* set the chip select for NAND Transaction */
@@ -228,8 +223,8 @@ static void fsl_ifc_run_command(struct mif (nctrl->eccread) {int errors;int bufnum = nctrl->page & priv->bufnum_mask; -int sector = bufnum * chip->ecc.steps; -int sector_end = sector + chip->ecc.steps -1 ; +int sector_start = bufnum * chip->ecc.steps; +int sector_end = sector_start + chip->ecc.steps -1 ;__be32 *eccstat_regs;if (ctrl->version >=FSL_IFC_VERSION_2_0_0 )
@@ -237,10 +232,12 @@ static void fsl_ifc_run_command(struct melse eccstat_regs = ifc->ifc_nand.v1_nand_eccstat; -for (i = sector /4 ; i <= sector_end /4 ; i++) - eccstat[i] =ifc_in32 (&eccstat_regs[i]); + eccstat =ifc_in32 (&eccstat_regs[sector_start /4 ]); + +for (i = sector_start; i <= sector_end; i++) { +if (i != sector_start && !(i %4 )) + eccstat =ifc_in32 (&eccstat_regs[i /4 ]); -for (i = sector; i <= sector_end; i++) { errors =check_read_ecc (mtd, ctrl, eccstat, i);if (errors ==15 ) {
Greg Kroah-Hartman
maintainerPATCH 68/101 mtd: nand: fsl_ifc: Read ECCSTAT0 and ECCSTAT1 registers for IFC 2.0
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jagdish Gediya <[email protected]>
commit 6b00c35138b404be98b85f4a703be594cbed501c upstream.
Due to missing information in Hardware manual, current implementation doesn't read ECCSTAT0 and ECCSTAT1 registers for IFC 2.0.
Add support to read ECCSTAT0 and ECCSTAT1 registers during ecccheck for IFC 2.0.
--- drivers/mtd/nand/fsl_ifc_nand.c | 6 +----- include/linux/fsl_ifc.h | 6 +----- 2 files changed, 2 insertions(+), 10 deletions(-)
drivers/mtd/nand/fsl_ifc_nand.c +1 -5
@@ -227,11 +227,7 @@ static void fsl_ifc_run_command(struct mint sector_end = sector_start + chip->ecc.steps -1 ;__be32 *eccstat_regs; -if (ctrl->version >=FSL_IFC_VERSION_2_0_0 ) - eccstat_regs = ifc->ifc_nand.v2_nand_eccstat; -else - eccstat_regs = ifc->ifc_nand.v1_nand_eccstat; - + eccstat_regs = ifc->ifc_nand.nand_eccstat; eccstat =ifc_in32 (&eccstat_regs[sector_start /4 ]);for (i = sector_start; i <= sector_end; i++) {
include/linux/fsl_ifc.h +1 -5
@@ -734,11 +734,7 @@ struct fsl_ifc_nand {u32 res19[0x10 ];__be32 nand_fsr;u32 res20; -/* The V1 nand_eccstat is actually 4 words that overlaps the -* V2 nand_eccstat. -*/ -__be32 v1_nand_eccstat[2 ]; -__be32 v2_nand_eccstat[6 ]; +__be32 nand_eccstat[8 ];u32 res21[0x1c ];__be32 nanndcr;u32 res22[0x2 ];
Greg Kroah-Hartman
maintainerPATCH 69/101 staging: ncpfs: memory corruption in ncp_read_kernel()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dan Carpenter <[email protected]>
commit 4c41aa24baa4ed338241d05494f2c595c885af8f upstream.
If the server is malicious then *bytes_read could be larger than the size of the "target" buffer. It would lead to memory corruption when we do the memcpy().
--- fs/ncpfs/ncplib_kernel.c | 4 ++++ 1 file changed, 4 insertions(+)
fs/ncpfs/ncplib_kernel.c +4 -0
@@ -981,6 +981,10 @@ ncp_read_kernel(struct ncp_server *serve goto out; } *bytes_read =ncp_reply_be16 (server,0 ); +if (*bytes_read > to_read) { + result = -EINVAL ; + goto out; + } source =ncp_reply_data (server,2 + (offset &1 ));memcpy (target, source, *bytes_read);
Greg Kroah-Hartman
maintainerPATCH 70/101 can: peak/pcie_fd: fix echo_skb is occupied! bug
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Stephane Grosjean <[email protected]>
commit e6048a00cfd0863d32f53b226e0b9a3633fc3332 upstream.
This patch makes atomic the handling of the linux-can echo_skb array and the network tx queue. This prevents from the "BUG! echo_skb is occupied!" message to be printed by the linux-can core, in SMP environments.
--- drivers/net/can/peak_canfd/peak_canfd.c | 12 ++++++------ drivers/net/can/peak_canfd/peak_pciefd_main.c | 8 ++++++-- 2 files changed, 12 insertions(+), 8 deletions(-)
drivers/net/can/peak_canfd/peak_canfd.c +6 -6
@@ -262,7 +262,6 @@ static int pucan_handle_can_rx(struct pespin_lock_irqsave (&priv->echo_lock, flags);can_get_echo_skb (priv->ndev, msg->client); -spin_unlock_irqrestore (&priv->echo_lock, flags);/* count bytes of the echo instead of skb */ stats->tx_bytes += cf_len;
@@ -271,6 +270,7 @@ static int pucan_handle_can_rx(struct pe/* restart tx queue (a slot is free) */ netif_wake_queue (priv->ndev); +spin_unlock_irqrestore (&priv->echo_lock, flags);return 0 ; }
@@ -726,11 +726,6 @@ static netdev_tx_t peak_canfd_start_xmit*/ should_stop_tx_queue = !!(priv->can.echo_skb[priv->echo_idx]); -spin_unlock_irqrestore (&priv->echo_lock, flags); - -/* write the skb on the interface */ - priv->write_tx_msg (priv, msg); -/* stop network tx queue if not enough room to save one more msg too */ if (priv->can.ctrlmode &CAN_CTRLMODE_FD ) should_stop_tx_queue |= (room_left <
@@ -742,6 +737,11 @@ static netdev_tx_t peak_canfd_start_xmitif (should_stop_tx_queue)netif_stop_queue (ndev); +spin_unlock_irqrestore (&priv->echo_lock, flags); + +/* write the skb on the interface */ + priv->write_tx_msg (priv, msg); +return NETDEV_TX_OK ; }
drivers/net/can/peak_canfd/peak_pciefd_main.c +6 -2
@@ -349,8 +349,12 @@ static irqreturn_t pciefd_irq_handler(in priv->tx_pages_free++;spin_unlock_irqrestore (&priv->tx_lock, flags); -/* wake producer up */ -netif_wake_queue (priv->ucan.ndev); +/* wake producer up (only if enough room in echo_skb array) */ +spin_lock_irqsave (&priv->ucan.echo_lock, flags); +if (!priv->ucan.can.echo_skb[priv->ucan.echo_idx]) +netif_wake_queue (priv->ucan.ndev); + +spin_unlock_irqrestore (&priv->ucan.echo_lock, flags); }/* re-enable Rx DMA transfer for this CAN */
Greg Kroah-Hartman
maintainerPATCH 71/101 can: peak/pcie_fd: remove useless code when interface starts
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Stephane Grosjean <[email protected]>
commit ffd137f7043cb30067e1bff6fe62a073ae190b23 upstream.
When an interface starts, the echo_skb array is empty and the network queue should be started only. This patch replaces useless code and locks when the internal RX_BARRIER message is received from the IP core, telling the driver that tx may start.
--- drivers/net/can/peak_canfd/peak_canfd.c | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-)
drivers/net/can/peak_canfd/peak_canfd.c +2 -11
@@ -333,7 +333,6 @@ static int pucan_handle_status(struct pe/* this STATUS is the CNF of the RX_BARRIER: Tx path can be setup */ if (pucan_status_is_rx_barrier (msg)) { -unsigned long flags;if (priv->enable_tx_path) {int err = priv->enable_tx_path (priv);
@@ -342,16 +341,8 @@ static int pucan_handle_status(struct pereturn err; } -/* restart network queue only if echo skb array is free */ -spin_lock_irqsave (&priv->echo_lock, flags); - -if (!priv->can.echo_skb[priv->echo_idx]) { -spin_unlock_irqrestore (&priv->echo_lock, flags); - -netif_wake_queue (ndev); - }else { -spin_unlock_irqrestore (&priv->echo_lock, flags); - } +/* start network queue (echo_skb array is empty) */ +netif_start_queue (ndev);return 0 ; }
Greg Kroah-Hartman
maintainerPATCH 72/101 can: ifi: Repair the error handling
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Marek Vasut <[email protected]>
commit 880dd464b4304583c557c4e5f5ecebfd55d232b1 upstream.
The new version of the IFI CANFD core has significantly less complex error state indication logic. In particular, the warning/error state bits are no longer all over the place, but are all present in the STATUS register. Moreover, there is a new IRQ register bit indicating transition between error states (active/warning/passive/busoff).
This patch makes use of this bit to weed out the obscure selective INTERRUPT register clearing, which was used to carry over the error state indication into the poll function. While at it, this patch fixes the handling of the ACTIVE state, since the hardware provides indication of the core being in ACTIVE state and that in turn fixes the state transition indication toward userspace. Finally, register reads in the poll function are moved to the matching subfunctions since those are also no longer needed in the poll function.
--- drivers/net/can/ifi_canfd/ifi_canfd.c | 64 +++++++++++++++++++--------------- 1 file changed, 37 insertions(+), 27 deletions(-)
drivers/net/can/ifi_canfd/ifi_canfd.c +37 -27
@@ -30,6 +30,7 @@#define IFI_CANFD_STCMD_ERROR_ACTIVE BIT(2)#define IFI_CANFD_STCMD_ERROR_PASSIVE BIT(3)#define IFI_CANFD_STCMD_BUSOFF BIT(4) +#define IFI_CANFD_STCMD_ERROR_WARNING BIT(5)#define IFI_CANFD_STCMD_BUSMONITOR BIT(16)#define IFI_CANFD_STCMD_LOOPBACK BIT(18)#define IFI_CANFD_STCMD_DISABLE_CANFD BIT(24)
@@ -52,7 +53,10 @@#define IFI_CANFD_TXSTCMD_OVERFLOW BIT(13)#define IFI_CANFD_INTERRUPT 0xc +#define IFI_CANFD_INTERRUPT_ERROR_BUSOFF BIT(0)#define IFI_CANFD_INTERRUPT_ERROR_WARNING BIT(1) +#define IFI_CANFD_INTERRUPT_ERROR_STATE_CHG BIT(2) +#define IFI_CANFD_INTERRUPT_ERROR_REC_TEC_INC BIT(3)#define IFI_CANFD_INTERRUPT_ERROR_COUNTER BIT(10)#define IFI_CANFD_INTERRUPT_TXFIFO_EMPTY BIT(16)#define IFI_CANFD_INTERRUPT_TXFIFO_REMOVE BIT(22)
@@ -61,6 +65,10 @@#define IFI_CANFD_INTERRUPT_SET_IRQ ((u32)BIT(31))#define IFI_CANFD_IRQMASK 0x10 +#define IFI_CANFD_IRQMASK_ERROR_BUSOFF BIT(0) +#define IFI_CANFD_IRQMASK_ERROR_WARNING BIT(1) +#define IFI_CANFD_IRQMASK_ERROR_STATE_CHG BIT(2) +#define IFI_CANFD_IRQMASK_ERROR_REC_TEC_INC BIT(3)#define IFI_CANFD_IRQMASK_SET_ERR BIT(7)#define IFI_CANFD_IRQMASK_SET_TS BIT(15)#define IFI_CANFD_IRQMASK_TXFIFO_EMPTY BIT(16)
@@ -220,7 +228,10 @@ static void ifi_canfd_irq_enable(structif (enable) { enirq =IFI_CANFD_IRQMASK_TXFIFO_EMPTY | -IFI_CANFD_IRQMASK_RXFIFO_NEMPTY ; +IFI_CANFD_IRQMASK_RXFIFO_NEMPTY | +IFI_CANFD_IRQMASK_ERROR_STATE_CHG | +IFI_CANFD_IRQMASK_ERROR_WARNING | +IFI_CANFD_IRQMASK_ERROR_BUSOFF ;if (priv->can.ctrlmode &CAN_CTRLMODE_BERR_REPORTING ) enirq |=IFI_CANFD_INTERRUPT_ERROR_COUNTER ; }
@@ -361,12 +372,13 @@ static int ifi_canfd_handle_lost_msg(strreturn 1 ; } -static int ifi_canfd_handle_lec_err (struct net_device *ndev,const u32 errctr) +static int ifi_canfd_handle_lec_err (struct net_device *ndev) {struct ifi_canfd_priv *priv =netdev_priv (ndev);struct net_device_stats *stats = &ndev->stats;struct can_frame *cf;struct sk_buff *skb; +u32 errctr =readl (priv->base +IFI_CANFD_ERROR_CTR );const u32 errmask =IFI_CANFD_ERROR_CTR_OVERLOAD_FIRST |IFI_CANFD_ERROR_CTR_ACK_ERROR_FIRST | IFI_CANFD_ERROR_CTR_BIT0_ERROR_FIRST |
@@ -449,6 +461,11 @@ static int ifi_canfd_handle_state_changeswitch (new_state) {case CAN_STATE_ERROR_ACTIVE : +/* error active state */ + priv->can.can_stats.error_warning++; + priv->can.state =CAN_STATE_ERROR_ACTIVE ; +break ; +case CAN_STATE_ERROR_WARNING :/* error warning state */ priv->can.can_stats.error_warning++; priv->can.state =CAN_STATE_ERROR_WARNING ;
@@ -477,7 +494,7 @@ static int ifi_canfd_handle_state_changeifi_canfd_get_berr_counter (ndev, &bec);switch (new_state) { -case CAN_STATE_ERROR_ACTIVE : +case CAN_STATE_ERROR_WARNING :/* error warning state */ cf->can_id |=CAN_ERR_CRTL ; cf->data[1] = (bec.txerr > bec.rxerr) ?
@@ -510,22 +527,21 @@ static int ifi_canfd_handle_state_changereturn 1 ; } -static int ifi_canfd_handle_state_errors (struct net_device *ndev,u32 stcmd) +static int ifi_canfd_handle_state_errors (struct net_device *ndev) {struct ifi_canfd_priv *priv =netdev_priv (ndev); +u32 stcmd =readl (priv->base +IFI_CANFD_STCMD );int work_done =0 ; -u32 isr; -/* -* The ErrWarn condition is a little special, since the bit is -* located in the INTERRUPT register instead of STCMD register. -*/ - isr =readl (priv->base +IFI_CANFD_INTERRUPT ); -if ((isr &IFI_CANFD_INTERRUPT_ERROR_WARNING ) && +if ((stcmd &IFI_CANFD_STCMD_ERROR_ACTIVE ) && + (priv->can.state !=CAN_STATE_ERROR_ACTIVE )) { +netdev_dbg (ndev,"Error, entered active state\n" ); + work_done +=ifi_canfd_handle_state_change (ndev, +CAN_STATE_ERROR_ACTIVE ); + } + +if ((stcmd &IFI_CANFD_STCMD_ERROR_WARNING ) && (priv->can.state !=CAN_STATE_ERROR_WARNING )) { -/* Clear the interrupt */ -writel (IFI_CANFD_INTERRUPT_ERROR_WARNING , - priv->base +IFI_CANFD_INTERRUPT );netdev_dbg (ndev,"Error, entered warning state\n" ); work_done +=ifi_canfd_handle_state_change (ndev,CAN_STATE_ERROR_WARNING );
@@ -552,18 +568,11 @@ static int ifi_canfd_poll(struct napi_st {struct net_device *ndev = napi->dev;struct ifi_canfd_priv *priv =netdev_priv (ndev); -const u32 stcmd_state_mask =IFI_CANFD_STCMD_ERROR_PASSIVE | -IFI_CANFD_STCMD_BUSOFF ; -int work_done =0 ; - -u32 stcmd =readl (priv->base +IFI_CANFD_STCMD );u32 rxstcmd =readl (priv->base +IFI_CANFD_RXSTCMD ); -u32 errctr =readl (priv->base +IFI_CANFD_ERROR_CTR ); +int work_done =0 ;/* Handle bus state changes */ -if ((stcmd & stcmd_state_mask) || - ((stcmd &IFI_CANFD_STCMD_ERROR_ACTIVE ) ==0 )) - work_done +=ifi_canfd_handle_state_errors (ndev, stcmd); + work_done +=ifi_canfd_handle_state_errors (ndev);/* Handle lost messages on RX */ if (rxstcmd &IFI_CANFD_RXSTCMD_OVERFLOW )
@@ -571,7 +580,7 @@ static int ifi_canfd_poll(struct napi_st/* Handle lec errors on the bus */ if (priv->can.ctrlmode &CAN_CTRLMODE_BERR_REPORTING ) - work_done +=ifi_canfd_handle_lec_err (ndev, errctr); + work_done +=ifi_canfd_handle_lec_err (ndev);/* Handle normal messages on RX */ if (!(rxstcmd &IFI_CANFD_RXSTCMD_EMPTY ))
@@ -592,12 +601,13 @@ static irqreturn_t ifi_canfd_isr(int irqstruct net_device_stats *stats = &ndev->stats;const u32 rx_irq_mask =IFI_CANFD_INTERRUPT_RXFIFO_NEMPTY |IFI_CANFD_INTERRUPT_RXFIFO_NEMPTY_PER | +IFI_CANFD_INTERRUPT_ERROR_COUNTER | +IFI_CANFD_INTERRUPT_ERROR_STATE_CHG |IFI_CANFD_INTERRUPT_ERROR_WARNING | -IFI_CANFD_INTERRUPT_ERROR_COUNTER ; +IFI_CANFD_INTERRUPT_ERROR_BUSOFF ;const u32 tx_irq_mask =IFI_CANFD_INTERRUPT_TXFIFO_EMPTY |IFI_CANFD_INTERRUPT_TXFIFO_REMOVE ; -const u32 clr_irq_mask = ~((u32)(IFI_CANFD_INTERRUPT_SET_IRQ | -IFI_CANFD_INTERRUPT_ERROR_WARNING )); +const u32 clr_irq_mask = ~((u32 )IFI_CANFD_INTERRUPT_SET_IRQ );u32 isr; isr =readl (priv->base +IFI_CANFD_INTERRUPT );
Greg Kroah-Hartman
maintainerPATCH 73/101 can: ifi: Check core revision upon probe
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Marek Vasut <[email protected]>
commit 591d65d5b15496af8d05e252bc1da611c66c0b79 upstream.
Older versions of the core are not compatible with the driver due to various intrusive fixes of the core. Read out the VER register, check the core revision bitfield and verify if the core in use is new enough (rev 2.1 or newer) to work correctly with this driver.
--- drivers/net/can/ifi_canfd/ifi_canfd.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)
drivers/net/can/ifi_canfd/ifi_canfd.c +10 -1
@@ -144,6 +144,8 @@#define IFI_CANFD_SYSCLOCK 0x50#define IFI_CANFD_VER 0x54 +#define IFI_CANFD_VER_REV_MASK 0xff +#define IFI_CANFD_VER_REV_MIN_SUPPORTED 0x15#define IFI_CANFD_IP_ID 0x58#define IFI_CANFD_IP_ID_VALUE 0xD073CAFD
@@ -943,7 +945,7 @@ static int ifi_canfd_plat_probe(struct pstruct resource *res;void __iomem *addr;int irq, ret; -u32 id; +u32 id, rev; res =platform_get_resource (pdev,IORESOURCE_MEM ,0 ); addr =devm_ioremap_resource (dev, res);
@@ -957,6 +959,13 @@ static int ifi_canfd_plat_probe(struct preturn -EINVAL ; } + rev =readl (addr +IFI_CANFD_VER ) &IFI_CANFD_VER_REV_MASK ; +if (rev <IFI_CANFD_VER_REV_MIN_SUPPORTED ) { +dev_err (dev,"This block is too old (rev %i), minimum supported is rev %i\n" , + rev,IFI_CANFD_VER_REV_MIN_SUPPORTED ); +return -EINVAL ; + } + ndev =alloc_candev (sizeof (*priv),1 );if (!ndev)return -ENOMEM ;
Greg Kroah-Hartman
maintainerPATCH 74/101 can: cc770: Fix stalls on rt-linux, remove redundant IRQ ack
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andri Yngvason <[email protected]>
commit f4353daf4905c0099fd25fa742e2ffd4a4bab26a upstream.
This has been reported to cause stalls on rt-linux.
--- drivers/net/can/cc770/cc770.c | 15 --------------- 1 file changed, 15 deletions(-)
drivers/net/can/cc770/cc770.c +0 -15
@@ -447,15 +447,6 @@ static netdev_tx_t cc770_start_xmit(stru stats->tx_bytes += dlc; - -/* -* HM: We had some cases of repeated IRQs so make sure the -* INT is acknowledged I know it's already further up, but -* doing again fixed the issue -*/ -cc770_write_reg (priv, msgobj[mo].ctrl0, -MSGVAL_UNC |TXIE_UNC |RXIE_UNC |INTPND_RES ); -return NETDEV_TX_OK ; }
@@ -684,12 +675,6 @@ static void cc770_tx_interrupt(struct ne/* Nothing more to send, switch off interrupts */ cc770_write_reg (priv, msgobj[mo].ctrl0,MSGVAL_RES |TXIE_RES |RXIE_RES |INTPND_RES ); -/* -* We had some cases of repeated IRQ so make sure the -* INT is acknowledged -*/ -cc770_write_reg (priv, msgobj[mo].ctrl0, -MSGVAL_UNC |TXIE_UNC |RXIE_UNC |INTPND_RES ); stats->tx_packets++;can_get_echo_skb (dev,0 );
Greg Kroah-Hartman
maintainerPATCH 75/101 can: cc770: Fix queue stall & dropped RTR reply
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andri Yngvason <[email protected]>
commit 746201235b3f876792099079f4c6fea941d76183 upstream.
While waiting for the TX object to send an RTR, an external message with a matching id can overwrite the TX data. In this case we must call the rx routine and then try transmitting the message that was overwritten again.
The queue was being stalled because the RX event did not generate an interrupt to wake up the queue again and the TX event did not happen because the TXRQST flag is reset by the chip when new data is received.
According to the CC770 datasheet the id of a message object should not be changed while the MSGVAL bit is set. This has been fixed by resetting the MSGVAL bit before modifying the object in the transmit function and setting it after. It is not enough to set & reset CPUUPD.
It is important to keep the MSGVAL bit reset while the message object is being modified. Otherwise, during RTR transmission, a frame with matching id could trigger an rx-interrupt, which would cause a race condition between the interrupt routine and the transmit function.
--- drivers/net/can/cc770/cc770.c | 94 +++++++++++++++++++++++++++++------------- drivers/net/can/cc770/cc770.h | 2 2 files changed, 68 insertions(+), 28 deletions(-)
drivers/net/can/cc770/cc770.c +66 -28
@@ -390,37 +390,23 @@ static int cc770_get_berr_counter(constreturn 0 ; } -static netdev_tx_t cc770_start_xmit (struct sk_buff *skb,struct net_device *dev) +static void cc770_tx (struct net_device *dev,int mo) {struct cc770_priv *priv =netdev_priv (dev); -struct net_device_stats *stats = &dev->stats; -struct can_frame *cf = (struct can_frame *)skb->data; -unsigned int mo =obj2msgobj (CC770_OBJ_TX ); +struct can_frame *cf = (struct can_frame *)priv->tx_skb->data;u8 dlc, rtr;u32 id;int i; -if (can_dropped_invalid_skb (dev, skb)) -return NETDEV_TX_OK ; - -if ((cc770_read_reg (priv, - msgobj[mo].ctrl1) &TXRQST_UNC ) ==TXRQST_SET ) { -netdev_err (dev,"TX register is still occupied!\n" ); -return NETDEV_TX_BUSY ; - } - -netif_stop_queue (dev); - dlc = cf->can_dlc; id = cf->can_id; -if (cf->can_id &CAN_RTR_FLAG ) - rtr =0 ; -else - rtr =MSGCFG_DIR ; + rtr = cf->can_id &CAN_RTR_FLAG ?0 :MSGCFG_DIR ; + +cc770_write_reg (priv, msgobj[mo].ctrl0, +MSGVAL_RES |TXIE_RES |RXIE_RES |INTPND_RES );cc770_write_reg (priv, msgobj[mo].ctrl1,RMTPND_RES |TXRQST_RES |CPUUPD_SET |NEWDAT_RES ); -cc770_write_reg (priv, msgobj[mo].ctrl0, -MSGVAL_SET |TXIE_SET |RXIE_RES |INTPND_RES ); +if (id &CAN_EFF_FLAG ) { id &=CAN_EFF_MASK ; cc770_write_reg(priv, msgobj[mo].config,
@@ -439,13 +425,30 @@ static netdev_tx_t cc770_start_xmit(strufor (i =0 ; i < dlc; i++)cc770_write_reg (priv, msgobj[mo].data[i], cf->data[i]); -/* Store echo skb before starting the transfer */ -can_put_echo_skb (skb, dev,0 ); -cc770_write_reg (priv, msgobj[mo].ctrl1, -RMTPND_RES |TXRQST_SET |CPUUPD_RES |NEWDAT_UNC ); +RMTPND_UNC |TXRQST_SET |CPUUPD_RES |NEWDAT_UNC ); +cc770_write_reg (priv, msgobj[mo].ctrl0, +MSGVAL_SET |TXIE_SET |RXIE_SET |INTPND_UNC ); +} + +static netdev_tx_t cc770_start_xmit (struct sk_buff *skb,struct net_device *dev) +{ +struct cc770_priv *priv =netdev_priv (dev); +unsigned int mo =obj2msgobj (CC770_OBJ_TX ); + +if (can_dropped_invalid_skb (dev, skb)) +return NETDEV_TX_OK ; + +netif_stop_queue (dev); + +if ((cc770_read_reg (priv, + msgobj[mo].ctrl1) &TXRQST_UNC ) ==TXRQST_SET ) { +netdev_err (dev,"TX register is still occupied!\n" ); +return NETDEV_TX_BUSY ; + } - stats->tx_bytes += dlc; + priv->tx_skb = skb; +cc770_tx (dev, mo);return NETDEV_TX_OK ; }
@@ -671,13 +674,47 @@ static void cc770_tx_interrupt(struct nestruct cc770_priv *priv =netdev_priv (dev);struct net_device_stats *stats = &dev->stats;unsigned int mo =obj2msgobj (o); +struct can_frame *cf; +u8 ctrl1; + + ctrl1 =cc770_read_reg (priv, msgobj[mo].ctrl1); -/* Nothing more to send, switch off interrupts */ cc770_write_reg (priv, msgobj[mo].ctrl0,MSGVAL_RES |TXIE_RES |RXIE_RES |INTPND_RES ); +cc770_write_reg (priv, msgobj[mo].ctrl1, +RMTPND_RES |TXRQST_RES |MSGLST_RES |NEWDAT_RES ); - stats->tx_packets++; +if (unlikely (!priv->tx_skb)) { +netdev_err (dev,"missing tx skb in tx interrupt\n" ); +return ; + } + +if (unlikely (ctrl1 &MSGLST_SET )) { + stats->rx_over_errors++; + stats->rx_errors++; + } + +/* When the CC770 is sending an RTR message and it receives a regular +* message that matches the id of the RTR message, it will overwrite the +* outgoing message in the TX register. When this happens we must +* process the received message and try to transmit the outgoing skb +* again. +*/ +if (unlikely (ctrl1 &NEWDAT_SET )) { +cc770_rx (dev, mo, ctrl1); +cc770_tx (dev, mo); +return ; + } + +can_put_echo_skb (priv->tx_skb, dev,0 );can_get_echo_skb (dev,0 ); + + cf = (struct can_frame *)priv->tx_skb->data; + stats->tx_bytes += cf->can_dlc; + stats->tx_packets++; + + priv->tx_skb =NULL ; +netif_wake_queue (dev); }
@@ -789,6 +826,7 @@ struct net_device *alloc_cc770dev(int si priv->can.do_set_bittiming = cc770_set_bittiming; priv->can.do_set_mode = cc770_set_mode; priv->can.ctrlmode_supported =CAN_CTRLMODE_3_SAMPLES ; + priv->tx_skb =NULL ;memcpy (priv->obj_flags, cc770_obj_flags,sizeof (cc770_obj_flags));
drivers/net/can/cc770/cc770.h +2 -0
@@ -193,6 +193,8 @@ struct cc770_priv {u8 cpu_interface;/* CPU interface register */ u8 clkout;/* Clock out register */ u8 bus_config;/* Bus conffiguration register */ + +struct sk_buff *tx_skb; };struct net_device *alloc_cc770dev (int sizeof_priv);
Greg Kroah-Hartman
maintainerPATCH 76/101 can: cc770: Fix use after free in cc770_tx_interrupt()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andri Yngvason <[email protected]>
commit 9ffd7503944ec7c0ef41c3245d1306c221aef2be upstream.
This fixes use after free introduced by the last cc770 patch.
--- drivers/net/can/cc770/cc770.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
drivers/net/can/cc770/cc770.c +2 -3
@@ -706,13 +706,12 @@ static void cc770_tx_interrupt(struct nereturn ; } -can_put_echo_skb (priv->tx_skb, dev,0 ); -can_get_echo_skb (dev,0 ); - cf = (struct can_frame *)priv->tx_skb->data; stats->tx_bytes += cf->can_dlc; stats->tx_packets++; +can_put_echo_skb (priv->tx_skb, dev,0 ); +can_get_echo_skb (dev,0 ); priv->tx_skb =NULL ;netif_wake_queue (dev);
Greg Kroah-Hartman
maintainerPATCH 77/101 tty: vt: fix up tabstops properly
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Linus Torvalds <[email protected]>
commit f1869a890cdedb92a3fab969db5d0fd982850273 upstream.
Tabs on a console with long lines do not wrap properly, so correctly account for the line length when computing the tab placement location.
--- drivers/tty/vt/vt.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/tty/vt/vt.c +4 -4
@@ -1725,7 +1725,7 @@ static void reset_terminal(struct vc_datdefault_attr (vc);update_attr (vc); - vc->vc_tab_stop[0 ] =0x01010100 ; + vc->vc_tab_stop[0 ] = vc->vc_tab_stop[1 ] = vc->vc_tab_stop[2 ] = vc->vc_tab_stop[3] =
@@ -1769,7 +1769,7 @@ static void do_con_trol(struct tty_struc vc->vc_pos -= (vc->vc_x <<1 );while (vc->vc_x < vc->vc_cols -1 ) { vc->vc_x++; -if (vc->vc_tab_stop[vc->vc_x >>5 ] & (1 << (vc->vc_x &31 ))) +if (vc->vc_tab_stop[7 & (vc->vc_x >>5 )] & (1 << (vc->vc_x &31 )))break ; } vc->vc_pos += (vc->vc_x <<1 );
@@ -1829,7 +1829,7 @@ static void do_con_trol(struct tty_struclf (vc);return ;case 'H' : - vc->vc_tab_stop[vc->vc_x >>5 ] |= (1 << (vc->vc_x &31 )); + vc->vc_tab_stop[7 & (vc->vc_x >>5 )] |= (1 << (vc->vc_x &31 ));return ;case 'Z' :respond_ID (tty);
@@ -2022,7 +2022,7 @@ static void do_con_trol(struct tty_strucreturn ;case 'g' :if (!vc->vc_par[0 ]) - vc->vc_tab_stop[vc->vc_x >>5 ] &= ~(1 << (vc->vc_x &31 )); + vc->vc_tab_stop[7 & (vc->vc_x >>5 )] &= ~(1 << (vc->vc_x &31 ));else if (vc->vc_par[0 ] ==3 ) { vc->vc_tab_stop[0 ] = vc->vc_tab_stop[1] =
Greg Kroah-Hartman
maintainerPATCH 78/101 x86/entry/64: Dont use IST entry for #BP stack
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andy Lutomirski <[email protected]>
commit d8ba61ba58c88d5207c1ba2f7d9a2280e7d03be9 upstream.
There's nothing IST-worthy about #BP/int3. We don't allow kprobes in the small handful of places in the kernel that run at CPL0 with an invalid stack, and 32-bit kernels have used normal interrupt gates for #BP forever.
Furthermore, we don't allow kprobes in places that have usergs while in kernel mode, so "paranoid" is also unnecessary.
--- arch/x86/entry/entry_64.S | 2 +- arch/x86/kernel/idt.c | 2 -- arch/x86/kernel/traps.c | 15 ++++++++------- 3 files changed, 9 insertions(+), 10 deletions(-)
arch/x86/entry/entry_64.S +1 -1
@@ -1091,7 +1091,7 @@ apicinterrupt3 HYPERVISOR_CALLBACK_VECTO #endif /* CONFIG_HYPERV */ idtentry debug do_debug has_error_code=0 paranoid=1 shift_ist=DEBUG_STACK -idtentry int3 do_int3 has_error_code=0 paranoid=1 shift_ist=DEBUG_STACK +idtentry int3 do_int3 has_error_code=0 idtentry stack_segment do_stack_segment has_error_code=1 #ifdef CONFIG_XEN
arch/x86/kernel/idt.c +0 -2
@@ -160,7 +160,6 @@ static const __initconst struct idt_data*/ static const __initconst struct idt_data dbg_idts[] = {INTG (X86_TRAP_DB , debug), -INTG (X86_TRAP_BP , int3), };#endif
@@ -183,7 +182,6 @@ gate_desc debug_idt_table[IDT_ENTRIES] _static const __initconst struct idt_data ist_idts[] = {ISTG (X86_TRAP_DB , debug,DEBUG_STACK ),ISTG (X86_TRAP_NMI , nmi,NMI_STACK ), -SISTG (X86_TRAP_BP , int3,DEBUG_STACK ),ISTG (X86_TRAP_DF , double_fault,DOUBLEFAULT_STACK ), #ifdef CONFIG_X86_MCE ISTG(X86_TRAP_MC, &machine_check, MCE_STACK),
arch/x86/kernel/traps.c +8 -7
@@ -571,7 +571,6 @@ do_general_protection(struct pt_regs *re }NOKPROBE_SYMBOL (do_general_protection); -/* May run on IST stack. */ dotraplinkage void notrace do_int3(struct pt_regs *regs, long error_code) {#ifdef CONFIG_DYNAMIC_FTRACE
@@ -586,6 +585,13 @@ dotraplinkage void notrace do_int3(strucif (poke_int3_handler (regs))return ; +/* +* Use ist_enter despite the fact that we don't use an IST stack. +* We can be called from a kprobe in non-CONTEXT_KERNEL kernel +* mode or even during context tracking state changes. +* +* This means that we can't schedule. That's okay. +*/ ist_enter (regs);RCU_LOCKDEP_WARN (!rcu_is_watching (),"entry code didn't wake RCU" );#ifdef CONFIG_KGDB_LOW_LEVEL_TRAP
@@ -603,15 +609,10 @@ dotraplinkage void notrace do_int3(struc SIGTRAP) == NOTIFY_STOP) goto exit; -/* -* Let others (NMI) know that the debug stack is in use -* as we may switch to the interrupt stack. -*/ -debug_stack_usage_inc ();cond_local_irq_enable (regs);do_trap (X86_TRAP_BP ,SIGTRAP ,"int3" , regs, error_code,NULL );cond_local_irq_disable (regs); -debug_stack_usage_dec (); + exit:ist_exit (regs); }
Greg Kroah-Hartman
maintainerPATCH 79/101 selftests/x86/ptrace_syscall: Fix for yet more glibc interference
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andy Lutomirski <luto at kernel.org>
commit 4b0b37d4cc54b21a6ecad7271cbc850555869c62 upstream.
glibc keeps getting cleverer, and my version now turns raise() into more than one syscall. Since the test relies on ptrace seeing an exact set of syscalls, this breaks the test. Replace raise(SIGSTOP) with syscall(SYS_tgkill, ...) to force glibc to get out of our way.
--- tools/testing/selftests/x86/ptrace_syscall.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)
tools/testing/selftests/x86/ptrace_syscall.c +6 -2
@@ -183,8 +183,10 @@ static void test_ptrace_syscall_restart(if (ptrace (PTRACE_TRACEME ,0 ,0 ,0 ) !=0 )err (1 ,"PTRACE_TRACEME" ); +pid_t pid =getpid (), tid =syscall (SYS_gettid); +printf ("\tChild will make one syscall\n" ); -raise (SIGSTOP ); +syscall (SYS_tgkill, pid, tid,SIGSTOP );syscall (SYS_gettid,10 ,11 ,12 ,13 ,14 ,15 );_exit (0 );
@@ -301,9 +303,11 @@ static void test_restart_under_ptrace(voif (ptrace (PTRACE_TRACEME ,0 ,0 ,0 ) !=0 )err (1 ,"PTRACE_TRACEME" ); +pid_t pid =getpid (), tid =syscall (SYS_gettid); +printf ("\tChild will take a nap until signaled\n" );setsigign (SIGUSR1 ,SA_RESTART ); -raise (SIGSTOP ); +syscall (SYS_tgkill, pid, tid,SIGSTOP );syscall (SYS_pause,0 ,0 ,0 ,0 ,0 ,0 );_exit (0 );
--
To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Greg Kroah-Hartman
maintainerPATCH 80/101 x86/vsyscall/64: Use proper accessor to update P4D entry
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Boris Ostrovsky <[email protected]>
commit 31ad7f8e7dc94d3b85ccf9b6141ce6dfd35a1781 upstream.
Writing to it directly does not work for Xen PV guests.
--- arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/x86/entry/vsyscall/vsyscall_64.c +1 -1
@@ -355,7 +355,7 @@ void __init set_vsyscall_pgtable_user_biset_pgd (pgd,__pgd (pgd_val (*pgd) | _PAGE_USER)); p4d =p4d_offset (pgd,VSYSCALL_ADDR );#if CONFIG_PGTABLE_LEVELS >=5 - p4d->p4d |= _PAGE_USER; +set_p4d (p4d,__p4d (p4d_val (*p4d) | _PAGE_USER));#endif pud =pud_offset (p4d,VSYSCALL_ADDR );set_pud (pud,__pud (pud_val (*pud) | _PAGE_USER));
Greg Kroah-Hartman
maintainerPATCH 81/101 x86/efi: Free efi_pgd with free_pages()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Waiman Long <[email protected]>
commit 06ace26f4e6fcf747e890a39193be811777a048a upstream.
The efi_pgd is allocated as PGD_ALLOCATION_ORDER pages and therefore must also be freed as PGD_ALLOCATION_ORDER pages with free_pages().
--- arch/x86/platform/efi/efi_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/x86/platform/efi/efi_64.c +1 -1
@@ -227,7 +227,7 @@ int __init efi_alloc_page_tables(void)if (!pud) {if (CONFIG_PGTABLE_LEVELS >4 )free_page ((unsigned long )pgd_page_vaddr (*pgd)); -free_page ((unsigned long )efi_pgd); +free_pages ((unsigned long )efi_pgd,PGD_ALLOCATION_ORDER );return -ENOMEM ; }
Greg Kroah-Hartman
maintainerPATCH 82/101 posix-timers: Protect posix clock array access against speculation
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Thomas Gleixner <[email protected]>
commit 19b558db12f9f4e45a22012bae7b4783e62224da upstream.
The clockid argument of clockid_to_kclock() comes straight from user space via various syscalls and is used as index into the posix_clocks array.
Protect it against spectre v1 array out of bounds speculation. Remove the redundant check for !posix_clock[id] as this is another source for speculation and does not provide any advantage over the return posix_clock[id] path which returns NULL in that case anyway.
--- kernel/time/posix-timers.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-)
kernel/time/posix-timers.c +8 -3
@@ -50,6 +50,7 @@#include <linux/export.h> #include <linux/hashtable.h> #include <linux/compat.h> +#include <linux/nospec.h> #include "timekeeping.h" #include "posix-timers.h"
@@ -1346,11 +1347,15 @@ static const struct k_clock * const posistatic const struct k_clock *clockid_to_kclock (const clockid_t id) { -if (id <0 ) +clockid_t idx = id; + +if (id <0 ) {return (id &CLOCKFD_MASK ) ==CLOCKFD ? &clock_posix_dynamic : &clock_posix_cpu; + } -if (id >=ARRAY_SIZE (posix_clocks) || !posix_clocks[id]) +if (id >=ARRAY_SIZE (posix_clocks))return NULL ; -return posix_clocks[id]; + +return posix_clocks[array_index_nospec (idx,ARRAY_SIZE (posix_clocks))]; }
Greg Kroah-Hartman
maintainerPATCH 83/101 kvm/x86: fix icebp instruction handling
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Linus Torvalds <[email protected]>
commit 32d43cd391bacb5f0814c2624399a5dad3501d09 upstream.
The undocumented 'icebp' instruction (aka 'int1') works pretty much like 'int3' in the absense of in-circuit probing equipment (except, obviously, that it raises #DB instead of raising #BP), and is used by some validation test-suites as such.
But Andy Lutomirski noticed that his test suite acted differently in kvm than on bare hardware.
The reason is that kvm used an inexact test for the icebp instruction:
it just assumed that an all-zero VM exit qualification value meant that the VM exit was due to icebp.
That is not unlike the guess that do_debug() does for the actual exception handling case, but it's purely a heuristic, not an absolute rule. do_debug() does it because it wants to ascribe _some_ reasons to the #DB that happened, and an empty %dr6 value means that 'icebp' is the most likely casue and we have no better information.
But kvm can just do it right, because unlike the do_debug() case, kvm actually sees the real reason for the #DB in the VM-exit interruption information field.
So instead of relying on an inexact heuristic, just use the actual VM exit information that says "it was 'icebp'".
Right now the 'icebp' instruction isn't technically documented by Intel, but that will hopefully change. The special "privileged software exception" information _is_ actually mentioned in the Intel SDM, even though the cause of it isn't enumerated.
--- arch/x86/include/asm/vmx.h | 1 + arch/x86/kvm/vmx.c | 9 ++++++++- 2 files changed, 9 insertions(+), 1 deletion(-)
arch/x86/include/asm/vmx.h +1 -0
@@ -352,6 +352,7 @@ enum vmcs_field {#define INTR_TYPE_NMI_INTR (2 << 8)/* NMI */ #define INTR_TYPE_HARD_EXCEPTION (3 << 8)/* processor exception */ #define INTR_TYPE_SOFT_INTR (4 << 8)/* software interrupt */ +#define INTR_TYPE_PRIV_SW_EXCEPTION (5 << 8)/* ICE breakpoint - undocumented */ #define INTR_TYPE_SOFT_EXCEPTION (6 << 8)/* software exception */ /* GUEST_INTERRUPTIBILITY_INFO flags. */
arch/x86/kvm/vmx.c +8 -1
@@ -1071,6 +1071,13 @@ static inline bool is_machine_check(u32 (INTR_TYPE_HARD_EXCEPTION |MC_VECTOR |INTR_INFO_VALID_MASK ); } +/* Undocumented: icebp/int1 */ +static inline bool is_icebp (u32 intr_info) +{ +return (intr_info & (INTR_INFO_INTR_TYPE_MASK |INTR_INFO_VALID_MASK )) + == (INTR_TYPE_PRIV_SW_EXCEPTION |INTR_INFO_VALID_MASK ); +} +static inline bool cpu_has_vmx_msr_bitmap (void ) {return vmcs_config.cpu_based_exec_ctrl &CPU_BASED_USE_MSR_BITMAPS ;
@@ -6169,7 +6176,7 @@ static int handle_exception(struct kvm_v (KVM_GUESTDBG_SINGLESTEP | KVM_GUESTDBG_USE_HW_BP))) { vcpu->arch.dr6 &= ~15 ; vcpu->arch.dr6 |= dr6 |DR6_RTM ; -if (!(dr6 & ~DR6_RESERVED ))/* icebp */ +if (is_icebp (intr_info))skip_emulated_instruction (vcpu);kvm_queue_exception (vcpu,DB_VECTOR );
Greg Kroah-Hartman
maintainerPATCH 84/101 x86/build/64: Force the linker to use 2MB page size
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: H.J. Lu <[email protected]>
commit e3d03598e8ae7d195af5d3d049596dec336f569f upstream.
Binutils 2.31 will enable -z separate-code by default for x86 to avoid mixing code pages with data to improve cache performance as well as security. To reduce x86-64 executable and shared object sizes, the maximum page size is reduced from 2MB to 4KB. But x86-64 kernel must be aligned to 2MB. Pass -z max-page-size=0x200000 to linker to force 2MB page size regardless of the default page size used by linker.
Tested with Linux kernel 4.15.6 on x86-64.
--- arch/x86/Makefile | 9 +++++++++ 1 file changed, 9 insertions(+)
arch/x86/Makefile +9 -0
@@ -223,6 +223,15 @@ KBUILD_CFLAGS += $(cfi) $(cfi-sigframe) LDFLAGS := -m elf_$(UTS_MACHINE) +# +# The 64-bit kernel must be aligned to 2MB. Pass -z max-page-size=0x200000 to +# the linker to force 2MB page size regardless of the default page size used +# by the linker. +# +ifdef CONFIG_X86_64 +LDFLAGS += $(call ld-option, -z max-page-size=0x200000) +endif + # Speed up the build KBUILD_CFLAGS += -pipe # Workaround for a gcc prelease that unfortunately was shipped in a suse release
Greg Kroah-Hartman
maintainerPATCH 85/101 x86/boot/64: Verify alignment of the LOAD segment
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: H.J. Lu <[email protected]>
commit c55b8550fa57ba4f5e507be406ff9fc2845713e8 upstream.
Since the x86-64 kernel must be aligned to 2MB, refuse to boot the kernel if the alignment of the LOAD segment isn't a multiple of 2MB.
--- arch/x86/boot/compressed/misc.c | 4 ++++ 1 file changed, 4 insertions(+)
arch/x86/boot/compressed/misc.c +4 -0
@@ -309,6 +309,10 @@ static void parse_elf(void *output)switch (phdr->p_type) {case PT_LOAD : +#ifdef CONFIG_X86_64 +if ((phdr->p_align %0x200000 ) !=0 ) +error ("Alignment of LOAD segment isn't multiple of 2MB" ); +#endif #ifdef CONFIG_RELOCATABLE dest = output; dest += (phdr->p_paddr -LOAD_PHYSICAL_ADDR );
Greg Kroah-Hartman
maintainerPATCH 86/101 drm/syncobj: Stop reusing the same struct file for all syncobj -> fd
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 e7cdf5c82f1773c3386b93bbcf13b9bfff29fa31 upstream.
The vk cts test:
dEQP-VK.api.external.semaphore.opaque_fd.export_multiple_times_temporary
triggers a lot of
VFS: Close: file count is 0
Dave pointed out that clearing the syncobj->file from drm_syncobj_file_release() was sufficient to silence the test, but that opens a can of worm since we assumed that the syncobj->file was never unset. Stop trying to reuse the same struct file for every fd pointing to the drm_syncobj, and allocate one file for each fd instead.
v2: Fixup return handling of drm_syncobj_fd_to_handle v2.1: [airlied: fix possible syncobj ref race] v2.2: [jekstrand: back-port to 4.14]
--- drivers/gpu/drm/drm_syncobj.c | 81 +++++++++++++++--------------------------- include/drm/drm_syncobj.h | 5 -- 2 files changed, 30 insertions(+), 56 deletions(-)
drivers/gpu/drm/drm_syncobj.c +30 -51
@@ -328,28 +328,11 @@ static const struct file_operations drm_ .release = drm_syncobj_file_release, }; -static int drm_syncobj_alloc_file (struct drm_syncobj *syncobj) -{ -struct file *file =anon_inode_getfile ("syncobj_file" , - &drm_syncobj_file_fops, - syncobj,0 ); -if (IS_ERR (file)) -return PTR_ERR (file); - -drm_syncobj_get (syncobj); -if (cmpxchg (&syncobj->file,NULL , file)) { -/* lost the race */ -fput (file); - } - -return 0 ; -} -static int drm_syncobj_handle_to_fd (struct drm_file *file_private,u32 handle,int *p_fd) {struct drm_syncobj *syncobj =drm_syncobj_find (file_private, handle); -int ret; +struct file *file;int fd;if (!syncobj)
@@ -361,46 +344,40 @@ static int drm_syncobj_handle_to_fd(strureturn fd; } -if (!syncobj->file) { - ret =drm_syncobj_alloc_file (syncobj); -if (ret) - goto out_put_fd; + file =anon_inode_getfile ("syncobj_file" , + &drm_syncobj_file_fops, + syncobj,0 ); +if (IS_ERR (file)) { +put_unused_fd (fd); +drm_syncobj_put (syncobj); +return PTR_ERR (file); } -fd_install (fd, syncobj->file); -drm_syncobj_put (syncobj); + +drm_syncobj_get (syncobj); +fd_install (fd, file); + *p_fd = fd;return 0 ; -out_put_fd: -put_unused_fd (fd); -drm_syncobj_put (syncobj); -return ret; } -static struct drm_syncobj *drm_syncobj_fdget (int fd) -{ -struct file *file =fget (fd); - -if (!file) -return NULL ; -if (file->f_op != &drm_syncobj_file_fops) - goto err; - -return file->private_data; -err: -fput (file); -return NULL ; -}; -static int drm_syncobj_fd_to_handle (struct drm_file *file_private,int fd,u32 *handle) { -struct drm_syncobj *syncobj =drm_syncobj_fdget (fd); +struct drm_syncobj *syncobj; +struct file *file;int ret; -if (!syncobj) + file =fget (fd); +if (!file) +return -EINVAL ; + +if (file->f_op != &drm_syncobj_file_fops) { +fput (file);return -EINVAL ; + }/* take a reference to put in the idr */ + syncobj = file->private_data;drm_syncobj_get (syncobj);idr_preload (GFP_KERNEL );
@@ -409,12 +386,14 @@ static int drm_syncobj_fd_to_handle(struspin_unlock (&file_private->syncobj_table_lock);idr_preload_end (); -if (ret <0 ) { -fput (syncobj->file); -return ret; - } - *handle = ret; -return 0 ; +if (ret >0 ) { + *handle = ret; + ret =0 ; + }else +drm_syncobj_put (syncobj); + +fput (file); +return ret; } int drm_syncobj_import_sync_file_fence(struct drm_file *file_private,
include/drm/drm_syncobj.h +0 -5
@@ -60,11 +60,6 @@ struct drm_syncobj {* locks cb_list and write-locks fence. */ spinlock_t lock; -/** -* @file: -* a file backing for this syncobj. -*/ -struct file *file; }; typedef void (*drm_syncobj_func_t)(struct drm_syncobj *syncobj,
Greg Kroah-Hartman
maintainerPATCH 87/101 perf/x86/intel/uncore: Fix Skylake UPI event format
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kan Liang <[email protected]>
commit 317660940fd9dddd3201c2f92e25c27902c753fa upstream.
There is no event extension (bit 21) for SKX UPI, so use 'event' instead of 'event_ext'.
--- arch/x86/events/intel/uncore_snbep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/x86/events/intel/uncore_snbep.c +1 -1
@@ -3598,7 +3598,7 @@ static struct intel_uncore_type skx_unco };static struct attribute *skx_upi_uncore_formats_attr[] = { - &format_attr_event_ext.attr, + &format_attr_event.attr, &format_attr_umask_ext.attr, &format_attr_edge.attr, &format_attr_inv.attr,
Greg Kroah-Hartman
maintainerPATCH 88/101 perf stat: Fix CVS output format for non-supported counters
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Ilya Pronin <[email protected]>
commit 40c21898ba5372c14ef71717040529794a91ccc2 upstream.
When printing stats in CSV mode, 'perf stat' appends extra separators when a counter is not supported:
<not supported>,,L1-dcache-store-misses,mesos/bd442f34-2b4a-47df-b966-9b281f9f56fc,0,100.00,,,,
Which causes a failure when parsing fields. The numbers of separators should be the same for each line, no matter if the counter is or not supported.
--- tools/perf/builtin-stat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
tools/perf/builtin-stat.c +1 -1
@@ -922,7 +922,7 @@ static void print_metric_csv(void *ctx,char buf[64 ], *vals, *ends;if (unit ==NULL || fmt ==NULL ) { -fprintf (out,"%s%s%s%s" , csv_sep, csv_sep, csv_sep, csv_sep); +fprintf (out,"%s%s" , csv_sep, csv_sep);return ; }snprintf (buf,sizeof (buf), fmt, val);
Greg Kroah-Hartman
maintainerPATCH 89/101 perf/core: Fix ctx_event_type in ctx_resched()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Song Liu <[email protected]>
commit bd903afeb504db5655a45bb4cf86f38be5b1bf62 upstream.
In ctx_resched(), EVENT_FLEXIBLE should be sched_out when EVENT_PINNED is added. However, ctx_resched() calculates ctx_event_type before checking this condition. As a result, pinned events will NOT get higher priority than flexible events.
The following shows this issue on an Intel CPU (where ref-cycles can only use one hardware counter).
1. First start:
perf stat -C 0 -e ref-cycles -I 1000
2. Then, in the second console, run:
perf stat -C 0 -e ref-cycles:D -I 1000
The second perf uses pinned events, which is expected to have higher priority. However, because it failed in ctx_resched(). It is never run.
This patch fixes this by calculating ctx_event_type after re-evaluating event_type.
--- kernel/events/core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
kernel/events/core.c +3 -1
@@ -2322,7 +2322,7 @@ static void ctx_resched(struct perf_cpu_struct perf_event_context *task_ctx, enum event_type_t event_type) { - enum event_type_t ctx_event_type = event_type & EVENT_ALL; + enum event_type_t ctx_event_type;bool cpu_event = !!(event_type &EVENT_CPU );/*
@@ -2332,6 +2332,8 @@ static void ctx_resched(struct perf_cpu_if (event_type &EVENT_PINNED ) event_type |=EVENT_FLEXIBLE ; + ctx_event_type = event_type &EVENT_ALL ; +perf_pmu_disable (cpuctx->ctx.pmu);if (task_ctx)task_ctx_sched_out (cpuctx, task_ctx, event_type);
Greg Kroah-Hartman
maintainerPATCH 90/101 perf/x86/intel: Dont accidentally clear high bits in bdw_limit_period()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dan Carpenter <[email protected]>
commit e5ea9b54a055619160bbfe527ebb7d7191823d66 upstream.
We intended to clear the lowest 6 bits but because of a type bug we clear the high 32 bits as well. Andi says that periods are rarely more than U32_MAX so this bug probably doesn't have a huge runtime impact.
--- arch/x86/events/intel/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/x86/events/intel/core.c +1 -1
@@ -3194,7 +3194,7 @@ static unsigned bdw_limit_period(struct X86_CONFIG(.event=0xc0, .umask=0x01)) {if (left <128 ) left =128 ; - left &= ~0x3fu ; + left &= ~0x3fULL ; }return left; }
Greg Kroah-Hartman
maintainerPATCH 91/101 perf/x86/intel/uncore: Fix multi-domain PCI CHA enumeration bug on Skylake servers
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Kan Liang <[email protected]>
commit 320b0651f32b830add6497fcdcfdcb6ae8c7b8a0 upstream.
The number of CHAs is miscalculated on multi-domain PCI Skylake server systems, resulting in an uncore driver initialization error.
Gary Kroening explains:
"For systems with a single PCI segment, it is sufficient to look for the
bus number to change in order to determine that all of the CHa's have been counted for a single socket.
However, for multi PCI segment systems, each socket is given a new segment and the bus number does NOT change. So looking only for the bus number to change ends up counting all of the CHa's on all sockets in the system. This leads to writing CPU MSRs beyond a valid range and causes an error in ivbep_uncore_msr_init_box()."
To fix this bug, query the number of CHAs from the CAPID6 register:
it should read bits 27:0 in the CAPID6 register located at Device 30, Function 3, Offset 0x9C. These 28 bits form a bit vector of available LLC slices and the CHAs that manage those slices.
--- arch/x86/events/intel/uncore_snbep.c | 31 +++++++++++++++++-------------- 1 file changed, 17 insertions(+), 14 deletions(-)
arch/x86/events/intel/uncore_snbep.c +17 -14
@@ -3554,24 +3554,27 @@ static struct intel_uncore_type *skx_msr NULL, }; +/* +* To determine the number of CHAs, it should read bits 27:0 in the CAPID6 +* register which located at Device 30, Function 3, Offset 0x9C. PCI ID 0x2083. +*/ +#define SKX_CAPID6 0x9c +#define SKX_CHA_BIT_MASK GENMASK(27, 0) +static int skx_count_chabox (void ) { -struct pci_dev *chabox_dev =NULL ; -int bus, count =0 ; +struct pci_dev *dev =NULL ; +u32 val =0 ; -while (1 ) { - chabox_dev =pci_get_device (PCI_VENDOR_ID_INTEL ,0x208d , chabox_dev); -if (!chabox_dev) -break ; -if (count ==0 ) - bus = chabox_dev->bus->number; -if (bus != chabox_dev->bus->number) -break ; - count++; - } + dev =pci_get_device (PCI_VENDOR_ID_INTEL ,0x2083 , dev); +if (!dev) + goto out; -pci_dev_put (chabox_dev); -return count; +pci_read_config_dword (dev,SKX_CAPID6 , &val); + val &=SKX_CHA_BIT_MASK ; +out: +pci_dev_put (dev); +return hweight32 (val); }void skx_uncore_cpu_init (void )
Greg Kroah-Hartman
maintainerPATCH 92/101 iio: ABI: Fix name of timestamp sysfs file
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Linus Walleij <[email protected]>
commit b9a3589332c2a25fb7edad25a26fcaada3209126 upstream.
The name of the file is "current_timetamp_clock" not "timestamp_clock".
--- Documentation/ABI/testing/sysfs-bus-iio | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Documentation/ABI/testing/sysfs-bus-iio +1 -1
@@ -32,7 +32,7 @@ Description: Description of the physical chip / device for device X. Typically a part number. -What: /sys/bus/iio/devices/iio:deviceX/timestamp_clock +What: /sys/bus/iio/devices/iio:deviceX/current_timestamp_clock KernelVersion: 4.5 Contact: [email protected] Description:
Greg Kroah-Hartman
maintainerPATCH 93/101 iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Lorenzo Bianconi <[email protected]>
commit 7b9ebe428266fb7e0a6d769bb3ff3fcb6044b15e upstream.
Apply le16_to_cpu() to data read from the sensor in order to take into account architecture endianness
--- drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c +1 -1
@@ -392,7 +392,7 @@ static int st_lsm6dsx_read_oneshot(strucst_lsm6dsx_sensor_disable (sensor); - *val = (s16 )data; + *val = (s16 )le16_to_cpu (data);return IIO_VAL_INT ; }
Greg Kroah-Hartman
maintainerPATCH 94/101 staging: android: ion: Zero CMA allocated memory
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Liam Mark <[email protected]>
commit 6d79bd5bb6c79a9dba4842040c9adf39e7806330 upstream.
Since commit 204f672255c2 ("staging: android: ion: Use CMA APIs directly") the CMA API is now used directly and therefore the allocated memory is no longer automatically zeroed.
Explicitly zero CMA allocated memory to ensure that no data is exposed to userspace.
--- drivers/staging/android/ion/ion_cma_heap.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)
drivers/staging/android/ion/ion_cma_heap.c +17 -0
@@ -21,6 +21,7 @@#include <linux/err.h> #include <linux/cma.h> #include <linux/scatterlist.h> +#include <linux/highmem.h> #include "ion.h"
@@ -51,6 +52,22 @@ static int ion_cma_allocate(struct ion_hif (!pages)return -ENOMEM ; +if (PageHighMem (pages)) { +unsigned long nr_clear_pages = nr_pages; +struct page *page = pages; + +while (nr_clear_pages >0 ) { +void *vaddr =kmap_atomic (page); + +memset (vaddr,0 ,PAGE_SIZE ); +kunmap_atomic (vaddr); + page++; + nr_clear_pages--; + } + }else { +memset (page_address (pages),0 , size); + } + table =kmalloc (sizeof (*table),GFP_KERNEL );if (!table) goto err;
Greg Kroah-Hartman
maintainerPATCH 95/101 staging: lustre: ptlrpc: kfree used instead of kvfree
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Nadav Amit <[email protected]>
commit c3eec59659cf25916647d2178c541302bb4822ad upstream.
rq_reqbuf is allocated using kvmalloc() but released in one occasion using kfree() instead of kvfree().
The issue was found using grep based on a similar bug.
--- drivers/staging/lustre/lustre/ptlrpc/sec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/lustre/lustre/ptlrpc/sec.c +1 -1
@@ -847,7 +847,7 @@ void sptlrpc_request_out_callback(structif (req->rq_pool || !req->rq_reqbuf)return ; -kfree (req->rq_reqbuf); +kvfree (req->rq_reqbuf); req->rq_reqbuf =NULL ; req->rq_reqbuf_len =0 ; }
Greg Kroah-Hartman
maintainerPATCH 96/101 usb: xhci: Disable slot even when virt-dev is null
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Lu Baolu <[email protected]>
commit b64149ca016c25f30b39ac5a8f37cfb9017e19bb upstream.
xhci_disable_slot() is a helper for disabling a slot when a device goes away or recovers from error situations. Currently, it checks the corespoding virt-dev pointer and returns directly (w/o issuing disable slot command) if it's null.
This is unnecessary and will cause problems in case where virt-dev allocation fails and xhci_disable_slot() is called to roll back the hardware state. Refer to the implementation of xhci_alloc_dev().
This patch removes lines to check virt-dev in xhci_disable_slot().
--- drivers/usb/host/xhci-hub.c | 3 +++ drivers/usb/host/xhci.c | 4 ---- 2 files changed, 3 insertions(+), 4 deletions(-)
drivers/usb/host/xhci-hub.c +3 -0
@@ -634,6 +634,9 @@ static int xhci_enter_test_mode(struct xxhci_dbg (xhci,"Disable all slots\n" );spin_unlock_irqrestore (&xhci->lock, *flags);for (i =1 ; i <=HCS_MAX_SLOTS (xhci->hcs_params1); i++) { +if (!xhci->devs[i]) +continue ; + retval =xhci_disable_slot (xhci,NULL , i);if (retval) xhci_err(xhci, "Failed to disable slot %d, %d. Enter test mode anyway\n",
drivers/usb/host/xhci.c +0 -4
@@ -3571,11 +3571,7 @@ int xhci_disable_slot(struct xhci_hcd *xunsigned long flags;u32 state;int ret =0 ; -struct xhci_virt_device *virt_dev; - virt_dev = xhci->devs[slot_id]; -if (!virt_dev) -return -EINVAL ;if (!command) command =xhci_alloc_command (xhci, false, false,GFP_KERNEL );if (!command)
Greg Kroah-Hartman
maintainerPATCH 97/101 usb: xhci: Fix potential memory leak in xhci_disable_slot()
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Lu Baolu <[email protected]>
commit cd3f1790b006d91786728c20a01da21ee277aff1 upstream.
xhci_disable_slot() allows the invoker to pass a command pointer as paramenter. Otherwise, it will allocate one. This will cause memory leak when a command structure was allocated inside of this function while queuing command trb fails. Another problem comes up when the invoker passed a command pointer, but xhci_disable_slot() frees it when it detects a dead host.
This patch fixes these two problems by removing the command parameter from xhci_disable_slot().
--- drivers/usb/host/xhci-hub.c | 2 +- drivers/usb/host/xhci.c | 30 +++++++++--------------------- drivers/usb/host/xhci.h | 3 +-- 3 files changed, 11 insertions(+), 24 deletions(-)
drivers/usb/host/xhci-hub.c +1 -1
@@ -637,7 +637,7 @@ static int xhci_enter_test_mode(struct xif (!xhci->devs[i])continue ; - retval =xhci_disable_slot (xhci,NULL , i); + retval =xhci_disable_slot (xhci, i);if (retval)xhci_err (xhci,"Failed to disable slot %d, %d. Enter test mode anyway\n" , i, retval);
drivers/usb/host/xhci.c +9 -21
@@ -3523,11 +3523,6 @@ static void xhci_free_dev(struct usb_hcdstruct xhci_virt_device *virt_dev;struct xhci_slot_ctx *slot_ctx;int i, ret; -struct xhci_command *command; - - command =xhci_alloc_command (xhci, false, false,GFP_KERNEL ); -if (!command) -return ;#ifndef CONFIG_USB_DEFAULT_PERSIST /*
@@ -3543,10 +3538,8 @@ static void xhci_free_dev(struct usb_hcd/* If the host is halted due to driver unload, we still need to free the * device. */ -if (ret <=0 && ret != -ENODEV ) { -kfree (command); +if (ret <=0 && ret != -ENODEV )return ; - } virt_dev = xhci->devs[udev->slot_id]; slot_ctx =xhci_get_slot_ctx (xhci, virt_dev->out_ctx);
@@ -3558,22 +3551,21 @@ static void xhci_free_dev(struct usb_hcddel_timer_sync (&virt_dev->eps[i].stop_cmd_timer); } -xhci_disable_slot (xhci, command, udev->slot_id); +xhci_disable_slot (xhci, udev->slot_id);/* * Event command completion handler will free any data structures * associated with the slot. XXX Can free sleep? */ } -int xhci_disable_slot (struct xhci_hcd *xhci,struct xhci_command *command, -u32 slot_id) +int xhci_disable_slot (struct xhci_hcd *xhci,u32 slot_id) { +struct xhci_command *command;unsigned long flags;u32 state;int ret =0 ; -if (!command) - command =xhci_alloc_command (xhci, false, false,GFP_KERNEL ); + command =xhci_alloc_command (xhci, false, false,GFP_KERNEL );if (!command)return -ENOMEM ;
@@ -3591,7 +3583,7 @@ int xhci_disable_slot(struct xhci_hcd *x slot_id);if (ret) {spin_unlock_irqrestore (&xhci->lock, flags); -xhci_dbg (xhci,"FIXME: allocate a command ring segment\n" ); +kfree (command);return ret; }xhci_ring_cmd_db (xhci);
@@ -3666,6 +3658,8 @@ int xhci_alloc_dev(struct usb_hcd *hcd,return 0 ; } +xhci_free_command (xhci, command); +if ((xhci->quirks &XHCI_EP_LIMIT_QUIRK )) {spin_lock_irqsave (&xhci->lock, flags); ret =xhci_reserve_host_control_ep_resources (xhci);
@@ -3701,18 +3695,12 @@ int xhci_alloc_dev(struct usb_hcd *hcd,pm_runtime_get_noresume (hcd->self.controller);#endif - -xhci_free_command (xhci, command);/* Is this a LS or FS device under a HS hub? */ /* Hub or peripherial? */ return 1 ; disable_slot: -/* Disable slot, if we can do it without mem alloc */ -kfree (command->completion); - command->completion =NULL ; - command->status =0 ; -return xhci_disable_slot (xhci, command, udev->slot_id); +return xhci_disable_slot (xhci, udev->slot_id); }/*
drivers/usb/host/xhci.h +1 -2
@@ -2014,8 +2014,7 @@ int xhci_run(struct usb_hcd *hcd);int xhci_gen_setup (struct usb_hcd *hcd,xhci_get_quirks_t get_quirks);void xhci_init_driver (struct hc_driver *drv,const struct xhci_driver_overrides *over); -int xhci_disable_slot (struct xhci_hcd *xhci, -struct xhci_command *command,u32 slot_id); +int xhci_disable_slot (struct xhci_hcd *xhci,u32 slot_id);int xhci_suspend (struct xhci_hcd *xhci,bool do_wakeup);int xhci_resume (struct xhci_hcd *xhci,bool hibernated);
Greg Kroah-Hartman
maintainerPATCH 98/101 x86/pkeys/selftests: Rename si_pkey to siginfo_pkey
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dave Hansen <[email protected]>
commit 91c49c2deb96ffc3c461eaae70219d89224076b7 upstream.
'si_pkey' is now #defined to be the name of the new siginfo field that protection keys uses. Rename it not to conflict.
--- tools/testing/selftests/x86/protection_keys.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)
tools/testing/selftests/x86/protection_keys.c +5 -5
@@ -250,7 +250,7 @@ void signal_handler(int signum, siginfo_unsigned long ip;char *fpregs;u32 *pkru_ptr; -u64 si_pkey; +u64 siginfo_pkey;u32 *si_pkey_ptr;int pkru_offset;fpregset_t fpregset;
@@ -292,9 +292,9 @@ void signal_handler(int signum, siginfo_ si_pkey_ptr = (u32 *)(((u8 *)si) + si_pkey_offset);dprintf1 ("si_pkey_ptr: %p\n" , si_pkey_ptr);dump_mem (si_pkey_ptr -8 ,24 ); - si_pkey = *si_pkey_ptr; -pkey_assert (si_pkey <NR_PKEYS ); - last_si_pkey = si_pkey; + siginfo_pkey = *si_pkey_ptr; +pkey_assert (siginfo_pkey <NR_PKEYS ); + last_si_pkey = siginfo_pkey; if ((si->si_code == SEGV_MAPERR) || (si->si_code == SEGV_ACCERR) ||
@@ -306,7 +306,7 @@ void signal_handler(int signum, siginfo_dprintf1 ("signal pkru from xsave: %08x\n" , *pkru_ptr);/* need __rdpkru() version so we do not do shadow_pkru checking */ dprintf1 ("signal pkru from pkru: %08x\n" ,__rdpkru ()); -dprintf1 ("si_pkey from siginfo: %jx\n" , si_pkey); +dprintf1 ("pkey from siginfo: %jx\n" , siginfo_pkey); *(u64 *)pkru_ptr =0x00000000 ;dprintf1 ("WARNING: set PRKU=0 to allow faulting instruction to continue\n" ); pkru_faults++;
Greg Kroah-Hartman
maintainerPATCH 99/101 kbuild: disable clangs default use of -fmerge-all-constants
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Daniel Borkmann <[email protected]>
commit 87e0d4f0f37fb0c8c4aeeac46fff5e957738df79 upstream.
Prasad reported that he has seen crashes in BPF subsystem with netd on Android with arm64 in the form of (note, the taint is unrelated):
[ 4134.721483] Unable to handle kernel paging request at virtual address 800000001 [ 4134.820925] Mem abort info: [ 4134.901283] Exception class = DABT (current EL), IL = 32 bits [ 4135.016736] SET = 0, FnV = 0 [ 4135.119820] EA = 0, S1PTW = 0 [ 4135.201431] Data abort info: [ 4135.301388] ISV = 0, ISS = 0x00000021 [ 4135.359599] CM = 0, WnR = 0 [ 4135.470873] user pgtable: 4k pages, 39-bit VAs, pgd = ffffffe39b946000 [ 4135.499757] [0000000800000001] *pgd=0000000000000000, *pud=0000000000000000 [ 4135.660725] Internal error: Oops: 96000021 [#1] PREEMPT SMP [ 4135.674610] Modules linked in: [ 4135.682883] CPU: 5 PID: 1260 Comm: netd Tainted: G S W 4.14.19+ #1 [ 4135.716188] task: ffffffe39f4aa380 task.stack: ffffff801d4e0000 [ 4135.731599] PC is at bpf_prog_add+0x20/0x68 [ 4135.741746] LR is at bpf_prog_inc+0x20/0x2c [ 4135.751788] pc : [<ffffff94ab7ad584>] lr : [<ffffff94ab7ad638>] pstate: 60400145 [ 4135.769062] sp : ffffff801d4e3ce0 [...] [ 4136.258315] Process netd (pid: 1260, stack limit = 0xffffff801d4e0000) [ 4136.273746] Call trace: [...] [ 4136.442494] 3ca0: ffffff94ab7ad584 0000000060400145 ffffffe3a01bf8f8 0000000000000006 [ 4136.460936] 3cc0: 0000008000000000 ffffff94ab844204 ffffff801d4e3cf0 ffffff94ab7ad584 [ 4136.479241] [<ffffff94ab7ad584>] bpf_prog_add+0x20/0x68 [ 4136.491767] [<ffffff94ab7ad638>] bpf_prog_inc+0x20/0x2c [ 4136.504536] [<ffffff94ab7b5d08>] bpf_obj_get_user+0x204/0x22c [ 4136.518746] [<ffffff94ab7ade68>] SyS_bpf+0x5a8/0x1a88
Android's netd was basically pinning the uid cookie BPF map in BPF fs (/sys/fs/bpf/traffic_cookie_uid_map) and later on retrieving it again resulting in above panic. Issue is that the map was wrongly identified as a prog! Above kernel was compiled with clang 4.0, and it turns out that clang decided to merge the bpf_prog_iops and bpf_map_iops into a single memory location, such that the two i_ops could then not be distinguished anymore.
Reason for this miscompilation is that clang has the more aggressive -fmerge-all-constants enabled by default. In fact, clang source code has a comment about it in lib/AST/ExprConstant.cpp on why it is okay to do so:
Pointers with different bases cannot represent the same object. (Note that clang defaults to -fmerge-all-constants, which can lead to inconsistent results for comparisons involving the address of a constant; this generally doesn't matter in practice.)
The issue never appeared with gcc however, since gcc does not enable -fmerge-all-constants by default and even *explicitly* states in it's option description that using this flag results in non-conforming behavior, quote from man gcc:
Languages like C or C++ require each variable, including multiple instances of the same variable in recursive calls, to have distinct locations, so using this option results in non-conforming behavior.
There are also various clang bug reports open on that matter [1], where clang developers acknowledge the non-conforming behavior, and refer to disabling it with -fno-merge-all-constants. But even if this gets fixed in clang today, there are already users out there that triggered this. Thus, fix this issue by explicitly adding -fno-merge-all-constants to the kernel's Makefile to generically disable this optimization, since potentially other places in the kernel could subtly break as well.
Note, there is also a flag called -fmerge-constants (not supported by clang), which is more conservative and only applies to strings and it's enabled in gcc's -O/-O2/-O3/-Os optimization levels. In gcc's code, the two flags -fmerge-{all-,}constants share the same variable internally, so when disabling it via -fno-merge-all-constants, then we really don't merge any const data (e.g. strings), and text size increases with gcc (14,927,214 -> 14,942,646 for vmlinux.o).
$ gcc -fverbose-asm -O2 foo.c -S -o foo.S
-> foo.S lists -fmerge-constants under options enabled
$ gcc -fverbose-asm -O2 -fno-merge-all-constants foo.c -S -o foo.S
-> foo.S doesn't list -fmerge-constants under options enabled
$ gcc -fverbose-asm -O2 -fno-merge-all-constants -fmerge-constants foo.c -S -o foo.S
-> foo.S lists -fmerge-constants under options enabledThus, as a workaround we need to set both -fno-merge-all-constants *and* -fmerge-constants in the Makefile in order for text size to stay as is.
[1] https://bugs.llvm.org/show_bug.cgi?id=18538
--- Makefile | 9 +++++++++ 1 file changed, 9 insertions(+)
Makefile +9 -0
@@ -805,6 +805,15 @@ KBUILD_CFLAGS += $(call cc-disable-warni # disable invalid "can't wrap" optimizations for signed / pointers KBUILD_CFLAGS += $(call cc-option,-fno-strict-overflow) +# clang sets -fmerge-all-constants by default as optimization, but this +# is non-conforming behavior for C and in fact breaks the kernel, so we +# need to disable it here generally. +KBUILD_CFLAGS += $(call cc-option,-fno-merge-all-constants) + +# for gcc -fno-merge-all-constants disables everything, but it is fine +# to have actual conforming behavior enabled. +KBUILD_CFLAGS += $(call cc-option,-fmerge-constants) + # Make sure -fstack-check isn't enabled (like gentoo apparently did) KBUILD_CFLAGS += $(call cc-option,-fno-stack-check,)
Greg Kroah-Hartman
maintainerPATCH 100/101 bpf: skip unnecessary capability check
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Chenbo Feng <[email protected]>
commit 0fa4fe85f4724fff89b09741c437cbee9cf8b008 upstream.
The current check statement in BPF syscall will do a capability check for CAP_SYS_ADMIN before checking sysctl_unprivileged_bpf_disabled. This code path will trigger unnecessary security hooks on capability checking and cause false alarms on unprivileged process trying to get CAP_SYS_ADMIN access. This can be resolved by simply switch the order of the statement and CAP_SYS_ADMIN is not required anyway if unprivileged bpf syscall is allowed.
--- kernel/bpf/syscall.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
kernel/bpf/syscall.c +1 -1
@@ -1455,7 +1455,7 @@ SYSCALL_DEFINE3(bpf, int, cmd, union bpfunion bpf_attr attr = {};int err; -if (!capable (CAP_SYS_ADMIN ) && sysctl_unprivileged_bpf_disabled) +if (sysctl_unprivileged_bpf_disabled && !capable (CAP_SYS_ADMIN ))return -EPERM ; err =check_uarg_tail_zero (uattr,sizeof (attr), size);
Greg Kroah-Hartman
maintainerPATCH 101/101 bpf, x64: increase number of passes
In reply to Greg Kroah-Hartman4.14-stable review patch. If anyone has any objections, please let me know.
------------------
From: Daniel Borkmann <[email protected]>
commit 6007b080d2e2adb7af22bf29165f0594ea12b34c upstream.
In Cilium some of the main programs we run today are hitting 9 passes on x64's JIT compiler, and we've had cases already where we surpassed the limit where the JIT then punts the program to the interpreter instead, leading to insertion failures due to CONFIG_BPF_JIT_ALWAYS_ON or insertion failures due to the prog array owner being JITed but the program to insert not (both must have the same JITed/non-JITed property).
One concrete case the program image shrunk from 12,767 bytes down to 10,288 bytes where the image converged after 16 steps. I've measured that this took 340us in the JIT until it converges on my i7-6600U. Thus, increase the original limit we had from day one where the JIT covered cBPF only back then before we run into the case (as similar with the complexity limit) where we trip over this and hit program rejections. Also add a cond_resched() into the compilation loop, the JIT process runs without any locks and may sleep anyway.
--- arch/x86/net/bpf_jit_comp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
arch/x86/net/bpf_jit_comp.c +2 -1
@@ -1156,7 +1156,7 @@ struct bpf_prog *bpf_int_jit_compile(str* may converge on the last pass. In such case do one more * pass to emit the final image */ -for (pass =0 ; pass <10 || image; pass++) { +for (pass =0 ; pass <20 || image; pass++) { proglen =do_jit (prog, addrs, image, oldproglen, &ctx);if (proglen <=0 ) { image =NULL ;
@@ -1183,6 +1183,7 @@ struct bpf_prog *bpf_int_jit_compile(str } } oldproglen = proglen; +cond_resched (); }if (bpf_jit_enable >1 )
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.31 release. There are 101 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 Thu Mar 29 16:27:21 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.31-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
stable-rc/linux-4.14.y boot: 126 boots: 0 failed, 60 passed with 63 offline, 3 untried/unknown (v4.14.30-102-g2d6120998a4c)
stable-rc/linux-4.14.y boot: 126 boots: 0 failed, 60 passed with 63 offline, 3 untried/unknown (v4.14.30-102-g2d6120998a4c)
Full Boot Summary: https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.14.y/kernel/v4.14.30-102-g2d6120998a4c/
Full Build Summary: https://kernelci.org/build/stable-rc/branch/linux-4.14.y/kernel/v4.14.30-102-g2d6120998a4c/
Tree: stable-rc
Branch: linux-4.14.y
Git Describe: v4.14.30-102-g2d6120998a4c
Git Commit: 2d6120998a4cef5d352e931b1c85f1e3d79081d5
Git URL: http://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
Tested: 68 unique boards, 23 SoC families, 17 builds out of 185
Offline Platforms:
arm:
sunxi_defconfig:
sun4i-a10-cubieboard: 1 offline lab
sun5i-r8-chip: 1 offline lab
sun7i-a20-bananapi: 1 offline lab
sun7i-a20-cubietruck: 1 offline lab
bcm2835_defconfig:
bcm2835-rpi-b: 1 offline lab
sama5_defconfig:
at91-sama5d4_xplained: 1 offline lab
multi_v7_defconfig:
alpine-db: 1 offline lab
am335x-boneblack: 1 offline lab armada-xp-openblocks-ax3-4: 1 offline lab
at91-sama5d4_xplained: 1 offline lab
exynos5250-arndale: 1 offline lab
exynos5410-odroidxu: 1 offline lab
exynos5420-arndale-octa: 1 offline lab
exynos5800-peach-pi: 1 offline lab
imx6dl-wandboard_dual: 1 offline lab
meson8b-odroidc1: 1 offline lab
mt7623n-bananapi-bpi-r2: 1 offline lab
omap3-beagle: 1 offline lab
omap3-beagle-xm: 1 offline lab
omap4-panda: 1 offline lab
qcom-apq8064-cm-qs600: 1 offline lab
qcom-apq8064-ifc6410: 1 offline lab socfpga_cyclone5_de0_sockit: 1 offline lab
stih410-b2120: 1 offline lab
sun4i-a10-cubieboard: 1 offline lab
sun5i-r8-chip: 1 offline lab
sun7i-a20-bananapi: 1 offline lab
sun7i-a20-cubietruck: 1 offline lab
tegra124-jetson-tk1: 1 offline lab
tegra20-iris-512: 1 offline lab
tegra30-beaver: 1 offline lab
vf610-colibri-eval-v3: 1 offline lab
zynq-zc702: 1 offline lab
tegra_defconfig:
tegra124-jetson-tk1: 1 offline lab
tegra20-iris-512: 1 offline lab
tegra30-beaver: 1 offline lab
imx_v6_v7_defconfig:
imx6dl-wandboard_dual: 1 offline lab
imx6dl-wandboard_solo: 1 offline lab
vf610-colibri-eval-v3: 1 offline lab
exynos_defconfig:
exynos5250-arndale: 1 offline lab
exynos5410-odroidxu: 1 offline lab
exynos5420-arndale-octa: 1 offline lab
exynos5422-odroidxu3: 1 offline lab
exynos5800-peach-pi: 1 offline lab
qcom_defconfig:
qcom-apq8064-cm-qs600: 1 offline lab
qcom-apq8064-ifc6410: 1 offline lab
omap2plus_defconfig:
am335x-boneblack: 1 offline lab
omap3-beagle: 1 offline lab
omap3-beagle-xm: 1 offline lab
omap4-panda: 1 offline lab
davinci_all_defconfig:
da850-lcdk: 1 offline lab
dm365evm,legacy: 1 offline lab
mvebu_v7_defconfig:
armada-xp-openblocks-ax3-4: 1 offline lab
arm64:
defconfig:
apq8016-sbc: 1 offline lab
meson-gxbb-odroidc2: 1 offline lab
meson-gxbb-p200: 1 offline lab
meson-gxl-s905d-p230: 1 offline lab
meson-gxl-s905x-khadas-vim: 1 offline lab
meson-gxl-s905x-nexbox-a95x: 1 offline lab
meson-gxl-s905x-p212: 1 offline lab
mt7622-rfb1: 1 offline lab
rk3399-firefly: 1 offline lab
sun50i-a64-pine64-plus: 1 offline lab---
For more info write to <[email protected]>
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.31 release. There are 101 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 Thu Mar 29 16:27:21 UTC 2018. Anything received after that time might be too late.
Build results:
total: 145 pass: 145 fail: 0 Qemu test results: total: 141 pass: 141 fail: 0
Guenter
Greg Kroah-Hartman
maintainerQuoted 6 lines · Guenter Roeck
Quoted 5 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.31 release. There are 101 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.
Quoted 8 lines
Quoted 3 lines
Responses should be made by Thu Mar 29 16:27:21 UTC 2018. Anything received after that time might be too late.
Build results:
total: 145 pass: 145 fail: 0 Qemu test results: total: 141 pass: 141 fail: 0
Thanks for testing all of these and letting me know.
greg k-h
Quoted 7 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.14.31 release. There are 101 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 Thu Mar 29 16:27:21 UTC 2018. Anything received after that time might be too late.
Results from Linaro’s test farm.
No regressions on arm64 and x86_64.
There is a regression on arm32 in libhugetlbfs/truncate_above_4GB-2M-32 that also exists in 4.15 and mainline. We'll investigate the root cause and report upstream in mainline. I suspect the cause is "hugetlbfs: check for pgoff value overflow", but have not verified yet.
Summary
------------------------------------------------------------------------
kernel: 4.14.31-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: 2d6120998a4cef5d352e931b1c85f1e3d79081d5
git describe: v4.14.30-102-g2d6120998a4c
Test details: https://qa-reports.linaro.org/lkft/linux-stable-rc-4.14-oe/build/v4.14.30-102-g2d6120998a4c
Regressions (compared to build v4.14.30)
------------------------------------------------------------------------
x15 - arm:
libhugetlbfs:
* truncate_above_4GB-2M-32
* test src: git://github.com/libhugetlbfs/libhugetlbfs.git
Boards, architectures and test suites:
-------------------------------------
dragonboard-410c
* boot - fail: 2, pass: 18
* kselftest - skip: 20, pass: 45
* libhugetlbfs - skip: 1, pass: 90
* ltp-cap_bounds-tests - pass: 2
* ltp-containers-tests - skip: 17, pass: 64
* ltp-fcntl-locktests-tests - pass: 2
* ltp-fs-tests - skip: 2, pass: 61
* ltp-fs_bind-tests - pass: 2
* ltp-fs_perms_simple-tests - pass: 19
* ltp-fsx-tests - pass: 2
* ltp-hugetlb-tests - skip: 1, pass: 21
* ltp-io-tests - pass: 3
* ltp-math-tests - pass: 11
* ltp-nptl-tests - pass: 2
* ltp-pty-tests - pass: 4
* ltp-sched-tests - pass: 14
* ltp-securebits-tests - pass: 4
* ltp-syscalls-tests - skip: 148, pass: 1002
* ltp-timers-tests - skip: 1, pass: 12
hi6220-hikey - arm64
* boot - pass: 20
* kselftest - skip: 17, pass: 48
* libhugetlbfs - skip: 1, pass: 90
* ltp-cap_bounds-tests - pass: 2
* ltp-containers-tests - skip: 17, pass: 64
* ltp-fcntl-locktests-tests - pass: 2
* ltp-filecaps-tests - pass: 2
* ltp-fs-tests - skip: 2, pass: 61
* ltp-fs_bind-tests - pass: 2
* ltp-fs_perms_simple-tests - pass: 19
* ltp-fsx-tests - pass: 2
* ltp-hugetlb-tests - skip: 1, pass: 21
* ltp-io-tests - pass: 3
* ltp-ipc-tests - pass: 9
* ltp-math-tests - pass: 11
* ltp-nptl-tests - pass: 2
* ltp-pty-tests - pass: 4
* ltp-sched-tests - skip: 4, pass: 10
* ltp-securebits-tests - pass: 4
* ltp-syscalls-tests - skip: 151, pass: 999
* ltp-timers-tests - skip: 1, pass: 12
juno-r2 - arm64
* boot - pass: 20
* kselftest - skip: 17, pass: 48
* libhugetlbfs - skip: 1, pass: 90
* ltp-cap_bounds-tests - pass: 2
* ltp-containers-tests - skip: 17, pass: 64
* ltp-fcntl-locktests-tests - pass: 2
* ltp-filecaps-tests - pass: 2
* ltp-fs-tests - skip: 2, pass: 61
* ltp-fs_bind-tests - pass: 2
* ltp-fs_perms_simple-tests - pass: 19
* ltp-fsx-tests - pass: 2
* ltp-hugetlb-tests - pass: 22
* ltp-io-tests - pass: 3
* ltp-ipc-tests - pass: 9
* ltp-math-tests - pass: 11
* ltp-nptl-tests - pass: 2
* ltp-pty-tests - pass: 4
* ltp-sched-tests - skip: 4, pass: 10
* ltp-securebits-tests - pass: 4
* ltp-syscalls-tests - skip: 149, pass: 1001
* ltp-timers-tests - skip: 1, pass: 12
qemu_x86_64
* boot - pass: 22
* kselftest - skip: 22, pass: 58
* kselftest-vsyscall-mode-native - skip: 22, pass: 58
* kselftest-vsyscall-mode-none - skip: 22, pass: 58
* libhugetlbfs - skip: 1, pass: 90
* ltp-cap_bounds-tests - pass: 2
* ltp-containers-tests - skip: 17, pass: 64
* ltp-fcntl-locktests-tests - pass: 2
* ltp-filecaps-tests - pass: 2
* ltp-fs-tests - skip: 6, pass: 57
* ltp-fs_bind-tests - pass: 2
* ltp-fs_perms_simple-tests - pass: 19
* ltp-fsx-tests - pass: 2
* ltp-hugetlb-tests - pass: 22
* ltp-io-tests - pass: 3
* ltp-ipc-tests - pass: 9
* ltp-math-tests - pass: 11
* ltp-nptl-tests - pass: 2
* ltp-pty-tests - pass: 4
* ltp-sched-tests - skip: 1, pass: 13
* ltp-securebits-tests - pass: 4
* ltp-timers-tests - skip: 1, pass: 12
x15 - arm
* boot - pass: 20
* kselftest - skip: 19, fail: 2, pass: 41
* libhugetlbfs - skip: 1, fail: 1, pass: 86
* ltp-cap_bounds-tests - pass: 2
* ltp-containers-tests - skip: 17, fail: 2, pass: 62
* ltp-fcntl-locktests-tests - pass: 2
* ltp-filecaps-tests - pass: 2
* ltp-fs-tests - skip: 2, pass: 61
* ltp-fs_bind-tests - pass: 2
* ltp-fs_perms_simple-tests - pass: 19
* ltp-fsx-tests - pass: 2
* ltp-hugetlb-tests - skip: 2, pass: 20
* ltp-io-tests - pass: 3
* ltp-ipc-tests - pass: 9
* ltp-math-tests - pass: 11
* ltp-nptl-tests - pass: 2
* ltp-pty-tests - pass: 4
* ltp-sched-tests - skip: 1, pass: 13
* ltp-securebits-tests - pass: 4
* ltp-syscalls-tests - skip: 97, pass: 1053
* ltp-timers-tests - skip: 1, pass: 12
x86_64
* boot - pass: 22
* kselftest - skip: 19, pass: 61
* kselftest-vsyscall-mode-native - skip: 19, pass: 61
* kselftest-vsyscall-mode-none - skip: 19, fail: 1, pass: 60
* libhugetlbfs - skip: 1, pass: 90
* ltp-cap_bounds-tests - pass: 2
* ltp-containers-tests - skip: 17, pass: 64
* ltp-fcntl-locktests-tests - pass: 2
* ltp-filecaps-tests - pass: 2
* ltp-fs-tests - skip: 1, pass: 62
* ltp-fs_bind-tests - pass: 2
* ltp-fs_perms_simple-tests - pass: 19
* ltp-fsx-tests - pass: 2
* ltp-hugetlb-tests - pass: 22
* ltp-io-tests - pass: 3
* ltp-ipc-tests - pass: 9
* ltp-math-tests - pass: 11
* ltp-nptl-tests - pass: 2
* ltp-pty-tests - pass: 4
* ltp-sched-tests - skip: 5, pass: 9
* ltp-securebits-tests - pass: 4
* ltp-syscalls-tests - skip: 119, pass: 1031
* ltp-timers-tests - skip: 1, pass: 12
--
Linaro QA (beta)
https://qa-reports.linaro.org