5.4.45-rc1 review
Subsystems (2)
Greg Kroah-Hartman
maintainercover 5.4.45-rc1 review
This is the start of the stable review cycle for the 5.4.45 release. There are 38 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 Sun, 07 Jun 2020 13:54:56 +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.45-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.45-rc1
Dinghao Liu <[email protected]>
net: smsc911x: Fix runtime PM imbalance on error
Amit Cohen <[email protected]>
selftests: mlxsw: qos_mc_aware: Specify arping timeout as an integer
Jonathan McDowell <[email protected]>
net: ethernet: stmmac: Enable interface clocks on probe for IPQ806x
Valentin Longchamp <[email protected]>
net/ethernet/freescale: rework quiesce/activate for ucc_geth
Chaitanya Kulkarni <[email protected]>
null_blk: return error for invalid zone size
Gerald Schaefer <[email protected]>
s390/mm: fix set_huge_pte_at() for empty ptes
Jan Schmidt <[email protected]>
drm/edid: Add Oculus Rift S to non-desktop list
Jeremy Kerr <[email protected]>
net: bmac: Fix read of MAC address from ROM
Nathan Chancellor <[email protected]>
x86/mmiotrace: Use cpumask_available() for cpumask_var_t variables
Jens Axboe <[email protected]>
io_uring: initialize ctx->sqo_wait earlier
Atsushi Nemoto <[email protected]>
i2c: altera: Fix race between xfer_msg and isr thread
Can Guo <[email protected]>
scsi: pm: Balance pm_only counter of request queue during system resume
Madhuparna Bhowmik <[email protected]>
evm: Fix RCU list related warnings
Vineet Gupta <[email protected]>
ARC: [plat-eznps]: Restrict to CONFIG_ISA_ARCOMPACT
Eugeniy Paltsev <[email protected]>
ARC: Fix ICCM & DCCM runtime size checks
Michal Kalderon <[email protected]>
RDMA/qedr: Fix synchronization methods and memory leaks in qedr
Michal Kalderon <[email protected]>
RDMA/qedr: Fix qpids xarray api used
Vasily Gorbik <[email protected]>
s390/ftrace: save traced function caller
Jaroslav Kysela <[email protected]>
ASoC: intel - fix the card names
Xinwei Kong <[email protected]>
spi: dw: use "smp_mb()" to avoid sending spi data error
Christopher M. Riedl <[email protected]>
powerpc/xmon: Restrict when kernel is locked down
Anju T Sudhakar <[email protected]>
powerpc/powernv: Avoid re-registration of imc debugfs directory
Xiang Chen <[email protected]>
scsi: hisi_sas: Check sas_port before using it
Lucas De Marchi <[email protected]>
drm/i915: fix port checks for MST support on gen >= 11
Dan Carpenter <[email protected]>
airo: Fix read overflows sending packets
DENG Qingfang <[email protected]>
net: dsa: mt7530: set CPU port to fallback mode
Can Guo <[email protected]>
scsi: ufs: Release clock if DMA map fails
Sakari Ailus <[email protected]>
media: staging: ipu3-imgu: Move alignment attribute to field
Sakari Ailus <[email protected]>
media: Revert "staging: imgu: Address a compiler warning on alignment"
Jérôme Pouiller <[email protected]>
mmc: fix compilation of user API
Daniel Axtens <[email protected]>
kernel/relay.c: handle alloc_percpu returning NULL in relay_open
Matthew Garrett <[email protected]>
mt76: mt76x02u: Add support for newer versions of the XBox One wifi adapter
Giuseppe Marco Randazzo <[email protected]>
p54usb: add AirVasT USB stick device-id
Julian Sax <[email protected]>
HID: i2c-hid: add Schneider SCL142ALM to descriptor override
Benjamin Tissoires <[email protected]>
HID: multitouch: enable multi-input as a quirk for some devices
Scott Shumate <[email protected]>
HID: sony: Fix for broken buttons on DS3 USB dongles
Fan Yang <[email protected]>
mm: Fix mremap not considering huge pmd devmap
Tejun Heo <[email protected]>
Revert "cgroup: Add memory barriers to plug cgroup_rstat_updated() race window"
-------------
Diffstat:
Makefile | 4 +- arch/arc/kernel/setup.c | 5 +- arch/arc/plat-eznps/Kconfig | 1 + arch/powerpc/platforms/powernv/opal-imc.c | 39 +++--- arch/powerpc/xmon/xmon.c | 103 +++++++++++--- arch/s390/kernel/mcount.S | 1 + arch/s390/mm/hugetlbpage.c | 9 +- arch/x86/include/asm/pgtable.h | 1 + arch/x86/mm/mmio-mod.c | 4 +- drivers/block/null_blk_zoned.c | 4 + drivers/gpu/drm/drm_edid.c | 3 +- drivers/gpu/drm/i915/display/intel_dp.c | 7 +- drivers/gpu/drm/i915/display/intel_dp_mst.c | 22 ++- drivers/hid/hid-multitouch.c | 26 ++++ drivers/hid/hid-sony.c | 17 +++ drivers/hid/i2c-hid/i2c-hid-dmi-quirks.c | 8 ++ drivers/i2c/busses/i2c-altera.c | 10 +- drivers/infiniband/hw/qedr/main.c | 2 +- drivers/infiniband/hw/qedr/qedr.h | 23 +++- drivers/infiniband/hw/qedr/qedr_iw_cm.c | 148 ++++++++++++++------- drivers/infiniband/hw/qedr/verbs.c | 64 +++++---- drivers/net/dsa/mt7530.c | 11 +- drivers/net/dsa/mt7530.h | 6 + drivers/net/ethernet/apple/bmac.c | 2 +- drivers/net/ethernet/freescale/ucc_geth.c | 13 +- drivers/net/ethernet/smsc/smsc911x.c | 9 +- .../net/ethernet/stmicro/stmmac/dwmac-ipq806x.c | 13 ++ drivers/net/wireless/cisco/airo.c | 12 ++ drivers/net/wireless/intersil/p54/p54usb.c | 1 + drivers/net/wireless/mediatek/mt76/mt76x02.h | 1 + drivers/net/wireless/mediatek/mt76/mt76x2/usb.c | 1 + drivers/scsi/hisi_sas/hisi_sas_main.c | 3 +- drivers/scsi/scsi_pm.c | 10 +- drivers/scsi/ufs/ufshcd.c | 1 + drivers/spi/spi-dw.c | 3 + drivers/staging/media/ipu3/include/intel-ipu3.h | 7 +- fs/io_uring.c | 2 +- include/linux/security.h | 2 + include/uapi/linux/mmc/ioctl.h | 1 + kernel/cgroup/rstat.c | 16 +-- kernel/relay.c | 5 + mm/mremap.c | 2 +- security/integrity/evm/evm_crypto.c | 2 +- security/integrity/evm/evm_main.c | 4 +- security/integrity/evm/evm_secfs.c | 9 +- security/lockdown/lockdown.c | 2 + .../soc/intel/boards/kbl_rt5663_rt5514_max98927.c | 2 +- sound/soc/intel/boards/skl_hda_dsp_generic.c | 2 +- sound/soc/intel/boards/sof_rt5682.c | 2 +- .../selftests/drivers/net/mlxsw/qos_mc_aware.sh | 2 +- 50 files changed, 461 insertions(+), 186 deletions(-)
Greg Kroah-Hartman
maintainerPATCH 1/38 Revert "cgroup: Add memory barriers to plug cgroup_rstat_updated() race window"
From: Tejun Heo <[email protected]>
[ Upstream commit d8ef4b38cb69d907f9b0e889c44d05fc0f890977 ]
This reverts commit 9a9e97b2f1f2 ("cgroup: Add memory barriers to plug cgroup_rstat_updated() race window").
The commit was added in anticipation of memcg rstat conversion which needed synchronous accounting for the event counters (e.g. oom kill count). However, the conversion didn't get merged due to percpu memory overhead concern which couldn't be addressed at the time.
Unfortunately, the patch's addition of smp_mb() to cgroup_rstat_updated() meant that every scheduling event now had to go through an additional full barrier and Mel Gorman noticed it as 1% regression in netperf UDP_STREAM test.
There's no need to have this barrier in tree now and even if we need synchronous accounting in the future, the right thing to do is separating that out to a separate function so that hot paths which don't care about synchronous behavior don't have to pay the overhead of the full barrier. Let's revert.
--- kernel/cgroup/rstat.c | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-)
kernel/cgroup/rstat.c +3 -13
@@ -33,12 +33,9 @@ void cgroup_rstat_updated(struct cgroupreturn ;/* -* Paired with the one in cgroup_rstat_cpu_pop_upated(). Either we -* see NULL updated_next or they see our updated stat. -*/ -smp_mb (); - -/* +* Speculative already-on-list test. This may race leading to +* temporary inaccuracies, which is fine. +* * Because @parent's updated_children is terminated with @parent * instead of NULL, we can tell whether @cgrp is on the list by * testing the next pointer for NULL.
@@ -134,13 +131,6 @@ static struct cgroup *cgroup_rstat_cpu_p *nextp = rstatc->updated_next; rstatc->updated_next =NULL ; -/* -* Paired with the one in cgroup_rstat_cpu_updated(). -* Either they see NULL updated_next or we see their -* updated stat. -*/ -smp_mb (); -return pos; }
Greg Kroah-Hartman
maintainerPATCH 2/38 mm: Fix mremap not considering huge pmd devmap
In reply to Greg Kroah-HartmanFrom: Fan Yang <[email protected]>
commit 5bfea2d9b17f1034a68147a8b03b9789af5700f9 upstream.
The original code in mm/mremap.c checks huge pmd by:
if (is_swap_pmd(*old_pmd) || pmd_trans_huge(*old_pmd)) {However, a DAX mapped nvdimm is mapped as huge page (by default) but it is not transparent huge page (_PAGE_PSE | PAGE_DEVMAP). This commit changes the condition to include the case.
This addresses CVE-2020-10757.
--- arch/x86/include/asm/pgtable.h | 1 + mm/mremap.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-)
arch/x86/include/asm/pgtable.h +1 -0
@@ -253,6 +253,7 @@ static inline int pmd_large(pmd_t pte) }#ifdef CONFIG_TRANSPARENT_HUGEPAGE +/* NOTE: when predicate huge page, consider also pmd_devmap, or use pmd_large */ static inline int pmd_trans_huge (pmd_t pmd) {return (pmd_val (pmd) & (_PAGE_PSE|_PAGE_DEVMAP)) == _PAGE_PSE;
mm/mremap.c +1 -1
@@ -266,7 +266,7 @@ unsigned long move_page_tables(struct vm new_pmd =alloc_new_pmd (vma->vm_mm, vma, new_addr);if (!new_pmd)break ; -if (is_swap_pmd (*old_pmd) ||pmd_trans_huge (*old_pmd)) { +if (is_swap_pmd (*old_pmd) ||pmd_trans_huge (*old_pmd) ||pmd_devmap (*old_pmd)) {if (extent ==HPAGE_PMD_SIZE ) {bool moved;/* See comment in move_ptes() */
Greg Kroah-Hartman
maintainerPATCH 3/38 HID: sony: Fix for broken buttons on DS3 USB dongles
In reply to Greg Kroah-HartmanFrom: Scott Shumate <[email protected]>
commit e72455b898ac678667c5674668186b4670d87d11 upstream.
Fix for non-working buttons on knock-off USB dongles for Sony controllers. These USB dongles are used to connect older Sony DA/DS1/DS2 controllers via USB and are common on Amazon, AliExpress, etc. Without the patch, the square, X, and circle buttons do not function. These dongles used to work prior to kernel 4.10 but removing the global DS3 report fixup in commit e19a267b9987 ("HID: sony: DS3 comply to Linux gamepad spec") exposed the problem.
Many people reported the problem on the Ubuntu forums and are working around the problem by falling back to the 4.9 hid-sony driver.
The problem stems from these dongles incorrectly reporting their button count as 13 instead of 16. This patch fixes up the report descriptor by changing the button report count to 16 and removing 3 padding bits.
--- drivers/hid/hid-sony.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)
drivers/hid/hid-sony.c +17 -0
@@ -867,6 +867,23 @@ static u8 *sony_report_fixup(struct hid_if (sc->quirks &PS3REMOTE )return ps3remote_fixup (hdev, rdesc, rsize); +/* +* Some knock-off USB dongles incorrectly report their button count +* as 13 instead of 16 causing three non-functional buttons. +*/ +if ((sc->quirks &SIXAXIS_CONTROLLER_USB ) && *rsize >=45 && +/* Report Count (13) */ + rdesc[23 ] ==0x95 && rdesc[24 ] ==0x0D && +/* Usage Maximum (13) */ + rdesc[37 ] ==0x29 && rdesc[38 ] ==0x0D && +/* Report Count (3) */ + rdesc[43 ] ==0x95 && rdesc[44 ] ==0x03 ) { +hid_info (hdev,"Fixing up USB dongle report descriptor\n" ); + rdesc[24 ] =0x10 ; + rdesc[38 ] =0x10 ; + rdesc[44 ] =0x00 ; + } +return rdesc; }
Greg Kroah-Hartman
maintainerPATCH 4/38 HID: multitouch: enable multi-input as a quirk for some devices
In reply to Greg Kroah-HartmanFrom: Benjamin Tissoires <[email protected]>
commit 40d5bb87377a599d0405af765290f28aaa6abb1e upstream.
Two touchpad/trackstick combos are currently not behaving properly. They define a mouse emulation collection, as per Win8 requirements, but also define a separate mouse collection for the trackstick.
The way the kernel currently treat the collections is that it merges both in one device. However, given that the first mouse collection already defines X,Y and left, right buttons, when mapping the events from the second mouse collection, hid-multitouch sees that these events are already mapped, and simply ignores them.
To be able to report events from the tracktick, add a new quirked class for it, and manually add the 2 devices we know about.
--- drivers/hid/hid-multitouch.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+)
drivers/hid/hid-multitouch.c +26 -0
@@ -69,6 +69,7 @@ MODULE_LICENSE("GPL");#define MT_QUIRK_ASUS_CUSTOM_UP BIT(17)#define MT_QUIRK_WIN8_PTP_BUTTONS BIT(18)#define MT_QUIRK_SEPARATE_APP_REPORT BIT(19) +#define MT_QUIRK_FORCE_MULTI_INPUT BIT(20)#define MT_INPUTMODE_TOUCHSCREEN 0x02#define MT_INPUTMODE_TOUCHPAD 0x03
@@ -189,6 +190,7 @@ static void mt_post_parse(struct mt_devi#define MT_CLS_WIN_8 0x0012#define MT_CLS_EXPORT_ALL_INPUTS 0x0013#define MT_CLS_WIN_8_DUAL 0x0014 +#define MT_CLS_WIN_8_FORCE_MULTI_INPUT 0x0015/* vendor specific classes */ #define MT_CLS_3M 0x0101
@@ -279,6 +281,15 @@ static const struct mt_class mt_classes[MT_QUIRK_CONTACT_CNT_ACCURATE | MT_QUIRK_WIN8_PTP_BUTTONS, .export_all_inputs = true }, + { .name = MT_CLS_WIN_8_FORCE_MULTI_INPUT, + .quirks =MT_QUIRK_ALWAYS_VALID | +MT_QUIRK_IGNORE_DUPLICATES | +MT_QUIRK_HOVERING | +MT_QUIRK_CONTACT_CNT_ACCURATE | +MT_QUIRK_STICKY_FINGERS | +MT_QUIRK_WIN8_PTP_BUTTONS | + MT_QUIRK_FORCE_MULTI_INPUT, + .export_all_inputs = true },/* * vendor specific classes
@@ -1714,6 +1725,11 @@ static int mt_probe(struct hid_device *hif (id->group !=HID_GROUP_MULTITOUCH_WIN_8 ) hdev->quirks |=HID_QUIRK_MULTI_INPUT ; +if (mtclass->quirks &MT_QUIRK_FORCE_MULTI_INPUT ) { + hdev->quirks &= ~HID_QUIRK_INPUT_PER_APP ; + hdev->quirks |=HID_QUIRK_MULTI_INPUT ; + } +timer_setup (&td->release_timer, mt_expired_timeout,0 ); ret =hid_parse (hdev);
@@ -1926,6 +1942,11 @@ static const struct hid_device_id mt_dev MT_USB_DEVICE(USB_VENDOR_ID_DWAV, USB_DEVICE_ID_DWAV_EGALAX_MULTITOUCH_C002) }, +/* Elan devices */ + { .driver_data = MT_CLS_WIN_8_FORCE_MULTI_INPUT, +HID_DEVICE (BUS_I2C ,HID_GROUP_MULTITOUCH_WIN_8 , + USB_VENDOR_ID_ELAN, 0x313a) }, + /* Elitegroup panel */ { .driver_data = MT_CLS_SERIAL, MT_USB_DEVICE(USB_VENDOR_ID_ELITEGROUP,
@@ -2056,6 +2077,11 @@ static const struct hid_device_id mt_dev MT_USB_DEVICE(USB_VENDOR_ID_STANTUM_STM, USB_DEVICE_ID_MTP_STM)}, +/* Synaptics devices */ + { .driver_data = MT_CLS_WIN_8_FORCE_MULTI_INPUT, +HID_DEVICE (BUS_I2C ,HID_GROUP_MULTITOUCH_WIN_8 , + USB_VENDOR_ID_SYNAPTICS, 0xce08) }, + /* TopSeed panels */ { .driver_data = MT_CLS_TOPSEED, MT_USB_DEVICE(USB_VENDOR_ID_TOPSEED2,
Greg Kroah-Hartman
maintainerPATCH 5/38 HID: i2c-hid: add Schneider SCL142ALM to descriptor override
In reply to Greg Kroah-HartmanFrom: Julian Sax <[email protected]>
commit 6507ef10660efdfee93f0f3b9fac24b5e4d83e56 upstream.
This device uses the SIPODEV SP1064 touchpad, which does not supply descriptors, so it has to be added to the override list.
--- drivers/hid/i2c-hid/i2c-hid-dmi-quirks.c | 8 ++++++++ 1 file changed, 8 insertions(+)
drivers/hid/i2c-hid/i2c-hid-dmi-quirks.c +8 -0
@@ -389,6 +389,14 @@ static const struct dmi_system_id i2c_hi }, .driver_data = (void *)&sipodev_desc }, + { + .ident = "Schneider SCL142ALM", + .matches = { + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "SCHNEIDER"), + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "SCL142ALM"), + }, + .driver_data = (void *)&sipodev_desc + }, { } /* Terminate list */ };
Greg Kroah-Hartman
maintainerPATCH 6/38 p54usb: add AirVasT USB stick device-id
In reply to Greg Kroah-HartmanFrom: Giuseppe Marco Randazzo <[email protected]>
commit 63e49a9fdac1b4e97ac26cb3fe953f210d83bc53 upstream.
This patch adds the AirVasT USB wireless devices 124a:4026 to the list of supported devices. It's using the ISL3886 usb firmware. Without this modification, the wiki adapter is not recognized.
--- drivers/net/wireless/intersil/p54/p54usb.c | 1 + 1 file changed, 1 insertion(+)
drivers/net/wireless/intersil/p54/p54usb.c +1 -0
@@ -61,6 +61,7 @@ static const struct usb_device_id p54u_t {USB_DEVICE(0x0db0, 0x6826)}, /* MSI UB54G (MS-6826) */ {USB_DEVICE(0x107b, 0x55f2)}, /* Gateway WGU-210 (Gemtek) */ {USB_DEVICE(0x124a, 0x4023)}, /* Shuttle PN15, Airvast WM168g, IOGear GWU513 */ + {USB_DEVICE(0x124a, 0x4026)}, /* AirVasT USB wireless device */ {USB_DEVICE(0x1435, 0x0210)}, /* Inventel UR054G */ {USB_DEVICE(0x15a9, 0x0002)}, /* Gemtek WUBI-100GW 802.11g */ {USB_DEVICE(0x1630, 0x0005)}, /* 2Wire 802.11g USB (v1) / Z-Com */
Greg Kroah-Hartman
maintainerPATCH 7/38 mt76: mt76x02u: Add support for newer versions of the XBox One wifi adapter
In reply to Greg Kroah-HartmanFrom: Matthew Garrett <[email protected]>
commit b2934279c3e9719145ff4090d4ab951e340df17e upstream.
The current version has a new USB ID and reports as an 0x7632 device. Adding the IDs results in it working out of the box.
--- drivers/net/wireless/mediatek/mt76/mt76x02.h | 1 + drivers/net/wireless/mediatek/mt76/mt76x2/usb.c | 1 + 2 files changed, 2 insertions(+)
drivers/net/wireless/mediatek/mt76/mt76x02.h +1 -0
@@ -212,6 +212,7 @@ static inline bool is_mt76x0(struct mt76static inline bool is_mt76x2 (struct mt76x02_dev *dev) {return mt76_chip (&dev->mt76) ==0x7612 || +mt76_chip (&dev->mt76) ==0x7632 ||mt76_chip (&dev->mt76) ==0x7662 ||mt76_chip (&dev->mt76) ==0x7602 ; }
drivers/net/wireless/mediatek/mt76/mt76x2/usb.c +1 -0
@@ -18,6 +18,7 @@ static const struct usb_device_id mt76x2 { USB_DEVICE(0x7392, 0xb711) }, /* Edimax EW 7722 UAC */ { USB_DEVICE(0x0846, 0x9053) }, /* Netgear A6210 */ { USB_DEVICE(0x045e, 0x02e6) }, /* XBox One Wireless Adapter */ + { USB_DEVICE(0x045e, 0x02fe) }, /* XBox One Wireless Adapter */ { }, };
Greg Kroah-Hartman
maintainerPATCH 8/38 kernel/relay.c: handle alloc_percpu returning NULL in relay_open
In reply to Greg Kroah-HartmanFrom: Daniel Axtens <[email protected]>
commit 54e200ab40fc14c863bcc80a51e20b7906608fce upstream.
alloc_percpu() may return NULL, which means chan->buf may be set to NULL. In that case, when we do *per_cpu_ptr(chan->buf, ...), we dereference an invalid pointer:
BUG: Unable to handle kernel data access at 0x7dae0000
Faulting instruction address: 0xc0000000003f3fec
...
NIP relay_open+0x29c/0x600
LR relay_open+0x270/0x600
Call Trace:
relay_open+0x264/0x600 (unreliable)
__blk_trace_setup+0x254/0x600
blk_trace_setup+0x68/0xa0
sg_ioctl+0x7bc/0x2e80
do_vfs_ioctl+0x13c/0x1300
ksys_ioctl+0x94/0x130
sys_ioctl+0x48/0xb0
system_call+0x5c/0x68Check if alloc_percpu returns NULL.
This was found by syzkaller both on x86 and powerpc, and the reproducer it found on powerpc is capable of hitting the issue as an unprivileged user.
--- kernel/relay.c | 5 +++++ 1 file changed, 5 insertions(+)
kernel/relay.c +5 -0
@@ -581,6 +581,11 @@ struct rchan *relay_open(const char *basreturn NULL ; chan->buf = alloc_percpu(struct rchan_buf *); +if (!chan->buf) { +kfree (chan); +return NULL ; + } + chan->version =RELAYFS_CHANNEL_VERSION ; chan->n_subbufs = n_subbufs; chan->subbuf_size = subbuf_size;
Greg Kroah-Hartman
maintainerPATCH 9/38 mmc: fix compilation of user API
In reply to Greg Kroah-HartmanFrom: Jérôme Pouiller <[email protected]>
commit 83fc5dd57f86c3ec7d6d22565a6ff6c948853b64 upstream.
The definitions of MMC_IOC_CMD and of MMC_IOC_MULTI_CMD rely on MMC_BLOCK_MAJOR:
#define MMC_IOC_CMD _IOWR(MMC_BLOCK_MAJOR, 0, struct mmc_ioc_cmd)
#define MMC_IOC_MULTI_CMD _IOWR(MMC_BLOCK_MAJOR, 1, struct mmc_ioc_multi_cmd)However, MMC_BLOCK_MAJOR is defined in linux/major.h and linux/mmc/ioctl.h did not include it.
--- include/uapi/linux/mmc/ioctl.h | 1 + 1 file changed, 1 insertion(+)
include/uapi/linux/mmc/ioctl.h +1 -0
@@ -3,6 +3,7 @@#define LINUX_MMC_IOCTL_H #include <linux/types.h> +#include <linux/major.h> struct mmc_ioc_cmd {/*
Greg Kroah-Hartman
maintainerPATCH 10/38 media: Revert "staging: imgu: Address a compiler warning on alignment"
In reply to Greg Kroah-HartmanFrom: Sakari Ailus <[email protected]>
commit 81d1adeb52c97fbe097e8c94e36c3eb702cdb110 upstream.
This reverts commit c9d52c114a9fcc61c30512c7f810247a9f2812af.
The patch being reverted changed the memory layout of struct ipu3_uapi_acc_param. Revert it, and address the compiler warning issues in further patches.
--- drivers/staging/media/ipu3/include/intel-ipu3.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/media/ipu3/include/intel-ipu3.h +1 -1
@@ -2472,7 +2472,7 @@ struct ipu3_uapi_acc_param { struct ipu3_uapi_yuvp1_yds_config yds2 __attribute__((aligned(32))); struct ipu3_uapi_yuvp2_tcc_static_config tcc __attribute__((aligned(32)));struct ipu3_uapi_anr_config anr; - struct ipu3_uapi_awb_fr_config_s awb_fr __attribute__((aligned(32))); +struct ipu3_uapi_awb_fr_config_s awb_fr;struct ipu3_uapi_ae_config ae;struct ipu3_uapi_af_config_s af;struct ipu3_uapi_awb_config awb;
Greg Kroah-Hartman
maintainerPATCH 11/38 media: staging: ipu3-imgu: Move alignment attribute to field
In reply to Greg Kroah-HartmanFrom: Sakari Ailus <[email protected]>
commit 8c038effd893920facedf18c2c0976cec4a33408 upstream.
Move the alignment attribute of struct ipu3_uapi_awb_fr_config_s to the field in struct ipu3_uapi_4a_config, the other location where the struct is used.
--- drivers/staging/media/ipu3/include/intel-ipu3.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
drivers/staging/media/ipu3/include/intel-ipu3.h +3 -2
@@ -450,7 +450,7 @@ struct ipu3_uapi_awb_fr_config_s {__u32 bayer_sign;__u8 bayer_nf;__u8 reserved2[7 ]; -} __attribute__((aligned(32))) __packed; +} __packed;/** * struct ipu3_uapi_4a_config - 4A config
@@ -466,7 +466,8 @@ struct ipu3_uapi_4a_config {struct ipu3_uapi_ae_grid_config ae_grd_config;__u8 padding[20 ];struct ipu3_uapi_af_config_s af_config; -struct ipu3_uapi_awb_fr_config_s awb_fr_config; + struct ipu3_uapi_awb_fr_config_s awb_fr_config + __attribute__((aligned(32))); } __packed;/**
Greg Kroah-Hartman
maintainerPATCH 12/38 scsi: ufs: Release clock if DMA map fails
In reply to Greg Kroah-HartmanFrom: Can Guo <[email protected]>
commit 17c7d35f141ef6158076adf3338f115f64fcf760 upstream.
In queuecommand path, if DMA map fails, it bails out with clock held. In this case, release the clock to keep its usage paired.
[mkp: applied by hand]
[EB: resolved cherry-pick conflict caused by newer kernels not having
the clear_bit_unlock() line]
--- drivers/scsi/ufs/ufshcd.c | 1 + 1 file changed, 1 insertion(+)
drivers/scsi/ufs/ufshcd.c +1 -0
@@ -2480,6 +2480,7 @@ static int ufshcd_queuecommand(struct Sc err =ufshcd_map_sg (hba, lrbp);if (err) { +ufshcd_release (hba); lrbp->cmd =NULL ;clear_bit_unlock (tag, &hba->lrb_in_use); goto out;
Greg Kroah-Hartman
maintainerPATCH 13/38 net: dsa: mt7530: set CPU port to fallback mode
In reply to Greg Kroah-HartmanFrom: DENG Qingfang <[email protected]>
commit 38152ea37d8bdaffa22603e0a5b5b86cfa8714c9 upstream.
Currently, setting a bridge's self PVID to other value and deleting the default VID 1 renders untagged ports of that VLAN unable to talk to the CPU port:
bridge vlan add dev br0 vid 2 pvid untagged self bridge vlan del dev br0 vid 1 self
bridge vlan add dev sw0p0 vid 2 pvid untagged bridge vlan del dev sw0p0 vid 1
# br0 cannot send untagged frames out of sw0p0 anymore
That is because the CPU port is set to security mode and its PVID is still 1, and untagged frames are dropped due to VLAN member violation.
Set the CPU port to fallback mode so untagged frames can pass through.
--- drivers/net/dsa/mt7530.c | 11 ++++++++--- drivers/net/dsa/mt7530.h | 6 ++++++ 2 files changed, 14 insertions(+), 3 deletions(-)
drivers/net/dsa/mt7530.c +8 -3
@@ -818,10 +818,15 @@ mt7530_port_set_vlan_aware(struct dsa_sw PCR_MATRIX_MASK, PCR_MATRIX(MT7530_ALL_MEMBERS));/* Trapped into security mode allows packet forwarding through VLAN -* table lookup. +* table lookup. CPU port is set to fallback mode to let untagged +* frames pass through. */ -mt7530_rmw (priv,MT7530_PCR_P (port),PCR_PORT_VLAN_MASK , -MT7530_PORT_SECURITY_MODE ); +if (dsa_is_cpu_port (ds, port)) +mt7530_rmw (priv,MT7530_PCR_P (port),PCR_PORT_VLAN_MASK , +MT7530_PORT_FALLBACK_MODE ); +else +mt7530_rmw (priv,MT7530_PCR_P (port),PCR_PORT_VLAN_MASK , +MT7530_PORT_SECURITY_MODE );/* Set the port as a user port which is to be able to recognize VID * from incoming packets before fetching entry within the VLAN table.
drivers/net/dsa/mt7530.h +6 -0
@@ -148,6 +148,12 @@ enum mt7530_port_mode {/* Port Matrix Mode: Frames are forwarded by the PCR_MATRIX members. */ MT7530_PORT_MATRIX_MODE =PORT_VLAN (0 ), +/* Fallback Mode: Forward received frames with ingress ports that do +* not belong to the VLAN member. Frames whose VID is not listed on +* the VLAN table are forwarded by the PCR_MATRIX members. +*/ + MT7530_PORT_FALLBACK_MODE = PORT_VLAN(1), +/* Security Mode: Discard any frame due to ingress membership * violation or VID missed on the VLAN table. */
Greg Kroah-Hartman
maintainerPATCH 14/38 airo: Fix read overflows sending packets
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 11e7a91994c29da96d847f676be023da6a2c1359 upstream.
The problem is that we always copy a minimum of ETH_ZLEN (60) bytes from skb->data even when skb->len is less than ETH_ZLEN so it leads to a read overflow.
The fix is to pad skb->data to at least ETH_ZLEN bytes.
--- drivers/net/wireless/cisco/airo.c | 12 ++++++++++++ 1 file changed, 12 insertions(+)
drivers/net/wireless/cisco/airo.c +12 -0
@@ -1925,6 +1925,10 @@ static netdev_tx_t mpi_start_xmit(structairo_print_err (dev->name,"%s: skb == NULL!" ,__func__);return NETDEV_TX_OK ; } +if (skb_padto (skb,ETH_ZLEN )) { + dev->stats.tx_dropped++; +return NETDEV_TX_OK ; + } npacks =skb_queue_len (&ai->txq);if (npacks >=MAXTXQ -1 ) {
@@ -2127,6 +2131,10 @@ static netdev_tx_t airo_start_xmit(strucairo_print_err (dev->name,"%s: skb == NULL!" , __func__);return NETDEV_TX_OK ; } +if (skb_padto (skb,ETH_ZLEN )) { + dev->stats.tx_dropped++; +return NETDEV_TX_OK ; + }/* Find a vacant FID */ for ( i =0 ; i <MAX_FIDS /2 && (fids[i] &0xffff0000 ); i++ );
@@ -2201,6 +2209,10 @@ static netdev_tx_t airo_start_xmit11(strairo_print_err (dev->name,"%s: skb == NULL!" , __func__);return NETDEV_TX_OK ; } +if (skb_padto (skb,ETH_ZLEN )) { + dev->stats.tx_dropped++; +return NETDEV_TX_OK ; + }/* Find a vacant FID */ for ( i =MAX_FIDS /2 ; i <MAX_FIDS && (fids[i] &0xffff0000 ); i++ );
Greg Kroah-Hartman
maintainerPATCH 15/38 drm/i915: fix port checks for MST support on gen >= 11
In reply to Greg Kroah-HartmanFrom: Lucas De Marchi <[email protected]>
[ Upstream commit 10d987fd1b7baceaafa78d805e71427ab735b4e4 ]
Both Ice Lake and Elkhart Lake (gen 11) support MST on all external connections except DDI A. Tiger Lake (gen 12) supports on all external connections.
Move the check to happen inside intel_dp_mst_encoder_init() and add specific platform checks.
v2: Replace != with == checks for ports on gen < 11 (Ville)
--- drivers/gpu/drm/i915/display/intel_dp.c | 7 ++----- drivers/gpu/drm/i915/display/intel_dp_mst.c | 22 +++++++++++++++------ 2 files changed, 18 insertions(+), 11 deletions(-)
drivers/gpu/drm/i915/display/intel_dp.c +2 -5
@@ -7218,11 +7218,8 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port, intel_connector->get_hw_state = intel_connector_get_hw_state;/* init MST on ports that can support it */ -if (HAS_DP_MST (dev_priv) && !intel_dp_is_edp (intel_dp) && - (port ==PORT_B || port ==PORT_C || - port ==PORT_D || port ==PORT_F )) -intel_dp_mst_encoder_init (intel_dig_port, - intel_connector->base.base.id); +intel_dp_mst_encoder_init (intel_dig_port, + intel_connector->base.base.id);if (!intel_edp_init_connector (intel_dp, intel_connector)) {intel_dp_aux_fini (intel_dp);
drivers/gpu/drm/i915/display/intel_dp_mst.c +16 -6
@@ -653,21 +653,31 @@ intel_dp_mst_encoder_active_links(struct intel_digital_port *intel_dig_port)int intel_dp_mst_encoder_init (struct intel_digital_port *intel_dig_port,int conn_base_id) { +struct drm_i915_private *i915 =to_i915 (intel_dig_port->base.base.dev);struct intel_dp *intel_dp = &intel_dig_port->dp; -struct drm_device *dev = intel_dig_port->base.base.dev; +enum port port = intel_dig_port->base.port;int ret; - intel_dp->can_mst = true; +if (!HAS_DP_MST (i915) ||intel_dp_is_edp (intel_dp)) +return 0 ; + +if (INTEL_GEN (i915) <12 && port ==PORT_A ) +return 0 ; + +if (INTEL_GEN (i915) <11 && port ==PORT_E ) +return 0 ; + intel_dp->mst_mgr.cbs = &mst_cbs;/* create encoders */ intel_dp_create_fake_mst_encoders (intel_dig_port); - ret =drm_dp_mst_topology_mgr_init (&intel_dp->mst_mgr, dev, + ret =drm_dp_mst_topology_mgr_init (&intel_dp->mst_mgr, &i915->drm, &intel_dp->aux,16 ,3 , conn_base_id); -if (ret) { - intel_dp->can_mst = false; +if (ret)return ret; - } + + intel_dp->can_mst = true; +return 0 ; }
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 16/38 scsi: hisi_sas: Check sas_port before using it
In reply to Greg Kroah-HartmanFrom: Xiang Chen <[email protected]>
[ Upstream commit 8c39673d5474b95374df2104dc1f65205c5278b8 ]
Need to check the structure sas_port before using it.
--- drivers/scsi/hisi_sas/hisi_sas_main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/scsi/hisi_sas/hisi_sas_main.c +2 -1
@@ -974,12 +974,13 @@ static void hisi_sas_port_notify_formed(struct asd_sas_phy *sas_phy)struct hisi_hba *hisi_hba = sas_ha->lldd_ha;struct hisi_sas_phy *phy = sas_phy->lldd_phy;struct asd_sas_port *sas_port = sas_phy->port; -struct hisi_sas_port *port =to_hisi_sas_port (sas_port); +struct hisi_sas_port *port;unsigned long flags;if (!sas_port)return ; + port =to_hisi_sas_port (sas_port);spin_lock_irqsave (&hisi_hba->lock, flags); port->port_attached =1 ; port->id = phy->port_id;
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 17/38 powerpc/powernv: Avoid re-registration of imc debugfs directory
In reply to Greg Kroah-HartmanFrom: Anju T Sudhakar <[email protected]>
[ Upstream commit 48e626ac85b43cc589dd1b3b8004f7f85f03544d ]
export_imc_mode_and_cmd() function which creates the debugfs interface for imc-mode and imc-command, is invoked when each nest pmu units is registered.
When the first nest pmu unit is registered, export_imc_mode_and_cmd() creates 'imc' directory under /debug/powerpc/. In the subsequent invocations debugfs_create_dir() function returns, since the directory already exists.
The recent commit <c33d442328f55> (debugfs: make error message a bit more verbose), throws a warning if we try to invoke debugfs_create_dir() with an already existing directory name.
Address this warning by making the debugfs directory registration in the opal_imc_counters_probe() function, i.e invoke export_imc_mode_and_cmd() function from the probe function.
--- arch/powerpc/platforms/powernv/opal-imc.c | 39 ++++++++++------------- 1 file changed, 16 insertions(+), 23 deletions(-)
arch/powerpc/platforms/powernv/opal-imc.c +16 -23
@@ -59,10 +59,6 @@ static void export_imc_mode_and_cmd(struct device_node *node, imc_debugfs_parent =debugfs_create_dir ("imc" , powerpc_debugfs_root); -/* -* Return here, either because 'imc' directory already exists, -* Or failed to create a new one. -*/ if (!imc_debugfs_parent)return ;
@@ -135,7 +131,6 @@ static int imc_get_mem_addr_nest(struct device_node *node, } pmu_ptr->imc_counter_mmaped = true; -export_imc_mode_and_cmd (node, pmu_ptr);kfree (base_addr_arr);kfree (chipid_arr);return 0 ;
@@ -151,7 +146,7 @@ static int imc_get_mem_addr_nest(struct device_node *node,* and domain as the inputs. * Allocates memory for the struct imc_pmu, sets up its domain, size and offsets */ -static int imc_pmu_create (struct device_node *parent,int pmu_index,int domain) +static struct imc_pmu *imc_pmu_create (struct device_node *parent,int pmu_index,int domain) {int ret =0 ;struct imc_pmu *pmu_ptr;
@@ -159,27 +154,23 @@ static int imc_pmu_create(struct device_node *parent, int pmu_index, int domain)/* Return for unknown domain */ if (domain <0 ) -return -EINVAL ; +return NULL ;/* memory for pmu */ pmu_ptr =kzalloc (sizeof (*pmu_ptr),GFP_KERNEL );if (!pmu_ptr) -return -ENOMEM ; +return NULL ;/* Set the domain */ pmu_ptr->domain = domain; ret =of_property_read_u32 (parent,"size" , &pmu_ptr->counter_mem_size); -if (ret) { - ret = -EINVAL ; +if (ret) goto free_pmu; - }if (!of_property_read_u32 (parent,"offset" , &offset)) { -if (imc_get_mem_addr_nest (parent, pmu_ptr, offset)) { - ret = -EINVAL ; +if (imc_get_mem_addr_nest (parent, pmu_ptr, offset)) goto free_pmu; - } }/* Function to register IMC pmu */
@@ -190,14 +181,14 @@ static int imc_pmu_create(struct device_node *parent, int pmu_index, int domain)if (pmu_ptr->domain ==IMC_DOMAIN_NEST )kfree (pmu_ptr->mem_info);kfree (pmu_ptr); -return ret; +return NULL ; } -return 0 ; +return pmu_ptr; free_pmu:kfree (pmu_ptr); -return ret; +return NULL ; }static void disable_nest_pmu_counters (void )
@@ -254,6 +245,7 @@ int get_max_nest_dev(void)static int opal_imc_counters_probe (struct platform_device *pdev) {struct device_node *imc_dev = pdev->dev.of_node; +struct imc_pmu *pmu;int pmu_count =0 , domain;bool core_imc_reg = false, thread_imc_reg = false;u32 type;
@@ -269,6 +261,7 @@ static int opal_imc_counters_probe(struct platform_device *pdev) }for_each_compatible_node (imc_dev,NULL ,IMC_DTB_UNIT_COMPAT ) { + pmu =NULL ;if (of_property_read_u32 (imc_dev,"type" , &type)) {pr_warn ("IMC Device without type property\n" );continue ;
@@ -300,9 +293,13 @@ static int opal_imc_counters_probe(struct platform_device *pdev)break ; } -if (!imc_pmu_create (imc_dev, pmu_count, domain)) { -if (domain ==IMC_DOMAIN_NEST ) + pmu =imc_pmu_create (imc_dev, pmu_count, domain); +if (pmu !=NULL ) { +if (domain ==IMC_DOMAIN_NEST ) { +if (!imc_debugfs_parent) +export_imc_mode_and_cmd (imc_dev, pmu); pmu_count++; + }if (domain ==IMC_DOMAIN_CORE ) core_imc_reg = true;if (domain ==IMC_DOMAIN_THREAD )
@@ -310,10 +307,6 @@ static int opal_imc_counters_probe(struct platform_device *pdev) } } -/* If none of the nest units are registered, remove debugfs interface */ -if (pmu_count ==0 ) -debugfs_remove_recursive (imc_debugfs_parent); -/* If core imc is not registered, unregister thread-imc */ if (!core_imc_reg && thread_imc_reg)unregister_thread_imc ();
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 18/38 powerpc/xmon: Restrict when kernel is locked down
In reply to Greg Kroah-HartmanFrom: Christopher M. Riedl <[email protected]>
[ Upstream commit 69393cb03ccdf29f3b452d3482ef918469d1c098 ]
Xmon should be either fully or partially disabled depending on the kernel lockdown state.
Put xmon into read-only mode for lockdown=integrity and prevent user entry into xmon when lockdown=confidentiality. Xmon checks the lockdown state on every attempted entry:
(1) during early xmon'ing
(2) when triggered via sysrq
(3) when toggled via debugfs
(4) when triggered via a previously enabled breakpoint
The following lockdown state transitions are handled:
(1) lockdown=none -> lockdown=integrity
set xmon read-only mode
(2) lockdown=none -> lockdown=confidentiality clear all breakpoints, set xmon read-only mode, prevent user re-entry into xmon
(3) lockdown=integrity -> lockdown=confidentiality clear all breakpoints, set xmon read-only mode, prevent user re-entry into xmon
--- arch/powerpc/xmon/xmon.c | 103 ++++++++++++++++++++++++++++------- include/linux/security.h | 2 + security/lockdown/lockdown.c | 2 + 3 files changed, 86 insertions(+), 21 deletions(-)
arch/powerpc/xmon/xmon.c +82 -21
@@ -25,6 +25,7 @@#include <linux/nmi.h> #include <linux/ctype.h> #include <linux/highmem.h> +#include <linux/security.h> #include <asm/debugfs.h> #include <asm/ptrace.h>
@@ -187,6 +188,8 @@ static void dump_tlb_44x(void);static void dump_tlb_book3e (void );#endif +static void clear_all_bpt (void ); +#ifdef CONFIG_PPC64 #define REG "%.16lx"#else
@@ -283,10 +286,38 @@ Commands:\n\" U show uptime information\n" " ? help\n" " # n limit output to n lines per page (for dp, dpa, dl)\n" -" zr reboot\n\ -zh halt\n" +" zr reboot\n" +" zh halt\n" ; +#ifdef CONFIG_SECURITY +static bool xmon_is_locked_down (void ) +{ +static bool lockdown; + +if (!lockdown) { + lockdown = !!security_locked_down (LOCKDOWN_XMON_RW ); +if (lockdown) { +printf ("xmon: Disabled due to kernel lockdown\n" ); + xmon_is_ro = true; + } + } + +if (!xmon_is_ro) { + xmon_is_ro = !!security_locked_down (LOCKDOWN_XMON_WR ); +if (xmon_is_ro) +printf ("xmon: Read-only due to kernel lockdown\n" ); + } + +return lockdown; +} +#else /* CONFIG_SECURITY */ +static inline bool xmon_is_locked_down (void ) +{ +return false; +} +#endif +static struct pt_regs *xmon_regs;static inline void sync (void )
@@ -438,7 +469,10 @@ static bool wait_for_other_cpus(int ncpus) return false; } -#endif /* CONFIG_SMP */ +#else /* CONFIG_SMP */ +static inline void get_output_lock (void ) {} +static inline void release_output_lock (void ) {} +#endif static inline int unrecoverable_excp (struct pt_regs *regs) {
@@ -455,6 +489,7 @@ static int xmon_core(struct pt_regs *regs, int fromipi)int cmd =0 ;struct bpt *bp;long recurse_jmp[JMP_BUF_LEN ]; +bool locked_down;unsigned long offset;unsigned long flags;#ifdef CONFIG_SMP
@@ -465,6 +500,8 @@ static int xmon_core(struct pt_regs *regs, int fromipi)local_irq_save (flags);hard_irq_disable (); + locked_down =xmon_is_locked_down (); +if (!fromipi) { tracing_enabled =tracing_is_on ();tracing_off ();
@@ -518,7 +555,8 @@ static int xmon_core(struct pt_regs *regs, int fromipi)if (!fromipi) {get_output_lock (); -excprint (regs); +if (!locked_down) +excprint (regs);if (bp) {printf ("cpu 0x%x stopped at breakpoint 0x%tx (" , cpu,BP_NUM (bp));
@@ -570,10 +608,14 @@ static int xmon_core(struct pt_regs *regs, int fromipi) }remove_bpts ();disable_surveillance (); -/* for breakpoint or single step, print the current instr. */ -if (bp ||TRAP (regs) ==0xd00 ) -ppc_inst_dump (regs->nip,1 ,0 ); -printf ("enter ? for help\n" ); + +if (!locked_down) { +/* for breakpoint or single step, print curr insn */ +if (bp ||TRAP (regs) ==0xd00 ) +ppc_inst_dump (regs->nip,1 ,0 ); +printf ("enter ? for help\n" ); + } +mb (); xmon_gate =1 ;barrier ();
@@ -597,8 +639,9 @@ static int xmon_core(struct pt_regs *regs, int fromipi)spin_cpu_relax ();touch_nmi_watchdog (); }else { - cmd =cmds (regs); -if (cmd !=0 ) { +if (!locked_down) + cmd =cmds (regs); +if (locked_down || cmd !=0 ) {/* exiting xmon */ insert_bpts (); xmon_gate =0 ;
@@ -635,13 +678,16 @@ static int xmon_core(struct pt_regs *regs, int fromipi) "can't continue\n");remove_bpts ();disable_surveillance (); -/* for breakpoint or single step, print the current instr. */ -if (bp ||TRAP (regs) ==0xd00 ) -ppc_inst_dump (regs->nip,1 ,0 ); -printf ("enter ? for help\n" ); +if (!locked_down) { +/* for breakpoint or single step, print current insn */ +if (bp ||TRAP (regs) ==0xd00 ) +ppc_inst_dump (regs->nip,1 ,0 ); +printf ("enter ? for help\n" ); + } } - cmd =cmds (regs); +if (!locked_down) + cmd =cmds (regs);insert_bpts (); in_xmon =0 ;
@@ -670,7 +716,10 @@ static int xmon_core(struct pt_regs *regs, int fromipi) } }#endif -insert_cpu_bpts (); +if (locked_down) +clear_all_bpt (); +else +insert_cpu_bpts ();touch_nmi_watchdog ();local_irq_restore (flags);
@@ -3761,6 +3810,11 @@ static void xmon_init(int enable)#ifdef CONFIG_MAGIC_SYSRQ static void sysrq_handle_xmon (int key) { +if (xmon_is_locked_down ()) { +clear_all_bpt (); +xmon_init (0 ); +return ; + }/* ensure xmon is enabled */ xmon_init (1 );debugger (get_irq_regs ());
@@ -3782,7 +3836,6 @@ static int __init setup_xmon_sysrq(void)device_initcall (setup_xmon_sysrq);#endif /* CONFIG_MAGIC_SYSRQ */ -#ifdef CONFIG_DEBUG_FS static void clear_all_bpt (void ) {int i;
@@ -3800,18 +3853,22 @@ static void clear_all_bpt(void) iabr =NULL ; dabr.enabled =0 ; } - -printf ("xmon: All breakpoints cleared\n" ); } +#ifdef CONFIG_DEBUG_FS static int xmon_dbgfs_set (void *data,u64 val) { xmon_on = !!val;xmon_init (xmon_on);/* make sure all breakpoints removed when disabling */ -if (!xmon_on) +if (!xmon_on) {clear_all_bpt (); +get_output_lock (); +printf ("xmon: All breakpoints cleared\n" ); +release_output_lock (); + } +return 0 ; }
@@ -3837,7 +3894,11 @@ static int xmon_early __initdata;static int __init early_parse_xmon (char *p) { -if (!p ||strncmp (p,"early" ,5 ) ==0 ) { +if (xmon_is_locked_down ()) { +xmon_init (0 ); + xmon_early =0 ; + xmon_on =0 ; + }else if (!p ||strncmp (p,"early" ,5 ) ==0 ) {/* just "xmon" is equivalent to "xmon=early" */ xmon_init (1 ); xmon_early =1 ;
include/linux/security.h +2 -0
@@ -117,12 +117,14 @@ enum lockdown_reason {LOCKDOWN_MODULE_PARAMETERS ,LOCKDOWN_MMIOTRACE ,LOCKDOWN_DEBUGFS , +LOCKDOWN_XMON_WR ,LOCKDOWN_INTEGRITY_MAX ,LOCKDOWN_KCORE ,LOCKDOWN_KPROBES ,LOCKDOWN_BPF_READ ,LOCKDOWN_PERF ,LOCKDOWN_TRACEFS , +LOCKDOWN_XMON_RW , LOCKDOWN_CONFIDENTIALITY_MAX, };
security/lockdown/lockdown.c +2 -0
@@ -32,12 +32,14 @@ static const char *const lockdown_reasons[LOCKDOWN_CONFIDENTIALITY_MAX+1] = { [LOCKDOWN_MODULE_PARAMETERS] = "unsafe module parameters", [LOCKDOWN_MMIOTRACE] = "unsafe mmio", [LOCKDOWN_DEBUGFS] = "debugfs access", + [LOCKDOWN_XMON_WR] = "xmon write access", [LOCKDOWN_INTEGRITY_MAX] = "integrity", [LOCKDOWN_KCORE] = "/proc/kcore access", [LOCKDOWN_KPROBES] = "use of kprobes", [LOCKDOWN_BPF_READ] = "use of bpf to read kernel RAM", [LOCKDOWN_PERF] = "unsafe use of perf", [LOCKDOWN_TRACEFS] = "use of tracefs", + [LOCKDOWN_XMON_RW] = "xmon read and write access", [LOCKDOWN_CONFIDENTIALITY_MAX] = "confidentiality", };
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 19/38 spi: dw: use "smp_mb()" to avoid sending spi data error
In reply to Greg Kroah-HartmanFrom: Xinwei Kong <[email protected]>
[ Upstream commit bfda044533b213985bc62bd7ca96f2b984d21b80 ]
Because of out-of-order execution about some CPU architecture, In this debug stage we find Completing spi interrupt enable -> prodrucing TXEI interrupt -> running "interrupt_transfer" function will prior to set "dw->rx and dws->rx_end" data, so this patch add memory barrier to enable dw->rx and dw->rx_end to be visible and solve to send SPI data error.
eg:
it will fix to this following low possibility error in testing environment which using SPI control to connect TPM Modules
kernel: tpm tpm0: Operation Timed out
kernel: tpm tpm0: tpm_relinquish_locality: : error -1
--- drivers/spi/spi-dw.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/spi/spi-dw.c +3 -0
@@ -297,6 +297,9 @@ static int dw_spi_transfer_one(struct spi_controller *master, dws->len = transfer->len;spin_unlock_irqrestore (&dws->buf_lock, flags); +/* Ensure dw->rx and dw->rx_end are visible */ +smp_mb (); +spi_enable_chip (dws,0 );/* Handle per transfer options for bpw and speed */
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 20/38 ASoC: intel - fix the card names
In reply to Greg Kroah-HartmanFrom: Jaroslav Kysela <[email protected]>
[ Upstream commit d745cc1ab65945b2d17ec9c5652f38299c054649 ]
Those strings are exposed to the user space as the card name thus used in the GUIs. The common standard is to avoid '_' here. The worst case is 'sof-skl_hda_card' string.
--- sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c | 2 +- sound/soc/intel/boards/skl_hda_dsp_generic.c | 2 +- sound/soc/intel/boards/sof_rt5682.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)
sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c +1 -1
@@ -626,7 +626,7 @@ static int kabylake_card_late_probe(struct snd_soc_card *card)* kabylake audio machine driver for MAX98927 + RT5514 + RT5663 */ static struct snd_soc_card kabylake_audio_card = { - .name ="kbl_r5514_5663_max" , + .name ="kbl-r5514-5663-max" , .owner =THIS_MODULE , .dai_link = kabylake_dais, .num_links =ARRAY_SIZE (kabylake_dais),
sound/soc/intel/boards/skl_hda_dsp_generic.c +1 -1
@@ -90,7 +90,7 @@ skl_hda_add_dai_link(struct snd_soc_card *card, struct snd_soc_dai_link *link) }static struct snd_soc_card hda_soc_card = { - .name ="skl_hda_card" , + .name ="hda-dsp" , .owner =THIS_MODULE , .dai_link = skl_hda_be_dai_links, .dapm_widgets = skl_hda_widgets,
sound/soc/intel/boards/sof_rt5682.c +1 -1
@@ -374,7 +374,7 @@ static int dmic_init(struct snd_soc_pcm_runtime *rtd)/* sof audio machine driver for rt5682 codec */ static struct snd_soc_card sof_audio_card_rt5682 = { - .name ="sof_rt5682" , + .name ="rt5682" ,/* the sof- prefix is added by the core */ .owner =THIS_MODULE , .controls = sof_controls, .num_controls =ARRAY_SIZE (sof_controls),
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 21/38 s390/ftrace: save traced function caller
In reply to Greg Kroah-HartmanFrom: Vasily Gorbik <[email protected]>
[ Upstream commit b4adfe55915d8363e244e42386d69567db1719b9 ]
A typical backtrace acquired from ftraced function currently looks like the following (e.g. for "path_openat"):
arch_stack_walk+0x15c/0x2d8 stack_trace_save+0x50/0x68 stack_trace_call+0x15a/0x3b8 ftrace_graph_caller+0x0/0x1c 0x3e0007e3c98 <- ftraced function caller (should be do_filp_open+0x7c/0xe8) do_open_execat+0x70/0x1b8 __do_execve_file.isra.0+0x7d8/0x860 __s390x_sys_execve+0x56/0x68 system_call+0xdc/0x2d8
Note random "0x3e0007e3c98" stack value as ftraced function caller. This value causes either imprecise unwinder result or unwinding failure. That "0x3e0007e3c98" comes from r14 of ftraced function stack frame, which it haven't had a chance to initialize since the very first instruction calls ftrace code ("ftrace_caller"). (ftraced function might never save r14 as well). Nevertheless according to s390 ABI any function is called with stack frame allocated for it and r14 contains return address. "ftrace_caller" itself is called with "brasl %r0,ftrace_caller". So, to fix this issue simply always save traced function caller onto ftraced function stack frame.
--- arch/s390/kernel/mcount.S | 1 + 1 file changed, 1 insertion(+)
arch/s390/kernel/mcount.S +1 -0
@@ -41,6 +41,7 @@ EXPORT_SYMBOL(_mcount) ENTRY(ftrace_caller) .globl ftrace_regs_caller .set ftrace_regs_caller,ftrace_caller + stg %r14,(__SF_GPRS+8*8)(%r15) # save traced function caller lgr %r1,%r15 #if !(defined(CC_USING_HOTPATCH) || defined(CC_USING_NOP_MCOUNT)) aghi %r0,MCOUNT_RETURN_FIXUP
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 22/38 RDMA/qedr: Fix qpids xarray api used
In reply to Greg Kroah-HartmanFrom: Michal Kalderon <[email protected]>
[ Upstream commit 5fdff18b4dc64e2d1e912ad2b90495cd487f791b ]
The qpids xarray isn't accessed from irq context and therefore there is no need to use the xa_XXX_irq version of the apis. Remove the _irq.
--- drivers/infiniband/hw/qedr/main.c | 2 +- drivers/infiniband/hw/qedr/qedr_iw_cm.c | 2 +- drivers/infiniband/hw/qedr/verbs.c | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-)
drivers/infiniband/hw/qedr/main.c +1 -1
@@ -360,7 +360,7 @@ static int qedr_alloc_resources(struct qedr_dev *dev)xa_init_flags (&dev->srqs,XA_FLAGS_LOCK_IRQ );if (IS_IWARP (dev)) { -xa_init_flags (&dev->qps,XA_FLAGS_LOCK_IRQ ); +xa_init (&dev->qps); dev->iwarp_wq =create_singlethread_workqueue ("qedr_iwarpq" ); }
drivers/infiniband/hw/qedr/qedr_iw_cm.c +1 -1
@@ -739,7 +739,7 @@ void qedr_iw_qp_rem_ref(struct ib_qp *ibqp)struct qedr_qp *qp =get_qedr_qp (ibqp);if (atomic_dec_and_test (&qp->refcnt)) { -xa_erase_irq (&qp->dev->qps, qp->qp_id); +xa_erase (&qp->dev->qps, qp->qp_id);kfree (qp); } }
drivers/infiniband/hw/qedr/verbs.c +2 -2
@@ -1926,7 +1926,7 @@ struct ib_qp *qedr_create_qp(struct ib_pd *ibpd, qp->ibqp.qp_num = qp->qp_id;if (rdma_protocol_iwarp (&dev->ibdev,1 )) { - rc =xa_insert_irq (&dev->qps, qp->qp_id, qp,GFP_KERNEL ); + rc =xa_insert (&dev->qps, qp->qp_id, qp,GFP_KERNEL );if (rc) goto err; }
@@ -2500,7 +2500,7 @@ int qedr_destroy_qp(struct ib_qp *ibqp, struct ib_udata *udata)if (atomic_dec_and_test (&qp->refcnt) &&rdma_protocol_iwarp (&dev->ibdev,1 )) { -xa_erase_irq (&dev->qps, qp->qp_id); +xa_erase (&dev->qps, qp->qp_id);kfree (qp); }return 0 ;
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 23/38 RDMA/qedr: Fix synchronization methods and memory leaks in qedr
In reply to Greg Kroah-HartmanFrom: Michal Kalderon <[email protected]>
[ Upstream commit 82af6d19d8d9227c22a53ff00b40fb2a4f9fce69 ]
Re-design of the iWARP CM related objects reference counting and synchronization methods, to ensure operations are synchronized correctly and that memory allocated for "ep" is properly released. Also makes sure QP memory is not released before ep is finished accessing it.
Where as the QP object is created/destroyed by external operations, the ep is created/destroyed by internal operations and represents the tcp connection associated with the QP.
QP destruction flow:
- needs to wait for ep establishment to complete (either successfully or with error)
- needs to wait for ep disconnect to be fully posted to avoid a race condition of disconnect being called after reset.
- both the operations above don't always happen, so we use atomic flags to indicate whether the qp destruction flow needs to wait for these completions or not, if the destroy is called before these operations began, the flows will check the flags and not execute them ( connect / disconnect).
We use completion structure for waiting for the completions mentioned above.
The QP refcnt was modified to kref object. The EP has a kref added to it to handle additional worker thread accessing it.
Memory Leaks - https://www.spinics.net/lists/linux-rdma/msg83762.html
Concurrency not managed correctly -
https://www.spinics.net/lists/linux-rdma/msg67949.html
--- drivers/infiniband/hw/qedr/qedr.h | 23 +++- drivers/infiniband/hw/qedr/qedr_iw_cm.c | 148 ++++++++++++++++-------- drivers/infiniband/hw/qedr/verbs.c | 62 ++++++---- 3 files changed, 158 insertions(+), 75 deletions(-)
drivers/infiniband/hw/qedr/qedr.h +19 -4
@@ -40,6 +40,7 @@#include <linux/qed/qed_rdma_if.h> #include <linux/qed/qede_rdma.h> #include <linux/qed/roce_common.h> +#include <linux/completion.h> #include "qedr_hsi_rdma.h" #define QEDR_NODE_DESC "QLogic 579xx RoCE HCA"
@@ -377,10 +378,20 @@ enum qedr_qp_err_bitmap { QEDR_QP_ERR_RQ_PBL_FULL = 32, }; +enum qedr_qp_create_type { +QEDR_QP_CREATE_NONE , +QEDR_QP_CREATE_USER , +QEDR_QP_CREATE_KERNEL , +}; + +enum qedr_iwarp_cm_flags { +QEDR_IWARP_CM_WAIT_FOR_CONNECT =BIT (0 ), +QEDR_IWARP_CM_WAIT_FOR_DISCONNECT =BIT (1 ), +}; +struct qedr_qp {struct ib_qp ibqp;/* must be first */ struct qedr_dev *dev; -struct qedr_iw_ep *ep;struct qedr_qp_hwq_info sq;struct qedr_qp_hwq_info rq;
@@ -395,6 +406,7 @@ struct qedr_qp {u32 id;struct qedr_pd *pd;enum ib_qp_type qp_type; +enum qedr_qp_create_type create_type;struct qed_rdma_qp *qed_qp;u32 qp_id;u16 icid;
@@ -437,8 +449,11 @@ struct qedr_qp {/* Relevant to qps created from user space only (applications) */ struct qedr_userq usq;struct qedr_userq urq; -atomic_t refcnt; -bool destroyed; + +/* synchronization objects used with iwarp ep */ +struct kref refcnt; +struct completion iwarp_cm_comp; +unsigned long iwarp_cm_flags;/* enum iwarp_cm_flags */ };struct qedr_ah {
@@ -531,7 +546,7 @@ struct qedr_iw_ep {struct iw_cm_id *cm_id;struct qedr_qp *qp;void *qed_context; -u8 during_connect; +struct kref refcnt; }; static inline
drivers/infiniband/hw/qedr/qedr_iw_cm.c +100 -48
295 lines not on this page · raw message
drivers/infiniband/hw/qedr/verbs.c +39 -23
105 lines not on this page · raw message
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 24/38 ARC: Fix ICCM & DCCM runtime size checks
In reply to Greg Kroah-HartmanFrom: Eugeniy Paltsev <[email protected]>
[ Upstream commit 43900edf67d7ef3ac8909854d75b8a1fba2d570c ]
As of today the ICCM and DCCM size checks are incorrectly using mismatched units (KiB checked against bytes). The CONFIG_ARC_DCCM_SZ and CONFIG_ARC_ICCM_SZ are in KiB, but the size calculated in runtime and stored in cpu->dccm.sz and cpu->iccm.sz is in bytes.
Fix that.
--- arch/arc/kernel/setup.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
arch/arc/kernel/setup.c +3 -2
@@ -12,6 +12,7 @@#include <linux/clocksource.h> #include <linux/console.h> #include <linux/module.h> +#include <linux/sizes.h> #include <linux/cpu.h> #include <linux/of_fdt.h> #include <linux/of.h>
@@ -409,12 +410,12 @@ static void arc_chk_core_config(void)if ((unsigned int )__arc_dccm_base != cpu->dccm.base_addr)panic ("Linux built with incorrect DCCM Base address\n" ); -if (CONFIG_ARC_DCCM_SZ != cpu->dccm.sz) +if (CONFIG_ARC_DCCM_SZ *SZ_1K != cpu->dccm.sz)panic ("Linux built with incorrect DCCM Size\n" );#endif #ifdef CONFIG_ARC_HAS_ICCM -if (CONFIG_ARC_ICCM_SZ != cpu->iccm.sz) +if (CONFIG_ARC_ICCM_SZ *SZ_1K != cpu->iccm.sz)panic ("Linux built with incorrect ICCM Size\n" );#endif
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 25/38 ARC: [plat-eznps]: Restrict to CONFIG_ISA_ARCOMPACT
In reply to Greg Kroah-HartmanFrom: Vineet Gupta <[email protected]>
[ Upstream commit 799587d5731db9dcdafaac4002463aa7d9cd6cf7 ]
Elide invalid configuration EZNPS + ARCv2, triggered by a make allyesconfig build.
Granted the root cause is in source code (asm/barrier.h) where we check for ARCv2 before PLAT_EZNPS, but it is better to avoid such combinations at onset rather then baking subtle nuances into code.
--- arch/arc/plat-eznps/Kconfig | 1 + 1 file changed, 1 insertion(+)
arch/arc/plat-eznps/Kconfig +1 -0
@@ -6,6 +6,7 @@ menuconfig ARC_PLAT_EZNPS bool "\"EZchip\" ARC dev platform" + depends on ISA_ARCOMPACT select CPU_BIG_ENDIAN select CLKSRC_NPS if !PHYS_ADDR_T_64BIT select EZNPS_GIC
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 26/38 evm: Fix RCU list related warnings
In reply to Greg Kroah-HartmanFrom: Madhuparna Bhowmik <[email protected]>
[ Upstream commit 770f60586d2af0590be263f55fd079226313922c ]
This patch fixes the following warning and few other instances of traversal of evm_config_xattrnames list:
[ 32.848432] ============================= [ 32.848707] WARNING: suspicious RCU usage [ 32.848966] 5.7.0-rc1-00006-ga8d5875ce5f0b #1 Not tainted [ 32.849308] ----------------------------- [ 32.849567] security/integrity/evm/evm_main.c:231 RCU-list traversed in non-reader section!!
Since entries are only added to the list and never deleted, use list_for_each_entry_lockless() instead of list_for_each_entry_rcu for traversing the list. Also, add a relevant comment in evm_secfs.c to indicate this fact.
--- security/integrity/evm/evm_crypto.c | 2 +- security/integrity/evm/evm_main.c | 4 ++-- security/integrity/evm/evm_secfs.c | 9 ++++++++- 3 files changed, 11 insertions(+), 4 deletions(-)
security/integrity/evm/evm_crypto.c +1 -1
@@ -209,7 +209,7 @@ static int evm_calc_hmac_or_hash(struct dentry *dentry, data->hdr.length =crypto_shash_digestsize (desc->tfm); error = -ENODATA ; -list_for_each_entry_rcu (xattr, &evm_config_xattrnames, list) { +list_for_each_entry_lockless (xattr, &evm_config_xattrnames, list) {bool is_ima = false;if (strcmp (xattr->name,XATTR_NAME_IMA ) ==0 )
security/integrity/evm/evm_main.c +2 -2
@@ -99,7 +99,7 @@ static int evm_find_protected_xattrs(struct dentry *dentry)if (!(inode->i_opflags &IOP_XATTR ))return -EOPNOTSUPP ; -list_for_each_entry_rcu (xattr, &evm_config_xattrnames, list) { +list_for_each_entry_lockless (xattr, &evm_config_xattrnames, list) { error =__vfs_getxattr (dentry, inode, xattr->name,NULL ,0 );if (error <0 ) {if (error == -ENODATA )
@@ -230,7 +230,7 @@ static int evm_protected_xattr(const char *req_xattr_name)struct xattr_list *xattr; namelen =strlen (req_xattr_name); -list_for_each_entry_rcu (xattr, &evm_config_xattrnames, list) { +list_for_each_entry_lockless (xattr, &evm_config_xattrnames, list) {if ((strlen (xattr->name) == namelen) && (strncmp (req_xattr_name, xattr->name, namelen) ==0 )) { found =1 ;
security/integrity/evm/evm_secfs.c +8 -1
@@ -234,7 +234,14 @@ static ssize_t evm_write_xattrs(struct file *file, const char __user *buf, goto out; } -/* Guard against races in evm_read_xattrs */ +/* +* xattr_list_mutex guards against races in evm_read_xattrs(). +* Entries are only added to the evm_config_xattrnames list +* and never deleted. Therefore, the list is traversed +* using list_for_each_entry_lockless() without holding +* the mutex in evm_calc_hmac_or_hash(), evm_find_protected_xattrs() +* and evm_protected_xattr(). +*/ mutex_lock (&xattr_list_mutex);list_for_each_entry (tmp, &evm_config_xattrnames, list) {if (strcmp (xattr->name, tmp->name) ==0 ) {
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 27/38 scsi: pm: Balance pm_only counter of request queue during system resume
In reply to Greg Kroah-HartmanFrom: Can Guo <[email protected]>
[ Upstream commit 05d18ae1cc8a0308b12f37b4ab94afce3535fac9 ]
During system resume, scsi_resume_device() decreases a request queue's pm_only counter if the scsi device was quiesced before. But after that, if the scsi device's RPM status is RPM_SUSPENDED, the pm_only counter is still held (non-zero). Current SCSI resume hook only sets the RPM status of the scsi_device and its request queue to RPM_ACTIVE, but leaves the pm_only counter unchanged. This may make the request queue's pm_only counter remain non-zero after resume hook returns, hence those who are waiting on the mq_freeze_wq would never be woken up. Fix this by calling blk_post_runtime_resume() if a sdev's RPM status was RPM_SUSPENDED.
(struct request_queue)0xFFFFFF815B69E938
pm_only = (counter = 2), rpm_status = 0, dev = 0xFFFFFF815B0511A0,
((struct device)0xFFFFFF815B0511A0)).power
is_suspended = FALSE,
runtime_status = RPM_ACTIVE,
(struct scsi_device)0xffffff815b051000
request_queue = 0xFFFFFF815B69E938, sdev_state = SDEV_RUNNING, quiesced_by = 0x0,
B::v.f_/task_0xFFFFFF810C246940
-000|__switch_to(prev = 0xFFFFFF810C246940, next = 0xFFFFFF80A49357C0) -001|context_switch(inline) -001|__schedule(?) -002|schedule() -003|blk_queue_enter(q = 0xFFFFFF815B69E938, flags = 0) -004|generic_make_request(?) -005|submit_bio(bio = 0xFFFFFF80A8195B80)
--- drivers/scsi/scsi_pm.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)
drivers/scsi/scsi_pm.c +8 -2
@@ -80,6 +80,10 @@ static int scsi_dev_type_resume(struct device *dev,dev_dbg (dev,"scsi resume: %d\n" , err);if (err ==0 ) { +bool was_runtime_suspended; + + was_runtime_suspended =pm_runtime_suspended (dev); +pm_runtime_disable (dev); err =pm_runtime_set_active (dev);pm_runtime_enable (dev);
@@ -93,8 +97,10 @@ static int scsi_dev_type_resume(struct device *dev,*/ if (!err &&scsi_is_sdev_device (dev)) {struct scsi_device *sdev =to_scsi_device (dev); - -blk_set_runtime_active (sdev->request_queue); +if (was_runtime_suspended) +blk_post_runtime_resume (sdev->request_queue,0 ); +else +blk_set_runtime_active (sdev->request_queue); } }
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 28/38 i2c: altera: Fix race between xfer_msg and isr thread
In reply to Greg Kroah-HartmanFrom: Atsushi Nemoto <[email protected]>
[ Upstream commit 5d4c7977499a736f3f80826bdc9744344ad55589 ]
Use a mutex to protect access to idev->msg_len, idev->buf, etc. which are modified by both altr_i2c_xfer_msg() and altr_i2c_isr().
This is the minimal fix for easy backporting. A cleanup to remove the spinlock will be added later.
--- drivers/i2c/busses/i2c-altera.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-)
drivers/i2c/busses/i2c-altera.c +9 -1
@@ -70,6 +70,7 @@* @isr_mask: cached copy of local ISR enables. * @isr_status: cached copy of local ISR status. * @lock: spinlock for IRQ synchronization. +* @isr_mutex: mutex for IRQ thread. */ struct altr_i2c_dev {void __iomem *base;
@@ -86,6 +87,7 @@ struct altr_i2c_dev {u32 isr_mask;u32 isr_status;spinlock_t lock;/* IRQ synchronization */ +struct mutex isr_mutex; }; static void
@@ -245,10 +247,11 @@ static irqreturn_t altr_i2c_isr(int irq, void *_dev)struct altr_i2c_dev *idev = _dev;u32 status = idev->isr_status; +mutex_lock (&idev->isr_mutex);if (!idev->msg) {dev_warn (idev->dev,"unexpected interrupt\n" );altr_i2c_int_clear (idev,ALTR_I2C_ALL_IRQ ); -return IRQ_HANDLED ; + goto out; } read = (idev->msg->flags &I2C_M_RD ) !=0 ;
@@ -301,6 +304,8 @@ static irqreturn_t altr_i2c_isr(int irq, void *_dev)complete (&idev->msg_complete);dev_dbg (idev->dev,"Message Complete\n" ); } +out: +mutex_unlock (&idev->isr_mutex);return IRQ_HANDLED ; }
@@ -312,6 +317,7 @@ static int altr_i2c_xfer_msg(struct altr_i2c_dev *idev, struct i2c_msg *msg)u32 value;u8 addr =i2c_8bit_addr_from_msg (msg); +mutex_lock (&idev->isr_mutex); idev->msg = msg; idev->msg_len = msg->len; idev->buf = msg->buf;
@@ -336,6 +342,7 @@ static int altr_i2c_xfer_msg(struct altr_i2c_dev *idev, struct i2c_msg *msg)altr_i2c_int_enable (idev, imask, true);altr_i2c_fill_tx_fifo (idev); } +mutex_unlock (&idev->isr_mutex); time_left =wait_for_completion_timeout (&idev->msg_complete,ALTR_I2C_XFER_TIMEOUT );
@@ -409,6 +416,7 @@ static int altr_i2c_probe(struct platform_device *pdev) idev->dev = &pdev->dev;init_completion (&idev->msg_complete);spin_lock_init (&idev->lock); +mutex_init (&idev->isr_mutex); ret =device_property_read_u32 (idev->dev,"fifo-size" , &idev->fifo_size);
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 29/38 io_uring: initialize ctx->sqo_wait earlier
In reply to Greg Kroah-HartmanFrom: Jens Axboe <[email protected]>
[ Upstream commit 583863ed918136412ddf14de2e12534f17cfdc6f ]
Ensure that ctx->sqo_wait is initialized as soon as the ctx is allocated, instead of deferring it to the offload setup. This fixes a syzbot reported lockdep complaint, which is really due to trying to wake_up on an uninitialized wait queue:
RSP: 002b:00007fffb1fb9aa8 EFLAGS: 00000246 ORIG_RAX: 00000000000001a9
RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 0000000000441319
RDX: 0000000000000001 RSI: 0000000020000140 RDI: 000000000000047b
RBP: 0000000000010475 R08: 0000000000000001 R09: 00000000004002c8
R10: 0000000000000000 R11: 0000000000000246 R12: 0000000000402260
R13: 00000000004022f0 R14: 0000000000000000 R15: 0000000000000000
INFO: trying to register non-static key.
the code is fine but needs lockdep annotation.
turning off the locking correctness validator.
CPU: 1 PID: 7090 Comm: syz-executor222 Not tainted 5.7.0-rc1-next-20200415-syzkaller #0 Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011 Call Trace: __dump_stack lib/dump_stack.c:77 [inline] dump_stack+0x188/0x20d lib/dump_stack.c:118 assign_lock_key kernel/locking/lockdep.c:913 [inline] register_lock_class+0x1664/0x1760 kernel/locking/lockdep.c:1225 __lock_acquire+0x104/0x4c50 kernel/locking/lockdep.c:4234 lock_acquire+0x1f2/0x8f0 kernel/locking/lockdep.c:4934 __raw_spin_lock_irqsave include/linux/spinlock_api_smp.h:110 [inline] _raw_spin_lock_irqsave+0x8c/0xbf kernel/locking/spinlock.c:159 __wake_up_common_lock+0xb4/0x130 kernel/sched/wait.c:122 io_cqring_ev_posted+0xa5/0x1e0 fs/io_uring.c:1160 io_poll_remove_all fs/io_uring.c:4357 [inline] io_ring_ctx_wait_and_kill+0x2bc/0x5a0 fs/io_uring.c:7305 io_uring_create fs/io_uring.c:7843 [inline] io_uring_setup+0x115e/0x22b0 fs/io_uring.c:7870 do_syscall_64+0xf6/0x7d0 arch/x86/entry/common.c:295 entry_SYSCALL_64_after_hwframe+0x49/0xb3 RIP: 0033:0x441319 Code: e8 5c ae 02 00 48 83 c4 18 c3 0f 1f 80 00 00 00 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 0f 83 bb 0a fc ff c3 66 2e 0f 1f 84 00 00 00 00 RSP: 002b:00007fffb1fb9aa8 EFLAGS: 00000246 ORIG_RAX: 00000000000001a9
--- fs/io_uring.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
fs/io_uring.c +1 -1
@@ -409,6 +409,7 @@ static struct io_ring_ctx *io_ring_ctx_alloc(struct io_uring_params *p) } ctx->flags = p->flags; +init_waitqueue_head (&ctx->sqo_wait);init_waitqueue_head (&ctx->cq_wait);init_completion (&ctx->ctx_done);init_completion (&ctx->sqo_thread_started);
@@ -3237,7 +3238,6 @@ static int io_sq_offload_start(struct io_ring_ctx *ctx, {int ret; -init_waitqueue_head (&ctx->sqo_wait);mmgrab (current->mm); ctx->sqo_mm = current->mm;
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 30/38 x86/mmiotrace: Use cpumask_available() for cpumask_var_t variables
In reply to Greg Kroah-HartmanFrom: Nathan Chancellor <[email protected]>
[ Upstream commit d7110a26e5905ec2fe3fc88bc6a538901accb72b ]
When building with Clang + -Wtautological-compare and CONFIG_CPUMASK_OFFSTACK unset:
arch/x86/mm/mmio-mod.c:375:6: warning: comparison of array 'downed_cpus' equal to a null pointer is always false [-Wtautological-pointer-compare]
if (downed_cpus == NULL &&
^~~~~~~~~~~ ~~~~
arch/x86/mm/mmio-mod.c:405:6: warning: comparison of array 'downed_cpus'
equal to a null pointer is always false [-Wtautological-pointer-compare]
if (downed_cpus == NULL || cpumask_weight(downed_cpus) == 0)
^~~~~~~~~~~ ~~~~2 warnings generated.
Commit
f7e30f01a9e2 ("cpumask: Add helper cpumask_available()")
added cpumask_available() to fix warnings of this nature. Use that here so that clang does not warn regardless of CONFIG_CPUMASK_OFFSTACK's value.
--- arch/x86/mm/mmio-mod.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
arch/x86/mm/mmio-mod.c +2 -2
@@ -372,7 +372,7 @@ static void enter_uniprocessor(void)int cpu;int err; -if (downed_cpus ==NULL && +if (!cpumask_available (downed_cpus) && !alloc_cpumask_var (&downed_cpus,GFP_KERNEL )) {pr_notice ("Failed to allocate mask\n" ); goto out;
@@ -402,7 +402,7 @@ static void leave_uniprocessor(void)int cpu;int err; -if (downed_cpus ==NULL ||cpumask_weight (downed_cpus) ==0 ) +if (!cpumask_available (downed_cpus) ||cpumask_weight (downed_cpus) ==0 )return ;pr_notice ("Re-enabling CPUs...\n" );for_each_cpu (cpu, downed_cpus) {
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 31/38 net: bmac: Fix read of MAC address from ROM
In reply to Greg Kroah-HartmanFrom: Jeremy Kerr <[email protected]>
[ Upstream commit ef01cee2ee1b369c57a936166483d40942bcc3e3 ]
In bmac_get_station_address, We're reading two bytes at a time from ROM, but we do that six times, resulting in 12 bytes of read & writes. This means we will write off the end of the six-byte destination buffer.
This change fixes the for-loop to only read/write six bytes.
Based on a proposed fix from Finn Thain <[email protected]>.
--- drivers/net/ethernet/apple/bmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/apple/bmac.c +1 -1
@@ -1182,7 +1182,7 @@ bmac_get_station_address(struct net_device *dev, unsigned char *ea)int i;unsigned short data; -for (i =0 ; i <6 ; i++) +for (i =0 ; i <3 ; i++) {reset_and_select_srom (dev); data =read_srom (dev, i + EnetAddressOffset/2 , SROMAddressBits);
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 32/38 drm/edid: Add Oculus Rift S to non-desktop list
In reply to Greg Kroah-HartmanFrom: Jan Schmidt <[email protected]>
[ Upstream commit 5a3f610877e9d08968ea7237551049581f02b163 ]
Add a quirk for the Oculus Rift S OVR0012 display so it shows up as a non-desktop display.
--- drivers/gpu/drm/drm_edid.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/gpu/drm/drm_edid.c +2 -1
@@ -191,10 +191,11 @@ static const struct edid_quirk { { "HVR", 0xaa01, EDID_QUIRK_NON_DESKTOP }, { "HVR", 0xaa02, EDID_QUIRK_NON_DESKTOP }, -/* Oculus Rift DK1, DK2, and CV1 VR Headsets */ +/* Oculus Rift DK1, DK2, CV1 and Rift S VR Headsets */ { "OVR", 0x0001, EDID_QUIRK_NON_DESKTOP }, { "OVR", 0x0003, EDID_QUIRK_NON_DESKTOP }, { "OVR", 0x0004, EDID_QUIRK_NON_DESKTOP }, + { "OVR", 0x0012, EDID_QUIRK_NON_DESKTOP },/* Windows Mixed Reality Headsets */ { "ACR", 0x7fce, EDID_QUIRK_NON_DESKTOP },
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 33/38 s390/mm: fix set_huge_pte_at() for empty ptes
In reply to Greg Kroah-HartmanFrom: Gerald Schaefer <[email protected]>
[ Upstream commit ac8372f3b4e41015549b331a4f350224661e7fc6 ]
On s390, the layout of normal and large ptes (i.e. pmds/puds) differs. Therefore, set_huge_pte_at() does a conversion from a normal pte to the corresponding large pmd/pud. So, when converting an empty pte, this should result in an empty pmd/pud, which would return true for pmd/pud_none().
However, after conversion we also mark the pmd/pud as large, and therefore present. For empty ptes, this will result in an empty pmd/pud that is also marked as large, and pmd/pud_none() would not return true.
There is currently no issue with this behaviour, as set_huge_pte_at() does not seem to be called for empty ptes. It would be valid though, so let's fix this by not marking empty ptes as large in set_huge_pte_at().
This was found by testing a patch from from Anshuman Khandual, which is currently discussed on LKML ("mm/debug: Add more arch page table helper tests").
--- arch/s390/mm/hugetlbpage.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
arch/s390/mm/hugetlbpage.c +6 -3
@@ -159,10 +159,13 @@ void set_huge_pte_at(struct mm_struct *mm, unsigned long addr, rste &= ~_SEGMENT_ENTRY_NOEXEC;/* Set correct table type for 2G hugepages */ -if ((pte_val (*ptep) & _REGION_ENTRY_TYPE_MASK) == _REGION_ENTRY_TYPE_R3) - rste |= _REGION_ENTRY_TYPE_R3 | _REGION3_ENTRY_LARGE; -else +if ((pte_val (*ptep) & _REGION_ENTRY_TYPE_MASK) == _REGION_ENTRY_TYPE_R3) { +if (likely (pte_present (pte))) + rste |= _REGION3_ENTRY_LARGE; + rste |= _REGION_ENTRY_TYPE_R3; + }else if (likely (pte_present (pte))) rste |= _SEGMENT_ENTRY_LARGE; +clear_huge_pte_skeys (mm, rste);pte_val (*ptep) = rste; }
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 34/38 null_blk: return error for invalid zone size
In reply to Greg Kroah-HartmanFrom: Chaitanya Kulkarni <[email protected]>
[ Upstream commit e274832590211c4b1b1e807ca66fad8b5bb8b328 ]
In null_init_zone_dev() check if the zone size is larger than device capacity, return error if needed.
This also fixes the following oops :-
null_blk: changed the number of conventional zones to 4294967295
BUG: kernel NULL pointer dereference, address: 0000000000000010 PGD 7d76c5067 P4D 7d76c5067 PUD 7d240c067 PMD 0 Oops: 0002 [#1] SMP NOPTI CPU: 4 PID: 5508 Comm: nullbtests.sh Tainted: G OE 5.7.0-rc4lblk-fnext0 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.12.0-59-gc9ba5276e4 RIP: 0010:null_init_zoned_dev+0x17a/0x27f [null_blk] RSP: 0018:ffffc90007007e00 EFLAGS: 00010246 RAX: 0000000000000020 RBX: ffff8887fb3f3c00 RCX: 0000000000000007 RDX: 0000000000000000 RSI: ffff8887ca09d688 RDI: ffff888810fea510 RBP: 0000000000000010 R08: ffff8887ca09d688 R09: 0000000000000000 R10: 0000000000000000 R11: 0000000000000000 R12: ffff8887c26e8000 R13: ffffffffa05e9390 R14: 0000000000000000 R15: 0000000000000001 FS: 00007fcb5256f740(0000) GS:ffff888810e00000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 CR2: 0000000000000010 CR3: 000000081e8fe000 CR4: 00000000003406e0 Call Trace: null_add_dev+0x534/0x71b [null_blk] nullb_device_power_store.cold.41+0x8/0x2e [null_blk] configfs_write_file+0xe6/0x150 vfs_write+0xba/0x1e0 ksys_write+0x5f/0xe0 do_syscall_64+0x60/0x250 entry_SYSCALL_64_after_hwframe+0x49/0xb3 RIP: 0033:0x7fcb51c71840
--- drivers/block/null_blk_zoned.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/block/null_blk_zoned.c +4 -0
@@ -20,6 +20,10 @@ int null_zone_init(struct nullb_device *dev)pr_err ("zone_size must be power-of-two\n" );return -EINVAL ; } +if (dev->zone_size > dev->size) { +pr_err ("Zone size larger than device capacity\n" ); +return -EINVAL ; + } dev->zone_size_sects = dev->zone_size <<ZONE_SIZE_SHIFT ; dev->nr_zones = dev_size >>
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 35/38 net/ethernet/freescale: rework quiesce/activate for ucc_geth
In reply to Greg Kroah-HartmanFrom: Valentin Longchamp <[email protected]>
[ Upstream commit 79dde73cf9bcf1dd317a2667f78b758e9fe139ed ]
ugeth_quiesce/activate are used to halt the controller when there is a link change that requires to reconfigure the mac.
The previous implementation called netif_device_detach(). This however causes the initial activation of the netdevice to fail precisely because it's detached. For details, see [1].
A possible workaround was the revert of commit
net: linkwatch: add check for netdevice being present to linkwatch_do_dev However, the check introduced in the above commit is correct and shall be kept.
The netif_device_detach() is thus replaced with netif_tx_stop_all_queues() that prevents any tranmission. This allows to perform mac config change required by the link change, without detaching the corresponding netdevice and thus not preventing its initial activation.
[1] https://lists.openwall.net/netdev/2020/01/08/201
--- drivers/net/ethernet/freescale/ucc_geth.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-)
drivers/net/ethernet/freescale/ucc_geth.c +7 -6
@@ -42,6 +42,7 @@#include <soc/fsl/qe/ucc.h> #include <soc/fsl/qe/ucc_fast.h> #include <asm/machdep.h> +#include <net/sch_generic.h> #include "ucc_geth.h"
@@ -1548,11 +1549,8 @@ static int ugeth_disable(struct ucc_geth_private *ugeth, enum comm_dir mode)static void ugeth_quiesce (struct ucc_geth_private *ugeth) { -/* Prevent any further xmits, plus detach the device. */ -netif_device_detach (ugeth->ndev); - -/* Wait for any current xmits to finish. */ -netif_tx_disable (ugeth->ndev); +/* Prevent any further xmits */ +netif_tx_stop_all_queues (ugeth->ndev);/* Disable the interrupt to avoid NAPI rescheduling. */ disable_irq (ugeth->ug_info->uf_info.irq);
@@ -1565,7 +1563,10 @@ static void ugeth_activate(struct ucc_geth_private *ugeth) {napi_enable (&ugeth->napi);enable_irq (ugeth->ug_info->uf_info.irq); -netif_device_attach (ugeth->ndev); + +/* allow to xmit again */ +netif_tx_wake_all_queues (ugeth->ndev); +__netdev_watchdog_up (ugeth->ndev); }/* Called every time the controller might need to be made
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 36/38 net: ethernet: stmmac: Enable interface clocks on probe for IPQ806x
In reply to Greg Kroah-HartmanFrom: Jonathan McDowell <[email protected]>
[ Upstream commit a96ac8a0045e3cbe3e5af6d1b3c78c6c2065dec5 ]
The ipq806x_gmac_probe() function enables the PTP clock but not the appropriate interface clocks. This means that if the bootloader hasn't done so attempting to bring up the interface will fail with an error like:
[ 59.028131] ipq806x-gmac-dwmac 37600000.ethernet: Failed to reset the dma [ 59.028196] ipq806x-gmac-dwmac 37600000.ethernet eth1: stmmac_hw_setup: DMA engine initialization failed [ 59.034056] ipq806x-gmac-dwmac 37600000.ethernet eth1: stmmac_open: Hw setup failed
This patch, a slightly cleaned up version of one posted by Sergey Sergeev in:
https://forum.openwrt.org/t/support-for-mikrotik-rb3011uias-rm/4064/257
correctly enables the clock; we have already configured the source just before this.
Tested on a MikroTik RB3011.
--- drivers/net/ethernet/stmicro/stmmac/dwmac-ipq806x.c | 13 +++++++++++++ 1 file changed, 13 insertions(+)
drivers/net/ethernet/stmicro/stmmac/dwmac-ipq806x.c +13 -0
@@ -318,6 +318,19 @@ static int ipq806x_gmac_probe(struct platform_device *pdev)/* Enable PTP clock */ regmap_read (gmac->nss_common,NSS_COMMON_CLK_GATE , &val); val |=NSS_COMMON_CLK_GATE_PTP_EN (gmac->id); +switch (gmac->phy_mode) { +case PHY_INTERFACE_MODE_RGMII : + val |=NSS_COMMON_CLK_GATE_RGMII_RX_EN (gmac->id) | +NSS_COMMON_CLK_GATE_RGMII_TX_EN (gmac->id); +break ; +case PHY_INTERFACE_MODE_SGMII : + val |=NSS_COMMON_CLK_GATE_GMII_RX_EN (gmac->id) | +NSS_COMMON_CLK_GATE_GMII_TX_EN (gmac->id); +break ; +default : +/* We don't get here; the switch above will have errored out */ +unreachable (); + }regmap_write (gmac->nss_common,NSS_COMMON_CLK_GATE , val);if (gmac->phy_mode ==PHY_INTERFACE_MODE_SGMII ) {
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 37/38 selftests: mlxsw: qos_mc_aware: Specify arping timeout as an integer
In reply to Greg Kroah-HartmanFrom: Amit Cohen <[email protected]>
[ Upstream commit 46ca11177ed593f39d534f8d2c74ec5344e90c11 ]
Starting from iputils s20190709 (used in Fedora 31), arping does not support timeout being specified as a decimal:
$ arping -c 1 -I swp1 -b 192.0.2.66 -q -w 0.1 arping: invalid argument: '0.1'
Previously, such timeouts were rounded to an integer.
Fix this by specifying the timeout as an integer.
--- tools/testing/selftests/drivers/net/mlxsw/qos_mc_aware.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
tools/testing/selftests/drivers/net/mlxsw/qos_mc_aware.sh +1 -1
@@ -300,7 +300,7 @@ test_uc_aware() local i for ((i = 0; i < attempts; ++i)); do - if $ARPING -c 1 -I $h1 -b 192.0.2.66 -q -w 0.1; then + if $ARPING -c 1 -I $h1 -b 192.0.2.66 -q -w 1; then ((passes++)) fi
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerPATCH 38/38 net: smsc911x: Fix runtime PM imbalance on error
In reply to Greg Kroah-HartmanFrom: Dinghao Liu <[email protected]>
[ Upstream commit 539d39ad0c61b35f69565a037d7586deaf6d6166 ]
Remove runtime PM usage counter decrement when the increment function has not been called to keep the counter balanced.
--- drivers/net/ethernet/smsc/smsc911x.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)
drivers/net/ethernet/smsc/smsc911x.c +5 -4
@@ -2502,20 +2502,20 @@ static int smsc911x_drv_probe(struct platform_device *pdev) retval =smsc911x_init (dev);if (retval <0 ) - goto out_disable_resources; + goto out_init_fail;netif_carrier_off (dev); retval =smsc911x_mii_init (pdev, dev);if (retval) {SMSC_WARN (pdata, probe,"Error %i initialising mii" , retval); - goto out_disable_resources; + goto out_init_fail; } retval =register_netdev (dev);if (retval) {SMSC_WARN (pdata, probe,"Error %i registering device" , retval); - goto out_disable_resources; + goto out_init_fail; }else {SMSC_TRACE (pdata, probe,"Network interface: \"%s\"" , dev->name);
@@ -2556,9 +2556,10 @@ static int smsc911x_drv_probe(struct platform_device *pdev)return 0 ; -out_disable_resources: +out_init_fail:pm_runtime_put (&pdev->dev);pm_runtime_disable (&pdev->dev); +out_disable_resources: (void )smsc911x_disable_resources (pdev); out_enable_resources_fail:smsc911x_free_resources (pdev);
Signature · 1 line
2.25.1
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.45 release. There are 38 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 Sun, 07 Jun 2020 13:54:56 +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.45-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.45 release. There are 38 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 Sun, 07 Jun 2020 13:54:56 +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.45-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-5.4:
11 builds: 11 pass, 0 fail
26 boots: 26 pass, 0 fail
50 tests: 50 pass, 0 failLinux version: 5.4.45-rc1-g8a32c9d517eb
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 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.45 release. There are 38 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 Sun, 07 Jun 2020 13:54:56 +0000. Anything received after that time might be too late.
Build results:
total: 157 pass: 157 fail: 0 Qemu test results: total: 430 pass: 430 fail: 0
Guenter
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.45 release. There are 38 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 Sun, 07 Jun 2020 13:54:56 +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.45-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.45-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: 0e4e419d5fc3f776cc5ac829737dd6020f89f2a6
git describe: v5.4.44-39-g0e4e419d5fc3
Test details: https://qa-reports.linaro.org/lkft/linux-stable-rc-5.4-oe/build/v5.4.44-39-g0e4e419d5fc3
No regressions (compared to build v5.4.44)
No fixes (compared to build v5.4.44)
Ran 32219 total tests in the following environments and test suites.
Environments
--------------
- dragonboard-410c
- hi6220-hikey
- i386
- juno-r2
- juno-r2-compat
- juno-r2-kasan
- nxp-ls2088
- qemu_arm
- qemu_arm64
- qemu_i386
- qemu_x86_64
- x15
- x86
- x86-kasan
Test Suites
-----------
* build
* install-android-platform-tools-r2600
* install-android-platform-tools-r2800
* kselftest
* kselftest/drivers
* kselftest/filesystems
* kselftest/net
* libgpiod
* linux-log-parser
* ltp-cap_bounds-tests
* ltp-commands-tests
* ltp-containers-tests
* ltp-cpuhotplug-tests
* ltp-crypto-tests
* ltp-dio-tests
* ltp-hugetlb-tests
* ltp-io-tests
* ltp-math-tests
* ltp-mm-tests
* ltp-sched-tests
* ltp-syscalls-tests
* perf
* v4l2-compliance
* libhugetlbfs
* ltp-cve-tests
* ltp-fcntl-locktests-tests
* ltp-filecaps-tests
* ltp-fs-tests
* ltp-fs_bind-tests
* ltp-fs_perms_simple-tests
* ltp-fsx-tests
* ltp-ipc-tests
* ltp-nptl-tests
* ltp-pty-tests
* ltp-securebits-tests
* network-basic-tests
* ltp-controllers-tests
* ltp-open-posix-tests
* kvm-unit-tests
* kselftest-vsyscall-mode-native
* kselftest-vsyscall-mode-native/drivers
* kselftest-vsyscall-mode-native/filesystems
* kselftest-vsyscall-mode-native/net
* kselftest-vsyscall-mode-none
* kselftest-vsyscall-mode-none/drivers
* kselftest-vsyscall-mode-none/filesystems
* kselftest-vsyscall-mode-none/net
Signature · 2 lines
Linaro LKFT https://lkft.linaro.org