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 X-Spam-Level: X-Spam-Status: No, score=-9.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,USER_AGENT_GIT autolearn=ham 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 2D827C10F14 for ; Thu, 11 Apr 2019 10:26:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 08D352184E for ; Thu, 11 Apr 2019 10:26:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726691AbfDKK0l (ORCPT ); Thu, 11 Apr 2019 06:26:41 -0400 Received: from mail-wm1-f68.google.com ([209.85.128.68]:34516 "EHLO mail-wm1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726635AbfDKK0k (ORCPT ); Thu, 11 Apr 2019 06:26:40 -0400 Received: by mail-wm1-f68.google.com with SMTP id r186so1609039wmf.1 for ; Thu, 11 Apr 2019 03:26:39 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=Gi3SOf8ZFfPE0FWhRc13sB95XG0PUYYR4jIgV6Oweg4=; b=WqdWRTJEuf2wFh7paFR+DijGSZQ743M2h0zKSRDxilPQpkCnukY9zjrppEd+x6ubY4 puCGOZ1RfLvsTSCM1jfrRCKUmLmYtIUbDftAWeIvzhfZVVniEWd037r0XE2nWEADexF7 b9Q/n6iPQ3GGtvSs0mWjaP9IWRCChA8Y40STmPt53PRsDX2GkkiC1n2+G6AjSxcY/byX q76AgYYckQ+rS40m7Ijy59Tn/1G+0o9D/ucewmJL4rua6FUbdh0AXdCJIJmUh7YGwTtI zMAymCLu67WruuBqlLWvXANPCbvancZBk6u6VbBTGCsbYVe7zScj/zeIIJx5ylpdI+K/ cSZg== X-Gm-Message-State: APjAAAUwLJ7Pp2wLuW2evrrpLbHj4Ib9/gmSIa/MfHWNKcn4+BPoDDpx 6srO1UfngOts6TJj1Fk2TGd4q6xyx1Y= X-Google-Smtp-Source: APXvYqy88040TxWxZ41cUacGsJ6l1tKbw3r1Ixd4i0FLCcUj1xFn8/py1a7gEPyCwm+h9/zlzieAxA== X-Received: by 2002:a05:600c:c4:: with SMTP id u4mr6328000wmm.80.1554978398248; Thu, 11 Apr 2019 03:26:38 -0700 (PDT) Received: from mcroce-redhat.mxp.redhat.com (nat-pool-mxp-t.redhat.com. [149.6.153.186]) by smtp.gmail.com with ESMTPSA id 204sm7681684wmc.1.2019.04.11.03.26.37 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 11 Apr 2019 03:26:37 -0700 (PDT) From: Matteo Croce To: netdev@vger.kernel.org Cc: Sunil Goutham , Robert Richter , linux-arm-kernel@lists.infradead.org, Ilias Apalodimas , Jesper Dangaard Brouer , David Miller Subject: [PATCH v3 2/2] net: thunderx: don't allow jumbo frames with XDP Date: Thu, 11 Apr 2019 12:26:33 +0200 Message-Id: <20190411102633.3685-3-mcroce@redhat.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190411102633.3685-1-mcroce@redhat.com> References: <20190411102633.3685-1-mcroce@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The thunderx driver forbids to load an eBPF program if the MTU is too high, but this can be circumvented by loading the eBPF, then raising the MTU. Fix this by limiting the MTU if an eBPF program is already loaded. Fixes: 05c773f52b96e ("net: thunderx: Add basic XDP support") Signed-off-by: Matteo Croce --- drivers/net/ethernet/cavium/thunder/nicvf_main.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/net/ethernet/cavium/thunder/nicvf_main.c b/drivers/net/ethernet/cavium/thunder/nicvf_main.c index debc8c861c6b..c032bef1b776 100644 --- a/drivers/net/ethernet/cavium/thunder/nicvf_main.c +++ b/drivers/net/ethernet/cavium/thunder/nicvf_main.c @@ -1589,6 +1589,15 @@ static int nicvf_change_mtu(struct net_device *netdev, int new_mtu) struct nicvf *nic = netdev_priv(netdev); int orig_mtu = netdev->mtu; + /* For now just support only the usual MTU sized frames, + * plus some headroom for VLAN, QinQ. + */ + if (nic->xdp_prog && new_mtu > MAX_XDP_MTU) { + netdev_warn(netdev, "Jumbo frames not yet supported with XDP, current MTU %d.\n", + netdev->mtu); + return -EINVAL; + } + netdev->mtu = new_mtu; if (!netif_running(netdev)) -- 2.21.0