diff options
author | Andrew Jeffery <andrew@aj.id.au> | 2019-02-21 16:58:16 +1030 |
---|---|---|
committer | Stewart Smith <stewart@linux.ibm.com> | 2019-02-24 17:43:37 -0600 |
commit | d4b870c8524c3abe9d5daf506d1ca30a07d70912 (patch) | |
tree | e24daeb9f84a6c7bac60f9b433190111a0a2b513 | |
parent | 5b12dfe8dd0d3dd62383796e6588bfa28c6b3a6d (diff) | |
download | skiboot-d4b870c8524c3abe9d5daf506d1ca30a07d70912.zip skiboot-d4b870c8524c3abe9d5daf506d1ca30a07d70912.tar.gz skiboot-d4b870c8524c3abe9d5daf506d1ca30a07d70912.tar.bz2 |
test-ipmi-hiomap: Add read-two-blocks test
Cc: stable
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
-rw-r--r-- | libflash/test/test-ipmi-hiomap.c | 105 |
1 files changed, 98 insertions, 7 deletions
diff --git a/libflash/test/test-ipmi-hiomap.c b/libflash/test/test-ipmi-hiomap.c index f9c8ada..4df3641 100644 --- a/libflash/test/test-ipmi-hiomap.c +++ b/libflash/test/test-ipmi-hiomap.c @@ -215,6 +215,20 @@ int64_t lpc_read(enum OpalLPCAddressType addr_type __attribute__((unused)), return 0; } +static bool lpc_read_success(const uint8_t *buf, size_t len) +{ + if (len < 64) + while (len--) + if (*buf++ != 0xaa) + return false; + + for (int i = 0; i < 64; i++) + if (buf[i] != 0xaa) + return false; + + return !memcmp(buf, buf + 64, len - 64); +} + /* Commonly used messages */ static const struct scenario_event hiomap_ack_call = { @@ -304,6 +318,31 @@ hiomap_create_read_window_qs0l1_rs0l1_call = { }, }; +static const struct scenario_event +hiomap_create_read_window_qs0l2_rs0l1_call = { + .type = scenario_cmd, + .c = { + .req = { + .cmd = HIOMAP_C_CREATE_READ_WINDOW, + .seq = 4, + .args = { + [0] = 0x00, [1] = 0x00, + [2] = 0x02, [3] = 0x00, + }, + }, + .cc = IPMI_CC_NO_ERROR, + .resp = { + .cmd = HIOMAP_C_CREATE_READ_WINDOW, + .seq = 4, + .args = { + [0] = 0xff, [1] = 0x0f, + [2] = 0x01, [3] = 0x00, + [4] = 0x00, [5] = 0x00, + }, + }, + }, +}; + static const struct scenario_event scenario_hiomap_init[] = { { .type = scenario_event_p, .p = &hiomap_ack_call, }, { .type = scenario_event_p, .p = &hiomap_get_info_call, }, @@ -686,21 +725,72 @@ static void test_hiomap_protocol_read_one_block(void) { struct blocklevel_device *bl; struct ipmi_hiomap *ctx; + uint8_t *buf; size_t len; - unsigned char *buf; - int i; scenario_enter(scenario_hiomap_protocol_read_one_block); assert(!ipmi_hiomap_init(&bl)); ctx = container_of(bl, struct ipmi_hiomap, bl); len = 1 << ctx->block_size_shift; - buf = malloc(len); + buf = calloc(1, len); + assert(buf); + assert(!bl->read(bl, 0, buf, len)); + assert(lpc_read_success(buf, len)); + free(buf); + ipmi_hiomap_exit(bl); + scenario_exit(); +} + +static const struct scenario_event +scenario_hiomap_protocol_read_two_blocks[] = { + { .type = scenario_event_p, .p = &hiomap_ack_call, }, + { .type = scenario_event_p, .p = &hiomap_get_info_call, }, + { .type = scenario_event_p, .p = &hiomap_get_flash_info_call, }, + { + .type = scenario_event_p, + .p = &hiomap_create_read_window_qs0l2_rs0l1_call, + }, + { + .type = scenario_cmd, + .c = { + .req = { + .cmd = HIOMAP_C_CREATE_READ_WINDOW, + .seq = 5, + .args = { + [0] = 0x01, [1] = 0x00, + [2] = 0x01, [3] = 0x00, + }, + }, + .cc = IPMI_CC_NO_ERROR, + .resp = { + .cmd = HIOMAP_C_CREATE_READ_WINDOW, + .seq = 5, + .args = { + [0] = 0xfe, [1] = 0x0f, + [2] = 0x01, [3] = 0x00, + [4] = 0x01, [5] = 0x00, + }, + }, + }, + }, + SCENARIO_SENTINEL, +}; + +static void test_hiomap_protocol_read_two_blocks(void) +{ + struct blocklevel_device *bl; + struct ipmi_hiomap *ctx; + uint8_t *buf; + size_t len; + + scenario_enter(scenario_hiomap_protocol_read_two_blocks); + assert(!ipmi_hiomap_init(&bl)); + ctx = container_of(bl, struct ipmi_hiomap, bl); + len = 2 * (1 << ctx->block_size_shift); + buf = calloc(1, len); assert(buf); assert(!bl->read(bl, 0, buf, len)); - assert(len > 64); - for (i = 0; i < 64; i++) - assert(buf[i] == 0xaa); - assert(!memcmp(buf, buf + 64, len - 64)); + assert(lpc_read_success(buf, len)); free(buf); ipmi_hiomap_exit(bl); scenario_exit(); @@ -750,6 +840,7 @@ struct test_case test_cases[] = { TEST_CASE(test_hiomap_event_daemon_regained_flash_control_dirty), TEST_CASE(test_hiomap_protocol_reset_recovery), TEST_CASE(test_hiomap_protocol_read_one_block), + TEST_CASE(test_hiomap_protocol_read_two_blocks), TEST_CASE(test_hiomap_protocol_persistent_error), { NULL, NULL }, }; |