5.4.17-stable review
Subsystems (3)
Greg Kroah-Hartman
maintainercover 5.4.17-stable review
This is the start of the stable review cycle for the 5.4.17 release. There are 110 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat, 01 Feb 2020 18:35:06 +0000. 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/v5.x/stable-review/patch-5.4.17-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-5.4.y
and the diffstat can be found below.
thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:
Greg Kroah-Hartman <[email protected]>
Linux 5.4.17-rc1
Paul Cercueil <[email protected]>
power/supply: ingenic-battery: Don't change scale if there's only one
Johannes Berg <[email protected]>
Revert "um: Enable CONFIG_CONSTRUCTORS"
Andrew Murray <[email protected]>
KVM: arm64: Write arch.mdcr_el2 changes since last vcpu_load on VHE
Herbert Xu <[email protected]>
crypto: pcrypt - Fix user-after-free on module unload
Iuliana Prodan <[email protected]>
crypto: caam - do not reset pointer size from MCFGR register
Daniel Axtens <[email protected]>
crypto: vmx - reject xts inputs that are too short
Herbert Xu <[email protected]>
crypto: af_alg - Use bh_lock_sock in sk_destruct
Johan Hovold <[email protected]>
rsi: fix non-atomic allocation in completion handler
Johan Hovold <[email protected]>
rsi: fix memory leak on failed URB submission
Johan Hovold <[email protected]>
rsi: fix use-after-free on probe errors
Johan Hovold <[email protected]>
rsi: fix use-after-free on failed probe and unbind
Tony Lindgren <[email protected]>
bus: ti-sysc: Fix missing force mstandby quirk handling
Andre Heider <[email protected]>
Bluetooth: btbcm: Use the BDADDR_PROPERTY quirk
Marcel Holtmann <[email protected]>
Bluetooth: Allow combination of BDADDR_PROPERTY and INVALID_BDADDR quirks
Hui Wang <[email protected]>
ALSA: hda/realtek - Move some alc236 pintbls to fallback table
Laura Abbott <[email protected]>
usb-storage: Disable UAS on JMicron SATA enclosure
Tony Lindgren <[email protected]>
bus: ti-sysc: Add module enable quirk for audio AESS
Adrian Hunter <[email protected]>
mmc: sdhci-pci: Add support for Intel JSL
Raul E Rangel <[email protected]>
mmc: sdhci-pci: Quirk for AMD SDHC Device 0x7906
Ben Dooks <[email protected]>
ARM: OMAP2+: SmartReflex: add omap_sr_pdata definition
Joel Stanley <[email protected]>
ARM: config: aspeed-g5: Enable 8250_DW quirks
Andy Shevchenko <[email protected]>
mfd: intel-lpss: Add Intel Comet Lake PCH-H PCI IDs
Joakim Zhang <[email protected]>
perf/imx_ddr: Add enhanced AXI ID filter support
Logan Gunthorpe <[email protected]>
iommu/amd: Support multiple PCI DMA aliases in IRQ Remapping
Logan Gunthorpe <[email protected]>
iommu/amd: Support multiple PCI DMA aliases in device table
Jarkko Nikula <[email protected]>
spi: pxa2xx: Add support for Intel Comet Lake-H
Tony Lindgren <[email protected]>
bus: ti-sysc: Use swsup quirks also for am335x musb
Tony Lindgren <[email protected]>
bus: ti-sysc: Handle mstandby quirk and use it for musb
Thomas Voegtle <[email protected]>
media: dvbsky: add support for eyeTV Geniatech T2 lite
Slawomir Pawlowski <[email protected]>
PCI: Add DMA alias quirk for Intel VCA NTB
Pacien TRAN-GIRARD <[email protected]>
platform/x86: dell-laptop: disable kbd backlight on Inspiron 10xx
Sergio Paracuellos <[email protected]>
staging: mt7621-pci: add quirks for 'E2' revision using 'soc_device_attribute'
Andrii Nakryiko <[email protected]>
libbpf: Fix BTF-defined map's __type macro handling of arrays
Jiange Zhao <[email protected]>
drm/amdgpu/SRIOV: add navi12 pci id for SRIOV (v2)
Sam McNally <[email protected]>
ASoC: Intel: cht_bsw_rt5645: Add quirk for boards using pmc_plt_clk_0
Yauhen Kharuzhy <[email protected]>
extcon-intel-cht-wc: Don't reset USB data connection at probe
Rodrigo Rivas Costa <[email protected]>
HID: steam: Fix input device disappearing
Arnd Bergmann <[email protected]>
atm: eni: fix uninitialized variable warning
Jiping Ma <[email protected]>
stmmac: debugfs entry name is not be changed when udev rename device name.
Thomas Anderson <[email protected]>
drm/amd/display: Reduce HDMI pixel encoding if max clock is exceeded
Qian Cai <[email protected]>
iommu/dma: fix variable 'cookie' set but not used
Dmitry Osipenko <[email protected]>
gpio: max77620: Add missing dependency on GPIOLIB_IRQCHIP
Krzysztof Kozlowski <[email protected]>
net: wan: sdla: Fix cast from pointer to integer of different size
Fenghua Yu <[email protected]>
drivers/net/b44: Change to non-atomic bit operations on pwol_mask
Liran Alon <[email protected]>
net: Google gve: Remove dma_wmb() before ringing doorbell
wuxu.wu <[email protected]>
spi: spi-dw: Add lock protect dw_spi rx/tx to prevent concurrent calls
Sean Nyekjaer <[email protected]>
can: tcan4x5x: tcan4x5x_parse_config(): reset device before register access
Paul Cercueil <[email protected]>
usb: musb: jz4740: Silence error if code is -EPROBE_DEFER
Russell King <[email protected]>
watchdog: orion: fix platform_get_irq() complaints
Andreas Kemnade <[email protected]>
watchdog: rn5t618_wdt: fix module aliases
David Engraf <[email protected]>
watchdog: max77620_wdt: fix potential build errors
Srinivas Pandruvada <[email protected]>
HID: intel-ish-hid: ipc: Add Tiger Lake PCI device ID
Tony Lindgren <[email protected]>
phy: cpcap-usb: Prevent USB line glitches from waking up modem
Dragos Tarcatu <[email protected]>
ASoC: topology: Prevent use-after-free in snd_soc_get_pcm_runtime()
Chuhong Yuan <[email protected]>
ASoC: fsl_audmix: add missed pm_runtime_disable
Bjorn Andersson <[email protected]>
phy: qcom-qmp: Increase PHY ready timeout
Pan Zhang <[email protected]>
drivers/hid/hid-multitouch.c: fix a possible null pointer access.
Pierre-Louis Bossart <[email protected]>
ASoC: SOF: Intel: hda: hda-dai: fix oops on hda_link .hw_free
Kai Vehmanen <[email protected]>
ASoC: SOF: fix fault at driver unload after failed probe
Jason Gerecke <[email protected]>
HID: wacom: Recognize new MobileStudio Pro PID
Even Xu <[email protected]>
HID: intel-ish-hid: ipc: add CMP device id
Pavel Balan <[email protected]>
HID: Add quirk for incorrect input length on Lenovo Y720
Hans de Goede <[email protected]>
HID: asus: Ignore Asus vendor-page usage-code 0xff events
Hans de Goede <[email protected]>
HID: ite: Add USB id match for Acer SW5-012 keyboard dock
Priit Laes <[email protected]>
HID: Add quirk for Xin-Mo Dual Controller
Randy Dunlap <[email protected]>
arc: eznps: fix allmodconfig kconfig warning
Aaron Ma <[email protected]>
HID: multitouch: Add LG MELF0410 I2C touchscreen support
David Howells <[email protected]>
rxrpc: Fix use-after-free in rxrpc_receive_data()
Stephen Worley <[email protected]>
net: include struct nhmsg size in nh nlmsg size
Christophe JAILLET <[email protected]>
mlxsw: minimal: Fix an error handling path in 'mlxsw_m_port_create()'
Willem de Bruijn <[email protected]>
udp: segment looped gso packets correctly
Lorenzo Bianconi <[email protected]>
net: socionext: fix xdp_result initialization in netsec_process_rx
Lorenzo Bianconi <[email protected]>
net: socionext: fix possible user-after-free in netsec_process_rx
Cong Wang <[email protected]>
net_sched: walk through all child classes in tc_bind_tclass()
Cong Wang <[email protected]>
net_sched: fix ops->bind_class() implementations
Eric Dumazet <[email protected]>
net_sched: ematch: reject invalid TCF_EM_SIMPLE
Johan Hovold <[email protected]>
zd1211rw: fix storage endpoint lookup
Johan Hovold <[email protected]>
rtl8xxxu: fix interface sanity check
Johan Hovold <[email protected]>
brcmfmac: fix interface sanity check
Johan Hovold <[email protected]>
ath9k: fix storage endpoint lookup
Paulo Alcantara (SUSE) <[email protected]>
cifs: Fix memory allocation in __smb2_handle_cancelled_cmd()
Ronnie Sahlberg <[email protected]>
cifs: set correct max-buffer-size for smb2_ioctl_init()
Vincent Whitchurch <[email protected]>
CIFS: Fix task struct use-after-free on reconnect
Eric Biggers <[email protected]>
crypto: chelsio - fix writing tfm flags to wrong place
Guenter Roeck <[email protected]>
driver core: Fix test_async_driver_probe if NUMA is disabled
Andy Shevchenko <[email protected]>
iio: st_gyro: Correct data for LSM9DS0 gyro
Olivier Moysan <[email protected]>
iio: adc: stm32-dfsdm: fix single conversion
Tomas Winkler <[email protected]>
mei: me: add comet point (lake) H device ids
Tomas Winkler <[email protected]>
mei: hdcp: bind only with i915 on the same PCH
Martin Fuzzey <[email protected]>
binder: fix log spam for existing debugfs file creation.
Lubomir Rintel <[email protected]>
component: do not dereference opaque pointer in debugfs
Eric Snowberg <[email protected]>
debugfs: Return -EPERM when locked down
Uwe Kleine-König <[email protected]>
serial: imx: fix a race condition in receive path
Lukas Wunner <[email protected]>
serial: 8250_bcm2835aux: Fix line mismatch on driver unbind
Malcolm Priestley <[email protected]>
staging: vt6656: Fix false Tx excessive retries reporting.
Malcolm Priestley <[email protected]>
staging: vt6656: use NULLFUCTION stack on mac80211
Malcolm Priestley <[email protected]>
staging: vt6656: correct packet types for CTS protect, mode.
Colin Ian King <[email protected]>
staging: wlan-ng: ensure error return is actually returned
Andrey Shvetsov <[email protected]>
staging: most: net: fix buffer overflow
Thomas Hebb <[email protected]>
usb: typec: fusb302: fix "op-sink-microwatt" default that was in mW
Thomas Hebb <[email protected]>
usb: typec: wcove: fix "op-sink-microwatt" default that was in mW
Bin Liu <[email protected]>
usb: dwc3: turn off VBUS when leaving host mode
Johan Hovold <[email protected]>
USB: serial: ir-usb: fix IrLAP framing
Johan Hovold <[email protected]>
USB: serial: ir-usb: fix link-speed handling
Johan Hovold <[email protected]>
USB: serial: ir-usb: add missing endpoint sanity check
Peter Robinson <[email protected]>
usb: host: xhci-tegra: set MODULE_FIRMWARE for tegra186
Heikki Krogerus <[email protected]>
usb: dwc3: pci: add ID for the Intel Comet Lake -V variant
Johan Hovold <[email protected]>
rsi_91x_usb: fix interface sanity check
Johan Hovold <[email protected]>
orinoco_usb: fix interface sanity check
Johan Hovold <[email protected]>
Bluetooth: btusb: fix non-atomic allocation in completion handler
-------------
Diffstat:
Makefile | 4 +- arch/arc/plat-eznps/Kconfig | 2 +- arch/arm/configs/aspeed_g5_defconfig | 1 + arch/arm64/kvm/debug.c | 6 +- arch/um/include/asm/common.lds.S | 2 +- arch/um/kernel/dyn.lds.S | 1 + crypto/af_alg.c | 6 +- crypto/pcrypt.c | 3 +- drivers/android/binder.c | 37 ++--- drivers/atm/eni.c | 4 +- drivers/base/component.c | 8 +- drivers/base/test/test_async_driver_probe.c | 3 +- drivers/bluetooth/btbcm.c | 6 + drivers/bluetooth/btusb.c | 2 +- drivers/bus/ti-sysc.c | 27 +++- drivers/crypto/caam/ctrl.c | 6 +- drivers/crypto/chelsio/chcr_algo.c | 16 +- drivers/crypto/vmx/aes_xts.c | 3 + drivers/extcon/extcon-intel-cht-wc.c | 16 +- drivers/gpio/Kconfig | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 1 + drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 45 +++--- drivers/hid/hid-asus.c | 3 +- drivers/hid/hid-ids.h | 3 + drivers/hid/hid-ite.c | 3 + drivers/hid/hid-multitouch.c | 5 +- drivers/hid/hid-quirks.c | 1 + drivers/hid/hid-steam.c | 4 + drivers/hid/i2c-hid/i2c-hid-core.c | 16 +- drivers/hid/intel-ish-hid/ipc/hw-ish.h | 2 + drivers/hid/intel-ish-hid/ipc/pci-ish.c | 2 + drivers/hid/wacom_wac.c | 6 +- drivers/iio/adc/stm32-dfsdm-adc.c | 2 + drivers/iio/gyro/st_gyro_core.c | 75 ++++++++- drivers/iommu/amd_iommu.c | 170 +++++++++++---------- drivers/iommu/amd_iommu_types.h | 2 +- drivers/iommu/dma-iommu.c | 3 - drivers/media/usb/dvb-usb-v2/dvbsky.c | 3 + drivers/mfd/intel-lpss-pci.c | 13 +- drivers/misc/mei/hdcp/mei_hdcp.c | 33 +++- drivers/misc/mei/hw-me-regs.h | 4 + drivers/misc/mei/pci-me.c | 2 + drivers/mmc/host/sdhci-pci-core.c | 53 ++++++- drivers/mmc/host/sdhci-pci.h | 2 + drivers/net/can/m_can/tcan4x5x.c | 27 +++- drivers/net/ethernet/broadcom/b44.c | 9 +- drivers/net/ethernet/google/gve/gve_rx.c | 2 - drivers/net/ethernet/google/gve/gve_tx.c | 6 - drivers/net/ethernet/mellanox/mlxsw/minimal.c | 2 +- drivers/net/ethernet/socionext/netsec.c | 4 +- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 32 ++++ drivers/net/wan/sdla.c | 2 +- drivers/net/wireless/ath/ath9k/hif_usb.c | 2 +- .../net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 4 +- .../net/wireless/intersil/orinoco/orinoco_usb.c | 4 +- .../net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c | 2 +- drivers/net/wireless/rsi/rsi_91x_hal.c | 12 +- drivers/net/wireless/rsi/rsi_91x_usb.c | 37 +++-- drivers/net/wireless/zydas/zd1211rw/zd_usb.c | 2 +- drivers/pci/quirks.c | 34 +++++ drivers/perf/fsl_imx8_ddr_perf.c | 63 +++++--- drivers/phy/motorola/phy-cpcap-usb.c | 18 ++- drivers/phy/qualcomm/phy-qcom-qmp.c | 2 +- drivers/platform/x86/dell-laptop.c | 26 ++++ drivers/power/supply/ingenic-battery.c | 15 +- drivers/spi/spi-dw.c | 15 +- drivers/spi/spi-dw.h | 1 + drivers/spi/spi-pxa2xx.c | 4 + drivers/staging/most/net/net.c | 10 ++ drivers/staging/mt7621-pci/pci-mt7621.c | 23 ++- drivers/staging/vt6656/device.h | 2 + drivers/staging/vt6656/int.c | 6 +- drivers/staging/vt6656/main_usb.c | 1 + drivers/staging/vt6656/rxtx.c | 26 ++-- drivers/staging/wlan-ng/prism2mgmt.c | 2 +- drivers/tty/serial/8250/8250_bcm2835aux.c | 2 +- drivers/tty/serial/imx.c | 51 +++++-- drivers/usb/dwc3/core.c | 3 + drivers/usb/dwc3/dwc3-pci.c | 4 + drivers/usb/host/xhci-tegra.c | 1 + drivers/usb/musb/jz4740.c | 7 +- drivers/usb/serial/ir-usb.c | 136 +++++++++++++---- drivers/usb/storage/unusual_uas.h | 7 +- drivers/usb/typec/tcpm/fusb302.c | 2 +- drivers/usb/typec/tcpm/wcove.c | 2 +- drivers/watchdog/Kconfig | 1 + drivers/watchdog/orion_wdt.c | 4 +- drivers/watchdog/rn5t618_wdt.c | 1 + fs/cifs/cifsglob.h | 1 + fs/cifs/smb2misc.c | 2 +- fs/cifs/smb2ops.c | 9 +- fs/cifs/smb2transport.c | 2 + fs/cifs/transport.c | 3 + fs/debugfs/file.c | 17 ++- include/linux/platform_data/ti-sysc.h | 2 + include/linux/power/smartreflex.h | 3 + include/linux/usb/irda.h | 13 +- include/media/dvb-usb-ids.h | 1 + include/net/pkt_cls.h | 33 ++-- include/net/sch_generic.h | 3 +- include/net/udp.h | 3 + init/Kconfig | 1 + kernel/gcov/Kconfig | 2 +- net/bluetooth/hci_core.c | 26 +++- net/ipv4/nexthop.c | 4 +- net/rxrpc/input.c | 12 +- net/sched/cls_basic.c | 11 +- net/sched/cls_bpf.c | 11 +- net/sched/cls_flower.c | 11 +- net/sched/cls_fw.c | 11 +- net/sched/cls_matchall.c | 11 +- net/sched/cls_route.c | 11 +- net/sched/cls_rsvp.h | 11 +- net/sched/cls_tcindex.c | 11 +- net/sched/cls_u32.c | 11 +- net/sched/ematch.c | 3 + net/sched/sch_api.c | 47 ++++-- sound/pci/hda/patch_realtek.c | 17 +-- sound/soc/fsl/fsl_audmix.c | 9 +- sound/soc/intel/boards/cht_bsw_rt5645.c | 26 +++- sound/soc/soc-topology.c | 6 +- sound/soc/sof/intel/hda-dai.c | 11 +- sound/soc/sof/ipc.c | 3 + tools/testing/selftests/bpf/bpf_helpers.h | 2 +- .../selftests/bpf/progs/test_get_stack_rawtp.c | 3 +- 125 files changed, 1162 insertions(+), 424 deletions(-)
Greg Kroah-Hartman
maintainerPATCH 1/110 Bluetooth: btusb: fix non-atomic allocation in completion handler
From: Johan Hovold <[email protected]>
commit 22cc6b7a1dbb58da4afc539d9b7d470b23a25eea upstream.
USB completion handlers are called in atomic context and must specifically not allocate memory using GFP_KERNEL.
--- drivers/bluetooth/btusb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/bluetooth/btusb.c +1 -1
@@ -2585,7 +2585,7 @@ static void btusb_mtk_wmt_recv(struct ur* and being processed the events from there then. */ if (test_bit (BTUSB_TX_WAIT_VND_EVT , &data->flags)) { - data->evt_skb =skb_clone (skb,GFP_KERNEL ); + data->evt_skb =skb_clone (skb,GFP_ATOMIC );if (!data->evt_skb) goto err_out; }
Greg Kroah-Hartman
maintainerPATCH 2/110 orinoco_usb: fix interface sanity check
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit b73e05aa543cf8db4f4927e36952360d71291d41 upstream.
Make sure to use the current alternate setting when verifying the interface descriptors to avoid binding to an invalid interface.
Failing to do so could cause the driver to misbehave or trigger a WARN() in usb_submit_urb() that kernels with panic_on_warn set would choke on.
--- drivers/net/wireless/intersil/orinoco/orinoco_usb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/wireless/intersil/orinoco/orinoco_usb.c +2 -2
@@ -1608,9 +1608,9 @@ static int ezusb_probe(struct usb_interf/* set up the endpoint information */ /* check out the endpoints */ - iface_desc = &interface->altsetting[0 ].desc; + iface_desc = &interface->cur_altsetting->desc;for (i =0 ; i < iface_desc->bNumEndpoints; ++i) { - ep = &interface->altsetting[0 ].endpoint[i].desc; + ep = &interface->cur_altsetting->endpoint[i].desc;if (usb_endpoint_is_bulk_in (ep)) {/* we found a bulk in endpoint */
Greg Kroah-Hartman
maintainerPATCH 3/110 rsi_91x_usb: fix interface sanity check
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 3139b180906af43bc09bd3373fc2338a8271d9d9 upstream.
Make sure to use the current alternate setting when verifying the interface descriptors to avoid binding to an invalid interface.
Failing to do so could cause the driver to misbehave or trigger a WARN() in usb_submit_urb() that kernels with panic_on_warn set would choke on.
--- drivers/net/wireless/rsi/rsi_91x_usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/rsi/rsi_91x_usb.c +1 -1
@@ -117,7 +117,7 @@ static int rsi_find_bulk_in_and_out_endp__le16 buffer_size;int ii, bin_found =0 , bout_found =0 ; - iface_desc = &(interface->altsetting[0 ]); + iface_desc = interface->cur_altsetting;for (ii =0 ; ii < iface_desc->desc.bNumEndpoints; ++ii) { endpoint = &(iface_desc->endpoint[ii].desc);
Greg Kroah-Hartman
maintainerPATCH 4/110 usb: dwc3: pci: add ID for the Intel Comet Lake -V variant
In reply to Greg Kroah-HartmanFrom: Heikki Krogerus <[email protected]>
commit f5ae8869095552e3396ee3e404f9586cc6a828f0 upstream.
There is one more Comet Lake PCH variant, CML-V, that has its own PCI ID.
--- drivers/usb/dwc3/dwc3-pci.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/usb/dwc3/dwc3-pci.c +4 -0
@@ -34,6 +34,7 @@#define PCI_DEVICE_ID_INTEL_GLK 0x31aa#define PCI_DEVICE_ID_INTEL_CNPLP 0x9dee#define PCI_DEVICE_ID_INTEL_CNPH 0xa36e +#define PCI_DEVICE_ID_INTEL_CNPV 0xa3b0#define PCI_DEVICE_ID_INTEL_ICLLP 0x34ee#define PCI_DEVICE_ID_INTEL_EHLLP 0x4b7e#define PCI_DEVICE_ID_INTEL_TGPLP 0xa0ee
@@ -342,6 +343,9 @@ static const struct pci_device_id dwc3_p {PCI_VDEVICE (INTEL ,PCI_DEVICE_ID_INTEL_CNPH ), (kernel_ulong_t) &dwc3_pci_intel_properties, }, + { PCI_VDEVICE(INTEL, PCI_DEVICE_ID_INTEL_CNPV), + (kernel_ulong_t) &dwc3_pci_intel_properties, }, + { PCI_VDEVICE(INTEL, PCI_DEVICE_ID_INTEL_ICLLP), (kernel_ulong_t) &dwc3_pci_intel_properties, },
Greg Kroah-Hartman
maintainerPATCH 5/110 usb: host: xhci-tegra: set MODULE_FIRMWARE for tegra186
In reply to Greg Kroah-HartmanFrom: Peter Robinson <[email protected]>
commit e1f236efd9c579a29d7df75aa052127d0d975267 upstream.
Set the MODULE_FIRMWARE for tegra186, it's registered for 124/210 and ensures the firmware is available at the appropriate time such as in the initrd, else if the firmware is unavailable the driver fails with the following errors:
tegra-xusb 3530000.usb: Direct firmware load for nvidia/tegra186/xusb.bin failed with error -2 tegra-xusb 3530000.usb: failed to request firmware: -2
tegra-xusb 3530000.usb: failed to load firmware: -2
tegra-xusb: probe of 3530000.usb failed with error -2
--- drivers/usb/host/xhci-tegra.c | 1 + 1 file changed, 1 insertion(+)
drivers/usb/host/xhci-tegra.c +1 -0
@@ -1413,6 +1413,7 @@ MODULE_FIRMWARE("nvidia/tegra210/xusb.bistatic const char *const tegra186_supply_names[] = { }; +MODULE_FIRMWARE ("nvidia/tegra186/xusb.bin" );static const struct tegra_xusb_phy_type tegra186_phy_types[] = { { .name ="usb3" , .num =3 , },
Greg Kroah-Hartman
maintainerPATCH 6/110 USB: serial: ir-usb: add missing endpoint sanity check
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 2988a8ae7476fe9535ab620320790d1714bdad1d upstream.
Add missing endpoint sanity check to avoid dereferencing a NULL-pointer on open() in case a device lacks a bulk-out endpoint.
Note that prior to commit f4a4cbb2047e ("USB: ir-usb: reimplement using generic framework") the oops would instead happen on open() if the device lacked a bulk-in endpoint and on write() if it lacked a bulk-out endpoint.
--- drivers/usb/serial/ir-usb.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/usb/serial/ir-usb.c +3 -0
@@ -195,6 +195,9 @@ static int ir_startup(struct usb_serialstruct usb_irda_cs_descriptor *irda_desc;int rates; +if (serial->num_bulk_in <1 || serial->num_bulk_out <1 ) +return -ENODEV ; + irda_desc =irda_usb_find_class_desc (serial,0 );if (!irda_desc) { dev_err(&serial->dev->dev,
Greg Kroah-Hartman
maintainerPATCH 7/110 USB: serial: ir-usb: fix link-speed handling
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 17a0184ca17e288decdca8b2841531e34d49285f upstream.
Commit e0d795e4f36c ("usb: irda: cleanup on ir-usb module") added a USB IrDA header with common defines, but mistakingly switched to using the class-descriptor baud-rate bitmask values for the outbound header.
This broke link-speed handling for rates above 9600 baud, but a device would also be able to operate at the default 9600 baud until a link-speed request was issued (e.g. using the TCGETS ioctl).
--- drivers/usb/serial/ir-usb.c | 20 ++++++++++---------- include/linux/usb/irda.h | 13 ++++++++++++- 2 files changed, 22 insertions(+), 11 deletions(-)
drivers/usb/serial/ir-usb.c +10 -10
@@ -335,34 +335,34 @@ static void ir_set_termios(struct tty_stswitch (baud) {case 2400 : - ir_baud =USB_IRDA_BR_2400 ; + ir_baud =USB_IRDA_LS_2400 ;break ;case 9600 : - ir_baud =USB_IRDA_BR_9600 ; + ir_baud =USB_IRDA_LS_9600 ;break ;case 19200 : - ir_baud =USB_IRDA_BR_19200 ; + ir_baud =USB_IRDA_LS_19200 ;break ;case 38400 : - ir_baud =USB_IRDA_BR_38400 ; + ir_baud =USB_IRDA_LS_38400 ;break ;case 57600 : - ir_baud =USB_IRDA_BR_57600 ; + ir_baud =USB_IRDA_LS_57600 ;break ;case 115200 : - ir_baud =USB_IRDA_BR_115200 ; + ir_baud =USB_IRDA_LS_115200 ;break ;case 576000 : - ir_baud =USB_IRDA_BR_576000 ; + ir_baud =USB_IRDA_LS_576000 ;break ;case 1152000 : - ir_baud =USB_IRDA_BR_1152000 ; + ir_baud =USB_IRDA_LS_1152000 ;break ;case 4000000 : - ir_baud =USB_IRDA_BR_4000000 ; + ir_baud =USB_IRDA_LS_4000000 ;break ;default : - ir_baud =USB_IRDA_BR_9600 ; + ir_baud =USB_IRDA_LS_9600 ; baud =9600 ; }
include/linux/usb/irda.h +12 -1
@@ -119,11 +119,22 @@ struct usb_irda_cs_descriptor {* 6 - 115200 bps * 7 - 576000 bps * 8 - 1.152 Mbps -* 9 - 5 mbps +* 9 - 4 Mbps * 10..15 - Reserved */ #define USB_IRDA_STATUS_LINK_SPEED 0x0f +#define USB_IRDA_LS_NO_CHANGE 0 +#define USB_IRDA_LS_2400 1 +#define USB_IRDA_LS_9600 2 +#define USB_IRDA_LS_19200 3 +#define USB_IRDA_LS_38400 4 +#define USB_IRDA_LS_57600 5 +#define USB_IRDA_LS_115200 6 +#define USB_IRDA_LS_576000 7 +#define USB_IRDA_LS_1152000 8 +#define USB_IRDA_LS_4000000 9 +/* The following is a 4-bit value used only for * outbound header: *
Greg Kroah-Hartman
maintainerPATCH 8/110 USB: serial: ir-usb: fix IrLAP framing
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 38c0d5bdf4973f9f5a888166e9d3e9ed0d32057a upstream.
Commit f4a4cbb2047e ("USB: ir-usb: reimplement using generic framework") switched to using the generic write implementation which may combine multiple write requests into larger transfers. This can break the IrLAP protocol where end-of-frame is determined using the USB short packet mechanism, for example, if multiple frames are sent in rapid succession.
--- drivers/usb/serial/ir-usb.c | 113 +++++++++++++++++++++++++++++++++++--------- 1 file changed, 91 insertions(+), 22 deletions(-)
drivers/usb/serial/ir-usb.c +91 -22
@@ -45,9 +45,10 @@ static int buffer_size;static int xbof =-1 ;static int ir_startup (struct usb_serial *serial); -static int ir_open (struct tty_struct *tty,struct usb_serial_port *port); -static int ir_prepare_write_buffer (struct usb_serial_port *port, -void *dest,size_t size); +static int ir_write (struct tty_struct *tty,struct usb_serial_port *port, +const unsigned char *buf,int count); +static int ir_write_room (struct tty_struct *tty); +static void ir_write_bulk_callback (struct urb *urb);static void ir_process_read_urb (struct urb *urb);static void ir_set_termios (struct tty_struct *tty,struct usb_serial_port *port,struct ktermios *old_termios);
@@ -77,8 +78,9 @@ static struct usb_serial_driver ir_devic .num_ports = 1, .set_termios = ir_set_termios, .attach = ir_startup, - .open = ir_open, - .prepare_write_buffer = ir_prepare_write_buffer, + .write = ir_write, + .write_room = ir_write_room, + .write_bulk_callback = ir_write_bulk_callback, .process_read_urb = ir_process_read_urb, };
@@ -254,35 +256,102 @@ static int ir_startup(struct usb_serialreturn 0 ; } -static int ir_open (struct tty_struct *tty,struct usb_serial_port *port) +static int ir_write (struct tty_struct *tty,struct usb_serial_port *port, +const unsigned char *buf,int count) { -int i; +struct urb *urb =NULL ; +unsigned long flags; +int ret; -for (i =0 ; i <ARRAY_SIZE (port->write_urbs); ++i) - port->write_urbs[i]->transfer_flags =URB_ZERO_PACKET ; +if (port->bulk_out_size ==0 ) +return -EINVAL ; -/* Start reading from the device */ -return usb_serial_generic_open (tty, port); -} +if (count ==0 ) +return 0 ; -static int ir_prepare_write_buffer (struct usb_serial_port *port, -void *dest,size_t size) -{ -unsigned char *buf = dest; -int count; + count =min (count, port->bulk_out_size -1 ); + +spin_lock_irqsave (&port->lock, flags); +if (__test_and_clear_bit (0 , &port->write_urbs_free)) { + urb = port->write_urbs[0 ]; + port->tx_bytes += count; + } +spin_unlock_irqrestore (&port->lock, flags); + +if (!urb) +return 0 ;/* * The first byte of the packet we send to the device contains an -* inbound header which indicates an additional number of BOFs and +* outbound header which indicates an additional number of BOFs and * a baud rate change. * * See section 5.4.2.2 of the USB IrDA spec. */ - *buf = ir_xbof | ir_baud; + *(u8 *)urb->transfer_buffer = ir_xbof | ir_baud; + +memcpy (urb->transfer_buffer +1 , buf, count); + + urb->transfer_buffer_length = count +1 ; + urb->transfer_flags =URB_ZERO_PACKET ; + + ret =usb_submit_urb (urb,GFP_ATOMIC ); +if (ret) { +dev_err (&port->dev,"failed to submit write urb: %d\n" , ret); + +spin_lock_irqsave (&port->lock, flags); +__set_bit (0 , &port->write_urbs_free); + port->tx_bytes -= count; +spin_unlock_irqrestore (&port->lock, flags); + +return ret; + } + +return count; +} + +static void ir_write_bulk_callback (struct urb *urb) +{ +struct usb_serial_port *port = urb->context; +int status = urb->status; +unsigned long flags; + +spin_lock_irqsave (&port->lock, flags); +__set_bit (0 , &port->write_urbs_free); + port->tx_bytes -= urb->transfer_buffer_length -1 ; +spin_unlock_irqrestore (&port->lock, flags); + +switch (status) { +case 0 : +break ; +case -ENOENT : +case -ECONNRESET : +case -ESHUTDOWN : +dev_dbg (&port->dev,"write urb stopped: %d\n" , status); +return ; +case -EPIPE : +dev_err (&port->dev,"write urb stopped: %d\n" , status); +return ; +default : +dev_err (&port->dev,"nonzero write-urb status: %d\n" , status); +break ; + } + +usb_serial_port_softint (port); +} + +static int ir_write_room (struct tty_struct *tty) +{ +struct usb_serial_port *port = tty->driver_data; +int count =0 ; + +if (port->bulk_out_size ==0 ) +return 0 ; + +if (test_bit (0 , &port->write_urbs_free)) + count = port->bulk_out_size -1 ; - count =kfifo_out_locked (&port->write_fifo, buf +1 , size -1 , - &port->lock); -return count +1 ; +return count; }static void ir_process_read_urb (struct urb *urb)
Greg Kroah-Hartman
maintainerPATCH 9/110 usb: dwc3: turn off VBUS when leaving host mode
In reply to Greg Kroah-HartmanFrom: Bin Liu <[email protected]>
commit 09ed259fac621634d51cd986aa8d65f035662658 upstream.
VBUS should be turned off when leaving the host mode.
Set GCTL_PRTCAP to device mode in teardown to de-assert DRVVBUS pin to turn off VBUS power.
--- drivers/usb/dwc3/core.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/usb/dwc3/core.c +3 -0
@@ -1209,6 +1209,9 @@ static void dwc3_core_exit_mode(struct d/* do nothing */ break ; } + +/* de-assert DRVVBUS for HOST and OTG mode */ +dwc3_set_prtcap (dwc,DWC3_GCTL_PRTCAP_DEVICE ); }static void dwc3_get_properties (struct dwc3 *dwc)
Greg Kroah-Hartman
maintainerPATCH 10/110 usb: typec: wcove: fix "op-sink-microwatt" default that was in mW
In reply to Greg Kroah-HartmanFrom: Thomas Hebb <[email protected]>
commit 0e64350bf4668d0fbbfec66fd8e637b971b4e976 upstream.
commit 4c912bff46cc ("usb: typec: wcove: Provide fwnode for the port") didn't convert this value from mW to uW when migrating to a new specification format like it should have.
--- drivers/usb/typec/tcpm/wcove.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/usb/typec/tcpm/wcove.c +1 -1
@@ -597,7 +597,7 @@ static const struct property_entry wcovePROPERTY_ENTRY_STRING ("try-power-role" ,"sink" ),PROPERTY_ENTRY_U32_ARRAY ("source-pdos" , src_pdo),PROPERTY_ENTRY_U32_ARRAY ("sink-pdos" , snk_pdo), - PROPERTY_ENTRY_U32("op-sink-microwatt", 15000), + PROPERTY_ENTRY_U32("op-sink-microwatt", 15000000), { } };
Greg Kroah-Hartman
maintainerPATCH 11/110 usb: typec: fusb302: fix "op-sink-microwatt" default that was in mW
In reply to Greg Kroah-HartmanFrom: Thomas Hebb <[email protected]>
commit eb7a3bb8c955b3694e0e0998413ce1563c02f90c upstream.
commit 8f6244055bd3 ("usb: typec: fusb302: Always provide fwnode for the port") didn't convert this value from mW to uW when migrating to a new specification format like it should have.
--- drivers/usb/typec/tcpm/fusb302.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/usb/typec/tcpm/fusb302.c +1 -1
@@ -1666,7 +1666,7 @@ static const struct property_entry port_PROPERTY_ENTRY_STRING ("try-power-role" ,"sink" ),PROPERTY_ENTRY_U32_ARRAY ("source-pdos" , src_pdo),PROPERTY_ENTRY_U32_ARRAY ("sink-pdos" , snk_pdo), - PROPERTY_ENTRY_U32("op-sink-microwatt", 2500), + PROPERTY_ENTRY_U32("op-sink-microwatt", 2500000), { } };
Greg Kroah-Hartman
maintainerPATCH 12/110 staging: most: net: fix buffer overflow
In reply to Greg Kroah-HartmanFrom: Andrey Shvetsov <[email protected]>
commit 4d1356ac12f4d5180d0df345d85ff0ee42b89c72 upstream.
If the length of the socket buffer is 0xFFFFFFFF (max size for an unsigned int), then payload_len becomes 0xFFFFFFF1 after subtracting 14 (ETH_HLEN). Then, mdp_len is set to payload_len + 16 (MDP_HDR_LEN) which overflows and results in a value of 2. These values for payload_len and mdp_len will pass current buffer size checks.
This patch checks if derived from skb->len sum may overflow.
The check is based on the following idea:
For any unsigned V1, V2 and derived unsigned SUM = V1 + V2, V1 + V2 overflows iif SUM < V1.
--- drivers/staging/most/net/net.c | 10 ++++++++++ 1 file changed, 10 insertions(+)
drivers/staging/most/net/net.c +10 -0
@@ -81,6 +81,11 @@ static int skb_to_mamac(const struct sk_unsigned int payload_len = skb->len -ETH_HLEN ;unsigned int mdp_len = payload_len +MDP_HDR_LEN ; +if (mdp_len < skb->len) { +pr_err ("drop: too large packet! (%u)\n" , skb->len); +return -EINVAL ; + } +if (mbo->buffer_length < mdp_len) {pr_err ("drop: too small buffer! (%d for %d)\n" , mbo->buffer_length, mdp_len);
@@ -128,6 +133,11 @@ static int skb_to_mep(const struct sk_buu8 *buff = mbo->virt_address;unsigned int mep_len = skb->len +MEP_HDR_LEN ; +if (mep_len < skb->len) { +pr_err ("drop: too large packet! (%u)\n" , skb->len); +return -EINVAL ; + } +if (mbo->buffer_length < mep_len) {pr_err ("drop: too small buffer! (%d for %d)\n" , mbo->buffer_length, mep_len);
Greg Kroah-Hartman
maintainerPATCH 13/110 staging: wlan-ng: ensure error return is actually returned
In reply to Greg Kroah-HartmanFrom: Colin Ian King <[email protected]>
commit 4cc41cbce536876678b35e03c4a8a7bb72c78fa9 upstream.
Currently when the call to prism2sta_ifst fails a netdev_err error is reported, error return variable result is set to -1 but the function always returns 0 for success. Fix this by returning the error value in variable result rather than 0.
Addresses-Coverity: ("Unused value")
--- drivers/staging/wlan-ng/prism2mgmt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/wlan-ng/prism2mgmt.c +1 -1
@@ -959,7 +959,7 @@ int prism2mgmt_flashdl_state(struct wlan } } -return 0 ; +return result; }/*----------------------------------------------------------------
Greg Kroah-Hartman
maintainerPATCH 14/110 staging: vt6656: correct packet types for CTS protect, mode.
In reply to Greg Kroah-HartmanFrom: Malcolm Priestley <[email protected]>
commit d971fdd3412f8342747778fb59b8803720ed82b1 upstream.
It appears that the driver still transmits in CTS protect mode even though it is not enabled in mac80211.
That is both packet types PK_TYPE_11GA and PK_TYPE_11GB both use CTS protect. The only difference between them GA does not use B rates.
Find if only B rate in GB or GA in protect mode otherwise transmit packets as PK_TYPE_11A.
--- drivers/staging/vt6656/device.h | 2 ++ drivers/staging/vt6656/rxtx.c | 12 ++++++++---- 2 files changed, 10 insertions(+), 4 deletions(-)
drivers/staging/vt6656/device.h +2 -0
@@ -52,6 +52,8 @@#define RATE_AUTO 12#define MAX_RATE 12 +#define VNT_B_RATES (BIT(RATE_1M) | BIT(RATE_2M) |\ + BIT(RATE_5M) | BIT(RATE_11M))/* * device specific
drivers/staging/vt6656/rxtx.c +8 -4
@@ -815,10 +815,14 @@ int vnt_tx_packet(struct vnt_private *prif (info->band ==NL80211_BAND_5GHZ ) { pkt_type =PK_TYPE_11A ; }else { -if (tx_rate->flags &IEEE80211_TX_RC_USE_CTS_PROTECT ) - pkt_type =PK_TYPE_11GB ; -else - pkt_type =PK_TYPE_11GA ; +if (tx_rate->flags &IEEE80211_TX_RC_USE_CTS_PROTECT ) { +if (priv->basic_rates &VNT_B_RATES ) + pkt_type =PK_TYPE_11GB ; +else + pkt_type =PK_TYPE_11GA ; + }else { + pkt_type =PK_TYPE_11A ; + } } }else { pkt_type =PK_TYPE_11B ;
Greg Kroah-Hartman
maintainerPATCH 15/110 staging: vt6656: use NULLFUCTION stack on mac80211
In reply to Greg Kroah-HartmanFrom: Malcolm Priestley <[email protected]>
commit d579c43c82f093e63639151625b2139166c730fd upstream.
It appears that the drivers does not go into power save correctly the NULL data packets are not being transmitted because it not enabled in mac80211.
The driver needs to capture ieee80211_is_nullfunc headers and copy the duration_id to it's own duration data header.
--- drivers/staging/vt6656/main_usb.c | 1 + drivers/staging/vt6656/rxtx.c | 14 +++++--------- 2 files changed, 6 insertions(+), 9 deletions(-)
drivers/staging/vt6656/main_usb.c +1 -0
@@ -1016,6 +1016,7 @@ vt6656_probe(struct usb_interface *intf,ieee80211_hw_set (priv->hw,RX_INCLUDES_FCS );ieee80211_hw_set (priv->hw,REPORTS_TX_ACK_STATUS );ieee80211_hw_set (priv->hw,SUPPORTS_PS ); +ieee80211_hw_set (priv->hw,PS_NULLFUNC_STACK ); priv->hw->max_signal =100 ;
drivers/staging/vt6656/rxtx.c +5 -9
@@ -278,11 +278,9 @@ static u16 vnt_rxtx_datahead_g(struct vn PK_TYPE_11B, &buf->b);/* Get Duration and TimeStamp */ -if (ieee80211_is_pspoll (hdr->frame_control)) { -__le16 dur =cpu_to_le16 (priv->current_aid |BIT (14 ) |BIT (15 )); - - buf->duration_a = dur; - buf->duration_b = dur; +if (ieee80211_is_nullfunc (hdr->frame_control)) { + buf->duration_a = hdr->duration_id; + buf->duration_b = hdr->duration_id; }else { buf->duration_a =vnt_get_duration_le (priv, tx_context->pkt_type, need_ack);
@@ -371,10 +369,8 @@ static u16 vnt_rxtx_datahead_ab(struct v tx_context->pkt_type, &buf->ab);/* Get Duration and TimeStampOff */ -if (ieee80211_is_pspoll (hdr->frame_control)) { -__le16 dur =cpu_to_le16 (priv->current_aid |BIT (14 ) |BIT (15 )); - - buf->duration = dur; +if (ieee80211_is_nullfunc (hdr->frame_control)) { + buf->duration = hdr->duration_id; }else { buf->duration =vnt_get_duration_le (priv, tx_context->pkt_type, need_ack);
Greg Kroah-Hartman
maintainerPATCH 16/110 staging: vt6656: Fix false Tx excessive retries reporting.
In reply to Greg Kroah-HartmanFrom: Malcolm Priestley <[email protected]>
commit 9dd631fa99dc0a0dfbd191173bf355ba30ea786a upstream.
The driver reporting IEEE80211_TX_STAT_ACK is not being handled correctly. The driver should only report on TSR_TMO flag is not set indicating no transmission errors and when not IEEE80211_TX_CTL_NO_ACK is being requested.
--- drivers/staging/vt6656/int.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
drivers/staging/vt6656/int.c +4 -2
@@ -99,9 +99,11 @@ static int vnt_int_report_rate(struct vn info->status.rates[0 ].count = tx_retry; -if (!(tsr & (TSR_TMO |TSR_RETRYTMO ))) { +if (!(tsr &TSR_TMO )) { info->status.rates[0 ].idx = idx; - info->flags |=IEEE80211_TX_STAT_ACK ; + +if (!(info->flags &IEEE80211_TX_CTL_NO_ACK )) + info->flags |=IEEE80211_TX_STAT_ACK ; }ieee80211_tx_status_irqsafe (priv->hw, context->skb);
Greg Kroah-Hartman
maintainerPATCH 17/110 serial: 8250_bcm2835aux: Fix line mismatch on driver unbind
In reply to Greg Kroah-HartmanFrom: Lukas Wunner <[email protected]>
commit dc76697d7e933d5e299116f219c890568785ea15 upstream.
Unbinding the bcm2835aux UART driver raises the following error if the maximum number of 8250 UARTs is set to 1 (via the 8250.nr_uarts module parameter or CONFIG_SERIAL_8250_RUNTIME_UARTS):
(NULL device *): Removing wrong port: a6f80333 != fa20408b
That's because bcm2835aux_serial_probe() retrieves UART line number 1 from the devicetree and stores it in data->uart.port.line, while serial8250_register_8250_port() instead uses UART line number 0, which is stored in data->line.
On driver unbind, bcm2835aux_serial_remove() uses data->uart.port.line, which contains the wrong number. Fix it.
The issue does not occur if the maximum number of 8250 UARTs is >= 2.
--- drivers/tty/serial/8250/8250_bcm2835aux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/tty/serial/8250/8250_bcm2835aux.c +1 -1
@@ -113,7 +113,7 @@ static int bcm2835aux_serial_remove(stru {struct bcm2835aux_data *data =platform_get_drvdata (pdev); -serial8250_unregister_port (data->uart.port.line); +serial8250_unregister_port (data->line);clk_disable_unprepare (data->clk);return 0 ;
Greg Kroah-Hartman
maintainerPATCH 18/110 serial: imx: fix a race condition in receive path
In reply to Greg Kroah-HartmanFrom: Uwe Kleine-König <[email protected]>
commit 101aa46bd221b768dfff8ef3745173fc8dbb85ee upstream.
The main irq handler function starts by first masking disabled interrupts in the status register values to ensure to only handle enabled interrupts. This is important as when the RX path in the hardware is disabled reading the RX fifo results in an external abort.
This checking must be done under the port lock, otherwise the following can happen:
CPU1 | CPU2
|
irq triggers as there are chars |
in the RX fifo | | grab port lock
imx_uart_int finds RRDY enabled |
and calls imx_uart_rxint which |
has to wait for port lock |
| disable RX (e.g. because we're | using RS485 with !RX_DURING_TX) |
| release port lock
read from RX fifo with RX |
disabled => exception |
So take the port lock only once in imx_uart_int() instead of in the functions called from there.
--- drivers/tty/serial/imx.c | 51 +++++++++++++++++++++++++++++++++++------------ 1 file changed, 38 insertions(+), 13 deletions(-)
drivers/tty/serial/imx.c +38 -13
@@ -700,22 +700,33 @@ static void imx_uart_start_tx(struct uar } } -static irqreturn_t imx_uart_rtsint (int irq,void *dev_id) +static irqreturn_t __imx_uart_rtsint (int irq,void *dev_id) {struct imx_port *sport = dev_id;u32 usr1; -spin_lock (&sport->port.lock); -imx_uart_writel (sport,USR1_RTSD ,USR1 ); usr1 =imx_uart_readl (sport,USR1 ) &USR1_RTSS ;uart_handle_cts_change (&sport->port, !!usr1);wake_up_interruptible (&sport->port.state->port.delta_msr_wait); -spin_unlock (&sport->port.lock);return IRQ_HANDLED ; } +static irqreturn_t imx_uart_rtsint (int irq,void *dev_id) +{ +struct imx_port *sport = dev_id; +irqreturn_t ret; + +spin_lock (&sport->port.lock); + + ret =__imx_uart_rtsint (irq, dev_id); + +spin_unlock (&sport->port.lock); + +return ret; +} +static irqreturn_t imx_uart_txint (int irq,void *dev_id) {struct imx_port *sport = dev_id;
@@ -726,14 +737,12 @@ static irqreturn_t imx_uart_txint(int irreturn IRQ_HANDLED ; } -static irqreturn_t imx_uart_rxint (int irq,void *dev_id) +static irqreturn_t __imx_uart_rxint (int irq,void *dev_id) {struct imx_port *sport = dev_id;unsigned int rx, flg, ignored =0 ;struct tty_port *port = &sport->port.state->port; -spin_lock (&sport->port.lock); -while (imx_uart_readl (sport,USR2 ) &USR2_RDR ) {u32 usr2;
@@ -792,11 +801,25 @@ static irqreturn_t imx_uart_rxint(int ir } out: -spin_unlock (&sport->port.lock);tty_flip_buffer_push (port); +return IRQ_HANDLED ; } +static irqreturn_t imx_uart_rxint (int irq,void *dev_id) +{ +struct imx_port *sport = dev_id; +irqreturn_t ret; + +spin_lock (&sport->port.lock); + + ret =__imx_uart_rxint (irq, dev_id); + +spin_unlock (&sport->port.lock); + +return ret; +} +static void imx_uart_clear_rx_errors (struct imx_port *sport);/*
@@ -855,6 +878,8 @@ static irqreturn_t imx_uart_int(int irq,unsigned int usr1, usr2, ucr1, ucr2, ucr3, ucr4;irqreturn_t ret =IRQ_NONE ; +spin_lock (&sport->port.lock); + usr1 =imx_uart_readl (sport,USR1 ); usr2 =imx_uart_readl (sport,USR2 ); ucr1 =imx_uart_readl (sport,UCR1 );
@@ -888,27 +913,25 @@ static irqreturn_t imx_uart_int(int irq, usr2 &= ~USR2_ORE ;if (usr1 & (USR1_RRDY |USR1_AGTIM )) { -imx_uart_rxint (irq, dev_id); +__imx_uart_rxint (irq, dev_id); ret =IRQ_HANDLED ; }if ((usr1 &USR1_TRDY ) || (usr2 &USR2_TXDC )) { -imx_uart_txint (irq, dev_id); +imx_uart_transmit_buffer (sport); ret =IRQ_HANDLED ; }if (usr1 &USR1_DTRD ) {imx_uart_writel (sport,USR1_DTRD ,USR1 ); -spin_lock (&sport->port.lock);imx_uart_mctrl_check (sport); -spin_unlock (&sport->port.lock); ret =IRQ_HANDLED ; }if (usr1 &USR1_RTSD ) { -imx_uart_rtsint (irq, dev_id); +__imx_uart_rtsint (irq, dev_id); ret =IRQ_HANDLED ; }
@@ -923,6 +946,8 @@ static irqreturn_t imx_uart_int(int irq, ret =IRQ_HANDLED ; } +spin_unlock (&sport->port.lock); +return ret; }
Greg Kroah-Hartman
maintainerPATCH 19/110 debugfs: Return -EPERM when locked down
In reply to Greg Kroah-HartmanFrom: Eric Snowberg <[email protected]>
commit a37f4958f7b63d2b3cd17a76151fdfc29ce1da5f upstream.
When lockdown is enabled, debugfs_is_locked_down returns 1. It will then trigger the following:
WARNING: CPU: 48 PID: 3747 CPU: 48 PID: 3743 Comm: bash Not tainted 5.4.0-1946.x86_64 #1 Hardware name: Oracle Corporation ORACLE SERVER X7-2/ASM, MB, X7-2, BIOS 41060400 05/20/2019 RIP: 0010:do_dentry_open+0x343/0x3a0 Code: 00 40 08 00 45 31 ff 48 c7 43 28 40 5b e7 89 e9 02 ff ff ff 48 8b 53 28 4c 8b 72 70 4d 85 f6 0f 84 10 fe ff ff e9 f5 fd ff ff <0f> 0b 41 bf ea ff ff ff e9 3b ff ff ff 41 bf e6 ff ff ff e9 b4 fe RSP: 0018:ffffb8740dde7ca0 EFLAGS: 00010202 RAX: ffffffff89e88a40 RBX: ffff928c8e6b6f00 RCX: 0000000000000000 RDX: 0000000000000000 RSI: ffff928dbfd97778 RDI: ffff9285cff685c0 RBP: ffffb8740dde7cc8 R08: 0000000000000821 R09: 0000000000000030 R10: 0000000000000057 R11: ffffb8740dde7a98 R12: ffff926ec781c900 R13: ffff928c8e6b6f10 R14: ffffffff8936e190 R15: 0000000000000001 FS: 00007f45f6777740(0000) GS:ffff928dbfd80000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 CR2: 00007fff95e0d5d8 CR3: 0000001ece562006 CR4: 00000000007606e0 DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400 PKRU: 55555554 Call Trace: vfs_open+0x2d/0x30 path_openat+0x2d4/0x1680 ? tty_mode_ioctl+0x298/0x4c0 do_filp_open+0x93/0x100 ? strncpy_from_user+0x57/0x1b0 ? __alloc_fd+0x46/0x150 do_sys_open+0x182/0x230 __x64_sys_openat+0x20/0x30 do_syscall_64+0x60/0x1b0 entry_SYSCALL_64_after_hwframe+0x170/0x1d5 RIP: 0033:0x7f45f5e5ce02 Code: 25 00 00 41 00 3d 00 00 41 00 74 4c 48 8d 05 25 59 2d 00 8b 00 85 c0 75 6d 89 f2 b8 01 01 00 00 48 89 fe bf 9c ff ff ff 0f 05 <48> 3d 00 f0 ff ff 0f 87 a2 00 00 00 48 8b 4c 24 28 64 48 33 0c 25 RSP: 002b:00007fff95e0d2e0 EFLAGS: 00000246 ORIG_RAX: 0000000000000101 RAX: ffffffffffffffda RBX: 0000561178c069b0 RCX: 00007f45f5e5ce02 RDX: 0000000000000241 RSI: 0000561178c08800 RDI: 00000000ffffff9c RBP: 00007fff95e0d3e0 R08: 0000000000000020 R09: 0000000000000005 R10: 00000000000001b6 R11: 0000000000000246 R12: 0000000000000000 R13: 0000000000000003 R14: 0000000000000001 R15: 0000561178c08800
Change the return type to int and return -EPERM when lockdown is enabled to remove the warning above. Also rename debugfs_is_locked_down to debugfs_locked_down to make it sound less like it returns a boolean.
--- fs/debugfs/file.c | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-)
fs/debugfs/file.c +10 -7
@@ -142,18 +142,21 @@ EXPORT_SYMBOL_GPL(debugfs_file_put);* We also need to exclude any file that has ways to write or alter it as root * can bypass the permissions check. */ -static bool debugfs_is_locked_down (struct inode *inode, -struct file *filp, -const struct file_operations *real_fops) +static int debugfs_locked_down (struct inode *inode, +struct file *filp, +const struct file_operations *real_fops) {if ((inode->i_mode &07777 ) ==0444 && !(filp->f_mode &FMODE_WRITE ) && !real_fops->unlocked_ioctl && !real_fops->compat_ioctl && !real_fops->mmap) -return false; +return 0 ; -return security_locked_down (LOCKDOWN_DEBUGFS ); +if (security_locked_down (LOCKDOWN_DEBUGFS )) +return -EPERM ; + +return 0 ; }static int open_proxy_open (struct inode *inode,struct file *filp)
@@ -168,7 +171,7 @@ static int open_proxy_open(struct inode real_fops =debugfs_real_fops (filp); - r =debugfs_is_locked_down (inode, filp, real_fops); + r =debugfs_locked_down (inode, filp, real_fops);if (r) goto out;
@@ -298,7 +301,7 @@ static int full_proxy_open(struct inode real_fops =debugfs_real_fops (filp); - r =debugfs_is_locked_down (inode, filp, real_fops); + r =debugfs_locked_down (inode, filp, real_fops);if (r) goto out;
Greg Kroah-Hartman
maintainerPATCH 20/110 component: do not dereference opaque pointer in debugfs
In reply to Greg Kroah-HartmanFrom: Lubomir Rintel <[email protected]>
commit ef9ffc1e5f1ac73ecd2fb3b70db2a3b2472ff2f7 upstream.
The match data does not have to be a struct device pointer, and indeed very often is not. Attempt to treat it as such easily results in a crash.
For the components that are not registered, we don't know which device is missing. Once it it is there, we can use the struct component to get the device and whether it's bound or not.
--- drivers/base/component.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/base/component.c +4 -4
@@ -102,11 +102,11 @@ static int component_devices_show(structseq_printf (s,"%-40s %20s\n" ,"device name" ,"status" );seq_puts (s,"-------------------------------------------------------------\n" );for (i =0 ; i < match->num; i++) { -struct device *d = (struct device *)match->compare[i].data; +struct component *component = match->compare[i].component; -seq_printf (s,"%-40s %20s\n" ,dev_name (d), - match->compare[i].component ? -"registered" :"not registered" ); +seq_printf (s,"%-40s %20s\n" , + component ?dev_name (component->dev) :"(unknown)" , + component ? (component->bound ?"bound" :"not bound" ) :"not registered" ); }mutex_unlock (&component_mutex);
Greg Kroah-Hartman
maintainerPATCH 21/110 binder: fix log spam for existing debugfs file creation.
In reply to Greg Kroah-HartmanFrom: Martin Fuzzey <[email protected]>
commit eb143f8756e77c8fcfc4d574922ae9efd3a43ca9 upstream.
Since commit 43e23b6c0b01 ("debugfs: log errors when something goes wrong") debugfs logs attempts to create existing files.
However binder attempts to create multiple debugfs files with the same name when a single PID has multiple contexts, this leads to log spamming during an Android boot (17 such messages during boot on my system).
Fix this by checking if we already know the PID and only create the debugfs entry for the first context per PID.
Do the same thing for binderfs for symmetry.
--- drivers/android/binder.c | 37 +++++++++++++++++++------------------ 1 file changed, 19 insertions(+), 18 deletions(-)
drivers/android/binder.c +19 -18
@@ -5203,10 +5203,11 @@ err_bad_arg:static int binder_open (struct inode *nodp,struct file *filp) { -struct binder_proc *proc; +struct binder_proc *proc, *itr;struct binder_device *binder_dev;struct binderfs_info *info;struct dentry *binder_binderfs_dir_entry_proc =NULL ; +bool existing_pid = false;binder_debug (BINDER_DEBUG_OPEN_CLOSE ,"%s: %d:%d\n" , __func__, current->group_leader->pid, current->pid);
@@ -5239,19 +5240,24 @@ static int binder_open(struct inode *nod filp->private_data = proc;mutex_lock (&binder_procs_lock); +hlist_for_each_entry (itr, &binder_procs, proc_node) { +if (itr->pid == proc->pid) { + existing_pid = true; +break ; + } + }hlist_add_head (&proc->proc_node, &binder_procs);mutex_unlock (&binder_procs_lock); -if (binder_debugfs_dir_entry_proc) { +if (binder_debugfs_dir_entry_proc && !existing_pid) {char strbuf[11 ];snprintf (strbuf,sizeof (strbuf),"%u" , proc->pid);/* -* proc debug entries are shared between contexts, so -* this will fail if the process tries to open the driver -* again with a different context. The priting code will -* anyway print all contexts that a given PID has, so this -* is not a problem. +* proc debug entries are shared between contexts. +* Only create for the first PID to avoid debugfs log spamming +* The printing code will anyway print all contexts for a given +* PID so this is not a problem. */ proc->debugfs_entry = debugfs_create_file(strbuf, 0444, binder_debugfs_dir_entry_proc,
@@ -5259,19 +5265,16 @@ static int binder_open(struct inode *nod &proc_fops); } -if (binder_binderfs_dir_entry_proc) { +if (binder_binderfs_dir_entry_proc && !existing_pid) {char strbuf[11 ];struct dentry *binderfs_entry;snprintf (strbuf,sizeof (strbuf),"%u" , proc->pid);/* * Similar to debugfs, the process specific log file is shared -* between contexts. If the file has already been created for a -* process, the following binderfs_create_file() call will -* fail with error code EEXIST if another context of the same -* process invoked binder_open(). This is ok since same as -* debugfs, the log file will contain information on all -* contexts of a given PID. +* between contexts. Only create for the first PID. +* This is ok since same as debugfs, the log file will contain +* information on all contexts of a given PID. */ binderfs_entry =binderfs_create_file (binder_binderfs_dir_entry_proc, strbuf, &proc_fops, (void *)(unsigned long )proc->pid);
@@ -5281,10 +5284,8 @@ static int binder_open(struct inode *nodint error; error =PTR_ERR (binderfs_entry); -if (error != -EEXIST ) { -pr_warn ("Unable to create file %s in binderfs (error %d)\n" , - strbuf, error); - } +pr_warn ("Unable to create file %s in binderfs (error %d)\n" , + strbuf, error); } }
Greg Kroah-Hartman
maintainerPATCH 22/110 mei: hdcp: bind only with i915 on the same PCH
In reply to Greg Kroah-HartmanFrom: Tomas Winkler <[email protected]>
commit 1e8d19d9b0dfcf11b61bac627203a290577e807a upstream.
The mei device and i915 must reside on the same PCH in order for HDCP to work. Make the component matching function enforce this requirement.
hdcp
|
i915 mei
| |
+----= PCH =----+--- drivers/misc/mei/hdcp/mei_hdcp.c | 33 ++++++++++++++++++++++++++++++--- 1 file changed, 30 insertions(+), 3 deletions(-)
drivers/misc/mei/hdcp/mei_hdcp.c +30 -3
@@ -758,11 +758,38 @@ static const struct component_master_ops .unbind = mei_component_master_unbind, }; +/** +* mei_hdcp_component_match - compare function for matching mei hdcp. +* +* The function checks if the driver is i915, the subcomponent is HDCP +* and the grand parent of hdcp and the parent of i915 are the same +* PCH device. +* +* @dev: master device +* @subcomponent: subcomponent to match (I915_COMPONENT_HDCP) +* @data: compare data (mei hdcp device) +* +* Return: +* * 1 - if components match +* * 0 - otherwise +*/ static int mei_hdcp_component_match (struct device *dev,int subcomponent,void *data) { -return !strcmp (dev->driver->name,"i915" ) && - subcomponent ==I915_COMPONENT_HDCP ; +struct device *base = data; + +if (strcmp (dev->driver->name,"i915" ) || + subcomponent !=I915_COMPONENT_HDCP ) +return 0 ; + + base = base->parent; +if (!base) +return 0 ; + + base = base->parent; + dev = dev->parent; + +return (base && dev && dev == base); } static int mei_hdcp_probe(struct mei_cl_device *cldev,
@@ -786,7 +813,7 @@ static int mei_hdcp_probe(struct mei_cl_ master_match =NULL ;component_match_add_typed (&cldev->dev, &master_match, - mei_hdcp_component_match, comp_master); + mei_hdcp_component_match, &cldev->dev);if (IS_ERR_OR_NULL (master_match)) { ret = -ENOMEM ; goto err_exit;
Greg Kroah-Hartman
maintainerPATCH 23/110 mei: me: add comet point (lake) H device ids
In reply to Greg Kroah-HartmanFrom: Tomas Winkler <[email protected]>
commit 559e575a8946a6561dfe8880de341d4ef78d5994 upstream.
Add Comet Point device IDs for Comet Lake H platforms.
--- drivers/misc/mei/hw-me-regs.h | 4 ++++ drivers/misc/mei/pci-me.c | 2 ++ 2 files changed, 6 insertions(+)
drivers/misc/mei/hw-me-regs.h +4 -0
@@ -81,8 +81,12 @@#define MEI_DEV_ID_CMP_LP 0x02e0/* Comet Point LP */ #define MEI_DEV_ID_CMP_LP_3 0x02e4/* Comet Point LP 3 (iTouch) */ +#define MEI_DEV_ID_CMP_V 0xA3BA/* Comet Point Lake V */ +#define MEI_DEV_ID_CMP_H 0x06e0/* Comet Lake H */ +#define MEI_DEV_ID_CMP_H_3 0x06e4/* Comet Lake H 3 (iTouch) */ +#define MEI_DEV_ID_ICP_LP 0x34E0/* Ice Lake Point LP */ #define MEI_DEV_ID_TGP_LP 0xA0E0/* Tiger Lake Point LP */
drivers/misc/mei/pci-me.c +2 -0
@@ -99,6 +99,8 @@ static const struct pci_device_id mei_me {MEI_PCI_DEVICE(MEI_DEV_ID_CMP_LP, MEI_ME_PCH12_CFG)}, {MEI_PCI_DEVICE(MEI_DEV_ID_CMP_LP_3, MEI_ME_PCH8_CFG)}, {MEI_PCI_DEVICE(MEI_DEV_ID_CMP_V, MEI_ME_PCH12_CFG)}, + {MEI_PCI_DEVICE(MEI_DEV_ID_CMP_H, MEI_ME_PCH12_CFG)}, + {MEI_PCI_DEVICE(MEI_DEV_ID_CMP_H_3, MEI_ME_PCH8_CFG)}, {MEI_PCI_DEVICE(MEI_DEV_ID_ICP_LP, MEI_ME_PCH12_CFG)},
Greg Kroah-Hartman
maintainerPATCH 24/110 iio: adc: stm32-dfsdm: fix single conversion
In reply to Greg Kroah-HartmanFrom: Olivier Moysan <[email protected]>
commit dc26935fb60e8da8d59655dd2ec0de47b20d7d8f upstream.
Apply data formatting to single conversion,
as this is already done in continuous and trigger modes.
--- drivers/iio/adc/stm32-dfsdm-adc.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/iio/adc/stm32-dfsdm-adc.c +2 -0
@@ -1204,6 +1204,8 @@ static int stm32_dfsdm_single_conv(strucstm32_dfsdm_stop_conv (adc); +stm32_dfsdm_process_data (adc, res); + stop_dfsdm:stm32_dfsdm_stop_dfsdm (adc->dfsdm);
Greg Kroah-Hartman
maintainerPATCH 25/110 iio: st_gyro: Correct data for LSM9DS0 gyro
In reply to Greg Kroah-HartmanFrom: Andy Shevchenko <[email protected]>
commit e825070f697abddf3b9b0a675ed0ff1884114818 upstream.
The commit 41c128cb25ce ("iio: st_gyro: Add lsm9ds0-gyro support") assumes that gyro in LSM9DS0 is the same as others with 0xd4 WAI ID, but datasheet tells slight different story, i.e. the first scale factor for the chip is 245 dps, and not 250 dps.
Correct this by introducing a separate settings for LSM9DS0.
Depends-on: 45a4e4220bf4 ("iio: gyro: st_gyro: fix L3GD20H support")
--- drivers/iio/gyro/st_gyro_core.c | 75 +++++++++++++++++++++++++++++++++++++++- 1 file changed, 74 insertions(+), 1 deletion(-)
drivers/iio/gyro/st_gyro_core.c +74 -1
@@ -139,7 +139,6 @@ static const struct st_sensor_settings s [2] = LSM330DLC_GYRO_DEV_NAME, [3] = L3G4IS_GYRO_DEV_NAME, [4 ] =LSM330_GYRO_DEV_NAME , - [5 ] =LSM9DS0_GYRO_DEV_NAME , }, .ch = (struct iio_chan_spec *)st_gyro_16bit_channels, .odr = {
@@ -203,6 +202,80 @@ static const struct st_sensor_settings s }, }, .sim = { + .addr = 0x23, + .value = BIT(0), + }, + .multi_read_bit = true, + .bootime = 2, + }, + { + .wai = 0xd4, + .wai_addr = ST_SENSORS_DEFAULT_WAI_ADDRESS, + .sensors_supported = { + [0] = LSM9DS0_GYRO_DEV_NAME, + }, + .ch = (struct iio_chan_spec *)st_gyro_16bit_channels, + .odr = { + .addr = 0x20, + .mask = GENMASK(7, 6), + .odr_avl = { + { .hz = 95, .value = 0x00, }, + { .hz = 190, .value = 0x01, }, + { .hz = 380, .value = 0x02, }, + { .hz = 760, .value = 0x03, }, + }, + }, + .pw = { + .addr = 0x20, + .mask = BIT(3), + .value_on = ST_SENSORS_DEFAULT_POWER_ON_VALUE, + .value_off = ST_SENSORS_DEFAULT_POWER_OFF_VALUE, + }, + .enable_axis = { + .addr = ST_SENSORS_DEFAULT_AXIS_ADDR, + .mask = ST_SENSORS_DEFAULT_AXIS_MASK, + }, + .fs = { + .addr = 0x23, + .mask = GENMASK(5, 4), + .fs_avl = { + [0] = { + .num = ST_GYRO_FS_AVL_245DPS, + .value = 0x00, + .gain = IIO_DEGREE_TO_RAD(8750), + }, + [1] = { + .num = ST_GYRO_FS_AVL_500DPS, + .value = 0x01, + .gain = IIO_DEGREE_TO_RAD(17500), + }, + [2] = { + .num = ST_GYRO_FS_AVL_2000DPS, + .value = 0x02, + .gain = IIO_DEGREE_TO_RAD(70000), + }, + }, + }, + .bdu = { + .addr = 0x23, + .mask = BIT(7), + }, + .drdy_irq = { + .int2 = { + .addr = 0x22, + .mask = BIT(3), + }, + /* + * The sensor has IHL (active low) and open + * drain settings, but only for INT1 and not + * for the DRDY line on INT2. + */ + .stat_drdy = { + .addr = ST_SENSORS_DEFAULT_STAT_ADDR, + .mask = GENMASK(2, 0), + }, + }, + .sim = { .addr = 0x23, .value = BIT(0), },
Greg Kroah-Hartman
maintainerPATCH 26/110 driver core: Fix test_async_driver_probe if NUMA is disabled
In reply to Greg Kroah-HartmanFrom: Guenter Roeck <[email protected]>
commit 264d25275a46fce5da501874fa48a2ae5ec571c8 upstream.
Since commit 57ea974fb871 ("driver core: Rewrite test_async_driver_probe to cover serialization and NUMA affinity"), running the test with NUMA disabled results in warning messages similar to the following.
test_async_driver test_async_driver.12: NUMA node mismatch -1 != 0
If CONFIG_NUMA=n, dev_to_node(dev) returns -1, and numa_node_id() returns 0. Both are widely used, so it appears risky to change return values. Augment the check with IS_ENABLED(CONFIG_NUMA) instead to fix the problem.
--- drivers/base/test/test_async_driver_probe.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/base/test/test_async_driver_probe.c +2 -1
@@ -44,7 +44,8 @@ static int test_probe(struct platform_de* performing an async init on that node. */ if (dev->driver->probe_type ==PROBE_PREFER_ASYNCHRONOUS ) { -if (dev_to_node (dev) !=numa_node_id ()) { +if (IS_ENABLED (CONFIG_NUMA ) && +dev_to_node (dev) !=numa_node_id ()) {dev_warn (dev,"NUMA node mismatch %d != %d\n" ,dev_to_node (dev),numa_node_id ());atomic_inc (&warnings);
Greg Kroah-Hartman
maintainerPATCH 27/110 crypto: chelsio - fix writing tfm flags to wrong place
In reply to Greg Kroah-HartmanFrom: Eric Biggers <[email protected]>
commit bd56cea012fc2d6381e8cd3209510ce09f9de8c9 upstream.
The chelsio crypto driver is casting 'struct crypto_aead' directly to 'struct crypto_tfm', which is incorrect because the crypto_tfm isn't the first field of 'struct crypto_aead'. Consequently, the calls to crypto_tfm_set_flags() are modifying some other field in the struct.
Also, the driver is setting CRYPTO_TFM_RES_BAD_KEY_LEN in ->setauthsize(), not just in ->setkey(). This is incorrect since this flag is for bad key lengths, not for bad authentication tag lengths.
Fix these bugs by removing the broken crypto_tfm_set_flags() calls from ->setauthsize() and by fixing them in ->setkey().
--- drivers/crypto/chelsio/chcr_algo.c | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-)
drivers/crypto/chelsio/chcr_algo.c +3 -13
@@ -3194,9 +3194,6 @@ static int chcr_gcm_setauthsize(struct c aeadctx->mayverify =VERIFY_SW ;break ;default : - -crypto_tfm_set_flags ((struct crypto_tfm *) tfm, -CRYPTO_TFM_RES_BAD_KEY_LEN );return -EINVAL ; }return crypto_aead_setauthsize (aeadctx->sw_cipher, authsize);
@@ -3221,8 +3218,6 @@ static int chcr_4106_4309_setauthsize(st aeadctx->mayverify =VERIFY_HW ;break ;default : -crypto_tfm_set_flags ((struct crypto_tfm *)tfm, -CRYPTO_TFM_RES_BAD_KEY_LEN );return -EINVAL ; }return crypto_aead_setauthsize (aeadctx->sw_cipher, authsize);
@@ -3263,8 +3258,6 @@ static int chcr_ccm_setauthsize(struct c aeadctx->mayverify =VERIFY_HW ;break ;default : -crypto_tfm_set_flags ((struct crypto_tfm *)tfm, -CRYPTO_TFM_RES_BAD_KEY_LEN );return -EINVAL ; }return crypto_aead_setauthsize (aeadctx->sw_cipher, authsize);
@@ -3289,8 +3282,7 @@ static int chcr_ccm_common_setkey(struct ck_size =CHCR_KEYCTX_CIPHER_KEY_SIZE_256 ; mk_size =CHCR_KEYCTX_MAC_KEY_SIZE_256 ; }else { -crypto_tfm_set_flags ((struct crypto_tfm *)aead, -CRYPTO_TFM_RES_BAD_KEY_LEN ); +crypto_aead_set_flags (aead,CRYPTO_TFM_RES_BAD_KEY_LEN ); aeadctx->enckey_len =0 ;return -EINVAL ; }
@@ -3328,8 +3320,7 @@ static int chcr_aead_rfc4309_setkey(struint error;if (keylen <3 ) { -crypto_tfm_set_flags ((struct crypto_tfm *)aead, -CRYPTO_TFM_RES_BAD_KEY_LEN ); +crypto_aead_set_flags (aead,CRYPTO_TFM_RES_BAD_KEY_LEN ); aeadctx->enckey_len =0 ;return -EINVAL ; }
@@ -3379,8 +3370,7 @@ static int chcr_gcm_setkey(struct crypto }else if (keylen ==AES_KEYSIZE_256 ) { ck_size =CHCR_KEYCTX_CIPHER_KEY_SIZE_256 ; }else { -crypto_tfm_set_flags ((struct crypto_tfm *)aead, -CRYPTO_TFM_RES_BAD_KEY_LEN ); +crypto_aead_set_flags (aead,CRYPTO_TFM_RES_BAD_KEY_LEN );pr_err ("GCM: Invalid key length %d\n" , keylen); ret = -EINVAL ; goto out;
Greg Kroah-Hartman
maintainerPATCH 28/110 CIFS: Fix task struct use-after-free on reconnect
In reply to Greg Kroah-HartmanFrom: Vincent Whitchurch <[email protected]>
commit f1f27ad74557e39f67a8331a808b860f89254f2d upstream.
The task which created the MID may be gone by the time cifsd attempts to call the callbacks on MIDs from cifs_reconnect().
This leads to a use-after-free of the task struct in cifs_wake_up_task:
==================================================================
BUG: KASAN: use-after-free in __lock_acquire+0x31a0/0x3270
Read of size 8 at addr ffff8880103e3a68 by task cifsd/630
CPU: 0 PID: 630 Comm: cifsd Not tainted 5.5.0-rc6+ #119
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.10.2-1 04/01/2014
Call Trace:
dump_stack+0x8e/0xcb
print_address_description.constprop.5+0x1d3/0x3c0
? __lock_acquire+0x31a0/0x3270
__kasan_report+0x152/0x1aa
? __lock_acquire+0x31a0/0x3270
? __lock_acquire+0x31a0/0x3270
kasan_report+0xe/0x20
__lock_acquire+0x31a0/0x3270
? __wake_up_common+0x1dc/0x630
? _raw_spin_unlock_irqrestore+0x4c/0x60
? mark_held_locks+0xf0/0xf0
? _raw_spin_unlock_irqrestore+0x39/0x60
? __wake_up_common_lock+0xd5/0x130
? __wake_up_common+0x630/0x630
lock_acquire+0x13f/0x330
? try_to_wake_up+0xa3/0x19e0
_raw_spin_lock_irqsave+0x38/0x50
? try_to_wake_up+0xa3/0x19e0
try_to_wake_up+0xa3/0x19e0
? cifs_compound_callback+0x178/0x210
? set_cpus_allowed_ptr+0x10/0x10
cifs_reconnect+0xa1c/0x15d0
? generic_ip_connect+0x1860/0x1860
? rwlock_bug.part.0+0x90/0x90
cifs_readv_from_socket+0x479/0x690
cifs_read_from_socket+0x9d/0xe0
? cifs_readv_from_socket+0x690/0x690
? mempool_resize+0x690/0x690
? rwlock_bug.part.0+0x90/0x90
? memset+0x1f/0x40
? allocate_buffers+0xff/0x340
cifs_demultiplex_thread+0x388/0x2a50
? cifs_handle_standard+0x610/0x610
? rcu_read_lock_held_common+0x120/0x120
? mark_lock+0x11b/0xc00
? __lock_acquire+0x14ed/0x3270
? __kthread_parkme+0x78/0x100
? lockdep_hardirqs_on+0x3e8/0x560
? lock_downgrade+0x6a0/0x6a0
? lockdep_hardirqs_on+0x3e8/0x560
? _raw_spin_unlock_irqrestore+0x39/0x60
? cifs_handle_standard+0x610/0x610
kthread+0x2bb/0x3a0
? kthread_create_worker_on_cpu+0xc0/0xc0
ret_from_fork+0x3a/0x50
Allocated by task 649:
save_stack+0x19/0x70
__kasan_kmalloc.constprop.5+0xa6/0xf0
kmem_cache_alloc+0x107/0x320
copy_process+0x17bc/0x5370
_do_fork+0x103/0xbf0
__x64_sys_clone+0x168/0x1e0
do_syscall_64+0x9b/0xec0
entry_SYSCALL_64_after_hwframe+0x49/0xbe
Freed by task 0:
save_stack+0x19/0x70
__kasan_slab_free+0x11d/0x160
kmem_cache_free+0xb5/0x3d0
rcu_core+0x52f/0x1230
__do_softirq+0x24d/0x962
The buggy address belongs to the object at ffff8880103e32c0
which belongs to the cache task_struct of size 6016
The buggy address is located 1960 bytes inside of
6016-byte region [ffff8880103e32c0, ffff8880103e4a40)
The buggy address belongs to the page:
page:ffffea000040f800 refcount:1 mapcount:0 mapping:ffff8880108da5c0
index:0xffff8880103e4c00 compound_mapcount: 0
raw: 4000000000010200 ffffea00001f2208 ffffea00001e3408 ffff8880108da5c0
raw: ffff8880103e4c00 0000000000050003 00000001ffffffff 0000000000000000
page dumped because: kasan: bad access detected
Memory state around the buggy address:
ffff8880103e3900: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
ffff8880103e3980: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>ffff8880103e3a00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
^
ffff8880103e3a80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
ffff8880103e3b00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
==================================================================This can be reliably reproduced by adding the below delay to cifs_reconnect(), running find(1) on the mount, restarting the samba server while find is running, and killing find during the delay:
spin_unlock(&GlobalMid_Lock); mutex_unlock(&server->srv_mutex);
+ msleep(10000);
+
cifs_dbg(FYI, "%s: issuing mid callbacks\n", __func__); list_for_each_safe(tmp, tmp2, &retry_list) {
mid_entry = list_entry(tmp, struct mid_q_entry, qhead);
Fix this by holding a reference to the task struct until the MID is freed.
--- fs/cifs/cifsglob.h | 1 + fs/cifs/smb2transport.c | 2 ++ fs/cifs/transport.c | 3 +++ 3 files changed, 6 insertions(+)
fs/cifs/cifsglob.h +1 -0
@@ -1538,6 +1538,7 @@ struct mid_q_entry {mid_callback_t *callback;/* call completion callback */ mid_handle_t *handle;/* call handle mid callback */ void *callback_data;/* general purpose pointer for callback */ +struct task_struct *creator;void *resp_buf;/* pointer to received SMB header */ unsigned int resp_buf_size;int mid_state;/* wish this were enum but can not pass to wait_event */
fs/cifs/smb2transport.c +2 -0
@@ -599,6 +599,8 @@ smb2_mid_entry_alloc(const struct smb2_s* The default is for the mid to be synchronous, so the * default callback just wakes up the current task. */ +get_task_struct (current); + temp->creator = current; temp->callback = cifs_wake_up_task; temp->callback_data = current;
fs/cifs/transport.c +3 -0
@@ -76,6 +76,8 @@ AllocMidQEntry(const struct smb_hdr *smb* The default is for the mid to be synchronous, so the * default callback just wakes up the current task. */ +get_task_struct (current); + temp->creator = current; temp->callback = cifs_wake_up_task; temp->callback_data = current;
@@ -158,6 +160,7 @@ static void _cifs_mid_q_entry_release(st } }#endif +put_task_struct (midEntry->creator);mempool_free (midEntry, cifs_mid_poolp); }
Greg Kroah-Hartman
maintainerPATCH 29/110 cifs: set correct max-buffer-size for smb2_ioctl_init()
In reply to Greg Kroah-HartmanFrom: Ronnie Sahlberg <[email protected]>
commit 731b82bb1750a906c1e7f070aedf5505995ebea7 upstream.
Fix two places where we need to adjust down the max response size for ioctl when it is used together with compounding.
--- fs/cifs/smb2ops.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)
fs/cifs/smb2ops.c +7 -2
@@ -1461,7 +1461,9 @@ smb2_ioctl_query_info(const unsigned intCOMPOUND_FID ,COMPOUND_FID , qi.info_type, true, buffer, qi.output_buffer_length, - CIFSMaxBufSize); + CIFSMaxBufSize - +MAX_SMB2_CREATE_RESPONSE_SIZE - + MAX_SMB2_CLOSE_RESPONSE_SIZE); } }else if (qi.flags ==PASSTHRU_SET_INFO ) {/* Can eventually relax perm check since server enforces too */
@@ -2634,7 +2636,10 @@ smb2_query_symlink(const unsigned int xi rc =SMB2_ioctl_init (tcon, &rqst[1 ], fid.persistent_fid, fid.volatile_fid,FSCTL_GET_REPARSE_POINT , - true/* is_fctl */ ,NULL ,0 , CIFSMaxBufSize); + true/* is_fctl */ ,NULL ,0 , + CIFSMaxBufSize - +MAX_SMB2_CREATE_RESPONSE_SIZE - +MAX_SMB2_CLOSE_RESPONSE_SIZE );if (rc) goto querty_exit;
Greg Kroah-Hartman
maintainerPATCH 30/110 cifs: Fix memory allocation in __smb2_handle_cancelled_cmd()
In reply to Greg Kroah-HartmanFrom: Paulo Alcantara (SUSE) <[email protected]>
commit 0a5a98863c9debc02387b3d23c46d187756f5e2b upstream.
__smb2_handle_cancelled_cmd() is called under a spin lock held in cifs_mid_q_entry_release(), so make its memory allocation GFP_ATOMIC.
This issue was observed when running xfstests generic/028:
[ 1722.589204] CIFS VFS: \\192.168.30.26 Cancelling wait for mid 72064 cmd: 5 [ 1722.590687] CIFS VFS: \\192.168.30.26 Cancelling wait for mid 72065 cmd: 17 [ 1722.593529] CIFS VFS: \\192.168.30.26 Cancelling wait for mid 72066 cmd: 6 [ 1723.039014] BUG: sleeping function called from invalid context at mm/slab.h:565 [ 1723.040710] in_atomic(): 1, irqs_disabled(): 0, non_block: 0, pid: 30877, name: cifsd [ 1723.045098] CPU: 3 PID: 30877 Comm: cifsd Not tainted 5.5.0-rc4+ #313 [ 1723.046256] Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS rel-1.12.0-59-gc9ba527-rebuilt.opensuse.org 04/01/2014 [ 1723.048221] Call Trace: [ 1723.048689] dump_stack+0x97/0xe0 [ 1723.049268] ___might_sleep.cold+0xd1/0xe1 [ 1723.050069] kmem_cache_alloc_trace+0x204/0x2b0 [ 1723.051051] __smb2_handle_cancelled_cmd+0x40/0x140 [cifs] [ 1723.052137] smb2_handle_cancelled_mid+0xf6/0x120 [cifs] [ 1723.053247] cifs_mid_q_entry_release+0x44d/0x630 [cifs] [ 1723.054351] ? cifs_reconnect+0x26a/0x1620 [cifs] [ 1723.055325] cifs_demultiplex_thread+0xad4/0x14a0 [cifs] [ 1723.056458] ? cifs_handle_standard+0x2c0/0x2c0 [cifs] [ 1723.057365] ? kvm_sched_clock_read+0x14/0x30 [ 1723.058197] ? sched_clock+0x5/0x10 [ 1723.058838] ? sched_clock_cpu+0x18/0x110 [ 1723.059629] ? lockdep_hardirqs_on+0x17d/0x250 [ 1723.060456] kthread+0x1ab/0x200 [ 1723.061149] ? cifs_handle_standard+0x2c0/0x2c0 [cifs] [ 1723.062078] ? kthread_create_on_node+0xd0/0xd0 [ 1723.062897] ret_from_fork+0x3a/0x50
--- fs/cifs/smb2misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
fs/cifs/smb2misc.c +1 -1
@@ -750,7 +750,7 @@ __smb2_handle_cancelled_close(struct cif {struct close_cancelled_open *cancelled; - cancelled =kzalloc (sizeof (*cancelled),GFP_KERNEL ); + cancelled =kzalloc (sizeof (*cancelled),GFP_ATOMIC );if (!cancelled)return -ENOMEM ;
Greg Kroah-Hartman
maintainerPATCH 31/110 ath9k: fix storage endpoint lookup
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 0ef332951e856efa89507cdd13ba8f4fb8d4db12 upstream.
Make sure to use the current alternate setting when verifying the storage interface descriptors to avoid submitting an URB to an invalid endpoint.
Failing to do so could cause the driver to misbehave or trigger a WARN() in usb_submit_urb() that kernels with panic_on_warn set would choke on.
--- drivers/net/wireless/ath/ath9k/hif_usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/ath/ath9k/hif_usb.c +1 -1
@@ -1216,7 +1216,7 @@ err_fw:static int send_eject_command (struct usb_interface *interface) {struct usb_device *udev =interface_to_usbdev (interface); -struct usb_host_interface *iface_desc = &interface->altsetting[0 ]; +struct usb_host_interface *iface_desc = interface->cur_altsetting;struct usb_endpoint_descriptor *endpoint;unsigned char *cmd;u8 bulk_out_ep;
Greg Kroah-Hartman
maintainerPATCH 32/110 brcmfmac: fix interface sanity check
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 3428fbcd6e6c0850b1a8b2a12082b7b2aabb3da3 upstream.
Make sure to use the current alternate setting when verifying the interface descriptors to avoid binding to an invalid interface.
Failing to do so could cause the driver to misbehave or trigger a WARN() in usb_submit_urb() that kernels with panic_on_warn set would choke on.
--- drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c +2 -2
@@ -1348,7 +1348,7 @@ brcmf_usb_probe(struct usb_interface *in goto fail; } - desc = &intf->altsetting[0 ].desc; + desc = &intf->cur_altsetting->desc;if ((desc->bInterfaceClass !=USB_CLASS_VENDOR_SPEC ) || (desc->bInterfaceSubClass !=2 ) || (desc->bInterfaceProtocol !=0xff )) {
@@ -1361,7 +1361,7 @@ brcmf_usb_probe(struct usb_interface *in num_of_eps = desc->bNumEndpoints;for (ep =0 ; ep < num_of_eps; ep++) { - endpoint = &intf->altsetting[0 ].endpoint[ep].desc; + endpoint = &intf->cur_altsetting->endpoint[ep].desc; endpoint_num =usb_endpoint_num (endpoint);if (!usb_endpoint_xfer_bulk (endpoint))continue ;
Greg Kroah-Hartman
maintainerPATCH 33/110 rtl8xxxu: fix interface sanity check
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 39a4281c312f2d226c710bc656ce380c621a2b16 upstream.
Make sure to use the current alternate setting when verifying the interface descriptors to avoid binding to an invalid interface.
Failing to do so could cause the driver to misbehave or trigger a WARN() in usb_submit_urb() that kernels with panic_on_warn set would choke on.
--- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c +1 -1
@@ -5915,7 +5915,7 @@ static int rtl8xxxu_parse_usb(struct rtlu8 dir, xtype, num;int ret =0 ; - host_interface = &interface->altsetting[0 ]; + host_interface = interface->cur_altsetting; interface_desc = &host_interface->desc; endpoints = interface_desc->bNumEndpoints;
Greg Kroah-Hartman
maintainerPATCH 34/110 zd1211rw: fix storage endpoint lookup
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 2d68bb2687abb747558b933e80845ff31570a49c upstream.
Make sure to use the current alternate setting when verifying the storage interface descriptors to avoid submitting an URB to an invalid endpoint.
Failing to do so could cause the driver to misbehave or trigger a WARN() in usb_submit_urb() that kernels with panic_on_warn set would choke on.
--- drivers/net/wireless/zydas/zd1211rw/zd_usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/zydas/zd1211rw/zd_usb.c +1 -1
@@ -1263,7 +1263,7 @@ static void print_id(struct usb_device *static int eject_installer (struct usb_interface *intf) {struct usb_device *udev =interface_to_usbdev (intf); -struct usb_host_interface *iface_desc = &intf->altsetting[0 ]; +struct usb_host_interface *iface_desc = intf->cur_altsetting;struct usb_endpoint_descriptor *endpoint;unsigned char *cmd;u8 bulk_out_ep;
Greg Kroah-Hartman
maintainerPATCH 35/110 net_sched: ematch: reject invalid TCF_EM_SIMPLE
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
[ Upstream commit 55cd9f67f1e45de8517cdaab985fb8e56c0bc1d8 ]
It is possible for malicious userspace to set TCF_EM_SIMPLE bit even for matches that should not have this bit set.
This can fool two places using tcf_em_is_simple()
1) tcf_em_tree_destroy() -> memory leak of em->data if ops->destroy() is NULL
2) tcf_em_tree_dump() wrongly report/leak 4 low-order bytes of a kernel pointer.
BUG: memory leak
unreferenced object 0xffff888121850a40 (size 32):
comm "syz-executor927", pid 7193, jiffies 4294941655 (age 19.840s)
hex dump (first 32 bytes):
00 00 00 00 01 00 00 00 00 00 00 00 00 00 00 00 ................
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
backtrace:
[<00000000f67036ea>] kmemleak_alloc_recursive include/linux/kmemleak.h:43 [inline]
[<00000000f67036ea>] slab_post_alloc_hook mm/slab.h:586 [inline]
[<00000000f67036ea>] slab_alloc mm/slab.c:3320 [inline]
[<00000000f67036ea>] __do_kmalloc mm/slab.c:3654 [inline]
[<00000000f67036ea>] __kmalloc_track_caller+0x165/0x300 mm/slab.c:3671
[<00000000fab0cc8e>] kmemdup+0x27/0x60 mm/util.c:127
[<00000000d9992e0a>] kmemdup include/linux/string.h:453 [inline]
[<00000000d9992e0a>] em_nbyte_change+0x5b/0x90 net/sched/em_nbyte.c:32
[<000000007e04f711>] tcf_em_validate net/sched/ematch.c:241 [inline]
[<000000007e04f711>] tcf_em_tree_validate net/sched/ematch.c:359 [inline]
[<000000007e04f711>] tcf_em_tree_validate+0x332/0x46f net/sched/ematch.c:300
[<000000007a769204>] basic_set_parms net/sched/cls_basic.c:157 [inline]
[<000000007a769204>] basic_change+0x1d7/0x5f0 net/sched/cls_basic.c:219
[<00000000e57a5997>] tc_new_tfilter+0x566/0xf70 net/sched/cls_api.c:2104
[<0000000074b68559>] rtnetlink_rcv_msg+0x3b2/0x4b0 net/core/rtnetlink.c:5415
[<00000000b7fe53fb>] netlink_rcv_skb+0x61/0x170 net/netlink/af_netlink.c:2477
[<00000000e83a40d0>] rtnetlink_rcv+0x1d/0x30 net/core/rtnetlink.c:5442
[<00000000d62ba933>] netlink_unicast_kernel net/netlink/af_netlink.c:1302 [inline]
[<00000000d62ba933>] netlink_unicast+0x223/0x310 net/netlink/af_netlink.c:1328
[<0000000088070f72>] netlink_sendmsg+0x2c0/0x570 net/netlink/af_netlink.c:1917
[<00000000f70b15ea>] sock_sendmsg_nosec net/socket.c:639 [inline]
[<00000000f70b15ea>] sock_sendmsg+0x54/0x70 net/socket.c:659
[<00000000ef95a9be>] ____sys_sendmsg+0x2d0/0x300 net/socket.c:2330
[<00000000b650f1ab>] ___sys_sendmsg+0x8a/0xd0 net/socket.c:2384
[<0000000055bfa74a>] __sys_sendmsg+0x80/0xf0 net/socket.c:2417
[<000000002abac183>] __do_sys_sendmsg net/socket.c:2426 [inline]
[<000000002abac183>] __se_sys_sendmsg net/socket.c:2424 [inline]
[<000000002abac183>] __x64_sys_sendmsg+0x23/0x30 net/socket.c:2424--- net/sched/ematch.c | 3 +++ 1 file changed, 3 insertions(+)
net/sched/ematch.c +3 -0
@@ -238,6 +238,9 @@ static int tcf_em_validate(struct tcf_pr goto errout;if (em->ops->change) { + err = -EINVAL ; +if (em_hdr->flags &TCF_EM_SIMPLE ) + goto errout; err = em->ops->change (net, data, data_len, em);if (err <0 ) goto errout;
Greg Kroah-Hartman
maintainerPATCH 36/110 net_sched: fix ops->bind_class() implementations
In reply to Greg Kroah-HartmanFrom: Cong Wang <[email protected]>
[ Upstream commit 2e24cd755552350b94a7617617c6877b8cbcb701 ]
The current implementations of ops->bind_class() are merely searching for classid and updating class in the struct tcf_result, without invoking either of cl_ops->bind_tcf() or cl_ops->unbind_tcf(). This breaks the design of them as qdisc's like cbq use them to count filters too. This is why syzbot triggered the warning in cbq_destroy_class().
In order to fix this, we have to call cl_ops->bind_tcf() and cl_ops->unbind_tcf() like the filter binding path. This patch does so by refactoring out two helper functions __tcf_bind_filter() and __tcf_unbind_filter(), which are lockless and accept a Qdisc pointer, then teaching each implementation to call them correctly.
Note, we merely pass the Qdisc pointer as an opaque pointer to each filter, they only need to pass it down to the helper functions without understanding it at all.
--- include/net/pkt_cls.h | 33 +++++++++++++++++++-------------- include/net/sch_generic.h | 3 ++- net/sched/cls_basic.c | 11 ++++++++--- net/sched/cls_bpf.c | 11 ++++++++--- net/sched/cls_flower.c | 11 ++++++++--- net/sched/cls_fw.c | 11 ++++++++--- net/sched/cls_matchall.c | 11 ++++++++--- net/sched/cls_route.c | 11 ++++++++--- net/sched/cls_rsvp.h | 11 ++++++++--- net/sched/cls_tcindex.c | 11 ++++++++--- net/sched/cls_u32.c | 11 ++++++++--- net/sched/sch_api.c | 6 ++++-- 12 files changed, 97 insertions(+), 44 deletions(-)
include/net/pkt_cls.h +19 -14
@@ -141,31 +141,38 @@ __cls_set_class(unsigned long *clp, unsireturn xchg (clp, cl); } -static inline unsigned long -cls_set_class (struct Qdisc *q,unsigned long *clp,unsigned long cl) +static inline void +__tcf_bind_filter (struct Qdisc *q,struct tcf_result *r,unsigned long base) { -unsigned long old_cl; +unsigned long cl; -sch_tree_lock (q); - old_cl =__cls_set_class (clp, cl); -sch_tree_unlock (q); -return old_cl; + cl = q->ops->cl_ops->bind_tcf (q, base, r->classid); + cl =__cls_set_class (&r->class, cl); +if (cl) + q->ops->cl_ops->unbind_tcf (q, cl); }static inline void tcf_bind_filter (struct tcf_proto *tp,struct tcf_result *r,unsigned long base) {struct Qdisc *q = tp->chain->block->q; -unsigned long cl;/* Check q as it is not set for shared blocks. In that case, * setting class is not supported. */ if (!q)return ; - cl = q->ops->cl_ops->bind_tcf (q, base, r->classid); - cl =cls_set_class (q, &r->class, cl); -if (cl) +sch_tree_lock (q); +__tcf_bind_filter (q, r, base); +sch_tree_unlock (q); +} + +static inline void +__tcf_unbind_filter (struct Qdisc *q,struct tcf_result *r) +{ +unsigned long cl; + +if ((cl =__cls_set_class (&r->class,0 )) !=0 ) q->ops->cl_ops->unbind_tcf (q, cl); }
@@ -173,12 +180,10 @@ static inline void tcf_unbind_filter(struct tcf_proto *tp, struct tcf_result *r) { struct Qdisc *q = tp->chain->block->q; -unsigned long cl;if (!q)return ; -if ((cl =__cls_set_class (&r->class,0 )) !=0 ) - q->ops->cl_ops->unbind_tcf (q, cl); +__tcf_unbind_filter (q, r); } struct tcf_exts {
include/net/sch_generic.h +2 -1
@@ -318,7 +318,8 @@ struct tcf_proto_ops { void *type_data);void (*hw_del)(struct tcf_proto *tp,void *type_data); -void (*bind_class)(void *,u32 ,unsigned long ); +void (*bind_class)(void *,u32 ,unsigned long , +void *,unsigned long ); void * (*tmplt_create)(struct net *net, struct tcf_chain *chain, struct nlattr **tca,
net/sched/cls_basic.c +8 -3
@@ -263,12 +263,17 @@ skip: } } -static void basic_bind_class (void *fh,u32 classid,unsigned long cl) +static void basic_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct basic_filter *f = fh; -if (f && f->res.classid == classid) - f->res.class = cl; +if (f && f->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &f->res, base); +else +__tcf_unbind_filter (q, &f->res); + } } static int basic_dump(struct net *net, struct tcf_proto *tp, void *fh,
net/sched/cls_bpf.c +8 -3
@@ -631,12 +631,17 @@ nla_put_failure:return -1 ; } -static void cls_bpf_bind_class (void *fh,u32 classid,unsigned long cl) +static void cls_bpf_bind_class (void *fh,u32 classid,unsigned long cl, +void *q,unsigned long base) {struct cls_bpf_prog *prog = fh; -if (prog && prog->res.classid == classid) - prog->res.class = cl; +if (prog && prog->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &prog->res, base); +else +__tcf_unbind_filter (q, &prog->res); + } } static void cls_bpf_walk(struct tcf_proto *tp, struct tcf_walker *arg,
net/sched/cls_flower.c +8 -3
@@ -2511,12 +2511,17 @@ nla_put_failure:return -EMSGSIZE ; } -static void fl_bind_class (void *fh,u32 classid,unsigned long cl) +static void fl_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct cls_fl_filter *f = fh; -if (f && f->res.classid == classid) - f->res.class = cl; +if (f && f->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &f->res, base); +else +__tcf_unbind_filter (q, &f->res); + } }static bool fl_delete_empty (struct tcf_proto *tp)
net/sched/cls_fw.c +8 -3
@@ -419,12 +419,17 @@ nla_put_failure:return -1 ; } -static void fw_bind_class (void *fh,u32 classid,unsigned long cl) +static void fw_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct fw_filter *f = fh; -if (f && f->res.classid == classid) - f->res.class = cl; +if (f && f->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &f->res, base); +else +__tcf_unbind_filter (q, &f->res); + } }static struct tcf_proto_ops cls_fw_ops__read_mostly = {
net/sched/cls_matchall.c +8 -3
@@ -393,12 +393,17 @@ nla_put_failure:return -1 ; } -static void mall_bind_class (void *fh,u32 classid,unsigned long cl) +static void mall_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct cls_mall_head *head = fh; -if (head && head->res.classid == classid) - head->res.class = cl; +if (head && head->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &head->res, base); +else +__tcf_unbind_filter (q, &head->res); + } }static struct tcf_proto_ops cls_mall_ops__read_mostly = {
net/sched/cls_route.c +8 -3
@@ -641,12 +641,17 @@ nla_put_failure:return -1 ; } -static void route4_bind_class (void *fh,u32 classid,unsigned long cl) +static void route4_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct route4_filter *f = fh; -if (f && f->res.classid == classid) - f->res.class = cl; +if (f && f->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &f->res, base); +else +__tcf_unbind_filter (q, &f->res); + } }static struct tcf_proto_ops cls_route4_ops__read_mostly = {
net/sched/cls_rsvp.h +8 -3
@@ -738,12 +738,17 @@ nla_put_failure:return -1 ; } -static void rsvp_bind_class (void *fh,u32 classid,unsigned long cl) +static void rsvp_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct rsvp_filter *f = fh; -if (f && f->res.classid == classid) - f->res.class = cl; +if (f && f->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &f->res, base); +else +__tcf_unbind_filter (q, &f->res); + } }static struct tcf_proto_ops RSVP_OPS __read_mostly = {
net/sched/cls_tcindex.c +8 -3
@@ -654,12 +654,17 @@ nla_put_failure:return -1 ; } -static void tcindex_bind_class (void *fh,u32 classid,unsigned long cl) +static void tcindex_bind_class (void *fh,u32 classid,unsigned long cl, +void *q,unsigned long base) {struct tcindex_filter_result *r = fh; -if (r && r->res.classid == classid) - r->res.class = cl; +if (r && r->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &r->res, base); +else +__tcf_unbind_filter (q, &r->res); + } }static struct tcf_proto_ops cls_tcindex_ops__read_mostly = {
net/sched/cls_u32.c +8 -3
@@ -1255,12 +1255,17 @@ static int u32_reoffload(struct tcf_protreturn 0 ; } -static void u32_bind_class (void *fh,u32 classid,unsigned long cl) +static void u32_bind_class (void *fh,u32 classid,unsigned long cl,void *q, +unsigned long base) {struct tc_u_knode *n = fh; -if (n && n->res.classid == classid) - n->res.class = cl; +if (n && n->res.classid == classid) { +if (cl) +__tcf_bind_filter (q, &n->res, base); +else +__tcf_unbind_filter (q, &n->res); + } } static int u32_dump(struct net *net, struct tcf_proto *tp, void *fh,
net/sched/sch_api.c +4 -2
@@ -1891,8 +1891,9 @@ static int tclass_del_notify(struct netstruct tcf_bind_args {struct tcf_walker w; -u32 classid; +unsigned long base;unsigned long cl; +u32 classid; };static int tcf_node_bind (struct tcf_proto *tp,void *n,struct tcf_walker *arg)
@@ -1903,7 +1904,7 @@ static int tcf_node_bind(struct tcf_protstruct Qdisc *q =tcf_block_q (tp->chain->block);sch_tree_lock (q); - tp->ops->bind_class (n, a->classid, a->cl); + tp->ops->bind_class (n, a->classid, a->cl, q, a->base);sch_tree_unlock (q); }return 0 ;
@@ -1936,6 +1937,7 @@ static void tc_bind_tclass(struct Qdisc arg.w.fn = tcf_node_bind; arg.classid = clid; + arg.base = cl; arg.cl = new_cl; tp->ops->walk (tp, &arg.w, true); }
Greg Kroah-Hartman
maintainerPATCH 37/110 net_sched: walk through all child classes in tc_bind_tclass()
In reply to Greg Kroah-HartmanFrom: Cong Wang <[email protected]>
[ Upstream commit 760d228e322e99cdf6d81b4b60a268b8f13cf67a ]
In a complex TC class hierarchy like this:
tc qdisc add dev eth0 root handle 1:0 cbq bandwidth 100Mbit \
avpkt 1000 cell 8
tc class add dev eth0 parent 1:0 classid 1:1 cbq bandwidth 100Mbit \
rate 6Mbit weight 0.6Mbit prio 8 allot 1514 cell 8 maxburst 20 \
avpkt 1000 bounded
tc filter add dev eth0 parent 1:0 protocol ip prio 1 u32 match ip \
sport 80 0xffff flowid 1:3
tc filter add dev eth0 parent 1:0 protocol ip prio 1 u32 match ip \
sport 25 0xffff flowid 1:4
tc class add dev eth0 parent 1:1 classid 1:3 cbq bandwidth 100Mbit \
rate 5Mbit weight 0.5Mbit prio 5 allot 1514 cell 8 maxburst 20 \
avpkt 1000
tc class add dev eth0 parent 1:1 classid 1:4 cbq bandwidth 100Mbit \
rate 3Mbit weight 0.3Mbit prio 5 allot 1514 cell 8 maxburst 20 \
avpkt 1000
where filters are installed on qdisc 1:0, so we can't merely search from class 1:1 when creating class 1:3 and class 1:4. We have to walk through all the child classes of the direct parent qdisc. Otherwise we would miss filters those need reverse binding.
--- net/sched/sch_api.c | 41 ++++++++++++++++++++++++++++++----------- 1 file changed, 30 insertions(+), 11 deletions(-)
net/sched/sch_api.c +30 -11
@@ -1910,22 +1910,24 @@ static int tcf_node_bind(struct tcf_protreturn 0 ; } -static void tc_bind_tclass (struct Qdisc *q,u32 portid,u32 clid, -unsigned long new_cl) +struct tc_bind_class_args { +struct qdisc_walker w; +unsigned long new_cl; +u32 portid; +u32 clid; +}; + +static int tc_bind_class_walker (struct Qdisc *q,unsigned long cl, +struct qdisc_walker *w) { +struct tc_bind_class_args *a = (struct tc_bind_class_args *)w;const struct Qdisc_class_ops *cops = q->ops->cl_ops;struct tcf_block *block;struct tcf_chain *chain; -unsigned long cl; - cl = cops->find (q, portid); -if (!cl) -return ; -if (!cops->tcf_block) -return ; block = cops->tcf_block (q, cl,NULL );if (!block) -return ; +return 0 ;for (chain =tcf_get_next_chain (block,NULL ); chain; chain =tcf_get_next_chain (block, chain)) {
@@ -1936,12 +1938,29 @@ static void tc_bind_tclass(struct Qdiscstruct tcf_bind_args arg = {}; arg.w.fn = tcf_node_bind; - arg.classid = clid; + arg.classid = a->clid; arg.base = cl; - arg.cl = new_cl; + arg.cl = a->new_cl; tp->ops->walk(tp, &arg.w, true); } } + +return 0 ; +} + +static void tc_bind_tclass (struct Qdisc *q,u32 portid,u32 clid, +unsigned long new_cl) +{ +const struct Qdisc_class_ops *cops = q->ops->cl_ops; +struct tc_bind_class_args args = {}; + +if (!cops->tcf_block) +return ; + args.portid = portid; + args.clid = clid; + args.new_cl = new_cl; + args.w.fn = tc_bind_class_walker; + q->ops->cl_ops->walk (q, &args.w); }#else
Greg Kroah-Hartman
maintainerPATCH 38/110 net: socionext: fix possible user-after-free in netsec_process_rx
In reply to Greg Kroah-HartmanFrom: Lorenzo Bianconi <[email protected]>
[ Upstream commit b5e82e3c89c78ee0407ea8e8087af5519b6c7bae ]
Fix possible use-after-free in in netsec_process_rx that can occurs if the first packet is sent to the normal networking stack and the following one is dropped by the bpf program attached to the xdp hook. Fix the issue defining the skb pointer in the 'budget' loop
--- drivers/net/ethernet/socionext/netsec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/socionext/netsec.c +1 -1
@@ -928,7 +928,6 @@ static int netsec_process_rx(struct netsstruct netsec_rx_pkt_info rx_info;enum dma_data_direction dma_dir;struct bpf_prog *xdp_prog; -struct sk_buff *skb =NULL ;u16 xdp_xmit =0 ;u32 xdp_act =0 ;int done =0 ;
@@ -942,6 +941,7 @@ static int netsec_process_rx(struct netsstruct netsec_de *de = dring->vaddr + (DESC_SZ * idx);struct netsec_desc *desc = &dring->desc[idx];struct page *page =virt_to_page (desc->addr); +struct sk_buff *skb =NULL ;u32 xdp_result =XDP_PASS ;u16 pkt_len, desc_len;dma_addr_t dma_handle;
Greg Kroah-Hartman
maintainerPATCH 39/110 net: socionext: fix xdp_result initialization in netsec_process_rx
In reply to Greg Kroah-HartmanFrom: Lorenzo Bianconi <[email protected]>
[ Upstream commit 02758cb6dac31a2b4bd9e535cffbe718acd46404 ]
Fix xdp_result initialization in netsec_process_rx in order to not increase rx counters if there is no bpf program attached to the xdp hook and napi_gro_receive returns GRO_DROP
--- drivers/net/ethernet/socionext/netsec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/socionext/netsec.c +1 -1
@@ -941,8 +941,8 @@ static int netsec_process_rx(struct netsstruct netsec_de *de = dring->vaddr + (DESC_SZ * idx);struct netsec_desc *desc = &dring->desc[idx];struct page *page =virt_to_page (desc->addr); +u32 xdp_result =NETSEC_XDP_PASS ;struct sk_buff *skb =NULL ; -u32 xdp_result =XDP_PASS ;u16 pkt_len, desc_len;dma_addr_t dma_handle;struct xdp_buff xdp;
Greg Kroah-Hartman
maintainerPATCH 40/110 udp: segment looped gso packets correctly
In reply to Greg Kroah-HartmanFrom: Willem de Bruijn <[email protected]>
[ Upstream commit 6cd021a58c18a1731f7e47f83e172c0c302d65e5 ]
Multicast and broadcast packets can be looped from egress to ingress pre segmentation with dev_loopback_xmit. That function unconditionally sets ip_summed to CHECKSUM_UNNECESSARY.
udp_rcv_segment segments gso packets in the udp rx path. Segmentation usually executes on egress, and does not expect packets of this type. __udp_gso_segment interprets !CHECKSUM_PARTIAL as CHECKSUM_NONE. But the offsets are not correct for gso_make_checksum.
UDP GSO packets are of type CHECKSUM_PARTIAL, with their uh->check set to the correct pseudo header checksum. Reset ip_summed to this type. (CHECKSUM_PARTIAL is allowed on ingress, see comments in skbuff.h)
--- include/net/udp.h | 3 +++ 1 file changed, 3 insertions(+)
include/net/udp.h +3 -0
@@ -476,6 +476,9 @@ static inline struct sk_buff *udp_rcv_seif (!inet_get_convert_csum (sk)) features |=NETIF_F_IP_CSUM |NETIF_F_IPV6_CSUM ; +if (skb->pkt_type ==PACKET_LOOPBACK ) + skb->ip_summed =CHECKSUM_PARTIAL ; +/* the GSO CB lays after the UDP one, no need to save and restore any * CB fragment */
Greg Kroah-Hartman
maintainerPATCH 41/110 mlxsw: minimal: Fix an error handling path in mlxsw_m_port_create()
In reply to Greg Kroah-HartmanFrom: Christophe JAILLET <[email protected]>
[ Upstream commit 6dd4b4f3936e17fedea1308bc70e9716f68bf232 ]
An 'alloc_etherdev()' called is not ballanced by a corresponding 'free_netdev()' call in one error handling path.
Slighly reorder the error handling code to catch the missed case.
--- drivers/net/ethernet/mellanox/mlxsw/minimal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/mellanox/mlxsw/minimal.c +1 -1
@@ -204,8 +204,8 @@ mlxsw_m_port_create(struct mlxsw_m *mlxs err_register_netdev: mlxsw_m->ports[local_port] =NULL ; -free_netdev (dev); err_dev_addr_get: +free_netdev (dev); err_alloc_etherdev:mlxsw_core_port_fini (mlxsw_m->core, local_port);return err;
Greg Kroah-Hartman
maintainerPATCH 42/110 net: include struct nhmsg size in nh nlmsg size
In reply to Greg Kroah-HartmanFrom: Stephen Worley <[email protected]>
[ Upstream commit f9e95555757915fc194288862d2978e370fe316b ]
Include the size of struct nhmsg size when calculating how much of a payload to allocate in a new netlink nexthop notification message.
Without this, we will fail to fill the skbuff at certain nexthop group sizes.
You can reproduce the failure with the following iproute2 commands:
ip link add dummy1 type dummy
ip link add dummy2 type dummy
ip link add dummy3 type dummy
ip link add dummy4 type dummy
ip link add dummy5 type dummy
ip link add dummy6 type dummy
ip link add dummy7 type dummy
ip link add dummy8 type dummy
ip link add dummy9 type dummy
ip link add dummy10 type dummy
ip link add dummy11 type dummy
ip link add dummy12 type dummy
ip link add dummy13 type dummy
ip link add dummy14 type dummy
ip link add dummy15 type dummy
ip link add dummy16 type dummy
ip link add dummy17 type dummy
ip link add dummy18 type dummy
ip link add dummy19 type dummy
ip ro add 1.1.1.1/32 dev dummy1
ip ro add 1.1.1.2/32 dev dummy2
ip ro add 1.1.1.3/32 dev dummy3
ip ro add 1.1.1.4/32 dev dummy4
ip ro add 1.1.1.5/32 dev dummy5
ip ro add 1.1.1.6/32 dev dummy6
ip ro add 1.1.1.7/32 dev dummy7
ip ro add 1.1.1.8/32 dev dummy8
ip ro add 1.1.1.9/32 dev dummy9
ip ro add 1.1.1.10/32 dev dummy10 ip ro add 1.1.1.11/32 dev dummy11 ip ro add 1.1.1.12/32 dev dummy12 ip ro add 1.1.1.13/32 dev dummy13 ip ro add 1.1.1.14/32 dev dummy14 ip ro add 1.1.1.15/32 dev dummy15 ip ro add 1.1.1.16/32 dev dummy16 ip ro add 1.1.1.17/32 dev dummy17 ip ro add 1.1.1.18/32 dev dummy18 ip ro add 1.1.1.19/32 dev dummy19
ip next add id 1 via 1.1.1.1 dev dummy1 ip next add id 2 via 1.1.1.2 dev dummy2 ip next add id 3 via 1.1.1.3 dev dummy3 ip next add id 4 via 1.1.1.4 dev dummy4 ip next add id 5 via 1.1.1.5 dev dummy5 ip next add id 6 via 1.1.1.6 dev dummy6 ip next add id 7 via 1.1.1.7 dev dummy7 ip next add id 8 via 1.1.1.8 dev dummy8 ip next add id 9 via 1.1.1.9 dev dummy9 ip next add id 10 via 1.1.1.10 dev dummy10 ip next add id 11 via 1.1.1.11 dev dummy11 ip next add id 12 via 1.1.1.12 dev dummy12 ip next add id 13 via 1.1.1.13 dev dummy13 ip next add id 14 via 1.1.1.14 dev dummy14 ip next add id 15 via 1.1.1.15 dev dummy15 ip next add id 16 via 1.1.1.16 dev dummy16 ip next add id 17 via 1.1.1.17 dev dummy17 ip next add id 18 via 1.1.1.18 dev dummy18 ip next add id 19 via 1.1.1.19 dev dummy19
ip next add id 1111 group 1/2/3/4/5/6/7/8/9/10/11/12/13/14/15/16/17/18/19 ip next del id 1111
--- net/ipv4/nexthop.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
net/ipv4/nexthop.c +3 -1
@@ -322,7 +322,9 @@ static size_t nh_nlmsg_size_single(strucstatic size_t nh_nlmsg_size (struct nexthop *nh) { -size_t sz =nla_total_size (4 );/* NHA_ID */ +size_t sz =NLMSG_ALIGN (sizeof (struct nhmsg )); + + sz +=nla_total_size (4 );/* NHA_ID */ if (nh->is_group) sz +=nh_nlmsg_size_grp (nh);
Greg Kroah-Hartman
maintainerPATCH 43/110 rxrpc: Fix use-after-free in rxrpc_receive_data()
In reply to Greg Kroah-HartmanFrom: David Howells <[email protected]>
[ Upstream commit 122d74fac84204b9a98263636f6f9a3b2e665639 ]
The subpacket scanning loop in rxrpc_receive_data() references the subpacket count in the private data part of the sk_buff in the loop termination condition. However, when the final subpacket is pasted into the ring buffer, the function is no longer has a ref on the sk_buff and should not be looking at sp->* any more. This point is actually marked in the code when skb is cleared (but sp is not - which is an error).
Fix this by caching sp->nr_subpackets in a local variable and using that instead.
Also clear 'sp' to catch accesses after that point.
This can show up as an oops in rxrpc_get_skb() if sp->nr_subpackets gets trashed by the sk_buff getting freed and reused in the meantime.
--- net/rxrpc/input.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-)
net/rxrpc/input.c +7 -5
@@ -413,7 +413,7 @@ static void rxrpc_input_data(struct rxrp {struct rxrpc_skb_priv *sp =rxrpc_skb (skb);enum rxrpc_call_state state; -unsigned int j; +unsigned int j, nr_subpackets;rxrpc_serial_t serial = sp->hdr.serial, ack_serial =0 ;rxrpc_seq_t seq0 = sp->hdr.seq, hard_ack;bool immediate_ack = false, jumbo_bad = false;
@@ -457,7 +457,8 @@ static void rxrpc_input_data(struct rxrp call->ackr_prev_seq = seq0; hard_ack =READ_ONCE (call->rx_hard_ack); -if (sp->nr_subpackets >1 ) { + nr_subpackets = sp->nr_subpackets; +if (nr_subpackets >1 ) {if (call->nr_jumbo_bad >3 ) { ack =RXRPC_ACK_NOSPACE ; ack_serial = serial;
@@ -465,11 +466,11 @@ static void rxrpc_input_data(struct rxrp } } -for (j =0 ; j < sp->nr_subpackets; j++) { +for (j =0 ; j < nr_subpackets; j++) {rxrpc_serial_t serial = sp->hdr.serial + j;rxrpc_seq_t seq = seq0 + j;unsigned int ix = seq &RXRPC_RXTX_BUFF_MASK ; -bool terminal = (j == sp->nr_subpackets -1 ); +bool terminal = (j == nr_subpackets -1 );bool last = terminal && (sp->rx_flags &RXRPC_SKB_INCL_LAST );u8 flags, annotation = j;
@@ -506,7 +507,7 @@ static void rxrpc_input_data(struct rxrp }if (call->rxtx_buffer[ix]) { -rxrpc_input_dup_data (call, seq, sp->nr_subpackets >1 , +rxrpc_input_dup_data (call, seq, nr_subpackets >1 , &jumbo_bad);if (ack !=RXRPC_ACK_DUPLICATE ) { ack =RXRPC_ACK_DUPLICATE ;
@@ -564,6 +565,7 @@ static void rxrpc_input_data(struct rxrp* ring. */ skb =NULL ; + sp =NULL ; }if (last) {
Greg Kroah-Hartman
maintainerPATCH 44/110 HID: multitouch: Add LG MELF0410 I2C touchscreen support
In reply to Greg Kroah-HartmanFrom: Aaron Ma <[email protected]>
[ Upstream commit 348b80b273fbf4ce2a307f9e38eadecf37828cad ]
Add multitouch support for LG MELF I2C touchscreen. Apply the same workaround as LG USB touchscreen.
--- drivers/hid/hid-ids.h | 1 + drivers/hid/hid-multitouch.c | 3 +++ 2 files changed, 4 insertions(+)
drivers/hid/hid-ids.h +1 -0
@@ -730,6 +730,7 @@#define USB_DEVICE_ID_LG_MULTITOUCH 0x0064#define USB_DEVICE_ID_LG_MELFAS_MT 0x6007#define I2C_DEVICE_ID_LG_8001 0x8001 +#define I2C_DEVICE_ID_LG_7010 0x7010#define USB_VENDOR_ID_LOGITECH 0x046d#define USB_DEVICE_ID_LOGITECH_AUDIOHUB 0x0a0e
drivers/hid/hid-multitouch.c +3 -0
@@ -1985,6 +1985,9 @@ static const struct hid_device_id mt_devices[] = { { .driver_data = MT_CLS_LG,HID_USB_DEVICE (USB_VENDOR_ID_LG , USB_DEVICE_ID_LG_MELFAS_MT) }, + { .driver_data = MT_CLS_LG, +HID_DEVICE (BUS_I2C ,HID_GROUP_GENERIC , + USB_VENDOR_ID_LG, I2C_DEVICE_ID_LG_7010) }, /* MosArt panels */ { .driver_data = MT_CLS_CONFIDENCE_MINUS_ONE,
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 45/110 arc: eznps: fix allmodconfig kconfig warning
In reply to Greg Kroah-HartmanFrom: Randy Dunlap <[email protected]>
[ Upstream commit 1928b36cfa4df1aeedf5f2644d0c33f3a1fcfd7b ]
Fix kconfig warning for arch/arc/plat-eznps/Kconfig allmodconfig:
WARNING: unmet direct dependencies detected for CLKSRC_NPS
Depends on [n]: GENERIC_CLOCKEVENTS [=y] && !PHYS_ADDR_T_64BIT [=y] Selected by [y]:
- ARC_PLAT_EZNPS [=y]
--- arch/arc/plat-eznps/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arc/plat-eznps/Kconfig +1 -1
@@ -7,7 +7,7 @@ menuconfig ARC_PLAT_EZNPS bool "\"EZchip\" ARC dev platform" select CPU_BIG_ENDIAN - select CLKSRC_NPS + select CLKSRC_NPS if !PHYS_ADDR_T_64BIT select EZNPS_GIC select EZCHIP_NPS_MANAGEMENT_ENET if ETHERNET help
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 46/110 HID: Add quirk for Xin-Mo Dual Controller
In reply to Greg Kroah-HartmanFrom: Priit Laes <[email protected]>
[ Upstream commit c62f7cd8ed066a93a243643ebf57ca99f754388e ]
Without the quirk, joystick shows up as single controller for both first and second player pads/pins.
--- drivers/hid/hid-quirks.c | 1 + 1 file changed, 1 insertion(+)
drivers/hid/hid-quirks.c +1 -0
@@ -174,6 +174,7 @@ static const struct hid_device_id hid_quirks[] = { { HID_USB_DEVICE(USB_VENDOR_ID_WALTOP, USB_DEVICE_ID_WALTOP_SIRIUS_BATTERY_FREE_TABLET), HID_QUIRK_MULTI_INPUT }, { HID_USB_DEVICE(USB_VENDOR_ID_WISEGROUP_LTD2, USB_DEVICE_ID_SMARTJOY_DUAL_PLUS), HID_QUIRK_NOGET | HID_QUIRK_MULTI_INPUT }, { HID_USB_DEVICE(USB_VENDOR_ID_WISEGROUP, USB_DEVICE_ID_QUAD_USB_JOYPAD), HID_QUIRK_NOGET | HID_QUIRK_MULTI_INPUT }, + { HID_USB_DEVICE(USB_VENDOR_ID_XIN_MO, USB_DEVICE_ID_XIN_MO_DUAL_ARCADE), HID_QUIRK_MULTI_INPUT }, { 0 } };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 47/110 HID: ite: Add USB id match for Acer SW5-012 keyboard dock
In reply to Greg Kroah-HartmanFrom: Hans de Goede <[email protected]>
[ Upstream commit 8f18eca9ebc57d6b150237033f6439242907e0ba ]
The Acer SW5-012 2-in-1 keyboard dock uses a Synaptics S91028 touchpad which is connected to an ITE 8595 USB keyboard controller chip.
This keyboard has the same quirk for its rfkill / airplane mode hotkey as other keyboards with the ITE 8595 chip, it only sends a single release event when pressed and released, it never sends a press event.
This commit adds this keyboards USB id to the hid-ite id-table, fixing the rfkill key not working on this keyboard.
--- drivers/hid/hid-ids.h | 1 + drivers/hid/hid-ite.c | 3 +++ 2 files changed, 4 insertions(+)
drivers/hid/hid-ids.h +1 -0
@@ -1099,6 +1099,7 @@#define USB_DEVICE_ID_SYNAPTICS_LTS2 0x1d10#define USB_DEVICE_ID_SYNAPTICS_HD 0x0ac3#define USB_DEVICE_ID_SYNAPTICS_QUAD_HD 0x1ac3 +#define USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012 0x2968#define USB_DEVICE_ID_SYNAPTICS_TP_V103 0x5710#define USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5 0x81a7
drivers/hid/hid-ite.c +3 -0
@@ -40,6 +40,9 @@ static int ite_event(struct hid_device *hdev, struct hid_field *field,static const struct hid_device_id ite_devices[] = { {HID_USB_DEVICE (USB_VENDOR_ID_ITE ,USB_DEVICE_ID_ITE8595 ) }, {HID_USB_DEVICE (USB_VENDOR_ID_258A ,USB_DEVICE_ID_258A_6A88 ) }, +/* ITE8595 USB kbd ctlr, with Synaptics touchpad connected to it. */ + {HID_USB_DEVICE (USB_VENDOR_ID_SYNAPTICS , +USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012 ) }, { } };MODULE_DEVICE_TABLE (hid, ite_devices);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 48/110 HID: asus: Ignore Asus vendor-page usage-code 0xff events
In reply to Greg Kroah-HartmanFrom: Hans de Goede <[email protected]>
[ Upstream commit c07a0254c89e4bb69ca781cd488baa5b628e2754 ]
At least on a T100HA an Asus vendor-page usage-code 0xff event is send on every suspend and again on resume, resulting in the following warning:
asus 0003:0B05:1807.0002: Unmapped Asus vendor usagepage code 0xff
being logged twice on every suspend/resume.
This commit silences the "Unmapped Asus vendor usagepage code ..." warning for usage-code 0xff to avoid these warnings being logged.
--- drivers/hid/hid-asus.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/hid/hid-asus.c +2 -1
@@ -261,7 +261,8 @@ static int asus_event(struct hid_device *hdev, struct hid_field *field, struct hid_usage *usage, __s32 value) {if ((usage->hid &HID_USAGE_PAGE ) ==0xff310000 && - (usage->hid &HID_USAGE ) !=0x00 && !usage->type) { + (usage->hid &HID_USAGE ) !=0x00 && + (usage->hid &HID_USAGE ) !=0xff && !usage->type) {hid_warn (hdev,"Unmapped Asus vendor usagepage code 0x%02x\n" , usage->hid &HID_USAGE ); }
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 49/110 HID: Add quirk for incorrect input length on Lenovo Y720
In reply to Greg Kroah-HartmanFrom: Pavel Balan <[email protected]>
[ Upstream commit fd0913768701612fc2b8ab9c8a5c019133e8d978 ]
Apply it to the Lenovo Y720 gaming laptop I2C peripheral then.
This fixes dmesg being flooded with errors visible on un-suspend in Linux Mint 19 Cinnamon.
Example of error log:
<...>
[ 4.326588] i2c_hid i2c-ITE33D1:00: i2c_hid_get_input: incomplete report (2/4) [ 4.326845] i2c_hid i2c-ITE33D1:00: i2c_hid_get_input: incomplete report (2/4) [ 4.327095] i2c_hid i2c-ITE33D1:00: i2c_hid_get_input: incomplete report (2/4) [ 4.327341] i2c_hid i2c-ITE33D1:00: i2c_hid_get_input: incomplete report (2/4) [ 4.327609] i2c_hid i2c-ITE33D1:00: i2c_hid_get_input: incomplete report (2/4)
<...>
Example of fixed log (debug on)
<...>
[ 3731.333183] i2c_hid i2c-ITE33D1:00: input: 02 00 [ 3731.333581] i2c_hid i2c-ITE33D1:00: input: 02 00 [ 3731.333842] i2c_hid i2c-ITE33D1:00: input: 02 00 [ 3731.334107] i2c_hid i2c-ITE33D1:00: input: 02 00 [ 3731.334367] i2c_hid i2c-ITE33D1:00: input: 02 00
<...>
--- drivers/hid/hid-ids.h | 1 + drivers/hid/i2c-hid/i2c-hid-core.c | 16 +++++++++++++--- 2 files changed, 14 insertions(+), 3 deletions(-)
drivers/hid/hid-ids.h +1 -0
@@ -631,6 +631,7 @@#define USB_VENDOR_ID_ITE 0x048d#define USB_DEVICE_ID_ITE_LENOVO_YOGA 0x8386#define USB_DEVICE_ID_ITE_LENOVO_YOGA2 0x8350 +#define I2C_DEVICE_ID_ITE_LENOVO_LEGION_Y720 0x837a#define USB_DEVICE_ID_ITE_LENOVO_YOGA900 0x8396#define USB_DEVICE_ID_ITE8595 0x8595
drivers/hid/i2c-hid/i2c-hid-core.c +13 -3
@@ -49,6 +49,8 @@#define I2C_HID_QUIRK_NO_IRQ_AFTER_RESET BIT(1)#define I2C_HID_QUIRK_BOGUS_IRQ BIT(4)#define I2C_HID_QUIRK_RESET_ON_RESUME BIT(5) +#define I2C_HID_QUIRK_BAD_INPUT_SIZE BIT(6) +/* flags */ #define I2C_HID_STARTED 0
@@ -177,6 +179,8 @@ static const struct i2c_hid_quirks { I2C_HID_QUIRK_BOGUS_IRQ }, {USB_VENDOR_ID_ALPS_JP ,HID_ANY_ID , I2C_HID_QUIRK_RESET_ON_RESUME }, + { USB_VENDOR_ID_ITE, I2C_DEVICE_ID_ITE_LENOVO_LEGION_Y720, + I2C_HID_QUIRK_BAD_INPUT_SIZE }, { 0, 0 } };
@@ -498,9 +502,15 @@ static void i2c_hid_get_input(struct i2c_hid *ihid) }if ((ret_size > size) || (ret_size <2 )) { -dev_err (&ihid->client->dev,"%s: incomplete report (%d/%d)\n" , - __func__, size, ret_size); -return ; +if (ihid->quirks &I2C_HID_QUIRK_BAD_INPUT_SIZE ) { + ihid->inbuf[0 ] = size &0xff ; + ihid->inbuf[1 ] = size >>8 ; + ret_size = size; + }else { +dev_err (&ihid->client->dev,"%s: incomplete report (%d/%d)\n" , + __func__, size, ret_size); +return ; + } }i2c_hid_dbg (ihid,"input: %*ph\n" , ret_size, ihid->inbuf);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 50/110 HID: intel-ish-hid: ipc: add CMP device id
In reply to Greg Kroah-HartmanFrom: Even Xu <[email protected]>
[ Upstream commit abb33ee8a8c0d146b4b2c52937dc86a15ec92d05 ]
Add Comet Lake H into ishtp support list.
--- drivers/hid/intel-ish-hid/ipc/hw-ish.h | 1 + drivers/hid/intel-ish-hid/ipc/pci-ish.c | 1 + 2 files changed, 2 insertions(+)
drivers/hid/intel-ish-hid/ipc/hw-ish.h +1 -0
@@ -24,6 +24,7 @@#define ICL_MOBILE_DEVICE_ID 0x34FC#define SPT_H_DEVICE_ID 0xA135#define CML_LP_DEVICE_ID 0x02FC +#define CMP_H_DEVICE_ID 0x06FC#define EHL_Ax_DEVICE_ID 0x4BB3#define REVISION_ID_CHT_A0 0x6
drivers/hid/intel-ish-hid/ipc/pci-ish.c +1 -0
@@ -34,6 +34,7 @@ static const struct pci_device_id ish_pci_tbl[] = { {PCI_DEVICE(PCI_VENDOR_ID_INTEL, ICL_MOBILE_DEVICE_ID)}, {PCI_DEVICE(PCI_VENDOR_ID_INTEL, SPT_H_DEVICE_ID)}, {PCI_DEVICE(PCI_VENDOR_ID_INTEL, CML_LP_DEVICE_ID)}, + {PCI_DEVICE(PCI_VENDOR_ID_INTEL, CMP_H_DEVICE_ID)}, {PCI_DEVICE(PCI_VENDOR_ID_INTEL, EHL_Ax_DEVICE_ID)}, {0, } };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 51/110 HID: wacom: Recognize new MobileStudio Pro PID
In reply to Greg Kroah-HartmanFrom: Jason Gerecke <[email protected]>
[ Upstream commit fe4e940f0f91b4a506f048b42e00386f5ad322b6 ]
A new PID is in use for repaired MobileStudio Pro devices. Add it to the list of devices that need special-casing in wacom_wac_pad_event.
--- drivers/hid/wacom_wac.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
drivers/hid/wacom_wac.c +4 -2
@@ -2096,14 +2096,16 @@ static void wacom_wac_pad_event(struct hid_device *hdev, struct hid_field *field (hdev->product ==0x34d || hdev->product ==0x34e ||/* MobileStudio Pro */ hdev->product ==0x357 || hdev->product ==0x358 ||/* Intuos Pro 2 */ hdev->product ==0x392 ||/* Intuos Pro 2 */ - hdev->product == 0x398 || hdev->product == 0x399)) { /* MobileStudio Pro */ + hdev->product ==0x398 || hdev->product ==0x399 ||/* MobileStudio Pro */ + hdev->product == 0x3AA)) { /* MobileStudio Pro */ value = (field->logical_maximum - value);if (hdev->product ==0x357 || hdev->product ==0x358 || hdev->product ==0x392 ) value =wacom_offset_rotation (input, usage, value,3 ,16 );else if (hdev->product ==0x34d || hdev->product ==0x34e || - hdev->product ==0x398 || hdev->product ==0x399 ) + hdev->product ==0x398 || hdev->product ==0x399 || + hdev->product ==0x3AA ) value =wacom_offset_rotation (input, usage, value,1 ,2 ); } else {
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 52/110 ASoC: SOF: fix fault at driver unload after failed probe
In reply to Greg Kroah-HartmanFrom: Kai Vehmanen <[email protected]>
[ Upstream commit b06e46427f987bf83dcb6a69516b57276eb8ec0c ]
If sof_machine_check() fails during driver probe, the IPC state is not initialized and this will lead to a NULL dereference at driver unload. Example log is as follows:
[ 1535.980630] sof-audio-pci 0000:00:1f.3: error: no matching ASoC machine driver found - aborting probe [ 1535.980631] sof-audio-pci 0000:00:1f.3: error: failed to get machine info -19 [ 1535.980632] sof-audio-pci 0000:00:1f.3: error: sof_probe_work failed err: -19 [ 1550.798373] BUG: kernel NULL pointer dereference, address: 0000000000000008 ... [ 1550.798393] Call Trace: [ 1550.798397] snd_sof_ipc_free+0x15/0x30 [snd_sof] [ 1550.798399] snd_sof_device_remove+0x29/0xa0 [snd_sof] [ 1550.798400] sof_pci_remove+0x10/0x30 [snd_sof_pci]
--- sound/soc/sof/ipc.c | 3 +++ 1 file changed, 3 insertions(+)
sound/soc/sof/ipc.c +3 -0
@@ -834,6 +834,9 @@ void snd_sof_ipc_free(struct snd_sof_dev *sdev) {struct snd_sof_ipc *ipc = sdev->ipc; +if (!ipc) +return ; +/* disable sending of ipc's */ mutex_lock (&ipc->tx_mutex); ipc->disable_ipc_tx = true;
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 53/110 ASoC: SOF: Intel: hda: hda-dai: fix oops on hda_link .hw_free
In reply to Greg Kroah-HartmanFrom: Pierre-Louis Bossart <[email protected]>
[ Upstream commit 921162c81a089aa2f442103290f1af9ba281fc9f ]
When the PCM_PARAM IPC fails while configuring the FE, the kernel oopses in the HDaudio link DMA .hw_free operation. The root cause is a NULL dma_data since the BE .hw_params was never called by the SOC core.
This error can also happen if the HDaudio link DMA configuration IPC fails in the BE .hw_params.
This patches makes sure the dma_data is properly saved in .hw_params, and tested before being use in hw_free.
GitHub issue: https://github.com/thesofproject/linux/issues/1417
--- sound/soc/sof/intel/hda-dai.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-)
sound/soc/sof/intel/hda-dai.c +9 -2
@@ -216,6 +216,8 @@ static int hda_link_hw_params(struct snd_pcm_substream *substream, link_dev =hda_link_stream_assign (bus, substream);if (!link_dev)return -EBUSY ; + +snd_soc_dai_set_dma_data (dai, substream, (void *)link_dev); } stream_tag =hdac_stream (link_dev)->stream_tag;
@@ -228,8 +230,6 @@ static int hda_link_hw_params(struct snd_pcm_substream *substream,if (ret <0 )return ret; -snd_soc_dai_set_dma_data (dai, substream, (void *)link_dev); - link =snd_hdac_ext_bus_get_link (bus, codec_dai->component->name);if (!link)return -EINVAL ;
@@ -361,6 +361,13 @@ static int hda_link_hw_free(struct snd_pcm_substream *substream, bus = hstream->bus; rtd =snd_pcm_substream_chip (substream); link_dev =snd_soc_dai_get_dma_data (dai, substream); + +if (!link_dev) { +dev_dbg (dai->dev, +"%s: link_dev is not assigned\n" , __func__); +return -EINVAL ; + } + hda_stream =hstream_to_sof_hda_stream (link_dev);/* free the link DMA channel in the FW */
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 54/110 drivers/hid/hid-multitouch.c: fix a possible null pointer access.
In reply to Greg Kroah-HartmanFrom: Pan Zhang <[email protected]>
[ Upstream commit 306d5acbfc66e7cccb4d8f91fc857206b8df80d1 ]
1002 if ((quirks & MT_QUIRK_IGNORE_DUPLICATES) && mt) {
1003 struct input_mt_slot *i_slot = &mt->slots[slotnum];
1004
1005 if (input_mt_is_active(i_slot) &&
1006 input_mt_is_used(mt, i_slot))
1007 return -EAGAIN;
1008 }
We previously assumed 'mt' could be null (see line 1002).
The following situation is similar, so add a judgement.
--- drivers/hid/hid-multitouch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/hid/hid-multitouch.c +1 -1
@@ -1019,7 +1019,7 @@ static int mt_process_slot(struct mt_device *td, struct input_dev *input, tool =MT_TOOL_DIAL ; else if (unlikely(!confidence_state)) { tool =MT_TOOL_PALM ; -if (!active && +if (!active && mt &&input_mt_is_active (&mt->slots[slotnum])) {/* * The non-confidence was reported for
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 55/110 phy: qcom-qmp: Increase PHY ready timeout
In reply to Greg Kroah-HartmanFrom: Bjorn Andersson <[email protected]>
[ Upstream commit cd217ee6867d285ceecd610fa1006975d5c683fa ]
It's typical for the QHP PHY to take slightly above 1ms to initialize, so increase the timeout of the PHY ready check to 10ms - as already done in the downstream PCIe driver.
--- drivers/phy/qualcomm/phy-qcom-qmp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/phy/qualcomm/phy-qcom-qmp.c +1 -1
@@ -66,7 +66,7 @@/* QPHY_V3_PCS_MISC_CLAMP_ENABLE register bits */ #define CLAMP_EN BIT(0)/* enables i/o clamp_n */ -#define PHY_INIT_COMPLETE_TIMEOUT 1000 +#define PHY_INIT_COMPLETE_TIMEOUT 10000#define POWER_DOWN_DELAY_US_MIN 10#define POWER_DOWN_DELAY_US_MAX 11
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 56/110 ASoC: fsl_audmix: add missed pm_runtime_disable
In reply to Greg Kroah-HartmanFrom: Chuhong Yuan <[email protected]>
[ Upstream commit 77fffa742285f2b587648d6c72b5c705633f146f ]
The driver forgets to call pm_runtime_disable in probe failure and remove.
Add the missed calls to fix it.
--- sound/soc/fsl/fsl_audmix.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
sound/soc/fsl/fsl_audmix.c +8 -1
@@ -505,15 +505,20 @@ static int fsl_audmix_probe(struct platform_device *pdev) ARRAY_SIZE(fsl_audmix_dai));if (ret) {dev_err (dev,"failed to register ASoC DAI\n" ); -return ret; + goto err_disable_pm; } priv->pdev =platform_device_register_data (dev, mdrv,0 ,NULL ,0 );if (IS_ERR (priv->pdev)) { ret =PTR_ERR (priv->pdev);dev_err (dev,"failed to register platform %s: %d\n" , mdrv, ret); + goto err_disable_pm; } +return 0 ; + +err_disable_pm: +pm_runtime_disable (dev);return ret; }
@@ -521,6 +526,8 @@ static int fsl_audmix_remove(struct platform_device *pdev) {struct fsl_audmix *priv =dev_get_drvdata (&pdev->dev); +pm_runtime_disable (&pdev->dev); +if (priv->pdev)platform_device_unregister (priv->pdev);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 57/110 ASoC: topology: Prevent use-after-free in snd_soc_get_pcm_runtime()
In reply to Greg Kroah-HartmanFrom: Dragos Tarcatu <[email protected]>
[ Upstream commit dd836ddf4e4e1c7f1eb2ae44783ccd70872ef24e ]
remove_link() is currently calling snd_soc_remove_dai_link() after it has already freed the memory for the link name. But this is later read from snd_soc_get_pcm_runtime() causing a KASAN use-after-free warning. Reorder the cleanups to fix this issue.
--- sound/soc/soc-topology.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
sound/soc/soc-topology.c +3 -3
@@ -548,12 +548,12 @@ static void remove_link(struct snd_soc_component *comp,if (dobj->ops && dobj->ops->link_unload) dobj->ops->link_unload (comp, dobj); +list_del (&dobj->list); +snd_soc_remove_dai_link (comp->card, link); +kfree (link->name);kfree (link->stream_name);kfree (link->cpus->dai_name); - -list_del (&dobj->list); -snd_soc_remove_dai_link (comp->card, link);kfree (link); }
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 58/110 phy: cpcap-usb: Prevent USB line glitches from waking up modem
In reply to Greg Kroah-HartmanFrom: Tony Lindgren <[email protected]>
[ Upstream commit 63078b6ba09e842f09df052c5728857389fddcd2 ]
The micro-USB connector on Motorola Mapphone devices can be muxed between the SoC and the mdm6600 modem. But even when used for the SoC, configuring the PHY with ID pin grounded will wake up the modem from idle state. Looks like the issue is probably caused by line glitches.
We can prevent the glitches by using a previously unknown mode of the GPIO mux to prevent the USB lines from being connected to the moden while configuring the USB PHY, and enable the USB lines after configuring the PHY.
Note that this only prevents waking up mdm6600 as regular USB A-host mode, and does not help when connected to a lapdock. The lapdock specific issue still needs to be debugged separately.
--- drivers/phy/motorola/phy-cpcap-usb.c | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-)
drivers/phy/motorola/phy-cpcap-usb.c +15 -3
@@ -115,7 +115,7 @@ struct cpcap_usb_ints_state {enum cpcap_gpio_mode {CPCAP_DM_DP ,CPCAP_MDM_RX_TX , -CPCAP_UNKNOWN , +CPCAP_UNKNOWN_DISABLED ,/* Seems to disable USB lines */ CPCAP_OTG_DM_DP , };
@@ -381,7 +381,8 @@ static int cpcap_usb_set_uart_mode(struct cpcap_phy_ddata *ddata) {int error; - error =cpcap_usb_gpio_set_mode (ddata,CPCAP_DM_DP ); +/* Disable lines to prevent glitches from waking up mdm6600 */ + error =cpcap_usb_gpio_set_mode (ddata,CPCAP_UNKNOWN_DISABLED );if (error) goto out_err;
@@ -408,6 +409,11 @@ static int cpcap_usb_set_uart_mode(struct cpcap_phy_ddata *ddata)if (error) goto out_err; +/* Enable UART mode */ + error =cpcap_usb_gpio_set_mode (ddata,CPCAP_DM_DP ); +if (error) + goto out_err; +return 0 ; out_err:
@@ -420,7 +426,8 @@ static int cpcap_usb_set_usb_mode(struct cpcap_phy_ddata *ddata) {int error; - error =cpcap_usb_gpio_set_mode (ddata,CPCAP_OTG_DM_DP ); +/* Disable lines to prevent glitches from waking up mdm6600 */ + error =cpcap_usb_gpio_set_mode (ddata,CPCAP_UNKNOWN_DISABLED );if (error)return error;
@@ -460,6 +467,11 @@ static int cpcap_usb_set_usb_mode(struct cpcap_phy_ddata *ddata)if (error) goto out_err; +/* Enable USB mode */ + error =cpcap_usb_gpio_set_mode (ddata,CPCAP_OTG_DM_DP ); +if (error) + goto out_err; +return 0 ; out_err:
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 59/110 HID: intel-ish-hid: ipc: Add Tiger Lake PCI device ID
In reply to Greg Kroah-HartmanFrom: Srinivas Pandruvada <[email protected]>
[ Upstream commit 1479a82d82df68dfac29c72c774cb8bdc17d4eb1 ]
Added Tiger Lake PCI device ID to the supported device list.
--- drivers/hid/intel-ish-hid/ipc/hw-ish.h | 1 + drivers/hid/intel-ish-hid/ipc/pci-ish.c | 1 + 2 files changed, 2 insertions(+)
drivers/hid/intel-ish-hid/ipc/hw-ish.h +1 -0
@@ -26,6 +26,7 @@#define CML_LP_DEVICE_ID 0x02FC#define CMP_H_DEVICE_ID 0x06FC#define EHL_Ax_DEVICE_ID 0x4BB3 +#define TGL_LP_DEVICE_ID 0xA0FC#define REVISION_ID_CHT_A0 0x6#define REVISION_ID_CHT_Ax_SI 0x0
drivers/hid/intel-ish-hid/ipc/pci-ish.c +1 -0
@@ -36,6 +36,7 @@ static const struct pci_device_id ish_pci_tbl[] = { {PCI_DEVICE(PCI_VENDOR_ID_INTEL, CML_LP_DEVICE_ID)}, {PCI_DEVICE(PCI_VENDOR_ID_INTEL, CMP_H_DEVICE_ID)}, {PCI_DEVICE(PCI_VENDOR_ID_INTEL, EHL_Ax_DEVICE_ID)}, + {PCI_DEVICE(PCI_VENDOR_ID_INTEL, TGL_LP_DEVICE_ID)}, {0, } };MODULE_DEVICE_TABLE (pci, ish_pci_tbl);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 60/110 watchdog: max77620_wdt: fix potential build errors
In reply to Greg Kroah-HartmanFrom: David Engraf <[email protected]>
[ Upstream commit da9e3f4e30a53cd420cf1e6961c3b4110f0f21f0 ]
max77620_wdt uses watchdog core functions. Enable CONFIG_WATCHDOG_CORE to fix potential build errors.
--- drivers/watchdog/Kconfig | 1 + 1 file changed, 1 insertion(+)
drivers/watchdog/Kconfig +1 -0
@@ -687,6 +687,7 @@ config MAX63XX_WATCHDOG config MAX77620_WATCHDOG tristate "Maxim Max77620 Watchdog Timer" depends on MFD_MAX77620 || COMPILE_TEST + select WATCHDOG_CORE help This is the driver for the Max77620 watchdog timer. Say 'Y' here to enable the watchdog timer support for
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 61/110 watchdog: rn5t618_wdt: fix module aliases
In reply to Greg Kroah-HartmanFrom: Andreas Kemnade <[email protected]>
[ Upstream commit a76dfb859cd42df6e3d1910659128ffcd2fb6ba2 ]
Platform device aliases were missing so module autoloading did not work.
--- drivers/watchdog/rn5t618_wdt.c | 1 + 1 file changed, 1 insertion(+)
drivers/watchdog/rn5t618_wdt.c +1 -0
@@ -188,6 +188,7 @@ static struct platform_driver rn5t618_wdt_driver = {module_platform_driver (rn5t618_wdt_driver); +MODULE_ALIAS ("platform:rn5t618-wdt" );MODULE_AUTHOR ("Beniamino Galvani <[email protected]>" );MODULE_DESCRIPTION ("RN5T618 watchdog driver" );MODULE_LICENSE ("GPL v2" );
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 62/110 watchdog: orion: fix platform_get_irq() complaints
In reply to Greg Kroah-HartmanFrom: Russell King <[email protected]>
[ Upstream commit dcbce5fbcc69bf2553f650004aad44bf390eca73 ]
Fix:
orion_wdt f1020300.watchdog: IRQ index 1 not found
which is caused by platform_get_irq() now complaining when optional IRQs are not found. Neither interrupt for orion is required, so make them both optional.
--- drivers/watchdog/orion_wdt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/watchdog/orion_wdt.c +2 -2
@@ -602,7 +602,7 @@ static int orion_wdt_probe(struct platform_device *pdev)set_bit (WDOG_HW_RUNNING , &dev->wdt.status);/* Request the IRQ only after the watchdog is disabled */ - irq =platform_get_irq (pdev,0 ); + irq =platform_get_irq_optional (pdev,0 );if (irq >0 ) {/* * Not all supported platforms specify an interrupt for the
@@ -617,7 +617,7 @@ static int orion_wdt_probe(struct platform_device *pdev) }/* Optional 2nd interrupt for pretimeout */ - irq =platform_get_irq (pdev,1 ); + irq =platform_get_irq_optional (pdev,1 );if (irq >0 ) { orion_wdt_info.options |=WDIOF_PRETIMEOUT ; ret = devm_request_irq(&pdev->dev, irq, orion_wdt_pre_irq,
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 63/110 usb: musb: jz4740: Silence error if code is -EPROBE_DEFER
In reply to Greg Kroah-HartmanFrom: Paul Cercueil <[email protected]>
[ Upstream commit ce03cbcb4b4fd2a3817f32366001f1ca45d213b8 ]
Avoid printing any error message if the error code is -EPROBE_DEFER.
--- drivers/usb/musb/jz4740.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
drivers/usb/musb/jz4740.c +5 -2
@@ -75,14 +75,17 @@ static struct musb_hdrc_platform_data jz4740_musb_platform_data = {static int jz4740_musb_init (struct musb *musb) {struct device *dev = musb->controller->parent; +int err;if (dev->of_node) musb->xceiv =devm_usb_get_phy_by_phandle (dev,"phys" ,0 );else musb->xceiv =devm_usb_get_phy (dev,USB_PHY_TYPE_USB2 );if (IS_ERR (musb->xceiv)) { -dev_err (dev,"No transceiver configured\n" ); -return PTR_ERR (musb->xceiv); + err =PTR_ERR (musb->xceiv); +if (err != -EPROBE_DEFER ) +dev_err (dev,"No transceiver configured: %d" , err); +return err; }/* Silicon does not implement ConfigData register.
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 64/110 can: tcan4x5x: tcan4x5x_parse_config(): reset device before register access
In reply to Greg Kroah-HartmanFrom: Sean Nyekjaer <[email protected]>
[ Upstream commit c3083124e6a1c0d6cd4fe3b3f627b488bd3b10c4 ]
It's a good idea to reset a ip-block/spi device before using it, this patch will reset the device.
And a generic reset function if needed elsewhere.
--- drivers/net/can/m_can/tcan4x5x.c | 27 ++++++++++++++++++++++++++- 1 file changed, 26 insertions(+), 1 deletion(-)
drivers/net/can/m_can/tcan4x5x.c +26 -1
@@ -164,6 +164,28 @@ static void tcan4x5x_check_wake(struct tcan4x5x_priv *priv) } } +static int tcan4x5x_reset (struct tcan4x5x_priv *priv) +{ +int ret =0 ; + +if (priv->reset_gpio) { +gpiod_set_value (priv->reset_gpio,1 ); + +/* tpulse_width minimum 30us */ +usleep_range (30 ,100 ); +gpiod_set_value (priv->reset_gpio,0 ); + }else { + ret =regmap_write (priv->regmap,TCAN4X5X_CONFIG , +TCAN4X5X_SW_RESET ); +if (ret) +return ret; + } + +usleep_range (700 ,1000 ); + +return ret; +} +static int regmap_spi_gather_write (void *context,const void *reg,size_t reg_len,const void *val,size_t val_len)
@@ -341,6 +363,7 @@ static int tcan4x5x_init(struct m_can_classdev *cdev)static int tcan4x5x_parse_config (struct m_can_classdev *cdev) {struct tcan4x5x_priv *tcan4x5x = cdev->device_data; +int ret; tcan4x5x->device_wake_gpio =devm_gpiod_get (cdev->dev,"device-wake" ,GPIOD_OUT_HIGH );
@@ -354,7 +377,9 @@ static int tcan4x5x_parse_config(struct m_can_classdev *cdev)if (IS_ERR (tcan4x5x->reset_gpio)) tcan4x5x->reset_gpio =NULL ; -usleep_range (700 ,1000 ); + ret =tcan4x5x_reset (tcan4x5x); +if (ret) +return ret; tcan4x5x->device_state_gpio = devm_gpiod_get_optional(cdev->dev, "device-state",
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 65/110 spi: spi-dw: Add lock protect dw_spi rx/tx to prevent concurrent calls
In reply to Greg Kroah-HartmanFrom: wuxu.wu <[email protected]>
[ Upstream commit 19b61392c5a852b4e8a0bf35aecb969983c5932d ]
dw_spi_irq() and dw_spi_transfer_one concurrent calls.
I find a panic in dw_writer(): txw = *(u8 *)(dws->tx), when dw->tx==null, dw->len==4, and dw->tx_end==1.
When tpm driver's message overtime dw_spi_irq() and dw_spi_transfer_one may concurrent visit dw_spi, so I think dw_spi structure lack of protection.
Otherwise dw_spi_transfer_one set dw rx/tx buffer and then open irq, store dw rx/tx instructions and other cores handle irq load dw rx/tx instructions may out of order.
[ 1025.321302] Call trace: ... [ 1025.321319] __crash_kexec+0x98/0x148 [ 1025.321323] panic+0x17c/0x314 [ 1025.321329] die+0x29c/0x2e8 [ 1025.321334] die_kernel_fault+0x68/0x78 [ 1025.321337] __do_kernel_fault+0x90/0xb0 [ 1025.321346] do_page_fault+0x88/0x500 [ 1025.321347] do_translation_fault+0xa8/0xb8 [ 1025.321349] do_mem_abort+0x68/0x118 [ 1025.321351] el1_da+0x20/0x8c [ 1025.321362] dw_writer+0xc8/0xd0 [ 1025.321364] interrupt_transfer+0x60/0x110 [ 1025.321365] dw_spi_irq+0x48/0x70 ...
--- drivers/spi/spi-dw.c | 15 ++++++++++++--- drivers/spi/spi-dw.h | 1 + 2 files changed, 13 insertions(+), 3 deletions(-)
drivers/spi/spi-dw.c +12 -3
@@ -172,9 +172,11 @@ static inline u32 rx_max(struct dw_spi *dws)static void dw_writer (struct dw_spi *dws) { -u32 max =tx_max (dws); +u32 max;u16 txw =0 ; +spin_lock (&dws->buf_lock); + max =tx_max (dws);while (max--) {/* Set the tx word if the transfer's original "tx" is not null */ if (dws->tx_end - dws->len) {
@@ -186,13 +188,16 @@ static void dw_writer(struct dw_spi *dws)dw_write_io_reg (dws,DW_SPI_DR , txw); dws->tx += dws->n_bytes; } +spin_unlock (&dws->buf_lock); }static void dw_reader (struct dw_spi *dws) { -u32 max =rx_max (dws); +u32 max;u16 rxw; +spin_lock (&dws->buf_lock); + max =rx_max (dws);while (max--) { rxw =dw_read_io_reg (dws,DW_SPI_DR );/* Care rx only if the transfer's original "rx" is not null */
@@ -204,6 +209,7 @@ static void dw_reader(struct dw_spi *dws) } dws->rx += dws->n_bytes; } +spin_unlock (&dws->buf_lock); }static void int_error_stop (struct dw_spi *dws,const char *msg)
@@ -276,18 +282,20 @@ static int dw_spi_transfer_one(struct spi_controller *master, {struct dw_spi *dws =spi_controller_get_devdata (master);struct chip_data *chip =spi_get_ctldata (spi); +unsigned long flags;u8 imask =0 ;u16 txlevel =0 ;u32 cr0;int ret; dws->dma_mapped =0 ; - +spin_lock_irqsave (&dws->buf_lock, flags); dws->tx = (void *)transfer->tx_buf; dws->tx_end = dws->tx + transfer->len; dws->rx = transfer->rx_buf; dws->rx_end = dws->rx + transfer->len; dws->len = transfer->len; +spin_unlock_irqrestore (&dws->buf_lock, flags);spi_enable_chip (dws,0 );
@@ -471,6 +479,7 @@ int dw_spi_add_host(struct device *dev, struct dw_spi *dws) dws->type =SSI_MOTO_SPI ; dws->dma_inited =0 ; dws->dma_addr = (dma_addr_t)(dws->paddr +DW_SPI_DR ); +spin_lock_init (&dws->buf_lock);spi_controller_set_devdata (master, dws);
drivers/spi/spi-dw.h +1 -0
@@ -120,6 +120,7 @@ struct dw_spi {size_t len;void *tx;void *tx_end; +spinlock_t buf_lock;void *rx;void *rx_end;int dma_mapped;
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 66/110 net: Google gve: Remove dma_wmb() before ringing doorbell
In reply to Greg Kroah-HartmanFrom: Liran Alon <[email protected]>
[ Upstream commit b54ef37b1ce892fdf6b632d566246d2f2f539910 ]
Current code use dma_wmb() to ensure Rx/Tx descriptors are visible to device before writing to doorbell.
However, these dma_wmb() are wrong and unnecessary. Therefore, they should be removed.
iowrite32be() called from gve_rx_write_doorbell()/gve_tx_put_doorbell() should guaratee that all previous writes to WB/UC memory is visible to device before the write done by iowrite32be().
E.g. On ARM64, iowrite32be() calls __iowmb() which expands to dma_wmb() and only then calls __raw_writel().
--- drivers/net/ethernet/google/gve/gve_rx.c | 2 -- drivers/net/ethernet/google/gve/gve_tx.c | 6 ------ 2 files changed, 8 deletions(-)
drivers/net/ethernet/google/gve/gve_rx.c +0 -2
@@ -418,8 +418,6 @@ bool gve_clean_rx_done(struct gve_rx_ring *rx, int budget, rx->cnt = cnt; rx->fill_cnt += work_done; -/* restock desc ring slots */ -dma_wmb ();/* Ensure descs are visible before ringing doorbell */ gve_rx_write_doorbell (priv, rx);return gve_rx_work_pending (rx); }
drivers/net/ethernet/google/gve/gve_tx.c +0 -6
@@ -487,10 +487,6 @@ netdev_tx_t gve_tx(struct sk_buff *skb, struct net_device *dev)* may have added descriptors without ringing the doorbell. */ -/* Ensure tx descs from a prior gve_tx are visible before -* ringing doorbell. -*/ -dma_wmb ();gve_tx_put_doorbell (priv, tx->q_resources, tx->req);return NETDEV_TX_BUSY ; }
@@ -505,8 +501,6 @@ netdev_tx_t gve_tx(struct sk_buff *skb, struct net_device *dev)if (!netif_xmit_stopped (tx->netdev_txq) &&netdev_xmit_more ())return NETDEV_TX_OK ; -/* Ensure tx descs are visible before ringing doorbell */ -dma_wmb ();gve_tx_put_doorbell (priv, tx->q_resources, tx->req);return NETDEV_TX_OK ; }
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 67/110 drivers/net/b44: Change to non-atomic bit operations on pwol_mask
In reply to Greg Kroah-HartmanFrom: Fenghua Yu <[email protected]>
[ Upstream commit f11421ba4af706cb4f5703de34fa77fba8472776 ]
Atomic operations that span cache lines are super-expensive on x86 (not just to the current processor, but also to other processes as all memory operations are blocked until the operation completes). Upcoming x86 processors have a switch to cause such operations to generate a #AC trap. It is expected that some real time systems will enable this mode in BIOS.
In preparation for this, it is necessary to fix code that may execute atomic instructions with operands that cross cachelines because the #AC trap will crash the kernel.
Since "pwol_mask" is local and never exposed to concurrency, there is no need to set bits in pwol_mask using atomic operations.
Directly operate on the byte which contains the bit instead of using __set_bit() to avoid any big endian concern due to type cast to unsigned long in __set_bit().
--- drivers/net/ethernet/broadcom/b44.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
drivers/net/ethernet/broadcom/b44.c +6 -3
@@ -1519,8 +1519,10 @@ static int b44_magic_pattern(u8 *macaddr, u8 *ppattern, u8 *pmask, int offset)int ethaddr_bytes =ETH_ALEN ;memset (ppattern + offset,0xff , magicsync); -for (j =0 ; j < magicsync; j++) -set_bit (len++, (unsigned long *) pmask); +for (j =0 ; j < magicsync; j++) { + pmask[len >>3 ] |=BIT (len &7 ); + len++; + }for (j =0 ; j <B44_MAX_PATTERNS ; j++) {if ((B44_PATTERN_SIZE - len) >=ETH_ALEN )
@@ -1532,7 +1534,8 @@ static int b44_magic_pattern(u8 *macaddr, u8 *ppattern, u8 *pmask, int offset)for (k =0 ; k< ethaddr_bytes; k++) { ppattern[offset + magicsync + (j *ETH_ALEN ) + k] = macaddr[k]; -set_bit (len++, (unsigned long *) pmask); + pmask[len >>3 ] |=BIT (len &7 ); + len++; } }return len -1 ;
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 68/110 net: wan: sdla: Fix cast from pointer to integer of different size
In reply to Greg Kroah-HartmanFrom: Krzysztof Kozlowski <[email protected]>
[ Upstream commit 00c0688cecadbf7ac2f5b4cdb36d912a2d3f0cca ]
Since net_device.mem_start is unsigned long, it should not be cast to int right before casting to pointer. This fixes warning (compile testing on alpha architecture):
drivers/net/wan/sdla.c: In function ‘sdla_transmit’:
drivers/net/wan/sdla.c:711:13: warning:
cast to pointer from integer of different size [-Wint-to-pointer-cast]
--- drivers/net/wan/sdla.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wan/sdla.c +1 -1
@@ -708,7 +708,7 @@ static netdev_tx_t sdla_transmit(struct sk_buff *skb,spin_lock_irqsave (&sdla_lock, flags);SDLA_WINDOW (dev, addr); - pbuf = (void *)(((int ) dev->mem_start) + (addr &SDLA_ADDR_MASK )); + pbuf = (void *)(dev->mem_start + (addr &SDLA_ADDR_MASK ));__sdla_write (dev, pbuf->buf_addr, skb->data, skb->len);SDLA_WINDOW (dev, addr); pbuf->opp_flag =1 ;
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 69/110 gpio: max77620: Add missing dependency on GPIOLIB_IRQCHIP
In reply to Greg Kroah-HartmanFrom: Dmitry Osipenko <[email protected]>
[ Upstream commit c5706c7defc79de68a115b5536376298a8fef111 ]
Driver fails to compile in a minimized kernel's configuration because of the missing dependency on GPIOLIB_IRQCHIP.
error: ‘struct gpio_chip’ has no member named ‘irq’
44 | virq = irq_find_mapping(gpio->gpio_chip.irq.domain, offset);
--- drivers/gpio/Kconfig | 1 + 1 file changed, 1 insertion(+)
drivers/gpio/Kconfig +1 -0
@@ -1120,6 +1120,7 @@ config GPIO_MADERA config GPIO_MAX77620 tristate "GPIO support for PMIC MAX77620 and MAX20024" depends on MFD_MAX77620 + select GPIOLIB_IRQCHIP help GPIO driver for MAX77620 and MAX20024 PMIC from Maxim Semiconductor. MAX77620 PMIC has 8 pins that can be configured as GPIOs. The
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 70/110 iommu/dma: fix variable cookie set but not used
In reply to Greg Kroah-HartmanFrom: Qian Cai <[email protected]>
[ Upstream commit 55817b340a31951d23d1692db45522560b1d20f9 ]
The commit c18647900ec8 ("iommu/dma: Relax locking in iommu_dma_prepare_msi()") introduced a compliation warning,
drivers/iommu/dma-iommu.c: In function 'iommu_dma_prepare_msi':drivers/iommu/dma-iommu.c:1206:27: warning: variable 'cookie' set but not used [-Wunused-but-set-variable]
struct iommu_dma_cookie *cookie;
^~~~~~--- drivers/iommu/dma-iommu.c | 3 --- 1 file changed, 3 deletions(-)
drivers/iommu/dma-iommu.c +0 -3
@@ -1177,7 +1177,6 @@ int iommu_dma_prepare_msi(struct msi_desc *desc, phys_addr_t msi_addr) {struct device *dev =msi_desc_to_dev (desc);struct iommu_domain *domain =iommu_get_domain_for_dev (dev); -struct iommu_dma_cookie *cookie;struct iommu_dma_msi_page *msi_page;static DEFINE_MUTEX (msi_prepare_lock);/* see below */
@@ -1186,8 +1185,6 @@ int iommu_dma_prepare_msi(struct msi_desc *desc, phys_addr_t msi_addr)return 0 ; } - cookie = domain->iova_cookie; -/* * In fact the whole prepare operation should already be serialised by * irq_domain_mutex further up the callchain, but that's pretty subtle
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 71/110 drm/amd/display: Reduce HDMI pixel encoding if max clock is exceeded
In reply to Greg Kroah-HartmanFrom: Thomas Anderson <[email protected]>
[ Upstream commit 840c90fce6c78bc6b2c4cb9e836d70985ed32066 ]
For high-res (8K) or HFR (4K120) displays, using uncompressed pixel formats like YCbCr444 would exceed the bandwidth of HDMI 2.0, so the "interesting" modes would be disabled, leaving only low-res or low framerate modes.
This change lowers the pixel encoding to 4:2:2 or 4:2:0 if the max TMDS clock is exceeded. Verified that 8K30 and 4K120 are now available and working with a Samsung Q900R over an HDMI 2.0b link from a Radeon 5700.
--- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 45 ++++++++++--------- 1 file changed, 23 insertions(+), 22 deletions(-)
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +23 -22
@@ -3266,27 +3266,21 @@ get_output_color_space(const struct dc_crtc_timing *dc_crtc_timing)return color_space; } -static void reduce_mode_colour_depth (struct dc_crtc_timing *timing_out) -{ -if (timing_out->display_color_depth <=COLOR_DEPTH_888 ) -return ; - - timing_out->display_color_depth--; -} - -static void adjust_colour_depth_from_display_info (struct dc_crtc_timing *timing_out, -const struct drm_display_info *info) +static bool adjust_colour_depth_from_display_info ( +struct dc_crtc_timing *timing_out, +const struct drm_display_info *info) { +enum dc_color_depth depth = timing_out->display_color_depth;int normalized_clk; -if (timing_out->display_color_depth <=COLOR_DEPTH_888 ) -return ; do { normalized_clk = timing_out->pix_clk_100hz / 10; /* YCbCr 4:2:0 requires additional adjustment of 1/2 */ if (timing_out->pixel_encoding == PIXEL_ENCODING_YCBCR420) normalized_clk /= 2; /* Adjusting pix clock following on HDMI spec based on colour depth */ - switch (timing_out->display_color_depth) { + switch (depth) { + case COLOR_DEPTH_888: + break; case COLOR_DEPTH_101010: normalized_clk = (normalized_clk * 30) / 24; break;
@@ -3297,14 +3291,15 @@ static void adjust_colour_depth_from_display_info(struct dc_crtc_timing *timing_ normalized_clk = (normalized_clk *48 ) /24 ;break ;default : -return ; +/* The above depths are the only ones valid for HDMI. */ +return false; } -if (normalized_clk <= info->max_tmds_clock) -return ; -reduce_mode_colour_depth (timing_out); - - }while (timing_out->display_color_depth >COLOR_DEPTH_888 ); - +if (normalized_clk <= info->max_tmds_clock) { + timing_out->display_color_depth = depth; +return true; + } + }while (--depth >COLOR_DEPTH_666 ); +return false; } static void fill_stream_properties_from_drm_display_mode(
@@ -3370,8 +3365,14 @@ static void fill_stream_properties_from_drm_display_mode( stream->out_transfer_func->type =TF_TYPE_PREDEFINED ; stream->out_transfer_func->tf =TRANSFER_FUNCTION_SRGB ; -if (stream->signal ==SIGNAL_TYPE_HDMI_TYPE_A ) -adjust_colour_depth_from_display_info (timing_out, info); +if (stream->signal ==SIGNAL_TYPE_HDMI_TYPE_A ) { +if (!adjust_colour_depth_from_display_info (timing_out, info) && +drm_mode_is_420_also (info, mode_in) && + timing_out->pixel_encoding !=PIXEL_ENCODING_YCBCR420 ) { + timing_out->pixel_encoding =PIXEL_ENCODING_YCBCR420 ; +adjust_colour_depth_from_display_info (timing_out, info); + } + } } static void fill_audio_info(struct audio_info *audio_info,
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 72/110 stmmac: debugfs entry name is not be changed when udev rename device name.
In reply to Greg Kroah-HartmanFrom: Jiping Ma <[email protected]>
[ Upstream commit 481a7d154cbbd5ca355cc01cc8969876b240eded ]
Add one notifier for udev changes net device name.
--- .../net/ethernet/stmicro/stmmac/stmmac_main.c | 32 +++++++++++++++++++ 1 file changed, 32 insertions(+)
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +32 -0
@@ -105,6 +105,7 @@ MODULE_PARM_DESC(chain_mode, "To use chain instead of ring mode");static irqreturn_t stmmac_interrupt (int irq,void *dev_id);#ifdef CONFIG_DEBUG_FS +static const struct net_device_ops stmmac_netdev_ops;static void stmmac_init_fs (struct net_device *dev);static void stmmac_exit_fs (struct net_device *dev);#endif
@@ -4175,6 +4176,34 @@ static int stmmac_dma_cap_show(struct seq_file *seq, void *v) }DEFINE_SHOW_ATTRIBUTE (stmmac_dma_cap); +/* Use network device events to rename debugfs file entries. +*/ +static int stmmac_device_event (struct notifier_block *unused, +unsigned long event,void *ptr) +{ +struct net_device *dev =netdev_notifier_info_to_dev (ptr); +struct stmmac_priv *priv =netdev_priv (dev); + +if (dev->netdev_ops != &stmmac_netdev_ops) + goto done; + +switch (event) { +case NETDEV_CHANGENAME : +if (priv->dbgfs_dir) + priv->dbgfs_dir =debugfs_rename (stmmac_fs_dir, + priv->dbgfs_dir, + stmmac_fs_dir, + dev->name); +break ; + } +done: +return NOTIFY_DONE ; +} + +static struct notifier_block stmmac_notifier = { + .notifier_call = stmmac_device_event, +}; +static void stmmac_init_fs (struct net_device *dev) {struct stmmac_priv *priv =netdev_priv (dev);
@@ -4189,12 +4218,15 @@ static void stmmac_init_fs(struct net_device *dev)/* Entry to report the DMA HW features */ debugfs_create_file ("dma_cap" ,0444 , priv->dbgfs_dir, dev, &stmmac_dma_cap_fops); + + register_netdevice_notifier(&stmmac_notifier); }static void stmmac_exit_fs (struct net_device *dev) {struct stmmac_priv *priv =netdev_priv (dev); +unregister_netdevice_notifier (&stmmac_notifier);debugfs_remove_recursive (priv->dbgfs_dir); }#endif /* CONFIG_DEBUG_FS */
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 73/110 atm: eni: fix uninitialized variable warning
In reply to Greg Kroah-HartmanFrom: Arnd Bergmann <[email protected]>
[ Upstream commit 30780d086a83332adcd9362281201cee7c3d9d19 ]
With -O3, gcc has found an actual unintialized variable stored into an mmio register in two instances:
drivers/atm/eni.c: In function 'discard':drivers/atm/eni.c:465:13: error: 'dma[1]' is used uninitialized in this function [-Werror=uninitialized]
writel(dma[i*2+1],eni_dev->rx_dma+dma_wr*8+4);
^drivers/atm/eni.c:465:13: error: 'dma[3]' is used uninitialized in this function [-Werror=uninitialized]
Change the code to always write zeroes instead.
--- drivers/atm/eni.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/atm/eni.c +2 -2
@@ -374,7 +374,7 @@ static int do_rx_dma(struct atm_vcc *vcc,struct sk_buff *skb, here = (eni_vcc->descr+skip) & (eni_vcc->words-1 ); dma[j++] = (here <<MID_DMA_COUNT_SHIFT ) | (vcc->vci <<MID_DMA_VCI_SHIFT ) |MID_DT_JK ; - j++; + dma[j++] =0 ; } here = (eni_vcc->descr+size+skip) & (eni_vcc->words-1 );if (!eff) size += skip;
@@ -447,7 +447,7 @@ static int do_rx_dma(struct atm_vcc *vcc,struct sk_buff *skb,if (size != eff) { dma[j++] = (here <<MID_DMA_COUNT_SHIFT ) | (vcc->vci <<MID_DMA_VCI_SHIFT ) |MID_DT_JK ; - j++; + dma[j++] =0 ; }if (!j || j >2 *RX_DMA_BUF ) { printk(KERN_CRIT DEV_LABEL "!j or j too big!!!\n");
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 74/110 HID: steam: Fix input device disappearing
In reply to Greg Kroah-HartmanFrom: Rodrigo Rivas Costa <[email protected]>
[ Upstream commit 20eee6e5af35d9586774e80b6e0b1850e7cc9899 ]
The connected value for wired devices was not properly initialized, it must be set to true upon creation, because wired devices do not generate connection events.
When a raw client (the Steam Client) uses the device, the input device is destroyed. Then, when the raw client finishes, it must be recreated. But since the connected variable was false this never happended.
--- drivers/hid/hid-steam.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/hid/hid-steam.c +4 -0
@@ -768,8 +768,12 @@ static int steam_probe(struct hid_device *hdev,if (steam->quirks &STEAM_QUIRK_WIRELESS ) {hid_info (hdev,"Steam wireless receiver connected" ); +/* If using a wireless adaptor ask for connection status */ + steam->connected = false;steam_request_conn_status (steam); }else { +/* A wired connection is always present */ + steam->connected = true; ret =steam_register (steam);if (ret) { hid_err(hdev,
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 75/110 extcon-intel-cht-wc: Dont reset USB data connection at probe
In reply to Greg Kroah-HartmanFrom: Yauhen Kharuzhy <[email protected]>
[ Upstream commit e81b88932985c9134d410f4eaaaa9b81a3b4bd0c ]
Intel Cherry Trail Whiskey Cove extcon driver connect USB data lines to PMIC at driver probing for further charger detection. This causes reset of USB data sessions and removing all devices from bus. If system was booted from Live CD or USB dongle, this makes system unusable.
Check if USB ID pin is floating and re-route data lines in this case only, don't touch otherwise.
--- drivers/extcon/extcon-intel-cht-wc.c | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-)
drivers/extcon/extcon-intel-cht-wc.c +14 -2
@@ -338,6 +338,7 @@ static int cht_wc_extcon_probe(struct platform_device *pdev)struct intel_soc_pmic *pmic =dev_get_drvdata (pdev->dev.parent);struct cht_wc_extcon_data *ext;unsigned long mask = ~(CHT_WC_PWRSRC_VBUS |CHT_WC_PWRSRC_USBID_MASK ); +int pwrsrc_sts, id;int irq, ret; irq =platform_get_irq (pdev,0 );
@@ -387,8 +388,19 @@ static int cht_wc_extcon_probe(struct platform_device *pdev) goto disable_sw_control; } -/* Route D+ and D- to PMIC for initial charger detection */ -cht_wc_extcon_set_phymux (ext,MUX_SEL_PMIC ); + ret =regmap_read (ext->regmap,CHT_WC_PWRSRC_STS , &pwrsrc_sts); +if (ret) { +dev_err (ext->dev,"Error reading pwrsrc status: %d\n" , ret); + goto disable_sw_control; + } + +/* +* If no USB host or device connected, route D+ and D- to PMIC for +* initial charger detection +*/ + id =cht_wc_extcon_get_id (ext, pwrsrc_sts); +if (id !=INTEL_USB_ID_GND ) +cht_wc_extcon_set_phymux (ext,MUX_SEL_PMIC );/* Get initial state */ cht_wc_extcon_pwrsrc_event (ext);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 76/110 ASoC: Intel: cht_bsw_rt5645: Add quirk for boards using pmc_plt_clk_0
In reply to Greg Kroah-HartmanFrom: Sam McNally <[email protected]>
[ Upstream commit adebb11139029ddf1fba6f796c4a476f17eacddc ]
As of commit 648e921888ad ("clk: x86: Stop marking clocks as CLK_IS_CRITICAL"), the cht_bsw_rt5645 driver needs to enable the clock it's using for the codec's mclk. It does this from commit 7735bce05a9c ("ASoC: Intel: boards: use devm_clk_get() unconditionally"), enabling pmc_plt_clk_3. However, Strago family Chromebooks use pmc_plt_clk_0 for the codec mclk, resulting in white noise with some digital microphones. Add a DMI-based quirk for Strago family Chromebooks to use pmc_plt_clk_0 instead - mirroring the changes made to cht_bsw_max98090_ti in commit a182ecd3809c ("ASoC: intel: cht_bsw_max98090_ti: Add quirk for boards using pmc_plt_clk_0") and making use of the existing dmi_check_system() call and related infrastructure added in commit 22af29114eb4 ("ASoC: Intel: cht-bsw-rt5645: add quirks for SSP0/AIF1/AIF2 routing").
--- sound/soc/intel/boards/cht_bsw_rt5645.c | 26 +++++++++++++++++++------ 1 file changed, 20 insertions(+), 6 deletions(-)
sound/soc/intel/boards/cht_bsw_rt5645.c +20 -6
@@ -48,6 +48,7 @@ struct cht_mc_private {#define CHT_RT5645_SSP2_AIF2 BIT(16)/* default is using AIF1 */ #define CHT_RT5645_SSP0_AIF1 BIT(17)#define CHT_RT5645_SSP0_AIF2 BIT(18) +#define CHT_RT5645_PMC_PLT_CLK_0 BIT(19)static unsigned long cht_rt5645_quirk =0 ;
@@ -59,6 +60,8 @@ static void log_quirks(struct device *dev)dev_info (dev,"quirk SSP0_AIF1 enabled" );if (cht_rt5645_quirk &CHT_RT5645_SSP0_AIF2 )dev_info (dev,"quirk SSP0_AIF2 enabled" ); +if (cht_rt5645_quirk &CHT_RT5645_PMC_PLT_CLK_0 ) +dev_info (dev,"quirk PMC_PLT_CLK_0 enabled" ); } static int platform_clock_control(struct snd_soc_dapm_widget *w,
@@ -226,15 +229,21 @@ static int cht_aif1_hw_params(struct snd_pcm_substream *substream,return 0 ; } -/* uncomment when we have a real quirk static int cht_rt5645_quirk_cb (const struct dmi_system_id *id) { cht_rt5645_quirk = (unsigned long )id->driver_data;return 1 ; } -*/ static const struct dmi_system_id cht_rt5645_quirk_table[] = { + { +/* Strago family Chromebooks */ + .callback = cht_rt5645_quirk_cb, + .matches = { +DMI_MATCH (DMI_PRODUCT_FAMILY ,"Intel_Strago" ), + }, + .driver_data = (void *)CHT_RT5645_PMC_PLT_CLK_0 , + }, { }, };
@@ -526,6 +535,7 @@ static int snd_cht_mc_probe(struct platform_device *pdev)int dai_index =0 ;int ret_val =0 ;int i; +const char *mclk_name; drv =devm_kzalloc (&pdev->dev,sizeof (*drv),GFP_KERNEL );if (!drv)
@@ -662,11 +672,15 @@ static int snd_cht_mc_probe(struct platform_device *pdev)if (ret_val)return ret_val; - drv->mclk =devm_clk_get (&pdev->dev,"pmc_plt_clk_3" ); +if (cht_rt5645_quirk &CHT_RT5645_PMC_PLT_CLK_0 ) + mclk_name ="pmc_plt_clk_0" ; +else + mclk_name ="pmc_plt_clk_3" ; + + drv->mclk =devm_clk_get (&pdev->dev, mclk_name);if (IS_ERR (drv->mclk)) { -dev_err (&pdev->dev, -"Failed to get MCLK from pmc_plt_clk_3: %ld\n" , -PTR_ERR (drv->mclk)); +dev_err (&pdev->dev,"Failed to get MCLK from %s: %ld\n" , + mclk_name,PTR_ERR (drv->mclk));return PTR_ERR (drv->mclk); }
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 77/110 drm/amdgpu/SRIOV: add navi12 pci id for SRIOV (v2)
In reply to Greg Kroah-HartmanFrom: Jiange Zhao <[email protected]>
[ Upstream commit 57d4f3b7fd65b56f98b62817f27c461142c0bc2a ]
Add Navi12 PCI id support.
v2: flag as experimental for now (Alex)
--- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 1 + 1 file changed, 1 insertion(+)
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c +1 -0
@@ -1023,6 +1023,7 @@ static const struct pci_device_id pciidlist[] = {/* Navi12 */ {0x1002, 0x7360, PCI_ANY_ID, PCI_ANY_ID, 0, 0, CHIP_NAVI12|AMD_EXP_HW_SUPPORT}, + {0x1002, 0x7362, PCI_ANY_ID, PCI_ANY_ID, 0, 0, CHIP_NAVI12|AMD_EXP_HW_SUPPORT}, {0, 0, 0} };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 78/110 libbpf: Fix BTF-defined maps __type macro handling of arrays
In reply to Greg Kroah-HartmanFrom: Andrii Nakryiko <[email protected]>
[ Upstream commit a53ba15d81995868651dd28a85d8045aef3d4e20 ]
Due to a quirky C syntax of declaring pointers to array or function prototype, existing __type() macro doesn't work with map key/value types that are array or function prototype. One has to create a typedef first and use it to specify key/value type for a BPF map. By using typeof(), pointer to type is now handled uniformly for all kinds of types. Convert one of self-tests as a demonstration.
--- tools/testing/selftests/bpf/bpf_helpers.h | 2 +- tools/testing/selftests/bpf/progs/test_get_stack_rawtp.c | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-)
tools/testing/selftests/bpf/bpf_helpers.h +1 -1
@@ -3,7 +3,7 @@#define __BPF_HELPERS__#define __uint (name, val) int (*name)[val] -#define __type (name, val) val *name +#define __type (name, val) typeof(val) *name/* helper macro to print out debug messages */ #define bpf_printk (fmt, ...) \
tools/testing/selftests/bpf/progs/test_get_stack_rawtp.c +1 -2
@@ -47,12 +47,11 @@ struct {* issue and avoid complicated C programming massaging. * This is an acceptable workaround since there is one entry here. */ -typedef __u64 raw_stack_trace_t [2 *MAX_STACK_RAWTP ];struct { __uint(type, BPF_MAP_TYPE_PERCPU_ARRAY); __uint(max_entries, 1); __type(key, __u32); - __type(value, raw_stack_trace_t); + __type(value, __u64[2 * MAX_STACK_RAWTP]); } rawdata_map SEC(".maps");SEC ("raw_tracepoint/sys_enter" )
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 79/110 staging: mt7621-pci: add quirks for E2 revision using soc_device_attribute
In reply to Greg Kroah-HartmanFrom: Sergio Paracuellos <[email protected]>
[ Upstream commit b483b4e4d3f6bfd5089b9e6dc9ba259879c6ce6f ]
Depending on revision of the chip, reset lines are inverted. Make code more readable making use of 'soc_device_match' in driver probe function.
--- drivers/staging/mt7621-pci/pci-mt7621.c | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-)
drivers/staging/mt7621-pci/pci-mt7621.c +17 -6
@@ -29,15 +29,14 @@#include <linux/phy/phy.h> #include <linux/platform_device.h> #include <linux/reset.h> +#include <linux/sys_soc.h> #include <mt7621.h> #include <ralink_regs.h> #include "../../pci/pci.h" /* sysctl */ -#define MT7621_CHIP_REV_ID 0x0c#define MT7621_GPIO_MODE 0x60 -#define CHIP_REV_MT7621_E2 0x0101/* MediaTek specific configuration registers */ #define PCIE_FTS_NUM 0x70c
@@ -126,6 +125,8 @@ struct mt7621_pcie_port {* @ports: pointer to PCIe port information * @perst: gpio reset * @rst: pointer to pcie reset +* @resets_inverted: depends on chip revision +* reset lines are inverted. */ struct mt7621_pcie {void __iomem *base;
@@ -140,6 +141,7 @@ struct mt7621_pcie {struct list_head ports;struct gpio_desc *perst;struct reset_control *rst; +bool resets_inverted; };static inline u32 pcie_read (struct mt7621_pcie *pcie,u32 reg)
@@ -229,9 +231,9 @@ static inline void mt7621_pcie_port_clk_disable(struct mt7621_pcie_port *port)static inline void mt7621_control_assert (struct mt7621_pcie_port *port) { -u32 chip_rev_id =rt_sysc_r32 (MT7621_CHIP_REV_ID ); +struct mt7621_pcie *pcie = port->pcie; -if ((chip_rev_id &0xFFFF ) ==CHIP_REV_MT7621_E2 ) +if (pcie->resets_inverted)reset_control_assert (port->pcie_rst);else reset_control_deassert (port->pcie_rst);
@@ -239,9 +241,9 @@ static inline void mt7621_control_assert(struct mt7621_pcie_port *port)static inline void mt7621_control_deassert (struct mt7621_pcie_port *port) { -u32 chip_rev_id =rt_sysc_r32 (MT7621_CHIP_REV_ID ); +struct mt7621_pcie *pcie = port->pcie; -if ((chip_rev_id &0xFFFF ) ==CHIP_REV_MT7621_E2 ) +if (pcie->resets_inverted)reset_control_deassert (port->pcie_rst);else reset_control_assert (port->pcie_rst);
@@ -641,9 +643,14 @@ static int mt7621_pcie_register_host(struct pci_host_bridge *host,return pci_host_probe (host); } +static const struct soc_device_attribute mt7621_pci_quirks_match[] = { + { .soc_id ="mt7621" , .revision ="E2" } +}; +static int mt7621_pci_probe (struct platform_device *pdev) {struct device *dev = &pdev->dev; +const struct soc_device_attribute *attr;struct mt7621_pcie *pcie;struct pci_host_bridge *bridge;int err;
@@ -661,6 +668,10 @@ static int mt7621_pci_probe(struct platform_device *pdev)platform_set_drvdata (pdev, pcie);INIT_LIST_HEAD (&pcie->ports); + attr =soc_device_match (mt7621_pci_quirks_match); +if (attr) + pcie->resets_inverted = true; + err =mt7621_pcie_parse_dt (pcie);if (err) {dev_err (dev,"Parsing DT failed\n" );
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 80/110 platform/x86: dell-laptop: disable kbd backlight on Inspiron 10xx
In reply to Greg Kroah-HartmanFrom: Pacien TRAN-GIRARD <[email protected]>
[ Upstream commit 10b65e2915b2fcc606d173e98a972850101fb4c4 ]
This patch adds a quirk disabling keyboard backlight support for the Dell Inspiron 1012 and 1018.
Those models wrongly report supporting keyboard backlight control features (through SMBIOS tokens) even though they're not equipped with a backlit keyboard. This led to broken controls being exposed through sysfs by this driver which froze the system when used.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=107651
--- drivers/platform/x86/dell-laptop.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+)
drivers/platform/x86/dell-laptop.c +26 -0
@@ -33,6 +33,7 @@struct quirk_entry {bool touchpad_led; +bool kbd_led_not_present;bool kbd_led_levels_off_1;bool kbd_missing_ac_tag;
@@ -73,6 +74,10 @@ static struct quirk_entry quirk_dell_latitude_e6410 = { .kbd_led_levels_off_1 = true, }; +static struct quirk_entry quirk_dell_inspiron_1012 = { + .kbd_led_not_present = true, +}; +static struct platform_driver platform_driver = { .driver = { .name ="dell-laptop" ,
@@ -310,6 +315,24 @@ static const struct dmi_system_id dell_quirks[] __initconst = { }, .driver_data = &quirk_dell_latitude_e6410, }, + { + .callback = dmi_matched, + .ident = "Dell Inspiron 1012", + .matches = { + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), + DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 1012"), + }, + .driver_data = &quirk_dell_inspiron_1012, + }, + { + .callback = dmi_matched, + .ident = "Dell Inspiron 1018", + .matches = { + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), + DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 1018"), + }, + .driver_data = &quirk_dell_inspiron_1012, + }, { } };
@@ -1493,6 +1516,9 @@ static void kbd_init(void) {int ret; +if (quirks && quirks->kbd_led_not_present) +return ; + ret =kbd_init_info ();kbd_init_tokens ();
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 81/110 PCI: Add DMA alias quirk for Intel VCA NTB
In reply to Greg Kroah-HartmanFrom: Slawomir Pawlowski <[email protected]>
[ Upstream commit 56b4cd4b7da9ee95778eb5c8abea49f641ebfd91 ]
Intel Visual Compute Accelerator (VCA) is a family of PCIe add-in devices exposing computational units via Non Transparent Bridges (NTB, PEX 87xx).
Similarly to MIC x200, we need to add DMA aliases to allow buffer access when IOMMU is enabled.
Add aliases to allow computational unit access to host memory. These aliases mark the whole VCA device as one IOMMU group.
All possible slot numbers (0x20) are used, since we are unable to tell what slot is used on other side. This quirk is intended for both host and computational unit sides. The VCA devices have up to five functions: four for DMA channels and one additional.
--- drivers/pci/quirks.c | 34 ++++++++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+)
drivers/pci/quirks.c +34 -0
@@ -4080,6 +4080,40 @@ static void quirk_mic_x200_dma_alias(struct pci_dev *pdev)DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2260 , quirk_mic_x200_dma_alias);DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2264 , quirk_mic_x200_dma_alias); +/* +* Intel Visual Compute Accelerator (VCA) is a family of PCIe add-in devices +* exposing computational units via Non Transparent Bridges (NTB, PEX 87xx). +* +* Similarly to MIC x200, we need to add DMA aliases to allow buffer access +* when IOMMU is enabled. These aliases allow computational unit access to +* host memory. These aliases mark the whole VCA device as one IOMMU +* group. +* +* All possible slot numbers (0x20) are used, since we are unable to tell +* what slot is used on other side. This quirk is intended for both host +* and computational unit sides. The VCA devices have up to five functions +* (four for DMA channels and one additional). +*/ +static void quirk_pex_vca_alias (struct pci_dev *pdev) +{ +const unsigned int num_pci_slots =0x20 ; +unsigned int slot; + +for (slot =0 ; slot < num_pci_slots; slot++) { +pci_add_dma_alias (pdev,PCI_DEVFN (slot,0x0 )); +pci_add_dma_alias (pdev,PCI_DEVFN (slot,0x1 )); +pci_add_dma_alias (pdev,PCI_DEVFN (slot,0x2 )); +pci_add_dma_alias (pdev,PCI_DEVFN (slot,0x3 )); +pci_add_dma_alias (pdev,PCI_DEVFN (slot,0x4 )); + } +} +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2954 , quirk_pex_vca_alias); +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2955 , quirk_pex_vca_alias); +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2956 , quirk_pex_vca_alias); +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2958 , quirk_pex_vca_alias); +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x2959 , quirk_pex_vca_alias); +DECLARE_PCI_FIXUP_HEADER (PCI_VENDOR_ID_INTEL ,0x295A , quirk_pex_vca_alias); +/* * The IOMMU and interrupt controller on Broadcom Vulcan/Cavium ThunderX2 are * associated not at the root bus, but at a bridge below. This quirk avoids
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 82/110 media: dvbsky: add support for eyeTV Geniatech T2 lite
In reply to Greg Kroah-HartmanFrom: Thomas Voegtle <[email protected]>
[ Upstream commit 14494583336880640654300c76d0f5df3360d85f ]
Adds USB ID for the eyeTV Geniatech T2 lite to the dvbsky driver. This is a Geniatech T230C based stick without IR and a different USB ID.
--- drivers/media/usb/dvb-usb-v2/dvbsky.c | 3 +++ include/media/dvb-usb-ids.h | 1 + 2 files changed, 4 insertions(+)
drivers/media/usb/dvb-usb-v2/dvbsky.c +3 -0
@@ -792,6 +792,9 @@ static const struct usb_device_id dvbsky_id_table[] = { {DVB_USB_DEVICE (USB_VID_CONEXANT ,USB_PID_MYGICA_T230C , &mygica_t230c_props,"MyGica Mini DVB-T2 USB Stick T230C" , RC_MAP_TOTAL_MEDIA_IN_HAND_02) }, + { DVB_USB_DEVICE(USB_VID_CONEXANT, USB_PID_MYGICA_T230C_LITE, + &mygica_t230c_props,"MyGica Mini DVB-T2 USB Stick T230C Lite" , + NULL) }, { DVB_USB_DEVICE(USB_VID_CONEXANT, USB_PID_MYGICA_T230C2, &mygica_t230c_props,"MyGica Mini DVB-T2 USB Stick T230C v2" , RC_MAP_TOTAL_MEDIA_IN_HAND_02) },
include/media/dvb-usb-ids.h +1 -0
@@ -389,6 +389,7 @@#define USB_PID_MYGICA_T230 0xc688#define USB_PID_MYGICA_T230C 0xc689#define USB_PID_MYGICA_T230C2 0xc68a +#define USB_PID_MYGICA_T230C_LITE 0xc699#define USB_PID_ELGATO_EYETV_DIVERSITY 0x0011#define USB_PID_ELGATO_EYETV_DTT 0x0021#define USB_PID_ELGATO_EYETV_DTT_2 0x003f
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 83/110 bus: ti-sysc: Handle mstandby quirk and use it for musb
In reply to Greg Kroah-HartmanFrom: Tony Lindgren <[email protected]>
[ Upstream commit 03856e928b0e1a1c274eece1dfe4330a362c37f3 ]
We need swsup quirks for sidle and mstandby for musb to work properly.
--- drivers/bus/ti-sysc.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)
drivers/bus/ti-sysc.c +8 -2
@@ -923,6 +923,9 @@ set_midle:return -EINVAL ; } +if (ddata->cfg.quirks &SYSC_QUIRK_SWSUP_MSTANDBY ) + best_mode =SYSC_IDLE_NO ; + reg &= ~(SYSC_IDLE_MASK << regbits->midle_shift); reg |= best_mode << regbits->midle_shift;sysc_write (ddata, ddata->offsets[SYSC_SYSCONFIG ], reg);
@@ -984,6 +987,9 @@ static int sysc_disable_module(struct device *dev)return ret; } +if (ddata->cfg.quirks &SYSC_QUIRK_SWSUP_MSTANDBY ) + best_mode =SYSC_IDLE_FORCE ; + reg &= ~(SYSC_IDLE_MASK << regbits->midle_shift); reg |= best_mode << regbits->midle_shift;sysc_write (ddata, ddata->offsets[SYSC_SYSCONFIG ], reg);
@@ -1257,6 +1263,8 @@ static const struct sysc_revision_quirk sysc_revision_quirks[] = {SYSC_QUIRK ("gpu" ,0x50000000 ,0x14 ,-1 ,-1 ,0x00010201 ,0xffffffff ,0 ),SYSC_QUIRK ("gpu" ,0x50000000 ,0xfe00 ,0xfe10 ,-1 ,0x40000000 ,0xffffffff ,SYSC_MODULE_QUIRK_SGX ), +SYSC_QUIRK ("usb_otg_hs" ,0 ,0x400 ,0x404 ,0x408 ,0x00000050 , +0xffffffff ,SYSC_QUIRK_SWSUP_SIDLE |SYSC_QUIRK_SWSUP_MSTANDBY ),SYSC_QUIRK ("wdt" ,0 ,0 ,0x10 ,0x14 ,0x502a0500 ,0xfffff0f0 , SYSC_MODULE_QUIRK_WDT),/* Watchdog on am3 and am4 */
@@ -1315,8 +1323,6 @@ static const struct sysc_revision_quirk sysc_revision_quirks[] = {SYSC_QUIRK ("usbhstll" ,0 ,0 ,0x10 ,0x14 ,0x00000008 ,0xffffffff ,0 ),SYSC_QUIRK ("usb_host_hs" ,0 ,0 ,0x10 ,0x14 ,0x50700100 ,0xffffffff ,0 ),SYSC_QUIRK ("usb_host_hs" ,0 ,0 ,0x10 ,-1 ,0x50700101 ,0xffffffff ,0 ), -SYSC_QUIRK ("usb_otg_hs" ,0 ,0x400 ,0x404 ,0x408 ,0x00000050 , -0xffffffff ,0 ), SYSC_QUIRK("vfpe", 0, 0, 0x104, -1, 0x4d001200, 0xffffffff, 0), #endif };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 84/110 bus: ti-sysc: Use swsup quirks also for am335x musb
In reply to Greg Kroah-HartmanFrom: Tony Lindgren <[email protected]>
[ Upstream commit 1819ef2e2d12d5b1a6ee54ac1c2afe35cffc677c ]
Also on am335x we need the swsup quirks for musb.
--- drivers/bus/ti-sysc.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/bus/ti-sysc.c +2 -0
@@ -1265,6 +1265,8 @@ static const struct sysc_revision_quirk sysc_revision_quirks[] = { SYSC_MODULE_QUIRK_SGX),SYSC_QUIRK ("usb_otg_hs" ,0 ,0x400 ,0x404 ,0x408 ,0x00000050 ,0xffffffff ,SYSC_QUIRK_SWSUP_SIDLE |SYSC_QUIRK_SWSUP_MSTANDBY ), +SYSC_QUIRK ("usb_otg_hs" ,0 ,0 ,0x10 ,-1 ,0x4ea2080d ,0xffffffff , +SYSC_QUIRK_SWSUP_SIDLE |SYSC_QUIRK_SWSUP_MSTANDBY ),SYSC_QUIRK ("wdt" ,0 ,0 ,0x10 ,0x14 ,0x502a0500 ,0xfffff0f0 , SYSC_MODULE_QUIRK_WDT),/* Watchdog on am3 and am4 */
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 85/110 spi: pxa2xx: Add support for Intel Comet Lake-H
In reply to Greg Kroah-HartmanFrom: Jarkko Nikula <[email protected]>
[ Upstream commit f0cf17ed76cffa365001d263ced1f130ec794917 ]
Add Intel Comet Lake-H LPSS SPI PCI IDs.
--- drivers/spi/spi-pxa2xx.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/spi/spi-pxa2xx.c +4 -0
@@ -1461,6 +1461,10 @@ static const struct pci_device_id pxa2xx_spi_pci_compound_match[] = { { PCI_VDEVICE(INTEL, 0x02aa), LPSS_CNL_SSP }, { PCI_VDEVICE(INTEL, 0x02ab), LPSS_CNL_SSP }, { PCI_VDEVICE(INTEL, 0x02fb), LPSS_CNL_SSP }, +/* CML-H */ + { PCI_VDEVICE(INTEL, 0x06aa), LPSS_CNL_SSP }, + { PCI_VDEVICE(INTEL, 0x06ab), LPSS_CNL_SSP }, + { PCI_VDEVICE(INTEL, 0x06fb), LPSS_CNL_SSP },/* TGL-LP */ { PCI_VDEVICE(INTEL, 0xa0aa), LPSS_CNL_SSP }, { PCI_VDEVICE(INTEL, 0xa0ab), LPSS_CNL_SSP },
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 86/110 iommu/amd: Support multiple PCI DMA aliases in device table
In reply to Greg Kroah-HartmanFrom: Logan Gunthorpe <[email protected]>
[ Upstream commit 3332364e4ebc0581d133a334645a20fd13b580f1 ]
Non-Transparent Bridge (NTB) devices (among others) may have many DMA aliases seeing the hardware will send requests with different device ids depending on their origin across the bridged hardware.
See commit ad281ecf1c7d ("PCI: Add DMA alias quirk for Microsemi Switchtec NTB") for more information on this.
The AMD IOMMU ignores all the PCI aliases except the last one so DMA transfers from these aliases will be blocked on AMD hardware with the IOMMU enabled.
To fix this, ensure the DTEs are cloned for every PCI alias. This is done by copying the DTE data for each alias as well as the IVRS alias every time it is changed.
--- drivers/iommu/amd_iommu.c | 133 +++++++++++++++----------------- drivers/iommu/amd_iommu_types.h | 2 +- 2 files changed, 62 insertions(+), 73 deletions(-)
drivers/iommu/amd_iommu.c +61 -72
@@ -226,71 +226,61 @@ static struct iommu_dev_data *search_dev_data(u16 devid)return NULL ; } -static int __last_alias (struct pci_dev *pdev,u16 alias,void *data) +static int clone_alias (struct pci_dev *pdev,u16 alias,void *data) { - *(u16 *)data = alias; -return 0 ; -} - -static u16 get_alias (struct device *dev) -{ -struct pci_dev *pdev =to_pci_dev (dev); -u16 devid, ivrs_alias, pci_alias; +u16 devid =pci_dev_id (pdev); -/* The callers make sure that get_device_id() does not fail here */ - devid =get_device_id (dev); - -/* For ACPI HID devices, we simply return the devid as such */ -if (!dev_is_pci (dev)) -return devid; +if (devid == alias) +return 0 ; - ivrs_alias = amd_iommu_alias_table[devid]; + amd_iommu_rlookup_table[alias] = + amd_iommu_rlookup_table[devid]; +memcpy (amd_iommu_dev_table[alias].data, + amd_iommu_dev_table[devid].data, +sizeof (amd_iommu_dev_table[alias].data)); -pci_for_each_dma_alias (pdev, __last_alias, &pci_alias); +return 0 ; +} -if (ivrs_alias == pci_alias) -return ivrs_alias; +static void clone_aliases (struct pci_dev *pdev) +{ +if (!pdev) +return ;/* -* DMA alias showdown -* -* The IVRS is fairly reliable in telling us about aliases, but it -* can't know about every screwy device. If we don't have an IVRS -* reported alias, use the PCI reported alias. In that case we may -* still need to initialize the rlookup and dev_table entries if the -* alias is to a non-existent device. +* The IVRS alias stored in the alias table may not be +* part of the PCI DMA aliases if it's bus differs +* from the original device. */ -if (ivrs_alias == devid) { -if (!amd_iommu_rlookup_table[pci_alias]) { - amd_iommu_rlookup_table[pci_alias] = - amd_iommu_rlookup_table[devid]; -memcpy (amd_iommu_dev_table[pci_alias].data, - amd_iommu_dev_table[devid].data, -sizeof (amd_iommu_dev_table[pci_alias].data)); - } +clone_alias (pdev, amd_iommu_alias_table[pci_dev_id (pdev)],NULL ); -return pci_alias; - } +pci_for_each_dma_alias (pdev, clone_alias,NULL ); +} -pci_info (pdev,"Using IVRS reported alias %02x:%02x.%d " -"for device [%04x:%04x], kernel reported alias " -"%02x:%02x.%d\n" ,PCI_BUS_NUM (ivrs_alias),PCI_SLOT (ivrs_alias), -PCI_FUNC (ivrs_alias), pdev->vendor, pdev->device, -PCI_BUS_NUM (pci_alias),PCI_SLOT (pci_alias), -PCI_FUNC (pci_alias)); +static struct pci_dev *setup_aliases (struct device *dev) +{ +struct pci_dev *pdev =to_pci_dev (dev); +u16 ivrs_alias; + +/* For ACPI HID devices, there are no aliases */ +if (!dev_is_pci (dev)) +return NULL ;/* -* If we don't have a PCI DMA alias and the IVRS alias is on the same -* bus, then the IVRS table may know about a quirk that we don't. +* Add the IVRS alias to the pci aliases if it is on the same +* bus. The IVRS table may know about a quirk that we don't. */ -if (pci_alias == devid && + ivrs_alias = amd_iommu_alias_table[pci_dev_id (pdev)]; +if (ivrs_alias !=pci_dev_id (pdev) &&PCI_BUS_NUM (ivrs_alias) == pdev->bus->number) {pci_add_dma_alias (pdev, ivrs_alias &0xff );pci_info (pdev,"Added PCI DMA alias %02x.%d\n" ,PCI_SLOT (ivrs_alias),PCI_FUNC (ivrs_alias)); } -return ivrs_alias; +clone_aliases (pdev); + +return pdev; }static struct iommu_dev_data *find_dev_data (u16 devid)
@@ -428,7 +418,7 @@ static int iommu_init_device(struct device *dev)if (!dev_data)return -ENOMEM ; - dev_data->alias =get_alias (dev); + dev_data->pdev =setup_aliases (dev);/* * By default we use passthrough mode for IOMMUv2 capable device.
@@ -453,20 +443,16 @@ static int iommu_init_device(struct device *dev)static void iommu_ignore_device (struct device *dev) { -u16 alias;int devid; devid =get_device_id (dev);if (devid <0 )return ; - alias =get_alias (dev); - + amd_iommu_rlookup_table[devid] =NULL ;memset (&amd_iommu_dev_table[devid],0 ,sizeof (struct dev_table_entry )); -memset (&amd_iommu_dev_table[alias],0 ,sizeof (struct dev_table_entry )); - amd_iommu_rlookup_table[devid] =NULL ; - amd_iommu_rlookup_table[alias] =NULL ; +setup_aliases (dev); }static void iommu_uninit_device (struct device *dev)
@@ -1236,6 +1222,13 @@ static int device_flush_iotlb(struct iommu_dev_data *dev_data,return iommu_queue_command (iommu, &cmd); } +static int device_flush_dte_alias (struct pci_dev *pdev,u16 alias,void *data) +{ +struct amd_iommu *iommu = data; + +return iommu_flush_dte (iommu, alias); +} +/* * Command send function for invalidating a device table entry */
@@ -1246,14 +1239,22 @@ static int device_flush_dte(struct iommu_dev_data *dev_data)int ret; iommu = amd_iommu_rlookup_table[dev_data->devid]; - alias = dev_data->alias; - ret =iommu_flush_dte (iommu, dev_data->devid); -if (!ret && alias != dev_data->devid) - ret =iommu_flush_dte (iommu, alias); +if (dev_data->pdev) + ret =pci_for_each_dma_alias (dev_data->pdev, + device_flush_dte_alias, iommu); +else + ret =iommu_flush_dte (iommu, dev_data->devid);if (ret)return ret; + alias = amd_iommu_alias_table[dev_data->devid]; +if (alias != dev_data->devid) { + ret =iommu_flush_dte (iommu, alias); +if (ret) +return ret; + } +if (dev_data->ats.enabled) ret =device_flush_iotlb (dev_data,0 , ~0UL );
@@ -2035,11 +2036,9 @@ static void do_attach(struct iommu_dev_data *dev_data, struct protection_domain *domain) {struct amd_iommu *iommu; -u16 alias;bool ats; iommu = amd_iommu_rlookup_table[dev_data->devid]; - alias = dev_data->alias; ats = dev_data->ats.enabled;/* Update data structures */
@@ -2052,8 +2051,7 @@ static void do_attach(struct iommu_dev_data *dev_data,/* Update device table */ set_dte_entry (dev_data->devid, domain, ats, dev_data->iommu_v2); -if (alias != dev_data->devid) -set_dte_entry (alias, domain, ats, dev_data->iommu_v2); +clone_aliases (dev_data->pdev);device_flush_dte (dev_data); }
@@ -2062,17 +2060,14 @@ static void do_detach(struct iommu_dev_data *dev_data) {struct protection_domain *domain = dev_data->domain;struct amd_iommu *iommu; -u16 alias; iommu = amd_iommu_rlookup_table[dev_data->devid]; - alias = dev_data->alias;/* Update data structures */ dev_data->domain =NULL ;list_del (&dev_data->list);clear_dte_entry (dev_data->devid); -if (alias != dev_data->devid) -clear_dte_entry (alias); +clone_aliases (dev_data->pdev);/* Flush the DTE entry */ device_flush_dte (dev_data);
@@ -2384,13 +2379,7 @@ static void update_device_table(struct protection_domain *domain)list_for_each_entry (dev_data, &domain->dev_list, list) {set_dte_entry (dev_data->devid, domain, dev_data->ats.enabled, dev_data->iommu_v2); - -if (dev_data->devid == dev_data->alias) -continue ; - -/* There is an alias, update device table entry for it */ -set_dte_entry (dev_data->alias, domain, dev_data->ats.enabled, - dev_data->iommu_v2); +clone_aliases (dev_data->pdev); } }
drivers/iommu/amd_iommu_types.h +1 -1
@@ -639,8 +639,8 @@ struct iommu_dev_data {struct list_head list;/* For domain->dev_list */ struct llist_node dev_data_list;/* For global dev_data_list */ struct protection_domain *domain;/* Domain the device is bound to */ +struct pci_dev *pdev;u16 devid;/* PCI Device ID */ -u16 alias;/* Alias Device ID */ bool iommu_v2;/* Device can make use of IOMMUv2 */ bool passthrough;/* Device is identity mapped */ struct {
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 87/110 iommu/amd: Support multiple PCI DMA aliases in IRQ Remapping
In reply to Greg Kroah-HartmanFrom: Logan Gunthorpe <[email protected]>
[ Upstream commit 3c124435e8dd516df4b2fc983f4415386fd6edae ]
Non-Transparent Bridge (NTB) devices (among others) may have many DMA aliases seeing the hardware will send requests with different device ids depending on their origin across the bridged hardware.
See commit ad281ecf1c7d ("PCI: Add DMA alias quirk for Microsemi Switchtec NTB") for more information on this.
The AMD IOMMU IRQ remapping functionality ignores all PCI aliases for IRQs so if devices send an interrupt from one of their aliases they will be blocked on AMD hardware with the IOMMU enabled.
To fix this, ensure IRQ remapping is enabled for all aliases with MSI interrupts.
This is analogous to the functionality added to the Intel IRQ remapping code in commit 3f0c625c6ae7 ("iommu/vt-d: Allow interrupts from the entire bus for aliased devices")
--- drivers/iommu/amd_iommu.c | 37 ++++++++++++++++++++++++++++++------- 1 file changed, 30 insertions(+), 7 deletions(-)
drivers/iommu/amd_iommu.c +30 -7
@@ -3741,7 +3741,20 @@ static void set_remap_table_entry(struct amd_iommu *iommu, u16 devid,iommu_flush_dte (iommu, devid); } -static struct irq_remap_table *alloc_irq_table (u16 devid) +static int set_remap_table_entry_alias (struct pci_dev *pdev,u16 alias, +void *data) +{ +struct irq_remap_table *table = data; + + irq_lookup_table[alias] = table; +set_dte_irq_entry (alias, table); + +iommu_flush_dte (amd_iommu_rlookup_table[alias], alias); + +return 0 ; +} + +static struct irq_remap_table *alloc_irq_table (u16 devid,struct pci_dev *pdev) {struct irq_remap_table *table =NULL ;struct irq_remap_table *new_table =NULL ;
@@ -3787,7 +3800,12 @@ static struct irq_remap_table *alloc_irq_table(u16 devid) table = new_table; new_table =NULL ; -set_remap_table_entry (iommu, devid, table); +if (pdev) +pci_for_each_dma_alias (pdev, set_remap_table_entry_alias, + table); +else +set_remap_table_entry (iommu, devid, table); +if (devid != alias)set_remap_table_entry (iommu, alias, table);
@@ -3804,7 +3822,8 @@ out_unlock:return table; } -static int alloc_irq_index (u16 devid,int count,bool align) +static int alloc_irq_index (u16 devid,int count,bool align, +struct pci_dev *pdev) {struct irq_remap_table *table;int index, c, alignment =1 ;
@@ -3814,7 +3833,7 @@ static int alloc_irq_index(u16 devid, int count, bool align)if (!iommu)return -ENODEV ; - table =alloc_irq_table (devid); + table =alloc_irq_table (devid, pdev);if (!table)return -ENODEV ;
@@ -4247,7 +4266,7 @@ static int irq_remapping_alloc(struct irq_domain *domain, unsigned int virq,struct irq_remap_table *table;struct amd_iommu *iommu; - table =alloc_irq_table (devid); + table =alloc_irq_table (devid,NULL );if (table) {if (!table->min_index) {/*
@@ -4264,11 +4283,15 @@ static int irq_remapping_alloc(struct irq_domain *domain, unsigned int virq, }else { index = -ENOMEM ; } - }else { + }else if (info->type ==X86_IRQ_ALLOC_TYPE_MSI || + info->type ==X86_IRQ_ALLOC_TYPE_MSIX ) {bool align = (info->type ==X86_IRQ_ALLOC_TYPE_MSI ); - index =alloc_irq_index (devid, nr_irqs, align); + index =alloc_irq_index (devid, nr_irqs, align, info->msi_dev); + }else { + index =alloc_irq_index (devid, nr_irqs, false,NULL ); } +if (index <0 ) {pr_warn ("Failed to allocate IRTE\n" ); ret = index;
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 88/110 perf/imx_ddr: Add enhanced AXI ID filter support
In reply to Greg Kroah-HartmanFrom: Joakim Zhang <[email protected]>
[ Upstream commit 44f8bd014a94ed679ddb77d0b92350d4ac4f23a5 ]
With DDR_CAP_AXI_ID_FILTER quirk, indicating HW supports AXI ID filter which only can get bursts from DDR transaction, i.e. DDR read/write requests.
This patch add DDR_CAP_AXI_ID_ENHANCED_FILTER quirk, indicating HW supports AXI ID filter which can get bursts and bytes from DDR transaction at the same time. We hope PMU always return bytes in the driver due to it is more meaningful for users.
--- drivers/perf/fsl_imx8_ddr_perf.c | 63 +++++++++++++++++++++----------- 1 file changed, 42 insertions(+), 21 deletions(-)
drivers/perf/fsl_imx8_ddr_perf.c +42 -21
@@ -45,7 +45,8 @@static DEFINE_IDA (ddr_ida);/* DDR Perf hardware feature */ -#define DDR_CAP_AXI_ID_FILTER 0x1/* support AXI ID filter */ +#define DDR_CAP_AXI_ID_FILTER 0x1/* support AXI ID filter */ +#define DDR_CAP_AXI_ID_FILTER_ENHANCED 0x3/* support enhanced AXI ID filter */ struct fsl_ddr_devtype_data {unsigned int quirks;/* quirks needed for different DDR Perf core */
@@ -178,6 +179,36 @@ static const struct attribute_group *attr_groups[] = { NULL, }; +static bool ddr_perf_is_filtered (struct perf_event *event) +{ +return event->attr.config ==0x41 || event->attr.config ==0x42 ; +} + +static u32 ddr_perf_filter_val (struct perf_event *event) +{ +return event->attr.config1; +} + +static bool ddr_perf_filters_compatible (struct perf_event *a, +struct perf_event *b) +{ +if (!ddr_perf_is_filtered (a)) +return true; +if (!ddr_perf_is_filtered (b)) +return true; +return ddr_perf_filter_val (a) ==ddr_perf_filter_val (b); +} + +static bool ddr_perf_is_enhanced_filtered (struct perf_event *event) +{ +unsigned int filt; +struct ddr_pmu *pmu =to_ddr_pmu (event->pmu); + + filt = pmu->devtype_data->quirks &DDR_CAP_AXI_ID_FILTER_ENHANCED ; +return (filt ==DDR_CAP_AXI_ID_FILTER_ENHANCED ) && +ddr_perf_is_filtered (event); +} +static u32 ddr_perf_alloc_counter (struct ddr_pmu *pmu,int event) {int i;
@@ -209,27 +240,17 @@ static void ddr_perf_free_counter(struct ddr_pmu *pmu, int counter)static u32 ddr_perf_read_counter (struct ddr_pmu *pmu,int counter) { -return readl_relaxed (pmu->base +COUNTER_READ + counter *4 ); -} - -static bool ddr_perf_is_filtered (struct perf_event *event) -{ -return event->attr.config ==0x41 || event->attr.config ==0x42 ; -} +struct perf_event *event = pmu->events[counter]; +void __iomem *base = pmu->base; -static u32 ddr_perf_filter_val (struct perf_event *event) -{ -return event->attr.config1; -} - -static bool ddr_perf_filters_compatible (struct perf_event *a, -struct perf_event *b) -{ -if (!ddr_perf_is_filtered (a)) -return true; -if (!ddr_perf_is_filtered (b)) -return true; -return ddr_perf_filter_val (a) ==ddr_perf_filter_val (b); +/* +* return bytes instead of bursts from ddr transaction for +* axid-read and axid-write event if PMU core supports enhanced +* filter. +*/ + base +=ddr_perf_is_enhanced_filtered (event) ?COUNTER_DPCR1 : +COUNTER_READ ; +return readl_relaxed (base + counter *4 ); }static int ddr_perf_event_init (struct perf_event *event)
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 89/110 mfd: intel-lpss: Add Intel Comet Lake PCH-H PCI IDs
In reply to Greg Kroah-HartmanFrom: Andy Shevchenko <[email protected]>
[ Upstream commit dd047dce3a6f5233b98e792e2287cc549da35879 ]
Intel Comet Lake PCH-H has the same LPSS than Intel Cannon Lake. Add the new IDs to the list of supported devices.
--- drivers/mfd/intel-lpss-pci.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-)
drivers/mfd/intel-lpss-pci.c +12 -1
@@ -140,7 +140,7 @@ static const struct intel_lpss_platform_info cnl_i2c_info = { };static const struct pci_device_id intel_lpss_pci_ids[] = { -/* CML */ +/* CML-LP */ {PCI_VDEVICE (INTEL ,0x02a8 ), (kernel_ulong_t)&spt_uart_info }, {PCI_VDEVICE (INTEL ,0x02a9 ), (kernel_ulong_t)&spt_uart_info }, {PCI_VDEVICE (INTEL ,0x02aa ), (kernel_ulong_t)&spt_info },
@@ -153,6 +153,17 @@ static const struct pci_device_id intel_lpss_pci_ids[] = { { PCI_VDEVICE(INTEL, 0x02ea), (kernel_ulong_t)&cnl_i2c_info }, { PCI_VDEVICE(INTEL, 0x02eb), (kernel_ulong_t)&cnl_i2c_info }, { PCI_VDEVICE(INTEL, 0x02fb), (kernel_ulong_t)&spt_info }, +/* CML-H */ + { PCI_VDEVICE(INTEL, 0x06a8), (kernel_ulong_t)&spt_uart_info }, + { PCI_VDEVICE(INTEL, 0x06a9), (kernel_ulong_t)&spt_uart_info }, + { PCI_VDEVICE(INTEL, 0x06aa), (kernel_ulong_t)&spt_info }, + { PCI_VDEVICE(INTEL, 0x06ab), (kernel_ulong_t)&spt_info }, + { PCI_VDEVICE(INTEL, 0x06c7), (kernel_ulong_t)&spt_uart_info }, + { PCI_VDEVICE(INTEL, 0x06e8), (kernel_ulong_t)&cnl_i2c_info }, + { PCI_VDEVICE(INTEL, 0x06e9), (kernel_ulong_t)&cnl_i2c_info }, + { PCI_VDEVICE(INTEL, 0x06ea), (kernel_ulong_t)&cnl_i2c_info }, + { PCI_VDEVICE(INTEL, 0x06eb), (kernel_ulong_t)&cnl_i2c_info }, + { PCI_VDEVICE(INTEL, 0x06fb), (kernel_ulong_t)&spt_info },/* BXT A-Step */ { PCI_VDEVICE(INTEL, 0x0aac), (kernel_ulong_t)&bxt_i2c_info }, { PCI_VDEVICE(INTEL, 0x0aae), (kernel_ulong_t)&bxt_i2c_info },
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 90/110 ARM: config: aspeed-g5: Enable 8250_DW quirks
In reply to Greg Kroah-HartmanFrom: Joel Stanley <[email protected]>
[ Upstream commit a5331a7a87ec81d5228b7421acf831b2d0c0de26 ]
This driver option is used by the AST2600 A0 boards to work around a hardware issue.
--- arch/arm/configs/aspeed_g5_defconfig | 1 + 1 file changed, 1 insertion(+)
arch/arm/configs/aspeed_g5_defconfig +1 -0
@@ -139,6 +139,7 @@ CONFIG_SERIAL_8250_RUNTIME_UARTS=6 CONFIG_SERIAL_8250_EXTENDED=y CONFIG_SERIAL_8250_ASPEED_VUART=y CONFIG_SERIAL_8250_SHARE_IRQ=y +CONFIG_SERIAL_8250_DW=y CONFIG_SERIAL_OF_PLATFORM=y CONFIG_ASPEED_KCS_IPMI_BMC=y CONFIG_ASPEED_BT_IPMI_BMC=y
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 91/110 ARM: OMAP2+: SmartReflex: add omap_sr_pdata definition
In reply to Greg Kroah-HartmanFrom: Ben Dooks <[email protected]>
[ Upstream commit 2079fe6ea8cbd2fb2fbadba911f1eca6c362eb9b ]
The omap_sr_pdata is not declared but is exported, so add a define for it to fix the following warning:
arch/arm/mach-omap2/pdata-quirks.c:609:36: warning: symbol 'omap_sr_pdata' was not declared. Should it be static?
--- include/linux/power/smartreflex.h | 3 +++ 1 file changed, 3 insertions(+)
include/linux/power/smartreflex.h +3 -0
@@ -293,6 +293,9 @@ struct omap_sr_data {struct voltagedomain *voltdm; }; + +extern struct omap_sr_data omap_sr_pdata[OMAP_SR_NR ]; +#ifdef CONFIG_POWER_AVS_OMAP /* Smartreflex module enable/disable interface */
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 92/110 mmc: sdhci-pci: Quirk for AMD SDHC Device 0x7906
In reply to Greg Kroah-HartmanFrom: Raul E Rangel <[email protected]>
[ Upstream commit 7a869f00bb15bcefb8804d798a49b086267b03e6 ]
AMD SDHC 0x7906 requires a hard reset to clear all internal state. Otherwise it can get into a bad state where the DATA lines are always read as zeros.
This change requires firmware that can transition the device into D3Cold for it to work correctly. If the firmware does not support transitioning to D3Cold then the power state transitions are a no-op.
--- drivers/mmc/host/sdhci-pci-core.c | 51 ++++++++++++++++++++++++++++++- 1 file changed, 50 insertions(+), 1 deletion(-)
drivers/mmc/host/sdhci-pci-core.c +50 -1
@@ -21,6 +21,7 @@#include <linux/mmc/mmc.h> #include <linux/scatterlist.h> #include <linux/io.h> +#include <linux/iopoll.h> #include <linux/gpio.h> #include <linux/pm_runtime.h> #include <linux/mmc/slot-gpio.h>
@@ -1598,11 +1599,59 @@ static int amd_probe(struct sdhci_pci_chip *chip)return 0 ; } +static u32 sdhci_read_present_state (struct sdhci_host *host) +{ +return sdhci_readl (host,SDHCI_PRESENT_STATE ); +} + +void amd_sdhci_reset (struct sdhci_host *host,u8 mask) +{ +struct sdhci_pci_slot *slot =sdhci_priv (host); +struct pci_dev *pdev = slot->chip->pdev; +u32 present_state; + +/* +* SDHC 0x7906 requires a hard reset to clear all internal state. +* Otherwise it can get into a bad state where the DATA lines are always +* read as zeros. +*/ +if (pdev->device ==0x7906 && (mask &SDHCI_RESET_ALL )) { +pci_clear_master (pdev); + +pci_save_state (pdev); + +pci_set_power_state (pdev, PCI_D3cold); +pr_debug ("%s: power_state=%u\n" ,mmc_hostname (host->mmc), + pdev->current_state); +pci_set_power_state (pdev,PCI_D0 ); + +pci_restore_state (pdev); + +/* +* SDHCI_RESET_ALL says the card detect logic should not be +* reset, but since we need to reset the entire controller +* we should wait until the card detect logic has stabilized. +* +* This normally takes about 40ms. +*/ +readx_poll_timeout ( + sdhci_read_present_state, + host, + present_state, + present_state &SDHCI_CD_STABLE , +10000 , +100000 + ); + } + +return sdhci_reset (host, mask); +} +static const struct sdhci_ops amd_sdhci_pci_ops = { .set_clock = sdhci_set_clock, .enable_dma = sdhci_pci_enable_dma, .set_bus_width = sdhci_set_bus_width, - .reset = sdhci_reset, + .reset = amd_sdhci_reset, .set_uhs_signaling = sdhci_set_uhs_signaling, };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 93/110 mmc: sdhci-pci: Add support for Intel JSL
In reply to Greg Kroah-HartmanFrom: Adrian Hunter <[email protected]>
[ Upstream commit 315e3bd7ac19b18ba704d96cbb9b79bad485c01f ]
Add PCI Ids for Intel JSL.
--- drivers/mmc/host/sdhci-pci-core.c | 2 ++ drivers/mmc/host/sdhci-pci.h | 2 ++ 2 files changed, 4 insertions(+)
drivers/mmc/host/sdhci-pci-core.c +2 -0
@@ -1730,6 +1730,8 @@ static const struct pci_device_id pci_ids[] = {SDHCI_PCI_DEVICE (INTEL ,CML_EMMC , intel_glk_emmc),SDHCI_PCI_DEVICE (INTEL ,CML_SD , intel_byt_sd),SDHCI_PCI_DEVICE (INTEL ,CMLH_SD , intel_byt_sd), +SDHCI_PCI_DEVICE (INTEL ,JSL_EMMC , intel_glk_emmc), +SDHCI_PCI_DEVICE (INTEL ,JSL_SD , intel_byt_sd),SDHCI_PCI_DEVICE (O2 ,8120 , o2),SDHCI_PCI_DEVICE (O2 ,8220 , o2), SDHCI_PCI_DEVICE(O2, 8221, o2),
drivers/mmc/host/sdhci-pci.h +2 -0
@@ -55,6 +55,8 @@#define PCI_DEVICE_ID_INTEL_CML_EMMC 0x02c4#define PCI_DEVICE_ID_INTEL_CML_SD 0x02f5#define PCI_DEVICE_ID_INTEL_CMLH_SD 0x06f5 +#define PCI_DEVICE_ID_INTEL_JSL_EMMC 0x4dc4 +#define PCI_DEVICE_ID_INTEL_JSL_SD 0x4df8#define PCI_DEVICE_ID_SYSKONNECT_8000 0x8000#define PCI_DEVICE_ID_VIA_95D0 0x95d0
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 94/110 bus: ti-sysc: Add module enable quirk for audio AESS
In reply to Greg Kroah-HartmanFrom: Tony Lindgren <[email protected]>
[ Upstream commit 020003f763e24e4ed0bb3d8909f3940891536d5d ]
We must set the autogating bit on enable for AESS (Audio Engine SubSystem) when probed with ti-sysc interconnect target module driver. Otherwise it won't idle properly.
--- drivers/bus/ti-sysc.c | 14 +++++++++++++- include/linux/platform_data/ti-sysc.h | 1 + 2 files changed, 14 insertions(+), 1 deletion(-)
drivers/bus/ti-sysc.c +13 -1
@@ -1248,6 +1248,8 @@ static const struct sysc_revision_quirk sysc_revision_quirks[] = { SYSC_QUIRK_SWSUP_SIDLE),/* Quirks that need to be set based on detected module */ +SYSC_QUIRK ("aess" ,0 ,0 ,0x10 ,-1 ,0x40000000 ,0xffffffff , +SYSC_MODULE_QUIRK_AESS ),SYSC_QUIRK ("hdq1w" ,0 ,0 ,0x14 ,0x18 ,0x00000006 ,0xffffffff ,SYSC_MODULE_QUIRK_HDQ1W ), SYSC_QUIRK("hdq1w", 0, 0, 0x14, 0x18, 0x0000000a, 0xffffffff,
@@ -1276,7 +1278,6 @@ static const struct sysc_revision_quirk sysc_revision_quirks[] = {#ifdef DEBUG SYSC_QUIRK ("adc" ,0 ,0 ,0x10 ,-1 ,0x47300001 ,0xffffffff ,0 ),SYSC_QUIRK ("atl" ,0 ,0 ,-1 ,-1 ,0x0a070100 ,0xffffffff ,0 ), -SYSC_QUIRK ("aess" ,0 ,0 ,0x10 ,-1 ,0x40000000 ,0xffffffff ,0 ),SYSC_QUIRK ("cm" ,0 ,0 ,-1 ,-1 ,0x40000301 ,0xffffffff ,0 ),SYSC_QUIRK ("control" ,0 ,0 ,0x10 ,-1 ,0x40000900 ,0xffffffff ,0 ), SYSC_QUIRK("cpgmac", 0, 0x1200, 0x1208, 0x1204, 0x4edb1902,
@@ -1408,6 +1409,14 @@ static void sysc_clk_enable_quirk_hdq1w(struct sysc *ddata)sysc_write (ddata, offset, val); } +/* AESS (Audio Engine SubSystem) needs autogating set after enable */ +static void sysc_module_enable_quirk_aess (struct sysc *ddata) +{ +int offset =0x7c ;/* AESS_AUTO_GATING_ENABLE */ + +sysc_write (ddata, offset,1 ); +} +/* I2C needs extra enable bit toggling for reset */ static void sysc_clk_quirk_i2c (struct sysc *ddata,bool enable) {
@@ -1490,6 +1499,9 @@ static void sysc_init_module_quirks(struct sysc *ddata)return ; } +if (ddata->cfg.quirks &SYSC_MODULE_QUIRK_AESS ) + ddata->module_enable_quirk = sysc_module_enable_quirk_aess; +if (ddata->cfg.quirks &SYSC_MODULE_QUIRK_SGX ) ddata->module_enable_quirk = sysc_module_enable_quirk_sgx;
include/linux/platform_data/ti-sysc.h +1 -0
@@ -49,6 +49,7 @@ struct sysc_regbits {s8 emufree_shift; }; +#define SYSC_MODULE_QUIRK_AESS BIT(19)#define SYSC_MODULE_QUIRK_SGX BIT(18)#define SYSC_MODULE_QUIRK_HDQ1W BIT(17)#define SYSC_MODULE_QUIRK_I2C BIT(16)
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 95/110 usb-storage: Disable UAS on JMicron SATA enclosure
In reply to Greg Kroah-HartmanFrom: Laura Abbott <[email protected]>
[ Upstream commit bc3bdb12bbb3492067c8719011576370e959a2e6 ]
Steve Ellis reported incorrect block sizes and alignement offsets with a SATA enclosure. Adding a quirk to disable UAS fixes the problems.
--- drivers/usb/storage/unusual_uas.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
drivers/usb/storage/unusual_uas.h +5 -2
@@ -87,12 +87,15 @@ UNUSUAL_DEV(0x2537, 0x1068, 0x0000, 0x9999, USB_SC_DEVICE, USB_PR_DEVICE, NULL, US_FL_IGNORE_UAS), -/* Reported-by: Takeo Nakayama <[email protected]> */ +/* +* Initially Reported-by: Takeo Nakayama <[email protected]> +* UAS Ignore Reported by Steven Ellis <[email protected]> +*/ UNUSUAL_DEV (0x357d ,0x7788 ,0x0000 ,0x9999 ,"JMicron" ,"JMS566" ,USB_SC_DEVICE ,USB_PR_DEVICE ,NULL , -US_FL_NO_REPORT_OPCODES ), +US_FL_NO_REPORT_OPCODES |US_FL_IGNORE_UAS ),/* Reported-by: Hans de Goede <[email protected]> */ UNUSUAL_DEV(0x4971, 0x1012, 0x0000, 0x9999,
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 96/110 ALSA: hda/realtek - Move some alc236 pintbls to fallback table
In reply to Greg Kroah-HartmanFrom: Hui Wang <[email protected]>
[ Upstream commit d64ebdbfd4f71406f58210f5ccb16977b4cd31d2 ]
We have a new Dell machine which needs to apply the quirk ALC255_FIXUP_DELL1_MIC_NO_PRESENCE, try to use the fallback table to fix it this time. And we could remove all pintbls of alc236 for applying DELL1_MIC_NO_PRESENCE on Dell machines.
--- sound/pci/hda/patch_realtek.c | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-)
sound/pci/hda/patch_realtek.c +3 -14
@@ -7563,20 +7563,6 @@ static const struct snd_hda_pin_quirk alc269_pin_fixup_tbl[] = { {0x19, 0x02a11020}, {0x1a, 0x02a11030}, {0x21, 0x0221101f}), -SND_HDA_PIN_QUIRK (0x10ec0236 ,0x1028 ,"Dell" ,ALC255_FIXUP_DELL1_MIC_NO_PRESENCE , - {0x12, 0x90a60140}, - {0x14, 0x90170110}, - {0x21, 0x02211020}), -SND_HDA_PIN_QUIRK (0x10ec0236 ,0x1028 ,"Dell" ,ALC255_FIXUP_DELL1_MIC_NO_PRESENCE , - {0x12, 0x90a60140}, - {0x14, 0x90170150}, - {0x21, 0x02211020}), -SND_HDA_PIN_QUIRK (0x10ec0236 ,0x1028 ,"Dell" ,ALC255_FIXUP_DELL1_MIC_NO_PRESENCE , - {0x21, 0x02211020}), -SND_HDA_PIN_QUIRK (0x10ec0236 ,0x1028 ,"Dell" ,ALC255_FIXUP_DELL1_MIC_NO_PRESENCE , - {0x12, 0x40000000}, - {0x14, 0x90170110}, - {0x21, 0x02211020}),SND_HDA_PIN_QUIRK (0x10ec0255 ,0x1028 ,"Dell" ,ALC255_FIXUP_DELL2_MIC_NO_PRESENCE , {0x14, 0x90170110}, {0x21, 0x02211020}),
@@ -7901,6 +7887,9 @@ static const struct snd_hda_pin_quirk alc269_fallback_pin_fixup_tbl[] = {SND_HDA_PIN_QUIRK (0x10ec0274 ,0x1028 ,"Dell" ,ALC274_FIXUP_DELL_AIO_LINEOUT_VERB , {0x19, 0x40000000}, {0x1a, 0x40000000}), +SND_HDA_PIN_QUIRK (0x10ec0236 ,0x1028 ,"Dell" ,ALC255_FIXUP_DELL1_MIC_NO_PRESENCE , + {0x19, 0x40000000}, + {0x1a, 0x40000000}), {} };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 97/110 Bluetooth: Allow combination of BDADDR_PROPERTY and INVALID_BDADDR quirks
In reply to Greg Kroah-HartmanFrom: Marcel Holtmann <[email protected]>
[ Upstream commit 7fdf6c6a0d0e032aac2aa4537a23af1e04a397ce ]
When utilizing BDADDR_PROPERTY and INVALID_BDADDR quirks together it results in an unconfigured controller even if the bootloader provides a valid address. Fix this by allowing a bootloader provided address to mark the controller as configured.
--- net/bluetooth/hci_core.c | 26 ++++++++++++++++++++++++-- 1 file changed, 24 insertions(+), 2 deletions(-)
net/bluetooth/hci_core.c +24 -2
@@ -1444,11 +1444,20 @@ static int hci_dev_do_open(struct hci_dev *hdev)if (hci_dev_test_flag (hdev,HCI_SETUP ) ||test_bit (HCI_QUIRK_NON_PERSISTENT_SETUP , &hdev->quirks)) { +bool invalid_bdaddr; +hci_sock_dev_event (hdev,HCI_DEV_SETUP );if (hdev->setup) ret = hdev->setup (hdev); +/* The transport driver can set the quirk to mark the +* BD_ADDR invalid before creating the HCI device or in +* its setup callback. +*/ + invalid_bdaddr =test_bit (HCI_QUIRK_INVALID_BDADDR , + &hdev->quirks); +if (ret) goto setup_failed;
@@ -1457,20 +1466,33 @@ static int hci_dev_do_open(struct hci_dev *hdev)hci_dev_get_bd_addr_from_property (hdev);if (bacmp (&hdev->public_addr,BDADDR_ANY ) && - hdev->set_bdaddr) + hdev->set_bdaddr) { ret = hdev->set_bdaddr (hdev, &hdev->public_addr); + +/* If setting of the BD_ADDR from the device +* property succeeds, then treat the address +* as valid even if the invalid BD_ADDR +* quirk indicates otherwise. +*/ +if (!ret) + invalid_bdaddr = false; + } } setup_failed:/* The transport driver can set these quirks before * creating the HCI device or in its setup callback. * +* For the invalid BD_ADDR quirk it is possible that +* it becomes a valid address if the bootloader does +* provide it (see above). +* * In case any of them is set, the controller has to * start up as unconfigured. */ if (test_bit (HCI_QUIRK_EXTERNAL_CONFIG , &hdev->quirks) || -test_bit (HCI_QUIRK_INVALID_BDADDR , &hdev->quirks)) + invalid_bdaddr)hci_dev_set_flag (hdev,HCI_UNCONFIGURED );/* For an unconfigured controller it is required to
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 98/110 Bluetooth: btbcm: Use the BDADDR_PROPERTY quirk
In reply to Greg Kroah-HartmanFrom: Andre Heider <[email protected]>
[ Upstream commit a4f95f31a9f38d9bb1fd313fcc2d0c0d48116ee3 ]
Some devices ship with the controller default address, like the Orange Pi 3 (BCM4345C5).
Allow the bootloader to set a valid address through the device tree.
--- drivers/bluetooth/btbcm.c | 6 ++++++ 1 file changed, 6 insertions(+)
drivers/bluetooth/btbcm.c +6 -0
@@ -440,6 +440,12 @@ int btbcm_finalize(struct hci_dev *hdev)set_bit (HCI_QUIRK_STRICT_DUPLICATE_FILTER , &hdev->quirks); +/* Some devices ship with the controller default address. +* Allow the bootloader to set a valid address through the +* device tree. +*/ +set_bit (HCI_QUIRK_USE_BDADDR_PROPERTY , &hdev->quirks); +return 0 ; }EXPORT_SYMBOL_GPL (btbcm_finalize);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 99/110 bus: ti-sysc: Fix missing force mstandby quirk handling
In reply to Greg Kroah-HartmanFrom: Tony Lindgren <[email protected]>
[ Upstream commit 93c60483b5feefced92b869d5f97769495bc6313 ]
Commit 03856e928b0e ("bus: ti-sysc: Handle mstandby quirk and use it for musb") added quirk handling for mstandby quirk but did not consider that we also need a quirk variant for SYSC_QUIRK_FORCE_MSTANDBY.
We need to use forced idle mode for both SYSC_QUIRK_SWSUP_MSTANDBY and SYSC_QUIRK_FORCE_MSTANDBY, but SYSC_QUIRK_SWSUP_MSTANDBY also need to additionally also configure no-idle mode when enabled.
--- drivers/bus/ti-sysc.c | 3 ++- include/linux/platform_data/ti-sysc.h | 1 + 2 files changed, 3 insertions(+), 1 deletion(-)
drivers/bus/ti-sysc.c +2 -1
@@ -987,7 +987,8 @@ static int sysc_disable_module(struct device *dev)return ret; } -if (ddata->cfg.quirks &SYSC_QUIRK_SWSUP_MSTANDBY ) +if (ddata->cfg.quirks & (SYSC_QUIRK_SWSUP_MSTANDBY ) || + ddata->cfg.quirks & (SYSC_QUIRK_FORCE_MSTANDBY )) best_mode =SYSC_IDLE_FORCE ; reg &= ~(SYSC_IDLE_MASK << regbits->midle_shift);
include/linux/platform_data/ti-sysc.h +1 -0
@@ -49,6 +49,7 @@ struct sysc_regbits {s8 emufree_shift; }; +#define SYSC_QUIRK_FORCE_MSTANDBY BIT(20)#define SYSC_MODULE_QUIRK_AESS BIT(19)#define SYSC_MODULE_QUIRK_SGX BIT(18)#define SYSC_MODULE_QUIRK_HDQ1W BIT(17)
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 100/110 rsi: fix use-after-free on failed probe and unbind
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit e93cd35101b61e4c79149be2cfc927c4b28dc60c upstream.
Make sure to stop both URBs before returning after failed probe as well as on disconnect to avoid use-after-free in the completion handler.
--- drivers/net/wireless/rsi/rsi_91x_usb.c | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-)
drivers/net/wireless/rsi/rsi_91x_usb.c +17 -1
@@ -292,6 +292,15 @@ out:dev_kfree_skb (rx_cb->rx_skb); } +static void rsi_rx_urb_kill (struct rsi_hw *adapter,u8 ep_num) +{ +struct rsi_91x_usbdev *dev = (struct rsi_91x_usbdev *)adapter->rsi_dev; +struct rx_usb_ctrl_block *rx_cb = &dev->rx_cb[ep_num -1 ]; +struct urb *urb = rx_cb->rx_urb; + +usb_kill_urb (urb); +} +/** * rsi_rx_urb_submit() - This function submits the given URB to the USB stack. * @adapter: Pointer to the adapter structure.
@@ -823,10 +832,13 @@ static int rsi_probe(struct usb_interfacif (adapter->priv->coex_mode >1 ) { status =rsi_rx_urb_submit (adapter,BT_EP );if (status) - goto err1; + goto err_kill_wlan_urb; }return 0 ; + +err_kill_wlan_urb: +rsi_rx_urb_kill (adapter,WLAN_EP ); err1:rsi_deinit_usb_interface (adapter); err:
@@ -857,6 +869,10 @@ static void rsi_disconnect(struct usb_in adapter->priv->bt_adapter =NULL ; } +if (adapter->priv->coex_mode >1 ) +rsi_rx_urb_kill (adapter,BT_EP ); +rsi_rx_urb_kill (adapter,WLAN_EP ); +rsi_reset_card (adapter);rsi_deinit_usb_interface (adapter);rsi_91x_deinit (adapter);
Greg Kroah-Hartman
maintainerPATCH 101/110 rsi: fix use-after-free on probe errors
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 92aafe77123ab478e5f5095878856ab0424910da upstream.
The driver would fail to stop the command timer in most error paths, something which specifically could lead to the timer being freed while still active on I/O errors during probe.
Fix this by making sure that each function starting the timer also stops it in all relevant error paths.
--- drivers/net/wireless/rsi/rsi_91x_hal.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
drivers/net/wireless/rsi/rsi_91x_hal.c +6 -6
@@ -622,6 +622,7 @@ static int bl_cmd(struct rsi_hw *adapterbl_start_cmd_timer (adapter, timeout); status =bl_write_cmd (adapter, cmd, exp_resp, ®out_val);if (status <0 ) { +bl_stop_cmd_timer (adapter);rsi_dbg (ERR_ZONE ,"%s: Command %s (%0x) writing failed..\n" , __func__, str, cmd);
@@ -737,10 +738,9 @@ static int ping_pong_write(struct rsi_hw } status =bl_cmd (adapter, cmd_req, cmd_resp, str); -if (status) { -bl_stop_cmd_timer (adapter); +if (status)return status; - } +return 0 ; }
@@ -828,10 +828,9 @@ static int auto_fw_upgrade(struct rsi_hw status =bl_cmd (adapter,EOF_REACHED ,FW_LOADING_SUCCESSFUL ,"EOF_REACHED" ); -if (status) { -bl_stop_cmd_timer (adapter); +if (status)return status; - } +rsi_dbg (INFO_ZONE ,"FW loading is done and FW is running..\n" );return 0 ; }
@@ -849,6 +848,7 @@ static int rsi_hal_prepare_fwload(struct ®out_val, RSI_COMMON_REG_SIZE);if (status <0 ) { +bl_stop_cmd_timer (adapter);rsi_dbg (ERR_ZONE ,"%s: REGOUT read failed\n" , __func__);return status;
Greg Kroah-Hartman
maintainerPATCH 102/110 rsi: fix memory leak on failed URB submission
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 47768297481184932844ab01a86752ba31a38861 upstream.
Make sure to free the skb on failed receive-URB submission (e.g. on disconnect or currently also due to a missing endpoint).
--- drivers/net/wireless/rsi/rsi_91x_usb.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/net/wireless/rsi/rsi_91x_usb.c +3 -1
@@ -338,8 +338,10 @@ static int rsi_rx_urb_submit(struct rsi_ rx_cb); status =usb_submit_urb (urb,GFP_KERNEL ); -if (status) +if (status) {rsi_dbg (ERR_ZONE ,"%s: Failed in urb submission\n" , __func__); +dev_kfree_skb (skb); + }return status; }
Greg Kroah-Hartman
maintainerPATCH 103/110 rsi: fix non-atomic allocation in completion handler
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit b9b9f9fea21830f85cf0148cd8dce001ae55ead1 upstream.
USB completion handlers are called in atomic context and must specifically not allocate memory using GFP_KERNEL.
--- drivers/net/wireless/rsi/rsi_91x_usb.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-)
drivers/net/wireless/rsi/rsi_91x_usb.c +7 -6
@@ -16,6 +16,7 @@*/ #include <linux/module.h> +#include <linux/types.h> #include <net/rsi_91x.h> #include "rsi_usb.h" #include "rsi_hal.h"
@@ -29,7 +30,7 @@ MODULE_PARM_DESC(dev_oper_mode,"9[Wi-Fi STA + BT LE], 13[Wi-Fi STA + BT classic + BT LE]\n" "6[AP + BT classic], 14[AP + BT classic + BT LE]"); -static int rsi_rx_urb_submit (struct rsi_hw *adapter,u8 ep_num); +static int rsi_rx_urb_submit (struct rsi_hw *adapter,u8 ep_num,gfp_t flags);/** * rsi_usb_card_write() - This function writes to the USB Card.
@@ -285,7 +286,7 @@ static void rsi_rx_done_handler(struct u status =0 ; out: -if (rsi_rx_urb_submit (dev->priv, rx_cb->ep_num)) +if (rsi_rx_urb_submit (dev->priv, rx_cb->ep_num,GFP_ATOMIC ))rsi_dbg (ERR_ZONE ,"%s: Failed in urb submission" , __func__);if (status)
@@ -307,7 +308,7 @@ static void rsi_rx_urb_kill(struct rsi_h* * Return: 0 on success, a negative error code on failure. */ -static int rsi_rx_urb_submit (struct rsi_hw *adapter,u8 ep_num) +static int rsi_rx_urb_submit (struct rsi_hw *adapter,u8 ep_num,gfp_t mem_flags) {struct rsi_91x_usbdev *dev = (struct rsi_91x_usbdev *)adapter->rsi_dev;struct rx_usb_ctrl_block *rx_cb = &dev->rx_cb[ep_num -1 ];
@@ -337,7 +338,7 @@ static int rsi_rx_urb_submit(struct rsi_ rsi_rx_done_handler, rx_cb); - status =usb_submit_urb (urb,GFP_KERNEL ); + status =usb_submit_urb (urb, mem_flags);if (status) {rsi_dbg (ERR_ZONE ,"%s: Failed in urb submission\n" , __func__);dev_kfree_skb (skb);
@@ -827,12 +828,12 @@ static int rsi_probe(struct usb_interfacrsi_dbg (INIT_ZONE ,"%s: Device Init Done\n" , __func__); } - status =rsi_rx_urb_submit (adapter,WLAN_EP ); + status =rsi_rx_urb_submit (adapter,WLAN_EP ,GFP_KERNEL );if (status) goto err1;if (adapter->priv->coex_mode >1 ) { - status =rsi_rx_urb_submit (adapter,BT_EP ); + status =rsi_rx_urb_submit (adapter,BT_EP ,GFP_KERNEL );if (status) goto err_kill_wlan_urb; }
Greg Kroah-Hartman
maintainerPATCH 104/110 crypto: af_alg - Use bh_lock_sock in sk_destruct
In reply to Greg Kroah-HartmanFrom: Herbert Xu <[email protected]>
commit 37f96694cf73ba116993a9d2d99ad6a75fa7fdb0 upstream.
As af_alg_release_parent may be called from BH context (most notably due to an async request that only completes after socket closure, or as reported here because of an RCU-delayed sk_destruct call), we must use bh_lock_sock instead of lock_sock.
--- crypto/af_alg.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
crypto/af_alg.c +4 -2
@@ -134,11 +134,13 @@ void af_alg_release_parent(struct sock * sk = ask->parent; ask =alg_sk (sk); -lock_sock (sk); +local_bh_disable (); +bh_lock_sock (sk); ask->nokey_refcnt -= nokey;if (!last) last = !--ask->refcnt; -release_sock (sk); +bh_unlock_sock (sk); +local_bh_enable ();if (last)sock_put (sk);
Greg Kroah-Hartman
maintainerPATCH 105/110 crypto: vmx - reject xts inputs that are too short
In reply to Greg Kroah-HartmanFrom: Daniel Axtens <[email protected]>
commit 1372a51b88fa0d5a8ed2803e4975c98da3f08463 upstream.
When the kernel XTS implementation was extended to deal with ciphertext stealing in commit 8083b1bf8163 ("crypto: xts - add support for ciphertext stealing"), a check was added to reject inputs that were too short.
However, in the vmx enablement - commit 239668419349 ("crypto: vmx/xts - use fallback for ciphertext stealing"), that check wasn't added to the vmx implementation. This disparity leads to errors like the following:
alg: skcipher: p8_aes_xts encryption unexpectedly succeeded on test vector "random: len=0 klen=64"; expected_error=-22, cfg="random: inplace may_sleep use_finup src_divs=[<flush>66.99%@+10, 33.1%@alignmask+1155]"
Return -EINVAL if asked to operate with a cryptlen smaller than the AES block size. This brings vmx in line with the generic implementation.
--- drivers/crypto/vmx/aes_xts.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/crypto/vmx/aes_xts.c +3 -0
@@ -84,6 +84,9 @@ static int p8_aes_xts_crypt(struct skcipu8 tweak[AES_BLOCK_SIZE ];int ret; +if (req->cryptlen <AES_BLOCK_SIZE ) +return -EINVAL ; +if (!crypto_simd_usable () || (req->cryptlen %XTS_BLOCK_SIZE ) !=0 ) {struct skcipher_request *subreq =skcipher_request_ctx (req);
Greg Kroah-Hartman
maintainerPATCH 106/110 crypto: caam - do not reset pointer size from MCFGR register
In reply to Greg Kroah-HartmanFrom: Iuliana Prodan <[email protected]>
commit 7278fa25aa0ebcc0e62c39b12071069df13f7e77 upstream.
In commit 'a1cf573ee95 ("crypto: caam - select DMA address size at runtime")' CAAM pointer size (caam_ptr_size) is changed from sizeof(dma_addr_t) to runtime value computed from MCFGR register. Therefore, do not reset MCFGR[PS].
--- drivers/crypto/caam/ctrl.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)
drivers/crypto/caam/ctrl.c +2 -4
@@ -685,11 +685,9 @@ static int caam_probe(struct platform_deof_node_put (np);if (!ctrlpriv->mc_en) -clrsetbits_32 (&ctrl->mcr,MCFGR_AWCACHE_MASK |MCFGR_LONG_PTR , +clrsetbits_32 (&ctrl->mcr,MCFGR_AWCACHE_MASK ,MCFGR_AWCACHE_CACH |MCFGR_AWCACHE_BUFF | -MCFGR_WDENABLE |MCFGR_LARGE_BURST | - (sizeof (dma_addr_t) ==sizeof (u64) ? -MCFGR_LONG_PTR :0 )); +MCFGR_WDENABLE |MCFGR_LARGE_BURST );handle_imx6_err005766 (&ctrl->mcr);
Greg Kroah-Hartman
maintainerPATCH 107/110 crypto: pcrypt - Fix user-after-free on module unload
In reply to Greg Kroah-HartmanFrom: Herbert Xu <[email protected]>
commit 07bfd9bdf568a38d9440c607b72342036011f727 upstream.
On module unload of pcrypt we must unregister the crypto algorithms first and then tear down the padata structure. As otherwise the crypto algorithms are still alive and can be used while the padata structure is being freed.
--- crypto/pcrypt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
crypto/pcrypt.c +2 -1
@@ -362,11 +362,12 @@ err: static void __exit pcrypt_exit(void) { +crypto_unregister_template (&pcrypt_tmpl); +pcrypt_fini_padata (pencrypt);pcrypt_fini_padata (pdecrypt);kset_unregister (pcrypt_kset); -crypto_unregister_template (&pcrypt_tmpl); }subsys_initcall (pcrypt_init);
Greg Kroah-Hartman
maintainerPATCH 108/110 KVM: arm64: Write arch.mdcr_el2 changes since last vcpu_load on VHE
In reply to Greg Kroah-HartmanFrom: Andrew Murray <[email protected]>
commit 4942dc6638b07b5326b6d2faa142635c559e7cd5 upstream.
On VHE systems arch.mdcr_el2 is written to mdcr_el2 at vcpu_load time to set options for self-hosted debug and the performance monitors extension.
Unfortunately the value of arch.mdcr_el2 is not calculated until kvm_arm_setup_debug() in the run loop after the vcpu has been loaded. This means that the initial brief iterations of the run loop use a zero value of mdcr_el2 - until the vcpu is preempted. This also results in a delay between changes to vcpu->guest_debug taking effect.
Fix this by writing to mdcr_el2 in kvm_arm_setup_debug() on VHE systems when a change to arch.mdcr_el2 has been detected.
--- arch/arm64/kvm/debug.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
arch/arm64/kvm/debug.c +5 -1
@@ -101,7 +101,7 @@ void kvm_arm_reset_debug_ptr(struct kvm_void kvm_arm_setup_debug (struct kvm_vcpu *vcpu) {bool trap_debug = !(vcpu->arch.flags &KVM_ARM64_DEBUG_DIRTY ); -unsigned long mdscr; +unsigned long mdscr, orig_mdcr_el2 = vcpu->arch.mdcr_el2;trace_kvm_arm_setup_debug (vcpu, vcpu->guest_debug);
@@ -197,6 +197,10 @@ void kvm_arm_setup_debug(struct kvm_vcpuif (vcpu_read_sys_reg (vcpu,MDSCR_EL1 ) & (DBG_MDSCR_KDE |DBG_MDSCR_MDE )) vcpu->arch.flags |=KVM_ARM64_DEBUG_DIRTY ; +/* Write mdcr_el2 changes since vcpu_load on VHE systems */ +if (has_vhe () && orig_mdcr_el2 != vcpu->arch.mdcr_el2) +write_sysreg (vcpu->arch.mdcr_el2, mdcr_el2); +trace_kvm_arm_set_dreg32 ("MDCR_EL2" , vcpu->arch.mdcr_el2);trace_kvm_arm_set_dreg32 ("MDSCR_EL1" ,vcpu_read_sys_reg (vcpu,MDSCR_EL1 )); }
Greg Kroah-Hartman
maintainerPATCH 109/110 Revert "um: Enable CONFIG_CONSTRUCTORS"
In reply to Greg Kroah-HartmanFrom: Johannes Berg <[email protected]>
commit 87c9366e17259040a9118e06b6dc8de986e5d3d1 upstream.
This reverts commit 786b2384bf1c ("um: Enable CONFIG_CONSTRUCTORS").
There are two issues with this commit, uncovered by Anton in tests on some (Debian) systems:
1) I completely forgot to call any constructors if CONFIG_CONSTRUCTORS isn't set. Don't recall now if it just wasn't needed on my system, or if I never tested this case.
2) With that fixed, it works - with CONFIG_CONSTRUCTORS *unset*. If I set CONFIG_CONSTRUCTORS, it fails again, which isn't totally unexpected since whatever wanted to run is likely to have to run before the kernel init etc. that calls the constructors in this case.
Basically, some constructors that gcc emits (libc has?) need to run very early during init; the failure mode otherwise was that the ptrace fork test already failed:
----------------------
$ ./linux mem=512M Core dump limits : soft - 0 hard - NONE Checking that ptrace can change system call numbers...check_ptrace : child exited with exitcode 6, while expecting 0; status 0x67f Aborted ----------------------
Thinking more about this, it's clear that we simply cannot support CONFIG_CONSTRUCTORS in UML. All the cases we need now (gcov, kasan) involve not use of the __attribute__((constructor)), but instead some constructor code/entry generated by gcc. Therefore, we cannot distinguish between kernel constructors and system constructors.
Thus, revert this commit.
--- arch/um/include/asm/common.lds.S | 2 +- arch/um/kernel/dyn.lds.S | 1 + init/Kconfig | 1 + kernel/gcov/Kconfig | 2 +- 4 files changed, 4 insertions(+), 2 deletions(-)
arch/um/include/asm/common.lds.S +1 -1
@@ -83,8 +83,8 @@ __preinit_array_end = .; } .init_array : { - /* dummy - we call this ourselves */ __init_array_start = .; + *(.init_array) __init_array_end = .; } .fini_array : {
arch/um/kernel/dyn.lds.S +1 -0
@@ -103,6 +103,7 @@ SECTIONS be empty, which isn't pretty. */ . = ALIGN(32 / 8); .preinit_array : { *(.preinit_array) } + .init_array : { *(.init_array) } .fini_array : { *(.fini_array) } .data : { INIT_TASK_DATA(KERNEL_STACK_SIZE)
init/Kconfig +1 -0
@@ -54,6 +54,7 @@ config CC_DISABLE_WARN_MAYBE_UNINITIALIZ config CONSTRUCTORS bool + depends on !UML config IRQ_WORK bool
kernel/gcov/Kconfig +1 -1
@@ -4,7 +4,7 @@ menu "GCOV-based kernel profiling" config GCOV_KERNEL bool "Enable gcov-based kernel profiling" depends on DEBUG_FS - select CONSTRUCTORS + select CONSTRUCTORS if !UML default n ---help--- This option enables gcov-based code profiling (e.g. for code coverage
Greg Kroah-Hartman
maintainerPATCH 110/110 power/supply: ingenic-battery: Dont change scale if theres only one
In reply to Greg Kroah-HartmanFrom: Paul Cercueil <[email protected]>
commit 86b9182df8bb12610d4d6feac45a69f3ed57bfd2 upstream.
The ADC in the JZ4740 can work either in high-precision mode with a 2.5V range, or in low-precision mode with a 7.5V range. The code in place in this driver will select the proper scale according to the maximum voltage of the battery.
The JZ4770 however only has one mode, with a 6.6V range. If only one scale is available, there's no need to change it (and nothing to change it to), and trying to do so will fail with -EINVAL.
--- drivers/power/supply/ingenic-battery.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-)
drivers/power/supply/ingenic-battery.c +11 -4
@@ -100,10 +100,17 @@ static int ingenic_battery_set_scale(strreturn -EINVAL ; } -return iio_write_channel_attribute (bat->channel, - scale_raw[best_idx], - scale_raw[best_idx +1 ], -IIO_CHAN_INFO_SCALE ); +/* Only set scale if there is more than one (fractional) entry */ +if (scale_len >2 ) { + ret =iio_write_channel_attribute (bat->channel, + scale_raw[best_idx], + scale_raw[best_idx +1 ], +IIO_CHAN_INFO_SCALE ); +if (ret) +return ret; + } + +return 0 ; }static enum power_supply_property ingenic_battery_properties[] = {
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.17 release. There are 110 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat, 01 Feb 2020 18:35:06 +0000. 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/v5.x/stable-review/patch-5.4.17-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-5.4.y
and the diffstat can be found below.thanks,
greg k-h
Compiled and booted on my test system. No dmesg regressions.
thanks,
-- Shuah
Jon Hunter
maintainerQuoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.17 release. There are 110 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat, 01 Feb 2020 18:35:06 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v5.x/stable-review/patch-5.4.17-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-5.4.y
and the diffstat can be found below.thanks,
greg k-h
All tests are passing for Tegra ...
Test results for stable-v5.4:
13 builds: 13 pass, 0 fail
22 boots: 22 pass, 0 fail
40 tests: 40 pass, 0 failLinux version: 5.4.17-rc1-g1bde1c513887
Boards tested: tegra124-jetson-tk1, tegra186-p2771-0000,
tegra194-p2972-0000, tegra20-ventana,
tegra210-p2371-2180, tegra210-p3450-0000,
tegra30-cardhu-a04Cheers
Jon
Signature · 1 line
nvpublic
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.17 release. There are 110 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat, 01 Feb 2020 18:35:06 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v5.x/stable-review/patch-5.4.17-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-5.4.y
and the diffstat can be found below.thanks,
greg k-h
Results from Linaro’s test farm.
No regressions on arm64, arm, x86_64, and i386.
Summary
------------------------------------------------------------------------
kernel: 5.4.17-rc1
git repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git branch: linux-5.4.y
git commit: 1bde1c513887dd7bac46be466f1cd06fe3a64292
git describe: v5.4.16-111-g1bde1c513887
Test details: https://qa-reports.linaro.org/lkft/linux-stable-rc-5.4-oe/build/v5.4.16-111-g1bde1c513887
No regressions (compared to build v5.4.16)
No fixes (compared to build v5.4.16)
Ran 23227 total tests in the following environments and test suites.
Environments
--------------
- dragonboard-410c
- hi6220-hikey
- i386
- juno-r2
- qemu_arm
- qemu_arm64
- qemu_i386
- qemu_x86_64
- x15
- x86
Test Suites
-----------
* build
* install-android-platform-tools-r2600
* kselftest
* libhugetlbfs
* linux-log-parser
* ltp-cap_bounds-tests
* ltp-cpuhotplug-tests
* ltp-cve-tests
* ltp-dio-tests
* ltp-fcntl-locktests-tests
* ltp-filecaps-tests
* ltp-fs-tests
* ltp-fs_bind-tests
* ltp-fs_perms_simple-tests
* ltp-fsx-tests
* ltp-io-tests
* ltp-ipc-tests
* ltp-nptl-tests
* ltp-pty-tests
* ltp-sched-tests
* ltp-securebits-tests
* perf
* spectre-meltdown-checker-test
* v4l2-compliance
* libgpiod
* ltp-commands-tests
* ltp-containers-tests
* ltp-math-tests
* ltp-syscalls-tests
* network-basic-tests
* ltp-hugetlb-tests
* ltp-mm-tests
* ltp-open-posix-tests
* kvm-unit-tests
* kselftest-vsyscall-mode-native
* kselftest-vsyscall-mode-none
* ssuite
Signature · 2 lines
Linaro LKFT https://lkft.linaro.org
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.17 release. There are 110 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat, 01 Feb 2020 18:35:06 +0000. Anything received after that time might be too late.
Build results:
total: 158 pass: 158 fail: 0 Qemu test results: total: 388 pass: 388 fail: 0
Guenter
Quoted 5 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.17 release. There are 110 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
Responses should be made by Sat, 01 Feb 2020 18:35:06 +0000. 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/v5.x/stable-review/patch-5.4.17-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-5.4.y
and the diffstat can be found below.
compiled and booted 5.4.17-rc1+ . No new errors according to "sudo dmesg -l err"
--
software engineer
rajagiri school of engineering and technology