From: Joshua Washington <joshwash@google.com>
To: Jeroen de Borst <jeroendb@google.com>,
Joshua Washington <joshwash@google.com>,
"Jasper Tran O'Leary" <jtranoleary@google.com>
Cc: dev@dpdk.org, stable@dpdk.org
Subject: [PATCH v2 8/9] net/gve: don't reset ring size bounds to default on reset
Date: Fri, 3 Jul 2026 06:13:05 -0700 [thread overview]
Message-ID: <20260703131308.2507403-9-joshwash@google.com> (raw)
In-Reply-To: <20260703131308.2507403-1-joshwash@google.com>
On device reset, GVE skips describe_device functionality, as the device
is not expected to change on a reset. However, before skipping the
describe_device functionality, GVE still sets the ring sizes to their
default values. This effectively removes the ability to create queues
with higher-than-default descriptor counts after a reset.
Fix this behavior by only setting the default ring size bounds is
describe_device is being executed.
Fixes: 1bf64edce3c4 ("net/gve: add reset path")
Cc: stable@dpdk.org
Signed-off-by: Joshua Washington <joshwash@google.com>
Reviewed-by: Jasper Tran O'Leary <jtranoleary@google.com>
---
drivers/net/gve/gve_ethdev.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/gve/gve_ethdev.c b/drivers/net/gve/gve_ethdev.c
index f73784a109..c990920a4d 100644
--- a/drivers/net/gve/gve_ethdev.c
+++ b/drivers/net/gve/gve_ethdev.c
@@ -1579,12 +1579,12 @@ gve_init_priv(struct gve_priv *priv, bool skip_describe_device)
goto free_adminq;
}
- /* Set default descriptor counts */
- gve_set_default_ring_size_bounds(priv);
-
if (skip_describe_device)
goto setup_device;
+ /* Set default descriptor counts */
+ gve_set_default_ring_size_bounds(priv);
+
/* Get the initial information we need from the device */
err = gve_adminq_describe_device(priv);
if (err) {
--
2.55.0.rc0.799.gd6f94ed593-goog
next prev parent reply other threads:[~2026-07-03 13:14 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-03 13:12 [PATCH v2 0/9] Stability fixes for GVE Joshua Washington
2026-07-03 13:12 ` [PATCH v2 1/9] net/gve: clear out shared memory region for stats report Joshua Washington
2026-07-03 13:12 ` [PATCH v2 2/9] net/gve: delay adding mbuf head to software ring Joshua Washington
2026-07-03 13:13 ` [PATCH v2 3/9] net/gve: copy data to QPL buffer when mbuf read does not Joshua Washington
2026-07-03 13:13 ` [PATCH v2 4/9] net/gve: validate buf ID before processing Rx packet Joshua Washington
2026-07-03 13:13 ` [PATCH v2 5/9] net/gve: set mbuf to null in software ring after use Joshua Washington
2026-07-03 13:13 ` [PATCH v2 6/9] net/gve: free ctx mbuf if packet dropped after first segment Joshua Washington
2026-07-03 13:13 ` [PATCH v2 7/9] net/gve: increase range of DMA memzone ids to 64 bits Joshua Washington
2026-07-03 13:13 ` Joshua Washington [this message]
2026-07-03 13:13 ` [PATCH v2 9/9] net/gve: restrict max ring size in GQ QPL to 2K Joshua Washington
2026-07-03 14:27 ` [PATCH v2 0/9] Stability fixes for GVE Stephen Hemminger
2026-07-07 16:02 ` Joshua Washington
2026-07-07 16:10 ` David Marchand
2026-07-07 16:36 ` Joshua Washington
2026-07-07 16:40 ` Bruce Richardson
2026-07-07 16:40 ` [PATCH v3 " Joshua Washington
2026-07-07 16:40 ` [PATCH v3 1/9] net/gve: clear out shared memory region for stats report Joshua Washington
2026-07-07 16:40 ` [PATCH v3 2/9] net/gve: delay adding mbuf head to software ring Joshua Washington
2026-07-07 16:40 ` [PATCH v3 3/9] net/gve: copy data to QPL buffer when mbuf read does not Joshua Washington
2026-07-07 16:40 ` [PATCH v3 4/9] net/gve: validate buf ID before processing Rx packet Joshua Washington
2026-07-07 16:40 ` [PATCH v3 5/9] net/gve: set mbuf to null in software ring after use Joshua Washington
2026-07-07 16:40 ` [PATCH v3 6/9] net/gve: free ctx mbuf if packet dropped after first segment Joshua Washington
2026-07-07 16:40 ` [PATCH v3 7/9] net/gve: increase range of DMA memzone ids to 64 bits Joshua Washington
2026-07-07 16:40 ` [PATCH v3 8/9] net/gve: don't reset ring size bounds to default on reset Joshua Washington
2026-07-07 16:40 ` [PATCH v3 9/9] net/gve: restrict max ring size in GQ QPL to 2K Joshua Washington
2026-07-07 22:13 ` Joshua Washington
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260703131308.2507403-9-joshwash@google.com \
--to=joshwash@google.com \
--cc=dev@dpdk.org \
--cc=jeroendb@google.com \
--cc=jtranoleary@google.com \
--cc=stable@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.