Rainer Sabelka wrote:
On Wednesday 11 June 2008 15:25, Pablo Neira Ayuso wrote:
Are your scripts committing the entries twice (ie. invoking conntrackd
-c several times)?
In my case - yes I did.
The only way to reproduce this that I have found is
to double insert an existing conntrack with some NAT handling. In the
upcoming 2.6.26 you'll get a EBUSY instead of EINVAL which sounds more
reasonable.
Anyhow, does the patch attached fix this behaviour? The idea behind it
is to check if there is a conntrack present in kernel, if so, just
update the attributes of the conntrack object that are changeable to
avoid the error. Would you mind testing it?
Thanks for the patch!
Now I see no more "commit: Invalid argument" in the logs. Instead I get
something like this, which looks much fiendlier:
Jun 11 15:36:48 fw1b conntrack-tools[13273]: committing external cache
Jun 11 15:36:48 fw1b conntrack-tools[13273]: Committed 69 new entries
Jun 11 15:36:48 fw1b conntrack-tools[13273]: 53 entries ignored, already exist
^^^
I'll also fix the message (those entries are not ignored but updated).
Then I'll commit the patch asap.
But in rare cases I can see "commit-create: Cannot allocate memory".
I also noticed this a few times before applying this patch. Is this something
I should worry about?
Yes, very strange. ctnetlink is hitting ENOMEM, ie. it cannot create the
conntrack because there's no memory available. As for now ctnetlink is
allocating conntracks via nf_conntrack_alloc() which uses GFP_ATOMIC.
I'll send you a patch to relax this to check if that is the problem,
otherwise I think that the error is bogus.
Jun 11 15:40:07 fw1b conntrack-tools[13383]: committing external cache
Jun 11 15:40:07 fw1b conntrack-tools[13383]: commit-create: Cannot allocate
memory
Jun 11 15:40:07 fw1b conntrack-tools[13383]: Committed 33 new entries
Jun 11 15:40:07 fw1b conntrack-tools[13383]: 25 entries ignored, already exist
Jun 11 15:40:07 fw1b conntrack-tools[13383]: 1 entries can't be committed
What it is really annoying is the fact that you hit error with that less
entries. I cannot reproduce that in my testbed with ~20000 entries. Are
you using some kind of embedded device? Architecture?
--
"Los honestos son inadaptados sociales" -- Les Luthiers
--
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
