Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SourceC: now it is FIFO #41

Merged
merged 1 commit into from
Aug 13, 2023
Merged

SourceC: now it is FIFO #41

merged 1 commit into from
Aug 13, 2023

Conversation

Ivyfeather
Copy link
Member

sourceC must be FIFO, a case:
L2 Release, then receives Probe and respond ProbeAck NtoN if ProbeAck NtoN were to be sent first in sourceC, then L3 will update client meta as L2 Invalid L3 then receives Release from L2, causing double release assertion failed

sourceC must be FIFO, a case:
L2 Release, then receives Probe and respond ProbeAck NtoN
if ProbeAck NtoN were to be sent first in sourceC, then L3 will update client meta as L2 Invalid
L3 then receives Release from L2, causing double release assertion failed
@Ivyfeather
Copy link
Member Author

already passed 16 tl-test

@Ivyfeather Ivyfeather merged commit 3d56922 into master Aug 13, 2023
1 check passed
@Ivyfeather Ivyfeather deleted the fix-sourceC branch August 13, 2023 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants