]> git.baikalelectronics.ru Git - kernel.git/commit
rxrpc: Use the UDP encap_rcv hook
authorDavid Howells <dhowells@redhat.com>
Thu, 4 Oct 2018 10:10:51 +0000 (11:10 +0100)
committerDavid Howells <dhowells@redhat.com>
Mon, 8 Oct 2018 14:45:18 +0000 (15:45 +0100)
commit912123eeaf6293a1599c76d48d78a14bf798797d
treed7ebc9c86a13557a6802fab23b7b1a25a708d6b5
parent9cd84904d68565735be786d070a6c8325fc4ddfd
rxrpc: Use the UDP encap_rcv hook

Use the UDP encap_rcv hook to cut the bit out of the rxrpc packet reception
in which a packet is placed onto the UDP receive queue and then immediately
removed again by rxrpc.  Going via the queue in this manner seems like it
should be unnecessary.

This does, however, require the invention of a value to place in encap_type
as that's one of the conditions to switch packets out to the encap_rcv
hook.  Possibly the value doesn't actually matter for anything other than
sockopts on the UDP socket, which aren't accessible outside of rxrpc
anyway.

This seems to cut a bit of time out of the time elapsed between each
sk_buff being timestamped and turning up in rxrpc (the final number in the
following trace excerpts).  I measured this by making the rxrpc_rx_packet
trace point print the time elapsed between the skb being timestamped and
the current time (in ns), e.g.:

... 424.278721: rxrpc_rx_packet: ...  ACK 25026

So doing a 512MiB DIO read from my test server, with an unmodified kernel:

N       min     max     sum mean    stddev
27605   2626    7581    7.83992e+07     2840.04 181.029

and with the patch applied:

N       min     max     sum mean    stddev
27547   1895    12165   6.77461e+07     2459.29 255.02

Signed-off-by: David Howells <dhowells@redhat.com>
include/uapi/linux/udp.h
net/rxrpc/ar-internal.h
net/rxrpc/input.c
net/rxrpc/local_object.c