Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Jan 2024 20:15:24 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: bf81fc51ccf0 - stable/13 - igb(4): Remove disconnected SYSCTL
Message-ID:  <202401182015.40IKFOXp068343@gitrepo.freebsd.org>

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

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

commit bf81fc51ccf04a6ae58f7c4d281266d5ab59d4e0
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:56 +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 bc8c3547230d..e3dc7da65713 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
@@ -119,10 +119,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 3e05d1a7b957..8de044169269 100644
--- a/sys/dev/igc/if_igc.c
+++ b/sys/dev/igc/if_igc.c
@@ -255,13 +255,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,
@@ -467,8 +460,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 382931468ade..fde9429b5360 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.40IKFOXp068343>