[openwrt/openwrt] kernel: remove unnecessary change from backlog threading fix

LEDE Commits lede-commits at lists.infradead.org
Tue Mar 28 04:19:31 PDT 2023


nbd pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/77f11d2772dcee35b4956d1d36d95b369b6b719c

commit 77f11d2772dcee35b4956d1d36d95b369b6b719c
Author: Felix Fietkau <nbd at nbd.name>
AuthorDate: Tue Mar 28 13:13:53 2023 +0200

    kernel: remove unnecessary change from backlog threading fix
    
    Fixes: f1c9bd6f6f41 ("kernel: fix kernel warning and network issue in the backlog threading patch")
    Signed-off-by: Felix Fietkau <nbd at nbd.name>
---
 ...d-optional-threading-for-backlog-processi.patch | 28 +++++++---------------
 1 file changed, 9 insertions(+), 19 deletions(-)

diff --git a/target/linux/generic/pending-5.15/760-net-core-add-optional-threading-for-backlog-processi.patch b/target/linux/generic/pending-5.15/760-net-core-add-optional-threading-for-backlog-processi.patch
index 6a13b3375c..fc83416b2a 100644
--- a/target/linux/generic/pending-5.15/760-net-core-add-optional-threading-for-backlog-processi.patch
+++ b/target/linux/generic/pending-5.15/760-net-core-add-optional-threading-for-backlog-processi.patch
@@ -30,17 +30,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  #endif
 --- a/net/core/dev.c
 +++ b/net/core/dev.c
-@@ -4558,7 +4558,8 @@ static void rps_trigger_softirq(void *da
- {
- 	struct softnet_data *sd = data;
- 
--	____napi_schedule(sd, &sd->backlog);
-+	if (!__test_and_set_bit(NAPI_STATE_SCHED, &sd->backlog.state))
-+		____napi_schedule(sd, &sd->backlog);
- 	sd->received_rps++;
- }
- 
-@@ -4574,7 +4575,7 @@ static int rps_ipi_queued(struct softnet
+@@ -4574,7 +4574,7 @@ static int rps_ipi_queued(struct softnet
  #ifdef CONFIG_RPS
  	struct softnet_data *mysd = this_cpu_ptr(&softnet_data);
  
@@ -49,7 +39,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  		sd->rps_ipi_next = mysd->rps_ipi_list;
  		mysd->rps_ipi_list = sd;
  
-@@ -5755,6 +5756,8 @@ static DEFINE_PER_CPU(struct work_struct
+@@ -5755,6 +5755,8 @@ static DEFINE_PER_CPU(struct work_struct
  /* Network device is going away, flush any packets still pending */
  static void flush_backlog(struct work_struct *work)
  {
@@ -58,7 +48,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  	struct sk_buff *skb, *tmp;
  	struct softnet_data *sd;
  
-@@ -5770,9 +5773,18 @@ static void flush_backlog(struct work_st
+@@ -5770,9 +5772,18 @@ static void flush_backlog(struct work_st
  			input_queue_head_incr(sd);
  		}
  	}
@@ -77,7 +67,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  	skb_queue_walk_safe(&sd->process_queue, skb, tmp) {
  		if (skb->dev->reg_state == NETREG_UNREGISTERING) {
  			__skb_unlink(skb, &sd->process_queue);
-@@ -5780,7 +5792,18 @@ static void flush_backlog(struct work_st
+@@ -5780,7 +5791,18 @@ static void flush_backlog(struct work_st
  			input_queue_head_incr(sd);
  		}
  	}
@@ -96,7 +86,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  }
  
  static bool flush_required(int cpu)
-@@ -6463,6 +6486,7 @@ static int process_backlog(struct napi_s
+@@ -6463,6 +6485,7 @@ static int process_backlog(struct napi_s
  
  		local_irq_disable();
  		rps_lock(sd);
@@ -104,7 +94,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  		if (skb_queue_empty(&sd->input_pkt_queue)) {
  			/*
  			 * Inline a custom version of __napi_complete().
-@@ -6472,7 +6496,8 @@ static int process_backlog(struct napi_s
+@@ -6472,7 +6495,8 @@ static int process_backlog(struct napi_s
  			 * We can use a plain write instead of clear_bit(),
  			 * and we dont need an smp_mb() memory barrier.
  			 */
@@ -114,7 +104,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  			again = false;
  		} else {
  			skb_queue_splice_tail_init(&sd->input_pkt_queue,
-@@ -6889,6 +6914,57 @@ int dev_set_threaded(struct net_device *
+@@ -6889,6 +6913,57 @@ int dev_set_threaded(struct net_device *
  }
  EXPORT_SYMBOL(dev_set_threaded);
  
@@ -172,7 +162,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  void netif_napi_add(struct net_device *dev, struct napi_struct *napi,
  		    int (*poll)(struct napi_struct *, int), int weight)
  {
-@@ -11367,6 +11443,9 @@ static int dev_cpu_dead(unsigned int old
+@@ -11367,6 +11442,9 @@ static int dev_cpu_dead(unsigned int old
  	raise_softirq_irqoff(NET_TX_SOFTIRQ);
  	local_irq_enable();
  
@@ -182,7 +172,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
  #ifdef CONFIG_RPS
  	remsd = oldsd->rps_ipi_list;
  	oldsd->rps_ipi_list = NULL;
-@@ -11706,6 +11785,7 @@ static int __init net_dev_init(void)
+@@ -11706,6 +11784,7 @@ static int __init net_dev_init(void)
  		sd->cpu = i;
  #endif
  




More information about the lede-commits mailing list