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

Add text to PNA specification on encrypt/decrypt operations #89

Open
jfingerh opened this issue Nov 15, 2022 · 4 comments
Open

Add text to PNA specification on encrypt/decrypt operations #89

jfingerh opened this issue Nov 15, 2022 · 4 comments

Comments

@jfingerh
Copy link
Contributor

jfingerh commented Nov 15, 2022

While it is definitely valuable to have comments in the definition of the extern, it would also be useful to give some text in the body of the PNA specification, in its own section, describing the basic idea of this extern, when and how its methods should be called, and what their effect is.

@pbhide Is this something you would be willing to write? Or anyone you can suggest that you can persuade to volunteer for the task?

@jfingerh jfingerh changed the title Add text to PSA specification on encrypt/decrypt operations Add text to PNA specification on encrypt/decrypt operations Nov 15, 2022
@pbhide
Copy link
Contributor

pbhide commented Nov 17, 2022

Will work on it in 1st week of dec.

@pbhide
Copy link
Contributor

pbhide commented Dec 5, 2022

@jfingerh any recommendation on the section in PNA spec where this should go?
I can think of adding it in section 6 (after 6.10 packet recirculation section, which seems empty btw). OR
I can add it in section 7.
I was looking for a section for "recommended/Required" externs in PNA spec but don't see one.
Are we going to specify certain extern functions/objects as - "PNA compliant target SHALL support the following extern methods and objects..."

@jafingerhut
Copy link
Collaborator

I would recommend a new sub-section of section 7, since it is an extern object we are explaining.

@jafingerhut
Copy link
Collaborator

So far there has been little use of MUST/SHALL/SHOULD capitalized like many IETF RFCs have in P4.org specifications. We can discuss whether there should be, but that is the current state of things.

jafingerhut added a commit that referenced this issue Dec 20, 2022
Issue #89: Description of Extern Objects for Inline Accelerators
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

No branches or pull requests

3 participants