Commit 8f073036 authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 'net-tls-fix-device-surprise-removal-with-offload' 



Jakub Kicinski says:

====================
net/tls: fix device surprise removal with offload

This series fixes two issues with device surprise removal.
First we need to take a read lock around resync, otherwise
netdev notifier handler may clear the structures from under
our feet.

Secondly we need to be careful about the interpretation
of device features.  Offload has to be properly cleaned
up even if the TLS device features got cleared after
connection state was installed.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 0d18c7bd c3f4a6c3

net/tls/tls_device.c

+12 −12
Original line number Diff line number Diff line
@@ -553,8 +553,8 @@ void tls_device_write_space(struct sock *sk, struct tls_context *ctx) 
void handle_device_resync(struct sock *sk, u32 seq, u64 rcd_sn)

{

	struct tls_context *tls_ctx = tls_get_ctx(sk);

	struct net_device *netdev = tls_ctx->netdev;

	struct tls_offload_context_rx *rx_ctx;

	struct net_device *netdev;

	u32 is_req_pending;

	s64 resync_req;

	u32 req_seq;
@@ -568,10 +568,15 @@ void handle_device_resync(struct sock *sk, u32 seq, u64 rcd_sn) 
	is_req_pending = resync_req;



	if (unlikely(is_req_pending) && req_seq == seq &&

	    atomic64_try_cmpxchg(&rx_ctx->resync_req, &resync_req, 0))

		netdev->tlsdev_ops->tls_dev_resync_rx(netdev, sk,

						      seq + TLS_HEADER_SIZE - 1,

	    atomic64_try_cmpxchg(&rx_ctx->resync_req, &resync_req, 0)) {

		seq += TLS_HEADER_SIZE - 1;

		down_read(&device_offload_lock);

		netdev = tls_ctx->netdev;

		if (netdev)

			netdev->tlsdev_ops->tls_dev_resync_rx(netdev, sk, seq,

							      rcd_sn);

		up_read(&device_offload_lock);

	}

}



static int tls_device_reencrypt(struct sock *sk, struct sk_buff *skb)
@@ -934,12 +939,6 @@ void tls_device_offload_cleanup_rx(struct sock *sk) 
	if (!netdev)

		goto out;



	if (!(netdev->features & NETIF_F_HW_TLS_RX)) {

		pr_err_ratelimited("%s: device is missing NETIF_F_HW_TLS_RX cap\n",

				   __func__);

		goto out;

	}



	netdev->tlsdev_ops->tls_dev_del(netdev, tls_ctx,

					TLS_OFFLOAD_CTX_DIR_RX);
@@ -998,7 +997,8 @@ static int tls_dev_event(struct notifier_block *this, unsigned long event, 
{

	struct net_device *dev = netdev_notifier_info_to_dev(ptr);



	if (!(dev->features & (NETIF_F_HW_TLS_RX | NETIF_F_HW_TLS_TX)))

	if (!dev->tlsdev_ops &&

	    !(dev->features & (NETIF_F_HW_TLS_RX | NETIF_F_HW_TLS_TX)))

		return NOTIFY_DONE;



	switch (event) {

CRA Git | Maintained and supported by SUSTech CRA and CCSE