Thread Links Date Links
Thread Prev Thread Next Thread Index Date Prev Date Next Date Index

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



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.





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,




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



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

Looks like a clever fix.




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



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.







Heath Stewart
Design Center Manager, Mixed Signal

Office   (805) 560-7658
Mobile  (805) 895-0499

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


Attachment: bullock_01_0317.pdf
Description: bullock_01_0317.pdf