On Wed, 7 Dec 2011, Johannes Weiner wrote:
> > @@ -4913,31 +4913,31 @@ void __init free_area_init_nodes(unsigned long *max_zone_pfn)
> > find_zone_movable_pfns_for_nodes(zone_movable_pfn);
> >
> > /* Print out the zone ranges */
> > - printk("Zone PFN ranges:\n");
> > + printk(KERN_INFO "Zone PFN ranges:\n");
> > for (i = 0; i < MAX_NR_ZONES; i++) {
> > if (i == ZONE_MOVABLE)
> > continue;
> > - printk(" %-8s ", zone_names[i]);
> > + printk(KERN_INFO " %-8s ", zone_names[i]);
> > if (arch_zone_lowest_possible_pfn[i] ==
> > arch_zone_highest_possible_pfn[i])
> > - printk("empty\n");
> > + printk(KERN_INFO "empty\n");
> > else
> > - printk("%0#10lx -> %0#10lx\n",
> > + printk(KERN_INFO "%0#10lx -> %0#10lx\n",
> > arch_zone_lowest_possible_pfn[i],
> > arch_zone_highest_possible_pfn[i]);
> > }
>
> Shouldn't continuation lines be KERN_CONT instead?
>
Indeed, and I think it would benefit from using pr_info() and pr_cont() as
well to avoid going over 80 characters per line.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>