[dpdk-dev] [Bug 22] Ixgbe driver sets RDRXCTL with the wrong RSCACKC and FCOE_WRFIX values

[dpdk-dev] [Bug 22] Ixgbe driver sets RDRXCTL with the wrong RSCACKC and FCOE_WRFIX values

[bugzilla]

https://dpdk.org/tracker/show_bug.cgi?id=22

            Bug ID: 22
           Summary: Ixgbe driver sets RDRXCTL with the wrong RSCACKC and
                    FCOE_WRFIX values
           Product: DPDK
           Version: unspecified
          Hardware: All
                OS: All
            Status: CONFIRMED
          Severity: normal
          Priority: Normal
         Component: ethdev
          Assignee: dev@dpdk.org
          Reporter: solal.pirelli@gmail.com
  Target Milestone: ---

The Intel 82599 data sheet
(https://www.intel.com/content/dam/www/public/us/en/documents/datasheets/82599-10-gbe-controller-datasheet.pdf,
§8.2.3.8.8) states that the RDRXCTL register's RSCACKC (bit 25) and FCOE_WRFIX
(bit 26) values, while reserved, should be set to 1 by software.

The ixgbe driver does not do that, e.g. in ixgbe_dev_rx_init
(http://dpdk.org/browse/dpdk/tree/drivers/net/ixgbe/ixgbe_rxtx.c?h=v18.02#n4855).

-- 
You are receiving this mail because:
You are the assignee for the bug.

Re: [dpdk-dev] [Bug 22] Ixgbe driver sets RDRXCTL with the wrong RSCACKC and FCOE_WRFIX values

[Stephen Hemminger]

On Sun, 01 Apr 2018 12:42:38 +0000
bugzilla@dpdk.org wrote:

> https://dpdk.org/tracker/show_bug.cgi?id=22
> 
>             Bug ID: 22
>            Summary: Ixgbe driver sets RDRXCTL with the wrong RSCACKC and
>                     FCOE_WRFIX values
>            Product: DPDK
>            Version: unspecified
>           Hardware: All
>                 OS: All
>             Status: CONFIRMED
>           Severity: normal
>           Priority: Normal
>          Component: ethdev
>           Assignee: dev@dpdk.org
>           Reporter: solal.pirelli@gmail.com
>   Target Milestone: ---
> 
> The Intel 82599 data sheet
> (https://www.intel.com/content/dam/www/public/us/en/documents/datasheets/82599-10-gbe-controller-datasheet.pdf,
> §8.2.3.8.8) states that the RDRXCTL register's RSCACKC (bit 25) and FCOE_WRFIX
> (bit 26) values, while reserved, should be set to 1 by software.
> 
> The ixgbe driver does not do that, e.g. in ixgbe_dev_rx_init
> (http://dpdk.org/browse/dpdk/tree/drivers/net/ixgbe/ixgbe_rxtx.c?h=v18.02#n4855).

It is done elsewhere.

static int
ixgbe_set_rsc(struct rte_eth_dev *dev)
{
...

	/* Set RDRXCTL.RSCACKC bit */
	rdrxctl = IXGBE_READ_REG(hw, IXGBE_RDRXCTL);
	rdrxctl |= IXGBE_RDRXCTL_RSCACKC;
	IXGBE_WRITE_REG(hw, IXGBE_RDRXCTL, rdrxctl);