Thread (184 messages) flat view 184 messages, 7 authors, 2021-10-27

Re: [dpdk-dev] [EXT] [PATCH v4 03/10] security: add ESN field to ipsec_xform

From: Anoob Joseph <hidden>
Date: 2021-09-06 13:50:44

Hi Radu, Akhil,

Please see inline

Thanks,
Anoob
On 9/6/2021 12:36 PM, Anoob Joseph wrote:
quoted
Hi Radu,
quoted
Hi Akhil, I suppose they can be complementary, with this one being a
hard ESN limit that the user can enforce by setting the initial ESN
value - but there is no requirement to do so. Also, this change
doesn't need explicit support added in the PMDs.
What is the actual use case of this field (ESN)? My impression was this is to
allow application to control sequence number. For normal use cases, it can be
like starting sequence number. And this can be used with
``rte_security_session_update`` to allow simulating corner cases (like large
anti-replay windows sizes with ESN enabled etc). Did I capture the intended
use case correctly?
quoted
If it is to set max sequence number to be handled by the session, then I
guess, this is getting addressed as part of SA lifetime spec proposal.
quoted
Can you confirm what is the intended use case?

Thanks,
Anoob
Hi Anoob, the purpose was to have a starting value controlled by the app and
I think you're right, it can be achieved with rte_security_session_update.
[Anoob] Thanks for the confirmation. In that case, I'm in agreement with this proposal. May be update the patch description to better explain the use case.

Acked-by: Anoob Joseph <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help