From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pablo Neira Ayuso Subject: Re: [PATCH 3/8] [PATCH] Helper modules load-on-demand support for ctnetlink Date: Thu, 09 Oct 2008 16:50:27 +0200 Message-ID: <48EE1A33.4010603@netfilter.org> References: <20081009083339.9824.24056.stgit@Decadence> <20081009083422.9824.35589.stgit@Decadence> <48EE085E.2020701@trash.net> <48EE0A9A.7010706@trash.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: netfilter-devel@vger.kernel.org To: Patrick McHardy Return-path: Received: from mail.us.es ([193.147.175.20]:43579 "EHLO us.es" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752945AbYJIPFq (ORCPT ); Thu, 9 Oct 2008 11:05:46 -0400 In-Reply-To: <48EE0A9A.7010706@trash.net> Sender: netfilter-devel-owner@vger.kernel.org List-ID: Patrick McHardy wrote: > Patrick McHardy wrote: >> This one doesn't apply: >> >> patching file include/net/netfilter/nf_conntrack_helper.h >> patching file net/netfilter/nf_conntrack_core.c >> Hunk #1 FAILED at 581. >> Hunk #2 succeeded at 753 (offset -5 lines). >> Hunk #3 succeeded at 764 (offset -5 lines). >> 1 out of 3 hunks FAILED -- saving rejects to file >> net/netfilter/nf_conntrack_core.c.rej >> patching file net/netfilter/nf_conntrack_helper.c >> patching file net/netfilter/nf_conntrack_netlink.c >> Hunk #2 succeeded at 1131 with fuzz 1. >> patching file net/netfilter/nfnetlink.c > > Oops, I think I didn't refresh my tree, so it didn't include any > of the netfilter patches I sent to Dave :) So, what do I have to do now? I'm cloning Dave's tree now to put the patches on top on it. Fine with it? > The remaining issues, especially the nfnl_lock think, should be > fixed though. OK. BTW, returning EOPNOTSUPP looks better to me. ENOENT is misleading since the user won't be able to differenciate between "I don't know anything about that helper" and "this conntrack entry does not exist". So, I prefer using EOPNOTSUPP. -- "Los honestos son inadaptados sociales" -- Les Luthiers