[Skiboot] [PATCH v2 9/9] core/pldm/test : pldm FRU implementation to retrieve the bmc information

Abhishek Singh Tomar abhishek at linux.ibm.com
Sat May 7 16:36:02 AEST 2022


This patch do self test to retrive BMC version information

Signed-off-by: Abhishek Singh Tomar <abhishek at linux.ibm.com>
---
 core/pldm/test/test_pldm-fru.c | 118 ++++++++++++++++++++++++++++++++-
 1 file changed, 115 insertions(+), 3 deletions(-)

diff --git a/core/pldm/test/test_pldm-fru.c b/core/pldm/test/test_pldm-fru.c
index 049dbce8..53232399 100644
--- a/core/pldm/test/test_pldm-fru.c
+++ b/core/pldm/test/test_pldm-fru.c
@@ -124,6 +124,56 @@ int pldm_test_reply_request(void *request_msg, size_t request_len,
 }
 
 
+/*
+ * This function tries to duplicate BMC functionality for Pldm self test
+ * It generate reply for PLDM_GET_FRU_RECORD_BY_OPTION for self test.
+ * It generates fru Dta structure with BMC version tlv
+ */
+int pldm_test_generate_reply_field_type_version(uint8_t record_type, uint8_t field_type,
+		uint8_t **fru_table, size_t *fru_table_size)
+{
+	struct pldm_fru_record_tlv *tlv;
+	int tlv_size, ret;
+	size_t curr_size = 0;
+	size_t	pad_bytes = 0;
+	uint32_t checksum;
+
+	if (field_type == PLDM_FRU_FIELD_TYPE_VERSION &&
+			record_type == PLDM_FRU_RECORD_TYPE_GENERAL) {
+
+		tlv_size = sizeof(struct pldm_fru_record_tlv) + strlen(TEST_BMC_VERSION) - 1;
+		tlv = malloc(tlv_size);
+		tlv->type = PLDM_FRU_FIELD_TYPE_VERSION;
+		tlv->length = strlen(TEST_BMC_VERSION);
+		memcpy(tlv->value, TEST_BMC_VERSION, tlv->length);
+		*fru_table_size = sizeof(struct pldm_fru_record_data_format) + tlv->length - 1;
+
+		/* Culculate pad bytes in fru */
+		if (*fru_table_size % 4)
+			pad_bytes = 4 - (*fru_table_size % 4);
+		else
+			pad_bytes = 0;
+
+		*fru_table = malloc(*fru_table_size + pad_bytes + sizeof(uint32_t));
+		ret = encode_fru_record(*fru_table, *fru_table_size, &curr_size, 0,
+				PLDM_FRU_RECORD_TYPE_GENERAL, 1, 1, (uint8_t *)tlv, tlv_size);
+		if (ret != PLDM_SUCCESS)
+			return ret;
+		*fru_table_size += pad_bytes + sizeof(uint32_t);
+
+		/* Pad with 0 */
+		memset(*fru_table + curr_size, 0, pad_bytes);
+
+		checksum = htole32(pldm_crc32(*fru_table, *fru_table_size - sizeof(uint32_t)));
+		memcpy(*fru_table + curr_size + pad_bytes, (void *)&checksum, sizeof(uint32_t));
+		free(tlv);
+	} else
+		return OPAL_PARAMETER;
+
+	return PLDM_SUCCESS;
+}
+
+
 /*
  * This function tries to duplicate BMC functionality for Pldm self test
  * it tries to handle PLDM_REQUEST for PLDM_FRU and reply with appropriate
@@ -132,12 +182,60 @@ int pldm_test_reply_request(void *request_msg, size_t request_len,
 int pldm_test_reply_request_fru(void *request_msg, size_t request_len,
 		void **response_msg, size_t *response_len)
 {
+	int  ret, payload_len = 0;
+	uint32_t transfer_handle;
+	uint16_t fru_table_handle;
+	uint16_t record_set_identifier;
+	uint8_t record_type;
+	uint8_t field_type;
+	uint8_t transfer_op_flag;
+	uint8_t *fru_ds;
+	size_t fru_ds_size;
+
+	/* Check PLDM command and reply with appropriate reply */
 	switch (((struct pldm_msg *)request_msg)->hdr.command) {
 	case PLDM_GET_FRU_RECORD_BY_OPTION:
-		(void)request_len;
-		*response_msg = NULL;
-		*response_len = 0;
+		payload_len = request_len - sizeof(struct pldm_msg_hdr);
+		ret = decode_get_fru_record_by_option_req(request_msg, payload_len,
+				&transfer_handle, &fru_table_handle, &record_set_identifier,
+				&record_type, &field_type, &transfer_op_flag);
+		if (ret != PLDM_SUCCESS)
+			return ret;
 
+		/*
+		 * Test if field type and record type is as expected i.e.
+		 * field type = PLDM_FRU_FIELD_TYPE_VERSION and
+		 * record_type == PLDM_FRU_RECORD_TYPE_GENERAL
+		 * else return error
+		 */
+		if (field_type == PLDM_FRU_FIELD_TYPE_VERSION &&
+				record_type == PLDM_FRU_RECORD_TYPE_GENERAL) {
+
+			/*
+			 * generate the fru data structure to reply request
+			 * on behalf on BMC for PLDM self test
+			 */
+			ret = pldm_test_generate_reply_field_type_version(
+					PLDM_FRU_RECORD_TYPE_GENERAL,
+					PLDM_FRU_FIELD_TYPE_VERSION, &fru_ds, &fru_ds_size);
+			if (ret != PLDM_SUCCESS)
+				return ret;
+
+			payload_len = fru_ds_size +
+				sizeof(struct pldm_get_fru_record_by_option_resp) - 1;
+			*response_len = sizeof(struct pldm_msg_hdr) + payload_len;
+			*response_msg = malloc(*response_len);
+
+			ret = encode_get_fru_record_by_option_resp(
+					((struct pldm_msg *)request_msg)->hdr.instance_id,
+					PLDM_SUCCESS, PLDM_GET_NEXTPART, PLDM_START_AND_END,
+					fru_ds, fru_ds_size, *response_msg, payload_len);
+			if (ret != PLDM_SUCCESS)
+				return ret;
+			free(fru_ds);
+			return PLDM_SUCCESS;
+		} else
+			return OPAL_PARAMETER;
 		break;
 	default:
 		return PLDM_ERROR_INVALID_DATA;
@@ -199,6 +297,7 @@ int main(void)
 	int ret;
 	void *fru_record_table;
 	uint32_t fru_record_table_size;
+	struct variable_field fru_structure_data;
 
 	/*
 	 * Trying to get fru table when fru table not created
@@ -220,5 +319,18 @@ int main(void)
 	if (ret != PLDM_SUCCESS)
 		return OPAL_PARAMETER;
 
+	/* retrieve the bmc information with
+	 * "FRU Field Type": Version
+	 * "FRU Record Set Identifier": 1,
+	 * "FRU Record Type": "General(1)"
+	 */
+	ret = pldm_fru_get_record_by_option(0, 1, PLDM_FRU_RECORD_TYPE_GENERAL,
+			PLDM_FRU_FIELD_TYPE_VERSION, &fru_structure_data);
+	if (ret != PLDM_SUCCESS) {
+		perror("encode_get_fru_record_table_metadata_req");
+		return ret;
+	}
+
+
 }
 
-- 
2.34.1



More information about the Skiboot mailing list