5.4.267-rc1 review
Subsystems (2)
Greg Kroah-Hartman
maintainercover 5.4.267-rc1 review
This is the start of the stable review cycle for the 5.4.267 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 Mon, 15 Jan 2024 09:41:55 +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.267-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.267-rc1
Jon Maxwell <[email protected]>
ipv6: remove max_size check inline with ipv4
Eric Dumazet <[email protected]>
ipv6: make ip6_rt_gc_expire an atomic_t
Eric Dumazet <[email protected]>
net/dst: use a smaller percpu_counter batch for dst entries accounting
Bartosz Pawlowski <[email protected]>
PCI: Disable ATS for specific Intel IPU E2000 devices
Bartosz Pawlowski <[email protected]>
PCI: Extract ATS disabling to a helper function
Phil Sutter <[email protected]>
netfilter: nf_tables: Reject tables of unsupported family
John Fastabend <[email protected]>
net: tls, update curr on splice as well
Douglas Anderson <[email protected]>
ath10k: Get rid of "per_ce_irq" hw param
Douglas Anderson <[email protected]>
ath10k: Keep track of which interrupts fired, don't poll them
Rakesh Pillai <[email protected]>
ath10k: Add interrupt summary based CE processing
Douglas Anderson <[email protected]>
ath10k: Wait until copy complete is actually done before completing
Wenchao Chen <[email protected]>
mmc: sdhci-sprd: Fix eMMC init failure after hw reset
Geert Uytterhoeven <[email protected]>
mmc: core: Cancel delayed work before releasing host
Jorge Ramirez-Ortiz <[email protected]>
mmc: rpmb: fixes pause retune on all RPMB partitions.
Jiajun Xie <[email protected]>
mm: fix unmap_mapping_range high bits shift bug
Benjamin Bara <[email protected]>
i2c: core: Fix atomic xfer check for non-preempt config
Takashi Sakamoto <[email protected]>
firewire: ohci: suppress unexpected system reboot in AMD Ryzen machines and ASM108x/VT630x PCIe cards
Matthew Wilcox (Oracle) <[email protected]>
mm/memory-failure: check the mapcount of the precise page
Thomas Lange <[email protected]>
net: Implement missing SO_TIMESTAMPING_NEW cmsg support
Michael Chan <[email protected]>
bnxt_en: Remove mis-applied code from bnxt_cfg_ntp_filters()
Chen Ni <[email protected]>
asix: Add check for usbnet_get_endpoints
Dinghao Liu <[email protected]>
net/qla3xxx: fix potential memleak in ql_alloc_buffer_queues
Christophe JAILLET <[email protected]>
net/qla3xxx: switch from 'pci_' to 'dma_' API
Andrii Staikov <[email protected]>
i40e: Restore VF MSI-X state during PCI reset
Mark Brown <[email protected]>
ASoC: meson: g12a-tohdmitx: Fix event generation for S/PDIF mux
Mark Brown <[email protected]>
ASoC: meson: g12a-tohdmitx: Validate written enum values
Jerome Brunet <[email protected]>
ASoC: meson: g12a: extract codec-to-codec utils
Ke Xiao <[email protected]>
i40e: fix use-after-free in i40e_aqc_add_filters()
Marc Dionne <[email protected]>
net: Save and restore msg_namelen in sock_sendmsg
Adrian Cinal <[email protected]>
net: bcmgenet: Fix FCS generation for fragmented skbuffs
Stefan Wahren <[email protected]>
ARM: sun9i: smp: Fix array-index-out-of-bounds read in sunxi_mc_smp_init
Vadim Fedorenko <[email protected]>
net-timestamp: extend SOF_TIMESTAMPING_OPT_ID to HW timestamps
Marc Kleine-Budde <[email protected]>
can: raw: add support for SO_MARK
Marc Kleine-Budde <[email protected]>
can: raw: add support for SO_TXTIME/SCM_TXTIME
Jörn-Thorben Hinz <[email protected]>
net: Implement missing getsockopt(SO_TIMESTAMPING_NEW)
Hangyu Hua <[email protected]>
net: sched: em_text: fix possible memory leak in em_text_destroy()
Sudheer Mogilappagari <[email protected]>
i40e: Fix filter input checks to prevent config with invalid values
Siddh Raman Pant <[email protected]>
nfc: llcp_core: Hold a ref to llcp_local->dev when holding a ref to llcp_local
-------------
Diffstat:
Makefile | 4 +- arch/arm/mach-sunxi/mc_smp.c | 4 +- drivers/firewire/ohci.c | 51 +++++ drivers/i2c/i2c-core.h | 4 +- drivers/mmc/core/block.c | 7 +- drivers/mmc/core/host.c | 1 + drivers/mmc/host/sdhci-sprd.c | 10 +- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 4 +- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 4 +- drivers/net/ethernet/intel/i40e/i40e_main.c | 11 +- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 34 ++- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h | 3 + drivers/net/ethernet/qlogic/qla3xxx.c | 198 ++++++++---------- drivers/net/usb/ax88172a.c | 4 +- drivers/net/wireless/ath/ath10k/ce.c | 79 +++---- drivers/net/wireless/ath/ath10k/ce.h | 15 +- drivers/net/wireless/ath/ath10k/core.c | 13 -- drivers/net/wireless/ath/ath10k/hw.h | 3 - drivers/net/wireless/ath/ath10k/snoc.c | 19 +- drivers/net/wireless/ath/ath10k/snoc.h | 1 + drivers/pci/quirks.c | 28 ++- include/net/dst_ops.h | 6 +- include/net/netns/ipv6.h | 4 +- mm/memory-failure.c | 6 +- mm/memory.c | 4 +- net/can/raw.c | 12 +- net/core/dst.c | 12 +- net/core/sock.c | 12 +- net/ipv4/ip_output.c | 2 +- net/ipv6/ip6_output.c | 2 +- net/ipv6/route.c | 25 +-- net/netfilter/nf_tables_api.c | 27 +++ net/nfc/llcp_core.c | 39 +++- net/sched/em_text.c | 4 +- net/socket.c | 2 + net/tls/tls_sw.c | 2 + sound/soc/meson/Kconfig | 4 + sound/soc/meson/Makefile | 2 + sound/soc/meson/g12a-tohdmitx.c | 227 +++++---------------- sound/soc/meson/meson-codec-glue.c | 149 ++++++++++++++ sound/soc/meson/meson-codec-glue.h | 32 +++ 41 files changed, 658 insertions(+), 412 deletions(-)
Greg Kroah-Hartman
maintainerPATCH 1/38 nfc: llcp_core: Hold a ref to llcp_local->dev when holding a ref to llcp_local
5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Siddh Raman Pant <[email protected]>
[ Upstream commit c95f919567d6f1914f13350af61a1b044ac85014 ]
llcp_sock_sendmsg() calls nfc_llcp_send_ui_frame() which in turn calls nfc_alloc_send_skb(), which accesses the nfc_dev from the llcp_sock for getting the headroom and tailroom needed for skb allocation.
Parallelly the nfc_dev can be freed, as the refcount is decreased via nfc_free_device(), leading to a UAF reported by Syzkaller, which can be summarized as follows:
(1) llcp_sock_sendmsg() -> nfc_llcp_send_ui_frame()
-> nfc_alloc_send_skb() -> Dereference *nfc_dev (2) virtual_ncidev_close() -> nci_free_device() -> nfc_free_device() -> put_device() -> nfc_release() -> Free *nfc_dev
When a reference to llcp_local is acquired, we do not acquire the same for the nfc_dev. This leads to freeing even when the llcp_local is in use, and this is the case with the UAF described above too.
Thus, when we acquire a reference to llcp_local, we should acquire a reference to nfc_dev, and release the references appropriately later.
References for llcp_local is initialized in nfc_llcp_register_device() (which is called by nfc_register_device()). Thus, we should acquire a reference to nfc_dev there.
nfc_unregister_device() calls nfc_llcp_unregister_device() which in turn calls nfc_llcp_local_put(). Thus, the reference to nfc_dev is appropriately released later.
--- net/nfc/llcp_core.c | 39 ++++++++++++++++++++++++++++++++++++--- 1 file changed, 36 insertions(+), 3 deletions(-)
net/nfc/llcp_core.c +36 -3
@@ -147,6 +147,13 @@ static void nfc_llcp_socket_release(struct nfc_llcp_local *local, bool device,static struct nfc_llcp_local *nfc_llcp_local_get (struct nfc_llcp_local *local) { +/* Since using nfc_llcp_local may result in usage of nfc_dev, whenever +* we hold a reference to local, we also need to hold a reference to +* the device to avoid UAF. +*/ +if (!nfc_get_device (local->dev->idx)) +return NULL ; +kref_get (&local->ref);return local;
@@ -179,10 +186,18 @@ static void local_release(struct kref *ref)int nfc_llcp_local_put (struct nfc_llcp_local *local) { +struct nfc_dev *dev; +int ret; +if (local ==NULL )return 0 ; -return kref_put (&local->ref, local_release); + dev = local->dev; + + ret =kref_put (&local->ref, local_release); +nfc_put_device (dev); + +return ret; } static struct nfc_llcp_sock *nfc_llcp_sock_get(struct nfc_llcp_local *local,
@@ -968,8 +983,17 @@ static void nfc_llcp_recv_connect(struct nfc_llcp_local *local, } new_sock =nfc_llcp_sock (new_sk); - new_sock->dev = local->dev; + new_sock->local =nfc_llcp_local_get (local); +if (!new_sock->local) { + reason =LLCP_DM_REJ ; +sock_put (&new_sock->sk); +release_sock (&sock->sk); +sock_put (&sock->sk); + goto fail; + } + + new_sock->dev = local->dev; new_sock->rw = sock->rw; new_sock->miux = sock->miux; new_sock->nfc_protocol = sock->nfc_protocol;
@@ -1607,7 +1631,16 @@ int nfc_llcp_register_device(struct nfc_dev *ndev)if (local ==NULL )return -ENOMEM ; - local->dev = ndev; +/* As we are going to initialize local's refcount, we need to get the +* nfc_dev to avoid UAF, otherwise there is no point in continuing. +* See nfc_llcp_local_get(). +*/ + local->dev =nfc_get_device (ndev->idx); +if (!local->dev) { +kfree (local); +return -ENODEV ; + } +INIT_LIST_HEAD (&local->list);kref_init (&local->ref);mutex_init (&local->sdp_lock);
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 2/38 i40e: Fix filter input checks to prevent config with invalid values
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Sudheer Mogilappagari <[email protected]>
[ Upstream commit 3e48041d9820c17e0a51599d12e66c6e12a8d08d ]
Prevent VF from configuring filters with unsupported actions or use REDIRECT action with invalid tc number. Current checks could cause out of bounds access on PF side.
--- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c +4 -4
@@ -3332,16 +3332,16 @@ static int i40e_validate_cloud_filter(struct i40e_vf *vf,bool found = false;int bkt; -if (!tc_filter->action) { +if (tc_filter->action !=VIRTCHNL_ACTION_TC_REDIRECT ) {dev_info (&pf->pdev->dev, -"VF %d: Currently ADq doesn't support Drop Action\n" , - vf->vf_id); +"VF %d: ADQ doesn't support this action (%d)\n" , + vf->vf_id, tc_filter->action); goto err; }/* action_meta is TC number here to which the filter is applied */ if (!tc_filter->action_meta || - tc_filter->action_meta >I40E_MAX_VF_VSI ) { + tc_filter->action_meta > vf->num_tc) {dev_info (&pf->pdev->dev,"VF %d: Invalid TC number %u\n" , vf->vf_id, tc_filter->action_meta); goto err;
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 3/38 net: sched: em_text: fix possible memory leak in em_text_destroy()
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hangyu Hua <[email protected]>
[ Upstream commit 8fcb0382af6f1ef50936f1be05b8149eb2f88496 ]
m->data needs to be freed when em_text_destroy is called.
--- net/sched/em_text.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
net/sched/em_text.c +3 -1
@@ -97,8 +97,10 @@ static int em_text_change(struct net *net, void *data, int len,static void em_text_destroy (struct tcf_ematch *m) { -if (EM_TEXT_PRIV (m) &&EM_TEXT_PRIV (m)->config) +if (EM_TEXT_PRIV (m) &&EM_TEXT_PRIV (m)->config) {textsearch_destroy (EM_TEXT_PRIV (m)->config); +kfree (EM_TEXT_PRIV (m)); + } }static int em_text_dump (struct sk_buff *skb,struct tcf_ematch *m)
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 4/38 net: Implement missing getsockopt(SO_TIMESTAMPING_NEW)
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jörn-Thorben Hinz <[email protected]>
[ Upstream commit 7f6ca95d16b96567ce4cf458a2790ff17fa620c3 ]
Commit 9718475e6908 ("socket: Add SO_TIMESTAMPING_NEW") added the new socket option SO_TIMESTAMPING_NEW. Setting the option is handled in sk_setsockopt(), querying it was not handled in sk_getsockopt(), though.
Following remarks on an earlier submission of this patch, keep the old behavior of getsockopt(SO_TIMESTAMPING_OLD) which returns the active flags even if they actually have been set through SO_TIMESTAMPING_NEW.
The new getsockopt(SO_TIMESTAMPING_NEW) is stricter, returning flags only if they have been set through the same option.
--- net/core/sock.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-)
net/core/sock.c +9 -2
@@ -1309,9 +1309,16 @@ int sock_getsockopt(struct socket *sock, int level, int optname,break ;case SO_LINGER : +case SO_TIMESTAMPING_NEW : lv =sizeof (v.ling); - v.ling.l_onoff =sock_flag (sk,SOCK_LINGER ); - v.ling.l_linger = sk->sk_lingertime /HZ ; +/* For the later-added case SO_TIMESTAMPING_NEW: Be strict about only +* returning the flags when they were set through the same option. +* Don't change the beviour for the old case SO_TIMESTAMPING_OLD. +*/ +if (optname ==SO_TIMESTAMPING_OLD ||sock_flag (sk,SOCK_TSTAMP_NEW )) { + v.ling.l_onoff =sock_flag (sk,SOCK_LINGER ); + v.ling.l_linger = sk->sk_lingertime /HZ ; + }break ;case SO_BSDCOMPAT :
Signature · 1 line
2.43.0
Re: [PATCH 5.4 04/38] net: Implement missing getsockopt(SO_TIMESTAMPING_NEW)
Hi Greg,
this patch is applied in the wrong place (the wrong case) here in sock_getsockopt(). The function seems to have changed in a number of places after 5.4, apparently too much for an automatic(?!) merge.
Quoted 64 lines · Greg Kroah-Hartman
5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jörn-Thorben Hinz <[email protected]>
[ Upstream commit
7f6ca95d16b96567ce4cf458a2790ff17fa620c3]Commit
9718475e6908("socket: AddSO_TIMESTAMPING_NEW") added the new socket optionSO_TIMESTAMPING_NEW. Setting the option is handled insk_setsockopt(), querying it was not handled insk_getsockopt(), though.Following remarks on an earlier submission of this patch, keep the old
behavior ofgetsockopt(SO_TIMESTAMPING_OLD)which returns the active flags even if they actually have been set throughSO_TIMESTAMPING_NEW.The new
getsockopt(SO_TIMESTAMPING_NEW)is stricter, returning flags only if they have been set through the same option.Fixes 9718475e6908 ("socket: Add SO_TIMESTAMPING_NEW")Link:
https://lore.kernel.org/lkml/[email protected]/
Link:
https://lore.kernel.org/netdev/[email protected]/Signed-off-by Jörn-Thorben Hinz <[email protected]>Reviewed-by Willem de Bruijn <[email protected]>Signed-off-by David S. Miller <[email protected]>Signed-off-by Sasha Levin <[email protected]>---
net/core/sock.c | 11 +++++++++--1 file changed, 9 insertions(+), 2 deletions(-)net/core/sock.c +0
-0@@ -1309,9 +1309,16 @@ int sock_getsockopt(struct socket *sock, int level, int optname, break;
caseSO_LINGER:
+ caseSO_TIMESTAMPING_NEW:
lv = sizeof(v.ling);
- v.ling.l_onoff =sock_flag(sk, SOCK_LINGER); - v.ling.l_linger =sk->sk_lingertime/ HZ; + /* For the later-added caseSO_TIMESTAMPING_NEW: Be strict about only
+ * returning the flags when they were set through the same option.
+ * Don't change the beviour for the old caseSO_TIMESTAMPING_OLD.
+ */
+ if (optname ==SO_TIMESTAMPING_OLD||sock_flag(sk, SOCK_TSTAMP_NEW)) {
+ v.ling.l_onoff =sock_flag(sk, SOCK_LINGER); + v.ling.l_linger =sk->sk_lingertime/ HZ; + }
break;
caseSO_BSDCOMPAT:
Greg Kroah-Hartman
maintainerPATCH 5/38 can: raw: add support for SO_TXTIME/SCM_TXTIME
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Marc Kleine-Budde <[email protected]>
[ Upstream commit 51a0d5e51178fcd147c1b8fdab2ed16b561326db ]
This patch calls into sock_cmsg_send() to parse the user supplied control information into a struct sockcm_cookie. Then assign the requested transmit time to the skb.
This makes it possible to use the Earliest TXTIME First (ETF) packet scheduler with the CAN_RAW protocol. The user can send a CAN_RAW frame with a TXTIME and the kernel (with the ETF scheduler) will take care of sending it to the network interface.
Stable-dep-of: 7f6ca95d16b9 ("net: Implement missing getsockopt(SO_TIMESTAMPING_NEW)")
--- net/can/raw.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)
net/can/raw.c +10 -1
@@ -770,6 +770,7 @@ static int raw_sendmsg(struct socket *sock, struct msghdr *msg, size_t size) {struct sock *sk = sock->sk;struct raw_sock *ro =raw_sk (sk); +struct sockcm_cookie sockc;struct sk_buff *skb;struct net_device *dev;int ifindex;
@@ -815,11 +816,19 @@ static int raw_sendmsg(struct socket *sock, struct msghdr *msg, size_t size)if (err <0 ) goto free_skb; -skb_setup_tx_timestamp (skb, sk->sk_tsflags); +sockcm_init (&sockc, sk); +if (msg->msg_controllen) { + err =sock_cmsg_send (sk, msg, &sockc); +if (unlikely (err)) + goto free_skb; + } skb->dev = dev; skb->sk = sk; skb->priority = sk->sk_priority; + skb->tstamp = sockc.transmit_time; + +skb_setup_tx_timestamp (skb, sockc.tsflags); err =can_send (skb, ro->loopback);
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 6/38 can: raw: add support for SO_MARK
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Marc Kleine-Budde <[email protected]>
[ Upstream commit 0826e82b8a32e646b7b32ba8b68ba30812028e47 ]
Add support for SO_MARK to the CAN_RAW protocol. This makes it possible to add traffic control filters based on the fwmark.
Stable-dep-of: 7f6ca95d16b9 ("net: Implement missing getsockopt(SO_TIMESTAMPING_NEW)")
--- net/can/raw.c | 1 + 1 file changed, 1 insertion(+)
net/can/raw.c +1 -0
@@ -826,6 +826,7 @@ static int raw_sendmsg(struct socket *sock, struct msghdr *msg, size_t size) skb->dev = dev; skb->sk = sk; skb->priority = sk->sk_priority; + skb->mark = sk->sk_mark; skb->tstamp = sockc.transmit_time;skb_setup_tx_timestamp (skb, sockc.tsflags);
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 7/38 net-timestamp: extend SOF_TIMESTAMPING_OPT_ID to HW timestamps
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Vadim Fedorenko <[email protected]>
[ Upstream commit 8ca5a5790b9a1ce147484d2a2c4e66d2553f3d6c ]
When the feature was added it was enabled for SW timestamps only but with current hardware the same out-of-order timestamps can be seen. Let's expand the area for the feature to all types of timestamps.
Stable-dep-of: 7f6ca95d16b9 ("net: Implement missing getsockopt(SO_TIMESTAMPING_NEW)")
--- net/ipv4/ip_output.c | 2 +- net/ipv6/ip6_output.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
net/ipv4/ip_output.c +1 -1
@@ -993,7 +993,7 @@ static int __ip_append_data(struct sock *sk, mtu = cork->gso_size ?IP_MAX_MTU : cork->fragsize; paged = !!cork->gso_size; -if (cork->tx_flags &SKBTX_ANY_SW_TSTAMP && +if (cork->tx_flags &SKBTX_ANY_TSTAMP && sk->sk_tsflags &SOF_TIMESTAMPING_OPT_ID ) tskey = sk->sk_tskey++;
net/ipv6/ip6_output.c +1 -1
@@ -1425,7 +1425,7 @@ static int __ip6_append_data(struct sock *sk, mtu = cork->gso_size ?IP6_MAX_MTU : cork->fragsize; orig_mtu = mtu; -if (cork->tx_flags &SKBTX_ANY_SW_TSTAMP && +if (cork->tx_flags &SKBTX_ANY_TSTAMP && sk->sk_tsflags &SOF_TIMESTAMPING_OPT_ID ) tskey = sk->sk_tskey++;
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 8/38 ARM: sun9i: smp: Fix array-index-out-of-bounds read in sunxi_mc_smp_init
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Stefan Wahren <[email protected]>
[ Upstream commit 72ad3b772b6d393701df58ba1359b0bb346a19ed ]
Running a multi-arch kernel (multi_v7_defconfig) on a Raspberry Pi 3B+ with enabled CONFIG_UBSAN triggers the following warning:
UBSAN: array-index-out-of-bounds in arch/arm/mach-sunxi/mc_smp.c:810:29 index 2 is out of range for type 'sunxi_mc_smp_data [2]' CPU: 0 PID: 1 Comm: swapper/0 Not tainted 6.7.0-rc6-00248-g5254c0cbc92d Hardware name: BCM2835 unwind_backtrace from show_stack+0x10/0x14 show_stack from dump_stack_lvl+0x40/0x4c dump_stack_lvl from ubsan_epilogue+0x8/0x34 ubsan_epilogue from __ubsan_handle_out_of_bounds+0x78/0x80 __ubsan_handle_out_of_bounds from sunxi_mc_smp_init+0xe4/0x4cc sunxi_mc_smp_init from do_one_initcall+0xa0/0x2fc do_one_initcall from kernel_init_freeable+0xf4/0x2f4 kernel_init_freeable from kernel_init+0x18/0x158 kernel_init from ret_from_fork+0x14/0x28
Since the enabled method couldn't match with any entry from sunxi_mc_smp_data, the value of the index shouldn't be used right after the loop. So move it after the check of ret in order to have a valid index.
--- arch/arm/mach-sunxi/mc_smp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
arch/arm/mach-sunxi/mc_smp.c +2 -2
@@ -808,12 +808,12 @@ static int __init sunxi_mc_smp_init(void)break ; } - is_a83t = sunxi_mc_smp_data[i].is_a83t; -of_node_put (node);if (ret)return -ENODEV ; + is_a83t = sunxi_mc_smp_data[i].is_a83t; +if (!sunxi_mc_smp_cpu_table_init ())return -EINVAL ;
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 9/38 net: bcmgenet: Fix FCS generation for fragmented skbuffs
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Adrian Cinal <[email protected]>
[ Upstream commit e584f2ff1e6cc9b1d99e8a6b0f3415940d1b3eb3 ]
The flag DMA_TX_APPEND_CRC was only written to the first DMA descriptor in the TX path, where each descriptor corresponds to a single skbuff fragment (or the skbuff head). This led to packets with no FCS appearing on the wire if the kernel allocated the packet in fragments, which would always happen when using PACKET_MMAP/TPACKET (cf. tpacket_fill_skb() in net/af_packet.c).
--- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/net/ethernet/broadcom/genet/bcmgenet.c +3 -1
@@ -1648,8 +1648,10 @@ static netdev_tx_t bcmgenet_xmit(struct sk_buff *skb, struct net_device *dev)/* Note: if we ever change from DMA_TX_APPEND_CRC below we * will need to restore software padding of "runt" packets */ + len_stat |=DMA_TX_APPEND_CRC ; +if (!i) { - len_stat |=DMA_TX_APPEND_CRC |DMA_SOP ; + len_stat |=DMA_SOP ;if (skb->ip_summed ==CHECKSUM_PARTIAL ) len_stat |=DMA_TX_DO_CSUM ; }
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 10/38 net: Save and restore msg_namelen in sock_sendmsg
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Marc Dionne <[email protected]>
[ Upstream commit 01b2885d9415152bcb12ff1f7788f500a74ea0ed ]
Commit 86a7e0b69bd5 ("net: prevent rewrite of msg_name in sock_sendmsg()") made sock_sendmsg save the incoming msg_name pointer and restore it before returning, to insulate the caller against msg_name being changed by the called code. If the address length was also changed however, we may return with an inconsistent structure where the length doesn't match the address, and attempts to reuse it may lead to lost packets.
For example, a kernel that doesn't have commit 1c5950fc6fe9 ("udp6: fix potential access to stale information") will replace a v4 mapped address with its ipv4 equivalent, and shorten namelen accordingly from 28 to 16. If the caller attempts to reuse the resulting msg structure, it will have the original ipv6 (v4 mapped) address but an incorrect v4 length.
--- net/socket.c | 2 ++ 1 file changed, 2 insertions(+)
net/socket.c +2 -0
@@ -661,6 +661,7 @@ int sock_sendmsg(struct socket *sock, struct msghdr *msg) {struct sockaddr_storage *save_addr = (struct sockaddr_storage *)msg->msg_name;struct sockaddr_storage address; +int save_len = msg->msg_namelen;int ret;if (msg->msg_name) {
@@ -670,6 +671,7 @@ int sock_sendmsg(struct socket *sock, struct msghdr *msg) ret =__sock_sendmsg (sock, msg); msg->msg_name = save_addr; + msg->msg_namelen = save_len;return ret; }
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 11/38 i40e: fix use-after-free in i40e_aqc_add_filters()
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Ke Xiao <[email protected]>
[ Upstream commit 6a15584e99db8918b60e507539c7446375dcf366 ]
Commit 3116f59c12bd ("i40e: fix use-after-free in i40e_sync_filters_subtask()") avoided use-after-free issues, by increasing refcount during update the VSI filter list to the HW. However, it missed the unicast situation.
When deleting an unicast FDB entry, the i40e driver will release the mac_filter, and i40e_service_task will concurrently request firmware to add the mac_filter, which will lead to the following use-after-free issue.
Fix again for both netdev->uc and netdev->mc.
BUG: KASAN: use-after-free in i40e_aqc_add_filters+0x55c/0x5b0 [i40e] Read of size 2 at addr ffff888eb3452d60 by task kworker/8:7/6379 CPU: 8 PID: 6379 Comm: kworker/8:7 Kdump: loaded Tainted: G Workqueue: i40e i40e_service_task [i40e] Call Trace: dump_stack+0x71/0xab print_address_description+0x6b/0x290 kasan_report+0x14a/0x2b0 i40e_aqc_add_filters+0x55c/0x5b0 [i40e] i40e_sync_vsi_filters+0x1676/0x39c0 [i40e] i40e_service_task+0x1397/0x2bb0 [i40e] process_one_work+0x56a/0x11f0 worker_thread+0x8f/0xf40 kthread+0x2a0/0x390 ret_from_fork+0x1f/0x40 Allocated by task 21948: kasan_kmalloc+0xa6/0xd0 kmem_cache_alloc_trace+0xdb/0x1c0 i40e_add_filter+0x11e/0x520 [i40e] i40e_addr_sync+0x37/0x60 [i40e] __hw_addr_sync_dev+0x1f5/0x2f0 i40e_set_rx_mode+0x61/0x1e0 [i40e] dev_uc_add_excl+0x137/0x190 i40e_ndo_fdb_add+0x161/0x260 [i40e] rtnl_fdb_add+0x567/0x950 rtnetlink_rcv_msg+0x5db/0x880 netlink_rcv_skb+0x254/0x380 netlink_unicast+0x454/0x610 netlink_sendmsg+0x747/0xb00 sock_sendmsg+0xe2/0x120 __sys_sendto+0x1ae/0x290 __x64_sys_sendto+0xdd/0x1b0 do_syscall_64+0xa0/0x370 entry_SYSCALL_64_after_hwframe+0x65/0xca Freed by task 21948: __kasan_slab_free+0x137/0x190 kfree+0x8b/0x1b0 __i40e_del_filter+0x116/0x1e0 [i40e] i40e_del_mac_filter+0x16c/0x300 [i40e] i40e_addr_unsync+0x134/0x1b0 [i40e] __hw_addr_sync_dev+0xff/0x2f0 i40e_set_rx_mode+0x61/0x1e0 [i40e] dev_uc_del+0x77/0x90 rtnl_fdb_del+0x6a5/0x860 rtnetlink_rcv_msg+0x5db/0x880 netlink_rcv_skb+0x254/0x380 netlink_unicast+0x454/0x610 netlink_sendmsg+0x747/0xb00 sock_sendmsg+0xe2/0x120 __sys_sendto+0x1ae/0x290 __x64_sys_sendto+0xdd/0x1b0 do_syscall_64+0xa0/0x370 entry_SYSCALL_64_after_hwframe+0x65/0xca
--- drivers/net/ethernet/intel/i40e/i40e_main.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
drivers/net/ethernet/intel/i40e/i40e_main.c +7 -1
@@ -110,12 +110,18 @@ static struct workqueue_struct *i40e_wq;static void netdev_hw_addr_refcnt (struct i40e_mac_filter *f,struct net_device *netdev,int delta) { +struct netdev_hw_addr_list *ha_list;struct netdev_hw_addr *ha;if (!f || !netdev)return ; -netdev_for_each_mc_addr (ha, netdev) { +if (is_unicast_ether_addr (f->macaddr) ||is_link_local_ether_addr (f->macaddr)) + ha_list = &netdev->uc; +else + ha_list = &netdev->mc; + +netdev_hw_addr_list_for_each (ha, ha_list) {if (ether_addr_equal (ha->addr, f->macaddr)) { ha->refcount += delta;if (ha->refcount <=0 )
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 12/38 ASoC: meson: g12a: extract codec-to-codec utils
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jerome Brunet <[email protected]>
[ Upstream commit 9c29fd9bdf92900dc0cc5c2d8f58951a7bdc0f41 ]
The hdmi routing mechanism used on g12a hdmi is also used:
* other Amlogic SoC types
* for the internal DAC path
Each of these codec glues are slightly different but the idea behind it remains the same. This change extract some helper functions from the g12a-tohdmitx driver to make them available for other Amlogic codecs.
Stable-dep-of: 1e001206804b ("ASoC: meson: g12a-tohdmitx: Validate written enum values")
--- sound/soc/meson/Kconfig | 4 + sound/soc/meson/Makefile | 2 + sound/soc/meson/g12a-tohdmitx.c | 219 ++++++----------------------- sound/soc/meson/meson-codec-glue.c | 149 ++++++++++++++++++++ sound/soc/meson/meson-codec-glue.h | 32 +++++ 5 files changed, 230 insertions(+), 176 deletions(-) create mode 100644 sound/soc/meson/meson-codec-glue.c create mode 100644 sound/soc/meson/meson-codec-glue.h
sound/soc/meson/Kconfig +4 -0
@@ -85,9 +85,13 @@ config SND_MESON_AXG_PDM Select Y or M to add support for PDM input embedded in the Amlogic AXG SoC family +config SND_MESON_CODEC_GLUE + tristate + config SND_MESON_G12A_TOHDMITX tristate "Amlogic G12A To HDMI TX Control Support" select REGMAP_MMIO + select SND_MESON_CODEC_GLUE imply SND_SOC_HDMI_CODEC help Select Y or M to add support for HDMI audio on the g12a SoC
sound/soc/meson/Makefile +2 -0
@@ -11,6 +11,7 @@ snd-soc-meson-axg-sound-card-objs := axg-card.o snd-soc-meson-axg-spdifin-objs := axg-spdifin.o snd-soc-meson-axg-spdifout-objs := axg-spdifout.o snd-soc-meson-axg-pdm-objs := axg-pdm.o +snd-soc-meson-codec-glue-objs := meson-codec-glue.o snd-soc-meson-g12a-tohdmitx-objs := g12a-tohdmitx.o obj-$(CONFIG_SND_MESON_AXG_FIFO) += snd-soc-meson-axg-fifo.o
@@ -24,4 +25,5 @@ obj-$(CONFIG_SND_MESON_AXG_SOUND_CARD) += snd-soc-meson-axg-sound-card.o obj-$(CONFIG_SND_MESON_AXG_SPDIFIN) += snd-soc-meson-axg-spdifin.o obj-$(CONFIG_SND_MESON_AXG_SPDIFOUT) += snd-soc-meson-axg-spdifout.o obj-$(CONFIG_SND_MESON_AXG_PDM) += snd-soc-meson-axg-pdm.o +obj-$(CONFIG_SND_MESON_CODEC_GLUE) += snd-soc-meson-codec-glue.o obj-$(CONFIG_SND_MESON_G12A_TOHDMITX) += snd-soc-meson-g12a-tohdmitx.o
sound/soc/meson/g12a-tohdmitx.c +43 -176
298 lines not on this page · raw message
sound/soc/meson/meson-codec-glue.c new file +149 -0
149 lines not on this page · raw message
sound/soc/meson/meson-codec-glue.h new file +32 -0
@@ -0,0 +1,32 @@ +/* SPDX-License-Identifier: GPL-2.0 +* +* Copyright (c) 2018 Baylibre SAS. +* Author: Jerome Brunet <[email protected]> +*/ + +#ifndef _MESON_CODEC_GLUE_H +#define _MESON_CODEC_GLUE_H + +#include <sound/soc.h> + +struct meson_codec_glue_input { +struct snd_soc_pcm_stream params; +unsigned int fmt; +}; + +/* Input helpers */ +struct meson_codec_glue_input * +meson_codec_glue_input_get_data (struct snd_soc_dai *dai); +int meson_codec_glue_input_hw_params (struct snd_pcm_substream *substream, +struct snd_pcm_hw_params *params, +struct snd_soc_dai *dai); +int meson_codec_glue_input_set_fmt (struct snd_soc_dai *dai, +unsigned int fmt); +int meson_codec_glue_input_dai_probe (struct snd_soc_dai *dai); +int meson_codec_glue_input_dai_remove (struct snd_soc_dai *dai); + +/* Output helpers */ +int meson_codec_glue_output_startup (struct snd_pcm_substream *substream, +struct snd_soc_dai *dai); + +#endif /* _MESON_CODEC_GLUE_H */
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 13/38 ASoC: meson: g12a-tohdmitx: Validate written enum values
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mark Brown <[email protected]>
[ Upstream commit 1e001206804be3f3d21f4a1cf16e5d059d75643f ]
When writing to an enum we need to verify that the value written is valid for the enumeration, the helper function snd_soc_item_enum_to_val() doesn't do it since it needs to return an unsigned (and in any case we'd need to check the return value).
--- sound/soc/meson/g12a-tohdmitx.c | 6 ++++++ 1 file changed, 6 insertions(+)
sound/soc/meson/g12a-tohdmitx.c +6 -0
@@ -44,6 +44,9 @@ static int g12a_tohdmitx_i2s_mux_put_enum(struct snd_kcontrol *kcontrol,struct soc_enum *e = (struct soc_enum *)kcontrol->private_value;unsigned int mux, changed; +if (ucontrol->value.enumerated.item[0 ] >= e->items) +return -EINVAL ; + mux =snd_soc_enum_item_to_val (e, ucontrol->value.enumerated.item[0 ]); changed = snd_soc_component_test_bits(component, e->reg, CTRL0_I2S_DAT_SEL,
@@ -92,6 +95,9 @@ static int g12a_tohdmitx_spdif_mux_put_enum(struct snd_kcontrol *kcontrol,struct soc_enum *e = (struct soc_enum *)kcontrol->private_value;unsigned int mux, changed; +if (ucontrol->value.enumerated.item[0 ] >= e->items) +return -EINVAL ; + mux =snd_soc_enum_item_to_val (e, ucontrol->value.enumerated.item[0 ]); changed = snd_soc_component_test_bits(component, TOHDMITX_CTRL0, CTRL0_SPDIF_SEL,
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 14/38 ASoC: meson: g12a-tohdmitx: Fix event generation for S/PDIF mux
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mark Brown <[email protected]>
[ Upstream commit b036d8ef3120b996751495ce25994eea58032a98 ]
When a control changes value the return value from _put() should be 1 so we get events generated to userspace notifying applications of the change. While the I2S mux gets this right the S/PDIF mux does not, fix the return value.
--- sound/soc/meson/g12a-tohdmitx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
sound/soc/meson/g12a-tohdmitx.c +1 -1
@@ -117,7 +117,7 @@ static int g12a_tohdmitx_spdif_mux_put_enum(struct snd_kcontrol *kcontrol,snd_soc_dapm_mux_update_power (dapm, kcontrol, mux, e,NULL ); -return 0 ; +return 1 ; } static SOC_ENUM_SINGLE_DECL(g12a_tohdmitx_spdif_mux_enum, TOHDMITX_CTRL0,
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 15/38 i40e: Restore VF MSI-X state during PCI reset
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Andrii Staikov <[email protected]>
[ Upstream commit 371e576ff3e8580d91d49026e5d5faebf5565558 ]
During a PCI FLR the MSI-X Enable flag in the VF PCI MSI-X capability register will be cleared. This can lead to issues when a VF is assigned to a VM because in these cases the VF driver receives no indication of the PF PCI error/reset and additionally it is incapable of restoring the cleared flag in the hypervisor configuration space without fully reinitializing the driver interrupt functionality.
Since the VF driver is unable to easily resolve this condition on its own, restore the VF MSI-X flag during the PF PCI reset handling.
--- drivers/net/ethernet/intel/i40e/i40e_main.c | 3 +++ .../ethernet/intel/i40e/i40e_virtchnl_pf.c | 26 +++++++++++++++++++ .../ethernet/intel/i40e/i40e_virtchnl_pf.h | 3 +++ 3 files changed, 32 insertions(+)
drivers/net/ethernet/intel/i40e/i40e_main.c +3 -0
@@ -15716,6 +15716,9 @@ static void i40e_pci_error_reset_done(struct pci_dev *pdev)struct i40e_pf *pf =pci_get_drvdata (pdev);i40e_reset_and_rebuild (pf, false, false); +#ifdef CONFIG_PCI_IOV +i40e_restore_all_vfs_msi_state (pdev); +#endif /* CONFIG_PCI_IOV */ }/**
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c +26 -0
@@ -99,6 +99,32 @@ void i40e_vc_notify_reset(struct i40e_pf *pf) (u8 *)&pfe, sizeof(struct virtchnl_pf_event)); } +#ifdef CONFIG_PCI_IOV +void i40e_restore_all_vfs_msi_state (struct pci_dev *pdev) +{ +u16 vf_id; +u16 pos; + +/* Continue only if this is a PF */ +if (!pdev->is_physfn) +return ; + +if (!pci_num_vf (pdev)) +return ; + + pos =pci_find_ext_capability (pdev,PCI_EXT_CAP_ID_SRIOV ); +if (pos) { +struct pci_dev *vf_dev =NULL ; + +pci_read_config_word (pdev, pos +PCI_SRIOV_VF_DID , &vf_id); +while ((vf_dev =pci_get_device (pdev->vendor, vf_id, vf_dev))) { +if (vf_dev->is_virtfn && vf_dev->physfn == pdev) +pci_restore_msi_state (vf_dev); + } + } +} +#endif /* CONFIG_PCI_IOV */ +/** * i40e_vc_notify_vf_reset * @vf: pointer to the VF structure
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h +3 -0
@@ -141,5 +141,8 @@ int i40e_ndo_set_vf_spoofchk(struct net_device *netdev, int vf_id, bool enable);void i40e_vc_notify_link_state (struct i40e_pf *pf);void i40e_vc_notify_reset (struct i40e_pf *pf); +#ifdef CONFIG_PCI_IOV +void i40e_restore_all_vfs_msi_state (struct pci_dev *pdev); +#endif /* CONFIG_PCI_IOV */ #endif /* _I40E_VIRTCHNL_PF_H_ */
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 16/38 net/qla3xxx: switch from pci_ to dma_ API
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Christophe JAILLET <[email protected]>
[ Upstream commit 41fb4c1ba7478fe34c7e094e124e4ee4513b9763 ]
The wrappers in include/linux/pci-dma-compat.h should go away.
The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag.
It has been compile tested.
When memory is allocated in 'ql_alloc_net_req_rsp_queues()' GFP_KERNEL can be used because it is only called from 'ql_alloc_mem_resources()' which already calls 'ql_alloc_buffer_queues()' which uses GFP_KERNEL. (see below)
When memory is allocated in 'ql_alloc_buffer_queues()' GFP_KERNEL can be used because this flag is already used just a few line above.
When memory is allocated in 'ql_alloc_small_buffers()' GFP_KERNEL can be used because it is only called from 'ql_alloc_mem_resources()' which already calls 'ql_alloc_buffer_queues()' which uses GFP_KERNEL. (see above)
When memory is allocated in 'ql_alloc_mem_resources()' GFP_KERNEL can be used because this function already calls 'ql_alloc_buffer_queues()' which uses GFP_KERNEL. (see above)
While at it, use 'dma_set_mask_and_coherent()' instead of 'dma_set_mask()/ dma_set_coherent_mask()' in order to slightly simplify code.
@@
@@
- PCI_DMA_BIDIRECTIONAL
+ DMA_BIDIRECTIONAL
@@
@@
- PCI_DMA_TODEVICE
+ DMA_TO_DEVICE
@@
@@
- PCI_DMA_FROMDEVICE
+ DMA_FROM_DEVICE
@@
@@
- PCI_DMA_NONE
+ DMA_NONE
@@
expression e1, e2, e3;
@@
- pci_alloc_consistent(e1, e2, e3)
+ dma_alloc_coherent(&e1->dev, e2, e3, GFP_)
@@
expression e1, e2, e3;
@@
- pci_zalloc_consistent(e1, e2, e3)
+ dma_alloc_coherent(&e1->dev, e2, e3, GFP_)
@@
expression e1, e2, e3, e4;
@@
- pci_free_consistent(e1, e2, e3, e4)
+ dma_free_coherent(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_map_single(e1, e2, e3, e4)
+ dma_map_single(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_unmap_single(e1, e2, e3, e4)
+ dma_unmap_single(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4, e5;
@@
- pci_map_page(e1, e2, e3, e4, e5)
+ dma_map_page(&e1->dev, e2, e3, e4, e5)
@@
expression e1, e2, e3, e4;
@@
- pci_unmap_page(e1, e2, e3, e4)
+ dma_unmap_page(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_map_sg(e1, e2, e3, e4)
+ dma_map_sg(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_unmap_sg(e1, e2, e3, e4)
+ dma_unmap_sg(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_dma_sync_single_for_cpu(e1, e2, e3, e4)
+ dma_sync_single_for_cpu(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_dma_sync_single_for_device(e1, e2, e3, e4)
+ dma_sync_single_for_device(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_dma_sync_sg_for_cpu(e1, e2, e3, e4)
+ dma_sync_sg_for_cpu(&e1->dev, e2, e3, e4)
@@
expression e1, e2, e3, e4;
@@
- pci_dma_sync_sg_for_device(e1, e2, e3, e4)
+ dma_sync_sg_for_device(&e1->dev, e2, e3, e4)
@@
expression e1, e2;
@@
- pci_dma_mapping_error(e1, e2)
+ dma_mapping_error(&e1->dev, e2)
@@
expression e1, e2;
@@
- pci_set_dma_mask(e1, e2)
+ dma_set_mask(&e1->dev, e2)
@@
expression e1, e2;
@@
- pci_set_consistent_dma_mask(e1, e2)
+ dma_set_coherent_mask(&e1->dev, e2)
Stable-dep-of: 89f45c30172c ("net/qla3xxx: fix potential memleak in ql_alloc_buffer_queues")
--- drivers/net/ethernet/qlogic/qla3xxx.c | 196 ++++++++++++-------------- 1 file changed, 87 insertions(+), 109 deletions(-)
drivers/net/ethernet/qlogic/qla3xxx.c +87 -109
@@ -316,12 +316,11 @@ static void ql_release_to_lrg_buf_free_list(struct ql3_adapter *qdev,* buffer */ skb_reserve (lrg_buf_cb->skb,QL_HEADER_SPACE ); - map =pci_map_single (qdev->pdev, + map =dma_map_single (&qdev->pdev->dev, lrg_buf_cb->skb->data, - qdev->lrg_buffer_len - -QL_HEADER_SPACE , -PCI_DMA_FROMDEVICE ); - err =pci_dma_mapping_error (qdev->pdev, map); + qdev->lrg_buffer_len -QL_HEADER_SPACE , +DMA_FROM_DEVICE ); + err =dma_mapping_error (&qdev->pdev->dev, map);if (err) { netdev_err(qdev->ndev, "PCI mapping failed with error: %d\n",
@@ -1803,13 +1802,12 @@ static int ql_populate_free_queue(struct ql3_adapter *qdev)* first buffer */ skb_reserve (lrg_buf_cb->skb,QL_HEADER_SPACE ); - map =pci_map_single (qdev->pdev, + map =dma_map_single (&qdev->pdev->dev, lrg_buf_cb->skb->data, - qdev->lrg_buffer_len - -QL_HEADER_SPACE , -PCI_DMA_FROMDEVICE ); + qdev->lrg_buffer_len -QL_HEADER_SPACE , +DMA_FROM_DEVICE ); - err =pci_dma_mapping_error (qdev->pdev, map); + err =dma_mapping_error (&qdev->pdev->dev, map);if (err) { netdev_err(qdev->ndev, "PCI mapping failed with error: %d\n",
@@ -1944,18 +1942,16 @@ static void ql_process_mac_tx_intr(struct ql3_adapter *qdev, goto invalid_seg_count; } -pci_unmap_single (qdev->pdev, +dma_unmap_single (&qdev->pdev->dev,dma_unmap_addr (&tx_cb->map[0 ], mapaddr), -dma_unmap_len (&tx_cb->map[0 ], maplen), -PCI_DMA_TODEVICE ); +dma_unmap_len (&tx_cb->map[0 ], maplen),DMA_TO_DEVICE ); tx_cb->seg_count--;if (tx_cb->seg_count) {for (i =1 ; i < tx_cb->seg_count; i++) { -pci_unmap_page (qdev->pdev, -dma_unmap_addr (&tx_cb->map[i], - mapaddr), +dma_unmap_page (&qdev->pdev->dev, +dma_unmap_addr (&tx_cb->map[i], mapaddr),dma_unmap_len (&tx_cb->map[i], maplen), -PCI_DMA_TODEVICE ); +DMA_TO_DEVICE ); } } qdev->ndev->stats.tx_packets++;
@@ -2022,10 +2018,9 @@ static void ql_process_mac_rx_intr(struct ql3_adapter *qdev, qdev->ndev->stats.rx_bytes += length;skb_put (skb, length); -pci_unmap_single (qdev->pdev, +dma_unmap_single (&qdev->pdev->dev,dma_unmap_addr (lrg_buf_cb2, mapaddr), -dma_unmap_len (lrg_buf_cb2, maplen), -PCI_DMA_FROMDEVICE ); +dma_unmap_len (lrg_buf_cb2, maplen),DMA_FROM_DEVICE );prefetch (skb->data);skb_checksum_none_assert (skb); skb->protocol =eth_type_trans (skb, qdev->ndev);
@@ -2068,10 +2063,9 @@ static void ql_process_macip_rx_intr(struct ql3_adapter *qdev, skb2 = lrg_buf_cb2->skb;skb_put (skb2, length);/* Just the second buffer length here. */ -pci_unmap_single (qdev->pdev, +dma_unmap_single (&qdev->pdev->dev,dma_unmap_addr (lrg_buf_cb2, mapaddr), -dma_unmap_len (lrg_buf_cb2, maplen), -PCI_DMA_FROMDEVICE ); +dma_unmap_len (lrg_buf_cb2, maplen),DMA_FROM_DEVICE );prefetch (skb2->data);skb_checksum_none_assert (skb2);
@@ -2320,9 +2314,9 @@ static int ql_send_map(struct ql3_adapter *qdev,/* * Map the skb buffer first. */ - map =pci_map_single (qdev->pdev, skb->data, len,PCI_DMA_TODEVICE ); + map =dma_map_single (&qdev->pdev->dev, skb->data, len,DMA_TO_DEVICE ); - err =pci_dma_mapping_error (qdev->pdev, map); + err =dma_mapping_error (&qdev->pdev->dev, map);if (err) {netdev_err (qdev->ndev,"PCI mapping failed with error: %d\n" , err);
@@ -2358,11 +2352,11 @@ static int ql_send_map(struct ql3_adapter *qdev, (seg == 7 && seg_cnt > 8) || (seg == 12 && seg_cnt > 13) || (seg == 17 && seg_cnt > 18)) { - map =pci_map_single (qdev->pdev, oal, + map =dma_map_single (&qdev->pdev->dev, oal,sizeof (struct oal ), -PCI_DMA_TODEVICE ); +DMA_TO_DEVICE ); - err =pci_dma_mapping_error (qdev->pdev, map); + err =dma_mapping_error (&qdev->pdev->dev, map);if (err) { netdev_err(qdev->ndev, "PCI mapping outbound address list with error: %d\n",
@@ -2424,24 +2418,24 @@ static int ql_send_map(struct ql3_adapter *qdev, (seg == 7 && seg_cnt > 8) || (seg == 12 && seg_cnt > 13) || (seg == 17 && seg_cnt > 18)) { -pci_unmap_single (qdev->pdev, -dma_unmap_addr (&tx_cb->map[seg], mapaddr), -dma_unmap_len (&tx_cb->map[seg], maplen), -PCI_DMA_TODEVICE ); +dma_unmap_single (&qdev->pdev->dev, +dma_unmap_addr (&tx_cb->map[seg], mapaddr), +dma_unmap_len (&tx_cb->map[seg], maplen), +DMA_TO_DEVICE ); oal++; seg++; } -pci_unmap_page (qdev->pdev, +dma_unmap_page (&qdev->pdev->dev,dma_unmap_addr (&tx_cb->map[seg], mapaddr),dma_unmap_len (&tx_cb->map[seg], maplen), -PCI_DMA_TODEVICE ); +DMA_TO_DEVICE ); } -pci_unmap_single (qdev->pdev, +dma_unmap_single (&qdev->pdev->dev,dma_unmap_addr (&tx_cb->map[0 ], mapaddr),dma_unmap_addr (&tx_cb->map[0 ], maplen), -PCI_DMA_TODEVICE ); +DMA_TO_DEVICE );return NETDEV_TX_BUSY ;
@@ -2527,9 +2521,8 @@ static int ql_alloc_net_req_rsp_queues(struct ql3_adapter *qdev)wmb (); qdev->req_q_virt_addr = -pci_alloc_consistent (qdev->pdev, - (size_t ) qdev->req_q_size, - &qdev->req_q_phy_addr); +dma_alloc_coherent (&qdev->pdev->dev, (size_t )qdev->req_q_size, + &qdev->req_q_phy_addr,GFP_KERNEL );if ((qdev->req_q_virt_addr ==NULL ) ||LS_64BITS (qdev->req_q_phy_addr) & (qdev->req_q_size -1 )) {
@@ -2538,16 +2531,14 @@ static int ql_alloc_net_req_rsp_queues(struct ql3_adapter *qdev) } qdev->rsp_q_virt_addr = -pci_alloc_consistent (qdev->pdev, - (size_t ) qdev->rsp_q_size, - &qdev->rsp_q_phy_addr); +dma_alloc_coherent (&qdev->pdev->dev, (size_t )qdev->rsp_q_size, + &qdev->rsp_q_phy_addr,GFP_KERNEL );if ((qdev->rsp_q_virt_addr ==NULL ) ||LS_64BITS (qdev->rsp_q_phy_addr) & (qdev->rsp_q_size -1 )) {netdev_err (qdev->ndev,"rspQ allocation failed\n" ); -pci_free_consistent (qdev->pdev, (size_t ) qdev->req_q_size, - qdev->req_q_virt_addr, - qdev->req_q_phy_addr); +dma_free_coherent (&qdev->pdev->dev, (size_t )qdev->req_q_size, + qdev->req_q_virt_addr, qdev->req_q_phy_addr);return -ENOMEM ; }
@@ -2563,15 +2554,13 @@ static void ql_free_net_req_rsp_queues(struct ql3_adapter *qdev)return ; } -pci_free_consistent (qdev->pdev, - qdev->req_q_size, - qdev->req_q_virt_addr, qdev->req_q_phy_addr); +dma_free_coherent (&qdev->pdev->dev, qdev->req_q_size, + qdev->req_q_virt_addr, qdev->req_q_phy_addr); qdev->req_q_virt_addr =NULL ; -pci_free_consistent (qdev->pdev, - qdev->rsp_q_size, - qdev->rsp_q_virt_addr, qdev->rsp_q_phy_addr); +dma_free_coherent (&qdev->pdev->dev, qdev->rsp_q_size, + qdev->rsp_q_virt_addr, qdev->rsp_q_phy_addr); qdev->rsp_q_virt_addr =NULL ;
@@ -2595,9 +2584,9 @@ static int ql_alloc_buffer_queues(struct ql3_adapter *qdev)return -ENOMEM ; qdev->lrg_buf_q_alloc_virt_addr = -pci_alloc_consistent (qdev->pdev, - qdev->lrg_buf_q_alloc_size, - &qdev->lrg_buf_q_alloc_phy_addr); +dma_alloc_coherent (&qdev->pdev->dev, + qdev->lrg_buf_q_alloc_size, + &qdev->lrg_buf_q_alloc_phy_addr,GFP_KERNEL );if (qdev->lrg_buf_q_alloc_virt_addr ==NULL ) {netdev_err (qdev->ndev,"lBufQ failed\n" );
@@ -2615,15 +2604,16 @@ static int ql_alloc_buffer_queues(struct ql3_adapter *qdev) qdev->small_buf_q_alloc_size = qdev->small_buf_q_size *2 ; qdev->small_buf_q_alloc_virt_addr = -pci_alloc_consistent (qdev->pdev, - qdev->small_buf_q_alloc_size, - &qdev->small_buf_q_alloc_phy_addr); +dma_alloc_coherent (&qdev->pdev->dev, + qdev->small_buf_q_alloc_size, + &qdev->small_buf_q_alloc_phy_addr,GFP_KERNEL );if (qdev->small_buf_q_alloc_virt_addr ==NULL ) {netdev_err (qdev->ndev,"Small Buffer Queue allocation failed\n" ); -pci_free_consistent (qdev->pdev, qdev->lrg_buf_q_alloc_size, - qdev->lrg_buf_q_alloc_virt_addr, - qdev->lrg_buf_q_alloc_phy_addr); +dma_free_coherent (&qdev->pdev->dev, + qdev->lrg_buf_q_alloc_size, + qdev->lrg_buf_q_alloc_virt_addr, + qdev->lrg_buf_q_alloc_phy_addr);return -ENOMEM ; }
@@ -2640,17 +2630,15 @@ static void ql_free_buffer_queues(struct ql3_adapter *qdev)return ; }kfree (qdev->lrg_buf); -pci_free_consistent (qdev->pdev, - qdev->lrg_buf_q_alloc_size, - qdev->lrg_buf_q_alloc_virt_addr, - qdev->lrg_buf_q_alloc_phy_addr); +dma_free_coherent (&qdev->pdev->dev, qdev->lrg_buf_q_alloc_size, + qdev->lrg_buf_q_alloc_virt_addr, + qdev->lrg_buf_q_alloc_phy_addr); qdev->lrg_buf_q_virt_addr =NULL ; -pci_free_consistent (qdev->pdev, - qdev->small_buf_q_alloc_size, - qdev->small_buf_q_alloc_virt_addr, - qdev->small_buf_q_alloc_phy_addr); +dma_free_coherent (&qdev->pdev->dev, qdev->small_buf_q_alloc_size, + qdev->small_buf_q_alloc_virt_addr, + qdev->small_buf_q_alloc_phy_addr); qdev->small_buf_q_virt_addr =NULL ;
@@ -2668,9 +2656,9 @@ static int ql_alloc_small_buffers(struct ql3_adapter *qdev) QL_SMALL_BUFFER_SIZE); qdev->small_buf_virt_addr = -pci_alloc_consistent (qdev->pdev, - qdev->small_buf_total_size, - &qdev->small_buf_phy_addr); +dma_alloc_coherent (&qdev->pdev->dev, + qdev->small_buf_total_size, + &qdev->small_buf_phy_addr,GFP_KERNEL );if (qdev->small_buf_virt_addr ==NULL ) {netdev_err (qdev->ndev,"Failed to get small buffer memory\n" );
@@ -2703,10 +2691,10 @@ static void ql_free_small_buffers(struct ql3_adapter *qdev)return ; }if (qdev->small_buf_virt_addr !=NULL ) { -pci_free_consistent (qdev->pdev, - qdev->small_buf_total_size, - qdev->small_buf_virt_addr, - qdev->small_buf_phy_addr); +dma_free_coherent (&qdev->pdev->dev, + qdev->small_buf_total_size, + qdev->small_buf_virt_addr, + qdev->small_buf_phy_addr); qdev->small_buf_virt_addr =NULL ; }
@@ -2721,10 +2709,10 @@ static void ql_free_large_buffers(struct ql3_adapter *qdev) lrg_buf_cb = &qdev->lrg_buf[i];if (lrg_buf_cb->skb) {dev_kfree_skb (lrg_buf_cb->skb); -pci_unmap_single (qdev->pdev, +dma_unmap_single (&qdev->pdev->dev,dma_unmap_addr (lrg_buf_cb, mapaddr),dma_unmap_len (lrg_buf_cb, maplen), -PCI_DMA_FROMDEVICE ); +DMA_FROM_DEVICE );memset (lrg_buf_cb,0 ,sizeof (struct ql_rcv_buf_cb )); }else {break ;
@@ -2776,13 +2764,11 @@ static int ql_alloc_large_buffers(struct ql3_adapter *qdev)* buffer */ skb_reserve (skb,QL_HEADER_SPACE ); - map =pci_map_single (qdev->pdev, - skb->data, - qdev->lrg_buffer_len - -QL_HEADER_SPACE , -PCI_DMA_FROMDEVICE ); + map =dma_map_single (&qdev->pdev->dev, skb->data, + qdev->lrg_buffer_len -QL_HEADER_SPACE , +DMA_FROM_DEVICE ); - err =pci_dma_mapping_error (qdev->pdev, map); + err =dma_mapping_error (&qdev->pdev->dev, map);if (err) { netdev_err(qdev->ndev, "PCI mapping failed with error: %d\n",
@@ -2867,8 +2853,8 @@ static int ql_alloc_mem_resources(struct ql3_adapter *qdev)* Network Completion Queue Producer Index Register */ qdev->shadow_reg_virt_addr = -pci_alloc_consistent (qdev->pdev, -PAGE_SIZE , &qdev->shadow_reg_phy_addr); +dma_alloc_coherent (&qdev->pdev->dev,PAGE_SIZE , + &qdev->shadow_reg_phy_addr,GFP_KERNEL );if (qdev->shadow_reg_virt_addr !=NULL ) { qdev->preq_consumer_index = qdev->shadow_reg_virt_addr;
@@ -2923,10 +2909,9 @@ static int ql_alloc_mem_resources(struct ql3_adapter *qdev) err_buffer_queues:ql_free_net_req_rsp_queues (qdev); err_req_rsp: -pci_free_consistent (qdev->pdev, -PAGE_SIZE , - qdev->shadow_reg_virt_addr, - qdev->shadow_reg_phy_addr); +dma_free_coherent (&qdev->pdev->dev,PAGE_SIZE , + qdev->shadow_reg_virt_addr, + qdev->shadow_reg_phy_addr);return -ENOMEM ; }
@@ -2939,10 +2924,9 @@ static void ql_free_mem_resources(struct ql3_adapter *qdev)ql_free_buffer_queues (qdev);ql_free_net_req_rsp_queues (qdev);if (qdev->shadow_reg_virt_addr !=NULL ) { -pci_free_consistent (qdev->pdev, -PAGE_SIZE , - qdev->shadow_reg_virt_addr, - qdev->shadow_reg_phy_addr); +dma_free_coherent (&qdev->pdev->dev,PAGE_SIZE , + qdev->shadow_reg_virt_addr, + qdev->shadow_reg_phy_addr); qdev->shadow_reg_virt_addr =NULL ; } }
@@ -3643,18 +3627,15 @@ static void ql_reset_work(struct work_struct *work)if (tx_cb->skb) {netdev_printk (KERN_DEBUG , ndev,"Freeing lost SKB\n" ); -pci_unmap_single (qdev->pdev, -dma_unmap_addr (&tx_cb->map[0 ], - mapaddr), -dma_unmap_len (&tx_cb->map[0 ], maplen), -PCI_DMA_TODEVICE ); +dma_unmap_single (&qdev->pdev->dev, +dma_unmap_addr (&tx_cb->map[0 ], mapaddr), +dma_unmap_len (&tx_cb->map[0 ], maplen), +DMA_TO_DEVICE );for (j =1 ; j < tx_cb->seg_count; j++) { -pci_unmap_page (qdev->pdev, -dma_unmap_addr (&tx_cb->map[j], - mapaddr), -dma_unmap_len (&tx_cb->map[j], - maplen), -PCI_DMA_TODEVICE ); +dma_unmap_page (&qdev->pdev->dev, +dma_unmap_addr (&tx_cb->map[j], mapaddr), +dma_unmap_len (&tx_cb->map[j], maplen), +DMA_TO_DEVICE ); }dev_kfree_skb (tx_cb->skb); tx_cb->skb =NULL ;
@@ -3786,13 +3767,10 @@ static int ql3xxx_probe(struct pci_dev *pdev,pci_set_master (pdev); -if (!pci_set_dma_mask (pdev,DMA_BIT_MASK (64 ))) { +if (!dma_set_mask_and_coherent (&pdev->dev,DMA_BIT_MASK (64 ))) pci_using_dac =1 ; - err =pci_set_consistent_dma_mask (pdev,DMA_BIT_MASK (64 )); - }else if (!(err =pci_set_dma_mask (pdev,DMA_BIT_MASK (32 )))) { +else if (!(err =dma_set_mask_and_coherent (&pdev->dev,DMA_BIT_MASK (32 )))) pci_using_dac =0 ; - err =pci_set_consistent_dma_mask (pdev,DMA_BIT_MASK (32 )); - }if (err) {pr_err ("%s no usable DMA configuration\n" ,pci_name (pdev));
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 17/38 net/qla3xxx: fix potential memleak in ql_alloc_buffer_queues
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dinghao Liu <[email protected]>
[ Upstream commit 89f45c30172c80e55c887f32f1af8e184124577b ]
When dma_alloc_coherent() fails, we should free qdev->lrg_buf to prevent potential memleak.
--- drivers/net/ethernet/qlogic/qla3xxx.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/net/ethernet/qlogic/qla3xxx.c +2 -0
@@ -2590,6 +2590,7 @@ static int ql_alloc_buffer_queues(struct ql3_adapter *qdev)if (qdev->lrg_buf_q_alloc_virt_addr ==NULL ) {netdev_err (qdev->ndev,"lBufQ failed\n" ); +kfree (qdev->lrg_buf);return -ENOMEM ; } qdev->lrg_buf_q_virt_addr = qdev->lrg_buf_q_alloc_virt_addr;
@@ -2614,6 +2615,7 @@ static int ql_alloc_buffer_queues(struct ql3_adapter *qdev) qdev->lrg_buf_q_alloc_size, qdev->lrg_buf_q_alloc_virt_addr, qdev->lrg_buf_q_alloc_phy_addr); +kfree (qdev->lrg_buf);return -ENOMEM ; }
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 18/38 asix: Add check for usbnet_get_endpoints
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Chen Ni <[email protected]>
[ Upstream commit eaac6a2d26b65511e164772bec6918fcbc61938e ]
Add check for usbnet_get_endpoints() and return the error if it fails in order to transfer the error.
--- drivers/net/usb/ax88172a.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/net/usb/ax88172a.c +3 -1
@@ -186,7 +186,9 @@ static int ax88172a_bind(struct usbnet *dev, struct usb_interface *intf)u8 buf[ETH_ALEN ];struct ax88172a_private *priv; -usbnet_get_endpoints (dev, intf); + ret =usbnet_get_endpoints (dev, intf); +if (ret) +return ret; priv =kzalloc (sizeof (*priv),GFP_KERNEL );if (!priv)
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 19/38 bnxt_en: Remove mis-applied code from bnxt_cfg_ntp_filters()
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Michael Chan <[email protected]>
[ Upstream commit e009b2efb7a8850498796b360043ac25c8d3d28f ]
The 2 lines to check for the BNXT_HWRM_PF_UNLOAD_SP_EVENT bit was mis-applied to bnxt_cfg_ntp_filters() and should have been applied to bnxt_sp_task().
--- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/broadcom/bnxt/bnxt.c +2 -2
@@ -10340,6 +10340,8 @@ static void bnxt_sp_task(struct work_struct *work)bnxt_cfg_ntp_filters (bp);if (test_and_clear_bit (BNXT_HWRM_EXEC_FWD_REQ_SP_EVENT , &bp->sp_event))bnxt_hwrm_exec_fwd_req (bp); +if (test_and_clear_bit (BNXT_HWRM_PF_UNLOAD_SP_EVENT , &bp->sp_event)) +netdev_info (bp->dev,"Receive PF driver unload event!\n" );if (test_and_clear_bit (BNXT_VXLAN_ADD_PORT_SP_EVENT , &bp->sp_event)) { bnxt_hwrm_tunnel_dst_port_alloc( bp, bp->vxlan_port,
@@ -11266,8 +11268,6 @@ static void bnxt_cfg_ntp_filters(struct bnxt *bp) } } } -if (test_and_clear_bit (BNXT_HWRM_PF_UNLOAD_SP_EVENT , &bp->sp_event)) -netdev_info (bp->dev,"Receive PF driver unload event!" ); }#else
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 20/38 net: Implement missing SO_TIMESTAMPING_NEW cmsg support
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Thomas Lange <[email protected]>
[ Upstream commit 382a32018b74f407008615e0e831d05ed28e81cd ]
Commit 9718475e6908 ("socket: Add SO_TIMESTAMPING_NEW") added the new socket option SO_TIMESTAMPING_NEW. However, it was never implemented in __sock_cmsg_send thus breaking SO_TIMESTAMPING cmsg for platforms using SO_TIMESTAMPING_NEW.
--- net/core/sock.c | 1 + 1 file changed, 1 insertion(+)
net/core/sock.c +1 -0
@@ -2310,6 +2310,7 @@ int __sock_cmsg_send(struct sock *sk, struct msghdr *msg, struct cmsghdr *cmsg, sockc->mark = *(u32 *)CMSG_DATA (cmsg);break ;case SO_TIMESTAMPING_OLD : +case SO_TIMESTAMPING_NEW :if (cmsg->cmsg_len !=CMSG_LEN (sizeof (u32)))return -EINVAL ;
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 21/38 mm/memory-failure: check the mapcount of the precise page
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Matthew Wilcox (Oracle) <[email protected]>
[ Upstream commit c79c5a0a00a9457718056b588f312baadf44e471 ]
A process may map only some of the pages in a folio, and might be missed if it maps the poisoned page but not the head page. Or it might be unnecessarily hit if it maps the head page, but not the poisoned page.
--- mm/memory-failure.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
mm/memory-failure.c +3 -3
@@ -986,7 +986,7 @@ static bool hwpoison_user_mappings(struct page *p, unsigned long pfn,* This check implies we don't kill processes if their pages * are in the swap cache early. Those are always late kills. */ -if (!page_mapped (hpage)) +if (!page_mapped (p))return true;if (PageKsm (p)) {
@@ -1030,10 +1030,10 @@ static bool hwpoison_user_mappings(struct page *p, unsigned long pfn,if (kill)collect_procs (hpage, &tokill, flags &MF_ACTION_REQUIRED ); - unmap_success =try_to_unmap (hpage, ttu); + unmap_success =try_to_unmap (p, ttu);if (!unmap_success)pr_err ("Memory failure: %#lx: failed to unmap page (mapcount=%d)\n" , - pfn,page_mapcount (hpage)); + pfn,page_mapcount (p));/* * try_to_unmap() might put mlocked page in lru cache, so call
Signature · 1 line
2.43.0
Greg Kroah-Hartman
maintainerPATCH 22/38 firewire: ohci: suppress unexpected system reboot in AMD Ryzen machines and ASM108x/VT630x PCIe cards
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Sakamoto <[email protected]>
commit ac9184fbb8478dab4a0724b279f94956b69be827 upstream.
VIA VT6306/6307/6308 provides PCI interface compliant to 1394 OHCI. When the hardware is combined with Asmedia ASM1083/1085 PCIe-to-PCI bus bridge, it appears that accesses to its 'Isochronous Cycle Timer' register (offset 0xf0 on PCI memory space) often causes unexpected system reboot in any type of AMD Ryzen machine (both 0x17 and 0x19 families). It does not appears in the other type of machine (AMD pre-Ryzen machine, Intel machine, at least), or in the other OHCI 1394 hardware (e.g. Texas Instruments).
The issue explicitly appears at a commit dcadfd7f7c74 ("firewire: core:
use union for callback of transaction completion") added to v6.5 kernel. It changed 1394 OHCI driver to access to the register every time to dispatch local asynchronous transaction. However, the issue exists in older version of kernel as long as it runs in AMD Ryzen machine, since the access to the register is required to maintain bus time. It is not hard to imagine that users experience the unexpected system reboot when generating bus reset by plugging any devices in, or reading the register by time-aware application programs; e.g. audio sample processing.
This commit suppresses the unexpected system reboot in the combination of hardware. It avoids the access itself. As a result, the software stack can not provide the hardware time anymore to unit drivers, userspace applications, and nodes in the same IEEE 1394 bus. It brings apparent disadvantage since time-aware application programs require it, while time-unaware applications are available again; e.g. sbp2.
--- drivers/firewire/ohci.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 51 insertions(+)
drivers/firewire/ohci.c +51 -0
@@ -279,6 +279,51 @@ static char ohci_driver_name[] = KBUILD_#define QUIRK_TI_SLLZ059 0x20#define QUIRK_IR_WAKE 0x40 +// On PCI Express Root Complex in any type of AMD Ryzen machine, VIA VT6306/6307/6308 with Asmedia +// ASM1083/1085 brings an inconvenience that the read accesses to 'Isochronous Cycle Timer' register +// (at offset 0xf0 in PCI I/O space) often causes unexpected system reboot. The mechanism is not +// clear, since the read access to the other registers is enough safe; e.g. 'Node ID' register, +// while it is probable due to detection of any type of PCIe error. +#define QUIRK_REBOOT_BY_CYCLE_TIMER_READ 0x80000000 + +#if IS_ENABLED (CONFIG_X86 ) + +static bool has_reboot_by_cycle_timer_read_quirk (const struct fw_ohci *ohci) +{ +return !!(ohci->quirks &QUIRK_REBOOT_BY_CYCLE_TIMER_READ ); +} + +#define PCI_DEVICE_ID_ASMEDIA_ASM108X 0x1080 + +static bool detect_vt630x_with_asm1083_on_amd_ryzen_machine (const struct pci_dev *pdev) +{ +const struct pci_dev *pcie_to_pci_bridge; + +// Detect any type of AMD Ryzen machine. +if (!static_cpu_has (X86_FEATURE_ZEN )) +return false; + +// Detect VIA VT6306/6307/6308. +if (pdev->vendor !=PCI_VENDOR_ID_VIA ) +return false; +if (pdev->device !=PCI_DEVICE_ID_VIA_VT630X ) +return false; + +// Detect Asmedia ASM1083/1085. + pcie_to_pci_bridge = pdev->bus->self; +if (pcie_to_pci_bridge->vendor !=PCI_VENDOR_ID_ASMEDIA ) +return false; +if (pcie_to_pci_bridge->device !=PCI_DEVICE_ID_ASMEDIA_ASM108X ) +return false; + +return true; +} + +#else +#define has_reboot_by_cycle_timer_read_quirk (ohci) false +#define detect_vt630x_with_asm1083_on_amd_ryzen_machine (pdev) false +#endif +/* In case of multiple matches in ohci_quirks[], only the first one is used. */ static const struct { unsigned short vendor, device, revision, flags;
@@ -1717,6 +1762,9 @@ static u32 get_cycle_time(struct fw_ohcis32 diff01, diff12;int i; +if (has_reboot_by_cycle_timer_read_quirk (ohci)) +return 0 ; + c2 =reg_read (ohci, OHCI1394_IsochronousCycleTimer);if (ohci->quirks &QUIRK_CYCLE_TIMER ) {
@@ -3619,6 +3667,9 @@ static int pci_probe(struct pci_dev *devif (param_quirks) ohci->quirks = param_quirks; +if (detect_vt630x_with_asm1083_on_amd_ryzen_machine (dev)) + ohci->quirks |=QUIRK_REBOOT_BY_CYCLE_TIMER_READ ; +/* * Because dma_alloc_coherent() allocates at least one page, * we save space by using a common buffer for the AR request/
Greg Kroah-Hartman
maintainerPATCH 23/38 i2c: core: Fix atomic xfer check for non-preempt config
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Benjamin Bara <[email protected]>
commit a3368e1186e3ce8e38f78cbca019622095b1f331 upstream.
Since commit aa49c90894d0 ("i2c: core: Run atomic i2c xfer when !preemptible"), the whole reboot/power off sequence on non-preempt kernels is using atomic i2c xfer, as !preemptible() always results to 1.
During device_shutdown(), the i2c might be used a lot and not all busses have implemented an atomic xfer handler. This results in a lot of avoidable noise, like:
[ 12.687169] No atomic I2C transfer handler for 'i2c-0' [ 12.692313] WARNING: CPU: 6 PID: 275 at drivers/i2c/i2c-core.h:40 i2c_smbus_xfer+0x100/0x118
...
Fix this by allowing non-atomic xfer when the interrupts are enabled, as it was before.
--- drivers/i2c/i2c-core.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/i2c/i2c-core.h +3 -1
@@ -3,6 +3,7 @@* i2c-core.h - interfaces internal to the I2C framework */ +#include <linux/kconfig.h> #include <linux/rwsem.h> struct i2c_devinfo {
@@ -29,7 +30,8 @@ int i2c_dev_irq_from_resources(const str*/ static inline bool i2c_in_atomic_xfer_mode (void ) { -return system_state >SYSTEM_RUNNING && !preemptible (); +return system_state >SYSTEM_RUNNING && + (IS_ENABLED (CONFIG_PREEMPT_COUNT ) ? !preemptible () :irqs_disabled ()); }static inline int __i2c_lock_bus_helper (struct i2c_adapter *adap)
Greg Kroah-Hartman
maintainerPATCH 24/38 mm: fix unmap_mapping_range high bits shift bug
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jiajun Xie <[email protected]>
commit 9eab0421fa94a3dde0d1f7e36ab3294fc306c99d upstream.
The bug happens when highest bit of holebegin is 1, suppose holebegin is 0x8000000111111000, after shift, hba would be 0xfff8000000111111, then vma_interval_tree_foreach would look it up fail or leads to the wrong result.
error call seq e.g.:
- mmap(..., offset=0x8000000111111000)
|- syscall(mmap, ... unsigned long, off):
|- ksys_mmap_pgoff( ... , off >> PAGE_SHIFT);
here pgoff is correctly shifted to 0x8000000111111, but pass 0x8000000111111000 as holebegin to unmap would then cause terrible result, as shown below:
- unmap_mapping_range(..., loff_t const holebegin)
|- pgoff_t hba = holebegin >> PAGE_SHIFT;
/* hba = 0xfff8000000111111 unexpectedly */The issue happens in Heterogeneous computing, where the device(e.g. gpu) and host share the same virtual address space.
A simple workflow pattern which hit the issue is:
/* host */
1. userspace first mmap a file backed VA range with specified offset. e.g. (offset=0x800..., mmap return: va_a)
2. write some data to the corresponding sys page
e.g. (va_a = 0xAABB)
/* device */
3. gpu workload touches VA, triggers gpu fault and notify the host. /* host */
4. reviced gpu fault notification, then it will:
4.1 unmap host pages and also takes care of cpu tlb
(use unmap_mapping_range with offset=0x800...)
4.2 migrate sys page to device
4.3 setup device page table and resolve device fault.
/* device */
5. gpu workload continued, it accessed va_a and got 0xAABB.
6. gpu workload continued, it wrote 0xBBCC to va_a.
/* host */
7. userspace access va_a, as expected, it will:
7.1 trigger cpu vm fault.
7.2 driver handling fault to migrate gpu local page to host.
8. userspace then could correctly get 0xBBCC from va_a
9. done
But in step 4.1, if we hit the bug this patch mentioned, then userspace would never trigger cpu fault, and still get the old value: 0xAABB.
Making holebegin unsigned first fixes the bug.
--- mm/memory.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
mm/memory.c +2 -2
@@ -2872,8 +2872,8 @@ void unmap_mapping_pages(struct address_void unmap_mapping_range (struct address_space *mapping,loff_t const holebegin,loff_t const holelen,int even_cows) { -pgoff_t hba = holebegin >>PAGE_SHIFT ; -pgoff_t hlen = (holelen +PAGE_SIZE -1 ) >>PAGE_SHIFT ; +pgoff_t hba = (pgoff_t)(holebegin) >>PAGE_SHIFT ; +pgoff_t hlen = ((pgoff_t)(holelen) +PAGE_SIZE -1 ) >>PAGE_SHIFT ;/* Check for overflow. */ if (sizeof (holelen) >sizeof (hlen)) {
Greg Kroah-Hartman
maintainerPATCH 25/38 mmc: rpmb: fixes pause retune on all RPMB partitions.
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jorge Ramirez-Ortiz <[email protected]>
commit e7794c14fd73e5eb4a3e0ecaa5334d5a17377c50 upstream.
When RPMB was converted to a character device, it added support for multiple RPMB partitions (Commit 97548575bef3 ("mmc: block: Convert RPMB to a character device").
One of the changes in this commit was transforming the variable target_part defined in __mmc_blk_ioctl_cmd into a bitmask. This inadvertently regressed the validation check done in mmc_blk_part_switch_pre() and mmc_blk_part_switch_post(), so let's fix it.
--- drivers/mmc/core/block.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)
drivers/mmc/core/block.c +4 -3
@@ -850,9 +850,10 @@ static const struct block_device_operatistatic int mmc_blk_part_switch_pre (struct mmc_card *card,unsigned int part_type) { +const unsigned int mask =EXT_CSD_PART_CONFIG_ACC_RPMB ;int ret =0 ; -if (part_type ==EXT_CSD_PART_CONFIG_ACC_RPMB ) { +if ((part_type & mask) == mask) {if (card->ext_csd.cmdq_en) { ret =mmc_cmdq_disable (card);if (ret)
@@ -867,9 +868,10 @@ static int mmc_blk_part_switch_pre(strucstatic int mmc_blk_part_switch_post (struct mmc_card *card,unsigned int part_type) { +const unsigned int mask =EXT_CSD_PART_CONFIG_ACC_RPMB ;int ret =0 ; -if (part_type ==EXT_CSD_PART_CONFIG_ACC_RPMB ) { +if ((part_type & mask) == mask) {mmc_retune_unpause (card->host);if (card->reenable_cmdq && !card->ext_csd.cmdq_en) ret =mmc_cmdq_enable (card);
@@ -3091,4 +3093,3 @@ module_exit(mmc_blk_exit);MODULE_LICENSE ("GPL" );MODULE_DESCRIPTION ("Multimedia Card (MMC) block device driver" ); -
Greg Kroah-Hartman
maintainerPATCH 26/38 mmc: core: Cancel delayed work before releasing host
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Geert Uytterhoeven <[email protected]>
commit 1036f69e251380573e256568cf814506e3fb9988 upstream.
On RZ/Five SMARC EVK, where probing of SDHI is deferred due to probe deferral of the vqmmc-supply regulator:
------------[ cut here ]------------
WARNING: CPU: 0 PID: 0 at kernel/time/timer.c:1738 __run_timers.part.0+0x1d0/0x1e8
Modules linked in:
CPU: 0 PID: 0 Comm: swapper Not tainted 6.7.0-rc4 #101
Hardware name: Renesas SMARC EVK based on r9a07g043f01 (DT)
epc : __run_timers.part.0+0x1d0/0x1e8
ra : __run_timers.part.0+0x134/0x1e8
epc : ffffffff800771a4 ra : ffffffff80077108 sp : ffffffc800003e60
gp : ffffffff814f5028 tp : ffffffff8140c5c0 t0 : ffffffc800000000
t1 : 0000000000000001 t2 : ffffffff81201300 s0 : ffffffc800003f20
s1 : ffffffd8023bc4a0 a0 : 00000000fffee6b0 a1 : 0004010000400000
a2 : ffffffffc0000016 a3 : ffffffff81488640 a4 : ffffffc800003e60
a5 : 0000000000000000 a6 : 0000000004000000 a7 : ffffffc800003e68
s2 : 0000000000000122 s3 : 0000000000200000 s4 : 0000000000000000
s5 : ffffffffffffffff s6 : ffffffff81488678 s7 : ffffffff814886c0
s8 : ffffffff814f49c0 s9 : ffffffff81488640 s10: 0000000000000000
s11: ffffffc800003e60 t3 : 0000000000000240 t4 : 0000000000000a52
t5 : ffffffd8024ae018 t6 : ffffffd8024ae038
status: 0000000200000100 badaddr: 0000000000000000 cause: 0000000000000003
[<ffffffff800771a4>] __run_timers.part.0+0x1d0/0x1e8
[<ffffffff800771e0>] run_timer_softirq+0x24/0x4a
[<ffffffff80809092>] __do_softirq+0xc6/0x1fa
[<ffffffff80028e4c>] irq_exit_rcu+0x66/0x84
[<ffffffff80800f7a>] handle_riscv_irq+0x40/0x4e
[<ffffffff80808f48>] call_on_irq_stack+0x1c/0x28
---[ end trace 0000000000000000 ]---What happens?
renesas_sdhi_probe()
{
tmio_mmc_host_alloc()
mmc_alloc_host()
INIT_DELAYED_WORK(&host->detect, mmc_rescan);
devm_request_irq(tmio_mmc_irq);
/*
* After this, the interrupt handler may be invoked at any time
*
* tmio_mmc_irq()
* {
* __tmio_mmc_card_detect_irq()
* mmc_detect_change()
* _mmc_detect_change()
* mmc_schedule_delayed_work(&host->detect, delay);
* }
*/
tmio_mmc_host_probe()
tmio_mmc_init_ocr()
-EPROBE_DEFER
tmio_mmc_host_free()
mmc_free_host()
}When expire_timers() runs later, it warns because the MMC host structure containing the delayed work was freed, and now contains an invalid work function pointer.
Fix this by cancelling any pending delayed work before releasing the MMC host structure.
--- drivers/mmc/core/host.c | 1 + 1 file changed, 1 insertion(+)
drivers/mmc/core/host.c +1 -0
@@ -570,6 +570,7 @@ EXPORT_SYMBOL(mmc_remove_host);*/ void mmc_free_host (struct mmc_host *host) { +cancel_delayed_work_sync (&host->detect);mmc_pwrseq_free (host);put_device (&host->class_dev); }
Greg Kroah-Hartman
maintainerPATCH 27/38 mmc: sdhci-sprd: Fix eMMC init failure after hw reset
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Wenchao Chen <[email protected]>
commit 8abf77c88929b6d20fa4f9928b18d6448d64e293 upstream.
Some eMMC devices that do not close the auto clk gate after hw reset will cause eMMC initialization to fail. Let's fix this.
--- drivers/mmc/host/sdhci-sprd.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-)
drivers/mmc/host/sdhci-sprd.c +7 -3
@@ -223,15 +223,19 @@ static inline void _sdhci_sprd_set_clock div = ((div &0x300 ) >>2 ) | ((div &0xFF ) <<8 );sdhci_enable_clk (host, div); + val =sdhci_readl (host,SDHCI_SPRD_REG_32_BUSY_POSI ); + mask =SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN ;/* Enable CLK_AUTO when the clock is greater than 400K. */ if (clk >400000 ) { - val =sdhci_readl (host,SDHCI_SPRD_REG_32_BUSY_POSI ); - mask =SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | -SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN ;if (mask != (val & mask)) { val |= mask;sdhci_writel (host, val,SDHCI_SPRD_REG_32_BUSY_POSI ); } + }else { +if (val & mask) { + val &= ~mask; +sdhci_writel (host, val,SDHCI_SPRD_REG_32_BUSY_POSI ); + } } }
Greg Kroah-Hartman
maintainerPATCH 28/38 ath10k: Wait until copy complete is actually done before completing
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Douglas Anderson <[email protected]>
[ Upstream commit 8f9ed93d09a97444733d492a3bbf66bcb786a777 ]
On wcn3990 we have "per_ce_irq = true". That makes the ath10k_ce_interrupt_summary() function always return 0xfff. The ath10k_ce_per_engine_service_any() function will see this and think that _all_ copy engines have an interrupt. Without checking, the ath10k_ce_per_engine_service() assumes that if it's called that the "copy complete" (cc) interrupt fired. This combination seems bad.
Let's add a check to make sure that the "copy complete" interrupt actually fired in ath10k_ce_per_engine_service().
This might fix a hard-to-reproduce failure where it appears that the copy complete handlers run before the copy is really complete. Specifically a symptom was that we were seeing this on a Qualcomm sc7180 board:
arm-smmu 15000000.iommu: Unhandled context fault:
fsr=0x402, iova=0x7fdd45780, fsynr=0x30003, cbfrsynra=0xc1, cb=10
Even on platforms that don't have wcn3990 this still seems like it would be a sane thing to do. Specifically the current IRQ handler comments indicate that there might be other misc interrupt sources firing that need to be cleared. If one of those sources was the one that caused the IRQ handler to be called it would also be important to double-check that the interrupt we cared about actually fired.
Tested-on: WCN3990 SNOC WLAN.HL.3.2.2-00490-QCAHLSWMTPL-1
Stable-dep-of: 170c75d43a77 ("ath10k: Don't touch the CE interrupt registers after power up")
--- drivers/net/wireless/ath/ath10k/ce.c | 30 +++++++++++++++++++++--------- 1 file changed, 21 insertions(+), 9 deletions(-)
drivers/net/wireless/ath/ath10k/ce.c +21 -9
@@ -481,6 +481,15 @@ static inline void ath10k_ce_engine_int_ath10k_ce_write32 (ar, ce_ctrl_addr + wm_regs->addr, mask); } +static inline bool ath10k_ce_engine_int_status_check (struct ath10k *ar, +u32 ce_ctrl_addr, +unsigned int mask) +{ +struct ath10k_hw_ce_host_wm_regs *wm_regs = ar->hw_ce_regs->wm_regs; + +return ath10k_ce_read32 (ar, ce_ctrl_addr + wm_regs->addr) & mask; +} +/* * Guts of ath10k_ce_send. * The caller takes responsibility for any needed locking.
@@ -1301,19 +1310,22 @@ void ath10k_ce_per_engine_service(structspin_lock_bh (&ce->ce_lock); -/* Clear the copy-complete interrupts that will be handled here. */ -ath10k_ce_engine_int_status_clear (ar, ctrl_addr, - wm_regs->cc_mask); +if (ath10k_ce_engine_int_status_check (ar, ctrl_addr, + wm_regs->cc_mask)) { +/* Clear before handling */ +ath10k_ce_engine_int_status_clear (ar, ctrl_addr, + wm_regs->cc_mask); -spin_unlock_bh (&ce->ce_lock); +spin_unlock_bh (&ce->ce_lock); -if (ce_state->recv_cb) - ce_state->recv_cb (ce_state); +if (ce_state->recv_cb) + ce_state->recv_cb (ce_state); -if (ce_state->send_cb) - ce_state->send_cb (ce_state); +if (ce_state->send_cb) + ce_state->send_cb (ce_state); -spin_lock_bh (&ce->ce_lock); +spin_lock_bh (&ce->ce_lock); + }/* * Misc CE interrupts are not being handled, but still need
Greg Kroah-Hartman
maintainerPATCH 29/38 ath10k: Add interrupt summary based CE processing
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Rakesh Pillai <[email protected]>
[ Upstream commit b92aba35d39d10d8a6bdf2495172fd490c598b4a ]
Currently the NAPI processing loops through all the copy engines and processes a particular copy engine is the copy completion is set for that copy engine. The host driver is not supposed to access any copy engine register after clearing the interrupt status register.
This might result in kernel crash like the one below
[ 1159.220143] Call trace: [ 1159.220170] ath10k_snoc_read32+0x20/0x40 [ath10k_snoc] [ 1159.220193] ath10k_ce_per_engine_service_any+0x78/0x130 [ath10k_core] [ 1159.220203] ath10k_snoc_napi_poll+0x38/0x8c [ath10k_snoc] [ 1159.220270] net_rx_action+0x100/0x3b0 [ 1159.220312] __do_softirq+0x164/0x30c [ 1159.220345] run_ksoftirqd+0x2c/0x64 [ 1159.220380] smpboot_thread_fn+0x1b0/0x288 [ 1159.220405] kthread+0x11c/0x12c [ 1159.220423] ret_from_fork+0x10/0x18
To avoid such a scenario, we generate an interrupt summary by reading the copy completion for all the copy engine before actually processing any of them. This will avoid reading the interrupt status register for any CE after the interrupt status is cleared.
Tested-on: WCN3990 hw1.0 SNOC WLAN.HL.3.1-01040-QCAHLSWMTPLZ-1
Stable-dep-of: 170c75d43a77 ("ath10k: Don't touch the CE interrupt registers after power up")
--- drivers/net/wireless/ath/ath10k/ce.c | 63 +++++++++++++++++++++-------------- drivers/net/wireless/ath/ath10k/ce.h | 5 +- 2 files changed, 42 insertions(+), 26 deletions(-)
drivers/net/wireless/ath/ath10k/ce.c +39 -24
@@ -481,15 +481,38 @@ static inline void ath10k_ce_engine_int_ath10k_ce_write32 (ar, ce_ctrl_addr + wm_regs->addr, mask); } -static inline bool ath10k_ce_engine_int_status_check (struct ath10k *ar, -u32 ce_ctrl_addr, -unsigned int mask) +static bool ath10k_ce_engine_int_status_check (struct ath10k *ar,u32 ce_ctrl_addr, +unsigned int mask) {struct ath10k_hw_ce_host_wm_regs *wm_regs = ar->hw_ce_regs->wm_regs;return ath10k_ce_read32 (ar, ce_ctrl_addr + wm_regs->addr) & mask; } +u32 ath10k_ce_gen_interrupt_summary (struct ath10k *ar) +{ +struct ath10k_hw_ce_host_wm_regs *wm_regs = ar->hw_ce_regs->wm_regs; +struct ath10k_ce_pipe *ce_state; +struct ath10k_ce *ce; +u32 irq_summary =0 ; +u32 ctrl_addr; +u32 ce_id; + + ce =ath10k_ce_priv (ar); + +for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) { + ce_state = &ce->ce_states[ce_id]; + ctrl_addr = ce_state->ctrl_addr; +if (ath10k_ce_engine_int_status_check (ar, ctrl_addr, + wm_regs->cc_mask)) { + irq_summary |=BIT (ce_id); + } + } + +return irq_summary; +} +EXPORT_SYMBOL (ath10k_ce_gen_interrupt_summary); +/* * Guts of ath10k_ce_send. * The caller takes responsibility for any needed locking.
@@ -1308,32 +1331,24 @@ void ath10k_ce_per_engine_service(structstruct ath10k_hw_ce_host_wm_regs *wm_regs = ar->hw_ce_regs->wm_regs;u32 ctrl_addr = ce_state->ctrl_addr; -spin_lock_bh (&ce->ce_lock); - -if (ath10k_ce_engine_int_status_check (ar, ctrl_addr, - wm_regs->cc_mask)) { -/* Clear before handling */ -ath10k_ce_engine_int_status_clear (ar, ctrl_addr, - wm_regs->cc_mask); - -spin_unlock_bh (&ce->ce_lock); - -if (ce_state->recv_cb) - ce_state->recv_cb (ce_state); - -if (ce_state->send_cb) - ce_state->send_cb (ce_state); - -spin_lock_bh (&ce->ce_lock); - } -/* +* Clear before handling +* * Misc CE interrupts are not being handled, but still need * to be cleared. +* +* NOTE: When the last copy engine interrupt is cleared the +* hardware will go to sleep. Once this happens any access to +* the CE registers can cause a hardware fault. */ -ath10k_ce_engine_int_status_clear (ar, ctrl_addr, wm_regs->wm_mask); +ath10k_ce_engine_int_status_clear (ar, ctrl_addr, + wm_regs->cc_mask | wm_regs->wm_mask); -spin_unlock_bh (&ce->ce_lock); +if (ce_state->recv_cb) + ce_state->recv_cb (ce_state); + +if (ce_state->send_cb) + ce_state->send_cb (ce_state); }EXPORT_SYMBOL (ath10k_ce_per_engine_service);
drivers/net/wireless/ath/ath10k/ce.h +3 -2
@@ -259,6 +259,8 @@ int ath10k_ce_disable_interrupts(structvoid ath10k_ce_enable_interrupts (struct ath10k *ar);void ath10k_ce_dump_registers (struct ath10k *ar,struct ath10k_fw_crash_data *crash_data); + +u32 ath10k_ce_gen_interrupt_summary (struct ath10k *ar);void ath10k_ce_alloc_rri (struct ath10k *ar);void ath10k_ce_free_rri (struct ath10k *ar);
@@ -369,7 +371,6 @@ static inline u32 ath10k_ce_base_address (((x) &CE_WRAPPER_INTERRUPT_SUMMARY_HOST_MSI_MASK ) >> \ CE_WRAPPER_INTERRUPT_SUMMARY_HOST_MSI_LSB)#define CE_WRAPPER_INTERRUPT_SUMMARY_ADDRESS 0x0000 -#define CE_INTERRUPT_SUMMARY (GENMASK(CE_COUNT_MAX - 1, 0))static inline u32 ath10k_ce_interrupt_summary (struct ath10k *ar) {
@@ -380,7 +381,7 @@ static inline u32 ath10k_ce_interrupt_su ce->bus_ops->read32 ((ar),CE_WRAPPER_BASE_ADDRESS + CE_WRAPPER_INTERRUPT_SUMMARY_ADDRESS)); else - return CE_INTERRUPT_SUMMARY; + return ath10k_ce_gen_interrupt_summary(ar); }/* Host software's Copy Engine configuration. */
Greg Kroah-Hartman
maintainerPATCH 30/38 ath10k: Keep track of which interrupts fired, dont poll them
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Douglas Anderson <[email protected]>
[ Upstream commit d66d24ac300cf41c6b88367fc9b4b6348679273d ]
If we have a per CE (Copy Engine) IRQ then we have no summary register. Right now the code generates a summary register by iterating over all copy engines and seeing if they have an interrupt pending.
This has a problem. Specifically if _none_ if the Copy Engines have an interrupt pending then they might go into low power mode and reading from their address space will cause a full system crash. This was seen to happen when two interrupts went off at nearly the same time. Both were handled by a single call of ath10k_snoc_napi_poll() but, because there were two interrupts handled and thus two calls to napi_schedule() there was still a second call to ath10k_snoc_napi_poll() which ran with no interrupts pending.
Instead of iterating over all the copy engines, let's just keep track of the IRQs that fire. Then we can effectively generate our own summary without ever needing to read the Copy Engines.
Tested-on: WCN3990 SNOC WLAN.HL.3.2.2-00490-QCAHLSWMTPL-1
Stable-dep-of: 170c75d43a77 ("ath10k: Don't touch the CE interrupt registers after power up")
--- drivers/net/wireless/ath/ath10k/ce.c | 84 ++++++++++++--------------------- drivers/net/wireless/ath/ath10k/ce.h | 14 ++--- drivers/net/wireless/ath/ath10k/snoc.c | 19 +++++-- drivers/net/wireless/ath/ath10k/snoc.h | 1 4 files changed, 52 insertions(+), 66 deletions(-)
drivers/net/wireless/ath/ath10k/ce.c +31 -53
@@ -481,38 +481,6 @@ static inline void ath10k_ce_engine_int_ath10k_ce_write32 (ar, ce_ctrl_addr + wm_regs->addr, mask); } -static bool ath10k_ce_engine_int_status_check (struct ath10k *ar,u32 ce_ctrl_addr, -unsigned int mask) -{ -struct ath10k_hw_ce_host_wm_regs *wm_regs = ar->hw_ce_regs->wm_regs; - -return ath10k_ce_read32 (ar, ce_ctrl_addr + wm_regs->addr) & mask; -} - -u32 ath10k_ce_gen_interrupt_summary (struct ath10k *ar) -{ -struct ath10k_hw_ce_host_wm_regs *wm_regs = ar->hw_ce_regs->wm_regs; -struct ath10k_ce_pipe *ce_state; -struct ath10k_ce *ce; -u32 irq_summary =0 ; -u32 ctrl_addr; -u32 ce_id; - - ce =ath10k_ce_priv (ar); - -for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) { - ce_state = &ce->ce_states[ce_id]; - ctrl_addr = ce_state->ctrl_addr; -if (ath10k_ce_engine_int_status_check (ar, ctrl_addr, - wm_regs->cc_mask)) { - irq_summary |=BIT (ce_id); - } - } - -return irq_summary; -} -EXPORT_SYMBOL (ath10k_ce_gen_interrupt_summary); -/* * Guts of ath10k_ce_send. * The caller takes responsibility for any needed locking.
@@ -1399,45 +1367,55 @@ static void ath10k_ce_per_engine_handlerath10k_ce_watermark_intr_disable (ar, ctrl_addr); } -int ath10k_ce_disable_interrupts (struct ath10k *ar) +void ath10k_ce_disable_interrupt (struct ath10k *ar,int ce_id) {struct ath10k_ce *ce =ath10k_ce_priv (ar);struct ath10k_ce_pipe *ce_state;u32 ctrl_addr; -int ce_id; -for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) { - ce_state = &ce->ce_states[ce_id]; -if (ce_state->attr_flags &CE_ATTR_POLL ) -continue ; + ce_state = &ce->ce_states[ce_id]; +if (ce_state->attr_flags &CE_ATTR_POLL ) +return ; - ctrl_addr =ath10k_ce_base_address (ar, ce_id); + ctrl_addr =ath10k_ce_base_address (ar, ce_id); -ath10k_ce_copy_complete_intr_disable (ar, ctrl_addr); -ath10k_ce_error_intr_disable (ar, ctrl_addr); -ath10k_ce_watermark_intr_disable (ar, ctrl_addr); - } +ath10k_ce_copy_complete_intr_disable (ar, ctrl_addr); +ath10k_ce_error_intr_disable (ar, ctrl_addr); +ath10k_ce_watermark_intr_disable (ar, ctrl_addr); +} +EXPORT_SYMBOL (ath10k_ce_disable_interrupt); -return 0 ; +void ath10k_ce_disable_interrupts (struct ath10k *ar) +{ +int ce_id; + +for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) +ath10k_ce_disable_interrupt (ar, ce_id); }EXPORT_SYMBOL (ath10k_ce_disable_interrupts); -void ath10k_ce_enable_interrupts (struct ath10k *ar) +void ath10k_ce_enable_interrupt (struct ath10k *ar,int ce_id) {struct ath10k_ce *ce =ath10k_ce_priv (ar); -int ce_id;struct ath10k_ce_pipe *ce_state; + ce_state = &ce->ce_states[ce_id]; +if (ce_state->attr_flags &CE_ATTR_POLL ) +return ; + +ath10k_ce_per_engine_handler_adjust (ce_state); +} +EXPORT_SYMBOL (ath10k_ce_enable_interrupt); + +void ath10k_ce_enable_interrupts (struct ath10k *ar) +{ +int ce_id; +/* Enable interrupts for copy engine that * are not using polling mode. */ -for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) { - ce_state = &ce->ce_states[ce_id]; -if (ce_state->attr_flags &CE_ATTR_POLL ) -continue ; - -ath10k_ce_per_engine_handler_adjust (ce_state); - } +for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) +ath10k_ce_enable_interrupt (ar, ce_id); }EXPORT_SYMBOL (ath10k_ce_enable_interrupts);
drivers/net/wireless/ath/ath10k/ce.h +6 -8
@@ -255,12 +255,13 @@ int ath10k_ce_cancel_send_next(struct at/*==================CE Interrupt Handlers====================*/ void ath10k_ce_per_engine_service_any (struct ath10k *ar);void ath10k_ce_per_engine_service (struct ath10k *ar,unsigned int ce_id); -int ath10k_ce_disable_interrupts (struct ath10k *ar); +void ath10k_ce_disable_interrupt (struct ath10k *ar,int ce_id); +void ath10k_ce_disable_interrupts (struct ath10k *ar); +void ath10k_ce_enable_interrupt (struct ath10k *ar,int ce_id);void ath10k_ce_enable_interrupts (struct ath10k *ar);void ath10k_ce_dump_registers (struct ath10k *ar,struct ath10k_fw_crash_data *crash_data); -u32 ath10k_ce_gen_interrupt_summary (struct ath10k *ar);void ath10k_ce_alloc_rri (struct ath10k *ar);void ath10k_ce_free_rri (struct ath10k *ar);
@@ -376,12 +377,9 @@ static inline u32 ath10k_ce_interrupt_su {struct ath10k_ce *ce =ath10k_ce_priv (ar); -if (!ar->hw_params.per_ce_irq) -return CE_WRAPPER_INTERRUPT_SUMMARY_HOST_MSI_GET ( - ce->bus_ops->read32 ((ar),CE_WRAPPER_BASE_ADDRESS + -CE_WRAPPER_INTERRUPT_SUMMARY_ADDRESS )); -else -return ath10k_ce_gen_interrupt_summary (ar); +return CE_WRAPPER_INTERRUPT_SUMMARY_HOST_MSI_GET ( + ce->bus_ops->read32 ((ar),CE_WRAPPER_BASE_ADDRESS + +CE_WRAPPER_INTERRUPT_SUMMARY_ADDRESS )); }/* Host software's Copy Engine configuration. */
drivers/net/wireless/ath/ath10k/snoc.c +14 -5
@@ -3,6 +3,7 @@* Copyright (c) 2018 The Linux Foundation. All rights reserved. */ +#include <linux/bits.h> #include <linux/clk.h> #include <linux/kernel.h> #include <linux/module.h>
@@ -927,6 +928,7 @@ static int ath10k_snoc_hif_start(struct {struct ath10k_snoc *ar_snoc =ath10k_snoc_priv (ar); +bitmap_clear (ar_snoc->pending_ce_irqs,0 ,CE_COUNT_MAX );napi_enable (&ar->napi);ath10k_snoc_irq_enable (ar);ath10k_snoc_rx_post (ar);
@@ -1166,7 +1168,9 @@ static irqreturn_t ath10k_snoc_per_enginreturn IRQ_HANDLED ; } -ath10k_snoc_irq_disable (ar); +ath10k_ce_disable_interrupt (ar, ce_id); +set_bit (ce_id, ar_snoc->pending_ce_irqs); +napi_schedule (&ar->napi);return IRQ_HANDLED ;
@@ -1175,20 +1179,25 @@ static irqreturn_t ath10k_snoc_per_enginstatic int ath10k_snoc_napi_poll (struct napi_struct *ctx,int budget) { struct ath10k *ar = container_of(ctx, struct ath10k, napi); +struct ath10k_snoc *ar_snoc =ath10k_snoc_priv (ar);int done =0 ; +int ce_id;if (test_bit (ATH10K_FLAG_CRASH_FLUSH , &ar->dev_flags)) {napi_complete (ctx);return done; } -ath10k_ce_per_engine_service_any (ar); +for (ce_id =0 ; ce_id <CE_COUNT ; ce_id++) +if (test_and_clear_bit (ce_id, ar_snoc->pending_ce_irqs)) { +ath10k_ce_per_engine_service (ar, ce_id); +ath10k_ce_enable_interrupt (ar, ce_id); + } + done =ath10k_htt_txrx_compl_task (ar, budget); -if (done < budget) { +if (done < budget)napi_complete (ctx); -ath10k_snoc_irq_enable (ar); - }return done; }
drivers/net/wireless/ath/ath10k/snoc.h +1 -0
@@ -81,6 +81,7 @@ struct ath10k_snoc {struct ath10k_clk_info *clk;struct ath10k_qmi *qmi;unsigned long flags; +DECLARE_BITMAP (pending_ce_irqs,CE_COUNT_MAX ); };static inline struct ath10k_snoc *ath10k_snoc_priv (struct ath10k *ar)
Greg Kroah-Hartman
maintainerPATCH 31/38 ath10k: Get rid of "per_ce_irq" hw param
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Douglas Anderson <[email protected]>
[ Upstream commit 7f86551665121931ecd6d327e019e7a69782bfcd ]
As of the patch ("ath10k: Keep track of which interrupts fired, don't poll them") we now have no users of this hardware parameter. Remove it.
Stable-dep-of: 170c75d43a77 ("ath10k: Don't touch the CE interrupt registers after power up")
--- drivers/net/wireless/ath/ath10k/core.c | 13 ------------- drivers/net/wireless/ath/ath10k/hw.h | 3 --- 2 files changed, 16 deletions(-)
drivers/net/wireless/ath/ath10k/core.c +0 -13
@@ -118,7 +118,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -154,7 +153,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -217,7 +215,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -252,7 +249,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -287,7 +283,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -325,7 +320,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -366,7 +360,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -414,7 +407,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -459,7 +451,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -494,7 +485,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -531,7 +521,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -573,7 +562,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = 0x20, .target_64bit = false, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL, - .per_ce_irq = false, .shadow_reg_support = false, .rri_on_ddr = false, .hw_filter_reset_required = true,
@@ -601,7 +589,6 @@ static const struct ath10k_hw_params ath .num_wds_entries = TARGET_HL_TLV_NUM_WDS_ENTRIES, .target_64bit = true, .rx_ring_fill_level = HTT_RX_RING_FILL_LEVEL_DUAL_MAC, - .per_ce_irq = true, .shadow_reg_support = true, .rri_on_ddr = true, .hw_filter_reset_required = false,
drivers/net/wireless/ath/ath10k/hw.h +0 -3
@@ -590,9 +590,6 @@ struct ath10k_hw_params {/* Target rx ring fill level */ u32 rx_ring_fill_level; -/* target supporting per ce IRQ */ -bool per_ce_irq; -/* target supporting shadow register for ce write */ bool shadow_reg_support;
Greg Kroah-Hartman
maintainerPATCH 32/38 net: tls, update curr on splice as well
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: John Fastabend <[email protected]>
commit c5a595000e2677e865a39f249c056bc05d6e55fd upstream.
The curr pointer must also be updated on the splice similar to how we do this for other copy types.
--- net/tls/tls_sw.c | 2 ++ 1 file changed, 2 insertions(+)
net/tls/tls_sw.c +2 -0
@@ -1211,6 +1211,8 @@ alloc_payload: }sk_msg_page_add (msg_pl, page, copy, offset); + msg_pl->sg.copybreak =0 ; + msg_pl->sg.curr = msg_pl->sg.end;sk_mem_charge (sk, copy); offset += copy;
Greg Kroah-Hartman
maintainerPATCH 33/38 netfilter: nf_tables: Reject tables of unsupported family
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Phil Sutter <[email protected]>
commit f1082dd31fe461d482d69da2a8eccfeb7bf07ac2 upstream.
An nftables family is merely a hollow container, its family just a number and such not reliant on compile-time options other than nftables support itself. Add an artificial check so attempts at using a family the kernel can't support fail as early as possible. This helps user space detect kernels which lack e.g. NFPROTO_INET.
--- net/netfilter/nf_tables_api.c | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+)
net/netfilter/nf_tables_api.c +27 -0
@@ -1005,6 +1005,30 @@ static int nft_objname_hash_cmp(struct rreturn strcmp (obj->key.name, k->name); } +static bool nft_supported_family (u8 family) +{ + return false +#ifdef CONFIG_NF_TABLES_INET + || family ==NFPROTO_INET +#endif +#ifdef CONFIG_NF_TABLES_IPV4 + || family ==NFPROTO_IPV4 +#endif +#ifdef CONFIG_NF_TABLES_ARP + || family ==NFPROTO_ARP +#endif +#ifdef CONFIG_NF_TABLES_NETDEV + || family == NFPROTO_NETDEV +#endif +#if IS_ENABLED(CONFIG_NF_TABLES_BRIDGE) + || family ==NFPROTO_BRIDGE +#endif +#ifdef CONFIG_NF_TABLES_IPV6 + || family ==NFPROTO_IPV6 +#endif + ; +} + static int nf_tables_newtable(struct net *net, struct sock *nlsk, struct sk_buff *skb, const struct nlmsghdr *nlh, const struct nlattr * const nla[],
@@ -1020,6 +1044,9 @@ static int nf_tables_newtable(struct netstruct nft_ctx ctx;int err; +if (!nft_supported_family (family)) +return -EOPNOTSUPP ; +lockdep_assert_held (&nft_net->commit_mutex); attr = nla[NFTA_TABLE_NAME ]; table =nft_table_lookup (net, attr, family, genmask);
Greg Kroah-Hartman
maintainerPATCH 34/38 PCI: Extract ATS disabling to a helper function
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Bartosz Pawlowski <[email protected]>
commit f18b1137d38c091cc8c16365219f0a1d4a30b3d1 upstream.
Introduce quirk_no_ats() helper function to provide a standard way to disable ATS capability in PCI quirks.
--- drivers/pci/quirks.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)
drivers/pci/quirks.c +7 -2
@@ -5389,6 +5389,12 @@ DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SEDECLARE_PCI_FIXUP_EARLY (PCI_VENDOR_ID_SERVERWORKS ,0x0422 , quirk_no_ext_tags);#ifdef CONFIG_PCI_ATS +static void quirk_no_ats (struct pci_dev *pdev) +{ +pci_info (pdev,"disabling ATS\n" ); + pdev->ats_cap =0 ; +} +/* * Some devices require additional driver setup to enable ATS. Don't use * ATS for those devices as ATS will be enabled before the driver has had a
@@ -5401,8 +5407,7 @@ static void quirk_amd_harvest_no_ats(str (pdev->device == 0x7341 && pdev->revision != 0x00)) return; -pci_info (pdev,"disabling ATS\n" ); - pdev->ats_cap =0 ; +quirk_no_ats (pdev); }/* AMD Stoney platform GPU */
Greg Kroah-Hartman
maintainerPATCH 35/38 PCI: Disable ATS for specific Intel IPU E2000 devices
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Bartosz Pawlowski <[email protected]>
commit a18615b1cfc04f00548c60eb9a77e0ce56e848fd upstream.
Due to a hardware issue in A and B steppings of Intel IPU E2000, it expects wrong endianness in ATS invalidation message body. This problem can lead to outdated translations being returned as valid and finally cause system instability.
To prevent such issues, add quirk_intel_e2000_no_ats() to disable ATS for vulnerable IPU E2000 devices.
--- drivers/pci/quirks.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)
drivers/pci/quirks.c +19 -0
@@ -5419,6 +5419,25 @@ DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AT/* AMD Navi14 dGPU */ DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_ATI ,0x7340 , quirk_amd_harvest_no_ats);DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_ATI ,0x7341 , quirk_amd_harvest_no_ats); + +/* +* Intel IPU E2000 revisions before C0 implement incorrect endianness +* in ATS Invalidate Request message body. Disable ATS for those devices. +*/ +static void quirk_intel_e2000_no_ats (struct pci_dev *pdev) +{ +if (pdev->revision <0x20 ) +quirk_no_ats (pdev); +} +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1451 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1452 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1453 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1454 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1455 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1457 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x1459 , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x145a , quirk_intel_e2000_no_ats); +DECLARE_PCI_FIXUP_FINAL (PCI_VENDOR_ID_INTEL ,0x145c , quirk_intel_e2000_no_ats);#endif /* CONFIG_PCI_ATS */ /* Freescale PCIe doesn't support MSI in RC mode */
Greg Kroah-Hartman
maintainerPATCH 36/38 net/dst: use a smaller percpu_counter batch for dst entries accounting
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Eric Dumazet <[email protected]>
commit cf86a086a18095e33e0637cb78cda1fcf5280852 upstream.
percpu_counter_add() uses a default batch size which is quite big on platforms with 256 cpus. (2*256 -> 512)
This means dst_entries_get_fast() can be off by +/- 2*(nr_cpus^2) (131072 on servers with 256 cpus)
Reduce the batch size to something more reasonable, and add logic to ip6_dst_gc() to call dst_entries_get_slow() before calling the _very_ expensive fib6_run_gc() function.
--- include/net/dst_ops.h | 4 +++- net/core/dst.c | 8 ++++---- net/ipv6/route.c | 3 +++ 3 files changed, 10 insertions(+), 5 deletions(-)
include/net/dst_ops.h +3 -1
@@ -53,9 +53,11 @@ static inline int dst_entries_get_slow(sreturn percpu_counter_sum_positive (&dst->pcpuc_entries); } +#define DST_PERCPU_COUNTER_BATCH 32static inline void dst_entries_add (struct dst_ops *dst,int val) { -percpu_counter_add (&dst->pcpuc_entries, val); +percpu_counter_add_batch (&dst->pcpuc_entries, val, +DST_PERCPU_COUNTER_BATCH ); }static inline int dst_entries_init (struct dst_ops *dst)
net/core/dst.c +4 -4
@@ -81,11 +81,11 @@ void *dst_alloc(struct dst_ops *ops, str {struct dst_entry *dst; -if (ops->gc &&dst_entries_get_fast (ops) > ops->gc_thresh) { +if (ops->gc && + !(flags &DST_NOCOUNT ) && +dst_entries_get_fast (ops) > ops->gc_thresh) {if (ops->gc (ops)) { -printk_ratelimited (KERN_NOTICE "Route cache is full: " -"consider increasing sysctl " -"net.ipv[4|6].route.max_size.\n" ); +pr_notice_ratelimited ("Route cache is full: consider increasing sysctl net.ipv6.route.max_size.\n" );return NULL ; } }
net/ipv6/route.c +3 -0
@@ -3218,6 +3218,9 @@ static int ip6_dst_gc(struct dst_ops *opint entries; entries =dst_entries_get_fast (ops); +if (entries > rt_max_size) + entries =dst_entries_get_slow (ops); +if (time_after (rt_last_gc + rt_min_interval, jiffies) && entries <= rt_max_size) goto out;
Greg Kroah-Hartman
maintainerPATCH 37/38 ipv6: make ip6_rt_gc_expire an atomic_t
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Eric Dumazet <[email protected]>
commit 9cb7c013420f98fa6fd12fc6a5dc055170c108db upstream.
Reads and Writes to ip6_rt_gc_expire always have been racy, as syzbot reported lately [1]
There is a possible risk of under-flow, leading
to unexpected high value passed to fib6_run_gc(),
although I have not observed this in the field.
Hosts hitting ip6_dst_gc() very hard are under pretty bad state anyway.
[1]
BUG: KCSAN: data-race in ip6_dst_gc / ip6_dst_gc
read-write to 0xffff888102110744 of 4 bytes by task 13165 on cpu 1:
ip6_dst_gc+0x1f3/0x220 net/ipv6/route.c:3311 dst_alloc+0x9b/0x160 net/core/dst.c:86 ip6_dst_alloc net/ipv6/route.c:344 [inline] icmp6_dst_alloc+0xb2/0x360 net/ipv6/route.c:3261 mld_sendpack+0x2b9/0x580 net/ipv6/mcast.c:1807 mld_send_cr net/ipv6/mcast.c:2119 [inline] mld_ifc_work+0x576/0x800 net/ipv6/mcast.c:2651 process_one_work+0x3d3/0x720 kernel/workqueue.c:2289 worker_thread+0x618/0xa70 kernel/workqueue.c:2436 kthread+0x1a9/0x1e0 kernel/kthread.c:376 ret_from_fork+0x1f/0x30
read-write to 0xffff888102110744 of 4 bytes by task 11607 on cpu 0:
ip6_dst_gc+0x1f3/0x220 net/ipv6/route.c:3311 dst_alloc+0x9b/0x160 net/core/dst.c:86 ip6_dst_alloc net/ipv6/route.c:344 [inline] icmp6_dst_alloc+0xb2/0x360 net/ipv6/route.c:3261 mld_sendpack+0x2b9/0x580 net/ipv6/mcast.c:1807 mld_send_cr net/ipv6/mcast.c:2119 [inline] mld_ifc_work+0x576/0x800 net/ipv6/mcast.c:2651 process_one_work+0x3d3/0x720 kernel/workqueue.c:2289 worker_thread+0x618/0xa70 kernel/workqueue.c:2436 kthread+0x1a9/0x1e0 kernel/kthread.c:376 ret_from_fork+0x1f/0x30
value changed: 0x00000bb3 -> 0x00000ba9
Reported by Kernel Concurrency Sanitizer on:
CPU: 0 PID: 11607 Comm: kworker/0:21 Not tainted 5.18.0-rc1-syzkaller-00037-g42e7a03d3bad-dirty #0 Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011 Workqueue: mld mld_ifc_work
--- include/net/netns/ipv6.h | 4 ++-- net/ipv6/route.c | 11 ++++++----- 2 files changed, 8 insertions(+), 7 deletions(-)
include/net/netns/ipv6.h +2 -2
@@ -78,8 +78,8 @@ struct netns_ipv6 {struct dst_ops ip6_dst_ops;rwlock_t fib6_walker_lock;spinlock_t fib6_gc_lock; -unsigned int ip6_rt_gc_expire; -unsigned long ip6_rt_last_gc; +atomic_t ip6_rt_gc_expire; +unsigned long ip6_rt_last_gc;#ifdef CONFIG_IPV6_MULTIPLE_TABLES unsigned int fib6_rules_require_fldissect;bool fib6_has_custom_rules;
net/ipv6/route.c +6 -5
@@ -3215,6 +3215,7 @@ static int ip6_dst_gc(struct dst_ops *opint rt_elasticity = net->ipv6.sysctl.ip6_rt_gc_elasticity;int rt_gc_timeout = net->ipv6.sysctl.ip6_rt_gc_timeout;unsigned long rt_last_gc = net->ipv6.ip6_rt_last_gc; +unsigned int val;int entries; entries =dst_entries_get_fast (ops);
@@ -3225,13 +3226,13 @@ static int ip6_dst_gc(struct dst_ops *op entries <= rt_max_size) goto out; - net->ipv6.ip6_rt_gc_expire++; -fib6_run_gc (net->ipv6.ip6_rt_gc_expire, net, true); +fib6_run_gc (atomic_inc_return (&net->ipv6.ip6_rt_gc_expire), net, true); entries =dst_entries_get_slow (ops);if (entries < ops->gc_thresh) - net->ipv6.ip6_rt_gc_expire = rt_gc_timeout>>1 ; +atomic_set (&net->ipv6.ip6_rt_gc_expire, rt_gc_timeout >>1 ); out: - net->ipv6.ip6_rt_gc_expire -= net->ipv6.ip6_rt_gc_expire>>rt_elasticity; + val =atomic_read (&net->ipv6.ip6_rt_gc_expire); +atomic_set (&net->ipv6.ip6_rt_gc_expire, val - (val >> rt_elasticity));return entries > rt_max_size; }
@@ -6329,7 +6330,7 @@ static int __net_init ip6_route_net_init net->ipv6.sysctl.ip6_rt_min_advmss =IPV6_MIN_MTU -20 -40 ; net->ipv6.sysctl.skip_notify_on_dev_down =0 ; - net->ipv6.ip6_rt_gc_expire =30 *HZ ; +atomic_set (&net->ipv6.ip6_rt_gc_expire,30 *HZ ); ret =0 ; out:
Greg Kroah-Hartman
maintainerPATCH 38/38 ipv6: remove max_size check inline with ipv4
In reply to Greg Kroah-Hartman5.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jon Maxwell <[email protected]>
commit af6d10345ca76670c1b7c37799f0d5576ccef277 upstream.
In ip6_dst_gc() replace:
if (entries > gc_thresh)
With:
if (entries > ops->gc_thresh)
Sending Ipv6 packets in a loop via a raw socket triggers an issue where a route is cloned by ip6_rt_cache_alloc() for each packet sent. This quickly consumes the Ipv6 max_size threshold which defaults to 4096 resulting in these warnings:
[1] 99.187805] dst_alloc: 7728 callbacks suppressed
[2] Route cache is full: consider increasing sysctl net.ipv6.route.max_size. .
.
[300] Route cache is full: consider increasing sysctl net.ipv6.route.max_size.
When this happens the packet is dropped and sendto() gets a network is unreachable error:
remaining pkt 200557 errno 101
remaining pkt 196462 errno 101
.
.
remaining pkt 126821 errno 101
Implement David Aherns suggestion to remove max_size check seeing that Ipv6 has a GC to manage memory usage. Ipv4 already does not check max_size.
Here are some memory comparisons for Ipv4 vs Ipv6 with the patch:
Test by running 5 instances of a program that sends UDP packets to a raw socket 5000000 times. Compare Ipv4 and Ipv6 performance with a similar program.
Ipv4:
Before test:
MemFree: 29427108 kB Slab: 237612 kB
ip6_dst_cache 1912 2528 256 32 2 : tunables 0 0 0 xfrm_dst_cache 0 0 320 25 2 : tunables 0 0 0 ip_dst_cache 2881 3990 192 42 2 : tunables 0 0 0
During test:
MemFree: 29417608 kB Slab: 247712 kB
ip6_dst_cache 1912 2528 256 32 2 : tunables 0 0 0 xfrm_dst_cache 0 0 320 25 2 : tunables 0 0 0 ip_dst_cache 44394 44394 192 42 2 : tunables 0 0 0
After test:
MemFree: 29422308 kB Slab: 238104 kB
ip6_dst_cache 1912 2528 256 32 2 : tunables 0 0 0 xfrm_dst_cache 0 0 320 25 2 : tunables 0 0 0 ip_dst_cache 3048 4116 192 42 2 : tunables 0 0 0
Ipv6 with patch:
Errno 101 errors are not observed anymore with the patch.
Before test:
MemFree: 29422308 kB Slab: 238104 kB
ip6_dst_cache 1912 2528 256 32 2 : tunables 0 0 0 xfrm_dst_cache 0 0 320 25 2 : tunables 0 0 0 ip_dst_cache 3048 4116 192 42 2 : tunables 0 0 0
During Test:
MemFree: 29431516 kB Slab: 240940 kB
ip6_dst_cache 11980 12064 256 32 2 : tunables 0 0 0 xfrm_dst_cache 0 0 320 25 2 : tunables 0 0 0 ip_dst_cache 3048 4116 192 42 2 : tunables 0 0 0
After Test:
MemFree: 29441816 kB Slab: 238132 kB
ip6_dst_cache 1902 2432 256 32 2 : tunables 0 0 0 xfrm_dst_cache 0 0 320 25 2 : tunables 0 0 0 ip_dst_cache 3048 4116 192 42 2 : tunables 0 0 0
--- include/net/dst_ops.h | 2 +- net/core/dst.c | 8 ++------ net/ipv6/route.c | 13 +++++-------- 3 files changed, 8 insertions(+), 15 deletions(-)
include/net/dst_ops.h +1 -1
@@ -16,7 +16,7 @@ struct dst_ops {unsigned short family;unsigned int gc_thresh; -int (*gc)(struct dst_ops *ops); +void (*gc)(struct dst_ops *ops);struct dst_entry * (*check)(struct dst_entry *,__u32 cookie);unsigned int (*default_advmss)(const struct dst_entry *);unsigned int (*mtu)(const struct dst_entry *);
net/core/dst.c +2 -6
@@ -83,12 +83,8 @@ void *dst_alloc(struct dst_ops *ops, strif (ops->gc && !(flags &DST_NOCOUNT ) && -dst_entries_get_fast (ops) > ops->gc_thresh) { -if (ops->gc (ops)) { -pr_notice_ratelimited ("Route cache is full: consider increasing sysctl net.ipv6.route.max_size.\n" ); -return NULL ; - } - } +dst_entries_get_fast (ops) > ops->gc_thresh) + ops->gc (ops); dst =kmem_cache_alloc (ops->kmem_cachep,GFP_ATOMIC );if (!dst)
net/ipv6/route.c +5 -8
@@ -88,7 +88,7 @@ static struct dst_entry *ip6_negative_adstatic void ip6_dst_destroy (struct dst_entry *);static void ip6_dst_ifdown (struct dst_entry *,struct net_device *dev,int how); -static int ip6_dst_gc (struct dst_ops *ops); +static void ip6_dst_gc (struct dst_ops *ops);static int ip6_pkt_discard (struct sk_buff *skb);static int ip6_pkt_discard_out (struct net *net,struct sock *sk,struct sk_buff *skb);
@@ -3207,11 +3207,10 @@ out:return dst; } -static int ip6_dst_gc (struct dst_ops *ops) +static void ip6_dst_gc (struct dst_ops *ops) { struct net *net = container_of(ops, struct net, ipv6.ip6_dst_ops);int rt_min_interval = net->ipv6.sysctl.ip6_rt_gc_min_interval; -int rt_max_size = net->ipv6.sysctl.ip6_rt_max_size;int rt_elasticity = net->ipv6.sysctl.ip6_rt_gc_elasticity;int rt_gc_timeout = net->ipv6.sysctl.ip6_rt_gc_timeout;unsigned long rt_last_gc = net->ipv6.ip6_rt_last_gc;
@@ -3219,11 +3218,10 @@ static int ip6_dst_gc(struct dst_ops *opint entries; entries =dst_entries_get_fast (ops); -if (entries > rt_max_size) +if (entries > ops->gc_thresh) entries =dst_entries_get_slow (ops); -if (time_after (rt_last_gc + rt_min_interval, jiffies) && - entries <= rt_max_size) +if (time_after (rt_last_gc + rt_min_interval, jiffies)) goto out;fib6_run_gc (atomic_inc_return (&net->ipv6.ip6_rt_gc_expire), net, true);
@@ -3233,7 +3231,6 @@ static int ip6_dst_gc(struct dst_ops *op out: val =atomic_read (&net->ipv6.ip6_rt_gc_expire);atomic_set (&net->ipv6.ip6_rt_gc_expire, val - (val >> rt_elasticity)); -return entries > rt_max_size; } static int ip6_nh_lookup_table(struct net *net, struct fib6_config *cfg,
@@ -6321,7 +6318,7 @@ static int __net_init ip6_route_net_init#endif net->ipv6.sysctl.flush_delay =0 ; - net->ipv6.sysctl.ip6_rt_max_size =4096 ; + net->ipv6.sysctl.ip6_rt_max_size =INT_MAX ; net->ipv6.sysctl.ip6_rt_gc_min_interval =HZ /2 ; net->ipv6.sysctl.ip6_rt_gc_timeout =60 *HZ ; net->ipv6.sysctl.ip6_rt_gc_interval =30 *HZ ;
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.267 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 Mon, 15 Jan 2024 09:41:55 +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.267-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.
## Build
* kernel: 5.4.267-rc1
* git: https://gitlab.com/Linaro/lkft/mirrors/stable/linux-stable-rc
* git branch: linux-5.4.y
* git commit: 86b133f1f903a253af84a135ff1afeec162143e7
* git describe: v5.4.266-39-g86b133f1f903
* test details:
https://qa-reports.linaro.org/lkft/linux-stable-rc-linux-5.4.y/build/v5.4.266-39-g86b133f1f903
## Test Regressions (compared to v5.4.266)
## Metric Regressions (compared to v5.4.266)
## Test Fixes (compared to v5.4.266)
## Metric Fixes (compared to v5.4.266)
## Test result summary
total: 176285, pass: 139633, fail: 3376, skip: 33179, xfail: 97
## Build Summary
* arc: 10 total, 10 passed, 0 failed
* arm: 285 total, 285 passed, 0 failed
* arm64: 81 total, 77 passed, 4 failed
* i386: 51 total, 39 passed, 12 failed
* mips: 52 total, 52 passed, 0 failed
* parisc: 6 total, 0 passed, 6 failed
* powerpc: 62 total, 62 passed, 0 failed
* riscv: 27 total, 27 passed, 0 failed
* s390: 14 total, 14 passed, 0 failed
* sh: 20 total, 20 passed, 0 failed
* sparc: 14 total, 14 passed, 0 failed
* x86_64: 70 total, 70 passed, 0 failed
## Test suites summary
* boot
* kselftest-android
* kselftest-arm64
* kselftest-breakpoints
* kselftest-capabilities
* kselftest-cgroup
* kselftest-clone3
* kselftest-core
* kselftest-cpu-hotplug
* kselftest-cpufreq
* kselftest-drivers-dma-buf
* kselftest-efivarfs
* kselftest-exec
* kselftest-filesystems
* kselftest-filesystems-binderfs
* kselftest-filesystems-epoll
* kselftest-firmware
* kselftest-fpu
* kselftest-ftrace
* kselftest-futex
* kselftest-gpio
* kselftest-intel_pstate
* kselftest-ipc
* kselftest-ir
* kselftest-kcmp
* kselftest-kexec
* kselftest-kvm
* kselftest-lib
* kselftest-membarrier
* kselftest-memfd
* kselftest-memory-hotplug
* kselftest-mincore
* kselftest-mount
* kselftest-mqueue
* kselftest-net
* kselftest-net-forwarding
* kselftest-net-mptcp
* kselftest-netfilter
* kselftest-nsfs
* kselftest-openat2
* kselftest-pid_namespace
* kselftest-pidfd
* kselftest-proc
* kselftest-pstore
* kselftest-ptrace
* kselftest-rseq
* kselftest-rtc
* kselftest-sigaltstack
* kselftest-size
* kselftest-tc-testing
* kselftest-timens
* kselftest-tmpfs
* kselftest-tpm2
* kselftest-user
* kselftest-user_events
* kselftest-vDSO
* kselftest-vm
* kselftest-x86
* kselftest-zram
* kunit
* log-parser-boot
* log-parser-test
* ltp-cap_bounds
* ltp-commands
* ltp-containers
* ltp-controllers
* ltp-cpuhotplug
* ltp-crypto
* ltp-cve
* ltp-dio
* ltp-fcntl-locktests
* ltp-filecaps
* ltp-fs
* ltp-fs_bind
* ltp-fs_perms_simple
* ltp-fsx
* ltp-hugetlb
* ltp-io
* ltp-ipc
* ltp-math
* ltp-mm
* ltp-nptl
* ltp-pty
* ltp-sched
* ltp-securebits
* ltp-smoke
* ltp-syscalls
* ltp-tracing
* perf
* rcutorture
--
Linaro LKFT
https://lkft.linaro.org
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.267 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 Mon, 15 Jan 2024 09:41:55 +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.267-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 passing for Tegra ...
Test results for stable-v5.4:
10 builds: 10 pass, 0 fail
24 boots: 24 pass, 0 fail
54 tests: 54 pass, 0 failLinux version: 5.4.267-rc1-g86b133f1f903
Boards tested: tegra124-jetson-tk1, tegra186-p2771-0000,
tegra194-p2972-0000, tegra20-ventana,
tegra210-p2371-2180, tegra210-p3450-0000,
tegra30-cardhu-a04Jon
Hi Greg,
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.267 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 Mon, 15 Jan 2024 09:41:55 +0000. Anything received after that time might be too late.
No problems seen on x86_64 and aarch64 with our testing.
Thanks,
Harshit
Quoted 9 lines
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.267-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
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 5.4.267 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 Mon, 15 Jan 2024 09:41:55 +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.267-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
On ARCH_BRCMSTB using 32-bit and 64-bit ARM kernels, build tested on BMIPS_GENERIC:
Signature · 1 line
Florian