netdev
[Top] [All Lists]

Re: [XFRM] Make XFRM core subsystem af-independent

To: yoshfuji@xxxxxxxxxxxxxx (YOSHIFUJI Hideaki / ????)
Subject: Re: [XFRM] Make XFRM core subsystem af-independent
From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date: Fri, 17 Sep 2004 22:40:52 +1000
Cc: davem@xxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <20040917.205038.61628907.yoshfuji@xxxxxxxxxxxxxx>
Organization: Core
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: tin/1.7.4-20040225 ("Benbecula") (UNIX) (Linux/2.4.26-1-686-smp (i686))
YOSHIFUJI Hideaki / ???? <yoshfuji@xxxxxxxxxxxxxx> wrote:
>
> ChangeSet@xxxxxx, 2004-09-17 20:13:22+09:00, yoshfuji@xxxxxxxxxxxxxx
> [IPV4] replace ip_route_output_key() with ip_route_output_flow().
> 
> Since ip_route_output_key() is identical to ip_route_output_flow(),
> except for the lack of the sk and the flags arguments, 
> let's use the later generic one.
> 
> Signed-off-by: Hideaki YOSHIFUJI <yoshfuji@xxxxxxxxxxxxxx>
> 
> diff -Nru a/net/atm/clip.c b/net/atm/clip.c
> --- a/net/atm/clip.c    2004-09-17 20:14:58 +09:00
> +++ b/net/atm/clip.c    2004-09-17 20:14:58 +09:00
> @@ -556,7 +556,7 @@
>                unlink_clip_vcc(clip_vcc);
>                return 0;
>        }
> -       error = ip_route_output_key(&rt,&fl);
> +       error = ip_route_output_flow(&rt, &fl, NULL, 0);

Yuck.  You've bloated the source as well as the binary for the sake
of deleting one function.

Please provide ip_route_output_key() as an inline wrapper around
ip_route_output_flow() so that its users don't have to change.
-- 
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>