From patchwork Tue Mar 7 11:36:13 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gautam Dawar X-Patchwork-Id: 13163234 X-Patchwork-Delegate: kuba@kernel.org 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 3D941C678D5 for ; Tue, 7 Mar 2023 11:42:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231388AbjCGLmK (ORCPT ); Tue, 7 Mar 2023 06:42:10 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39702 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231256AbjCGLkb (ORCPT ); Tue, 7 Mar 2023 06:40:31 -0500 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2076.outbound.protection.outlook.com [40.107.220.76]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 443057BA03; Tue, 7 Mar 2023 03:39:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nAAfpxFbPzQ/zBPMA88Bc70bOSFMOoKP+gHfSl4MDWOA/Vin+SUlw+4aNRSAx19octC6rX4q7u5gIroIAcZyXlKChbH1MTGJR61lr+JTt55MzqfhrN4Q8kl9293FDiGbeIbuqnb+zl1x0qn2AnwJ7tN3nSAFJ8hHAmN2nDfSajDO/IrEdI/DLXHE1MrReKt/bEn5smnckwuKZf8zU4WszROTD/IPBoYbog+q1ncfsTnSspCiFcHb04RsbfqfR1KLoqJ2COjiiNsati+xnRmhO+MKjUYq7bWByOgf/HK7jA7vPKMS0i0HiPjvoFPs1IwM9luF097arN1cj8txCCw9MQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=01BOhMw9BsMj5+QAtrPIInhWA9hS/wgAkqiKIqsfZYE=; b=Mvtxv8BNXT+p8GnnUF8MNywyQRLhS4psyD/tCUsB/M9POgkjvg14Q5tg2OClj/wADfKTVoLtCwUwLHJMuJj5E+EchFoCHvoFug4srtWCYjdGlh/OuhxynWadOZJteqlBYHdyNSZNGejnSBcIsr6vDhG7LWCQyRIqo56Xvjp1yO8gO6htwp/Mx7JtrRKh/ExqLULcmxLxRqZrLJjMsjsFhhoILAgfQHMTPm+pcTUPZcmssKyXmEx0ItSYHBe9J4aTcWHCtx2LtCWv8/zJ23HMW6I4eYZinpoAv4aXXdLoyp6s2Vc5Z3m/FFdTwbtd8rVY4TpwLImwjm9XoXzFUsoDNA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=redhat.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=01BOhMw9BsMj5+QAtrPIInhWA9hS/wgAkqiKIqsfZYE=; b=o7dOdq5C+j0DqQ+r2BR7bj8RGwHZe4TpUA35YEmpEus2xAiNyClIyNFwTT4uxmZGFnFgkVUXp6jnmEedTmvTDiN05lSLDXRvdOUJ7NLXax2BSFH+Ttb9OGozsNCtiYZSkPnt3gZt5djABaI6CnfX8VUCzjMShAWAw72r+LGGIXg= Received: from DM6PR12CA0014.namprd12.prod.outlook.com (2603:10b6:5:1c0::27) by IA1PR12MB7709.namprd12.prod.outlook.com (2603:10b6:208:423::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6156.29; Tue, 7 Mar 2023 11:38:13 +0000 Received: from DS1PEPF0000E640.namprd02.prod.outlook.com (2603:10b6:5:1c0:cafe::cd) by DM6PR12CA0014.outlook.office365.com (2603:10b6:5:1c0::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6156.29 via Frontend Transport; Tue, 7 Mar 2023 11:38:13 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by DS1PEPF0000E640.mail.protection.outlook.com (10.167.17.202) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6178.13 via Frontend Transport; Tue, 7 Mar 2023 11:38:12 +0000 Received: from SATLEXMB07.amd.com (10.181.41.45) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Tue, 7 Mar 2023 05:38:11 -0600 Received: from SATLEXMB03.amd.com (10.181.40.144) by SATLEXMB07.amd.com (10.181.41.45) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Tue, 7 Mar 2023 03:38:10 -0800 Received: from xndengvm004102.xilinx.com (10.180.168.240) by SATLEXMB03.amd.com (10.181.40.144) with Microsoft SMTP Server id 15.1.2375.34 via Frontend Transport; Tue, 7 Mar 2023 05:38:06 -0600 From: Gautam Dawar To: , , Edward Cree , Martin Habets , "David S. Miller" , Eric Dumazet , "Jakub Kicinski" , Paolo Abeni , Richard Cochran , , CC: , , , , Gautam Dawar Subject: [PATCH net-next v2 11/14] sfc: use PF's IOMMU domain for running VF's MCDI commands Date: Tue, 7 Mar 2023 17:06:13 +0530 Message-ID: <20230307113621.64153-12-gautam.dawar@amd.com> X-Mailer: git-send-email 2.30.1 In-Reply-To: <20230307113621.64153-1-gautam.dawar@amd.com> References: <20230307113621.64153-1-gautam.dawar@amd.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS1PEPF0000E640:EE_|IA1PR12MB7709:EE_ X-MS-Office365-Filtering-Correlation-Id: 80fadda8-9347-4c1c-9f78-08db1f006f49 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HydmbrCOhr7xCuyWEK9G88uRpVIAOmXAzvw9uIJuDvIgqMeK/ZFrltog33d0QPIQvtraad/+9rA4cneuz9Cmx/qYwdXJtcNoXqubAyzuA5/Vpz9fw0V7pDCC+Kq50cLEMV0V8iABcB3IbRirrOp29Jrf1WLzfigltH8qQNl7CM6XUJp4cOhEIlbgU7Za8LL15M2tkF5Z6dIcgb92qiW5lDdg8+1Ywm8WyixxG8xPjyQ5+btJEQAoyGj6BfhIr66umOsHPNEPDYKZame6YSC/ydpgL/w8epD4qxqqOpv2PE4otheSV6yk4+CPFIkf++btQTTx6yv9QmSee7zugkZYmSN/efeKjYRGZl82URtaFgVmHu/g2d7p/z68kB8p8UHLYpGivxb0y30oyjzL8uUoLcjN8FPkf8BiL6kDw1OH7G7Jt87fxmzOqJBNSFDRxp3dEdvgTwAULhGgamuebPmC9Hpmf/2HKIvGTLkIjPvrBVn3/vFI5+UtN2Mdbz9gcQT6XAptmZaXz5o+Hqq/856cqEcdxBwd3n9afSAEMO18SuEdEsMKzOXL+PAfsDJTm/evBzL8Uc0N6FKACzEiRnEuoxYLOlvLHjB/sMVlKotX55C5g/OTgQONw7bH+S1udq5w5suB74WE8zqBTFyELSDC4hMLhHlsRuAAkYcppUQvlpVUqAEDJU2xj3zS9Kja2M3x/8T85XsaIDSYcJFJ4dvHDc+h+WN3Wr+ZPoKhR8gKFm3mfgxpsVLLLLAKQY4+mANr X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230025)(4636009)(396003)(376002)(346002)(136003)(39860400002)(451199018)(40470700004)(36840700001)(46966006)(70206006)(8676002)(4326008)(70586007)(41300700001)(81166007)(82740400003)(316002)(8936002)(54906003)(110136005)(5660300002)(7416002)(44832011)(478600001)(30864003)(82310400005)(2906002)(40460700003)(356005)(921005)(36756003)(36860700001)(40480700001)(1076003)(6666004)(83380400001)(26005)(186003)(2616005)(86362001)(336012)(47076005)(426003)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Mar 2023 11:38:12.9832 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 80fadda8-9347-4c1c-9f78-08db1f006f49 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: DS1PEPF0000E640.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB7709 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org This changeset uses MC_CMD_CLIENT_CMD to execute VF's MCDI commands when running in vDPA mode (STATE_VDPA). Also, use the PF's IOMMU domain for executing the encapsulated VF's MCDI commands to isolate DMA of guest buffers in the VF's IOMMU domain. This patch also updates the PCIe FN's client id in the efx_nic structure which is required while running MC_CMD_CLIENT_CMD. Signed-off-by: Gautam Dawar --- drivers/net/ethernet/sfc/ef100.c | 1 + drivers/net/ethernet/sfc/ef100_nic.c | 35 +++++++++ drivers/net/ethernet/sfc/mcdi.c | 108 ++++++++++++++++++++++---- drivers/net/ethernet/sfc/mcdi.h | 2 +- drivers/net/ethernet/sfc/net_driver.h | 2 + drivers/net/ethernet/sfc/ptp.c | 4 +- 6 files changed, 132 insertions(+), 20 deletions(-) diff --git a/drivers/net/ethernet/sfc/ef100.c b/drivers/net/ethernet/sfc/ef100.c index c1c69783db7b..8453c9ba0f41 100644 --- a/drivers/net/ethernet/sfc/ef100.c +++ b/drivers/net/ethernet/sfc/ef100.c @@ -465,6 +465,7 @@ static int ef100_pci_probe(struct pci_dev *pci_dev, efx->type = (const struct efx_nic_type *)entry->driver_data; efx->pci_dev = pci_dev; + efx->client_id = MC_CMD_CLIENT_ID_SELF; pci_set_drvdata(pci_dev, efx); rc = efx_init_struct(efx, pci_dev); if (rc) diff --git a/drivers/net/ethernet/sfc/ef100_nic.c b/drivers/net/ethernet/sfc/ef100_nic.c index bda4fcbe1126..cd9f724a9e64 100644 --- a/drivers/net/ethernet/sfc/ef100_nic.c +++ b/drivers/net/ethernet/sfc/ef100_nic.c @@ -206,9 +206,11 @@ static int efx_ef100_init_datapath_caps(struct efx_nic *efx) "firmware reports num_mac_stats = %u\n", efx->num_mac_stats); +#ifdef CONFIG_SFC_VDPA nic_data->vdpa_supported = efx_ef100_has_cap(nic_data->datapath_caps3, CLIENT_CMD_VF_PROXY) && efx->type->is_vf; +#endif return 0; } @@ -1086,6 +1088,35 @@ static int ef100_check_design_params(struct efx_nic *efx) return rc; } +static int efx_ef100_update_client_id(struct efx_nic *efx) +{ + struct ef100_nic_data *nic_data = efx->nic_data; + unsigned int pf_index = PCIE_FUNCTION_PF_NULL; + unsigned int vf_index = PCIE_FUNCTION_VF_NULL; + efx_qword_t pciefn; + int rc; + + if (efx->pci_dev->is_virtfn) + vf_index = nic_data->vf_index; + else + pf_index = nic_data->pf_index; + + /* Construct PCIE_FUNCTION structure */ + EFX_POPULATE_QWORD_3(pciefn, + PCIE_FUNCTION_PF, pf_index, + PCIE_FUNCTION_VF, vf_index, + PCIE_FUNCTION_INTF, PCIE_INTERFACE_CALLER); + /* look up self client ID */ + rc = efx_ef100_lookup_client_id(efx, pciefn, &efx->client_id); + if (rc) { + pci_warn(efx->pci_dev, + "%s: Failed to get client ID, rc %d\n", + __func__, rc); + } + + return rc; +} + /* NIC probe and remove */ static int ef100_probe_main(struct efx_nic *efx) @@ -1173,6 +1204,10 @@ static int ef100_probe_main(struct efx_nic *efx) goto fail; efx->port_num = rc; + rc = efx_ef100_update_client_id(efx); + if (rc) + goto fail; + efx_mcdi_print_fwver(efx, fw_version, sizeof(fw_version)); pci_dbg(efx->pci_dev, "Firmware version %s\n", fw_version); diff --git a/drivers/net/ethernet/sfc/mcdi.c b/drivers/net/ethernet/sfc/mcdi.c index a7f2c31071e8..3bf1ebe05775 100644 --- a/drivers/net/ethernet/sfc/mcdi.c +++ b/drivers/net/ethernet/sfc/mcdi.c @@ -145,14 +145,15 @@ void efx_mcdi_fini(struct efx_nic *efx) kfree(efx->mcdi); } -static void efx_mcdi_send_request(struct efx_nic *efx, unsigned cmd, - const efx_dword_t *inbuf, size_t inlen) +static void efx_mcdi_send_request(struct efx_nic *efx, u32 client_id, + unsigned int cmd, const efx_dword_t *inbuf, + size_t inlen) { struct efx_mcdi_iface *mcdi = efx_mcdi(efx); #ifdef CONFIG_SFC_MCDI_LOGGING char *buf = mcdi->logging_buffer; /* page-sized */ #endif - efx_dword_t hdr[2]; + efx_dword_t hdr[5]; size_t hdr_len; u32 xflags, seqno; @@ -179,7 +180,7 @@ static void efx_mcdi_send_request(struct efx_nic *efx, unsigned cmd, MCDI_HEADER_XFLAGS, xflags, MCDI_HEADER_NOT_EPOCH, !mcdi->new_epoch); hdr_len = 4; - } else { + } else if (client_id == efx->client_id) { /* MCDI v2 */ BUG_ON(inlen > MCDI_CTL_SDU_LEN_MAX_V2); EFX_POPULATE_DWORD_7(hdr[0], @@ -194,6 +195,35 @@ static void efx_mcdi_send_request(struct efx_nic *efx, unsigned cmd, MC_CMD_V2_EXTN_IN_EXTENDED_CMD, cmd, MC_CMD_V2_EXTN_IN_ACTUAL_LEN, inlen); hdr_len = 8; + } else { + /* MCDI v2 */ + WARN_ON(inlen > MCDI_CTL_SDU_LEN_MAX_V2); + /* MCDI v2 with credentials of a different client */ + BUILD_BUG_ON(MC_CMD_CLIENT_CMD_IN_LEN != 4); + /* Outer CLIENT_CMD wrapper command with client ID */ + EFX_POPULATE_DWORD_7(hdr[0], + MCDI_HEADER_RESPONSE, 0, + MCDI_HEADER_RESYNC, 1, + MCDI_HEADER_CODE, MC_CMD_V2_EXTN, + MCDI_HEADER_DATALEN, 0, + MCDI_HEADER_SEQ, seqno, + MCDI_HEADER_XFLAGS, xflags, + MCDI_HEADER_NOT_EPOCH, !mcdi->new_epoch); + EFX_POPULATE_DWORD_2(hdr[1], + MC_CMD_V2_EXTN_IN_EXTENDED_CMD, + MC_CMD_CLIENT_CMD, + MC_CMD_V2_EXTN_IN_ACTUAL_LEN, inlen + 12); + MCDI_SET_DWORD(&hdr[2], + CLIENT_CMD_IN_CLIENT_ID, client_id); + + /* MCDIv2 header for inner command */ + EFX_POPULATE_DWORD_2(hdr[3], + MCDI_HEADER_CODE, MC_CMD_V2_EXTN, + MCDI_HEADER_DATALEN, 0); + EFX_POPULATE_DWORD_2(hdr[4], + MC_CMD_V2_EXTN_IN_EXTENDED_CMD, cmd, + MC_CMD_V2_EXTN_IN_ACTUAL_LEN, inlen); + hdr_len = 20; } #ifdef CONFIG_SFC_MCDI_LOGGING @@ -474,7 +504,8 @@ static void efx_mcdi_release(struct efx_mcdi_iface *mcdi) &mcdi->async_list, struct efx_mcdi_async_param, list); if (async) { mcdi->state = MCDI_STATE_RUNNING_ASYNC; - efx_mcdi_send_request(efx, async->cmd, + efx_mcdi_send_request(efx, efx->client_id, + async->cmd, (const efx_dword_t *)(async + 1), async->inlen); mod_timer(&mcdi->async_timer, @@ -797,7 +828,7 @@ static int efx_mcdi_proxy_wait(struct efx_nic *efx, u32 handle, bool quiet) return mcdi->proxy_rx_status; } -static int _efx_mcdi_rpc(struct efx_nic *efx, unsigned int cmd, +static int _efx_mcdi_rpc(struct efx_nic *efx, u32 client_id, unsigned int cmd, const efx_dword_t *inbuf, size_t inlen, efx_dword_t *outbuf, size_t outlen, size_t *outlen_actual, bool quiet, int *raw_rc) @@ -811,7 +842,7 @@ static int _efx_mcdi_rpc(struct efx_nic *efx, unsigned int cmd, return -EINVAL; } - rc = efx_mcdi_rpc_start(efx, cmd, inbuf, inlen); + rc = efx_mcdi_rpc_start(efx, client_id, cmd, inbuf, inlen); if (rc) return rc; @@ -836,7 +867,8 @@ static int _efx_mcdi_rpc(struct efx_nic *efx, unsigned int cmd, /* We now retry the original request. */ mcdi->state = MCDI_STATE_RUNNING_SYNC; - efx_mcdi_send_request(efx, cmd, inbuf, inlen); + efx_mcdi_send_request(efx, efx->client_id, cmd, + inbuf, inlen); rc = _efx_mcdi_rpc_finish(efx, cmd, inlen, outbuf, outlen, outlen_actual, @@ -855,16 +887,44 @@ static int _efx_mcdi_rpc(struct efx_nic *efx, unsigned int cmd, return rc; } +#ifdef CONFIG_SFC_VDPA +static bool is_mode_vdpa(struct efx_nic *efx) +{ + if (efx->pci_dev->is_virtfn && + efx->pci_dev->physfn && + efx->state == STATE_VDPA && + efx->vdpa_nic) + return true; + + return false; +} +#endif + static int _efx_mcdi_rpc_evb_retry(struct efx_nic *efx, unsigned cmd, const efx_dword_t *inbuf, size_t inlen, efx_dword_t *outbuf, size_t outlen, size_t *outlen_actual, bool quiet) { +#ifdef CONFIG_SFC_VDPA + struct efx_nic *efx_pf; +#endif int raw_rc = 0; int rc; - rc = _efx_mcdi_rpc(efx, cmd, inbuf, inlen, - outbuf, outlen, outlen_actual, true, &raw_rc); +#ifdef CONFIG_SFC_VDPA + if (is_mode_vdpa(efx)) { + efx_pf = pci_get_drvdata(efx->pci_dev->physfn); + rc = _efx_mcdi_rpc(efx_pf, efx->client_id, cmd, inbuf, + inlen, outbuf, outlen, outlen_actual, + true, &raw_rc); + } else { +#endif + rc = _efx_mcdi_rpc(efx, efx->client_id, cmd, inbuf, + inlen, outbuf, outlen, outlen_actual, true, + &raw_rc); +#ifdef CONFIG_SFC_VDPA + } +#endif if ((rc == -EPROTO) && (raw_rc == MC_CMD_ERR_NO_EVB_PORT) && efx->type->is_vf) { @@ -881,9 +941,22 @@ static int _efx_mcdi_rpc_evb_retry(struct efx_nic *efx, unsigned cmd, do { usleep_range(delay_us, delay_us + 10000); - rc = _efx_mcdi_rpc(efx, cmd, inbuf, inlen, - outbuf, outlen, outlen_actual, - true, &raw_rc); +#ifdef CONFIG_SFC_VDPA + if (is_mode_vdpa(efx)) { + efx_pf = pci_get_drvdata(efx->pci_dev->physfn); + rc = _efx_mcdi_rpc(efx_pf, efx->client_id, cmd, + inbuf, inlen, outbuf, outlen, + outlen_actual, true, + &raw_rc); + } else { +#endif + rc = _efx_mcdi_rpc(efx, efx->client_id, + cmd, inbuf, inlen, outbuf, + outlen, outlen_actual, true, + &raw_rc); +#ifdef CONFIG_SFC_VDPA + } +#endif if (delay_us < 100000) delay_us <<= 1; } while ((rc == -EPROTO) && @@ -939,7 +1012,7 @@ int efx_mcdi_rpc(struct efx_nic *efx, unsigned cmd, * function and is then responsible for calling efx_mcdi_display_error * as needed. */ -int efx_mcdi_rpc_quiet(struct efx_nic *efx, unsigned cmd, +int efx_mcdi_rpc_quiet(struct efx_nic *efx, unsigned int cmd, const efx_dword_t *inbuf, size_t inlen, efx_dword_t *outbuf, size_t outlen, size_t *outlen_actual) @@ -948,7 +1021,7 @@ int efx_mcdi_rpc_quiet(struct efx_nic *efx, unsigned cmd, outlen_actual, true); } -int efx_mcdi_rpc_start(struct efx_nic *efx, unsigned cmd, +int efx_mcdi_rpc_start(struct efx_nic *efx, u32 client_id, unsigned int cmd, const efx_dword_t *inbuf, size_t inlen) { struct efx_mcdi_iface *mcdi = efx_mcdi(efx); @@ -965,7 +1038,7 @@ int efx_mcdi_rpc_start(struct efx_nic *efx, unsigned cmd, return -ENETDOWN; efx_mcdi_acquire_sync(mcdi); - efx_mcdi_send_request(efx, cmd, inbuf, inlen); + efx_mcdi_send_request(efx, client_id, cmd, inbuf, inlen); return 0; } @@ -1009,7 +1082,8 @@ static int _efx_mcdi_rpc_async(struct efx_nic *efx, unsigned int cmd, */ if (mcdi->async_list.next == &async->list && efx_mcdi_acquire_async(mcdi)) { - efx_mcdi_send_request(efx, cmd, inbuf, inlen); + efx_mcdi_send_request(efx, efx->client_id, + cmd, inbuf, inlen); mod_timer(&mcdi->async_timer, jiffies + MCDI_RPC_TIMEOUT); } diff --git a/drivers/net/ethernet/sfc/mcdi.h b/drivers/net/ethernet/sfc/mcdi.h index dafab52aaef7..2c526d2edeb6 100644 --- a/drivers/net/ethernet/sfc/mcdi.h +++ b/drivers/net/ethernet/sfc/mcdi.h @@ -150,7 +150,7 @@ int efx_mcdi_rpc_quiet(struct efx_nic *efx, unsigned cmd, efx_dword_t *outbuf, size_t outlen, size_t *outlen_actual); -int efx_mcdi_rpc_start(struct efx_nic *efx, unsigned cmd, +int efx_mcdi_rpc_start(struct efx_nic *efx, u32 client_id, unsigned int cmd, const efx_dword_t *inbuf, size_t inlen); int efx_mcdi_rpc_finish(struct efx_nic *efx, unsigned cmd, size_t inlen, efx_dword_t *outbuf, size_t outlen, diff --git a/drivers/net/ethernet/sfc/net_driver.h b/drivers/net/ethernet/sfc/net_driver.h index 1da71deac71c..948c7a06403a 100644 --- a/drivers/net/ethernet/sfc/net_driver.h +++ b/drivers/net/ethernet/sfc/net_driver.h @@ -859,6 +859,7 @@ struct efx_mae; * @secondary_list: List of &struct efx_nic instances for the secondary PCI * functions of the controller, if this is for the primary function. * Serialised by rtnl_lock. + * @client_id: client ID of this PCIe function * @type: Controller type attributes * @legacy_irq: IRQ number * @workqueue: Workqueue for port reconfigures and the HW monitor. @@ -1022,6 +1023,7 @@ struct efx_nic { struct list_head secondary_list; struct pci_dev *pci_dev; unsigned int port_num; + u32 client_id; const struct efx_nic_type *type; int legacy_irq; bool eeh_disabled_legacy_irq; diff --git a/drivers/net/ethernet/sfc/ptp.c b/drivers/net/ethernet/sfc/ptp.c index 9f07e1ba7780..d90d4f6b3824 100644 --- a/drivers/net/ethernet/sfc/ptp.c +++ b/drivers/net/ethernet/sfc/ptp.c @@ -1052,8 +1052,8 @@ static int efx_ptp_synchronize(struct efx_nic *efx, unsigned int num_readings) /* Clear flag that signals MC ready */ WRITE_ONCE(*start, 0); - rc = efx_mcdi_rpc_start(efx, MC_CMD_PTP, synch_buf, - MC_CMD_PTP_IN_SYNCHRONIZE_LEN); + rc = efx_mcdi_rpc_start(efx, MC_CMD_CLIENT_ID_SELF, MC_CMD_PTP, + synch_buf, MC_CMD_PTP_IN_SYNCHRONIZE_LEN); EFX_WARN_ON_ONCE_PARANOID(rc); /* Wait for start from MCDI (or timeout) */