[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