From: Sabrina Dubroca <sd@queasysnail.net>
To: Antony Antony <antony.antony@secunet.com>
Cc: Steffen Klassert <steffen.klassert@secunet.com>,
Herbert Xu <herbert@gondor.apana.org.au>,
netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
devel@linux-ipsec.org, Leon Romanovsky <leon@kernel.org>,
Eyal Birger <eyal.birger@gmail.com>,
Nicolas Dichtel <nicolas.dichtel@6wind.com>
Subject: Re: [PATCH ipsec-next v14 0/4] xfrm: Introduce direction attribute for SA
Date: Tue, 30 Apr 2024 11:33:38 +0200 [thread overview]
Message-ID: <ZjC68onD2DvsX6Qy@hog> (raw)
In-Reply-To: <cover.1714460330.git.antony.antony@secunet.com>
2024-04-30, 09:08:06 +0200, Antony Antony wrote:
> Hi,
>
> Inspired by the upcoming IP-TFS patch set, and confusions experienced in
> the past due to lack of direction attribute on SAs, add a new direction
> "dir" attribute. It aims to streamline the SA configuration process and
> enhance the clarity of existing SA attributes.
>
> This patch set introduces the 'dir' attribute to SA, aka xfrm_state,
> ('in' for input or 'out' for output). Alsp add validations of existing
> direction-specific SA attributes during configuration and in the data
> path lookup.
>
> This change would not affect any existing use case or way of configuring
> SA. You will notice improvements when the new 'dir' attribute is set.
>
> v14: add more SA flag checks.
> v13: has one fix, minor documenation updates, and function renaming.
>
> Antony Antony (4):
> xfrm: Add Direction to the SA in or out
> xfrm: Add dir validation to "out" data path lookup
> xfrm: Add dir validation to "in" data path lookup
> xfrm: Restrict SA direction attribute to specific netlink message
> types
Reviewed-by: Sabrina Dubroca <sd@queasysnail.net>
Thanks Antony.
Patches 2 and 3 are identical to v13 so you could have kept Nicolas's
Reviewed-by tags. Steffen, I guess you can copy them in case Nicolas
doesn't look at v14 by the time you apply it?
--
Sabrina
next prev parent reply other threads:[~2024-04-30 9:33 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-30 7:08 [PATCH ipsec-next v14 0/4] xfrm: Introduce direction attribute for SA Antony Antony
2024-04-30 7:08 ` [PATCH ipsec-next v14 1/4] xfrm: Add Direction to the SA in or out Antony Antony
2024-04-30 7:09 ` [PATCH ipsec-next v14 2/4] xfrm: Add dir validation to "out" data path lookup Antony Antony
2024-05-02 7:55 ` Nicolas Dichtel
2024-04-30 7:09 ` [PATCH ipsec-next v14 3/4] xfrm: Add dir validation to "in" " Antony Antony
2024-05-02 7:56 ` Nicolas Dichtel
2024-04-30 7:09 ` [PATCH ipsec-next v14 4/4] xfrm: Restrict SA direction attribute to specific netlink message types Antony Antony
2024-04-30 9:33 ` Sabrina Dubroca [this message]
2024-05-02 7:58 ` [PATCH ipsec-next v14 0/4] xfrm: Introduce direction attribute for SA Nicolas Dichtel
2024-05-02 8:11 ` Steffen Klassert
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZjC68onD2DvsX6Qy@hog \
--to=sd@queasysnail.net \
--cc=antony.antony@secunet.com \
--cc=davem@davemloft.net \
--cc=devel@linux-ipsec.org \
--cc=edumazet@google.com \
--cc=eyal.birger@gmail.com \
--cc=herbert@gondor.apana.org.au \
--cc=kuba@kernel.org \
--cc=leon@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nicolas.dichtel@6wind.com \
--cc=pabeni@redhat.com \
--cc=steffen.klassert@secunet.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.