diff mbox series

[V3,11/15] core/pldm: Decode the GetFruRecordTable request

Message ID 20220913102724.65563-12-clombard@linux.vnet.ibm.com
State Superseded
Headers show
Series Complete PLDM responder and enable PLDM support | expand

Commit Message

Christophe Lombard Sept. 13, 2022, 10:27 a.m. UTC
The GetFRURecordTable command is used to get the FRU Record Table data.
This command is defined to allow the FRU Record Table data to be
transferred using a sequence of one or more command/response messages.

Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
---
 core/pldm/pldm-fru-requests.c | 12 ++++++
 core/pldm/pldm-responder.c    | 78 +++++++++++++++++++++++++++++++++++
 core/pldm/pldm.h              |  2 +
 3 files changed, 92 insertions(+)

Comments

Abhishek Singh Tomar Oct. 3, 2022, 5:25 a.m. UTC | #1
On Tue, Sep 13, 2022 at 12:27:20PM +0200, Christophe Lombard wrote:
> The GetFRURecordTable command is used to get the FRU Record Table data.
> This command is defined to allow the FRU Record Table data to be
> transferred using a sequence of one or more command/response messages.
> 
> Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
Reviewed-by: Abhishek Singh Tomar <abhishek@linux.ibm.com>
> ---
>  core/pldm/pldm-fru-requests.c | 12 ++++++
>  core/pldm/pldm-responder.c    | 78 +++++++++++++++++++++++++++++++++++
>  core/pldm/pldm.h              |  2 +
>  3 files changed, 92 insertions(+)
> 
> diff --git a/core/pldm/pldm-fru-requests.c b/core/pldm/pldm-fru-requests.c
> index 80a13ef1..c90f08b3 100644
> --- a/core/pldm/pldm-fru-requests.c
> +++ b/core/pldm/pldm-fru-requests.c
> @@ -228,6 +228,18 @@ void pldm_fru_add_record(uint32_t *table_length,
>  	local_fru_table_length = *table_length;
>  }
> 
> +int pldm_fru_get_local_table(void **fru_record_table_bytes,
> +			     uint32_t *fru_record_table_size)
> +{
> +	if (!local_fru_record_table)
> +		return OPAL_PARAMETER;
> +
> +	*fru_record_table_bytes = local_fru_record_table;
> +	*fru_record_table_size = local_fru_table_length;
> +
> +	return OPAL_SUCCESS;
> +}
> +
>  int pldm_fru_init(void)
>  {
>  	int rc;
> diff --git a/core/pldm/pldm-responder.c b/core/pldm/pldm-responder.c
> index cd2cbc1c..2c5b3677 100644
> --- a/core/pldm/pldm-responder.c
> +++ b/core/pldm/pldm-responder.c
> @@ -995,6 +995,83 @@ static struct pldm_cmd pldm_fru_get_record_table_metadata = {
>  	.handler = fru_get_record_table_metadata_handler,
>  };
> 
> +/*
> + * GetFRURecordTable (0X02)
> + * The GetFRURecordTable command is used to get the FRU Record Table
> + * data. This command is defined to allow the FRU Record Table data to
> + * be transferred using a sequence of one or more command/response
> + * messages.
> + */
> +static int fru_get_record_table_handler(const struct pldm_rx_data *req)
> +{
> +	struct pldm_get_fru_record_table_resp *resp;
> +	void *fru_record_table_bytes;
> +	uint32_t fru_record_table_size;
> +	size_t response_length;
> +	struct pldm_msg *msg;
> +	char *response_msg;
> +	int rc;
> +
> +	/* The getFruRecordTable requests do have request data, but it's
> +	 * only related to multi-part transfers which we don't support
> +	 * and which the BMC will not send us.
> +	 */
> +
> +	/* get local fru record table */
> +	rc = pldm_fru_get_local_table(&fru_record_table_bytes, &fru_record_table_size);
> +	if (rc) {
> +		prlog(PR_ERR, "Failed to get Fru Record Table\n");
> +		cc_resp(req, req->hdrinf.pldm_type,
> +			req->hdrinf.command, PLDM_ERROR);
> +		return OPAL_PARAMETER;
> +	}
> +
> +	/* create a PLDM response message for GetFRURecordTable */
> +	response_length = sizeof(struct pldm_msg_hdr) +
> +			  sizeof(struct pldm_get_fru_record_table_resp) +
> +			  fru_record_table_size;
> +	response_msg = zalloc(response_length);
> +	memset(response_msg, 0, response_length);
> +
> +	rc = encode_get_fru_record_table_resp(
> +				req->hdrinf.instance,
> +				PLDM_SUCCESS,
> +				0, // No next transfer handle
> +				PLDM_START_AND_END,
> +				(struct pldm_msg *)response_msg);
> +	if (rc != PLDM_SUCCESS) {
> +		prlog(PR_ERR, "Encode GetFruRecordTable Error, rc: %d\n", rc);
> +		cc_resp(req, req->hdrinf.pldm_type,
> +			req->hdrinf.command, PLDM_ERROR);
> +		free(response_msg);
> +		return OPAL_PARAMETER;
> +	}
> +
> +	msg = (struct pldm_msg *)response_msg;
> +	resp = (struct pldm_get_fru_record_table_resp *)(msg->payload);
> +	memcpy(resp->fru_record_table_data,
> +	       fru_record_table_bytes,
> +	       fru_record_table_size);
> +
> +	/* send PLDM message over MCTP */
> +	rc = pldm_mctp_message_tx(req->source_eid, response_msg, response_length - 1);
> +	if (rc) {
> +		prlog(PR_ERR, "Failed to send GetFruRecordTable response, rc = %d\n", rc);
> +		free(response_msg);
> +		return OPAL_HARDWARE;
> +	}
> +
> +	free(response_msg);
> +
> +	return OPAL_SUCCESS;
> +}
> +
> +static struct pldm_cmd pldm_fru_get_record_table = {
> +	.name = "PLDM_GET_FRU_RECORD_TABLE",
> +	.pldm_cmd_id = PLDM_GET_FRU_RECORD_TABLE,
> +	.handler = fru_get_record_table_handler,
> +};
> +
>  int pldm_responder_handle_request(struct pldm_rx_data *rx)
>  {
>  	const struct pldm_type *type;
> @@ -1046,6 +1123,7 @@ int pldm_responder_init(void)
>  	/* Register fru commands we'll respond to - DSP0257 */
>  	add_type(&pldm_fru_type);
>  	add_cmd(&pldm_fru_type, &pldm_fru_get_record_table_metadata);
> +	add_cmd(&pldm_fru_type, &pldm_fru_get_record_table);
> 
>  	return OPAL_SUCCESS;
>  }
> diff --git a/core/pldm/pldm.h b/core/pldm/pldm.h
> index fea06b0b..dd6f2352 100644
> --- a/core/pldm/pldm.h
> +++ b/core/pldm/pldm.h
> @@ -78,6 +78,8 @@ int pldm_fru_get_bmc_version(void *bv);
>  void pldm_fru_add_record(uint32_t *fru_table_length,
>  			 uint16_t *total_record_set_identifiers,
>  			 uint16_t *total_table_records);
> +int pldm_fru_get_local_table(void **fru_record_table_bytes,
> +			     uint32_t *fru_record_table_size);
>  int pldm_fru_init(void);
> 
>  int pldm_bios_find_lid_by_attr_name(const char *name, char **lid);
> -- 
> 2.37.3
> 
> _______________________________________________
> Skiboot mailing list
> Skiboot@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/skiboot
diff mbox series

Patch

diff --git a/core/pldm/pldm-fru-requests.c b/core/pldm/pldm-fru-requests.c
index 80a13ef1..c90f08b3 100644
--- a/core/pldm/pldm-fru-requests.c
+++ b/core/pldm/pldm-fru-requests.c
@@ -228,6 +228,18 @@  void pldm_fru_add_record(uint32_t *table_length,
 	local_fru_table_length = *table_length;
 }
 
+int pldm_fru_get_local_table(void **fru_record_table_bytes,
+			     uint32_t *fru_record_table_size)
+{
+	if (!local_fru_record_table)
+		return OPAL_PARAMETER;
+
+	*fru_record_table_bytes = local_fru_record_table;
+	*fru_record_table_size = local_fru_table_length;
+
+	return OPAL_SUCCESS;
+}
+
 int pldm_fru_init(void)
 {
 	int rc;
diff --git a/core/pldm/pldm-responder.c b/core/pldm/pldm-responder.c
index cd2cbc1c..2c5b3677 100644
--- a/core/pldm/pldm-responder.c
+++ b/core/pldm/pldm-responder.c
@@ -995,6 +995,83 @@  static struct pldm_cmd pldm_fru_get_record_table_metadata = {
 	.handler = fru_get_record_table_metadata_handler,
 };
 
+/*
+ * GetFRURecordTable (0X02)
+ * The GetFRURecordTable command is used to get the FRU Record Table
+ * data. This command is defined to allow the FRU Record Table data to
+ * be transferred using a sequence of one or more command/response
+ * messages.
+ */
+static int fru_get_record_table_handler(const struct pldm_rx_data *req)
+{
+	struct pldm_get_fru_record_table_resp *resp;
+	void *fru_record_table_bytes;
+	uint32_t fru_record_table_size;
+	size_t response_length;
+	struct pldm_msg *msg;
+	char *response_msg;
+	int rc;
+
+	/* The getFruRecordTable requests do have request data, but it's
+	 * only related to multi-part transfers which we don't support
+	 * and which the BMC will not send us.
+	 */
+
+	/* get local fru record table */
+	rc = pldm_fru_get_local_table(&fru_record_table_bytes, &fru_record_table_size);
+	if (rc) {
+		prlog(PR_ERR, "Failed to get Fru Record Table\n");
+		cc_resp(req, req->hdrinf.pldm_type,
+			req->hdrinf.command, PLDM_ERROR);
+		return OPAL_PARAMETER;
+	}
+
+	/* create a PLDM response message for GetFRURecordTable */
+	response_length = sizeof(struct pldm_msg_hdr) +
+			  sizeof(struct pldm_get_fru_record_table_resp) +
+			  fru_record_table_size;
+	response_msg = zalloc(response_length);
+	memset(response_msg, 0, response_length);
+
+	rc = encode_get_fru_record_table_resp(
+				req->hdrinf.instance,
+				PLDM_SUCCESS,
+				0, // No next transfer handle
+				PLDM_START_AND_END,
+				(struct pldm_msg *)response_msg);
+	if (rc != PLDM_SUCCESS) {
+		prlog(PR_ERR, "Encode GetFruRecordTable Error, rc: %d\n", rc);
+		cc_resp(req, req->hdrinf.pldm_type,
+			req->hdrinf.command, PLDM_ERROR);
+		free(response_msg);
+		return OPAL_PARAMETER;
+	}
+
+	msg = (struct pldm_msg *)response_msg;
+	resp = (struct pldm_get_fru_record_table_resp *)(msg->payload);
+	memcpy(resp->fru_record_table_data,
+	       fru_record_table_bytes,
+	       fru_record_table_size);
+
+	/* send PLDM message over MCTP */
+	rc = pldm_mctp_message_tx(req->source_eid, response_msg, response_length - 1);
+	if (rc) {
+		prlog(PR_ERR, "Failed to send GetFruRecordTable response, rc = %d\n", rc);
+		free(response_msg);
+		return OPAL_HARDWARE;
+	}
+
+	free(response_msg);
+
+	return OPAL_SUCCESS;
+}
+
+static struct pldm_cmd pldm_fru_get_record_table = {
+	.name = "PLDM_GET_FRU_RECORD_TABLE",
+	.pldm_cmd_id = PLDM_GET_FRU_RECORD_TABLE,
+	.handler = fru_get_record_table_handler,
+};
+
 int pldm_responder_handle_request(struct pldm_rx_data *rx)
 {
 	const struct pldm_type *type;
@@ -1046,6 +1123,7 @@  int pldm_responder_init(void)
 	/* Register fru commands we'll respond to - DSP0257 */
 	add_type(&pldm_fru_type);
 	add_cmd(&pldm_fru_type, &pldm_fru_get_record_table_metadata);
+	add_cmd(&pldm_fru_type, &pldm_fru_get_record_table);
 
 	return OPAL_SUCCESS;
 }
diff --git a/core/pldm/pldm.h b/core/pldm/pldm.h
index fea06b0b..dd6f2352 100644
--- a/core/pldm/pldm.h
+++ b/core/pldm/pldm.h
@@ -78,6 +78,8 @@  int pldm_fru_get_bmc_version(void *bv);
 void pldm_fru_add_record(uint32_t *fru_table_length,
 			 uint16_t *total_record_set_identifiers,
 			 uint16_t *total_table_records);
+int pldm_fru_get_local_table(void **fru_record_table_bytes,
+			     uint32_t *fru_record_table_size);
 int pldm_fru_init(void);
 
 int pldm_bios_find_lid_by_attr_name(const char *name, char **lid);