|Thread Links||Date Links|
|Thread Prev||Thread Next||Thread Index||Date Prev||Date Next||Date Index|
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.
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.
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
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.