From nobody Wed Nov 27 07:39:38 2024 Received: from trager.us (trager.us [52.5.81.116]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0A98C32C8B; Tue, 22 Oct 2024 01:41:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=52.5.81.116 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729561276; cv=none; b=b156OwDxSTmr5mSwBVQRqJiDbuNeXdG3qJ0u9rHwPrBPhfbuj0xhKv6dXfYZMad241N7aEWRz08pKMJ9oujhlm8wId7aIIcXveZZe+n0En7ByeFiefZBn7FAYU+9RY+FU22uuzmcZEkkuB9Ey2pVMlwnmyirB3MFbWHHkR5nU04= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729561276; c=relaxed/simple; bh=+bEwcUGfC3Kt5K5pJyllFNbCpdJvsEcyVMhOF7XqMTE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=m9S+7eCkflN9cEWscpkymVGfb3ESlaZnOZ42srnSTgNtDIjUaBzBqMaEXQFNCjM1ahhlK+BylejcxlU5bzXKOwg1k8Eb6MZxdoQ2MuvD92N+RYWKog4grEno7fg5cf/wIc3357zNh3DmiMBbTQXgDL1fCGypfUUhk7UkosgA0uI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us; spf=pass smtp.mailfrom=trager.us; arc=none smtp.client-ip=52.5.81.116 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=trager.us Received: from [163.114.132.6] (helo=localhost) by trager.us with esmtpsa (TLSv1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.92.3) (envelope-from ) id 1t33tS-0003yD-Ts; Tue, 22 Oct 2024 01:41:07 +0000 From: Lee Trager To: Alexander Duyck , Jakub Kicinski , kernel-team@meta.com, "David S. Miller" , Eric Dumazet , Paolo Abeni , Jonathan Corbet , Andrew Lunn , Lee Trager , Sanman Pradhan , Al Viro , Mohsin Bashir , Simon Horman Cc: netdev@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v2 1/2] eth: fbnic: Add mailbox support for PLDM updates Date: Mon, 21 Oct 2024 18:37:46 -0700 Message-ID: <20241022013941.3764567-2-lee@trager.us> X-Mailer: git-send-email 2.43.5 In-Reply-To: <20241012023646.3124717-1-lee@trager.us> References: <20241012023646.3124717-1-lee@trager.us> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" This adds driver support to utilize the kernel completion API. This allows the driver to block on a response from firmware. Initially fbnic_fw_completion only has support for updates, future patches will add additional features. Signed-off-by: Lee Trager Reviewed-by: Simon Horman --- drivers/net/ethernet/meta/fbnic/fbnic_fw.c | 263 +++++++++++++++++++++ drivers/net/ethernet/meta/fbnic/fbnic_fw.h | 64 ++++- 2 files changed, 323 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_fw.c b/drivers/net/ether= net/meta/fbnic/fbnic_fw.c index 8f7a2a19ddf8..d46afc38b4a5 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic_fw.c +++ b/drivers/net/ethernet/meta/fbnic/fbnic_fw.c @@ -207,6 +207,38 @@ static int fbnic_mbx_map_tlv_msg(struct fbnic_dev *fbd, return err; } +static int fbnic_mbx_map_req_w_cmpl(struct fbnic_dev *fbd, + struct fbnic_tlv_msg *msg, + struct fbnic_fw_completion *cmpl_data) +{ + unsigned long flags; + int err; + + spin_lock_irqsave(&fbd->fw_tx_lock, flags); + + /* If we are already waiting on a completion then abort */ + if (cmpl_data && fbd->cmpl_data) { + err =3D -EBUSY; + goto unlock_mbx; + } + + /* Record completion location and submit request */ + if (cmpl_data) + fbd->cmpl_data =3D cmpl_data; + + err =3D fbnic_mbx_map_msg(fbd, FBNIC_IPC_MBX_TX_IDX, msg, + le16_to_cpu(msg->hdr.len) * sizeof(u32), 1); + + /* If msg failed then clear completion data for next caller */ + if (err && cmpl_data) + fbd->cmpl_data =3D NULL; + +unlock_mbx: + spin_unlock_irqrestore(&fbd->fw_tx_lock, flags); + + return err; +} + static void fbnic_mbx_process_tx_msgs(struct fbnic_dev *fbd) { struct fbnic_fw_mbx *tx_mbx =3D &fbd->mbx[FBNIC_IPC_MBX_TX_IDX]; @@ -651,6 +683,225 @@ void fbnic_fw_check_heartbeat(struct fbnic_dev *fbd) dev_warn(fbd->dev, "Failed to send heartbeat message\n"); } +/** + * fbnic_fw_xmit_fw_start_upgrade - Create and transmit a start update mes= sage + * @fbd: FBNIC device structure + * @cmpl_data: Completion data for upgrade process + * @id: Component ID + * @len: Length of FW update package data + * + * Return: zero on success, negative value on failure + * + * Asks the FW to prepare for starting a firmware upgrade + */ +int fbnic_fw_xmit_fw_start_upgrade(struct fbnic_dev *fbd, + struct fbnic_fw_completion *cmpl_data, + unsigned int id, unsigned int len) +{ + struct fbnic_tlv_msg *msg; + int err; + + if (!len) + return -EINVAL; + + msg =3D fbnic_tlv_msg_alloc(FBNIC_TLV_MSG_ID_FW_START_UPGRADE_REQ); + if (!msg) + return -ENOMEM; + + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_START_UPGRADE_SECTION, id); + if (err) + goto free_message; + + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_START_UPGRADE_IMAGE_LENGTH, + len); + if (err) + goto free_message; + + err =3D fbnic_mbx_map_req_w_cmpl(fbd, msg, cmpl_data); + if (err) + goto free_message; + + return 0; + +free_message: + free_page((unsigned long)msg); + return err; +} + +static const struct fbnic_tlv_index fbnic_fw_start_upgrade_resp_index[] = =3D { + FBNIC_TLV_ATTR_S32(FBNIC_FW_START_UPGRADE_ERROR), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_start_upgrade_resp(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_fw_completion *cmpl_data; + struct fbnic_dev *fbd =3D opaque; + int err =3D 0; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ) + return -ENOSPC; + + /* Check for errors */ + get_signed_result(FBNIC_FW_START_UPGRADE_ERROR, err); + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + + return 0; +} + +static const struct fbnic_tlv_index fbnic_fw_write_chunk_req_index[] =3D { + FBNIC_TLV_ATTR_U32(FBNIC_FW_WRITE_CHUNK_OFFSET), + FBNIC_TLV_ATTR_U32(FBNIC_FW_WRITE_CHUNK_LENGTH), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_write_chunk_req(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_fw_completion *cmpl_data; + struct fbnic_dev *fbd =3D opaque; + u32 length =3D 0, offset =3D 0; + struct fbnic_tlv_msg *msg; + int err; + + /* Start by attempting to allocate a response to the message */ + msg =3D fbnic_tlv_msg_alloc(FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_RESP); + if (!msg) + return -ENOMEM; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ) { + err =3D -ENOSPC; + goto msg_err; + } + + /* Notify FW if the data link has been severed */ + if (!cmpl_data->fw_update.data) { + err =3D -ECANCELED; + goto msg_err; + } + + /* Pull length/offset pair and mark it as complete */ + get_unsigned_result(FBNIC_FW_WRITE_CHUNK_OFFSET, offset); + get_unsigned_result(FBNIC_FW_WRITE_CHUNK_LENGTH, length); + + /* Record offset and length for the response */ + if (offset) { + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_WRITE_CHUNK_OFFSET, + offset); + if (err) + goto msg_err; + } + + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_WRITE_CHUNK_LENGTH, + length); + if (err) + goto msg_err; + + /* Verify length */ + if (!length || length > TLV_MAX_DATA) { + err =3D -EINVAL; + goto msg_err; + } + + /* Verify offset and length are within bounds */ + if (offset >=3D cmpl_data->fw_update.size || + offset + length > cmpl_data->fw_update.size) { + err =3D -EFAULT; + goto msg_err; + } + + /* Add outbound data to message */ + err =3D fbnic_tlv_attr_put_value(msg, FBNIC_FW_WRITE_CHUNK_DATA, + cmpl_data->fw_update.data + offset, + length); + + /* Notify the waiting thread that we processed a message */ + if (!err) + cmpl_data->fw_update.last_offset =3D offset; + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + +msg_err: + /* Report error to FW if one occurred */ + if (err) + fbnic_tlv_attr_put_int(msg, FBNIC_FW_WRITE_CHUNK_ERROR, err); + + /* Map and send the response */ + err =3D fbnic_mbx_map_tlv_msg(fbd, msg); + if (err) + free_page((unsigned long)msg); + + return err; +} + +static const struct fbnic_tlv_index fbnic_fw_verify_image_resp_index[] =3D= { + FBNIC_TLV_ATTR_S32(FBNIC_FW_VERIFY_IMAGE_ERROR), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_verify_image_resp(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_fw_completion *cmpl_data; + struct fbnic_dev *fbd =3D opaque; + int err =3D 0; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_VERIFY_IMAGE_RESP) + return -ENOSPC; + + /* Check for errors */ + get_signed_result(FBNIC_FW_VERIFY_IMAGE_ERROR, err); + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + + return err; +} + +static const struct fbnic_tlv_index fbnic_fw_finish_upgrade_req_index[] = =3D { + FBNIC_TLV_ATTR_S32(FBNIC_FW_FINISH_UPGRADE_ERROR), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_finish_upgrade_req(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_fw_completion *cmpl_data; + struct fbnic_dev *fbd =3D opaque; + int err =3D 0; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ) + return -ENOSPC; + + /* Check for errors */ + get_signed_result(FBNIC_FW_FINISH_UPGRADE_ERROR, err); + + /* Close out update by clearing data pointer */ + cmpl_data->fw_update.last_offset =3D cmpl_data->fw_update.size; + cmpl_data->fw_update.data =3D NULL; + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + + return 0; +} + static const struct fbnic_tlv_parser fbnic_fw_tlv_parser[] =3D { FBNIC_TLV_PARSER(FW_CAP_RESP, fbnic_fw_cap_resp_index, fbnic_fw_parse_cap_resp), @@ -658,6 +909,18 @@ static const struct fbnic_tlv_parser fbnic_fw_tlv_pars= er[] =3D { fbnic_fw_parse_ownership_resp), FBNIC_TLV_PARSER(HEARTBEAT_RESP, fbnic_heartbeat_resp_index, fbnic_fw_parse_heartbeat_resp), + FBNIC_TLV_PARSER(FW_START_UPGRADE_RESP, + fbnic_fw_start_upgrade_resp_index, + fbnic_fw_parse_fw_start_upgrade_resp), + FBNIC_TLV_PARSER(FW_WRITE_CHUNK_REQ, + fbnic_fw_write_chunk_req_index, + fbnic_fw_parse_fw_write_chunk_req), + FBNIC_TLV_PARSER(FW_VERIFY_IMAGE_RESP, + fbnic_fw_verify_image_resp_index, + fbnic_fw_parse_fw_verify_image_resp), + FBNIC_TLV_PARSER(FW_FINISH_UPGRADE_REQ, + fbnic_fw_finish_upgrade_req_index, + fbnic_fw_parse_fw_finish_upgrade_req), FBNIC_TLV_MSG_ERROR }; diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_fw.h b/drivers/net/ether= net/meta/fbnic/fbnic_fw.h index 7cd8841920e4..347e60fee551 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic_fw.h +++ b/drivers/net/ethernet/meta/fbnic/fbnic_fw.h @@ -45,10 +45,20 @@ struct fbnic_fw_cap { }; struct fbnic_fw_completion { - struct { - s32 millivolts; - s32 millidegrees; - } tsene; + u32 msg_type; + struct completion done; + int result; + union { + struct { + u32 size; + u32 last_offset; + const u8 *data; + } fw_update; + struct { + s32 millivolts; + s32 millidegrees; + } tsene; + }; }; void fbnic_mbx_init(struct fbnic_dev *fbd); @@ -59,6 +69,9 @@ void fbnic_mbx_flush_tx(struct fbnic_dev *fbd); int fbnic_fw_xmit_ownership_msg(struct fbnic_dev *fbd, bool take_ownership= ); int fbnic_fw_init_heartbeat(struct fbnic_dev *fbd, bool poll); void fbnic_fw_check_heartbeat(struct fbnic_dev *fbd); +int fbnic_fw_xmit_fw_start_upgrade(struct fbnic_dev *fbd, + struct fbnic_fw_completion *cmpl_data, + unsigned int id, unsigned int len); #define fbnic_mk_full_fw_ver_str(_rev_id, _delim, _commit, _str, _str_sz) \ do { \ @@ -74,6 +87,15 @@ do { \ #define fbnic_mk_fw_ver_str(_rev_id, _str) \ fbnic_mk_full_fw_ver_str(_rev_id, "", "", _str, sizeof(_str)) +enum { + QSPI_SECTION_CMRT =3D 0, + QSPI_SECTION_CONTROL_FW =3D 1, + QSPI_SECTION_UCODE =3D 2, + QSPI_SECTION_OPTION_ROM =3D 3, + QSPI_SECTION_USER =3D 4, + QSPI_SECTION_INVALID, +}; + #define FW_HEARTBEAT_PERIOD (10 * HZ) enum { @@ -83,6 +105,14 @@ enum { FBNIC_TLV_MSG_ID_OWNERSHIP_RESP =3D 0x13, FBNIC_TLV_MSG_ID_HEARTBEAT_REQ =3D 0x14, FBNIC_TLV_MSG_ID_HEARTBEAT_RESP =3D 0x15, + FBNIC_TLV_MSG_ID_FW_START_UPGRADE_REQ =3D 0x22, + FBNIC_TLV_MSG_ID_FW_START_UPGRADE_RESP =3D 0x23, + FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ =3D 0x24, + FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_RESP =3D 0x25, + FBNIC_TLV_MSG_ID_FW_VERIFY_IMAGE_REQ =3D 0x26, + FBNIC_TLV_MSG_ID_FW_VERIFY_IMAGE_RESP =3D 0x27, + FBNIC_TLV_MSG_ID_FW_FINISH_UPGRADE_REQ =3D 0x28, + FBNIC_TLV_MSG_ID_FW_FINISH_UPGRADE_RESP =3D 0x29, }; #define FBNIC_FW_CAP_RESP_VERSION_MAJOR CSR_GENMASK(31, 24) @@ -128,4 +158,30 @@ enum { FBNIC_FW_OWNERSHIP_FLAG =3D 0x0, FBNIC_FW_OWNERSHIP_MSG_MAX }; + +enum { + FBNIC_FW_START_UPGRADE_ERROR =3D 0x0, + FBNIC_FW_START_UPGRADE_SECTION =3D 0x1, + FBNIC_FW_START_UPGRADE_IMAGE_LENGTH =3D 0x2, + FBNIC_FW_START_UPGRADE_MSG_MAX +}; + +enum { + FBNIC_FW_WRITE_CHUNK_OFFSET =3D 0x0, + FBNIC_FW_WRITE_CHUNK_LENGTH =3D 0x1, + FBNIC_FW_WRITE_CHUNK_DATA =3D 0x2, + FBNIC_FW_WRITE_CHUNK_ERROR =3D 0x3, + FBNIC_FW_WRITE_CHUNK_MSG_MAX +}; + +enum { + FBNIC_FW_VERIFY_IMAGE_ERROR =3D 0x0, + FBNIC_FW_VERIFY_IMAGE_MSG_MAX +}; + +enum { + FBNIC_FW_FINISH_UPGRADE_ERROR =3D 0x0, + FBNIC_FW_FINISH_UPGRADE_MSG_MAX +}; + #endif /* _FBNIC_FW_H_ */ -- 2.43.5 From nobody Wed Nov 27 07:39:38 2024 Received: from trager.us (trager.us [52.5.81.116]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8809417BA0; Sat, 12 Oct 2024 02:38:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=52.5.81.116 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728700712; cv=none; b=NMj6VbmOSTe37hfCXq8SeQNxLk0WfE9guEQtXFAATNI3ln4e5wAhWWnpePRmtJfFLflqwfeW9cLaFip60TrSYL29PBlWAmFlcxdjbLcabtmVjDRwjiCEPe2oiXQW74cF7t16sdRT5rrvNhPMoGyHQeHdGeFDjZ2+TlFa7Tg0KM4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728700712; c=relaxed/simple; bh=MlYqaemN4ty1btUPl2dmYBhHK8J+e8/x/odbjpL7Gi4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Y9EP4bbyD6JbF7lAF3RHzi2B9Gj7XpquQ6cl9PHEsT2XJTF2AngrJGFtJe4+569ztaqzfilxD1p0dk2EBGexe7kgDeGQ/wAuAEXpzu4fvc4aQSIQVXptqOzSUUm+DTBnWi72SZVDAdYu7Se8jW17GzJKL4wFcn/59XPHYfQ+9aw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us; spf=pass smtp.mailfrom=trager.us; arc=none smtp.client-ip=52.5.81.116 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=trager.us Received: from c-76-104-255-50.hsd1.wa.comcast.net ([76.104.255.50] helo=localhost) by trager.us with esmtpsa (TLSv1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.92.3) (envelope-from ) id 1szS1O-0005hV-1L; Sat, 12 Oct 2024 02:38:22 +0000 From: Lee Trager To: Alexander Duyck , Jakub Kicinski , kernel-team@meta.com, "David S. Miller" , Eric Dumazet , Paolo Abeni , Jonathan Corbet , Mohsin Bashir , Simon Horman , Lee Trager , Vadim Fedorenko , Sanman Pradhan , Al Viro Cc: netdev@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next 1/2] eth: fbnic: Add mailbox support for PLDM updates Date: Fri, 11 Oct 2024 19:34:03 -0700 Message-ID: <20241012023646.3124717-2-lee@trager.us> X-Mailer: git-send-email 2.43.5 In-Reply-To: <20241012023646.3124717-1-lee@trager.us> References: <20241012023646.3124717-1-lee@trager.us> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" This adds driver support to utilize the kernel completion API. This allows the driver to block on a response from firmware. Initially fbnic_fw_completion only has support for updates, future patches will add additional features. Signed-off-by: Lee Trager --- drivers/net/ethernet/meta/fbnic/fbnic.h | 1 + drivers/net/ethernet/meta/fbnic/fbnic_fw.c | 263 +++++++++++++++++++++ drivers/net/ethernet/meta/fbnic/fbnic_fw.h | 59 +++++ 3 files changed, 323 insertions(+) diff --git a/drivers/net/ethernet/meta/fbnic/fbnic.h b/drivers/net/ethernet= /meta/fbnic/fbnic.h index ca59261f0155..f58727e6e45a 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic.h +++ b/drivers/net/ethernet/meta/fbnic/fbnic.h @@ -31,6 +31,7 @@ struct fbnic_dev { struct fbnic_fw_mbx mbx[FBNIC_IPC_MBX_INDICES]; struct fbnic_fw_cap fw_cap; + struct fbnic_fw_completion *cmpl_data; /* Lock protecting Tx Mailbox queue to prevent possible races */ spinlock_t fw_tx_lock; diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_fw.c b/drivers/net/ether= net/meta/fbnic/fbnic_fw.c index 8f7a2a19ddf8..deebff3b6821 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic_fw.c +++ b/drivers/net/ethernet/meta/fbnic/fbnic_fw.c @@ -207,6 +207,38 @@ static int fbnic_mbx_map_tlv_msg(struct fbnic_dev *fbd, return err; } +static int fbnic_mbx_map_req_w_cmpl(struct fbnic_dev *fbd, + struct fbnic_tlv_msg *msg, + struct fbnic_fw_completion *cmpl_data) +{ + unsigned long flags; + int err; + + spin_lock_irqsave(&fbd->fw_tx_lock, flags); + + /* If we are already waiting on a completion then abort */ + if (cmpl_data && fbd->cmpl_data) { + err =3D -EBUSY; + goto unlock_mbx; + } + + /* Record completion location and submit request */ + if (cmpl_data) + fbd->cmpl_data =3D cmpl_data; + + err =3D fbnic_mbx_map_msg(fbd, FBNIC_IPC_MBX_TX_IDX, msg, + le16_to_cpu(msg->hdr.len) * sizeof(u32), 1); + + /* If msg failed then clear completion data for next caller */ + if (err && cmpl_data) + fbd->cmpl_data =3D NULL; + +unlock_mbx: + spin_unlock_irqrestore(&fbd->fw_tx_lock, flags); + + return err; +} + static void fbnic_mbx_process_tx_msgs(struct fbnic_dev *fbd) { struct fbnic_fw_mbx *tx_mbx =3D &fbd->mbx[FBNIC_IPC_MBX_TX_IDX]; @@ -651,6 +683,225 @@ void fbnic_fw_check_heartbeat(struct fbnic_dev *fbd) dev_warn(fbd->dev, "Failed to send heartbeat message\n"); } +/** + * fbnic_fw_xmit_fw_start_upgrade - Create and transmit a start update mes= sage + * @fbd: FBNIC device structure + * @cmpl_data: Completion data for upgrade process + * @id: Component ID + * @len: Length of FW update package data + * + * Return: zero on success, negative value on failure + * + * Asks the FW to prepare for starting a firmware upgrade + */ +int fbnic_fw_xmit_fw_start_upgrade(struct fbnic_dev *fbd, + struct fbnic_fw_completion *cmpl_data, + unsigned int id, unsigned int len) +{ + struct fbnic_tlv_msg *msg; + int err; + + if (!len) + return -EINVAL; + + msg =3D fbnic_tlv_msg_alloc(FBNIC_TLV_MSG_ID_FW_START_UPGRADE_REQ); + if (!msg) + return -ENOMEM; + + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_START_UPGRADE_SECTION, id); + if (err) + goto free_message; + + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_START_UPGRADE_IMAGE_LENGTH, + len); + if (err) + goto free_message; + + err =3D fbnic_mbx_map_req_w_cmpl(fbd, msg, cmpl_data); + if (err) + goto free_message; + + return 0; + +free_message: + free_page((unsigned long)msg); + return err; +} + +static const struct fbnic_tlv_index fbnic_fw_start_upgrade_resp_index[] = =3D { + FBNIC_TLV_ATTR_S32(FBNIC_FW_START_UPGRADE_ERROR), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_start_upgrade_resp(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_dev *fbd =3D opaque; + struct fbnic_fw_completion *cmpl_data; + int err =3D 0; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ) + return -ENOSPC; + + /* Check for errors */ + get_signed_result(FBNIC_FW_START_UPGRADE_ERROR, err); + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + + return 0; +} + +static const struct fbnic_tlv_index fbnic_fw_write_chunk_req_index[] =3D { + FBNIC_TLV_ATTR_U32(FBNIC_FW_WRITE_CHUNK_OFFSET), + FBNIC_TLV_ATTR_U32(FBNIC_FW_WRITE_CHUNK_LENGTH), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_write_chunk_req(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_dev *fbd =3D opaque; + struct fbnic_fw_completion *cmpl_data; + u32 length =3D 0, offset =3D 0; + struct fbnic_tlv_msg *msg; + int err; + + /* Start by attempting to allocate a response to the message */ + msg =3D fbnic_tlv_msg_alloc(FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_RESP); + if (!msg) + return -ENOMEM; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ) { + err =3D -ENOSPC; + goto msg_err; + } + + /* Notify FW if the data link has been severed */ + if (!cmpl_data->fw_update.data) { + err =3D -ECANCELED; + goto msg_err; + } + + /* Pull length/offset pair and mark it as complete */ + get_unsigned_result(FBNIC_FW_WRITE_CHUNK_OFFSET, offset); + get_unsigned_result(FBNIC_FW_WRITE_CHUNK_LENGTH, length); + + /* Record offset and length for the response */ + if (offset) { + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_WRITE_CHUNK_OFFSET, + offset); + if (err) + goto msg_err; + } + + err =3D fbnic_tlv_attr_put_int(msg, FBNIC_FW_WRITE_CHUNK_LENGTH, + length); + if (err) + goto msg_err; + + /* Verify length */ + if (!length || length > TLV_MAX_DATA) { + err =3D -EINVAL; + goto msg_err; + } + + /* Verify offset and length are within bounds */ + if (offset >=3D cmpl_data->fw_update.size || + offset + length > cmpl_data->fw_update.size) { + err =3D -EFAULT; + goto msg_err; + } + + /* Add outbound data to message */ + err =3D fbnic_tlv_attr_put_value(msg, FBNIC_FW_WRITE_CHUNK_DATA, + cmpl_data->fw_update.data + offset, + length); + + /* Notify the waiting thread that we processed a message */ + if (!err) + cmpl_data->fw_update.last_offset =3D offset; + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + +msg_err: + /* Report error to FW if one occurred */ + if (err) + fbnic_tlv_attr_put_int(msg, FBNIC_FW_WRITE_CHUNK_ERROR, err); + + /* Map and send the response */ + err =3D fbnic_mbx_map_tlv_msg(fbd, msg); + if (err) + free_page((unsigned long)msg); + + return err; +} + +static const struct fbnic_tlv_index fbnic_fw_verify_image_resp_index[] =3D= { + FBNIC_TLV_ATTR_S32(FBNIC_FW_VERIFY_IMAGE_ERROR), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_verify_image_resp(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_dev *fbd =3D opaque; + struct fbnic_fw_completion *cmpl_data; + int err =3D 0; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_VERIFY_IMAGE_RESP) + return -ENOSPC; + + /* Check for errors */ + get_signed_result(FBNIC_FW_VERIFY_IMAGE_ERROR, err); + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + + return err; +} + +static const struct fbnic_tlv_index fbnic_fw_finish_upgrade_req_index[] = =3D { + FBNIC_TLV_ATTR_S32(FBNIC_FW_FINISH_UPGRADE_ERROR), + FBNIC_TLV_ATTR_LAST +}; + +static int fbnic_fw_parse_fw_finish_upgrade_req(void *opaque, + struct fbnic_tlv_msg **results) +{ + struct fbnic_dev *fbd =3D opaque; + struct fbnic_fw_completion *cmpl_data; + int err =3D 0; + + /* Verify we have a completion pointer */ + cmpl_data =3D READ_ONCE(fbd->cmpl_data); + if (!cmpl_data || + cmpl_data->msg_type !=3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ) + return -ENOSPC; + + /* Check for errors */ + get_signed_result(FBNIC_FW_FINISH_UPGRADE_ERROR, err); + + /* Close out update by clearing data pointer */ + cmpl_data->fw_update.last_offset =3D cmpl_data->fw_update.size; + cmpl_data->fw_update.data =3D NULL; + + cmpl_data->result =3D err; + complete(&cmpl_data->done); + + return 0; +} + static const struct fbnic_tlv_parser fbnic_fw_tlv_parser[] =3D { FBNIC_TLV_PARSER(FW_CAP_RESP, fbnic_fw_cap_resp_index, fbnic_fw_parse_cap_resp), @@ -658,6 +909,18 @@ static const struct fbnic_tlv_parser fbnic_fw_tlv_pars= er[] =3D { fbnic_fw_parse_ownership_resp), FBNIC_TLV_PARSER(HEARTBEAT_RESP, fbnic_heartbeat_resp_index, fbnic_fw_parse_heartbeat_resp), + FBNIC_TLV_PARSER(FW_START_UPGRADE_RESP, + fbnic_fw_start_upgrade_resp_index, + fbnic_fw_parse_fw_start_upgrade_resp), + FBNIC_TLV_PARSER(FW_WRITE_CHUNK_REQ, + fbnic_fw_write_chunk_req_index, + fbnic_fw_parse_fw_write_chunk_req), + FBNIC_TLV_PARSER(FW_VERIFY_IMAGE_RESP, + fbnic_fw_verify_image_resp_index, + fbnic_fw_parse_fw_verify_image_resp), + FBNIC_TLV_PARSER(FW_FINISH_UPGRADE_REQ, + fbnic_fw_finish_upgrade_req_index, + fbnic_fw_parse_fw_finish_upgrade_req), FBNIC_TLV_MSG_ERROR }; diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_fw.h b/drivers/net/ether= net/meta/fbnic/fbnic_fw.h index 221faf8c6756..a638d73d2da5 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic_fw.h +++ b/drivers/net/ethernet/meta/fbnic/fbnic_fw.h @@ -44,6 +44,19 @@ struct fbnic_fw_cap { u8 link_fec; }; +struct fbnic_fw_completion { + u32 msg_type; + struct completion done; + int result; + union { + struct { + u32 size; + u32 last_offset; + const u8 *data; + } fw_update; + }; +}; + void fbnic_mbx_init(struct fbnic_dev *fbd); void fbnic_mbx_clean(struct fbnic_dev *fbd); void fbnic_mbx_poll(struct fbnic_dev *fbd); @@ -52,6 +65,9 @@ void fbnic_mbx_flush_tx(struct fbnic_dev *fbd); int fbnic_fw_xmit_ownership_msg(struct fbnic_dev *fbd, bool take_ownership= ); int fbnic_fw_init_heartbeat(struct fbnic_dev *fbd, bool poll); void fbnic_fw_check_heartbeat(struct fbnic_dev *fbd); +int fbnic_fw_xmit_fw_start_upgrade(struct fbnic_dev *fbd, + struct fbnic_fw_completion *cmpl_data, + unsigned int id, unsigned int len); #define fbnic_mk_full_fw_ver_str(_rev_id, _delim, _commit, _str, _str_sz) \ do { \ @@ -67,6 +83,15 @@ do { \ #define fbnic_mk_fw_ver_str(_rev_id, _str) \ fbnic_mk_full_fw_ver_str(_rev_id, "", "", _str, sizeof(_str)) +enum { + QSPI_SECTION_CMRT =3D 0, + QSPI_SECTION_CONTROL_FW =3D 1, + QSPI_SECTION_UCODE =3D 2, + QSPI_SECTION_OPTION_ROM =3D 3, + QSPI_SECTION_USER =3D 4, + QSPI_SECTION_INVALID, +}; + #define FW_HEARTBEAT_PERIOD (10 * HZ) enum { @@ -76,6 +101,14 @@ enum { FBNIC_TLV_MSG_ID_OWNERSHIP_RESP =3D 0x13, FBNIC_TLV_MSG_ID_HEARTBEAT_REQ =3D 0x14, FBNIC_TLV_MSG_ID_HEARTBEAT_RESP =3D 0x15, + FBNIC_TLV_MSG_ID_FW_START_UPGRADE_REQ =3D 0x22, + FBNIC_TLV_MSG_ID_FW_START_UPGRADE_RESP =3D 0x23, + FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ =3D 0x24, + FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_RESP =3D 0x25, + FBNIC_TLV_MSG_ID_FW_VERIFY_IMAGE_REQ =3D 0x26, + FBNIC_TLV_MSG_ID_FW_VERIFY_IMAGE_RESP =3D 0x27, + FBNIC_TLV_MSG_ID_FW_FINISH_UPGRADE_REQ =3D 0x28, + FBNIC_TLV_MSG_ID_FW_FINISH_UPGRADE_RESP =3D 0x29, }; #define FBNIC_FW_CAP_RESP_VERSION_MAJOR CSR_GENMASK(31, 24) @@ -121,4 +154,30 @@ enum { FBNIC_FW_OWNERSHIP_FLAG =3D 0x0, FBNIC_FW_OWNERSHIP_MSG_MAX }; + +enum { + FBNIC_FW_START_UPGRADE_ERROR =3D 0x0, + FBNIC_FW_START_UPGRADE_SECTION =3D 0x1, + FBNIC_FW_START_UPGRADE_IMAGE_LENGTH =3D 0x2, + FBNIC_FW_START_UPGRADE_MSG_MAX +}; + +enum { + FBNIC_FW_WRITE_CHUNK_OFFSET =3D 0x0, + FBNIC_FW_WRITE_CHUNK_LENGTH =3D 0x1, + FBNIC_FW_WRITE_CHUNK_DATA =3D 0x2, + FBNIC_FW_WRITE_CHUNK_ERROR =3D 0x3, + FBNIC_FW_WRITE_CHUNK_MSG_MAX +}; + +enum { + FBNIC_FW_VERIFY_IMAGE_ERROR =3D 0x0, + FBNIC_FW_VERIFY_IMAGE_MSG_MAX +}; + +enum { + FBNIC_FW_FINISH_UPGRADE_ERROR =3D 0x0, + FBNIC_FW_FINISH_UPGRADE_MSG_MAX +}; + #endif /* _FBNIC_FW_H_ */ -- 2.43.5 From nobody Wed Nov 27 07:39:38 2024 Received: from trager.us (trager.us [52.5.81.116]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9EFB284D29; Tue, 22 Oct 2024 01:44:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=52.5.81.116 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729561461; cv=none; b=c3ehO7sPzDvKFDk3BOLL50DYnu4upwWGrQznB6cypMbRZw1MBBBJ2KEUmzjY3jwvX2C1q2AKkDNrheyaRE/nEiH4h4UsO67rZZHsvK9ff7+ZeiVuEml59yZWCtYYrWXzljSBQECNcxAOF6X7IgJtHLo+QiDWT+hH9YYFHnQlg5g= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729561461; c=relaxed/simple; bh=6Z+AxkKR9zlrGHBwt2JjK+ih49DyBUV8ksoi07TJ5Tc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=r0OR8zkeqtOUqJF8PSn1+40qzBSSXnmfXVkWgDt585jl+2ASKmroQTdum1NRovBLs65z5VyXFKzkDsO9Lc5FW2M1l+zT1j5KTzfAHMQ4GZTx8iSSj6/Tq5YmHM9zql/zIijlL6eWdsy9cYEtuEmmgGxtLjH66O7kHT8pcC7GCQo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us; spf=pass smtp.mailfrom=trager.us; arc=none smtp.client-ip=52.5.81.116 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=trager.us Received: from [163.114.132.6] (helo=localhost) by trager.us with esmtpsa (TLSv1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.92.3) (envelope-from ) id 1t33wQ-0003yc-CK; Tue, 22 Oct 2024 01:44:10 +0000 From: Lee Trager To: Alexander Duyck , Jakub Kicinski , kernel-team@meta.com, "David S. Miller" , Eric Dumazet , Paolo Abeni , Jonathan Corbet , Andrew Lunn , Lee Trager , Sanman Pradhan , Al Viro , Simon Horman , Mohsin Bashir Cc: netdev@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v2 2/2] eth: fbnic: Add devlink dev flash support Date: Mon, 21 Oct 2024 18:42:24 -0700 Message-ID: <20241022014319.3791797-1-lee@trager.us> X-Mailer: git-send-email 2.43.5 In-Reply-To: <20241012023646.3124717-1-lee@trager.us> References: <20241012023646.3124717-1-lee@trager.us> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" fbnic supports updating firmware using a PLDM image signed and distributed by Meta. PLDM images are written into stored flashed. Flashing does not interrupt operation. On host reboot the newly flashed UEFI driver will be used. To run new control or cmrt firmware the NIC must be power cycled. Signed-off-by: Lee Trager Reviewed-by: Simon Horman --- .../device_drivers/ethernet/meta/fbnic.rst | 11 + drivers/net/ethernet/meta/Kconfig | 1 + .../net/ethernet/meta/fbnic/fbnic_devlink.c | 269 +++++++++++++++++- 3 files changed, 280 insertions(+), 1 deletion(-) diff --git a/Documentation/networking/device_drivers/ethernet/meta/fbnic.rs= t b/Documentation/networking/device_drivers/ethernet/meta/fbnic.rst index 32ff114f5c26..d6726c254818 100644 --- a/Documentation/networking/device_drivers/ethernet/meta/fbnic.rst +++ b/Documentation/networking/device_drivers/ethernet/meta/fbnic.rst @@ -27,3 +27,14 @@ driver takes over. devlink dev info provides version information for all three components. In addition to the version the hg commit hash of the build is included as a separate entry. + +Upgrading Firmware +------------------ + +fbnic supports upgrading firmware using devlink dev flash. Firmware images +are signed and distributed by Meta. All firmware is bundled into a single +PLDM image which is written into stored flash. Flashing firmware does not +interrupt operation. + +On host reboot the newly flashed UEFI driver will be used. To run new cont= rol +or cmrt firmware the NIC must be power cycled. diff --git a/drivers/net/ethernet/meta/Kconfig b/drivers/net/ethernet/meta/= Kconfig index 831921b9d4d5..3ba527514f1e 100644 --- a/drivers/net/ethernet/meta/Kconfig +++ b/drivers/net/ethernet/meta/Kconfig @@ -27,6 +27,7 @@ config FBNIC select NET_DEVLINK select PAGE_POOL select PHYLINK + select PLDMFW help This driver supports Meta Platforms Host Network Interface. diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_devlink.c b/drivers/net/= ethernet/meta/fbnic/fbnic_devlink.c index 0072d612215e..bd718b3e314b 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic_devlink.c +++ b/drivers/net/ethernet/meta/fbnic/fbnic_devlink.c @@ -3,6 +3,7 @@ #include #include +#include #include #include @@ -109,8 +110,274 @@ static int fbnic_devlink_info_get(struct devlink *dev= link, return 0; } +/** + * fbnic_send_package_data - Send record package data to firmware + * @context: PLDM FW update structure + * @data: pointer to the package data + * @length: length of the package data + * + * Send a copy of the package data associated with the PLDM record matching + * this device to the firmware. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_send_package_data(struct pldmfw *context, const u8 *data, + u16 length) +{ + struct device *dev =3D context->dev; + + /* Temp placeholder required by devlink */ + dev_info(dev, + "Sending %u bytes of PLDM record package data to firmware\n", + length); + + return 0; +} + +/** + * fbnic_send_component_table - Send PLDM component table to the firmware + * @context: PLDM FW update structure + * @component: The component to send + * @transfer_flag: Flag indication location in component tables + * + * Read relevant data from component table and forward it to the firmware. + * Check response to verify if the firmware indicates that it wishes to + * proceed with the update. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_send_component_table(struct pldmfw *context, + struct pldmfw_component *component, + u8 transfer_flag) +{ + struct device *dev =3D context->dev; + u16 id =3D component->identifier; + u8 test_string[80]; + + switch (id) { + case QSPI_SECTION_CMRT: + case QSPI_SECTION_CONTROL_FW: + case QSPI_SECTION_OPTION_ROM: + break; + default: + dev_err(dev, "Unknown component ID %u\n", id); + return -EINVAL; + } + + dev_dbg(dev, "Sending PLDM component table to firmware\n"); + + /* Temp placeholder */ + strscpy(test_string, component->version_string, + min_t(u8, component->version_len, 79)); + dev_info(dev, "PLDMFW: Component ID: %u version %s\n", + id, test_string); + + return 0; +} + +/** + * fbnic_flash_component - Flash a component of the QSPI + * @context: PLDM FW update structure + * @component: The component table to send to FW + * + * Map contents of component and make it available for FW to download + * so that it can update the contents of the QSPI Flash. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_flash_component(struct pldmfw *context, + struct pldmfw_component *component) +{ + const u8 *data =3D component->component_data; + u32 size =3D component->component_size; + struct fbnic_fw_completion *fw_cmpl; + struct device *dev =3D context->dev; + struct pci_dev *pdev =3D to_pci_dev(dev); + u16 id =3D component->identifier; + const char *component_name; + int retries =3D 2; + int err; + + struct devlink *devlink; + struct fbnic_dev *fbd; + + switch (id) { + case QSPI_SECTION_CMRT: + component_name =3D "boot1"; + break; + case QSPI_SECTION_CONTROL_FW: + component_name =3D "boot2"; + break; + case QSPI_SECTION_OPTION_ROM: + component_name =3D "option-rom"; + break; + default: + dev_err(dev, "Unknown component ID %u\n", id); + return -EINVAL; + } + + fw_cmpl =3D kzalloc(sizeof(*fw_cmpl), GFP_KERNEL); + if (!fw_cmpl) + return -ENOMEM; + + pdev =3D to_pci_dev(dev); + fbd =3D pci_get_drvdata(pdev); + devlink =3D priv_to_devlink(fbd); + + /* Initialize completion and queue it for FW to process */ + fw_cmpl->msg_type =3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ; + init_completion(&fw_cmpl->done); + + fw_cmpl->fw_update.last_offset =3D 0; + fw_cmpl->fw_update.data =3D data; + fw_cmpl->fw_update.size =3D size; + + err =3D fbnic_fw_xmit_fw_start_upgrade(fbd, fw_cmpl, id, size); + if (err) + goto cmpl_free; + + /* Monitor completions and report status of update */ + while (fw_cmpl->fw_update.data) { + u32 offset =3D fw_cmpl->fw_update.last_offset; + + devlink_flash_update_status_notify(devlink, "Flashing", + component_name, offset, + size); + + /* Allow 5 seconds for reply, resend and try up to 2 times */ + if (wait_for_completion_timeout(&fw_cmpl->done, 5 * HZ)) { + reinit_completion(&fw_cmpl->done); + /* If we receive a reply, reinit our retry counter */ + retries =3D 2; + } else if (--retries =3D=3D 0) { + dev_err(fbd->dev, "Timed out waiting on update\n"); + err =3D -ETIMEDOUT; + goto cmpl_cleanup; + } + } + + err =3D fw_cmpl->result; + if (err) + goto cmpl_cleanup; + + devlink_flash_update_status_notify(devlink, "Flashing", + component_name, size, size); + +cmpl_cleanup: + fbd->cmpl_data =3D NULL; +cmpl_free: + kfree(fw_cmpl); + + return err; +} + +/** + * fbnic_finalize_update - Perform last steps to complete device update + * @context: PLDM FW update structure + * + * Notify FW that update is complete and that it can take any actions + * needed to finalize the FW update. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_finalize_update(struct pldmfw *context) +{ + struct device *dev =3D context->dev; + + /* Temp placeholder required by devlink */ + dev_info(dev, "PLDMFW: Finalize update\n"); + + return 0; +} + +static const struct pldmfw_ops fbnic_pldmfw_ops =3D { + .match_record =3D pldmfw_op_pci_match_record, + .send_package_data =3D fbnic_send_package_data, + .send_component_table =3D fbnic_send_component_table, + .flash_component =3D fbnic_flash_component, + .finalize_update =3D fbnic_finalize_update, +}; + +static void fbnic_devlink_flash_update_report_err(struct fbnic_dev *fbd, + struct devlink *devlink, + const char *err_msg, + int err) +{ + char err_str[128]; + + snprintf(err_str, sizeof(err_str), + "Failed to flash PLDM Image: %s (error: %d)", + err_msg, err); + devlink_flash_update_status_notify(devlink, err_str, NULL, 0, 0); + dev_err(fbd->dev, "%s\n", err_str); +} + +static int +fbnic_devlink_flash_update(struct devlink *devlink, + struct devlink_flash_update_params *params, + struct netlink_ext_ack *extack) +{ + struct fbnic_dev *fbd =3D devlink_priv(devlink); + const struct firmware *fw =3D params->fw; + struct device *dev =3D fbd->dev; + struct pldmfw context; + char *err_msg; + int err; + + if (!fw || !fw->data || !fw->size) + return -EINVAL; + + devlink_flash_update_status_notify(devlink, "Preparing to flash", + NULL, 0, 0); + + context.ops =3D &fbnic_pldmfw_ops; + context.dev =3D dev; + + err =3D pldmfw_flash_image(&context, fw); + if (err) { + switch (err) { + case -EINVAL: + err_msg =3D "Invalid image"; + break; + case -EOPNOTSUPP: + err_msg =3D "Unsupported image"; + break; + case -ENOMEM: + err_msg =3D "Out of memory"; + break; + case -EFAULT: + err_msg =3D "Invalid header"; + break; + case -ENOENT: + err_msg =3D "No matching record"; + break; + case -ENODEV: + err_msg =3D "No matching device"; + break; + case -ETIMEDOUT: + err_msg =3D "Timed out waiting for reply"; + break; + default: + err_msg =3D "Unknown error"; + break; + } + fbnic_devlink_flash_update_report_err(fbd, devlink, + err_msg, err); + } else { + devlink_flash_update_status_notify(devlink, "Flashing done", + NULL, 0, 0); + } + + return err; +} + static const struct devlink_ops fbnic_devlink_ops =3D { - .info_get =3D fbnic_devlink_info_get, + .info_get =3D fbnic_devlink_info_get, + .flash_update =3D fbnic_devlink_flash_update, }; void fbnic_devlink_free(struct fbnic_dev *fbd) -- 2.43.5 From nobody Wed Nov 27 07:39:38 2024 Received: from trager.us (trager.us [52.5.81.116]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4D58B171AA; Sat, 12 Oct 2024 02:39:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=52.5.81.116 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728700797; cv=none; b=DWGOwtrajoB+WrDUcLIIyq6moc8MJewowA+GbCXVXOC7+QNjLuhtgWywRUZez6IwmSUjSDFKOwIj73OkdhTvvCcf5sec8DSnBdvvYY5qeTRuxEH3ubAB1D3+Q2pNappt+8iEskaOS1gy6fKDy3EiX1i+bL1gIgkDa8ERQwqEiZI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728700797; c=relaxed/simple; bh=8OERmuYJLC96clMNISMjG1RlHs20BVp7aY+U3fmctW4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=aYUdaWzKh7Wk+8VrGhzHU1QgSK5fUJwFK0V414pHq1xH7M78pAB1mQx5vsmE1ycYHJNJZiJUVQVx0aG2/akgzALI0MRkeczTTnMHYKUoReTvw3rQTj2taIjTnbO/ZCrx3he7rKGnJo+Pgjuu4DU1tXzhsuL0q9No9RgrqTkFbhc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us; spf=pass smtp.mailfrom=trager.us; arc=none smtp.client-ip=52.5.81.116 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=trager.us Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=trager.us Received: from c-76-104-255-50.hsd1.wa.comcast.net ([76.104.255.50] helo=localhost) by trager.us with esmtpsa (TLSv1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.92.3) (envelope-from ) id 1szS2m-0005hn-BX; Sat, 12 Oct 2024 02:39:48 +0000 From: Lee Trager To: Alexander Duyck , Jakub Kicinski , kernel-team@meta.com, "David S. Miller" , Eric Dumazet , Paolo Abeni , Jonathan Corbet , Mohsin Bashir , Simon Horman , Vadim Fedorenko , Lee Trager , Sanman Pradhan , Al Viro Cc: netdev@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next 2/2] eth: fbnic: Add devlink dev flash support Date: Fri, 11 Oct 2024 19:34:04 -0700 Message-ID: <20241012023646.3124717-3-lee@trager.us> X-Mailer: git-send-email 2.43.5 In-Reply-To: <20241012023646.3124717-1-lee@trager.us> References: <20241012023646.3124717-1-lee@trager.us> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" fbnic supports updating firmware using a PLDM image signed and distributed by Meta. PLDM images are written into stored flashed. Flashing does not interrupt operation. On host reboot the newly flashed UEFI driver will be used. To run new control or cmrt firmware the NIC must be power cycled. Signed-off-by: Lee Trager --- .../device_drivers/ethernet/meta/fbnic.rst | 11 + drivers/net/ethernet/meta/Kconfig | 1 + .../net/ethernet/meta/fbnic/fbnic_devlink.c | 270 +++++++++++++++++- 3 files changed, 281 insertions(+), 1 deletion(-) diff --git a/Documentation/networking/device_drivers/ethernet/meta/fbnic.rs= t b/Documentation/networking/device_drivers/ethernet/meta/fbnic.rst index 32ff114f5c26..d6726c254818 100644 --- a/Documentation/networking/device_drivers/ethernet/meta/fbnic.rst +++ b/Documentation/networking/device_drivers/ethernet/meta/fbnic.rst @@ -27,3 +27,14 @@ driver takes over. devlink dev info provides version information for all three components. In addition to the version the hg commit hash of the build is included as a separate entry. + +Upgrading Firmware +------------------ + +fbnic supports upgrading firmware using devlink dev flash. Firmware images +are signed and distributed by Meta. All firmware is bundled into a single +PLDM image which is written into stored flash. Flashing firmware does not +interrupt operation. + +On host reboot the newly flashed UEFI driver will be used. To run new cont= rol +or cmrt firmware the NIC must be power cycled. diff --git a/drivers/net/ethernet/meta/Kconfig b/drivers/net/ethernet/meta/= Kconfig index 85519690b837..f5d2c6b6399f 100644 --- a/drivers/net/ethernet/meta/Kconfig +++ b/drivers/net/ethernet/meta/Kconfig @@ -26,6 +26,7 @@ config FBNIC select NET_DEVLINK select PAGE_POOL select PHYLINK + select PLDMFW help This driver supports Meta Platforms Host Network Interface. diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_devlink.c b/drivers/net/= ethernet/meta/fbnic/fbnic_devlink.c index 0072d612215e..d487ae7f1126 100644 --- a/drivers/net/ethernet/meta/fbnic/fbnic_devlink.c +++ b/drivers/net/ethernet/meta/fbnic/fbnic_devlink.c @@ -3,6 +3,7 @@ #include #include +#include #include #include @@ -109,8 +110,275 @@ static int fbnic_devlink_info_get(struct devlink *dev= link, return 0; } +/** + * fbnic_send_package_data - Send record package data to firmware + * @context: PLDM FW update structure + * @data: pointer to the package data + * @length: length of the package data + * + * Send a copy of the package data associated with the PLDM record matching + * this device to the firmware. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_send_package_data(struct pldmfw *context, const u8 *data, + u16 length) +{ + struct device *dev =3D context->dev; + + /* Temp placeholder required by devlink */ + dev_info(dev, + "Sending %u bytes of PLDM record package data to firmware\n", + length); + + return 0; +} + +/** + * fbnic_send_component_table - Send PLDM component table to the firmware + * @context: PLDM FW update structure + * @component: The component to send + * @transfer_flag: Flag indication location in component tables + * + * Read relevant data from component table and forward it to the firmware. + * Check response to verify if the firmware indicates that it wishes to + * proceed with the update. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_send_component_table(struct pldmfw *context, + struct pldmfw_component *component, + u8 transfer_flag) +{ + struct device *dev =3D context->dev; + u16 id =3D component->identifier; + u8 test_string[80]; + + switch (id) { + case QSPI_SECTION_CMRT: + case QSPI_SECTION_CONTROL_FW: + case QSPI_SECTION_OPTION_ROM: + break; + default: + dev_err(dev, "Unknown component ID %u\n", id); + return -EINVAL; + } + + dev_dbg(dev, "Sending PLDM component table to firmware\n"); + + /* Temp placeholder */ + memcpy(test_string, component->version_string, + min_t(u8, component->version_len, 79)); + test_string[min_t(u8, component->version_len, 79)] =3D 0; + dev_info(dev, "PLDMFW: Component ID: %u version %s\n", + id, test_string); + + return 0; +} + +/** + * fbnic_flash_component - Flash a component of the QSPI + * @context: PLDM FW update structure + * @component: The component table to send to FW + * + * Map contents of component and make it available for FW to download + * so that it can update the contents of the QSPI Flash. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_flash_component(struct pldmfw *context, + struct pldmfw_component *component) +{ + const u8 *data =3D component->component_data; + u32 size =3D component->component_size; + struct fbnic_fw_completion *fw_cmpl; + struct device *dev =3D context->dev; + struct pci_dev *pdev =3D to_pci_dev(dev); + u16 id =3D component->identifier; + const char *component_name; + int retries =3D 2; + int err; + + struct devlink *devlink; + struct fbnic_dev *fbd; + + switch (id) { + case QSPI_SECTION_CMRT: + component_name =3D "boot1"; + break; + case QSPI_SECTION_CONTROL_FW: + component_name =3D "boot2"; + break; + case QSPI_SECTION_OPTION_ROM: + component_name =3D "option-rom"; + break; + default: + dev_err(dev, "Unknown component ID %u\n", id); + return -EINVAL; + } + + fw_cmpl =3D kzalloc(sizeof(*fw_cmpl), GFP_KERNEL); + if (!fw_cmpl) + return -ENOMEM; + + pdev =3D to_pci_dev(dev); + fbd =3D pci_get_drvdata(pdev); + devlink =3D priv_to_devlink(fbd); + + /* Initialize completion and queue it for FW to process */ + fw_cmpl->msg_type =3D FBNIC_TLV_MSG_ID_FW_WRITE_CHUNK_REQ; + init_completion(&fw_cmpl->done); + + fw_cmpl->fw_update.last_offset =3D 0; + fw_cmpl->fw_update.data =3D data; + fw_cmpl->fw_update.size =3D size; + + err =3D fbnic_fw_xmit_fw_start_upgrade(fbd, fw_cmpl, id, size); + if (err) + goto cmpl_free; + + /* Monitor completions and report status of update */ + while (fw_cmpl->fw_update.data) { + u32 offset =3D fw_cmpl->fw_update.last_offset; + + devlink_flash_update_status_notify(devlink, "Flashing", + component_name, offset, + size); + + /* Allow 5 seconds for reply, resend and try up to 2 times */ + if (wait_for_completion_timeout(&fw_cmpl->done, 5 * HZ)) { + reinit_completion(&fw_cmpl->done); + /* If we receive a reply, reinit our retry counter */ + retries =3D 2; + } else if (--retries =3D=3D 0) { + dev_err(fbd->dev, "Timed out waiting on update\n"); + err =3D -ETIMEDOUT; + goto cmpl_cleanup; + } + } + + err =3D fw_cmpl->result; + if (err) + goto cmpl_cleanup; + + devlink_flash_update_status_notify(devlink, "Flashing", + component_name, size, size); + +cmpl_cleanup: + fbd->cmpl_data =3D NULL; +cmpl_free: + kfree(fw_cmpl); + + return err; +} + +/** + * fbnic_finalize_update - Perform last steps to complete device update + * @context: PLDM FW update structure + * + * Notify FW that update is complete and that it can take any actions + * needed to finalize the FW update. + * + * Return: zero on success + * negative error code on failure + */ +static int fbnic_finalize_update(struct pldmfw *context) +{ + struct device *dev =3D context->dev; + + /* Temp placeholder required by devlink */ + dev_info(dev, "PLDMFW: Finalize update\n"); + + return 0; +} + +static const struct pldmfw_ops fbnic_pldmfw_ops =3D { + .match_record =3D pldmfw_op_pci_match_record, + .send_package_data =3D fbnic_send_package_data, + .send_component_table =3D fbnic_send_component_table, + .flash_component =3D fbnic_flash_component, + .finalize_update =3D fbnic_finalize_update, +}; + +static void fbnic_devlink_flash_update_report_err(struct fbnic_dev *fbd, + struct devlink *devlink, + const char *err_msg, + int err) +{ + char err_str[128]; + + snprintf(err_str, sizeof(err_str), + "Failed to flash PLDM Image: %s (error: %d)", + err_msg, err); + devlink_flash_update_status_notify(devlink, err_str, NULL, 0, 0); + dev_err(fbd->dev, "%s\n", err_str); +} + +static int +fbnic_devlink_flash_update(struct devlink *devlink, + struct devlink_flash_update_params *params, + struct netlink_ext_ack *extack) +{ + struct fbnic_dev *fbd =3D devlink_priv(devlink); + const struct firmware *fw =3D params->fw; + struct device *dev =3D fbd->dev; + struct pldmfw context; + char *err_msg; + int err; + + if (!fw || !fw->data || !fw->size) + return -EINVAL; + + devlink_flash_update_status_notify(devlink, "Preparing to flash", + NULL, 0, 0); + + context.ops =3D &fbnic_pldmfw_ops; + context.dev =3D dev; + + err =3D pldmfw_flash_image(&context, fw); + if (err) { + switch (err) { + case -EINVAL: + err_msg =3D "Invalid image"; + break; + case -EOPNOTSUPP: + err_msg =3D "Unsupported image"; + break; + case -ENOMEM: + err_msg =3D "Out of memory"; + break; + case -EFAULT: + err_msg =3D "Invalid header"; + break; + case -ENOENT: + err_msg =3D "No matching record"; + break; + case -ENODEV: + err_msg =3D "No matching device"; + break; + case -ETIMEDOUT: + err_msg =3D "Timed out waiting for reply"; + break; + default: + err_msg =3D "Unknown error"; + break; + } + fbnic_devlink_flash_update_report_err(fbd, devlink, + err_msg, err); + } else { + devlink_flash_update_status_notify(devlink, "Flashing done", + NULL, 0, 0); + } + + return err; +} + static const struct devlink_ops fbnic_devlink_ops =3D { - .info_get =3D fbnic_devlink_info_get, + .info_get =3D fbnic_devlink_info_get, + .flash_update =3D fbnic_devlink_flash_update, }; void fbnic_devlink_free(struct fbnic_dev *fbd) -- 2.43.5