mirror of
https://github.com/torvalds/linux.git
synced 2026-11-06 21:58:01 +01:00
netdevsim: psp: drop tx key ops
netdevsim has no SADB. The usage of psp_assoc_drv_data() was always obsolete given psp_validate_xmit(), so we can remove nsim_assoc_add() and nsim_assoc_del(). netdevsim::assoc_cnt can also be removed. PSP core can track the balance of tx_key_add vs tx_key_del calls in a future change. Delete psp_assoc_drv_data() because there are no more callers left in the tree. mlx5 accesses pas->drv_data directly. Signed-off-by: Daniel Zahka <daniel.zahka@gmail.com> Link: https://patch.msgid.link/20260903-psp-prep-v1-4-d47e9c4c375d@gmail.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
1e16b30310
commit
da630d1da2
|
|
@ -122,7 +122,6 @@ struct netdevsim {
|
|||
struct dentry *rereg;
|
||||
struct mutex rereg_lock;
|
||||
u32 spi;
|
||||
u32 assoc_cnt;
|
||||
} psp;
|
||||
|
||||
struct nsim_bus_dev *nsim_bus_dev;
|
||||
|
|
|
|||
|
|
@ -23,7 +23,6 @@ nsim_do_psp(struct sk_buff *skb, struct netdevsim *ns,
|
|||
struct psp_assoc *pas;
|
||||
struct net *net;
|
||||
int psp_len;
|
||||
void **ptr;
|
||||
|
||||
rcu_read_lock();
|
||||
pas = psp_skb_get_assoc_rcu(skb);
|
||||
|
|
@ -37,12 +36,6 @@ nsim_do_psp(struct sk_buff *skb, struct netdevsim *ns,
|
|||
goto out_unlock;
|
||||
}
|
||||
|
||||
ptr = psp_assoc_drv_data(pas);
|
||||
if (*ptr != ns) {
|
||||
rc = SKB_DROP_REASON_PSP_OUTPUT;
|
||||
goto out_unlock;
|
||||
}
|
||||
|
||||
net = sock_net(skb->sk);
|
||||
if (!psp_dev_encapsulate(net, skb, pas->tx.spi, pas->version, 0)) {
|
||||
rc = SKB_DROP_REASON_PSP_OUTPUT;
|
||||
|
|
@ -149,19 +142,6 @@ nsim_rx_spi_alloc(struct psp_dev *psd, u32 version,
|
|||
return 0;
|
||||
}
|
||||
|
||||
static int nsim_assoc_add(struct psp_dev *psd, struct psp_assoc *pas,
|
||||
struct netlink_ext_ack *extack)
|
||||
{
|
||||
struct netdevsim *ns = psd->drv_priv;
|
||||
void **ptr = psp_assoc_drv_data(pas);
|
||||
|
||||
/* Copy drv_priv from psd to assoc */
|
||||
*ptr = psd->drv_priv;
|
||||
ns->psp.assoc_cnt++;
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int nsim_key_rotate(struct psp_dev *psd, struct netlink_ext_ack *extack)
|
||||
{
|
||||
struct netdevsim *ns = psd->drv_priv;
|
||||
|
|
@ -177,15 +157,6 @@ static int nsim_key_rotate(struct psp_dev *psd, struct netlink_ext_ack *extack)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static void nsim_assoc_del(struct psp_dev *psd, struct psp_assoc *pas)
|
||||
{
|
||||
struct netdevsim *ns = psd->drv_priv;
|
||||
void **ptr = psp_assoc_drv_data(pas);
|
||||
|
||||
*ptr = NULL;
|
||||
ns->psp.assoc_cnt--;
|
||||
}
|
||||
|
||||
static void nsim_get_stats(struct psp_dev *psd, struct psp_dev_stats *stats)
|
||||
{
|
||||
struct netdevsim *ns = psd->drv_priv;
|
||||
|
|
@ -204,8 +175,6 @@ static void nsim_get_stats(struct psp_dev *psd, struct psp_dev_stats *stats)
|
|||
static struct psp_dev_ops nsim_psp_ops = {
|
||||
.set_config = nsim_psp_set_config,
|
||||
.rx_spi_alloc = nsim_rx_spi_alloc,
|
||||
.tx_key_add = nsim_assoc_add,
|
||||
.tx_key_del = nsim_assoc_del,
|
||||
.key_rotate = nsim_key_rotate,
|
||||
.get_stats = nsim_get_stats,
|
||||
};
|
||||
|
|
@ -215,7 +184,6 @@ static struct psp_dev_caps nsim_psp_caps = {
|
|||
1 << PSP_VERSION_HDR0_AES_GMAC_128 |
|
||||
1 << PSP_VERSION_HDR0_AES_GCM_256 |
|
||||
1 << PSP_VERSION_HDR0_AES_GMAC_256,
|
||||
.assoc_drv_spc = sizeof(void *),
|
||||
};
|
||||
|
||||
static void __nsim_psp_uninit(struct netdevsim *ns, bool teardown)
|
||||
|
|
@ -230,7 +198,6 @@ static void __nsim_psp_uninit(struct netdevsim *ns, bool teardown)
|
|||
synchronize_rcu();
|
||||
psp_dev_unregister(psd);
|
||||
}
|
||||
WARN_ON(ns->psp.assoc_cnt);
|
||||
}
|
||||
|
||||
void nsim_psp_uninit(struct netdevsim *ns)
|
||||
|
|
|
|||
|
|
@ -24,11 +24,6 @@ int psp_dev_rcv(struct sk_buff *skb, u16 dev_id, u8 generation, bool strip_icv);
|
|||
/* Kernel-facing API */
|
||||
void psp_assoc_put(struct psp_assoc *pas);
|
||||
|
||||
static inline void *psp_assoc_drv_data(struct psp_assoc *pas)
|
||||
{
|
||||
return pas->drv_data;
|
||||
}
|
||||
|
||||
#if IS_ENABLED(CONFIG_INET_PSP)
|
||||
unsigned int psp_key_size(u32 version);
|
||||
void psp_sk_assoc_free(struct sock *sk);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user