-
Notifications
You must be signed in to change notification settings - Fork 4
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
Potential Backwards Indexing #5
Comments
The crux of the issue is this statement from section 13.8 from the P4-16 specification
Currently, we're doing the arguably more intuitive thing of extracting the first packet bit to the first bit of a |
rcgoodfellow
added a commit
that referenced
this issue
Nov 10, 2022
The p4 spec states that the first bit from a packet extracts into the last bit of bit-string types, this means that we go from protocol-endian to p4-endian and then back to protocol-endian on the way out. Fixes #5.
rcgoodfellow
added a commit
that referenced
this issue
Nov 10, 2022
The p4 spec states that the first bit from a packet extracts into the last bit of bit-string types, this means that we go from protocol-endian to p4-endian and then back to protocol-endian on the way out. Fixes #5.
rcgoodfellow
added a commit
that referenced
this issue
Nov 10, 2022
The p4 spec states that the first bit from a packet extracts into the last bit of bit-string types, this means that we go from protocol-endian to p4-endian and then back to protocol-endian on the way out. Fixes #5.
rcgoodfellow
added a commit
that referenced
this issue
Nov 10, 2022
The p4 spec states that the first bit from a packet extracts into the last bit of bit-string types, this means that we go from protocol-endian to p4-endian and then back to protocol-endian on the way out. Fixes #5.
rcgoodfellow
added a commit
that referenced
this issue
Nov 10, 2022
The p4 spec states that the first bit from a packet extracts into the last bit of bit-string types, this means that we go from protocol-endian to p4-endian and then back to protocol-endian on the way out. Fixes #5.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See:
p4/test/src/p4/sidecar-lite.p4
Lines 397 to 399 in c53f861
The text was updated successfully, but these errors were encountered: