2008-10-11 20:58:37

by Alexander Beregalov

Subject: [PATCH] kernel/resource: fix printk format warning

Signed-off-by: Alexander Beregalov <[email protected]>
---

kernel/resource.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/resource.c b/kernel/resource.c
index 414d6fc..cfe0933 100644
--- a/kernel/resource.c
+++ b/kernel/resource.c
@@ -550,8 +550,8 @@ 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);
+ conflict->name, (unsigned long long)conflict->start, (unsigned long long)conflict->end,
+ name, (unsigned long long)start, (unsigned long long)end);

/* failed, split and try again */

2008-10-11 21:53:59

by Yinghai Lu

Subject: Re: [PATCH] kernel/resource: fix printk format warning

On Sat, Oct 11, 2008 at 1:58 PM, Alexander Beregalov
<[email protected]> wrote:
>
>
> Signed-off-by: Alexander Beregalov <[email protected]>
> ---
>
> kernel/resource.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/kernel/resource.c b/kernel/resource.c
> index 414d6fc..cfe0933 100644
> --- a/kernel/resource.c
> +++ b/kernel/resource.c
> @@ -550,8 +550,8 @@ 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);
> + conflict->name, (unsigned long long)conflict->start, (unsigned long long)conflict->end,
> + name, (unsigned long long)start, (unsigned long long)end);
>
> /* failed, split and try again */

Ingo should already have one patch in tip to remove that print out.

YH