From: Sasha Levin <sashal@kernel.org>
To: stable@vger.kernel.org, florian.fainelli@broadcom.com
Cc: Sasha Levin <sashal@kernel.org>
Subject: Re: [PATCH stable 5.15 v2 1/2] net: openvswitch: fix race on port output
Date: Wed, 26 Mar 2025 13:55:07 -0400 [thread overview]
Message-ID: <20250326132132-157910064b9f9bd2@stable.kernel.org> (raw)
In-Reply-To: <20250326155856.4133986-2-florian.fainelli@broadcom.com>
[ Sasha's backport helper bot ]
Hi,
Summary of potential issues:
⚠️ Found follow-up fixes in mainline
The upstream commit SHA1 provided is correct: 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8
WARNING: Author mismatch between patch and upstream commit:
Backport author: Florian Fainelli<florian.fainelli@broadcom.com>
Commit author: Felix Huettner<felix.huettner@mail.schwarz>
Status in newer kernel trees:
6.13.y | Present (exact SHA1)
6.12.y | Present (exact SHA1)
6.6.y | Present (exact SHA1)
6.1.y | Present (different SHA1: 644b3051b06b)
Found fixes commits:
47e55e4b410f openvswitch: fix lockup on tx to unregistering netdev with carrier
Note: The patch differs from the upstream commit:
---
1: 066b86787fa3d ! 1: 3e36758ea4208 net: openvswitch: fix race on port output
@@ Metadata
## Commit message ##
net: openvswitch: fix race on port output
+ [ Upstream commit 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8 ]
+
assume the following setup on a single machine:
1. An openvswitch instance with one bridge and default flows
2. two network namespaces "server" and "client"
@@ Commit message
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Link: https://lore.kernel.org/r/ZC0pBXBAgh7c76CA@kernel-bug-kernel-bug
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
+ Signed-off-by: Carlos Soto <carlos.soto@broadcom.com>
+ Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com>
## net/core/dev.c ##
@@ net/core/dev.c: static u16 skb_tx_hash(const struct net_device *dev,
}
if (skb_rx_queue_recorded(skb)) {
-+ DEBUG_NET_WARN_ON_ONCE(qcount == 0);
++ BUILD_BUG_ON_INVALID(qcount == 0);
hash = skb_get_rx_queue(skb);
if (hash >= qoffset)
hash -= qoffset;
---
Results of testing on various branches:
| Branch | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-5.15.y | Success | Success |
next prev parent reply other threads:[~2025-03-26 17:55 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-26 15:58 [PATCH stable 5.15 v3 0/2] openvswitch port output fixes Florian Fainelli
2025-03-26 15:58 ` [PATCH stable 5.15 v2 1/2] net: openvswitch: fix race on port output Florian Fainelli
2025-03-26 17:55 ` Sasha Levin [this message]
2025-03-26 15:58 ` [PATCH stable 5.15 v2 2/2] openvswitch: fix lockup on tx to unregistering netdev with carrier Florian Fainelli
2025-03-26 17:54 ` Sasha Levin
-- strict thread matches above, loose matches on Subject: below --
2025-03-25 19:22 [PATCH stable 5.15 v2 0/2] openvswitch port output fixes Florian Fainelli
2025-03-25 19:22 ` [PATCH stable 5.15 v2 1/2] net: openvswitch: fix race on port output Florian Fainelli
2025-03-26 2:23 ` Sasha Levin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250326132132-157910064b9f9bd2@stable.kernel.org \
--to=sashal@kernel.org \
--cc=florian.fainelli@broadcom.com \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.