diff options
| author | Ilya Smirnov <ismirno@us.ibm.com> | 2018-11-08 12:56:45 -0600 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2019-01-04 14:18:46 -0600 |
| commit | 36bd1dae218877ae8124a0ce8a9420c04e20c1d0 (patch) | |
| tree | f08fd193013bb0da0b34233442af05a6bee3e0e9 /src/usr/nvram | |
| parent | 24f66ecb7882be36eba5ef13cbc414acb642c2af (diff) | |
| download | talos-hostboot-36bd1dae218877ae8124a0ce8a9420c04e20c1d0.tar.gz talos-hostboot-36bd1dae218877ae8124a0ce8a9420c04e20c1d0.zip | |
Unit Tests For NVRAM Checking/Reading
This change introduces the unit tests for testing the ported
NVRAM checking and reading logic. nvram_format was also ported
to be able to format the test NVRAM "partition" to then run
the tests on it.
Change-Id: Idca01dc84d43caf0e32ba556b6407c78d66f053a
RTC: 192411
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68612
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Roland Veloz <rveloz@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/nvram')
| -rw-r--r-- | src/usr/nvram/import/nvram-format.C | 79 | ||||
| -rw-r--r-- | src/usr/nvram/makefile | 2 | ||||
| -rw-r--r-- | src/usr/nvram/test/makefile | 32 | ||||
| -rw-r--r-- | src/usr/nvram/test/testnvram.H | 100 |
4 files changed, 206 insertions, 7 deletions
diff --git a/src/usr/nvram/import/nvram-format.C b/src/usr/nvram/import/nvram-format.C index d57f9ab51..072fb8f4d 100644 --- a/src/usr/nvram/import/nvram-format.C +++ b/src/usr/nvram/import/nvram-format.C @@ -25,13 +25,6 @@ #include <skiboot.h> #include <nvram.h> -struct chrp_nvram_hdr { - uint8_t sig; - uint8_t cksum; - be16 len; - char name[12]; -}; - static struct chrp_nvram_hdr *skiboot_part_hdr; static uint8_t chrp_nv_cksum(struct chrp_nvram_hdr *hdr) @@ -54,10 +47,82 @@ static uint8_t chrp_nv_cksum(struct chrp_nvram_hdr *hdr) #define NVRAM_SIG_FW_PRIV 0x51 #define NVRAM_SIG_SYSTEM 0x70 +#define NVRAM_SIG_FREE 0x7f #define NVRAM_NAME_COMMON "common" #define NVRAM_NAME_FW_PRIV "ibm,skiboot" +#define NVRAM_NAME_FREE "wwwwwwwwwwww" +#define NVRAM_SIZE_COMMON 0x10000 #define NVRAM_SIZE_FW_PRIV 0x1000 +int nvram_format(void *nvram_image, uint32_t nvram_size) +{ + struct chrp_nvram_hdr *h; + unsigned int offset = 0; + + prerror("NVRAM: Re-initializing (size: 0x%08x)\n", nvram_size); + memset(nvram_image, 0, nvram_size); + + /* Create private partition */ + if (nvram_size - offset < NVRAM_SIZE_FW_PRIV) + return -1; + h = +#ifdef __HOSTBOOT_MODULE + reinterpret_cast<chrp_nvram_hdr*>( + static_cast<uint8_t*>(nvram_image) + offset); +#else + nvram_image + offset; +#endif + h->sig = NVRAM_SIG_FW_PRIV; + h->len = cpu_to_be16(NVRAM_SIZE_FW_PRIV >> 4); + strcpy(h->name, NVRAM_NAME_FW_PRIV); + h->cksum = chrp_nv_cksum(h); + prlog(PR_DEBUG, "NVRAM: Created '%s' partition at 0x%08x" + " for size 0x%08x with cksum 0x%02x\n", + NVRAM_NAME_FW_PRIV, offset, + be16_to_cpu(h->len), h->cksum); + offset += NVRAM_SIZE_FW_PRIV; + + /* Create common partition */ + if (nvram_size - offset < NVRAM_SIZE_COMMON) + return -1; + h = +#ifdef __HOSTBOOT_MODULE + reinterpret_cast<chrp_nvram_hdr*>( + static_cast<uint8_t*>(nvram_image) + offset); +#else + nvram_image + offset; +#endif + h->sig = NVRAM_SIG_SYSTEM; + h->len = cpu_to_be16(NVRAM_SIZE_COMMON >> 4); + strcpy(h->name, NVRAM_NAME_COMMON); + h->cksum = chrp_nv_cksum(h); + prlog(PR_DEBUG, "NVRAM: Created '%s' partition at 0x%08x" + " for size 0x%08x with cksum 0x%02x\n", + NVRAM_NAME_COMMON, offset, + be16_to_cpu(h->len), h->cksum); + offset += NVRAM_SIZE_COMMON; + + /* Create free space partition */ + if (nvram_size - offset < sizeof(struct chrp_nvram_hdr)) + return -1; + h = +#ifdef __HOSTBOOT_MODULE + reinterpret_cast<chrp_nvram_hdr*>( + static_cast<uint8_t*>(nvram_image) + offset); +#else + nvram_image + offset; +#endif + h->sig = NVRAM_SIG_FREE; + h->len = cpu_to_be16((nvram_size - offset) >> 4); + /* We have the full 12 bytes here */ + memcpy(h->name, NVRAM_NAME_FREE, 12); + h->cksum = chrp_nv_cksum(h); + prlog(PR_DEBUG, "NVRAM: Created '%s' partition at 0x%08x" + " for size 0x%08x with cksum 0x%02x\n", + NVRAM_NAME_FREE, offset, be16_to_cpu(h->len), h->cksum); + return 0; +} + static const char *find_next_key(const char *start, const char *end) { /* diff --git a/src/usr/nvram/makefile b/src/usr/nvram/makefile index 672e85529..74fea325f 100644 --- a/src/usr/nvram/makefile +++ b/src/usr/nvram/makefile @@ -34,4 +34,6 @@ OBJS += nvram-format.o VPATH += import +SUBDIRS += test.d + include ${ROOTPATH}/config.mk diff --git a/src/usr/nvram/test/makefile b/src/usr/nvram/test/makefile new file mode 100644 index 000000000..8b951863e --- /dev/null +++ b/src/usr/nvram/test/makefile @@ -0,0 +1,32 @@ +# IBM_PROLOG_BEGIN_TAG +# This is an automatically generated prolog. +# +# $Source: src/usr/nvram/test/makefile $ +# +# OpenPOWER HostBoot Project +# +# Contributors Listed Below - COPYRIGHT 2018 +# [+] International Business Machines Corp. +# +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or +# implied. See the License for the specific language governing +# permissions and limitations under the License. +# +# IBM_PROLOG_END_TAG +ROOTPATH = ../../../.. + +MODULE = testnvram + +TESTS = testnvram.H + +include ${ROOTPATH}/config.mk + diff --git a/src/usr/nvram/test/testnvram.H b/src/usr/nvram/test/testnvram.H new file mode 100644 index 000000000..88016ce95 --- /dev/null +++ b/src/usr/nvram/test/testnvram.H @@ -0,0 +1,100 @@ +/* IBM_PROLOG_BEGIN_TAG */ +/* This is an automatically generated prolog. */ +/* */ +/* $Source: src/usr/nvram/test/testnvram.H $ */ +/* */ +/* OpenPOWER HostBoot Project */ +/* */ +/* Contributors Listed Below - COPYRIGHT 2018 */ +/* [+] International Business Machines Corp. */ +/* */ +/* */ +/* Licensed under the Apache License, Version 2.0 (the "License"); */ +/* you may not use this file except in compliance with the License. */ +/* You may obtain a copy of the License at */ +/* */ +/* http://www.apache.org/licenses/LICENSE-2.0 */ +/* */ +/* Unless required by applicable law or agreed to in writing, software */ +/* distributed under the License is distributed on an "AS IS" BASIS, */ +/* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or */ +/* implied. See the License for the specific language governing */ +/* permissions and limitations under the License. */ +/* */ +/* IBM_PROLOG_END_TAG */ +#ifndef __TEST_NVRAM_H +#define __TEST_NVRAM_H + +#include <cxxtest/TestSuite.H> +#include <nvram/import/nvram.h> +#include <stdlib.h> +#include <string.h> + +class NvramTestSuite : public CxxTest::TestSuite +{ +public: + + void testNvram() const + { + TS_INFO(ENTER_MRK"testNvram"); + + const uint32_t NVRAM_SIZE = 0x90000; + uint8_t* l_nvramImg = static_cast<uint8_t*>(malloc(NVRAM_SIZE)); + + // Unformatted NVRAM - the check should fail + int l_rc = nvram_check(l_nvramImg, NVRAM_SIZE); + if(l_rc == 0) + { + TS_FAIL("testNvram: nvram_check returned clean on unformatted NVRAM"); + } + + // Format the memory area to look like NVRAM from skiboot perspective + l_rc = nvram_format(l_nvramImg, NVRAM_SIZE); + if(l_rc) + { + TS_FAIL("testNvram: could not nvram_format; rc = %d", l_rc); + } + + // Now check it + l_rc = nvram_check(l_nvramImg, NVRAM_SIZE); + if(l_rc) + { + TS_FAIL("testNvram: nvram_check failed; rc = %d", l_rc); + } + + char l_test_kv[] = "test=val"; + + // Insert the test K/V right after the header + memcpy(l_nvramImg + sizeof(chrp_nvram_hdr), l_test_kv, + sizeof(l_test_kv)/sizeof(l_test_kv[0])); + + const char* l_key = "test"; + const char* l_expectedVal = "val"; + const char* l_val = nvram_query(l_key); + if(!l_val) + { + TS_FAIL("testNvram: test key not found!"); + } + else + { + if(strcmp(l_val, l_expectedVal)) + { + TS_FAIL("testNvram: unexpected test value returned. Expected \"%s\", actual \"%s\"", l_expectedVal, l_val); + } + } + + const char* l_notExistingKey = "key_dne"; + l_val = nvram_query(l_notExistingKey); + if(l_val) + { + TS_FAIL("testNvram: non-existing key found!"); + } + + free(l_nvramImg); + l_nvramImg = nullptr; + + TS_INFO(EXIT_MRK"testNvram"); + } +}; + +#endif |

