From mboxrd@z Thu Jan 1 00:00:00 1970 From: Carl-Daniel Hailfinger Subject: Re: [PATCH 02/02] add mask options to fwmark masking code Date: Mon, 20 Feb 2006 20:58:39 +0100 Message-ID: <43FA1F6F.2080408@gmx.net> References: <23140.1140452813@sandelman.ottawa.on.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: kuznet@ms2.inr.ac.ru, netfilter-devel , Jamal Hadi Salim , netdev@vger.kernel.org Return-path: To: Michael Richardson In-Reply-To: <23140.1140452813@sandelman.ottawa.on.ca> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: netfilter-devel-bounces@lists.netfilter.org Errors-To: netfilter-devel-bounces@lists.netfilter.org List-Id: netdev.vger.kernel.org Michael Richardson schrieb: > [PATCH] This patch introduces a mask to the fwmark test cases in the advanced > routing. This let's one test individual bits of the fwmark to determine > how things should be routed (pick a routing table). This patch retains > compatibility with tests that do not set the mask by assuming a mask > of 0 is equivalent to a mask of 0xffffffff. Sorry if I misunderstood the intention of your patch, but isn't similar code already in mainline? linux-2.6.16-rc3/net/sched/cls_u32.c:146 #ifdef CONFIG_CLS_U32_MARK if ((skb->nfmark & n->mark.mask) != n->mark.val) { Regards, Carl-Daniel -- http://www.hailfinger.org/