From: Florian Westphal > David Laight <David.Laight@xxxxxxxxxx> wrote: > > From: Pablo Neira Ayuso > > > +#define CONNLIMIT_SLOTS 256 > > > +#define CONNLIMIT_LOCK_SLOTS 32 > > > > You might want to make these explicitly unsigned to ensure > > the divisions are unsigned (so can be done with a simple mask). > > I can do this but I checked that at least on my machine gcc already > emits a 'add' instruction to reduce the hash value. I think you meant 'and' :-) Yes the hash itself is probably unsigned so forces an unsigned modulus. It just seemed better to force it from the other argument as well. David -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
