From patchwork Wed Dec 9 13:57:36 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Toke_H=C3=B8iland-J=C3=B8rgensen?= X-Patchwork-Id: 11961547 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 56F91C4361B for ; Wed, 9 Dec 2020 13:59:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 26ED723B31 for ; Wed, 9 Dec 2020 13:59:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732065AbgLIN7H (ORCPT ); Wed, 9 Dec 2020 08:59:07 -0500 Received: from us-smtp-delivery-124.mimecast.com ([63.128.21.124]:52457 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726864AbgLIN7H (ORCPT ); Wed, 9 Dec 2020 08:59:07 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1607522260; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=DeJICkmUhNlMDPR60YNJsvyrM89/RB2B3sOMO3Orpnc=; b=jM5bbl2NMOz5cGGsAa3q++GW3BkSK2ONnTD4bo3bquYt4AagruggRu9+ZAfFEka2rb5X40 vxSJL1S5o5TVWAam5a5AH1l43oiOu+uGiAGgJ6+M5BhjL4sCD2HnmqYa7omFTCogB5KBI5 DUCBOA3BCMq/anKgxbOmmL6db2Gun3E= Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-196-lEGOM1O7N1e-YwTLoYN22Q-1; Wed, 09 Dec 2020 08:57:39 -0500 X-MC-Unique: lEGOM1O7N1e-YwTLoYN22Q-1 Received: by mail-wr1-f70.google.com with SMTP id v1so687573wri.16 for ; Wed, 09 Dec 2020 05:57:38 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:cc:date:message-id:user-agent :mime-version:content-transfer-encoding; bh=DeJICkmUhNlMDPR60YNJsvyrM89/RB2B3sOMO3Orpnc=; b=OhJBqHwUyNrIMlnfv17m9nggcLJpBNOwf4usst0xCyoQR0wI1riHKQFUt2zqdBh8zk ej+Vp0/ltiy8pBp8D+EdglCvdyrVYoCcL7LmujpHqMnT9D5U5Z+LzmMy77k3DSFywNwY IfwnGiZvEGdCIF2cGxe2nl0/qZaPNbM2WU10ztV+rmmRxaefQxKRu4okEjDJnsiRVEn6 n/e72qmKpOWFrm+zt07NptL2NLtCnjuuTuyzwLpWP7sv44DplVwA8OVqWZuz/qpvgUMp oPhZd6gnVqE0vNSKUzkWbgf9uHLJGq4S+tKUInH8wh3l+9a1KiX8ypfYtAdbmkH0e5gS CmHQ== X-Gm-Message-State: AOAM532KoPXqJf6Xs0Hvao045/J+GuR83ScKswrfvmH4hlzdoSeC6dM0 7bhAVBexigQyzawKI1358R2b0AmbFtDAPzWBtJst3XUNJ4ioRRfqak0SLQeTxLFWhvY0yp5knsG 9SVkIbNlkDXQU/xy/ X-Received: by 2002:a5d:4ece:: with SMTP id s14mr2809542wrv.427.1607522257605; Wed, 09 Dec 2020 05:57:37 -0800 (PST) X-Google-Smtp-Source: ABdhPJyidXDr7AGB88RVIa+qRbPwe75cFxUfdyia554wfcWfHyTBhIc9MtCkfrARVIyBxhWAKdS3PQ== X-Received: by 2002:a5d:4ece:: with SMTP id s14mr2809518wrv.427.1607522257401; Wed, 09 Dec 2020 05:57:37 -0800 (PST) Received: from alrua-x1.borgediget.toke.dk ([45.145.92.2]) by smtp.gmail.com with ESMTPSA id m21sm3556677wml.13.2020.12.09.05.57.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Dec 2020 05:57:36 -0800 (PST) Received: by alrua-x1.borgediget.toke.dk (Postfix, from userid 1000) id 7C78F180003; Wed, 9 Dec 2020 14:57:36 +0100 (CET) Subject: [PATCH bpf v4 0/7] selftests/bpf: Restore test_offload.py to working order From: =?utf-8?q?Toke_H=C3=B8iland-J=C3=B8rgensen?= To: Jakub Kicinski Cc: "David S. Miller" , Daniel Borkmann , Alexei Starovoitov , Andrii Nakryiko , Martin KaFai Lau , Song Liu , Yonghong Song , John Fastabend , KP Singh , Jesper Dangaard Brouer , "Michael S. Tsirkin" , Romain Perier , Allen Pais , Grygorii Strashko , Simon Horman , "Gustavo A. R. Silva" , Lorenzo Bianconi , Wei Yongjun , Jiri Benc , oss-drivers@netronome.com, linux-omap@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org Date: Wed, 09 Dec 2020 14:57:36 +0100 Message-ID: <160752225643.110217.4104692937165406635.stgit@toke.dk> User-Agent: StGit/0.23 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: bpf@iogearbox.net This series restores the test_offload.py selftest to working order. It seems a number of subtle behavioural changes have crept into various subsystems which broke test_offload.py in a number of ways. Most of these are fairly benign changes where small adjustments to the test script seems to be the best fix, but one is an actual kernel bug that I've observed in the wild caused by a bad interaction between xdp_attachment_flags_ok() and the rework of XDP program handling in the core netdev code. Patch 1 fixes the bug by removing xdp_attachment_flags_ok(), and the reminder of the patches are adjustments to test_offload.py, including a new feature for netdevsim to force a BPF verification fail. Please see the individual patches for details. Changelog: v4: - Accidentally truncated the Fixes: hashes in patches 3/4 to 11 chars v3: - Add Fixes: tags v2: - Replace xdp_attachment_flags_ok() with a check in dev_xdp_attach() - Better packing of struct nsim_dev --- Toke Høiland-Jørgensen (7): xdp: remove the xdp_attachment_flags_ok() callback selftests/bpf/test_offload.py: Remove check for program load flags match netdevsim: Add debugfs toggle to reject BPF programs in verifier selftests/bpf/test_offload.py: only check verifier log on verification fails selftests/bpf/test_offload.py: fix expected case of extack messages selftests/bpf/test_offload.py: reset ethtool features after failed setting selftests/bpf/test_offload.py: filter bpftool internal map when counting maps drivers/net/netdevsim/bpf.c | 12 ++++- drivers/net/netdevsim/netdevsim.h | 1 + tools/testing/selftests/bpf/test_offload.py | 53 +++++++++++---------- 3 files changed, 40 insertions(+), 26 deletions(-)