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.

Sign up for GitHub

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

Jump to bottom

PNA spec declares headers on deparser are RO #125

Closed
jhsmt opened this issue Dec 21, 2023 · 2 comments
Closed

PNA spec declares headers on deparser are RO #125

jhsmt opened this issue Dec 21, 2023 · 2 comments

Comments

@jhsmt
Copy link

jhsmt commented Dec 21, 2023

Comparing PNA: https://p4.org/p4-spec/docs/pna-working-draft-html-version.html#sec-programmable-blocks
vs PSA: https://p4.org/p4-spec/docs/PSA.html#sec-programmable-blocks ..

The deparser on PSA indicates headers are read/write whereas on PNA it is read-only.

We have a program that need to update ipv4 checksum on the deparser. While computing the checksum is allowed on the deparser (per https://p4.org/p4-spec/docs/pna-working-draft-html-version.html#sec-extern-restrictions), updating the computed value on a header is not. Is this the intended behavior?
@jafingerhut
Copy link
Collaborator

This PR is instended to address this issue. Please read it and see if it is missing anything you were hoping for: #127

@jhsmt
Copy link
Author

jhsmt commented Jan 31, 2024

Closing based on #127

@jhsmt jhsmt closed this as completed Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jafingerhut @jhsmt