> -----Original Message-----
> From: Ye, MingjinX <[email protected]>
> Sent: Monday, November 21, 2022 10:55 AM
> To: [email protected]
> Cc: Yang, Qiming <[email protected]>; [email protected]; Zhou, YidingX
> <[email protected]>; Ye, MingjinX <[email protected]>; Zhang, Qi
> Z <[email protected]>; Jie Zhou <[email protected]>; Menon, Ranjit
> <[email protected]>; Ferruh Yigit <[email protected]>; Kadam,
> Pallavi <[email protected]>
> Subject: [PATCH v6] doc: add PMD known issue
> 
> Add a known issue: Rx path dynamic routing is not supported for PMD.
> 
> Fixes: de853a3bb151 ("net/ice: disable DDP package on Windows")
> Cc: [email protected]
> 
> Signed-off-by: Mingjin Ye <[email protected]>
> ---
>  doc/guides/nics/ice.rst | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/doc/guides/nics/ice.rst b/doc/guides/nics/ice.rst index
> ce075e067c..60fd7834ed 100644
> --- a/doc/guides/nics/ice.rst
> +++ b/doc/guides/nics/ice.rst
> @@ -395,3 +395,20 @@ file is used by both the kernel driver and the DPDK
> PMD.
> 
>        Windows support: The DDP package is not supported on Windows so,
>        loading of the package is disabled on Windows.
> +
> +ice: Rx path is not supported after PF or DCF add vlan offload
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> ~
> +
> +If pmd does not enable Vlan offload during initialization, it will
> +automatically select Rx paths that do not support offload. Even if Vlan
> +offload is subsequently enabled through the API, Vlan offload will not
> +work because the selected Rx path does not support Vlan offload.
> +
> +Rx path dynamic routing is not supported. When the offload features is
> +switched, the queue needs to be reconfigured, then takes effect. It
> +would take additional workload for the network card to deal with.

If offload features is switched, we need to re-configure the queue, this does 
not looks like a limitation.
Better to describe this from APIs calls 

> +
> +When applying VLAN offload on the PF or DCF, it must be configured
> +firstly by the startup parameters.

Better to explain what is the startup parameter precisely.

> +
> --
> 2.34.1

Reply via email to