2023-11-28 18:27:18 +00:00
|
|
|
From cf74eb5a5bc867258e7d0b0d1c3c4a60e1e3de2f Mon Sep 17 00:00:00 2001
|
|
|
|
From: Jakub Kicinski <kuba@kernel.org>
|
|
|
|
Date: Mon, 14 Aug 2023 08:35:21 -0700
|
|
|
|
Subject: [PATCH] eth: r8152: try to use a normal budget
|
|
|
|
|
|
|
|
Mario reports that loading r8152 on his system leads to a:
|
|
|
|
|
|
|
|
netif_napi_add_weight() called with weight 256
|
|
|
|
|
|
|
|
warning getting printed. We don't have any solid data
|
|
|
|
on why such high budget was chosen, and it may cause
|
|
|
|
stalls in processing other softirqs and rt threads.
|
|
|
|
So try to switch back to the default (64) weight.
|
|
|
|
|
|
|
|
If this slows down someone's system we should investigate
|
|
|
|
which part of stopping starting the NAPI poll in this
|
|
|
|
driver are expensive.
|
|
|
|
|
|
|
|
Reported-by: Mario Limonciello <mario.limonciello@amd.com>
|
|
|
|
Link: https://lore.kernel.org/all/0bfd445a-81f7-f702-08b0-bd5a72095e49@amd.com/
|
|
|
|
Acked-by: Hayes Wang <hayeswang@realtek.com>
|
|
|
|
Link: https://lore.kernel.org/r/20230814153521.2697982-1-kuba@kernel.org
|
|
|
|
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
|
|
|
|
---
|
|
|
|
drivers/net/usb/r8152.c | 3 +--
|
|
|
|
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
|
|
|
|
|
|
--- a/drivers/net/usb/r8152.c
|
|
|
|
+++ b/drivers/net/usb/r8152.c
|
2023-12-13 19:52:38 +00:00
|
|
|
@@ -9784,8 +9784,7 @@ static int rtl8152_probe(struct usb_inte
|
2023-11-28 18:27:18 +00:00
|
|
|
|
|
|
|
usb_set_intfdata(intf, tp);
|
|
|
|
|
|
|
|
- netif_napi_add_weight(netdev, &tp->napi, r8152_poll,
|
|
|
|
- tp->support_2500full ? 256 : 64);
|
|
|
|
+ netif_napi_add(netdev, &tp->napi, r8152_poll);
|
|
|
|
|
|
|
|
ret = register_netdev(netdev);
|
|
|
|
if (ret != 0) {
|