Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Jan 2024 20:15:05 GMT
From:      Marius Strobl <marius@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 6d150c4c3f02 - stable/14 - igb(4): Remove disconnected SYSCTL
Message-ID:  <202401182015.40IKF52r067967@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/14 has been updated by marius:

URL: https://cgit.FreeBSD.org/src/commit/?id=6d150c4c3f0258b7d0fbaf0375b328b6ae84a169

commit 6d150c4c3f0258b7d0fbaf0375b328b6ae84a169
Author:     Marius Strobl <marius@FreeBSD.org>
AuthorDate: 2024-01-09 22:01:46 +0000
Commit:     Marius Strobl <marius@FreeBSD.org>
CommitDate: 2024-01-18 20:14:46 +0000

    igb(4): Remove disconnected SYSCTL
    
    The global hw.igb.rx_process_limit knob never was adhered to by the
    in-tree version of this driver but similar functionality is available
    via the device-specific dev.igb.N.iflib.rx_budget.
    
    While at it, remove the - besides initialization of tx_process_limit -
    unused {r,t}x_process_limit members.
    
    (cherry picked from commit f221d35be7a5749e2a1246c81adc9c03c1dc545c)
---
 share/man/man4/igc.4 | 6 +-----
 sys/dev/igc/if_igc.c | 9 ---------
 sys/dev/igc/if_igc.h | 2 --
 3 files changed, 1 insertion(+), 16 deletions(-)

diff --git a/share/man/man4/igc.4 b/share/man/man4/igc.4
index f77581edd00b..cf36e584a679 100644
--- a/share/man/man4/igc.4
+++ b/share/man/man4/igc.4
@@ -3,7 +3,7 @@
 .\" Copyright 2021 Rubicon Communications, LLC (Netgate)
 .\" SPDX-License-Identifier: BSD-3-Clause
 .\"
-.Dd May 10, 2021
+.Dd January 9, 2023
 .Dt IGC 4
 .Os
 .Sh NAME
@@ -118,10 +118,6 @@ maximum delay in which a transmit interrupt is generated.
 .It Va hw.igc.sbp
 Show bad packets when in promiscuous mode.
 Default is false.
-.It Va hw.igc.rx_process_limit
-Maximum number of received packets to process at a time.
-Default is 100.
-A value of -1 means unlimited.
 .It Va hw.igc.eee_setting
 Disable or enable Energy Efficient Ethernet.
 Default 1 (disabled).
diff --git a/sys/dev/igc/if_igc.c b/sys/dev/igc/if_igc.c
index a0f0906e30da..bfa33c82f7ba 100644
--- a/sys/dev/igc/if_igc.c
+++ b/sys/dev/igc/if_igc.c
@@ -254,13 +254,6 @@ static int igc_debug_sbp = true;
 SYSCTL_INT(_hw_igc, OID_AUTO, sbp, CTLFLAG_RDTUN, &igc_debug_sbp, 0,
     "Show bad packets in promiscuous mode");
 
-/* How many packets rxeof tries to clean at a time */
-static int igc_rx_process_limit = 100;
-SYSCTL_INT(_hw_igc, OID_AUTO, rx_process_limit, CTLFLAG_RDTUN,
-    &igc_rx_process_limit, 0,
-    "Maximum number of received packets to process "
-    "at a time, -1 means unlimited");
-
 /* Energy efficient ethernet - default to OFF */
 static int igc_eee_setting = 1;
 SYSCTL_INT(_hw_igc, OID_AUTO, eee_setting, CTLFLAG_RDTUN, &igc_eee_setting, 0,
@@ -466,8 +459,6 @@ igc_if_attach_pre(if_ctx_t ctx)
 	adapter->media = iflib_get_media(ctx);
 	hw = &adapter->hw;
 
-	adapter->tx_process_limit = scctx->isc_ntxd[0];
-
 	/* SYSCTL stuff */
 	SYSCTL_ADD_PROC(device_get_sysctl_ctx(dev),
 	    SYSCTL_CHILDREN(device_get_sysctl_tree(dev)),
diff --git a/sys/dev/igc/if_igc.h b/sys/dev/igc/if_igc.h
index 7a30f5956016..0c22ce9f76f4 100644
--- a/sys/dev/igc/if_igc.h
+++ b/sys/dev/igc/if_igc.h
@@ -372,8 +372,6 @@ struct igc_adapter {
 
         u32		txd_cmd;
 
-        u32             tx_process_limit;
-        u32             rx_process_limit;
 	u32		rx_mbuf_sz;
 
 	/* Management and WOL features */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202401182015.40IKF52r067967>