From patchwork Fri Jul 16 08:35:38 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Heinrich Kuhn X-Patchwork-Id: 95958 Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 52B35A0C50; Fri, 16 Jul 2021 10:36:12 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C495340151; Fri, 16 Jul 2021 10:36:11 +0200 (CEST) Received: from mail-ej1-f43.google.com (mail-ej1-f43.google.com [209.85.218.43]) by mails.dpdk.org (Postfix) with ESMTP id 458F94014D for ; Fri, 16 Jul 2021 10:36:10 +0200 (CEST) Received: by mail-ej1-f43.google.com with SMTP id hc15so13988173ejc.4 for ; Fri, 16 Jul 2021 01:36:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=netronome-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=P7kxB2QBAU2VYXDETUrSbWD9pd9jF1f5Wdx4YZrokUA=; b=mR2EGJCPm6SWXjLuOSPqP6sw5icWPrhDmpW/ZDqPm7HvS1QuMFFtlooChvJRVdEcHG aPE4f0B0wAlsQtMzCJfUV8LG9z8CAWtw3guHD9QEGE1Hg/mcyJ8Z0Y03XWs9s6Nmw4i+ WUARszcs+B1gZ55A/ec6ix0XsbkZ0SPMYX6EGDv2xpRohsO9nGkNwhf4u3Ijth5AMYJD kdDfXXyUcCLy3+yuApXeKjWMHczBKZgSRJ8qARGRmvdP4/oQnIDkD+3bh4ihvem8Suh0 UZh32LF6VVNY5AAdXGhI2ecBXrEiuWMdvGKX9w1xcHbYd4wxmzuIRFsPVKUNul6HcaVd K7OQ== 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=P7kxB2QBAU2VYXDETUrSbWD9pd9jF1f5Wdx4YZrokUA=; b=FkpkMp4WmV4o+tOTBOhjVt9nyc1BG5OcmARc/AXWm0fKsNQhmu+jU1Q5kqT8kzgNkL tRwgpGu/s+/bX7A5opvvCq4btX6lkpYyxYobDkqWVnH/DTa5gJ/dfJuLL59lyxSiz8ck 2BYATfsqhAvSh0WO2xeOpDkqLwaFiiwOgzglobpLy4m4ufinOtB19y8mahOGXimwa9Lk 1ofnos4eXTBHDskkih/XgvFBEBTewR2lUI45ZCZ4LIB5fGXSGYMvJ5dGDu3vBCdYSGmI ijqTw/HLx0TjdpG096oCXk7rMwTmdzAyOzTrzdBOBe2eBD9ATV3KBha+sbHG7mDaBhm1 J4bQ== X-Gm-Message-State: AOAM532F1zypuT+YZQl/NT56wg05gZkP8m8i55TP2D/BnB+Nx9QTU6X4 x71ztEMpMwOy6070+wzfAmJHUyTcwJK4G/Rk2B1qr/dXQYiFIQBijgkHkFmflGKbOI5thA7l14r fl+O+rX63gwcF26OzIArddp7EZlrZK/A8Ik6V8A6rsFtrv+VZZqkLt35Upw1syYvs X-Google-Smtp-Source: ABdhPJzLCnYwcLyTH2xmRUKxm6deH3/ksw3OYqyXLBG49tCGqxDwOvQ1rjQX5Wr0GiuGwkZLIw4mAQ== X-Received: by 2002:a17:906:8313:: with SMTP id j19mr10815184ejx.548.1626424569879; Fri, 16 Jul 2021 01:36:09 -0700 (PDT) Received: from localhost.localdomain ([155.93.216.150]) by smtp.gmail.com with ESMTPSA id e6sm3371650edk.63.2021.07.16.01.36.08 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 16 Jul 2021 01:36:09 -0700 (PDT) From: Heinrich Kuhn To: dev@dpdk.org Cc: Heinrich Kuhn Date: Fri, 16 Jul 2021 10:35:38 +0200 Message-Id: <20210716083545.34444-1-heinrich.kuhn@netronome.com> X-Mailer: git-send-email 2.30.1 (Apple Git-130) In-Reply-To: <20210716082314.33865-1-heinrich.kuhn@netronome.com> References: <20210716082314.33865-1-heinrich.kuhn@netronome.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v2 0/7] Refactor the NFP PMD X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This patch set restructures the NFP PMD, aligning it more with the common layout adopted by most other PMD's. Although the changes look fairly large, functionally nothing is added or removed from the driver and the existing code is mostly just reorganized into the familiar structure seen in other PMD's. Apart form adopting the common PMD layout this change should also aid in future feature development to the NFP PMD. The previous layout where most of the logic resided in a single file (nfp_net.c) would have become tedious to support going forward. v2: * Added missing sign-off's Heinrich Kuhn (7): net/nfp: split rxtx headers into separate file net/nfp: move rxtx functions to their own file net/nfp: move CPP bridge to a separate file net/nfp: prototype common functions in header file net/nfp: move VF functions into new file net/nfp: move PF functions into new file net/nfp: batch file rename for consistency drivers/net/nfp/meson.build | 6 +- drivers/net/nfp/nfp_common.c | 1322 ++++++ drivers/net/nfp/nfp_common.h | 413 ++ drivers/net/nfp/nfp_cpp_bridge.c | 392 ++ drivers/net/nfp/nfp_cpp_bridge.h | 36 + .../net/nfp/{nfp_net_ctrl.h => nfp_ctrl.h} | 6 +- drivers/net/nfp/nfp_ethdev.c | 1099 +++++ drivers/net/nfp/nfp_ethdev_vf.c | 504 +++ .../net/nfp/{nfp_net_logs.h => nfp_logs.h} | 6 +- drivers/net/nfp/nfp_net.c | 3921 ----------------- drivers/net/nfp/nfp_rxtx.c | 1002 +++++ drivers/net/nfp/{nfp_net_pmd.h => nfp_rxtx.h} | 279 +- 12 files changed, 4815 insertions(+), 4171 deletions(-) create mode 100644 drivers/net/nfp/nfp_common.c create mode 100644 drivers/net/nfp/nfp_common.h create mode 100644 drivers/net/nfp/nfp_cpp_bridge.c create mode 100644 drivers/net/nfp/nfp_cpp_bridge.h rename drivers/net/nfp/{nfp_net_ctrl.h => nfp_ctrl.h} (99%) create mode 100644 drivers/net/nfp/nfp_ethdev.c create mode 100644 drivers/net/nfp/nfp_ethdev_vf.c rename drivers/net/nfp/{nfp_net_logs.h => nfp_logs.h} (94%) delete mode 100644 drivers/net/nfp/nfp_net.c create mode 100644 drivers/net/nfp/nfp_rxtx.c rename drivers/net/nfp/{nfp_net_pmd.h => nfp_rxtx.h} (54%)