diff options
| author |   Florian Westphal <fw@strlen.de> | 2014-07-24 16:50:36 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-07-27 22:34:36 -0700 |
| commit | ab1c724f633080ed2e8a0cfe61654599b55cf8f9 (patch) | |
| tree | 215b28da3dc0fbbac8dd853501306f65b165a499 /net/ieee802154 | |
| parent | inet: frag: remove periodic secret rebuild timer (diff) | |
| download | linux-dev-ab1c724f633080ed2e8a0cfe61654599b55cf8f9.tar.xz linux-dev-ab1c724f633080ed2e8a0cfe61654599b55cf8f9.zip | |
inet: frag: use seqlock for hash rebuild
rehash is rare operation, don't force readers to take
the read-side rwlock.
Instead, we only have to detect the (rare) case where
the secret was altered while we are trying to insert
a new inetfrag queue into the table.
If it was changed, drop the bucket lock and recompute
the hash to get the 'new' chain bucket that we have to
insert into.
Joint work with Nikolay Aleksandrov.
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ieee802154')
| -rw-r--r-- | net/ieee802154/reassembly.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/ieee802154/reassembly.c b/net/ieee802154/reassembly.c index 20d219682d84..8da635d92a58 100644 --- a/net/ieee802154/reassembly.c +++ b/net/ieee802154/reassembly.c @@ -124,7 +124,6 @@ fq_find(struct net *net, const struct lowpan_frag_info *frag_info, arg.src = src; arg.dst = dst; - read_lock(&lowpan_frags.lock); hash = lowpan_hash_frag(frag_info->d_tag, frag_info->d_size, src, dst); q = inet_frag_find(&ieee802154_lowpan->frags, |