diff options
author | Michael Neuling <mikey@neuling.org> | 2019-05-13 17:09:39 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.ibm.com> | 2019-05-15 15:43:19 +1000 |
commit | 5beda3c6fe5b72aac95b4c13746ae598dfd64c01 (patch) | |
tree | 9f8db43611af7bf743c4a70c6843fb5e614be400 /hw/xscom.c | |
parent | c8b5e8a95caf029ffe73ea18769fdd7f2da48ab4 (diff) | |
download | skiboot-5beda3c6fe5b72aac95b4c13746ae598dfd64c01.zip skiboot-5beda3c6fe5b72aac95b4c13746ae598dfd64c01.tar.gz skiboot-5beda3c6fe5b72aac95b4c13746ae598dfd64c01.tar.bz2 |
nvram: Flag dangerous NVRAM options
Most nvram options used by skiboot are just for debug or testing for
regressions. They should never be used long term.
We've hit a number of issues in testing and the field where nvram
options have been set "temporarily" but haven't been properly cleared
after, resulting in crashes or real bugs being masked.
This patch marks most nvram options used by skiboot as dangerous and
prints a chicken to remind users of the problem.
Signed-off-by: Michael Neuling <mikey@neuling.org>
Reviewed-by: Samuel Mendoza-Jonas <sam@mendozajonas.com>
Acked-By: Alistair Popple <alistair@popple.id.au>
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'hw/xscom.c')
-rw-r--r-- | hw/xscom.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -833,7 +833,7 @@ int64_t xscom_trigger_xstop(void) int rc = OPAL_UNSUPPORTED; bool xstop_disabled = false; - if (nvram_query_eq("opal-sw-xstop", "disable")) + if (nvram_query_eq_dangerous("opal-sw-xstop", "disable")) xstop_disabled = true; if (xstop_disabled) { |