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
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>