netdev
[Top] [All Lists]

Re: [PATCH] Move inetdev/ifa over to RCU

To: "David S. Miller" <davem@xxxxxxxxxx>
Subject: Re: [PATCH] Move inetdev/ifa over to RCU
From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date: Sat, 14 Aug 2004 15:08:48 +1000
Cc: shemminger@xxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <20040814013030.GA2042@xxxxxxxxxxxxxxxxxxx>
References: <20040812165954.00429e65.davem@xxxxxxxxxx> <20040813090314.448c971d@xxxxxxxxxxxxxxxxxxxxx> <20040813093838.6961c0d4.davem@xxxxxxxxxx> <20040813215602.GA15870@xxxxxxxxxxxxxxxxxxx> <20040813151923.3311b4f0.davem@xxxxxxxxxx> <20040814003428.GA17760@xxxxxxxxxxxxxxxxxxx> <20040813173924.6d05be15.davem@xxxxxxxxxx> <20040814005411.GA18350@xxxxxxxxxxxxxxxxxxx> <20040814012513.GA721@xxxxxxxxxxxxxxxxxxx> <20040814013030.GA2042@xxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6+20040523i
Hi:

I'm just going through all the __in_dev_get() callers and the one
in ip_route_output_slow() looks fishy.  It appears to be checking
whether the subsequent inet_select_addr() calls will succeed or not.

But this is not reliable since the addresses can always disappear
between the check and the actual call.

Do we really care about the zero return value of inet_select_addr()
here? What about the other calls to inet_select_addr()?
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

<Prev in Thread] Current Thread [Next in Thread>