diff options
| author |   Gabriel Oginski <gabrielx.oginski@intel.com> | 2021-10-08 09:09:45 +0100 |
|---|---|---|
| committer |   Matthew Smith <mgsmith@netgate.com> | 2021-11-03 16:04:00 +0000 |
| commit | 4739c8833e2e8013417ef0e52610d8b310e97a93 (patch) | |
| tree | dbcc0c49dbeacad72005ef58c22d0e4ab13c2250 /src/plugins/wireguard/wireguard_input.c | |
| parent | 505fd37b31aa5a7d242184ebe281dbd4622ad277 (diff) | |
wireguard: reduce memcopy and prefetch header
Originally wireguard implementation does memory copy of the whole
packet in encryption and decryption.
This patch removes unnecessary packet copy in wireguard. In addition,
it contains some performance improvement such as prefetching header
and deleting unnecessary lock and unlock for decryption.
Type: improvement
Signed-off-by: Gabriel Oginski <gabrielx.oginski@intel.com>
Change-Id: I1fe8e54d749e6922465341083b448c842e2b670f
Diffstat (limited to 'src/plugins/wireguard/wireguard_input.c')
| -rw-r--r-- | src/plugins/wireguard/wireguard_input.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/src/plugins/wireguard/wireguard_input.c b/src/plugins/wireguard/wireguard_input.c index 4f5bd4de784..3e8ae9b2173 100644 --- a/src/plugins/wireguard/wireguard_input.c +++ b/src/plugins/wireguard/wireguard_input.c @@ -366,15 +366,9 @@ wg_input_inline (vlib_main_t *vm, vlib_node_runtime_t *node, goto out; } - u8 *decr_data = wmp->per_thread_data[thread_index].data; - - enum noise_state_crypt state_cr = noise_remote_decrypt (vm, - &peer->remote, - data->receiver_index, - data->counter, - data->encrypted_data, - encr_len, - decr_data); + enum noise_state_crypt state_cr = noise_remote_decrypt ( + vm, &peer->remote, data->receiver_index, data->counter, + data->encrypted_data, encr_len, data->encrypted_data); if (PREDICT_FALSE (state_cr == SC_CONN_RESET)) { @@ -392,7 +386,7 @@ wg_input_inline (vlib_main_t *vm, vlib_node_runtime_t *node, goto out; } - clib_memcpy (vlib_buffer_get_current (b[0]), decr_data, decr_len); + vlib_buffer_advance (b[0], sizeof (message_data_t)); b[0]->current_length = decr_len; vnet_buffer_offload_flags_clear (b[0], VNET_BUFFER_OFFLOAD_F_UDP_CKSUM); |