diff options
| author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-10-13 15:15:35 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-10-15 14:23:24 +1100 |
| commit | 324289829343bd11c2c0cc4cdbec80ce81bbf89d (patch) | |
| tree | 5c670de6abdaf94496a22d1b7956be5d189496ea /hdata/pcia.c | |
| parent | 07c91d4322eb3c5b0849f9ba85ec51a69c09eda0 (diff) | |
| download | skiboot-324289829343bd11c2c0cc4cdbec80ce81bbf89d.zip skiboot-324289829343bd11c2c0cc4cdbec80ce81bbf89d.tar.gz skiboot-324289829343bd11c2c0cc4cdbec80ce81bbf89d.tar.bz2 | |
decrease verbosity of logging CPU information.
If we have an UNAVAILABLE core (e.g. garded by FSP), we *WILL*
print that out with PR_WARNING, as that's probably something
that people want to pay attention to. Otherwise we stick with
PR_INFO, which should be enough for these kind of log messages.
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hdata/pcia.c')
| -rw-r--r-- | hdata/pcia.c | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/hdata/pcia.c b/hdata/pcia.c index d128a31..29a81c1 100644 --- a/hdata/pcia.c +++ b/hdata/pcia.c @@ -124,11 +124,11 @@ static struct dt_node *add_core_node(struct dt_node *cpus, >> CPU_ID_NUM_SECONDARY_THREAD_SHIFT) + 1; assert(threads <= PCIA_MAX_THREADS); - printf("CORE[%i]: PIR=%i RES=%i %s %s(%u threads)\n", - pcia_index(pcia), t->pir, t->proc_int_line, - ve_flags & CPU_ID_PACA_RESERVED - ? "**RESERVED**" : cpu_state(ve_flags), - be32_to_cpu(t->pir) == boot_cpu->pir ? "[boot] " : "", threads); + prlog(PR_INFO, "CORE[%i]: PIR=%i RES=%i %s %s(%u threads)\n", + pcia_index(pcia), t->pir, t->proc_int_line, + ve_flags & CPU_ID_PACA_RESERVED + ? "**RESERVED**" : cpu_state(ve_flags), + be32_to_cpu(t->pir) == boot_cpu->pir ? "[boot] " : "", threads); timebase = HDIF_get_idata(pcia, SPPCIA_IDATA_TIMEBASE, &size); if (!timebase || size < sizeof(*timebase)) { @@ -198,7 +198,7 @@ bool pcia_parse(void) if (!pcia) return false; - printf("Got PCIA !\n"); + prlog(PR_INFO, "Got PCIA !\n"); got_pcia = true; @@ -229,11 +229,12 @@ bool pcia_parse(void) okay = false; } - printf("CORE[%i]: HW_PROC_ID=%i PROC_CHIP_ID=%i EC=0x%x %s\n", - pcia_index(pcia), be32_to_cpu(id->hw_proc_id), - be32_to_cpu(id->proc_chip_id), - be32_to_cpu(id->chip_ec_level), - okay ? "OK" : "UNAVAILABLE"); + prlog(okay ? PR_INFO : PR_WARNING, + "CORE[%i]: HW_PROC_ID=%i PROC_CHIP_ID=%i EC=0x%x %s\n", + pcia_index(pcia), be32_to_cpu(id->hw_proc_id), + be32_to_cpu(id->proc_chip_id), + be32_to_cpu(id->chip_ec_level), + okay ? "OK" : "UNAVAILABLE"); if (!add_core_node(cpus, pcia, id, okay)) break; |