mirror of
https://git.openwrt.org/openwrt/openwrt.git
synced 2025-12-10 08:44:39 +01:00
1c92e468d5
Changelog: https://cdn.kernel.org/pub/linux/kernel/v6.x/ChangeLog-6.6.103 removed upstream patches: generic-backport/220-v6.16-powerpc-boot-fix-build-with-gcc-15.patch [1] generic-backport/847-v6.17-Revert-leds-trigger-netdev-Configure-LED-blink-inter.patch [2] update patch to upstream function change bcm53x/patches-6.6/180-usb-xhci-add-support-for-performing-fake-doorbell.patch changed function xhci_disable_and_free_slot() upstream [3] All other patches auto-refreshed. [1] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.6.103&id=93879b3ba967a33834727abf34ea08764339fe0b [2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.6.103&id=c66caf21b1d0a0847adc34d368e3f6753a2cbd53 [3] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/drivers/usb/host?h=v6.6.103&id=e600de541c37f97482fea2a7a26f186141e7ddea Suggested-by: Leo Barsky <leobrsky@proton.me> Signed-off-by: Goetz Goerisch <ggoerisch@gmail.com> Link: https://github.com/openwrt/openwrt/pull/19898 Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
75 lines
2.7 KiB
Diff
75 lines
2.7 KiB
Diff
From 56364c910691f6d10ba88c964c9041b9ab777bd6 Mon Sep 17 00:00:00 2001
|
|
From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
|
|
Date: Mon, 25 Mar 2024 08:40:28 +0100
|
|
Subject: [PATCH 1/4] net: Remove conditional threaded-NAPI wakeup based on
|
|
task state.
|
|
|
|
A NAPI thread is scheduled by first setting NAPI_STATE_SCHED bit. If
|
|
successful (the bit was not yet set) then the NAPI_STATE_SCHED_THREADED
|
|
is set but only if thread's state is not TASK_INTERRUPTIBLE (is
|
|
TASK_RUNNING) followed by task wakeup.
|
|
|
|
If the task is idle (TASK_INTERRUPTIBLE) then the
|
|
NAPI_STATE_SCHED_THREADED bit is not set. The thread is no relying on
|
|
the bit but always leaving the wait-loop after returning from schedule()
|
|
because there must have been a wakeup.
|
|
|
|
The smpboot-threads implementation for per-CPU threads requires an
|
|
explicit condition and does not support "if we get out of schedule()
|
|
then there must be something to do".
|
|
|
|
Removing this optimisation simplifies the following integration.
|
|
|
|
Set NAPI_STATE_SCHED_THREADED unconditionally on wakeup and rely on it
|
|
in the wait path by removing the `woken' condition.
|
|
|
|
Acked-by: Jakub Kicinski <kuba@kernel.org>
|
|
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
|
|
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
|
|
---
|
|
net/core/dev.c | 14 ++------------
|
|
1 file changed, 2 insertions(+), 12 deletions(-)
|
|
|
|
--- a/net/core/dev.c
|
|
+++ b/net/core/dev.c
|
|
@@ -4526,13 +4526,7 @@ static inline void ____napi_schedule(str
|
|
*/
|
|
thread = READ_ONCE(napi->thread);
|
|
if (thread) {
|
|
- /* Avoid doing set_bit() if the thread is in
|
|
- * INTERRUPTIBLE state, cause napi_thread_wait()
|
|
- * makes sure to proceed with napi polling
|
|
- * if the thread is explicitly woken from here.
|
|
- */
|
|
- if (READ_ONCE(thread->__state) != TASK_INTERRUPTIBLE)
|
|
- set_bit(NAPI_STATE_SCHED_THREADED, &napi->state);
|
|
+ set_bit(NAPI_STATE_SCHED_THREADED, &napi->state);
|
|
wake_up_process(thread);
|
|
return;
|
|
}
|
|
@@ -6688,8 +6682,6 @@ static int napi_poll(struct napi_struct
|
|
|
|
static int napi_thread_wait(struct napi_struct *napi)
|
|
{
|
|
- bool woken = false;
|
|
-
|
|
set_current_state(TASK_INTERRUPTIBLE);
|
|
|
|
while (!kthread_should_stop()) {
|
|
@@ -6698,15 +6690,13 @@ static int napi_thread_wait(struct napi_
|
|
* Testing SCHED bit is not enough because SCHED bit might be
|
|
* set by some other busy poll thread or by napi_disable().
|
|
*/
|
|
- if (test_bit(NAPI_STATE_SCHED_THREADED, &napi->state) || woken) {
|
|
+ if (test_bit(NAPI_STATE_SCHED_THREADED, &napi->state)) {
|
|
WARN_ON(!list_empty(&napi->poll_list));
|
|
__set_current_state(TASK_RUNNING);
|
|
return 0;
|
|
}
|
|
|
|
schedule();
|
|
- /* woken being true indicates this thread owns this napi. */
|
|
- woken = true;
|
|
set_current_state(TASK_INTERRUPTIBLE);
|
|
}
|
|
__set_current_state(TASK_RUNNING);
|