> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Vlad Zolotarov
> Sent: Monday, March 09, 2015 4:29 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH v2 2/3] ixgbe: Bug fix: Properly configure Rx CRC 
> stripping for x540 devices
> 
> According to x540 spec chapter 8.2.4.8.9 CRCSTRIP field of RDRXCTL should
> be configured to the same value as HLREG0.RXCRCSTRP.
> 
> Clearing the RDRXCTL.RSCFRSTSIZE field for x540 is not required by the spec
> but seems harmless.

Acked-by: Konstantin Ananyev <konstantin.ananyev at intel.com>

> 
> Signed-off-by: Vlad Zolotarov <vladz at cloudius-systems.com>
> ---
>  lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c 
> b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c
> index b033e04..ce9658e 100644
> --- a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c
> +++ b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c
> @@ -3677,7 +3677,8 @@ ixgbe_dev_rx_init(struct rte_eth_dev *dev)
> 
>       IXGBE_WRITE_REG(hw, IXGBE_RXCSUM, rxcsum);
> 
> -     if (hw->mac.type == ixgbe_mac_82599EB) {
> +     if (hw->mac.type == ixgbe_mac_82599EB ||
> +         hw->mac.type == ixgbe_mac_X540) {
>               rdrxctl = IXGBE_READ_REG(hw, IXGBE_RDRXCTL);
>               if (dev->data->dev_conf.rxmode.hw_strip_crc)
>                       rdrxctl |= IXGBE_RDRXCTL_CRCSTRIP;
> --
> 2.1.0

Reply via email to