From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: [net-next PATCH v5 3/3] net: reserve ports for applications using fixed port numbers Date: Sun, 21 Feb 2010 10:00:26 +0800 Message-ID: <4B8093BA.7010809@redhat.com> References: <1266532210-11536-1-git-send-email-opurdila@ixiacom.com> <1266532210-11536-3-git-send-email-opurdila@ixiacom.com> <4B7F9B65.1060309@redhat.com> <201002201527.37643.opurdila@ixiacom.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: David Miller , Linux Kernel Network Developers , Linux Kernel Developers , Neil Horman , Eric Dumazet , "Eric W. Biederman" To: Octavian Purdila Return-path: Received: from mx1.redhat.com ([209.132.183.28]:17767 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756239Ab0BUB5E (ORCPT ); Sat, 20 Feb 2010 20:57:04 -0500 In-Reply-To: <201002201527.37643.opurdila@ixiacom.com> Sender: netdev-owner@vger.kernel.org List-ID: Octavian Purdila wrote: > On Saturday 20 February 2010 10:20:53 you wrote: > >>> +unsigned long *sysctl_local_reserved_ports; >>> +EXPORT_SYMBOL(sysctl_local_reserved_ports); >>> + >> Sorry, this looks somewhat weird, why not just export >> inet_is_reserved_local_port()? >> > > My understanding is that if we do that than we won't be able to inline > inet_is_reserved_local_port(). And as David said previously that will have a > significant impact on performance. Oh, that would be true probably, so just leave as it is. Thanks.