Bug 23

Summary:	Ixgbe driver writes to reserved bit in the EIMC register
Product:	DPDK	Reporter:	Solal Pirelli (solal.pirelli)
Component:	ethdev	Assignee:	Konstantin Ananyev (konstantin.v.ananyev)
Status:	RESOLVED WONTFIX
Severity:	minor	CC:	ajit.khaparde, wei.zhao1
Priority:	Normal
Version:	unspecified
Target Milestone:	---
Hardware:	All
OS:	All

Description Solal Pirelli 2018-04-01 14:48:37 CEST

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.5.4) states that bit 31 of the EIMC register is reserved.

However, the ixgbe driver writes to this bit anyway, e.g. in ixgbe_stop_adapter_generic (http://dpdk.org/browse/dpdk/tree/drivers/net/ixgbe/base/ixgbe_common.c?h=v18.02#n1126 with the mask defined as all-1s in http://dpdk.org/browse/dpdk/tree/drivers/net/ixgbe/base/ixgbe_type.h?h=v18.02#n2047).

Comment 1 Ajit Khaparde 2018-08-29 19:40:49 CEST

Hi konstantin, can you please take a look? Thanks

Comment 2 Wei Zhao 2018-10-18 05:24:52 CEST

 
(In reply to Solal Pirelli from comment #0)
> 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.5.4) states that bit 31 of the
> EIMC register is reserved.
> 
> However, the ixgbe driver writes to this bit anyway, e.g. in
> ixgbe_stop_adapter_generic
> (http://dpdk.org/browse/dpdk/tree/drivers/net/ixgbe/base/ixgbe_common.
> c?h=v18.02#n1126 with the mask defined as all-1s in
> http://dpdk.org/browse/dpdk/tree/drivers/net/ixgbe/base/ixgbe_type.h?h=v18.
> 02#n2047).

In principle, the code in base folder will not be changed if there is no bug or issue. By now, the using of this reserved bit have no bad impact.  
So it is not a bug, and  change state to resolve and wontfix.

Comment 3 Wei Zhao 2018-10-18 09:40:08 CEST

it is not a bug, and change state to resolve and wontfix.