aboutsummaryrefslogtreecommitdiffstats
path: root/net/strparser/strparser.c
diff options
context:
space:
mode:
authorDoron Roberts-Kedes <doronrk@fb.com>2018-06-06 12:33:28 -0400
committerDavid S. Miller <davem@davemloft.net>2018-06-06 14:07:53 -0400
commit7170e6045a6a8b33f4fa5753589dc77b16198e2d (patch)
treed0b3c5b6035ce9ca8ef2fbfc01964221be142441 /net/strparser/strparser.c
parentfb1967a69f756073362b8f19347f863f227320ad (diff)
strparser: Add __strp_unpause and use it in ktls.
strp_unpause queues strp_work in order to parse any messages that arrived while the strparser was paused. However, the process invoking strp_unpause could eagerly parse a buffered message itself if it held the sock lock. __strp_unpause is an alternative to strp_pause that avoids the scheduling overhead that results when a receiving thread unpauses the strparser and waits for the next message to be delivered by the workqueue thread. This patch more than doubled the IOPS achieved in a benchmark of NBD traffic encrypted using ktls. Signed-off-by: Doron Roberts-Kedes <doronrk@fb.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/strparser/strparser.c')
-rw-r--r--net/strparser/strparser.c13
1 files changed, 13 insertions, 0 deletions
diff --git a/net/strparser/strparser.c b/net/strparser/strparser.c
index 092bebc70048..1a9695183599 100644
--- a/net/strparser/strparser.c
+++ b/net/strparser/strparser.c
@@ -512,6 +512,19 @@ int strp_init(struct strparser *strp, struct sock *sk,
512} 512}
513EXPORT_SYMBOL_GPL(strp_init); 513EXPORT_SYMBOL_GPL(strp_init);
514 514
515/* Sock process lock held (lock_sock) */
516void __strp_unpause(struct strparser *strp)
517{
518 strp->paused = 0;
519
520 if (strp->need_bytes) {
521 if (strp_peek_len(strp) < strp->need_bytes)
522 return;
523 }
524 strp_read_sock(strp);
525}
526EXPORT_SYMBOL_GPL(__strp_unpause);
527
515void strp_unpause(struct strparser *strp) 528void strp_unpause(struct strparser *strp)
516{ 529{
517 strp->paused = 0; 530 strp->paused = 0;