xref: /freebsd/sys/dev/ena/ena_sysctl.c (revision beaadec9eaec8e6b266faff3b0880a141728fcef)
19b8d05b8SZbigniew Bodek /*-
20835cc78SMarcin Wojtas  * SPDX-License-Identifier: BSD-2-Clause
39b8d05b8SZbigniew Bodek  *
4*beaadec9SMarcin Wojtas  * Copyright (c) 2015-2021 Amazon.com, Inc. or its affiliates.
59b8d05b8SZbigniew Bodek  * All rights reserved.
69b8d05b8SZbigniew Bodek  *
79b8d05b8SZbigniew Bodek  * Redistribution and use in source and binary forms, with or without
89b8d05b8SZbigniew Bodek  * modification, are permitted provided that the following conditions
99b8d05b8SZbigniew Bodek  * are met:
109b8d05b8SZbigniew Bodek  *
119b8d05b8SZbigniew Bodek  * 1. Redistributions of source code must retain the above copyright
129b8d05b8SZbigniew Bodek  *    notice, this list of conditions and the following disclaimer.
139b8d05b8SZbigniew Bodek  *
149b8d05b8SZbigniew Bodek  * 2. Redistributions in binary form must reproduce the above copyright
159b8d05b8SZbigniew Bodek  *    notice, this list of conditions and the following disclaimer in the
169b8d05b8SZbigniew Bodek  *    documentation and/or other materials provided with the distribution.
179b8d05b8SZbigniew Bodek  *
189b8d05b8SZbigniew Bodek  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
199b8d05b8SZbigniew Bodek  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
209b8d05b8SZbigniew Bodek  * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
219b8d05b8SZbigniew Bodek  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
229b8d05b8SZbigniew Bodek  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
239b8d05b8SZbigniew Bodek  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
249b8d05b8SZbigniew Bodek  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
259b8d05b8SZbigniew Bodek  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
269b8d05b8SZbigniew Bodek  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
279b8d05b8SZbigniew Bodek  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
289b8d05b8SZbigniew Bodek  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
299b8d05b8SZbigniew Bodek  */
309b8d05b8SZbigniew Bodek #include <sys/cdefs.h>
319b8d05b8SZbigniew Bodek __FBSDID("$FreeBSD$");
329b8d05b8SZbigniew Bodek 
339b8d05b8SZbigniew Bodek #include "ena_sysctl.h"
349b8d05b8SZbigniew Bodek 
350bdffe59SMarcin Wojtas static void	ena_sysctl_add_wd(struct ena_adapter *);
369b8d05b8SZbigniew Bodek static void	ena_sysctl_add_stats(struct ena_adapter *);
37f180142cSMarcin Wojtas static void	ena_sysctl_add_eni_metrics(struct ena_adapter *);
386064f289SMarcin Wojtas static void	ena_sysctl_add_tuneables(struct ena_adapter *);
396064f289SMarcin Wojtas static int	ena_sysctl_buf_ring_size(SYSCTL_HANDLER_ARGS);
406064f289SMarcin Wojtas static int	ena_sysctl_rx_queue_size(SYSCTL_HANDLER_ARGS);
4156d41ad5SMarcin Wojtas static int	ena_sysctl_io_queues_nb(SYSCTL_HANDLER_ARGS);
42f180142cSMarcin Wojtas static int	ena_sysctl_eni_metrics_interval(SYSCTL_HANDLER_ARGS);
43f180142cSMarcin Wojtas 
44f180142cSMarcin Wojtas /* Limit max ENI sample rate to be an hour. */
45f180142cSMarcin Wojtas #define ENI_METRICS_MAX_SAMPLE_INTERVAL 3600
466064f289SMarcin Wojtas 
477029da5cSPawel Biernacki static SYSCTL_NODE(_hw, OID_AUTO, ena, CTLFLAG_RD | CTLFLAG_MPSAFE, 0,
487029da5cSPawel Biernacki     "ENA driver parameters");
496064f289SMarcin Wojtas 
506064f289SMarcin Wojtas /*
516064f289SMarcin Wojtas  * Logging level for changing verbosity of the output
526064f289SMarcin Wojtas  */
536064f289SMarcin Wojtas int ena_log_level = ENA_ALERT | ENA_WARNING;
546064f289SMarcin Wojtas SYSCTL_INT(_hw_ena, OID_AUTO, log_level, CTLFLAG_RWTUN,
556064f289SMarcin Wojtas     &ena_log_level, 0, "Logging level indicating verbosity of the logs");
566064f289SMarcin Wojtas 
5785516621SMarcin Wojtas SYSCTL_CONST_STRING(_hw_ena, OID_AUTO, driver_version, CTLFLAG_RD,
5885516621SMarcin Wojtas     DRV_MODULE_VERSION, "ENA driver version");
5985516621SMarcin Wojtas 
6004cf2b88SMarcin Wojtas /*
6104cf2b88SMarcin Wojtas  * Use 9k mbufs for the Rx buffers. Default to 0 (use page size mbufs instead).
6204cf2b88SMarcin Wojtas  * Using 9k mbufs in low memory conditions might cause allocation to take a lot
6304cf2b88SMarcin Wojtas  * of time and lead to the OS instability as it needs to look for the contiguous
6404cf2b88SMarcin Wojtas  * pages.
6504cf2b88SMarcin Wojtas  * However, page size mbufs has a bit smaller throughput than 9k mbufs, so if
6604cf2b88SMarcin Wojtas  * the network performance is the priority, the 9k mbufs can be used.
6704cf2b88SMarcin Wojtas  */
6804cf2b88SMarcin Wojtas int ena_enable_9k_mbufs = 0;
6904cf2b88SMarcin Wojtas SYSCTL_INT(_hw_ena, OID_AUTO, enable_9k_mbufs, CTLFLAG_RDTUN,
7004cf2b88SMarcin Wojtas     &ena_enable_9k_mbufs, 0, "Use 9 kB mbufs for Rx descriptors");
719b8d05b8SZbigniew Bodek 
72*beaadec9SMarcin Wojtas /*
73*beaadec9SMarcin Wojtas  * Force the driver to use large LLQ (Low Latency Queue) header. Defaults to
74*beaadec9SMarcin Wojtas  * false. This option may be important for platforms, which often handle packet
75*beaadec9SMarcin Wojtas  * headers on Tx with total header size greater than 96B, as it may
76*beaadec9SMarcin Wojtas  * reduce the latency.
77*beaadec9SMarcin Wojtas  * It also reduces the maximum Tx queue size by half, so it may cause more Tx
78*beaadec9SMarcin Wojtas  * packet drops.
79*beaadec9SMarcin Wojtas  */
80*beaadec9SMarcin Wojtas bool ena_force_large_llq_header = false;
81*beaadec9SMarcin Wojtas SYSCTL_BOOL(_hw_ena, OID_AUTO, force_large_llq_header, CTLFLAG_RDTUN,
82*beaadec9SMarcin Wojtas     &ena_force_large_llq_header, 0,
83*beaadec9SMarcin Wojtas     "Increases maximum supported header size in LLQ mode to 224 bytes, while reducing the maximum Tx queue size by half.\n");
84*beaadec9SMarcin Wojtas 
859b8d05b8SZbigniew Bodek void
869b8d05b8SZbigniew Bodek ena_sysctl_add_nodes(struct ena_adapter *adapter)
879b8d05b8SZbigniew Bodek {
880bdffe59SMarcin Wojtas 	ena_sysctl_add_wd(adapter);
899b8d05b8SZbigniew Bodek 	ena_sysctl_add_stats(adapter);
90f180142cSMarcin Wojtas 	ena_sysctl_add_eni_metrics(adapter);
916064f289SMarcin Wojtas 	ena_sysctl_add_tuneables(adapter);
929b8d05b8SZbigniew Bodek }
939b8d05b8SZbigniew Bodek 
949b8d05b8SZbigniew Bodek static void
950bdffe59SMarcin Wojtas ena_sysctl_add_wd(struct ena_adapter *adapter)
960bdffe59SMarcin Wojtas {
970bdffe59SMarcin Wojtas 	device_t dev;
980bdffe59SMarcin Wojtas 
990bdffe59SMarcin Wojtas 	struct sysctl_ctx_list *ctx;
1000bdffe59SMarcin Wojtas 	struct sysctl_oid *tree;
1010bdffe59SMarcin Wojtas 	struct sysctl_oid_list *child;
1020bdffe59SMarcin Wojtas 
1030bdffe59SMarcin Wojtas 	dev = adapter->pdev;
1040bdffe59SMarcin Wojtas 
1050bdffe59SMarcin Wojtas 	ctx = device_get_sysctl_ctx(dev);
1060bdffe59SMarcin Wojtas 	tree = device_get_sysctl_tree(dev);
1070bdffe59SMarcin Wojtas 	child = SYSCTL_CHILDREN(tree);
1080bdffe59SMarcin Wojtas 
1090bdffe59SMarcin Wojtas 	/* Sysctl calls for Watchdog service */
1100bdffe59SMarcin Wojtas 	SYSCTL_ADD_INT(ctx, child, OID_AUTO, "wd_active",
1110bdffe59SMarcin Wojtas 	    CTLFLAG_RWTUN, &adapter->wd_active, 0,
1120bdffe59SMarcin Wojtas 	    "Watchdog is active");
1130bdffe59SMarcin Wojtas 
1140bdffe59SMarcin Wojtas 	SYSCTL_ADD_QUAD(ctx, child, OID_AUTO, "keep_alive_timeout",
1150bdffe59SMarcin Wojtas 	    CTLFLAG_RWTUN, &adapter->keep_alive_timeout,
1160bdffe59SMarcin Wojtas 	    "Timeout for Keep Alive messages");
1170bdffe59SMarcin Wojtas 
1180bdffe59SMarcin Wojtas 	SYSCTL_ADD_QUAD(ctx, child, OID_AUTO, "missing_tx_timeout",
1190bdffe59SMarcin Wojtas 	    CTLFLAG_RWTUN, &adapter->missing_tx_timeout,
1200bdffe59SMarcin Wojtas 	    "Timeout for TX completion");
1210bdffe59SMarcin Wojtas 
1220bdffe59SMarcin Wojtas 	SYSCTL_ADD_U32(ctx, child, OID_AUTO, "missing_tx_max_queues",
1230bdffe59SMarcin Wojtas 	    CTLFLAG_RWTUN, &adapter->missing_tx_max_queues, 0,
1240bdffe59SMarcin Wojtas 	    "Number of TX queues to check per run");
1250bdffe59SMarcin Wojtas 
1260bdffe59SMarcin Wojtas 	SYSCTL_ADD_U32(ctx, child, OID_AUTO, "missing_tx_threshold",
1270bdffe59SMarcin Wojtas 	    CTLFLAG_RWTUN, &adapter->missing_tx_threshold, 0,
1280bdffe59SMarcin Wojtas 	    "Max number of timeouted packets");
1290bdffe59SMarcin Wojtas }
1300bdffe59SMarcin Wojtas 
1310bdffe59SMarcin Wojtas static void
1329b8d05b8SZbigniew Bodek ena_sysctl_add_stats(struct ena_adapter *adapter)
1339b8d05b8SZbigniew Bodek {
1349b8d05b8SZbigniew Bodek 	device_t dev;
1359b8d05b8SZbigniew Bodek 
1369b8d05b8SZbigniew Bodek 	struct ena_ring *tx_ring;
1379b8d05b8SZbigniew Bodek 	struct ena_ring *rx_ring;
1389b8d05b8SZbigniew Bodek 
1399b8d05b8SZbigniew Bodek 	struct ena_hw_stats *hw_stats;
1409b8d05b8SZbigniew Bodek 	struct ena_stats_dev *dev_stats;
1419b8d05b8SZbigniew Bodek 	struct ena_stats_tx *tx_stats;
1429b8d05b8SZbigniew Bodek 	struct ena_stats_rx *rx_stats;
1439b8d05b8SZbigniew Bodek 	struct ena_com_stats_admin *admin_stats;
1449b8d05b8SZbigniew Bodek 
1459b8d05b8SZbigniew Bodek 	struct sysctl_ctx_list *ctx;
1469b8d05b8SZbigniew Bodek 	struct sysctl_oid *tree;
1479b8d05b8SZbigniew Bodek 	struct sysctl_oid_list *child;
1489b8d05b8SZbigniew Bodek 
1499b8d05b8SZbigniew Bodek 	struct sysctl_oid *queue_node, *tx_node, *rx_node, *hw_node;
1509b8d05b8SZbigniew Bodek 	struct sysctl_oid *admin_node;
1519b8d05b8SZbigniew Bodek 	struct sysctl_oid_list *queue_list, *tx_list, *rx_list, *hw_list;
1529b8d05b8SZbigniew Bodek 	struct sysctl_oid_list *admin_list;
1539b8d05b8SZbigniew Bodek 
1549b8d05b8SZbigniew Bodek #define QUEUE_NAME_LEN 32
1559b8d05b8SZbigniew Bodek 	char namebuf[QUEUE_NAME_LEN];
1569b8d05b8SZbigniew Bodek 	int i;
1579b8d05b8SZbigniew Bodek 
1589b8d05b8SZbigniew Bodek 	dev = adapter->pdev;
1599b8d05b8SZbigniew Bodek 
1609b8d05b8SZbigniew Bodek 	ctx = device_get_sysctl_ctx(dev);
1619b8d05b8SZbigniew Bodek 	tree = device_get_sysctl_tree(dev);
1629b8d05b8SZbigniew Bodek 	child = SYSCTL_CHILDREN(tree);
1639b8d05b8SZbigniew Bodek 
1649b8d05b8SZbigniew Bodek 	tx_ring = adapter->tx_ring;
1659b8d05b8SZbigniew Bodek 	rx_ring = adapter->rx_ring;
1669b8d05b8SZbigniew Bodek 
1679b8d05b8SZbigniew Bodek 	hw_stats = &adapter->hw_stats;
1689b8d05b8SZbigniew Bodek 	dev_stats = &adapter->dev_stats;
1699b8d05b8SZbigniew Bodek 	admin_stats = &adapter->ena_dev->admin_queue.stats;
1709b8d05b8SZbigniew Bodek 
1719b8d05b8SZbigniew Bodek 	SYSCTL_ADD_COUNTER_U64(ctx, child, OID_AUTO, "wd_expired",
1729b8d05b8SZbigniew Bodek 	    CTLFLAG_RD, &dev_stats->wd_expired,
1739b8d05b8SZbigniew Bodek 	    "Watchdog expiry count");
1749b8d05b8SZbigniew Bodek 	SYSCTL_ADD_COUNTER_U64(ctx, child, OID_AUTO, "interface_up",
1759b8d05b8SZbigniew Bodek 	    CTLFLAG_RD, &dev_stats->interface_up,
1769b8d05b8SZbigniew Bodek 	    "Network interface up count");
1779b8d05b8SZbigniew Bodek 	SYSCTL_ADD_COUNTER_U64(ctx, child, OID_AUTO, "interface_down",
1789b8d05b8SZbigniew Bodek 	    CTLFLAG_RD, &dev_stats->interface_down,
1799b8d05b8SZbigniew Bodek 	    "Network interface down count");
1809b8d05b8SZbigniew Bodek 	SYSCTL_ADD_COUNTER_U64(ctx, child, OID_AUTO, "admin_q_pause",
1819b8d05b8SZbigniew Bodek 	    CTLFLAG_RD, &dev_stats->admin_q_pause,
1829b8d05b8SZbigniew Bodek 	    "Admin queue pauses");
1839b8d05b8SZbigniew Bodek 
1847d8c4feeSMarcin Wojtas 	for (i = 0; i < adapter->num_io_queues; ++i, ++tx_ring, ++rx_ring) {
1859b8d05b8SZbigniew Bodek 		snprintf(namebuf, QUEUE_NAME_LEN, "queue%d", i);
1869b8d05b8SZbigniew Bodek 
1879b8d05b8SZbigniew Bodek 		queue_node = SYSCTL_ADD_NODE(ctx, child, OID_AUTO,
1887029da5cSPawel Biernacki 		    namebuf, CTLFLAG_RD | CTLFLAG_MPSAFE, NULL, "Queue Name");
1899b8d05b8SZbigniew Bodek 		queue_list = SYSCTL_CHILDREN(queue_node);
1909b8d05b8SZbigniew Bodek 
1919b8d05b8SZbigniew Bodek 		/* TX specific stats */
1929b8d05b8SZbigniew Bodek 		tx_node = SYSCTL_ADD_NODE(ctx, queue_list, OID_AUTO,
1937029da5cSPawel Biernacki 		    "tx_ring", CTLFLAG_RD | CTLFLAG_MPSAFE, NULL, "TX ring");
1949b8d05b8SZbigniew Bodek 		tx_list = SYSCTL_CHILDREN(tx_node);
1959b8d05b8SZbigniew Bodek 
1969b8d05b8SZbigniew Bodek 		tx_stats = &tx_ring->tx_stats;
1979b8d05b8SZbigniew Bodek 
1989b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
1999b8d05b8SZbigniew Bodek 		    "count", CTLFLAG_RD,
2009b8d05b8SZbigniew Bodek 		    &tx_stats->cnt, "Packets sent");
2019b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2029b8d05b8SZbigniew Bodek 		    "bytes", CTLFLAG_RD,
2039b8d05b8SZbigniew Bodek 		    &tx_stats->bytes, "Bytes sent");
2049b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2059b8d05b8SZbigniew Bodek 		    "prepare_ctx_err", CTLFLAG_RD,
2069b8d05b8SZbigniew Bodek 		    &tx_stats->prepare_ctx_err,
2079b8d05b8SZbigniew Bodek 		    "TX buffer preparation failures");
2089b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2099b8d05b8SZbigniew Bodek 		    "dma_mapping_err", CTLFLAG_RD,
2109b8d05b8SZbigniew Bodek 		    &tx_stats->dma_mapping_err, "DMA mapping failures");
2119b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2129b8d05b8SZbigniew Bodek 		    "doorbells", CTLFLAG_RD,
2139b8d05b8SZbigniew Bodek 		    &tx_stats->doorbells, "Queue doorbells");
2149b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2159b8d05b8SZbigniew Bodek 		    "missing_tx_comp", CTLFLAG_RD,
2169b8d05b8SZbigniew Bodek 		    &tx_stats->missing_tx_comp, "TX completions missed");
2179b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2189b8d05b8SZbigniew Bodek 		    "bad_req_id", CTLFLAG_RD,
2199b8d05b8SZbigniew Bodek 		    &tx_stats->bad_req_id, "Bad request id count");
2209b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2211b069f1cSZbigniew Bodek 		        "mbuf_collapses", CTLFLAG_RD,
2221b069f1cSZbigniew Bodek 		        &tx_stats->collapse,
2231b069f1cSZbigniew Bodek 		        "Mbuf collapse count");
2241e9fb899SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2251b069f1cSZbigniew Bodek 		        "mbuf_collapse_err", CTLFLAG_RD,
2261b069f1cSZbigniew Bodek 		        &tx_stats->collapse_err,
2271b069f1cSZbigniew Bodek 		        "Mbuf collapse failures");
2285cb9db07SMarcin Wojtas 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2295cb9db07SMarcin Wojtas 		    "queue_wakeups", CTLFLAG_RD,
2305cb9db07SMarcin Wojtas 		    &tx_stats->queue_wakeup, "Queue wakeups");
2315cb9db07SMarcin Wojtas 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2325cb9db07SMarcin Wojtas 		    "queue_stops", CTLFLAG_RD,
2335cb9db07SMarcin Wojtas 		    &tx_stats->queue_stop, "Queue stops");
2344fa9e02dSMarcin Wojtas 		SYSCTL_ADD_COUNTER_U64(ctx, tx_list, OID_AUTO,
2354fa9e02dSMarcin Wojtas 		    "llq_buffer_copy", CTLFLAG_RD,
2364fa9e02dSMarcin Wojtas 		    &tx_stats->llq_buffer_copy,
2374fa9e02dSMarcin Wojtas 		    "Header copies for llq transaction");
2389b8d05b8SZbigniew Bodek 
2399b8d05b8SZbigniew Bodek 		/* RX specific stats */
2409b8d05b8SZbigniew Bodek 		rx_node = SYSCTL_ADD_NODE(ctx, queue_list, OID_AUTO,
2417029da5cSPawel Biernacki 		    "rx_ring", CTLFLAG_RD | CTLFLAG_MPSAFE, NULL, "RX ring");
2429b8d05b8SZbigniew Bodek 		rx_list = SYSCTL_CHILDREN(rx_node);
2439b8d05b8SZbigniew Bodek 
2449b8d05b8SZbigniew Bodek 		rx_stats = &rx_ring->rx_stats;
2459b8d05b8SZbigniew Bodek 
2469b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2479b8d05b8SZbigniew Bodek 		    "count", CTLFLAG_RD,
2489b8d05b8SZbigniew Bodek 		    &rx_stats->cnt, "Packets received");
2499b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2509b8d05b8SZbigniew Bodek 		    "bytes", CTLFLAG_RD,
2519b8d05b8SZbigniew Bodek 		    &rx_stats->bytes, "Bytes received");
2529b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2539b8d05b8SZbigniew Bodek 		    "refil_partial", CTLFLAG_RD,
2549b8d05b8SZbigniew Bodek 		    &rx_stats->refil_partial, "Partial refilled mbufs");
2559b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2569b8d05b8SZbigniew Bodek 		    "bad_csum", CTLFLAG_RD,
2579b8d05b8SZbigniew Bodek 		    &rx_stats->bad_csum, "Bad RX checksum");
2589b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2599b8d05b8SZbigniew Bodek 		    "mbuf_alloc_fail", CTLFLAG_RD,
2609b8d05b8SZbigniew Bodek 		    &rx_stats->mbuf_alloc_fail, "Failed mbuf allocs");
2619b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2624727bda6SMarcin Wojtas 		    "mjum_alloc_fail", CTLFLAG_RD,
2634727bda6SMarcin Wojtas 		    &rx_stats->mjum_alloc_fail, "Failed jumbo mbuf allocs");
2644727bda6SMarcin Wojtas 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2659b8d05b8SZbigniew Bodek 		    "dma_mapping_err", CTLFLAG_RD,
2669b8d05b8SZbigniew Bodek 		    &rx_stats->dma_mapping_err, "DMA mapping errors");
2679b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
2689b8d05b8SZbigniew Bodek 		    "bad_desc_num", CTLFLAG_RD,
2699b8d05b8SZbigniew Bodek 		    &rx_stats->bad_desc_num, "Bad descriptor count");
2709b8d05b8SZbigniew Bodek 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
27143fefd16SMarcin Wojtas 		    "bad_req_id", CTLFLAG_RD,
27243fefd16SMarcin Wojtas 		    &rx_stats->bad_req_id, "Bad request id count");
273efe6ab18SMarcin Wojtas 		SYSCTL_ADD_COUNTER_U64(ctx, rx_list, OID_AUTO,
274efe6ab18SMarcin Wojtas 		    "empty_rx_ring", CTLFLAG_RD,
275efe6ab18SMarcin Wojtas 		    &rx_stats->empty_rx_ring, "RX descriptors depletion count");
2769b8d05b8SZbigniew Bodek 	}
2779b8d05b8SZbigniew Bodek 
2789b8d05b8SZbigniew Bodek 	/* Stats read from device */
2799b8d05b8SZbigniew Bodek 	hw_node = SYSCTL_ADD_NODE(ctx, child, OID_AUTO, "hw_stats",
2807029da5cSPawel Biernacki 	    CTLFLAG_RD | CTLFLAG_MPSAFE, NULL, "Statistics from hardware");
2819b8d05b8SZbigniew Bodek 	hw_list = SYSCTL_CHILDREN(hw_node);
2829b8d05b8SZbigniew Bodek 
28330217e2dSMarcin Wojtas 	SYSCTL_ADD_COUNTER_U64(ctx, hw_list, OID_AUTO, "rx_packets", CTLFLAG_RD,
28430217e2dSMarcin Wojtas 	    &hw_stats->rx_packets, "Packets received");
28530217e2dSMarcin Wojtas 	SYSCTL_ADD_COUNTER_U64(ctx, hw_list, OID_AUTO, "tx_packets", CTLFLAG_RD,
28630217e2dSMarcin Wojtas 	    &hw_stats->tx_packets, "Packets transmitted");
28730217e2dSMarcin Wojtas 	SYSCTL_ADD_COUNTER_U64(ctx, hw_list, OID_AUTO, "rx_bytes", CTLFLAG_RD,
28830217e2dSMarcin Wojtas 	    &hw_stats->rx_bytes, "Bytes received");
28930217e2dSMarcin Wojtas 	SYSCTL_ADD_COUNTER_U64(ctx, hw_list, OID_AUTO, "tx_bytes", CTLFLAG_RD,
29030217e2dSMarcin Wojtas 	    &hw_stats->tx_bytes, "Bytes transmitted");
29130217e2dSMarcin Wojtas 	SYSCTL_ADD_COUNTER_U64(ctx, hw_list, OID_AUTO, "rx_drops", CTLFLAG_RD,
29230217e2dSMarcin Wojtas 	    &hw_stats->rx_drops, "Receive packet drops");
2936c84cec3SMarcin Wojtas 	SYSCTL_ADD_COUNTER_U64(ctx, hw_list, OID_AUTO, "tx_drops", CTLFLAG_RD,
2946c84cec3SMarcin Wojtas 	    &hw_stats->tx_drops, "Transmit packet drops");
295a195fab0SMarcin Wojtas 
2969b8d05b8SZbigniew Bodek 	/* ENA Admin queue stats */
2979b8d05b8SZbigniew Bodek 	admin_node = SYSCTL_ADD_NODE(ctx, child, OID_AUTO, "admin_stats",
2987029da5cSPawel Biernacki 	    CTLFLAG_RD | CTLFLAG_MPSAFE, NULL, "ENA Admin Queue statistics");
2999b8d05b8SZbigniew Bodek 	admin_list = SYSCTL_CHILDREN(admin_node);
3009b8d05b8SZbigniew Bodek 
3018483b844SMarcin Wojtas 	SYSCTL_ADD_U64(ctx, admin_list, OID_AUTO, "aborted_cmd", CTLFLAG_RD,
3029b8d05b8SZbigniew Bodek 	    &admin_stats->aborted_cmd, 0, "Aborted commands");
3038483b844SMarcin Wojtas 	SYSCTL_ADD_U64(ctx, admin_list, OID_AUTO, "sumbitted_cmd", CTLFLAG_RD,
3049b8d05b8SZbigniew Bodek 	    &admin_stats->submitted_cmd, 0, "Submitted commands");
3058483b844SMarcin Wojtas 	SYSCTL_ADD_U64(ctx, admin_list, OID_AUTO, "completed_cmd", CTLFLAG_RD,
3069b8d05b8SZbigniew Bodek 	    &admin_stats->completed_cmd, 0, "Completed commands");
3078483b844SMarcin Wojtas 	SYSCTL_ADD_U64(ctx, admin_list, OID_AUTO, "out_of_space", CTLFLAG_RD,
3089b8d05b8SZbigniew Bodek 	    &admin_stats->out_of_space, 0, "Queue out of space");
3098483b844SMarcin Wojtas 	SYSCTL_ADD_U64(ctx, admin_list, OID_AUTO, "no_completion", CTLFLAG_RD,
3109b8d05b8SZbigniew Bodek 	    &admin_stats->no_completion, 0, "Commands not completed");
3119b8d05b8SZbigniew Bodek }
3129b8d05b8SZbigniew Bodek 
3136064f289SMarcin Wojtas static void
314f180142cSMarcin Wojtas ena_sysctl_add_eni_metrics(struct ena_adapter *adapter)
315f180142cSMarcin Wojtas {
316f180142cSMarcin Wojtas 	device_t dev;
317f180142cSMarcin Wojtas 	struct ena_admin_eni_stats *eni_metrics;
318f180142cSMarcin Wojtas 
319f180142cSMarcin Wojtas 	struct sysctl_ctx_list *ctx;
320f180142cSMarcin Wojtas 	struct sysctl_oid *tree;
321f180142cSMarcin Wojtas 	struct sysctl_oid_list *child;
322f180142cSMarcin Wojtas 
323f180142cSMarcin Wojtas 	struct sysctl_oid *eni_node;
324f180142cSMarcin Wojtas 	struct sysctl_oid_list *eni_list;
325f180142cSMarcin Wojtas 
326f180142cSMarcin Wojtas 	dev = adapter->pdev;
327f180142cSMarcin Wojtas 
328f180142cSMarcin Wojtas 	ctx = device_get_sysctl_ctx(dev);
329f180142cSMarcin Wojtas 	tree = device_get_sysctl_tree(dev);
330f180142cSMarcin Wojtas 	child = SYSCTL_CHILDREN(tree);
331f180142cSMarcin Wojtas 
332f180142cSMarcin Wojtas 	eni_node = SYSCTL_ADD_NODE(ctx, child, OID_AUTO, "eni_metrics",
333f180142cSMarcin Wojtas 	    CTLFLAG_RD | CTLFLAG_MPSAFE, NULL, "ENA's ENI metrics");
334f180142cSMarcin Wojtas 	eni_list = SYSCTL_CHILDREN(eni_node);
335f180142cSMarcin Wojtas 
336f180142cSMarcin Wojtas 	eni_metrics = &adapter->eni_metrics;
337f180142cSMarcin Wojtas 
338f180142cSMarcin Wojtas 	SYSCTL_ADD_U64(ctx, eni_list, OID_AUTO, "bw_in_allowance_exceeded",
339f180142cSMarcin Wojtas 	    CTLFLAG_RD, &eni_metrics->bw_in_allowance_exceeded, 0,
340f180142cSMarcin Wojtas 	    "Inbound BW allowance exceeded");
341f180142cSMarcin Wojtas 	SYSCTL_ADD_U64(ctx, eni_list, OID_AUTO, "bw_out_allowance_exceeded",
342f180142cSMarcin Wojtas 	    CTLFLAG_RD, &eni_metrics->bw_out_allowance_exceeded, 0,
343f180142cSMarcin Wojtas 	    "Outbound BW allowance exceeded");
344f180142cSMarcin Wojtas 	SYSCTL_ADD_U64(ctx, eni_list, OID_AUTO, "pps_allowance_exceeded",
345f180142cSMarcin Wojtas 	    CTLFLAG_RD, &eni_metrics->pps_allowance_exceeded, 0,
346f180142cSMarcin Wojtas 	    "PPS allowance exceeded");
347f180142cSMarcin Wojtas 	SYSCTL_ADD_U64(ctx, eni_list, OID_AUTO, "conntrack_allowance_exceeded",
348f180142cSMarcin Wojtas 	    CTLFLAG_RD, &eni_metrics->conntrack_allowance_exceeded, 0,
349f180142cSMarcin Wojtas 	    "Connection tracking allowance exceeded");
350f180142cSMarcin Wojtas 	SYSCTL_ADD_U64(ctx, eni_list, OID_AUTO, "linklocal_allowance_exceeded",
351f180142cSMarcin Wojtas 	    CTLFLAG_RD, &eni_metrics->linklocal_allowance_exceeded, 0,
352f180142cSMarcin Wojtas 	    "Linklocal packet rate allowance exceeded");
353f180142cSMarcin Wojtas 
354f180142cSMarcin Wojtas 	/*
355f180142cSMarcin Wojtas 	 * Tuneable, which determines how often ENI metrics will be read.
356f180142cSMarcin Wojtas 	 * 0 means it's turned off. Maximum allowed value is limited by:
357f180142cSMarcin Wojtas 	 * ENI_METRICS_MAX_SAMPLE_INTERVAL.
358f180142cSMarcin Wojtas 	 */
359f180142cSMarcin Wojtas 	SYSCTL_ADD_PROC(ctx, eni_list, OID_AUTO, "sample_interval",
360f180142cSMarcin Wojtas 	    CTLTYPE_U16 | CTLFLAG_RW | CTLFLAG_MPSAFE, adapter, 0,
361f180142cSMarcin Wojtas 	    ena_sysctl_eni_metrics_interval, "SU",
362f180142cSMarcin Wojtas 	    "Interval in seconds for updating ENI emetrics. 0 turns off the update.");
363f180142cSMarcin Wojtas }
364f180142cSMarcin Wojtas 
365f180142cSMarcin Wojtas static void
3666064f289SMarcin Wojtas ena_sysctl_add_tuneables(struct ena_adapter *adapter)
3676064f289SMarcin Wojtas {
3686064f289SMarcin Wojtas 	device_t dev;
3696064f289SMarcin Wojtas 
3706064f289SMarcin Wojtas 	struct sysctl_ctx_list *ctx;
3716064f289SMarcin Wojtas 	struct sysctl_oid *tree;
3726064f289SMarcin Wojtas 	struct sysctl_oid_list *child;
3736064f289SMarcin Wojtas 
3746064f289SMarcin Wojtas 	dev = adapter->pdev;
3756064f289SMarcin Wojtas 
3766064f289SMarcin Wojtas 	ctx = device_get_sysctl_ctx(dev);
3776064f289SMarcin Wojtas 	tree = device_get_sysctl_tree(dev);
3786064f289SMarcin Wojtas 	child = SYSCTL_CHILDREN(tree);
3796064f289SMarcin Wojtas 
3806064f289SMarcin Wojtas 	/* Tuneable number of buffers in the buf-ring (drbr) */
3817029da5cSPawel Biernacki 	SYSCTL_ADD_PROC(ctx, child, OID_AUTO, "buf_ring_size",
38221823546SMarcin Wojtas 	    CTLTYPE_U32 | CTLFLAG_RW | CTLFLAG_MPSAFE, adapter, 0,
38321823546SMarcin Wojtas 	    ena_sysctl_buf_ring_size, "I",
38421823546SMarcin Wojtas 	    "Size of the Tx buffer ring (drbr).");
3856064f289SMarcin Wojtas 
3867d8c4feeSMarcin Wojtas 	/* Tuneable number of the Rx ring size */
3877029da5cSPawel Biernacki 	SYSCTL_ADD_PROC(ctx, child, OID_AUTO, "rx_queue_size",
3887d8c4feeSMarcin Wojtas 	    CTLTYPE_U32 | CTLFLAG_RW | CTLFLAG_MPSAFE, adapter, 0,
3897d8c4feeSMarcin Wojtas 	    ena_sysctl_rx_queue_size, "I",
3907d8c4feeSMarcin Wojtas 	    "Size of the Rx ring. The size should be a power of 2.");
39156d41ad5SMarcin Wojtas 
39256d41ad5SMarcin Wojtas 	/* Tuneable number of IO queues */
39356d41ad5SMarcin Wojtas 	SYSCTL_ADD_PROC(ctx, child, OID_AUTO, "io_queues_nb",
39456d41ad5SMarcin Wojtas 	    CTLTYPE_U32 | CTLFLAG_RW | CTLFLAG_MPSAFE, adapter, 0,
39556d41ad5SMarcin Wojtas 	    ena_sysctl_io_queues_nb, "I", "Number of IO queues.");
3966064f289SMarcin Wojtas }
3976064f289SMarcin Wojtas 
3986064f289SMarcin Wojtas 
3996064f289SMarcin Wojtas static int
4006064f289SMarcin Wojtas ena_sysctl_buf_ring_size(SYSCTL_HANDLER_ARGS)
4016064f289SMarcin Wojtas {
4026064f289SMarcin Wojtas 	struct ena_adapter *adapter = arg1;
40321823546SMarcin Wojtas 	uint32_t val;
4046064f289SMarcin Wojtas 	int error;
4056064f289SMarcin Wojtas 
4066064f289SMarcin Wojtas 	val = 0;
40721823546SMarcin Wojtas 	error = sysctl_wire_old_buffer(req, sizeof(val));
4086064f289SMarcin Wojtas 	if (error == 0) {
4096064f289SMarcin Wojtas 		val = adapter->buf_ring_size;
410f180142cSMarcin Wojtas 		error = sysctl_handle_32(oidp, &val, 0, req);
4116064f289SMarcin Wojtas 	}
4126064f289SMarcin Wojtas 	if (error != 0 || req->newptr == NULL)
4136064f289SMarcin Wojtas 		return (error);
4146064f289SMarcin Wojtas 
41521823546SMarcin Wojtas 	if (!powerof2(val) || val == 0) {
4166064f289SMarcin Wojtas 		device_printf(adapter->pdev,
41721823546SMarcin Wojtas 		    "Requested new Tx buffer ring size (%u) is not a power of 2\n",
41821823546SMarcin Wojtas 		    val);
41921823546SMarcin Wojtas 		return (EINVAL);
4206064f289SMarcin Wojtas 	}
4216064f289SMarcin Wojtas 
42221823546SMarcin Wojtas 	if (val != adapter->buf_ring_size) {
42321823546SMarcin Wojtas 		device_printf(adapter->pdev,
42421823546SMarcin Wojtas 		    "Requested new Tx buffer ring size: %d. Old size: %d\n",
42521823546SMarcin Wojtas 		    val, adapter->buf_ring_size);
42621823546SMarcin Wojtas 
42721823546SMarcin Wojtas 		error = ena_update_buf_ring_size(adapter, val);
42821823546SMarcin Wojtas 	} else {
42921823546SMarcin Wojtas 		device_printf(adapter->pdev,
43021823546SMarcin Wojtas 		    "New Tx buffer ring size is the same as already used: %u\n",
43121823546SMarcin Wojtas 		    adapter->buf_ring_size);
43221823546SMarcin Wojtas 	}
43321823546SMarcin Wojtas 
43421823546SMarcin Wojtas 	return (error);
4356064f289SMarcin Wojtas }
4366064f289SMarcin Wojtas 
4376064f289SMarcin Wojtas static int
4386064f289SMarcin Wojtas ena_sysctl_rx_queue_size(SYSCTL_HANDLER_ARGS)
4396064f289SMarcin Wojtas {
4406064f289SMarcin Wojtas 	struct ena_adapter *adapter = arg1;
4417d8c4feeSMarcin Wojtas 	uint32_t val;
4426064f289SMarcin Wojtas 	int error;
4436064f289SMarcin Wojtas 
4446064f289SMarcin Wojtas 	val = 0;
4457d8c4feeSMarcin Wojtas 	error = sysctl_wire_old_buffer(req, sizeof(val));
4466064f289SMarcin Wojtas 	if (error == 0) {
4479762a033SMarcin Wojtas 		val = adapter->requested_rx_ring_size;
4487d8c4feeSMarcin Wojtas 		error = sysctl_handle_32(oidp, &val, 0, req);
4496064f289SMarcin Wojtas 	}
4506064f289SMarcin Wojtas 	if (error != 0 || req->newptr == NULL)
4516064f289SMarcin Wojtas 		return (error);
4526064f289SMarcin Wojtas 
4537d8c4feeSMarcin Wojtas 	if  (val < ENA_MIN_RING_SIZE || val > adapter->max_rx_ring_size) {
4546064f289SMarcin Wojtas 		device_printf(adapter->pdev,
4557d8c4feeSMarcin Wojtas 		    "Requested new Rx queue size (%u) is out of range: [%u, %u]\n",
4567d8c4feeSMarcin Wojtas 		    val, ENA_MIN_RING_SIZE, adapter->max_rx_ring_size);
4577d8c4feeSMarcin Wojtas 		return (EINVAL);
4586064f289SMarcin Wojtas 	}
4596064f289SMarcin Wojtas 
4607d8c4feeSMarcin Wojtas 	/* Check if the parameter is power of 2 */
4617d8c4feeSMarcin Wojtas 	if (!powerof2(val)) {
4627d8c4feeSMarcin Wojtas 		device_printf(adapter->pdev,
4637d8c4feeSMarcin Wojtas 		    "Requested new Rx queue size (%u) is not a power of 2\n",
4647d8c4feeSMarcin Wojtas 		    val);
4657d8c4feeSMarcin Wojtas 		return (EINVAL);
4667d8c4feeSMarcin Wojtas 	}
4677d8c4feeSMarcin Wojtas 
4689762a033SMarcin Wojtas 	if (val != adapter->requested_rx_ring_size) {
4697d8c4feeSMarcin Wojtas 		device_printf(adapter->pdev,
4707d8c4feeSMarcin Wojtas 		    "Requested new Rx queue size: %u. Old size: %u\n",
4719762a033SMarcin Wojtas 		    val, adapter->requested_rx_ring_size);
4727d8c4feeSMarcin Wojtas 
4739762a033SMarcin Wojtas 		error = ena_update_queue_size(adapter,
4749762a033SMarcin Wojtas 		    adapter->requested_tx_ring_size, val);
4757d8c4feeSMarcin Wojtas 	} else {
4767d8c4feeSMarcin Wojtas 		device_printf(adapter->pdev,
4777d8c4feeSMarcin Wojtas 		    "New Rx queue size is the same as already used: %u\n",
4789762a033SMarcin Wojtas 		    adapter->requested_rx_ring_size);
4797d8c4feeSMarcin Wojtas 	}
4807d8c4feeSMarcin Wojtas 
4817d8c4feeSMarcin Wojtas 	return (error);
4826064f289SMarcin Wojtas }
48356d41ad5SMarcin Wojtas 
48456d41ad5SMarcin Wojtas /*
48556d41ad5SMarcin Wojtas  * Change number of effectively used IO queues adapter->num_io_queues
48656d41ad5SMarcin Wojtas  */
48756d41ad5SMarcin Wojtas static int
48856d41ad5SMarcin Wojtas ena_sysctl_io_queues_nb(SYSCTL_HANDLER_ARGS)
48956d41ad5SMarcin Wojtas {
49056d41ad5SMarcin Wojtas 	struct ena_adapter *adapter = arg1;
49156d41ad5SMarcin Wojtas 	uint32_t tmp = 0;
49256d41ad5SMarcin Wojtas 	int error;
49356d41ad5SMarcin Wojtas 
49456d41ad5SMarcin Wojtas 	error = sysctl_wire_old_buffer(req, sizeof(tmp));
49556d41ad5SMarcin Wojtas 	if (error == 0) {
49656d41ad5SMarcin Wojtas 		tmp = adapter->num_io_queues;
49756d41ad5SMarcin Wojtas 		error = sysctl_handle_int(oidp, &tmp, 0, req);
49856d41ad5SMarcin Wojtas 	}
49956d41ad5SMarcin Wojtas 	if (error != 0 || req->newptr == NULL)
50056d41ad5SMarcin Wojtas 		return (error);
50156d41ad5SMarcin Wojtas 
50256d41ad5SMarcin Wojtas 	if (tmp == 0) {
50356d41ad5SMarcin Wojtas 		device_printf(adapter->pdev,
50456d41ad5SMarcin Wojtas 		    "Requested number of IO queues is zero\n");
50556d41ad5SMarcin Wojtas 		return (EINVAL);
50656d41ad5SMarcin Wojtas 	}
50756d41ad5SMarcin Wojtas 
50856d41ad5SMarcin Wojtas 	/*
50956d41ad5SMarcin Wojtas 	 * The adapter::max_num_io_queues is the HW capability. The system
51056d41ad5SMarcin Wojtas 	 * resources availability may potentially be a tighter limit. Therefore
51156d41ad5SMarcin Wojtas 	 * the relation `adapter::max_num_io_queues >= adapter::msix_vecs`
51256d41ad5SMarcin Wojtas 	 * always holds true, while the `adapter::msix_vecs` is variable across
51356d41ad5SMarcin Wojtas 	 * device reset (`ena_destroy_device()` + `ena_restore_device()`).
51456d41ad5SMarcin Wojtas 	 */
51556d41ad5SMarcin Wojtas 	if (tmp > (adapter->msix_vecs - ENA_ADMIN_MSIX_VEC)) {
51656d41ad5SMarcin Wojtas 		device_printf(adapter->pdev,
51756d41ad5SMarcin Wojtas 		    "Requested number of IO queues is higher than maximum "
51856d41ad5SMarcin Wojtas 		    "allowed (%u)\n", adapter->msix_vecs - ENA_ADMIN_MSIX_VEC);
51956d41ad5SMarcin Wojtas 		return (EINVAL);
52056d41ad5SMarcin Wojtas 	}
52156d41ad5SMarcin Wojtas 	if (tmp == adapter->num_io_queues) {
52256d41ad5SMarcin Wojtas 		device_printf(adapter->pdev,
52356d41ad5SMarcin Wojtas 		    "Requested number of IO queues is equal to current value "
52456d41ad5SMarcin Wojtas 		    "(%u)\n", adapter->num_io_queues);
52556d41ad5SMarcin Wojtas 	} else {
52656d41ad5SMarcin Wojtas 		device_printf(adapter->pdev,
52756d41ad5SMarcin Wojtas 		    "Requested new number of IO queues: %u, current value: "
52856d41ad5SMarcin Wojtas 		    "%u\n", tmp, adapter->num_io_queues);
52956d41ad5SMarcin Wojtas 
53056d41ad5SMarcin Wojtas 		error = ena_update_io_queue_nb(adapter, tmp);
53156d41ad5SMarcin Wojtas 	}
53256d41ad5SMarcin Wojtas 
53356d41ad5SMarcin Wojtas 	return (error);
53456d41ad5SMarcin Wojtas }
535f180142cSMarcin Wojtas 
536f180142cSMarcin Wojtas static int
537f180142cSMarcin Wojtas ena_sysctl_eni_metrics_interval(SYSCTL_HANDLER_ARGS)
538f180142cSMarcin Wojtas {
539f180142cSMarcin Wojtas 	struct ena_adapter *adapter = arg1;
540f180142cSMarcin Wojtas 	uint16_t interval;
541f180142cSMarcin Wojtas 	int error;
542f180142cSMarcin Wojtas 
543f180142cSMarcin Wojtas 	error = sysctl_wire_old_buffer(req, sizeof(interval));
544f180142cSMarcin Wojtas 	if (error == 0) {
545f180142cSMarcin Wojtas 		interval = adapter->eni_metrics_sample_interval;
546f180142cSMarcin Wojtas 		error = sysctl_handle_16(oidp, &interval, 0, req);
547f180142cSMarcin Wojtas 	}
548f180142cSMarcin Wojtas 	if (error != 0 || req->newptr == NULL)
549f180142cSMarcin Wojtas 		return (error);
550f180142cSMarcin Wojtas 
551f180142cSMarcin Wojtas 	if (interval > ENI_METRICS_MAX_SAMPLE_INTERVAL) {
552f180142cSMarcin Wojtas 		device_printf(adapter->pdev,
553f180142cSMarcin Wojtas 		    "ENI metrics update interval is out of range - maximum allowed value: %d seconds\n",
554f180142cSMarcin Wojtas 		    ENI_METRICS_MAX_SAMPLE_INTERVAL);
555f180142cSMarcin Wojtas 		return (EINVAL);
556f180142cSMarcin Wojtas 	}
557f180142cSMarcin Wojtas 
558f180142cSMarcin Wojtas 	if (interval == 0) {
559f180142cSMarcin Wojtas 		device_printf(adapter->pdev,
560f180142cSMarcin Wojtas 		    "ENI metrics update is now turned off\n");
561f180142cSMarcin Wojtas 		bzero(&adapter->eni_metrics, sizeof(adapter->eni_metrics));
562f180142cSMarcin Wojtas 	} else {
563f180142cSMarcin Wojtas 		device_printf(adapter->pdev,
564f180142cSMarcin Wojtas 		    "ENI metrics update interval is set to: %"PRIu16" seconds\n",
565f180142cSMarcin Wojtas 		    interval);
566f180142cSMarcin Wojtas 	}
567f180142cSMarcin Wojtas 
568f180142cSMarcin Wojtas 	adapter->eni_metrics_sample_interval = interval;
569f180142cSMarcin Wojtas 
570f180142cSMarcin Wojtas 	return (0);
571f180142cSMarcin Wojtas }
572