| To: | yoshfuji@xxxxxxxxxxxxxx |
|---|---|
| Subject: | Re: [PATCH] IPv6: Refine IPv6 Address Validation Timer |
| From: | "David S. Miller" <davem@xxxxxxxxxx> |
| Date: | Fri, 27 Sep 2002 02:25:15 -0700 (PDT) |
| Cc: | linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, usagi@xxxxxxxxxxxxxx, kuznet@xxxxxxxxxxxxx |
| In-reply-to: | <20020927.181256.112824147.yoshfuji@linux-ipv6.org> |
| References: | <20020927.181256.112824147.yoshfuji@linux-ipv6.org> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
From: YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>
Date: Fri, 27 Sep 2002 18:12:56 +0900 (JST)
This patch has problems.
@@ -1626,24 +1635,32 @@
for (ifp=inet6_addr_lst[i]; ifp; ifp=ifp->lst_next) {
unsigned long age;
- if (ifp->flags & IFA_F_PERMANENT)
+ spin_lock(&ifp->lock);
+ if (ifp->flags & IFA_F_PERMANENT) {
+ spin_unlock(&ifp->lock);
continue;
+ }
This is completely unnecessary. Nobody modifies the
IFA_F_PERMANENT flag after the addr entry has been added
to the hash table and this runs under the addrconf hash
lock already.
Alexey will have to comment on the rest.
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH] IPv6: Refine IPv6 Address Validation Timer, YOSHIFUJI Hideaki / 吉藤英明 |
|---|---|
| Next by Date: | Linux problems with hundreds of interfaces/routes, Petr Baudis |
| Previous by Thread: | [PATCH] IPv6: Refine IPv6 Address Validation Timer, YOSHIFUJI Hideaki / 吉藤英明 |
| Next by Thread: | Re: [PATCH] IPv6: Refine IPv6 Address Validation Timer, YOSHIFUJI Hideaki / 吉藤英明 |
| Indexes: | [Date] [Thread] [Top] [All Lists] |