From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [PATCH nf-next-2.6] conntrack: IPS_UNTRACKED bit Date: Tue, 08 Jun 2010 16:12:01 +0200 Message-ID: <4C0E4FB1.8080201@trash.net> References: <1271941082.14501.189.camel@jdb-workstation> <1271946961.7895.5665.camel@edumazet-laptop> <1271948029.7895.5707.camel@edumazet-laptop> <20100422155123.GA2524@linux.vnet.ibm.com> <1271952128.7895.5851.camel@edumazet-laptop> <1272056237.4599.7.camel@edumazet-laptop> <1272139861.20714.525.camel@edumazet-laptop> <1272292568.13192.43.camel@jdb-workstation> <1275340896.2478.26.camel@edumazet-laptop> <1275368732.2478.88.camel@edumazet-laptop> <4C04DE73.6050605@trash.net> <1275388310.2738.2.camel@edumazet-laptop> <4C04E3E2.7020209@trash.net> <1275409203.2738.227.camel@edumazet-laptop> <4C08E62A.9020607@trash.net> < AANLkTilKEpFTg5bH8d9UE3a3DVJNAGYz10Jgkt6lXtJ0@mail.gmail.com> <4C08F1A4.2050906@trash.net> <1275654964.2482.150.camel@edumazet-laptop> <1275668732.2482.201.camel@edumazet-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: Changli Gao , Netfilter Developers , netdev To: Eric Dumazet Return-path: Received: from stinky.trash.net ([213.144.137.162]:44094 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755320Ab0FHOMF (ORCPT ); Tue, 8 Jun 2010 10:12:05 -0400 In-Reply-To: <1275668732.2482.201.camel@edumazet-laptop> Sender: netfilter-devel-owner@vger.kernel.org List-ID: On 04.06.2010 18:25, Eric Dumazet wrote: > [PATCH nf-next-2.6] conntrack: IPS_UNTRACKED bit > > NOTRACK makes all cpus share a cache line on nf_conntrack_untracked > twice per packet. This is bad for performance. > __read_mostly annotation is also a bad choice. > > This patch introduces IPS_UNTRACKED bit so that we can use later a > per_cpu untrack structure more easily. > > A new helper, nf_ct_untracked_get() returns a pointer to > nf_conntrack_untracked. > > Another one, nf_ct_untracked_status_or() is used by nf_nat_init() to add > IPS_NAT_DONE_MASK bits to untracked status. > > nf_ct_is_untracked() prototype is changed to work on a nf_conn pointer. Applied, thanks Eric.