Bug 23 - Ixgbe driver writes to reserved bit in the EIMC register
Summary: Ixgbe driver writes to reserved bit in the EIMC register
Status: RESOLVED WONTFIX
Alias: None
Product: DPDK
Classification: Unclassified
Component: ethdev (show other bugs)
Version: unspecified
Hardware: All All
: Normal minor
Target Milestone: ---
Assignee: Konstantin Ananyev
URL:
Depends on:
Blocks:
 
Reported: 2018-04-01 14:48 CEST by Solal Pirelli
Modified: 2018-10-18 09:40 CEST (History)
2 users (show)



Attachments

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.

Note You need to log in before you can comment on or make changes to this bug.