Re: [Intel-wired-lan] Possible read-modify-write bug in ixgbe x550 phy setup

2018-02-02 Thread Shannon Nelson
On 2/2/2018 1:08 PM, Tantilov, Emil S wrote: Just FYI - we looked at the reads and confirmed that there is no functional bug in the code because as it happens the CX1/SR bits is the only bits that are read and set and as such we don't lose any data. This of course means that the read is not neede

RE: [Intel-wired-lan] Possible read-modify-write bug in ixgbe x550 phy setup

2018-02-02 Thread Tantilov, Emil S
un...@osuosl.org] On >>>> Behalf Of Shannon Nelson >>>> Sent: Thursday, February 01, 2018 3:46 PM >>>> To: Tantilov, Emil S >>>> Cc: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org >>>> Subject: [Intel-wired-lan] Possible read-modify-w

RE: [Intel-wired-lan] Possible read-modify-write bug in ixgbe x550 phy setup

2018-02-01 Thread Tantilov, Emil S
t;> To: Tantilov, Emil S >>> Cc: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org >>> Subject: [Intel-wired-lan] Possible read-modify-write bug in ixgbe >x550 >>> phy setup >>> >>> Hi Emil, >>> >>> I was looking through

Re: [Intel-wired-lan] Possible read-modify-write bug in ixgbe x550 phy setup

2018-02-01 Thread Shannon Nelson
Subject: [Intel-wired-lan] Possible read-modify-write bug in ixgbe x550 phy setup Hi Emil, I was looking through a set of ixgbe patches and came across this commit commit 410a494902777c11f95031d9ed757d7f8f09c5c6 ixgbe: add write flush when configuring CS4223/7 and am wondering about the

RE: [Intel-wired-lan] Possible read-modify-write bug in ixgbe x550 phy setup

2018-02-01 Thread Tantilov, Emil S
ed-lan] Possible read-modify-write bug in ixgbe x550 >phy setup > >Hi Emil, > >I was looking through a set of ixgbe patches and came across this commit > > commit 410a494902777c11f95031d9ed757d7f8f09c5c6 > ixgbe: add write flush when configuring CS4223/7 > >and a