diff options
author | Lars Ellenberg <lars.ellenberg@linbit.com> | 2010-05-25 08:23:57 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2010-06-01 05:12:27 -0400 |
commit | 344fa462e3246bd102059ccc3c59deef416676dd (patch) | |
tree | 1e66ef2ce11d25214004468d16bac6b7a1dc24c6 /drivers/block | |
parent | 2c8d196759054b632788633b20e39167df36041d (diff) |
drbd: improve network latency, TCP_QUICKACK
On Thu, Apr 29, 2010 at 04:00:50PM -0400, Eduard.Guzovsky@stratus.com
wrote on drbd-dev@lists.linbit.com
Subject: [Drbd-dev] DRBD small synchronous writes performance improvements
> 1. TCP_QUICKACK option is set incorrectly. The goal was force TCP to
> send and ACK as a "one time" event. Instead the code permanently sets
> connection in the QUICKACK mode.
He is right, we actually want to use an even val with TCP_QUICKACK.
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/drbd/drbd_int.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/drbd/drbd_int.h b/drivers/block/drbd/drbd_int.h index c194348a46ed..6d79a76ba597 100644 --- a/drivers/block/drbd/drbd_int.h +++ b/drivers/block/drbd/drbd_int.h | |||
@@ -1540,7 +1540,7 @@ static inline void drbd_tcp_nodelay(struct socket *sock) | |||
1540 | 1540 | ||
1541 | static inline void drbd_tcp_quickack(struct socket *sock) | 1541 | static inline void drbd_tcp_quickack(struct socket *sock) |
1542 | { | 1542 | { |
1543 | int __user val = 1; | 1543 | int __user val = 2; |
1544 | (void) drbd_setsockopt(sock, SOL_TCP, TCP_QUICKACK, | 1544 | (void) drbd_setsockopt(sock, SOL_TCP, TCP_QUICKACK, |
1545 | (char __user *)&val, sizeof(val)); | 1545 | (char __user *)&val, sizeof(val)); |
1546 | } | 1546 | } |