From owner-svn-src-all@FreeBSD.ORG Wed Mar 14 09:03:43 2012 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx2.freebsd.org (mx2.freebsd.org [69.147.83.53]) by hub.freebsd.org (Postfix) with ESMTP id 42257106566B; Wed, 14 Mar 2012 09:03:43 +0000 (UTC) (envelope-from melifaro@FreeBSD.org) Received: from dhcp170-36-red.yandex.net (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx2.freebsd.org (Postfix) with ESMTP id 2E78C14E13A; Wed, 14 Mar 2012 09:03:21 +0000 (UTC) Message-ID: <4F605E3A.5050702@FreeBSD.org> Date: Wed, 14 Mar 2012 13:00:42 +0400 From: "Alexander V. Chernikov" User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:8.0) Gecko/20111117 Thunderbird/8.0 MIME-Version: 1.0 To: Marko Zec References: <201203121541.q2CFfa5G071067@svn.freebsd.org> <201203121817.41503.zec@fer.hr> In-Reply-To: <201203121817.41503.zec@fer.hr> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r232868 - head/sys/netinet/ipfw X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Mar 2012 09:03:43 -0000 On 12.03.2012 21:17, Marko Zec wrote: > On Monday 12 March 2012 16:41:36 Alexander V. Chernikov wrote: >> Author: melifaro >> Date: Mon Mar 12 15:41:36 2012 >> New Revision: 232868 >> URL: http://svn.freebsd.org/changeset/base/232868 >> >> Log: >> Fix VNET build broken by r232865. >> Temporary remove the ability to assign different number of tables per >> VNET instance. > > Please don't let your code remain in this state for too long. You've declared > V_fw_tables_max to unconditionally resolve to a true global variable, which > violates the convention of using V_ prefix in networking code exclusively for > the purpose of denoting VNET-virtualized state. I understand this. I plan to change it to normal state today or tomorrow. Is this OK? Thanks for noting. > > Thanks, > > Marko > > >> Modified: >> head/sys/netinet/ipfw/ip_fw2.c >> head/sys/netinet/ipfw/ip_fw_private.h >> >> Modified: head/sys/netinet/ipfw/ip_fw2.c >> =========================================================================== >> === --- head/sys/netinet/ipfw/ip_fw2.c Mon Mar 12 15:13:17 2012 (r232867) >> +++ head/sys/netinet/ipfw/ip_fw2.c Mon Mar 12 15:41:36 2012 (r232868) @@ >> -117,7 +117,7 @@ VNET_DEFINE(int, autoinc_step); >> VNET_DEFINE(int, fw_one_pass) = 1; >> >> /* Use 128 tables by default */ >> -VNET_DEFINE(int, fw_tables_max) = IPFW_TABLES_MAX; >> +int fw_tables_max = IPFW_TABLES_MAX; >> >> /* >> * Each rule belongs to one of 32 different sets (0..31). >> >> Modified: head/sys/netinet/ipfw/ip_fw_private.h >> =========================================================================== >> === --- head/sys/netinet/ipfw/ip_fw_private.h Mon Mar 12 15:13:17 >> 2012 (r232867) +++ head/sys/netinet/ipfw/ip_fw_private.h Mon Mar 12 >> 15:41:36 2012 (r232868) @@ -209,8 +209,8 @@ VNET_DECLARE(u_int32_t, >> set_disable); >> VNET_DECLARE(int, autoinc_step); >> #define V_autoinc_step VNET(autoinc_step) >> >> -VNET_DECLARE(int, fw_tables_max); >> -#define V_fw_tables_max VNET(fw_tables_max) >> +extern int fw_tables_max; >> +#define V_fw_tables_max fw_tables_max >> >> struct ip_fw_chain { >> struct ip_fw *rules; /* list of rules */ > > > -- WBR, Alexander