| Message ID | 20181024000730.24162-1-joel@jms.id.au |
|---|---|
| State | Accepted |
| Headers | show |
| Series | cpu: Quieten OS endian switch messages | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/apply_patch | success | master/apply_patch Successfully applied |
| snowpatch_ozlabs/make_check | success | Test make_check on branch master |
Joel Stanley <joel@jms.id.au> writes: > Users see these when loading an OS from Petitboot: > > [ 119.486794100,5] OPAL: Switch to big-endian OS > [ 120.022302604,5] OPAL: Switch to little-endian OS > > Which is expected and doesn't provide any information the user can act > on. Switch them to PR_INFO so they still appear in the log, but not on > the serial console. Yeah, makes enough sense. We've previously kind of used it as a marker as to where everytnhing went horribly wrong during boot, but we should be able to always grab the OPAL in memory console out these days. Merged to master as of a3a8f7d0cf1119fba39ffd2e68c27eca1a60f06d
diff --git a/core/cpu.c b/core/cpu.c index cc5b88c5ce8a..4f518a4c10b2 100644 --- a/core/cpu.c +++ b/core/cpu.c @@ -1467,9 +1467,9 @@ static int64_t opal_reinit_cpus(uint64_t flags) prlog(PR_DEBUG, "OPAL: CPU re-init with flags: 0x%llx\n", flags); if (flags & OPAL_REINIT_CPUS_HILE_LE) - prlog(PR_NOTICE, "OPAL: Switch to little-endian OS\n"); + prlog(PR_INFO, "OPAL: Switch to little-endian OS\n"); else if (flags & OPAL_REINIT_CPUS_HILE_BE) - prlog(PR_NOTICE, "OPAL: Switch to big-endian OS\n"); + prlog(PR_INFO, "OPAL: Switch to big-endian OS\n"); again: lock(&reinit_lock);
Users see these when loading an OS from Petitboot: [ 119.486794100,5] OPAL: Switch to big-endian OS [ 120.022302604,5] OPAL: Switch to little-endian OS Which is expected and doesn't provide any information the user can act on. Switch them to PR_INFO so they still appear in the log, but not on the serial console. Signed-off-by: Joel Stanley <joel@jms.id.au> --- core/cpu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)