From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Maciej_=C5=BBenczykowski?= Subject: Re: [PATCH] netfilter: remove modulo operations Date: Sun, 24 Oct 2010 14:49:17 -0700 Message-ID: References: <1287891635-30207-1-git-send-email-xiaosuo@gmail.com> <20101024182508.GM15247@Chamillionaire.breakpoint.cc> <1287947511.2658.1678.camel@edumazet-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Florian Westphal , Changli Gao , Patrick McHardy , netfilter-devel@vger.kernel.org To: Eric Dumazet Return-path: Received: from mail-wy0-f174.google.com ([74.125.82.174]:35993 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932985Ab0JXVuB (ORCPT ); Sun, 24 Oct 2010 17:50:01 -0400 Received: by wyf28 with SMTP id 28so2707499wyf.19 for ; Sun, 24 Oct 2010 14:50:00 -0700 (PDT) In-Reply-To: <1287947511.2658.1678.camel@edumazet-laptop> Sender: netfilter-devel-owner@vger.kernel.org List-ID: > Absolutely not. total is a 32bit number, using full range. Strictly speaking I think you mean 'hash' here, which as a hash should indeed be a random value across a full 32-bit range. Total might very well be pretty small. Anyway, without having tested, the change looks fine to me.