[Skiboot] [PATCH 2/2] OPAL/FWTS: Reserved memory DT validation tests.
Pridhiviraj Paidipeddi
ppaidipe at linux.vnet.ibm.com
Thu Apr 6 23:34:38 AEST 2017
This patch includes the testcases for validating the
several DT properties for reserved memory.
Properties:
reserved-names
reserved-ranges
Nodes:
reserved-memory/
|sub-regions at unitaddress
|sub-regions at unitaddress
If patch looks good, will send it to fwts upstream.
Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com>
---
src/Makefile.am | 1 +
src/opal/reserv_mem.c | 241 ++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 242 insertions(+)
create mode 100644 src/opal/reserv_mem.c
diff --git a/src/Makefile.am b/src/Makefile.am
index e833554..ffbda0c 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -148,6 +148,7 @@ fwts_SOURCES = main.c \
opal/mtd_info.c \
opal/prd_info.c \
opal/power_mgmt_info.c \
+ opal/reserv_mem.c \
pci/aspm/aspm.c \
pci/crs/crs.c \
pci/maxreadreq/maxreadreq.c \
diff --git a/src/opal/reserv_mem.c b/src/opal/reserv_mem.c
new file mode 100644
index 0000000..7ae1edd
--- /dev/null
+++ b/src/opal/reserv_mem.c
@@ -0,0 +1,241 @@
+/*
+ * Copyright (C) 2010-2017 Canonical
+ * Some of this work - Copyright (C) 2016-2017 IBM
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ *
+ */
+
+#include <fcntl.h>
+#include <stdlib.h>
+#include <sys/ioctl.h>
+
+#include "fwts.h"
+
+#ifdef HAVE_LIBFDT
+#include <libfdt.h>
+#endif
+
+#define SLW_IMAGE_SIZE 0x100000UL /* 1MB */
+#define OCC_COMMON_SIZE 0x800000UL /* 8MB */
+#define HOMER_IMAGE_SIZE 0x400000UL /* 4MB */
+
+static const char *root_node = "/";
+
+static const char *reserv_mem_node = "/reserved-memory/";
+
+struct reserv_range {
+ uint64_t start;
+ uint64_t len;
+};
+
+struct reserv_region {
+ const char *name;
+ uint64_t start;
+ uint64_t len;
+};
+
+struct reserv_region *regions;
+
+static char *make_message(const char *fmt, ...) {
+ char *p = NULL;
+ size_t size = 128;
+ va_list ap;
+
+ if((p = malloc(size)) == NULL)
+ return NULL;
+
+ va_start(ap, fmt);
+ vsnprintf(p, size, fmt, ap);
+ va_end(ap);
+ return p;
+}
+
+static int reserv_mem_init(fwts_framework *fw)
+{
+ if (fwts_firmware_detect() != FWTS_FIRMWARE_OPAL) {
+ fwts_skipped(fw,
+ "The firmware type detected was non OPAL"
+ "so skipping the OPAL Reserve Memory DT checks.");
+ return FWTS_SKIP;
+ }
+
+ /* On an OPAL based system Device tree should be present */
+ if (!fw->fdt) {
+ fwts_failed(fw, LOG_LEVEL_HIGH, "NoDeviceTree",
+ "Device tree not found");
+ return FWTS_ERROR;
+ }
+
+ return FWTS_OK;
+}
+
+static int reserv_mem_limits_test(fwts_framework *fw)
+{
+ bool ok = true;
+ char *region_names;
+ const struct reserv_range *reserv_range;
+ int offset, len, nr_regions;
+
+ offset = fdt_path_offset(fw->fdt, root_node);
+ if (offset < 0) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "DTNodeMissing",
+ "DT root node %s is missing", root_node);
+ return FWTS_ERROR;
+ }
+
+ /* Get the number of memory reserved regions */
+ nr_regions = fwts_dt_stringlist_count(fw, fw->fdt, offset, "reserved-names");
+
+ /* Check for the reservd-names property */
+ region_names = (char *)fdt_getprop(fw->fdt, offset, "reserved-names", &len);
+ if (!region_names) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "DTPropertyMissing",
+ "DT Property reserved-names is missing %s", fdt_strerror(len));
+ return FWTS_ERROR;
+ }
+
+ regions = malloc(nr_regions*sizeof(struct reserv_region));
+ if(!regions) {
+ fwts_skipped(fw, "Unable to allocate memory for reserv_region structure");
+ return FWTS_SKIP;
+ }
+
+ for(int i = 0; i < nr_regions; i++) {
+ regions[i].name = strdup(region_names);
+ region_names += strlen(regions[i].name)+1;
+ }
+
+ /* Check for the reserved-ranges property */
+ reserv_range = fdt_getprop(fw->fdt, offset, "reserved-ranges", &len);
+ if (reserv_range == NULL) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "DTPropertyMissing",
+ "DT Property reserved-ranges is missing %s", fdt_strerror(len));
+ return FWTS_ERROR;
+ }
+
+ for(int j=0; j < nr_regions; j++) {
+ regions[j].start = (uint64_t )be64toh(reserv_range[j].start);
+ regions[j].len = (uint64_t )be64toh(reserv_range[j].len);
+ fwts_log_info(fw,"region name %s ,start: 0x%08lx, len: 0x%08lx\n",
+ regions[j].name, regions[j].start, regions[j].len);
+ }
+
+ offset = fdt_path_offset(fw->fdt, reserv_mem_node);
+ if (offset < 0) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "DTNodeMissing",
+ "reserve memory node %s is missing", reserv_mem_node);
+ return FWTS_ERROR;
+ }
+
+ /* Validate different cases */
+ for(int j=0; j < nr_regions; j++) {
+
+ char *buf=NULL;
+
+ /* Check for zero offset's */
+ if (regions[j].start == 0) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "ZeroStartAddress",
+ "memory region got zero start address");
+ ok = false;
+ }
+
+ /* Check for zero region sizes */
+ if (regions[j].len == 0) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "ZeroRegionSize",
+ "memory region got zero size");
+ ok = false;
+ }
+
+ /* Check corresponding memory region got reserved by looking
+ reserved-memory node entries */
+
+ /* Form the reserved-memory sub nodes for all the regions*/
+ if (!strstr(regions[j].name, "@"))
+ buf = make_message("%s%s@%lx", reserv_mem_node, regions[j].name,
+ regions[j].start);
+ else
+ buf = make_message("/%s/%s", reserv_mem_node, regions[j].name);
+
+ if(buf ==NULL) {
+ fwts_skipped(fw, "Unable to allocate memory for message buffer");
+ return FWTS_SKIP;
+ }
+
+ offset = fdt_path_offset(fw->fdt, buf);
+ if (offset < 0) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "DTNodeMissing",
+ "reserve memory region node %s is missing", buf);
+ ok = false;
+ }
+
+ /* Validate different Known image fixed sizes here */
+ if (strstr(regions[j].name, "homer-image")) {
+ if(regions[j].len != HOMER_IMAGE_SIZE) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "ImageSizeMismatch",
+ "Mismatch in homer-image size, expected: 0x%lx, actual: 0x%lx",
+ HOMER_IMAGE_SIZE, regions[j].len);
+ ok = false;
+ }
+ else
+ fwts_log_info(fw, "homer-image size is validated");
+ }
+
+ if (strstr(regions[j].name, "slw-image")) {
+ if(regions[j].len != SLW_IMAGE_SIZE) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "ImageSizeMismatch",
+ "Mismatch in slw-image size, expected: 0x%lx, actual: 0x%lx",
+ SLW_IMAGE_SIZE, regions[j].len);
+ ok = false;
+ }
+ else
+ fwts_log_info(fw, "slw-image size is validated");
+ }
+
+ if (strstr(regions[j].name, "occ-common-area")) {
+ if(regions[j].len != OCC_COMMON_SIZE) {
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "ImageSizeMismatch",
+ "Mismatch in occ-common-area size, expected: 0x%lx, actual: 0x%lx",
+ OCC_COMMON_SIZE, regions[j].len);
+ ok = false;
+ }
+ else
+ fwts_log_info(fw, "occ-common-area size is validated");
+ }
+ }
+
+ if (ok)
+ fwts_passed(fw, "Reserved memory validation tests passed");
+ else
+ fwts_failed(fw, LOG_LEVEL_HIGH, "ReservMemTestFail",
+ "One or few Reserved Memory DT validation tests failed");
+ free(regions);
+ return FWTS_OK;
+}
+
+static fwts_framework_minor_test reserv_mem_tests[] = {
+ { reserv_mem_limits_test, "OPAL Reserved memory DT Validation Info" },
+ { NULL, NULL }
+};
+
+static fwts_framework_ops reserv_mem_tests_ops = {
+ .description = "OPAL Reserved memory DT Validation Test",
+ .init = reserv_mem_init,
+ .minor_tests = reserv_mem_tests
+};
+
+FWTS_REGISTER_FEATURES("reserv_mem", &reserv_mem_tests_ops, FWTS_TEST_EARLY,
+ FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV,
+ FWTS_FW_FEATURE_DEVICETREE)
--
2.7.4
More information about the Skiboot
mailing list