From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BB879C4332F for ; Wed, 9 Nov 2022 02:42:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229818AbiKICmV (ORCPT ); Tue, 8 Nov 2022 21:42:21 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33920 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229453AbiKICmU (ORCPT ); Tue, 8 Nov 2022 21:42:20 -0500 Received: from mail-pf1-x449.google.com (mail-pf1-x449.google.com [IPv6:2607:f8b0:4864:20::449]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5D4EE1A83C for ; Tue, 8 Nov 2022 18:42:20 -0800 (PST) Received: by mail-pf1-x449.google.com with SMTP id cj8-20020a056a00298800b0056cee8a0cf8so8088950pfb.9 for ; Tue, 08 Nov 2022 18:42:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=4Mz4zEcIKBrlDyAmKkP8D55UhnpWD1j4J0ZUJ0geBMw=; b=JhcYXGD2COekAUcZzHL8U9BIh7stDBFD7YenyGw4kjTwzCbPWYUbVkDexYLlSQBXN5 bJT/6/x+DHJY7IRVw4FCcb4ZmLORBnMo6V6FJpc2BnaU8kYwKO5IZmJuRKVwG+9n7ULS pk1WClMef+rhKaEf+wtDtwIz/Pa2lH643l7E+gUnJn4iC1waoHqWkFzIJ5X3w+TMeDoX dpaGqib14Den7EzSKBSSlJ7ikjcvCd8ymDqxpfgDuDhoEH11OnBbbhLJd3kWdb+sJfI1 nPA1qi02HFS5VHxaSMdVApxfsemedy/mSyn4Vh7L1PEfrP4eja5/0z6oZQ9xSLgxUwXs zrsA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=4Mz4zEcIKBrlDyAmKkP8D55UhnpWD1j4J0ZUJ0geBMw=; b=tHcwyDzjYXZBIL+Gt+ncfEK1khPpZgTHuUkpH7YwAbMAgJyaL8BDRzn7SL2mT+i37M vbo5g3j4LEbFXAHHgOuRUwct973BRuFs4fuolSHKZnGvnliMReMPbhAxOCeNpihL4cTQ lc5CguBiD7iN6ZiY7tt8Carp3QuPg11DB/EnRzwS9F6WPWfaeZGUVjxPO+uEVYmdliyt 9YsRWt+a+qbQ4Qa/g/U61d/X6yc2jEDepVVMXIk0sLIfsiTYms1C+9z7NFruIxJdZP3y 7JkLWBVbQLJDh/cjU3FCRS3ZpjYjUpXmfl2vEzdvfVtmtPwxXucwBBjZu9RIumoWIxxk f+3w== X-Gm-Message-State: ACrzQf1PlQygVbUzfSpTl3hr2sKoeVjafWPCndmxnskqup9Ut0yH13DG H95hDK5e76ZL4vkVOejCxwNlvYYOAp6e1Cd7j9oddLDtOyq9gyc1iz3nFRGQUSse5DDenKjDaZS sco2n3kEu60XaM536ThCgGN+ZqzLUlo1045iWf2H/+OLE+HBpqr3oZ7fa87m+ X-Google-Smtp-Source: AMsMyM47HuLO5TgBNWTp7jQWcJrJ68COM+oS3++tmtUkWz28Cdhx1sZHVJKmHynm5AGq2rCNdfY1639MXAXk X-Received: from connoro.c.googlers.com ([fda3:e722:ac3:cc00:7f:e700:c0a8:a99]) (user=connoro job=sendgmr) by 2002:aa7:820f:0:b0:562:83c1:59e0 with SMTP id k15-20020aa7820f000000b0056283c159e0mr1116355pfi.34.1667961739606; Tue, 08 Nov 2022 18:42:19 -0800 (PST) Date: Wed, 9 Nov 2022 02:41:55 +0000 Mime-Version: 1.0 X-Mailer: git-send-email 2.38.1.431.g37b22c650d-goog Message-ID: <20221109024155.2810410-1-connoro@google.com> Subject: [PATCH bpf-next] bpf: btf: don't log ignored BTF mismatches From: "Connor O'Brien" To: bpf@vger.kernel.org Cc: Martin KaFai Lau , Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Song Liu , Yonghong Song , John Fastabend , KP Singh , Stanislav Fomichev , Hao Luo , Jiri Olsa , linux-kernel@vger.kernel.org, "Connor O'Brien" Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: bpf@vger.kernel.org Enabling CONFIG_MODULE_ALLOW_BTF_MISMATCH is an indication that BTF mismatches are expected and module loading should proceed anyway. Logging with pr_warn() on every one of these "benign" mismatches creates unnecessary noise when many such modules are loaded. Instead, limit logging to the case where a BTF mismatch actually prevents a module form loading. Signed-off-by: Connor O'Brien --- kernel/bpf/btf.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/kernel/bpf/btf.c b/kernel/bpf/btf.c index 5579ff3a5b54..406370487413 100644 --- a/kernel/bpf/btf.c +++ b/kernel/bpf/btf.c @@ -7190,11 +7190,12 @@ static int btf_module_notify(struct notifier_block *nb, unsigned long op, } btf = btf_parse_module(mod->name, mod->btf_data, mod->btf_data_size); if (IS_ERR(btf)) { - pr_warn("failed to validate module [%s] BTF: %ld\n", - mod->name, PTR_ERR(btf)); kfree(btf_mod); - if (!IS_ENABLED(CONFIG_MODULE_ALLOW_BTF_MISMATCH)) + if (!IS_ENABLED(CONFIG_MODULE_ALLOW_BTF_MISMATCH)) { + pr_warn("failed to validate module [%s] BTF: %ld\n", + mod->name, PTR_ERR(btf)); err = PTR_ERR(btf); + } goto out; } err = btf_alloc_id(btf); -- 2.38.1.431.g37b22c650d-goog