From patchwork Thu Feb 28 21:54:36 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 1049791 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=networkplumber.org Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=networkplumber-org.20150623.gappssmtp.com header.i=@networkplumber-org.20150623.gappssmtp.com header.b="twGE1dIs"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 449RDB517cz9s21 for ; Fri, 1 Mar 2019 08:54:50 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732592AbfB1Vys (ORCPT ); Thu, 28 Feb 2019 16:54:48 -0500 Received: from mail-pl1-f196.google.com ([209.85.214.196]:37220 "EHLO mail-pl1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726087AbfB1Vyr (ORCPT ); Thu, 28 Feb 2019 16:54:47 -0500 Received: by mail-pl1-f196.google.com with SMTP id q3so10390664pll.4 for ; Thu, 28 Feb 2019 13:54:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=wSnifz6wY42izTva3h55UdTQzyizkDrbVS8zPJ8WaH4=; b=twGE1dIs5sS9vOFy6eqAKsHUg1Z7oDggmQWJvaizZXUNKDVqVuvojuquNIJ/yj/soq QtToGu8OHpwuG6avD5dKupqi8MPtVrwxnMP2HPxtkEGvzZDQnb2M3Za7BoaySv4Qv4nc 4IlqoZ7T7P28GTCpLSk8SwzPl9JrUECxk1RR/Xuv1rgpS0xnDlLJHBgk0RICRK9mKETg 3wTa1398ArashkoX2/EMJWiHag+l0FnfoziaoEEjy+jcMcBbSMK4aU2CAfJftdpjZZTP w4l408f26dhuqzz0eYiBMYt/uG6hK9ERAuS9DJ2fH4I/FpmM4gmhoIRqmha5G2c2Ghky MS0w== 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; bh=wSnifz6wY42izTva3h55UdTQzyizkDrbVS8zPJ8WaH4=; b=pY5YlMgNrZvm9dsvVXK8sL3WfParNJzkPG/yLip2BcNAAWBH/xWEQBlXpYyLLv20ko AOI9u87vr30KSZJjNUfN6apwM5Ds0cRL4jXHIrIwlFRrYOa2OVrnVmp5vRiwJn3LaGQE pRjhYaaZlUfQHmBadJTU1ppe7kqgRnLjle8qRRGRnE78IXmHUF4eEvUq9yDGOWCKSIjX 68DERj+VSegyVMDGRomk+UqgK3s9QbLIO4wxumbpXjL8DSGIFJz1JjVqWSMTyZJMkgWT at4jNFb8EZXAFyxo9a125au0qXAg9T1FfG9KwQYPLf4AVzZe8KCY0szGd4R5uQjhrj+e /8kQ== X-Gm-Message-State: APjAAAXOMe+r/ImXu/v83R5QaupH7LiO/BxFEMkWlIJGaNQ7ipt3QMIW Els0bOgcgrDHNEMqwqZqn3DpepS4KLc= X-Google-Smtp-Source: APXvYqy1pcdZRrScf3/lUiR4ppHiFt6J8jia2eeWWoRcXKV906zHl6XNE75V0bynWiLE/KpnxDvoog== X-Received: by 2002:a17:902:4225:: with SMTP id g34mr1722634pld.152.1551390886444; Thu, 28 Feb 2019 13:54:46 -0800 (PST) Received: from shemminger-XPS-13-9360.lan (204-195-22-127.wavecable.com. [204.195.22.127]) by smtp.gmail.com with ESMTPSA id b138sm40078643pfb.48.2019.02.28.13.54.44 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 28 Feb 2019 13:54:44 -0800 (PST) From: Stephen Hemminger To: netdev@vger.kernel.org Cc: Stephen Hemminger Subject: [RFC v2 1/6] bnxt: use extack for xdp error messages Date: Thu, 28 Feb 2019 13:54:36 -0800 Message-Id: <20190228215441.28275-2-stephen@networkplumber.org> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190228215441.28275-1-stephen@networkplumber.org> References: <20190228215441.28275-1-stephen@networkplumber.org> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org XDP has a netlink error message buffer available that should be used for errors instead of console logging. Signed-off-by: Stephen Hemminger --- drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.c | 26 +++++++++---------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.c index 0184ef6f05a7..a9f6a3a66b4c 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.c @@ -150,19 +150,21 @@ bool bnxt_rx_xdp(struct bnxt *bp, struct bnxt_rx_ring_info *rxr, u16 cons, } /* Under rtnl_lock */ -static int bnxt_xdp_set(struct bnxt *bp, struct bpf_prog *prog) +static int bnxt_xdp_set(struct bnxt *bp, struct bpf_prog *prog, + struct netlink_ext_ack *extack) { struct net_device *dev = bp->dev; int tx_xdp = 0, rc, tc; struct bpf_prog *old; if (prog && bp->dev->mtu > BNXT_MAX_PAGE_MODE_MTU) { - netdev_warn(dev, "MTU %d larger than largest XDP supported MTU %d.\n", - bp->dev->mtu, BNXT_MAX_PAGE_MODE_MTU); + NL_SET_ERR_MSG_MOD(extack, + "MTU larger than largest XDP supported MTU"); return -EOPNOTSUPP; } if (!(bp->flags & BNXT_FLAG_SHARED_RINGS)) { - netdev_warn(dev, "ethtool rx/tx channels must be combined to support XDP.\n"); + NL_SET_ERR_MSG_MOD(extack, + "ethtool rx/tx channels must be combined to support XDP"); return -EOPNOTSUPP; } if (prog) @@ -174,7 +176,8 @@ static int bnxt_xdp_set(struct bnxt *bp, struct bpf_prog *prog) rc = bnxt_check_rings(bp, bp->tx_nr_rings_per_tc, bp->rx_nr_rings, true, tc, tx_xdp); if (rc) { - netdev_warn(dev, "Unable to reserve enough TX rings to support XDP.\n"); + NL_SET_ERR_MSG_MOD(extack, + "Unable to reserve enough TX rings to support XDP.\n"); return rc; } if (netif_running(dev)) @@ -211,19 +214,16 @@ static int bnxt_xdp_set(struct bnxt *bp, struct bpf_prog *prog) int bnxt_xdp(struct net_device *dev, struct netdev_bpf *xdp) { struct bnxt *bp = netdev_priv(dev); - int rc; switch (xdp->command) { case XDP_SETUP_PROG: - rc = bnxt_xdp_set(bp, xdp->prog); - break; + return bnxt_xdp_set(bp, xdp->prog, xdp->extack); + case XDP_QUERY_PROG: xdp->prog_id = bp->xdp_prog ? bp->xdp_prog->aux->id : 0; - rc = 0; - break; + return 0; default: - rc = -EINVAL; - break; + NL_SET_ERR_MSG_MOD(xdp->extack, "Unsupported XDP command"); + return -EINVAL; } - return rc; }