From patchwork Tue Jan 31 05:25:49 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 13122259 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 F29F7C38142 for ; Tue, 31 Jan 2023 05:26:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230166AbjAaF0M (ORCPT ); Tue, 31 Jan 2023 00:26:12 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36034 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230185AbjAaF0J (ORCPT ); Tue, 31 Jan 2023 00:26:09 -0500 Received: from mail-pl1-x62e.google.com (mail-pl1-x62e.google.com [IPv6:2607:f8b0:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 57BD9E38C for ; Mon, 30 Jan 2023 21:26:07 -0800 (PST) Received: by mail-pl1-x62e.google.com with SMTP id r8so7103639pls.2 for ; Mon, 30 Jan 2023 21:26:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:message-id:date:subject:cc:to:from:from:to:cc:subject :date:message-id:reply-to; bh=C47aBaTtHnSCIWGQBAQJB12y74ceeaAw2OO3uwBgLYA=; b=UVlbBkKaSDoGp0jxcc4RscEHOfW83QUotS2E+In/48yO+0cTbTFSiwOXhUl2ZQQAvg gxb1QQCrCoprtKVTmYMNukNAbF4eE15rKZlbezGzOhiMef3M35xalQQLjuIBJWzGc3a2 8GoPS5N0tDwH32g5NZHTKPZhxxZohHUQ12N24= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:subject:cc:to:from:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=C47aBaTtHnSCIWGQBAQJB12y74ceeaAw2OO3uwBgLYA=; b=sUqmqVCESHZ8/umYEvZLrdcb5nr86JYnFldRWaHgpo6e2ssH5Jt7Ke4FiPRXNMW8An brmqt0VLNWN+0jvm3qe4EH4e6FN2kxIx86XjdFYqqqF0fROOK7db2ymMdb7kYDOp5iSy a+N4c3//SV3uoa5rMI7KAzJlQT6wZ6+4AdajKPps0u/dhLGUbjoO+q0vd2T0xdaCLUjU 59HWj1ndDhocMDOKjrdCmHeUAeX3NO+k380xgK4q9Xd3IqcxGKoW+HyXgFGv3kLQKUPh Jyk4Rzd6x+VvF9gg3g3H+xZ47elS/qSzn9KeK60G9NCL73P6WPKOc04DRSTISf6QGpK3 BgUw== X-Gm-Message-State: AO0yUKU0mpOGjW84S4t+hZYuQltBznqJecWwVNzfefWXrXleX7EXBmCs 3GvFob6ASnaWbDA9zkX0vuJ/pA== X-Google-Smtp-Source: AK7set86xY9Im8SIP7cDDmlXpN9cMSvbkKcNJeI+c9Mtry3zTrRCe3UI5bh2TCldcl3a70c8DfAYTw== X-Received: by 2002:a17:902:f1ca:b0:198:94cb:82a5 with SMTP id e10-20020a170902f1ca00b0019894cb82a5mr1866239plc.4.1675142766736; Mon, 30 Jan 2023 21:26:06 -0800 (PST) Received: from localhost.localdomain ([2605:a601:a780:1400:a879:b64b:d9bd:3c1]) by smtp.gmail.com with ESMTPSA id jk15-20020a170903330f00b001960cccc318sm2310106plb.121.2023.01.30.21.26.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jan 2023 21:26:05 -0800 (PST) From: Ajit Khaparde To: ajit.khaparde@broadcom.com Cc: andrew.gospodarek@broadcom.com, davem@davemloft.net, edumazet@google.com, jgg@ziepe.ca, kuba@kernel.org, leon@kernel.org, linux-kernel@vger.kernel.org, linux-rdma@vger.kernel.org, michael.chan@broadcom.com, netdev@vger.kernel.org, pabeni@redhat.com, selvin.xavier@broadcom.com, gregkh@linuxfoundation.org Subject: [PATCH net-next v9 0/8] Add Auxiliary driver support Date: Mon, 30 Jan 2023 21:25:49 -0800 Message-Id: <20230131052557.99119-1-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.37.1 (Apple Git-137.1) MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org Add auxiliary device driver for Broadcom devices. The bnxt_en driver will register and initialize an aux device if RDMA is enabled in the underlying device. The bnxt_re driver will then probe and initialize the RoCE interfaces with the infiniband stack. We got rid of the bnxt_en_ops which the bnxt_re driver used to communicate with bnxt_en. Similarly We have tried to clean up most of the bnxt_ulp_ops. In most of the cases we used the functions and entry points provided by the auxiliary bus driver framework. And now these are the minimal functions needed to support the functionality. We will try to work on getting rid of the remaining if we find any other viable option in future. v1->v2: - Incorporated review comments including usage of ulp_id & complex function indirections. - Used function calls provided by the auxiliary bus interface instead of proprietary calls. - Refactor code to remove ROCE driver's access to bnxt structure. v2->v3: - Addressed review comments including cleanup of some unnecessary wrappers - Fixed warnings seen during cross compilation v3->v4: - Cleaned up bnxt_ulp.c and bnxt_ulp.h further - Removed some more dead code - Sending the patchset as a standalone series v4->v5: - Removed the SRIOV config callback which bnxt_en driver was calling into bnxt_re driver. - Removed excessive checks for rdev and other pointers. v5->v6: - Removed excessive checks for dev and other pointers - Remove runtime interrupt vector allocation. bnxt_en preallocates interrupt vectors for bnxt_re to use. v6->v7: - Removed incorrect usage of inline - Updated Kconfig to select AUXILIARY BUS support - Addressed various comments including removal of unnecessary forward declaration, using static functions where possible, unnecessary jump, cleanup logic, etc.. - Added Leon's Reviewed-by, to the commit log in the patches, from previous version. v7->v8: - Addressed various comments to remove unnecessary check for id, removed setting pointer to NULL after free, renamed private pointers to avoid confusing them with the auxiliary device names and refactored some code accordingly. - Auxiliary device will be released through auxiliary_device_uninit(); v8->v9: - Fixed the cleanup code to use the auxiliary driver release function to free allocated memory. This allows the cleanup to be performed once the device reference count drops to 0. - Refactored the code to address the above. Commit message uses Leon's Reviewed-by from earlier version. Please apply. Thanks. The following are changes since commit 90e8ca0abb05ada6c1e2710eaa21688dafca26f2 Merge branch 'devlink-next' and are available in the git repository at: https://github.com/ajitkhaparde1/net-next/tree/aux-bus-v9 Ajit Khaparde (7): bnxt_en: Add auxiliary driver support RDMA/bnxt_re: Use auxiliary driver interface bnxt_en: Remove usage of ulp_id bnxt_en: Use direct API instead of indirection bnxt_en: Use auxiliary bus calls over proprietary calls RDMA/bnxt_re: Remove the sriov config callback bnxt_en: Remove runtime interrupt vector allocation Hongguang Gao (1): bnxt_en: Remove struct bnxt access from RoCE driver drivers/infiniband/hw/bnxt_re/bnxt_re.h | 10 +- drivers/infiniband/hw/bnxt_re/main.c | 635 +++++++----------- drivers/net/ethernet/broadcom/Kconfig | 1 + drivers/net/ethernet/broadcom/bnxt/bnxt.c | 9 +- drivers/net/ethernet/broadcom/bnxt/bnxt.h | 8 + .../net/ethernet/broadcom/bnxt/bnxt_sriov.c | 7 +- drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c | 478 ++++++------- drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.h | 51 +- 8 files changed, 464 insertions(+), 735 deletions(-)