[Skiboot-stable] [PATCH v2 5/5] test-ipmi-hiomap: Add read-one-byte test

Vasant Hegde hegdevasant at linux.vnet.ibm.com
Mon Apr 8 16:05:41 AEST 2019


Add test case to read:
  - 1 byte
  - 1 block and 1 byte data

Cc: Andrew Jeffery <andrew at aj.id.au>
Cc: skiboot-stable at lists.ozlabs.org
Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
---
 libflash/test/test-ipmi-hiomap.c | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/libflash/test/test-ipmi-hiomap.c b/libflash/test/test-ipmi-hiomap.c
index abc97779c..e1bc5ef38 100644
--- a/libflash/test/test-ipmi-hiomap.c
+++ b/libflash/test/test-ipmi-hiomap.c
@@ -874,6 +874,24 @@ static void test_hiomap_protocol_read_one_block(void)
 	scenario_exit();
 }
 
+static void test_hiomap_protocol_read_one_byte(void)
+{
+	struct blocklevel_device *bl;
+	uint8_t *buf;
+	size_t len;
+
+	scenario_enter(scenario_hiomap_protocol_read_one_block);
+	assert(!ipmi_hiomap_init(&bl));
+	len = 1;
+	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, },
@@ -929,6 +947,26 @@ static void test_hiomap_protocol_read_two_blocks(void)
 	scenario_exit();
 }
 
+static void test_hiomap_protocol_read_1block_1byte(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 = (1 << ctx->block_size_shift) + 1;
+	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_one_block_twice[] = {
 	{ .type = scenario_event_p, .p = &hiomap_ack_call, },
@@ -3112,7 +3150,9 @@ 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_one_byte),
 	TEST_CASE(test_hiomap_protocol_read_two_blocks),
+	TEST_CASE(test_hiomap_protocol_read_1block_1byte),
 	TEST_CASE(test_hiomap_protocol_read_one_block_twice),
 	TEST_CASE(test_hiomap_protocol_event_before_read),
 	TEST_CASE(test_hiomap_protocol_event_during_read),
-- 
2.14.3



More information about the Skiboot-stable mailing list