Commit 2d6b51c6 authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller

net/tls: split the TLS_DRIVER_STATE_SIZE and bump TX to 16 bytes

8 bytes of driver state has been enough so far, but for drivers
which have to store 8 byte handle it's no longer practical to
store the state directly in the context.

Drivers generally don't need much extra state on RX side, while
TX side has to be tracking TCP sequence numbers.  Split the
lengths of max driver state size on RX and TX.

The struct tls_offload_context_tx currently stands at 616 bytes and
struct tls_offload_context_rx stands at 368 bytes.  Upcoming work
will consume extra 8 bytes in both for kernel-driven resync.
This means that we can bump TX side to 16 bytes and still fit
into the same number of cache lines but on RX side we would be 8
bytes over.
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: default avatarDirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5584c0f8
...@@ -202,12 +202,12 @@ struct tls_offload_context_tx { ...@@ -202,12 +202,12 @@ struct tls_offload_context_tx {
* Currently the belief is that there is not enough * Currently the belief is that there is not enough
* driver specific state to justify another layer of indirection * driver specific state to justify another layer of indirection
*/ */
#define TLS_DRIVER_STATE_SIZE (max_t(size_t, 8, sizeof(void *))) #define TLS_DRIVER_STATE_SIZE_TX 16
}; };
#define TLS_OFFLOAD_CONTEXT_SIZE_TX \ #define TLS_OFFLOAD_CONTEXT_SIZE_TX \
(ALIGN(sizeof(struct tls_offload_context_tx), sizeof(void *)) + \ (ALIGN(sizeof(struct tls_offload_context_tx), sizeof(void *)) + \
TLS_DRIVER_STATE_SIZE) TLS_DRIVER_STATE_SIZE_TX)
struct cipher_context { struct cipher_context {
char *iv; char *iv;
...@@ -307,11 +307,12 @@ struct tls_offload_context_rx { ...@@ -307,11 +307,12 @@ struct tls_offload_context_rx {
* Currently the belief is that there is not enough * Currently the belief is that there is not enough
* driver specific state to justify another layer of indirection * driver specific state to justify another layer of indirection
*/ */
#define TLS_DRIVER_STATE_SIZE_RX 8
}; };
#define TLS_OFFLOAD_CONTEXT_SIZE_RX \ #define TLS_OFFLOAD_CONTEXT_SIZE_RX \
(ALIGN(sizeof(struct tls_offload_context_rx), sizeof(void *)) + \ (ALIGN(sizeof(struct tls_offload_context_rx), sizeof(void *)) + \
TLS_DRIVER_STATE_SIZE) TLS_DRIVER_STATE_SIZE_RX)
int wait_on_pending_writer(struct sock *sk, long *timeo); int wait_on_pending_writer(struct sock *sk, long *timeo);
int tls_sk_query(struct sock *sk, int optname, char __user *optval, int tls_sk_query(struct sock *sk, int optname, char __user *optval,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment