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/npu2-common.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/npu2-common.c')
| -rw-r--r-- | hw/npu2-common.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/npu2-common.c b/hw/npu2-common.c index 16b285d..f3f2f45 100644 --- a/hw/npu2-common.c +++ b/hw/npu2-common.c @@ -663,7 +663,7 @@ void probe_npu2(void) } /* Check for a zcal override */ - zcal = nvram_query("nv_zcal_override"); + zcal = nvram_query_dangerous("nv_zcal_override"); if (zcal) { nv_zcal_nominal = atoi(zcal); prlog(PR_WARNING, "NPU2: Using ZCAL impedance override = %d\n", nv_zcal_nominal); |