Re: [PATCH v4 7/7] net/gve: update GVE documentation with RSS support

[Ferruh Yigit <ferruh.yigit@amd.com>]

On 1/26/2024 5:33 PM, Joshua Washington wrote:
> This patch updates the GVE doc page to communicate that GVE now supports
> RSS configuration and explains the limitations.
> 
> Signed-off-by: Joshua Washington <joshwash@google.com>
> ---
>  doc/guides/nics/gve.rst | 16 ++++++++++------
>  1 file changed, 10 insertions(+), 6 deletions(-)
> 
> diff --git a/doc/guides/nics/gve.rst b/doc/guides/nics/gve.rst
> index 1c3eaf03ef..80991e70cf 100644
> --- a/doc/guides/nics/gve.rst
> +++ b/doc/guides/nics/gve.rst
> @@ -70,6 +70,8 @@ Supported features of the GVE PMD are:
>  - Link state information
>  - Tx multi-segments (Scatter Tx)
>  - Tx UDP/TCP/SCTP Checksum
> +- RSS hash configuration
> +- RSS redirection table update and query
>  
>  Currently, only GQI_QPL and GQI_RDA queue format are supported in PMD.
>  Jumbo Frame is not supported in PMD for now.
> @@ -77,10 +79,12 @@ It'll be added in a future DPDK release.
>  Also, only GQI_QPL queue format is in use on GCP
>  since GQI_RDA hasn't been released in production.
>  
> -Currently, setting MTU with value larger than 1460 is not supported.
> +RSS
> +^^^
>  
> -Currently, only "RSS hash" is force enabled
> -so that the backend hardware device calculated hash values
> -could be shared with applications.
> -But for RSS, there is no such API to config RSS hash function or RETA table.
> -So, limited RSS is supported only with default config/setting.
> +GVE RSS can be enabled and configured using the standard interfaces. The driver
> +does not support querying the initial RSS configuration.
> +
> +The RSS hash key must be exactly 40 bytes, and the redirection table must have
> +128 entries. The RSS hash key must be configured before the redirection table
> +can be set up.
>

Similarly can you please update documentation on each relevant patch,
instead of having a separate documentation patch.