* [PATCH] include KERN_* constant in printk calls @ 2009-05-18 23:31 Chris Sanford 2009-05-19 9:00 ` Ingo Molnar 0 siblings, 1 reply; 9+ messages in thread From: Chris Sanford @ 2009-05-18 23:31 UTC (permalink / raw) To: linux-kernel, kernel-janitors; +Cc: mingo, peterz Add the KERN_CONT constant to two printk calls. Signed-off-by: Chris Sanford <crsanford@gmail.com> --- diff --git a/kernel/sched.c b/kernel/sched.c index 26efa47..6a2ed16 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -7319,7 +7319,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, printk(KERN_DEBUG "%*s domain %d: ", level, "", level); if (!(sd->flags & SD_LOAD_BALANCE)) { - printk("does not load-balance\n"); + printk(KERN_CONT "does not load-balance\n"); if (sd->parent) printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" " has parent"); @@ -7340,7 +7340,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, printk(KERN_DEBUG "%*s groups:", level + 1, ""); do { if (!group) { - printk("\n"); + printk(KERN_CONT "\n"); printk(KERN_ERR "ERROR: group is NULL\n"); break; } ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-18 23:31 [PATCH] include KERN_* constant in printk calls Chris Sanford @ 2009-05-19 9:00 ` Ingo Molnar 2009-05-20 4:32 ` Chris Sanford 0 siblings, 1 reply; 9+ messages in thread From: Ingo Molnar @ 2009-05-19 9:00 UTC (permalink / raw) To: linux-kernel, kernel-janitors, peterz * Chris Sanford <crsanford@gmail.com> wrote: > Add the KERN_CONT constant to two printk calls. > > Signed-off-by: Chris Sanford <crsanford@gmail.com> > > --- > > > diff --git a/kernel/sched.c b/kernel/sched.c > index 26efa47..6a2ed16 100644 > --- a/kernel/sched.c > +++ b/kernel/sched.c > @@ -7319,7 +7319,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > printk(KERN_DEBUG "%*s domain %d: ", level, "", level); > > if (!(sd->flags & SD_LOAD_BALANCE)) { > - printk("does not load-balance\n"); > + printk(KERN_CONT "does not load-balance\n"); > if (sd->parent) > printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" > " has parent"); > @@ -7340,7 +7340,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > printk(KERN_DEBUG "%*s groups:", level + 1, ""); > do { > if (!group) { > - printk("\n"); > + printk(KERN_CONT "\n"); > printk(KERN_ERR "ERROR: group is NULL\n"); > break; Please use pr_cont() - and while at it, perhaps convert all other printk()s in sched.c from KERN_* to the pr_*() notation as well? Thanks, Ingo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-19 9:00 ` Ingo Molnar @ 2009-05-20 4:32 ` Chris Sanford 2009-05-20 8:37 ` Ingo Molnar 0 siblings, 1 reply; 9+ messages in thread From: Chris Sanford @ 2009-05-20 4:32 UTC (permalink / raw) To: Ingo Molnar; +Cc: linux-kernel, kernel-janitors, peterz On Tue, May 19 09 11:00, Ingo Molnar wrote: > > * Chris Sanford <crsanford@gmail.com> wrote: > > > Add the KERN_CONT constant to two printk calls. > > > > Signed-off-by: Chris Sanford <crsanford@gmail.com> > > > > --- > > > > > > diff --git a/kernel/sched.c b/kernel/sched.c > > index 26efa47..6a2ed16 100644 > > --- a/kernel/sched.c > > +++ b/kernel/sched.c > > @@ -7319,7 +7319,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > > printk(KERN_DEBUG "%*s domain %d: ", level, "", level); > > > > if (!(sd->flags & SD_LOAD_BALANCE)) { > > - printk("does not load-balance\n"); > > + printk(KERN_CONT "does not load-balance\n"); > > if (sd->parent) > > printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" > > " has parent"); > > @@ -7340,7 +7340,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > > printk(KERN_DEBUG "%*s groups:", level + 1, ""); > > do { > > if (!group) { > > - printk("\n"); > > + printk(KERN_CONT "\n"); > > printk(KERN_ERR "ERROR: group is NULL\n"); > > break; > > Please use pr_cont() - and while at it, perhaps convert all other > printk()s in sched.c from KERN_* to the pr_*() notation as well? > > Thanks, > > Ingo > -- The KERN_DEBUG constant is used 5 times when DEBUG is undefined. Doing a straight conversion from printk(KERN_DEBUG to pr_debug( will result in no messages being printed. Should these calls have a different message type, should DEBUG be defined, or should something else be done? Thanks, Chris ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-20 4:32 ` Chris Sanford @ 2009-05-20 8:37 ` Ingo Molnar 2009-05-21 2:46 ` Chris Sanford 0 siblings, 1 reply; 9+ messages in thread From: Ingo Molnar @ 2009-05-20 8:37 UTC (permalink / raw) To: linux-kernel, kernel-janitors, peterz * Chris Sanford <crsanford@gmail.com> wrote: > On Tue, May 19 09 11:00, Ingo Molnar wrote: > > > > * Chris Sanford <crsanford@gmail.com> wrote: > > > > > Add the KERN_CONT constant to two printk calls. > > > > > > Signed-off-by: Chris Sanford <crsanford@gmail.com> > > > > > > --- > > > > > > > > > diff --git a/kernel/sched.c b/kernel/sched.c > > > index 26efa47..6a2ed16 100644 > > > --- a/kernel/sched.c > > > +++ b/kernel/sched.c > > > @@ -7319,7 +7319,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > > > printk(KERN_DEBUG "%*s domain %d: ", level, "", level); > > > > > > if (!(sd->flags & SD_LOAD_BALANCE)) { > > > - printk("does not load-balance\n"); > > > + printk(KERN_CONT "does not load-balance\n"); > > > if (sd->parent) > > > printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" > > > " has parent"); > > > @@ -7340,7 +7340,7 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > > > printk(KERN_DEBUG "%*s groups:", level + 1, ""); > > > do { > > > if (!group) { > > > - printk("\n"); > > > + printk(KERN_CONT "\n"); > > > printk(KERN_ERR "ERROR: group is NULL\n"); > > > break; > > > > Please use pr_cont() - and while at it, perhaps convert all other > > printk()s in sched.c from KERN_* to the pr_*() notation as well? > > > > Thanks, > > > > Ingo > > -- > > The KERN_DEBUG constant is used 5 times when DEBUG is undefined. > Doing a straight conversion from printk(KERN_DEBUG to pr_debug( > will result in no messages being printed. Should these calls have > a different message type, should DEBUG be defined, or should > something else be done? good point - i think it's fine to change it to pr_info(). Ingo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-20 8:37 ` Ingo Molnar @ 2009-05-21 2:46 ` Chris Sanford 2009-05-21 3:30 ` Joe Perches 2009-05-21 5:09 ` Michael Ellerman 0 siblings, 2 replies; 9+ messages in thread From: Chris Sanford @ 2009-05-21 2:46 UTC (permalink / raw) To: kernel-janitors; +Cc: linux-kernel, peterz Here is the updated patch for sched.c. It replaces all the printk calls with their corresponding pr_* call. The five KERN_DEBUG calls have been changed to pr_info. Signed-off-by: Chris Sanford <crsanford@gmail.com> --- diff --git a/kernel/sched.c b/kernel/sched.c index 26efa47..cfe42e7 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -4912,7 +4912,7 @@ static noinline void __schedule_bug(struct task_struct *prev) { struct pt_regs *regs = get_irq_regs(); - printk(KERN_ERR "BUG: scheduling while atomic: %s/%d/0x%08x\n", + pr_err("BUG: scheduling while atomic: %s/%d/0x%08x\n", prev->comm, prev->pid, preempt_count()); debug_show_held_locks(prev); @@ -6474,23 +6474,23 @@ void sched_show_task(struct task_struct *p) unsigned state; state = p->state ? __ffs(p->state) + 1 : 0; - printk(KERN_INFO "%-13.13s %c", p->comm, + pr_info("%-13.13s %c", p->comm, state < sizeof(stat_nam) - 1 ? stat_nam[state] : '?'); #if BITS_PER_LONG == 32 if (state == TASK_RUNNING) - printk(KERN_CONT " running "); + pr_cont(" running "); else - printk(KERN_CONT " %08lx ", thread_saved_pc(p)); + pr_cont(" %08lx ", thread_saved_pc(p)); #else if (state == TASK_RUNNING) - printk(KERN_CONT " running task "); + pr_cont(" running task "); else - printk(KERN_CONT " %016lx ", thread_saved_pc(p)); + pr_cont(" %016lx ", thread_saved_pc(p)); #endif #ifdef CONFIG_DEBUG_STACK_USAGE free = stack_not_used(p); #endif - printk(KERN_CONT "%5lu %5d %6d\n", free, + pr_cont("%5lu %5d %6d\n", free, task_pid_nr(p), task_pid_nr(p->real_parent)); show_stack(p, NULL); @@ -6501,10 +6501,10 @@ void show_state_filter(unsigned long state_filter) struct task_struct *g, *p; #if BITS_PER_LONG == 32 - printk(KERN_INFO + pr_info( " task PC stack pid father\n"); #else - printk(KERN_INFO + pr_info( " task PC stack pid father\n"); #endif read_lock(&tasklist_lock); @@ -6833,7 +6833,7 @@ again: * leave kernel. */ if (p->mm && printk_ratelimit()) { - printk(KERN_INFO "process %d (%s) no " + pr_info("process %d (%s) no " "longer affine to cpu%d\n", task_pid_nr(p), p->comm, dead_cpu); } @@ -7315,52 +7315,52 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, cpulist_scnprintf(str, sizeof(str), sched_domain_span(sd)); cpumask_clear(groupmask); - - printk(KERN_DEBUG "%*s domain %d: ", level, "", level); - + + pr_info("%*s domain %d: ", level, "", level); + if (!(sd->flags & SD_LOAD_BALANCE)) { - printk("does not load-balance\n"); + pr_cont("does not load-balance\n"); if (sd->parent) - printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" + pr_err("ERROR: !SD_LOAD_BALANCE domain" " has parent"); return -1; } - printk(KERN_CONT "span %s level %s\n", str, sd->name); + pr_cont("span %s level %s\n", str, sd->name); if (!cpumask_test_cpu(cpu, sched_domain_span(sd))) { - printk(KERN_ERR "ERROR: domain->span does not contain " + pr_err("ERROR: domain->span does not contain " "CPU%d\n", cpu); } if (!cpumask_test_cpu(cpu, sched_group_cpus(group))) { - printk(KERN_ERR "ERROR: domain->groups does not contain" + pr_err("ERROR: domain->groups does not contain" " CPU%d\n", cpu); } - printk(KERN_DEBUG "%*s groups:", level + 1, ""); + pr_info("%*s groups:", level + 1, ""); do { if (!group) { - printk("\n"); - printk(KERN_ERR "ERROR: group is NULL\n"); + pr_cont("\n"); + pr_err("ERROR: group is NULL\n"); break; } if (!group->__cpu_power) { - printk(KERN_CONT "\n"); - printk(KERN_ERR "ERROR: domain->cpu_power not " + pr_cont("\n"); + pr_err("ERROR: domain->cpu_power not " "set\n"); break; } if (!cpumask_weight(sched_group_cpus(group))) { - printk(KERN_CONT "\n"); - printk(KERN_ERR "ERROR: empty group\n"); + pr_cont("\n"); + pr_err("ERROR: empty group\n"); break; } if (cpumask_intersects(groupmask, sched_group_cpus(group))) { - printk(KERN_CONT "\n"); - printk(KERN_ERR "ERROR: repeated CPUs\n"); + pr_cont("\n"); + pr_err("ERROR: repeated CPUs\n"); break; } @@ -7368,22 +7368,22 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, cpulist_scnprintf(str, sizeof(str), sched_group_cpus(group)); - printk(KERN_CONT " %s", str); + pr_cont(" %s", str); if (group->__cpu_power != SCHED_LOAD_SCALE) { - printk(KERN_CONT " (__cpu_power = %d)", + pr_cont(" (__cpu_power = %d)", group->__cpu_power); } group = group->next; } while (group != sd->groups); - printk(KERN_CONT "\n"); + pr_cont("\n"); if (!cpumask_equal(sched_domain_span(sd), groupmask)) - printk(KERN_ERR "ERROR: groups don't span domain->span\n"); + pr_err("ERROR: groups don't span domain->span\n"); if (sd->parent && !cpumask_subset(groupmask, sched_domain_span(sd->parent))) - printk(KERN_ERR "ERROR: parent span is not a superset " + pr_err("ERROR: parent span is not a superset " "of domain->span\n"); return 0; } @@ -7394,14 +7394,14 @@ static void sched_domain_debug(struct sched_domain *sd, int cpu) int level = 0; if (!sd) { - printk(KERN_DEBUG "CPU%d attaching NULL sched-domain.\n", cpu); + pr_info("CPU%d attaching NULL sched-domain.\n", cpu); return; } - printk(KERN_DEBUG "CPU%d attaching sched-domain:\n", cpu); + pr_info("CPU%d attaching sched-domain:\n", cpu); if (!alloc_cpumask_var(&groupmask, GFP_KERNEL)) { - printk(KERN_DEBUG "Cannot load-balance (out of memory)\n"); + pr_info("Cannot load-balance (out of memory)\n"); return; } @@ -8095,14 +8095,14 @@ static int __build_sched_domains(const struct cpumask *cpu_map, sched_group_nodes = kcalloc(nr_node_ids, sizeof(struct sched_group *), GFP_KERNEL); if (!sched_group_nodes) { - printk(KERN_WARNING "Can not alloc sched group node list\n"); + pr_warning("Can not alloc sched group node list\n"); goto free_tmpmask; } #endif rd = alloc_rootdomain(); if (!rd) { - printk(KERN_WARNING "Cannot alloc root domain\n"); + pr_warning("Cannot alloc root domain\n"); goto free_sched_groups; } @@ -8241,7 +8241,7 @@ static int __build_sched_domains(const struct cpumask *cpu_map, sg = kmalloc_node(sizeof(struct sched_group) + cpumask_size(), GFP_KERNEL, i); if (!sg) { - printk(KERN_WARNING "Can not alloc domain group for " + pr_warning("Can not alloc domain group for " "node %d\n", i); goto error; } @@ -8275,7 +8275,7 @@ static int __build_sched_domains(const struct cpumask *cpu_map, cpumask_size(), GFP_KERNEL, i); if (!sg) { - printk(KERN_WARNING + pr_warning( "Can not alloc domain group for node %d\n", j); goto error; } @@ -9075,10 +9075,10 @@ void __might_sleep(char *file, int line) return; prev_jiffy = jiffies; - printk(KERN_ERR + pr_err( "BUG: sleeping function called from invalid context at %s:%d\n", file, line); - printk(KERN_ERR + pr_err( "in_atomic(): %d, irqs_disabled(): %d, pid: %d, name: %s\n", in_atomic(), irqs_disabled(), current->pid, current->comm); ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-21 2:46 ` Chris Sanford @ 2009-05-21 3:30 ` Joe Perches 2009-05-21 4:09 ` Chris Sanford 2009-05-21 5:09 ` Michael Ellerman 1 sibling, 1 reply; 9+ messages in thread From: Joe Perches @ 2009-05-21 3:30 UTC (permalink / raw) To: Chris Sanford; +Cc: kernel-janitors, linux-kernel, peterz Hi Chris. Minor comments. On Wed, 2009-05-20 at 19:46 -0700, Chris Sanford wrote: > diff --git a/kernel/sched.c b/kernel/sched.c > index 26efa47..cfe42e7 100644 > --- a/kernel/sched.c > +++ b/kernel/sched.c [] > @@ -6501,10 +6501,10 @@ void show_state_filter(unsigned long state_filter) > struct task_struct *g, *p; > > #if BITS_PER_LONG == 32 > - printk(KERN_INFO > + pr_info( > " task PC stack pid father\n"); > #else > - printk(KERN_INFO > + pr_info( > " task PC stack pid father\n"); > #endif > read_lock(&tasklist_lock); One line please. pr_info(" task etc... > @@ -6833,7 +6833,7 @@ again: > * leave kernel. > */ > if (p->mm && printk_ratelimit()) { > - printk(KERN_INFO "process %d (%s) no " > + pr_info("process %d (%s) no " > "longer affine to cpu%d\n", > task_pid_nr(p), p->comm, dead_cpu); > } pr_info("process $d (%s) no longer affine to cpu%d\n" > @@ -7315,52 +7315,52 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > > cpulist_scnprintf(str, sizeof(str), sched_domain_span(sd)); > cpumask_clear(groupmask); > - > - printk(KERN_DEBUG "%*s domain %d: ", level, "", level); > - > + > + pr_info("%*s domain %d: ", level, "", level); > + > if (!(sd->flags & SD_LOAD_BALANCE)) { > - printk("does not load-balance\n"); > + pr_cont("does not load-balance\n"); > if (sd->parent) > - printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" > + pr_err("ERROR: !SD_LOAD_BALANCE domain" > " has parent"); Make single line and add "\n"? pr_err("ERROR: !SD_LOAD_BALANCE domain has parent\n" > return -1; > } > > - printk(KERN_CONT "span %s level %s\n", str, sd->name); > + pr_cont("span %s level %s\n", str, sd->name); > > if (!cpumask_test_cpu(cpu, sched_domain_span(sd))) { > - printk(KERN_ERR "ERROR: domain->span does not contain " > + pr_err("ERROR: domain->span does not contain " > "CPU%d\n", cpu); Single line? > } > if (!cpumask_test_cpu(cpu, sched_group_cpus(group))) { > - printk(KERN_ERR "ERROR: domain->groups does not contain" > + pr_err("ERROR: domain->groups does not contain" > " CPU%d\n", cpu); Single line? > @@ -7368,22 +7368,22 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, > > cpulist_scnprintf(str, sizeof(str), sched_group_cpus(group)); > > - printk(KERN_CONT " %s", str); > + pr_cont(" %s", str); > if (group->__cpu_power != SCHED_LOAD_SCALE) { > - printk(KERN_CONT " (__cpu_power = %d)", > + pr_cont(" (__cpu_power = %d)", > group->__cpu_power); > } > > group = group->next; > } while (group != sd->groups); > - printk(KERN_CONT "\n"); > + pr_cont("\n"); > > if (!cpumask_equal(sched_domain_span(sd), groupmask)) > - printk(KERN_ERR "ERROR: groups don't span domain->span\n"); > + pr_err("ERROR: groups don't span domain->span\n"); > > if (sd->parent && > !cpumask_subset(groupmask, sched_domain_span(sd->parent))) > - printk(KERN_ERR "ERROR: parent span is not a superset " > + pr_err("ERROR: parent span is not a superset " > "of domain->span\n"); Single line? > @@ -8095,14 +8095,14 @@ static int __build_sched_domains(const struct cpumask *cpu_map, > sched_group_nodes = kcalloc(nr_node_ids, sizeof(struct sched_group *), > GFP_KERNEL); > if (!sched_group_nodes) { > - printk(KERN_WARNING "Can not alloc sched group node list\n"); > + pr_warning("Can not alloc sched group node list\n"); > goto free_tmpmask; > } > #endif > > rd = alloc_rootdomain(); > if (!rd) { > - printk(KERN_WARNING "Cannot alloc root domain\n"); > + pr_warning("Cannot alloc root domain\n"); > goto free_sched_groups; > } > "Can not" vs "Cannot", maybe choose a single style? > @@ -8241,7 +8241,7 @@ static int __build_sched_domains(const struct cpumask *cpu_map, > sg = kmalloc_node(sizeof(struct sched_group) + cpumask_size(), > GFP_KERNEL, i); > if (!sg) { > - printk(KERN_WARNING "Can not alloc domain group for " > + pr_warning("Can not alloc domain group for " > "node %d\n", i); Single line string? Maybe: pr_warning("Can not alloc domain group for node %d\n", i); > @@ -9075,10 +9075,10 @@ void __might_sleep(char *file, int line) > return; > prev_jiffy = jiffies; > > - printk(KERN_ERR > + pr_err( > "BUG: sleeping function called from invalid context at %s:%d\n", > file, line); pr_err("BUG: etc > - printk(KERN_ERR > + pr_err( > "in_atomic(): %d, irqs_disabled(): %d, pid: %d, name: %s\n", > in_atomic(), irqs_disabled(), > current->pid, current->comm); here too. cheers, Joe ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-21 3:30 ` Joe Perches @ 2009-05-21 4:09 ` Chris Sanford 0 siblings, 0 replies; 9+ messages in thread From: Chris Sanford @ 2009-05-21 4:09 UTC (permalink / raw) To: Joe Perches; +Cc: kernel-janitors, linux-kernel, peterz Here it is with your suggestions... diff --git a/kernel/sched.c b/kernel/sched.c index 26efa47..c4fb502 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -4912,7 +4912,7 @@ static noinline void __schedule_bug(struct task_struct *prev) { struct pt_regs *regs = get_irq_regs(); - printk(KERN_ERR "BUG: scheduling while atomic: %s/%d/0x%08x\n", + pr_err("BUG: scheduling while atomic: %s/%d/0x%08x\n", prev->comm, prev->pid, preempt_count()); debug_show_held_locks(prev); @@ -6474,23 +6474,23 @@ void sched_show_task(struct task_struct *p) unsigned state; state = p->state ? __ffs(p->state) + 1 : 0; - printk(KERN_INFO "%-13.13s %c", p->comm, + pr_info("%-13.13s %c", p->comm, state < sizeof(stat_nam) - 1 ? stat_nam[state] : '?'); #if BITS_PER_LONG == 32 if (state == TASK_RUNNING) - printk(KERN_CONT " running "); + pr_cont(" running "); else - printk(KERN_CONT " %08lx ", thread_saved_pc(p)); + pr_cont(" %08lx ", thread_saved_pc(p)); #else if (state == TASK_RUNNING) - printk(KERN_CONT " running task "); + pr_cont(" running task "); else - printk(KERN_CONT " %016lx ", thread_saved_pc(p)); + pr_cont(" %016lx ", thread_saved_pc(p)); #endif #ifdef CONFIG_DEBUG_STACK_USAGE free = stack_not_used(p); #endif - printk(KERN_CONT "%5lu %5d %6d\n", free, + pr_cont("%5lu %5d %6d\n", free, task_pid_nr(p), task_pid_nr(p->real_parent)); show_stack(p, NULL); @@ -6501,11 +6501,9 @@ void show_state_filter(unsigned long state_filter) struct task_struct *g, *p; #if BITS_PER_LONG == 32 - printk(KERN_INFO - " task PC stack pid father\n"); + pr_info(" task PC stack pid father\n"); #else - printk(KERN_INFO - " task PC stack pid father\n"); + pr_info(" task PC stack pid father\n"); #endif read_lock(&tasklist_lock); do_each_thread(g, p) { @@ -6833,9 +6831,8 @@ again: * leave kernel. */ if (p->mm && printk_ratelimit()) { - printk(KERN_INFO "process %d (%s) no " - "longer affine to cpu%d\n", - task_pid_nr(p), p->comm, dead_cpu); + pr_info("process %d (%s) no longer affine to cpu%d\n", + task_pid_nr(p), p->comm, dead_cpu); } } @@ -7315,52 +7312,48 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, cpulist_scnprintf(str, sizeof(str), sched_domain_span(sd)); cpumask_clear(groupmask); - - printk(KERN_DEBUG "%*s domain %d: ", level, "", level); - + + pr_info("%*s domain %d: ", level, "", level); + if (!(sd->flags & SD_LOAD_BALANCE)) { - printk("does not load-balance\n"); + pr_cont("does not load-balance\n"); if (sd->parent) - printk(KERN_ERR "ERROR: !SD_LOAD_BALANCE domain" - " has parent"); + pr_err("ERROR: !SD_LOAD_BALANCE domain has parent\n"); return -1; } - printk(KERN_CONT "span %s level %s\n", str, sd->name); + pr_cont("span %s level %s\n", str, sd->name); if (!cpumask_test_cpu(cpu, sched_domain_span(sd))) { - printk(KERN_ERR "ERROR: domain->span does not contain " - "CPU%d\n", cpu); + pr_err("ERROR: domain->span does not contain CPU%d\n", cpu); } if (!cpumask_test_cpu(cpu, sched_group_cpus(group))) { - printk(KERN_ERR "ERROR: domain->groups does not contain" - " CPU%d\n", cpu); + pr_err("ERROR: domain->groups does not contain CPU%d\n", cpu); } - printk(KERN_DEBUG "%*s groups:", level + 1, ""); + pr_info("%*s groups:", level + 1, ""); do { if (!group) { - printk("\n"); - printk(KERN_ERR "ERROR: group is NULL\n"); + pr_cont("\n"); + pr_err("ERROR: group is NULL\n"); break; } if (!group->__cpu_power) { - printk(KERN_CONT "\n"); - printk(KERN_ERR "ERROR: domain->cpu_power not " - "set\n"); + pr_cont("\n"); + pr_err("ERROR: domain->cpu_power not set\n"); break; } if (!cpumask_weight(sched_group_cpus(group))) { - printk(KERN_CONT "\n"); - printk(KERN_ERR "ERROR: empty group\n"); + pr_cont("\n"); + pr_err("ERROR: empty group\n"); break; } if (cpumask_intersects(groupmask, sched_group_cpus(group))) { - printk(KERN_CONT "\n"); - printk(KERN_ERR "ERROR: repeated CPUs\n"); + pr_cont("\n"); + pr_err("ERROR: repeated CPUs\n"); break; } @@ -7368,23 +7361,22 @@ static int sched_domain_debug_one(struct sched_domain *sd, int cpu, int level, cpulist_scnprintf(str, sizeof(str), sched_group_cpus(group)); - printk(KERN_CONT " %s", str); + pr_cont(" %s", str); if (group->__cpu_power != SCHED_LOAD_SCALE) { - printk(KERN_CONT " (__cpu_power = %d)", + pr_cont(" (__cpu_power = %d)", group->__cpu_power); } group = group->next; } while (group != sd->groups); - printk(KERN_CONT "\n"); + pr_cont("\n"); if (!cpumask_equal(sched_domain_span(sd), groupmask)) - printk(KERN_ERR "ERROR: groups don't span domain->span\n"); + pr_err("ERROR: groups don't span domain->span\n"); if (sd->parent && !cpumask_subset(groupmask, sched_domain_span(sd->parent))) - printk(KERN_ERR "ERROR: parent span is not a superset " - "of domain->span\n"); + pr_err("ERROR: parent span is not a superset of domain->span\n"); return 0; } @@ -7394,14 +7386,14 @@ static void sched_domain_debug(struct sched_domain *sd, int cpu) int level = 0; if (!sd) { - printk(KERN_DEBUG "CPU%d attaching NULL sched-domain.\n", cpu); + pr_info("CPU%d attaching NULL sched-domain.\n", cpu); return; } - printk(KERN_DEBUG "CPU%d attaching sched-domain:\n", cpu); + pr_info("CPU%d attaching sched-domain:\n", cpu); if (!alloc_cpumask_var(&groupmask, GFP_KERNEL)) { - printk(KERN_DEBUG "Cannot load-balance (out of memory)\n"); + pr_info("Cannot load-balance (out of memory)\n"); return; } @@ -8095,14 +8087,14 @@ static int __build_sched_domains(const struct cpumask *cpu_map, sched_group_nodes = kcalloc(nr_node_ids, sizeof(struct sched_group *), GFP_KERNEL); if (!sched_group_nodes) { - printk(KERN_WARNING "Can not alloc sched group node list\n"); + pr_warning("Cannot alloc sched group node list\n"); goto free_tmpmask; } #endif rd = alloc_rootdomain(); if (!rd) { - printk(KERN_WARNING "Cannot alloc root domain\n"); + pr_warning("Cannot alloc root domain\n"); goto free_sched_groups; } @@ -8241,8 +8233,8 @@ static int __build_sched_domains(const struct cpumask *cpu_map, sg = kmalloc_node(sizeof(struct sched_group) + cpumask_size(), GFP_KERNEL, i); if (!sg) { - printk(KERN_WARNING "Can not alloc domain group for " - "node %d\n", i); + pr_warning("Cannot alloc domain group for node %d\n", + i); goto error; } sched_group_nodes[i] = sg; @@ -8275,8 +8267,8 @@ static int __build_sched_domains(const struct cpumask *cpu_map, cpumask_size(), GFP_KERNEL, i); if (!sg) { - printk(KERN_WARNING - "Can not alloc domain group for node %d\n", j); + pr_warning( + "Cannot alloc domain group for node %d\n", j); goto error; } sg->__cpu_power = 0; @@ -9075,13 +9067,11 @@ void __might_sleep(char *file, int line) return; prev_jiffy = jiffies; - printk(KERN_ERR - "BUG: sleeping function called from invalid context at %s:%d\n", - file, line); - printk(KERN_ERR - "in_atomic(): %d, irqs_disabled(): %d, pid: %d, name: %s\n", - in_atomic(), irqs_disabled(), - current->pid, current->comm); + pr_err("BUG: sleeping function called from invalid context at %s:%d\n", + file, line); + pr_err("in_atomic(): %d, irqs_disabled(): %d, pid: %d, name: %s\n", + in_atomic(), irqs_disabled(), + current->pid, current->comm); debug_show_held_locks(current); if (irqs_disabled()) ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-21 2:46 ` Chris Sanford 2009-05-21 3:30 ` Joe Perches @ 2009-05-21 5:09 ` Michael Ellerman 2009-05-21 9:09 ` walter harms 1 sibling, 1 reply; 9+ messages in thread From: Michael Ellerman @ 2009-05-21 5:09 UTC (permalink / raw) To: Chris Sanford; +Cc: kernel-janitors, linux-kernel, peterz [-- Attachment #1: Type: text/plain, Size: 625 bytes --] On Wed, 2009-05-20 at 19:46 -0700, Chris Sanford wrote: > Here is the updated patch for sched.c. It replaces all the printk calls with their > corresponding pr_* call. The five KERN_DEBUG calls have been changed to > pr_info. Switching from KERN_DEBUG to KERN_INFO means the messages will now appear for most people, whereas previously they didn't. I'm not sure that's a great idea. If you really want to convert them, then I think we need a macro for KERN_DEBUG but not conditionally compiled like pr_debug. I sent a patch for that once, but I don't think anyone liked it, and I've lost it now :D cheers [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 197 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] include KERN_* constant in printk calls 2009-05-21 5:09 ` Michael Ellerman @ 2009-05-21 9:09 ` walter harms 0 siblings, 0 replies; 9+ messages in thread From: walter harms @ 2009-05-21 9:09 UTC (permalink / raw) To: michael; +Cc: Chris Sanford, kernel-janitors, linux-kernel, peterz Michael Ellerman schrieb: > On Wed, 2009-05-20 at 19:46 -0700, Chris Sanford wrote: >> Here is the updated patch for sched.c. It replaces all the printk calls with their >> corresponding pr_* call. The five KERN_DEBUG calls have been changed to >> pr_info. > > Switching from KERN_DEBUG to KERN_INFO means the messages will now > appear for most people, whereas previously they didn't. I'm not sure > that's a great idea. > > If you really want to convert them, then I think we need a macro for > KERN_DEBUG but not conditionally compiled like pr_debug. > > I sent a patch for that once, but I don't think anyone liked it, and > I've lost it now :D > beeing mainly a user theses days i would like to comment ... It is fine if debug statements are disabled within NO_DEBUG mode. As user i find them annoying because i do not care. KERN_INFO is ok and if people complain they need more info someone can always change from DEBUG to INFO. putting to much info into syslog simply drowns important informations. so keep it as pr_debug(). just my 2 cents, wh ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2009-05-21 9:39 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2009-05-18 23:31 [PATCH] include KERN_* constant in printk calls Chris Sanford 2009-05-19 9:00 ` Ingo Molnar 2009-05-20 4:32 ` Chris Sanford 2009-05-20 8:37 ` Ingo Molnar 2009-05-21 2:46 ` Chris Sanford 2009-05-21 3:30 ` Joe Perches 2009-05-21 4:09 ` Chris Sanford 2009-05-21 5:09 ` Michael Ellerman 2009-05-21 9:09 ` walter harms
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).