Commit 1cf44baa authored Sep 04, 2008 by Ingo Molnar

IO resources: fix/remove printk



Andrew Morton noticed that the printk in kernel/resource.c was buggy:

| start and end have type resource_size_t.  Such types CANNOT be printed
| unless cast to a known type.
|
| Because there is a %s following an incorrect %lld, the above code will
| crash the machine.

... and it's probably quite unneeded as well, so remove it.

Reported-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

parent 268364a0

kernel/resource.c

+1 −5

Original line number	Diff line number	Diff line
		@@ -549,13 +549,9 @@ static void __init __reserve_region_with_split(struct resource *root,
		}

		if (!res) {
		printk(KERN_DEBUG " __reserve_region_with_split: (%s) [%llx, %llx], res: (%s) [%llx, %llx]\n",
		conflict->name, conflict->start, conflict->end,
		name, start, end);

		/* failed, split and try again */

		/* conflict coverred whole area */
		/* conflict covered whole area */
		if (conflict->start <= start && conflict->end >= end)
		return;

Admin message