Re: [PATCH 3/6] Suspend: Use common prefix in messages

From: Pavel Machek
Date: Mon Nov 26 2007 - 19:21:59 EST


Hi!

> From: Rafael J. Wysocki <rjw@xxxxxxx>
>
> Make suspend messages start with one common prefix "PM: ".
>
> Signed-off-by: Rafael J. Wysocki <rjw@xxxxxxx>

I do not think I like the PREFIX idea (see other message).


> @@ -344,11 +344,12 @@ static int enter_state(suspend_state_t s
> if (!mutex_trylock(&pm_mutex))
> return -EBUSY;
>
> - printk("Syncing filesystems ... ");
> + printk(KERN_INFO PREFIX "Syncing filesystems ... ");
> sys_sync();
> - printk("done.\n");
> + printk(KERN_INFO "done.\n");


Plus this is wrong.
Pavel

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/