| To: | ja@xxxxxx (Julian Anastasov) |
|---|---|
| Subject: | Re: [IPv4]: More fib_alias insertion fixes |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Mon, 27 Sep 2004 07:23:16 +1000 |
| Cc: | herbert@xxxxxxxxxxxxxxxxxxx, davem@xxxxxxxxxxxxx, netdev@xxxxxxxxxxx |
| In-reply-to: | <Pine.LNX.4.58.0409261611230.8973@u.domain.uli> |
| Organization: | Core |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | tin/1.7.4-20040225 ("Benbecula") (UNIX) (Linux/2.4.27-hx-1-686-smp (i686)) |
Julian Anastasov <ja@xxxxxx> wrote: > > True, but in your patch2 you return fa after > list_for_each_entry which is not valid when the loop terminates. > Just change it to return the fa into the loop. You're right, we dereference it in fn_hash_insert. > [-- text/plain, encoding base64, charset: US-ASCII, 28 lines, name: > fib_ins3.diff --] > [-- Description: fib_alias ordered by TOS --] > > diff -ur v2.6.9-rc2-bk10/linux/net/ipv4/fib_hash.c linux/net/ipv4/fib_hash.c > --- v2.6.9-rc2-bk10/linux/net/ipv4/fib_hash.c 2004-09-26 16:52:11.000000000 > +0300 Looks good. Thanks, -- 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> |
|---|---|---|
| ||
| Previous by Date: | recommendations for NIC HW(/SW) design?, Lennert Buytenhek |
|---|---|
| Next by Date: | [PATCH] r8169 tx_timeout recovery and automatic DAC step-down, Francois Romieu |
| Previous by Thread: | Re: [IPv4]: More fib_alias insertion fixes, Julian Anastasov |
| Next by Thread: | Re: [IPv4]: More fib_alias insertion fixes, David S. Miller |
| Indexes: | [Date] [Thread] [Top] [All Lists] |