netdev
[Top] [All Lists]

Re: [RFC: 2.6 patch] net/ipv4/: possible cleanups

To: bunk@xxxxxxxxx
Subject: Re: [RFC: 2.6 patch] net/ipv4/: possible cleanups
From: "David S. Miller" <davem@xxxxxxxxxxxxx>
Date: Thu, 02 Jun 2005 13:13:14 -0700 (PDT)
Cc: akpm@xxxxxxxx, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20050530205651.GY10441@xxxxxxxxx>
References: <20050530205651.GY10441@xxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
From: Adrian Bunk <bunk@xxxxxxxxx>
Subject: [RFC: 2.6 patch] net/ipv4/: possible cleanups
Date: Mon, 30 May 2005 22:56:51 +0200

> This patch contains the following possible cleanups:
> - make needlessly global code static
> - #if 0 the following unused global function:
>   - xfrm4_state.c: xfrm4_state_fini
> - remove the following unneeded EXPORT_SYMBOL's:
>   - ip_output.c: ip_finish_output
>   - ip_output.c: sysctl_ip_default_ttl
>   - fib_frontend.c: ip_dev_find
>   - inetpeer.c: inet_peer_idlock
>   - ip_options.c: ip_options_compile
>   - ip_options.c: ip_options_undo
>   - tcp_ipv4.c: sysctl_max_syn_backlog
> 
> Please review which of these changes are correct and which might 
> conflict with pending patches.

Please keep all of the ECN implementation in the
tcp_ecn.h header file, even if the routine is only
called in one C file.

And therefore, please do not remove the tcp_enter_quickack_mode()
extern declaration from tcp.h

Thanks.

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [RFC: 2.6 patch] net/ipv4/: possible cleanups, David S. Miller <=