|SUCCESS| net/ixgbe: shrink size of context index

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/162947

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-04-09

git sha: 20260408154804.1596060-1-bruce.richardson at intel.com
Author: Bruce Richardson <bruce.richardson at intel.com>
Subject: [PATCH] net/ixgbe: shrink size of context index

The patch changes the data type of ctx_curr from uint32_t to uint8_t in
the ixgbe-specific portion of the ci_tx_queue union to reduce memory
footprint.

> diff --git a/drivers/net/intel/common/tx.h b/drivers/net/intel/common/tx.h
> index f2123f069c..8cb281aa20 100644
> --- a/drivers/net/intel/common/tx.h
> +++ b/drivers/net/intel/common/tx.h
> @@ -200,7 +200,7 @@ struct ci_tx_queue {
>  		struct { /* ixgbe specific values */
>  			const struct ixgbe_txq_ops *ops;
>  			struct ixgbe_advctx_info *ctx_cache;
> -			uint32_t ctx_curr;
> +			uint8_t ctx_curr;

Does this code properly handle all comparisons and assignments involving
ctx_curr?  If there are any implicit conversions from larger types or
signed comparisons, could those cause issues after narrowing the type?

>  			uint8_t pthresh;   /**< Prefetch threshold register. */
>  			uint8_t hthresh;   /**< Host threshold register. */
>  			uint8_t wthresh;   /**< Write-back threshold reg. */