Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp542486pxa; Fri, 31 Jul 2020 20:55:20 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxxna+WSzq3CCovHoS7FlyUyNRUOxuEsPKmA8O/8G7WR3a5TrzvgrLbKYI7BgXMj8X/pSxy X-Received: by 2002:a17:906:c213:: with SMTP id d19mr7442834ejz.291.1596254119809; Fri, 31 Jul 2020 20:55:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1596254119; cv=none; d=google.com; s=arc-20160816; b=zuueM21rFw2Jd/84XVeGcZboVchw3VuN8XLBc3Yd6zE/kyumpUIY5akeY2B/QQSYj1 b9ED0JNs4GlKHaX8jmvL3pGeQYRB05wNPL/gUa/noRoDmKgq7JftS0IpfGW1FKNZrWw0 CyNqIB5T0KyWJGTuhxu5fZ26VS1Wug8vqW9mbVXA2uohxoj9tvz8uRHabxpRTQSxKV1p 7uFtCiTWb6mL03obP3XDHgkvC/KCr1bzNaFZOFWn7rZ7XvwG1N/vprRgIzAwn8RteZAY bV8kmmOhvWPn4Ucdw8ZNHdQI0e15hN8zZAe0E36uCcVL9wJKvXfUWnpYOS1Ke7BcioML jZtQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=Gnn9lbsi+QeiyHza8FsKfu17u4FjzzG+2+QxHKbvkV0=; b=0R9GkTOOT3JvY59GMCzrJtxIsd2zewGGLYMcwX5F+cCplFy+TcHurfdlWzdPfe6fPn GDlgBA6DOwJLZYcuo0P7p2TAg9w35dYpMmPDPaUjOHHgn6jfAEL+Lfl7qi9TjQWq5WTS S+vTnaX4sr09YSsmdY5jH0CJTBemuP/zCHVCKnTp3BEJT+DRxkrXh7G85HVPhC34PC6K dUbn7DQYcfzArGaRo16Z57m1RIREwXSP/FskvjBxsXgQ8wPqlcC0mclKjYVqmgn2CTS4 Hxvo9EXyGz5GZMAJrkmpqHk8s7Gv9+1JjafbkBBPChs7LCFp5RSdon9gtcFsKQST4wwK FoJw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@infradead.org header.s=casper.20170209 header.b=koeUikLM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b24si3567499edy.309.2020.07.31.20.54.57; Fri, 31 Jul 2020 20:55:19 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@infradead.org header.s=casper.20170209 header.b=koeUikLM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728673AbgHADwi (ORCPT + 99 others); Fri, 31 Jul 2020 23:52:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60866 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728459AbgHADwh (ORCPT ); Fri, 31 Jul 2020 23:52:37 -0400 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 987BFC06174A; Fri, 31 Jul 2020 20:52:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=Content-Transfer-Encoding:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To:Subject:Sender :Reply-To:Content-ID:Content-Description; bh=Gnn9lbsi+QeiyHza8FsKfu17u4FjzzG+2+QxHKbvkV0=; b=koeUikLM4YRg0XyDmY1BhwjSlF uor/b+viYycgrcVXM4guo6CwQeLIHEQxmH4CguPqoRNS+l4qsDN/2PmO8M2FxWgLjyHjgjLENGqNr iT/GuKtmO5h6TH29C4GDiXSa0w8yZgN95XONwaxD52KbEb70ReRnZoxHp6g7jiqfRYf9vtJAy56sr 6CL/s96LlsZw6RGuqg/iwyB42lheEbryt0uA0AVDWDCT7RGfp2+RWThBjuwsVaGh3aivdUtH6Sbb3 FZL2tGXhLopnbwsqjbGDFyfocKIduTC6u1sS9/o/RS1kLbm+9ziQrzgQ3bNMX+ZzFh/TmzAit9517 taP6gV3A==; Received: from [2601:1c0:6280:3f0:897c:6038:c71d:ecac] by casper.infradead.org with esmtpsa (Exim 4.92.3 #3 (Red Hat Linux)) id 1k1iZV-0005AU-5K; Sat, 01 Aug 2020 03:52:33 +0000 Subject: Re: [PATCH net-next] fib: fix another fib_rules_ops indirect call wrapper problem To: Brian Vazquez , Brian Vazquez , Eric Dumazet , Paolo Abeni , "David S . Miller" Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Stephen Rothwell References: <20200801030110.747164-1-brianvv@google.com> From: Randy Dunlap Message-ID: <55292447-c426-8421-3147-ef7fa6e4b471@infradead.org> Date: Fri, 31 Jul 2020 20:52:29 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20200801030110.747164-1-brianvv@google.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 7/31/20 8:01 PM, Brian Vazquez wrote: > It turns out that on commit 41d707b7332f ("fib: fix fib_rules_ops > indirect calls wrappers") I forgot to include the case when > CONFIG_IP_MULTIPLE_TABLES is not set. > > Fixes: 41d707b7332f ("fib: fix fib_rules_ops indirect calls wrappers") > Reported-by: Randy Dunlap > Cc: Stephen Rothwell > Signed-off-by: Brian Vazquez Acked-by: Randy Dunlap # build-tested Thanks. > --- > net/core/fib_rules.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/net/core/fib_rules.c b/net/core/fib_rules.c > index fce645f6b9b10..a7a3f500a857b 100644 > --- a/net/core/fib_rules.c > +++ b/net/core/fib_rules.c > @@ -17,10 +17,16 @@ > #include > > #ifdef CONFIG_IPV6_MULTIPLE_TABLES > +#ifdef CONFIG_IP_MULTIPLE_TABLES > #define INDIRECT_CALL_MT(f, f2, f1, ...) \ > INDIRECT_CALL_INET(f, f2, f1, __VA_ARGS__) > #else > +#define INDIRECT_CALL_MT(f, f2, f1, ...) INDIRECT_CALL_1(f, f2, __VA_ARGS__) > +#endif > +#elif CONFIG_IP_MULTIPLE_TABLES > #define INDIRECT_CALL_MT(f, f2, f1, ...) INDIRECT_CALL_1(f, f1, __VA_ARGS__) > +#else > +#define INDIRECT_CALL_MT(f, f2, f1, ...) f(__VA_ARGS__) > #endif > > static const struct fib_kuid_range fib_kuid_range_unset = { > -- ~Randy