Re: [802.3_4PPOE] Comment #95 and #346 homework

["Yseboodt, Lennart" <lennart.yseboodt@xxxxxxxxxxx>]

Hi Chris,

We will need to propagate this fix also to the equivalent construct in the PD section, as well as the dual-sig PD section.

Given that the current scheme works and the only objection is that "it is not good practice", I'm not sure this is worth the effort.

Kind regards,

Lennart

---

From: Chris Bullock (bullock) <bullock@xxxxxxxxx>
Sent: Tuesday, March 14, 2017 16:23
To: Yseboodt, Lennart; STDS-802-3-4PPOE@xxxxxxxxxxxxxxxxx
Subject: RE: [802.3_4PPOE] Comment #95 and #346 homework

 

All,

 

Lennart is correct that the original proposed fix introduced a race condition.  I have an updated proposal that introduces a new variable to perform the hand-shaking.  There is still no need for a new state.  Please review the attached proposal, and let me know your thoughts.

 

Thanks,

Chris

 

From: Yseboodt, Lennart [mailto:lennart.yseboodt@xxxxxxxxxxx]
Sent: Tuesday, March 14, 2017 11:49 AM
To: STDS-802-3-4PPOE@xxxxxxxxxxxxxxxxx
Subject: Re: [802.3_4PPOE] Comment #95 and #346 homework

 

Unfortunately the proposed fix also introduces a race condition.

 

I'm not sure we really need to fix this, but if we do, we need to avoid that race condition.

 

Kind regards,

 

Lennart

 

---

From: George Zimmerman <george@xxxxxxxxxxxxxxxxxxxx>
Sent: Tuesday, March 14, 2017 9:26
To: STDS-802-3-4PPOE@xxxxxxxxxxxxxxxxx
Subject: Re: [802.3_4PPOE] Comment #95 and #346 homework

 

Agreed - adding a "done" exit condition is better than adding a state just to wait until done

George A. Zimmerman, Ph.D.

CME Consulting, Inc.

Experts in PHYsical Layer Communications

george@xxxxxxxxxxxxxxxxxxxx

310-920-3860

 

On Mar 14, 2017, at 9:23 AM, Heath Stewart <00000855853231d4-dmarc-request@xxxxxxxx> wrote:

Looks like a clever fix.

 

-Heath

 

On Mon, Mar 13, 2017 at 9:30 PM, Chris Bullock (bullock) <bullock@xxxxxxxxx> wrote:

All,

 

It turns out that we did not need to add a “wait state” to the DLL state-machine since we already had a UCT exit condition from MIRROR_UPDATE that we could adjust.  Please review the changes shown below to see if you agree that these changes satisfy the issue from comment #95 and #346.

 

Thanks,

Chris

 

 

--

Heath Stewart
Design Center Manager, Mixed Signal

Office   (805) 560-7658
Mobile  (805) 895-0499
Websites      analog.com, linear.com

Linear Technology is now part of Analog Devices.  Learn more.

<ADI-Logo.png>