Commit 0dd0219b authored by Theodore Y. Ts'o's avatar Theodore Y. Ts'o Committed by Linus Torvalds

[PATCH] dev/random: Fix latency in rekeying sequence number

Based on reports from Ingo's Latency Tracer that the TCP sequence number
rekey code is causing latency problems, I've moved the sequence number
rekey to be done out of a workqueue.
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 467474d0
...@@ -2247,30 +2247,35 @@ static struct keydata { ...@@ -2247,30 +2247,35 @@ static struct keydata {
static spinlock_t ip_lock = SPIN_LOCK_UNLOCKED; static spinlock_t ip_lock = SPIN_LOCK_UNLOCKED;
static unsigned int ip_cnt; static unsigned int ip_cnt;
static struct keydata *__check_and_rekey(time_t time) static void rekey_seq_generator(void *private_)
{ {
struct keydata *keyptr; struct keydata *keyptr;
struct timeval tv;
do_gettimeofday(&tv);
spin_lock_bh(&ip_lock); spin_lock_bh(&ip_lock);
keyptr = &ip_keydata[ip_cnt&1]; keyptr = &ip_keydata[ip_cnt&1];
if (!keyptr->rekey_time || (time - keyptr->rekey_time) > REKEY_INTERVAL) {
keyptr = &ip_keydata[1^(ip_cnt&1)]; keyptr = &ip_keydata[1^(ip_cnt&1)];
keyptr->rekey_time = time; keyptr->rekey_time = tv.tv_sec;
get_random_bytes(keyptr->secret, sizeof(keyptr->secret)); get_random_bytes(keyptr->secret, sizeof(keyptr->secret));
keyptr->count = (ip_cnt&COUNT_MASK)<<HASH_BITS; keyptr->count = (ip_cnt&COUNT_MASK)<<HASH_BITS;
mb(); mb();
ip_cnt++; ip_cnt++;
}
spin_unlock_bh(&ip_lock); spin_unlock_bh(&ip_lock);
return keyptr;
} }
static DECLARE_WORK(rekey_work, rekey_seq_generator, NULL);
static inline struct keydata *check_and_rekey(time_t time) static inline struct keydata *check_and_rekey(time_t time)
{ {
struct keydata *keyptr = &ip_keydata[ip_cnt&1]; struct keydata *keyptr = &ip_keydata[ip_cnt&1];
rmb(); rmb();
if (!keyptr->rekey_time || (time - keyptr->rekey_time) > REKEY_INTERVAL) { if (!keyptr->rekey_time || (time - keyptr->rekey_time) > REKEY_INTERVAL) {
keyptr = __check_and_rekey(time); schedule_work(&rekey_work);
} }
return keyptr; return keyptr;
......
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