2012-11-29 03:25:41

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: build warning after merge of the xen-two tree

Hi Konrad,

After merging the xen-two tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

arch/x86/xen/enlighten.c: In function 'init_hvm_pv_info':
arch/x86/xen/enlighten.c:1617:16: warning: unused variable 'ebx' [-Wunused-variable]
arch/x86/xen/enlighten.c:1617:11: warning: unused variable 'eax' [-Wunused-variable]

Introduced by commit 9d02b43dee0d ("xen PVonHVM: use E820_Reserved area
for shared_info").

--
Cheers,
Stephen Rothwell [email protected]

Attachments:
(No filename) (505.00 B)
 (No filename) (836.00 B)
 Download all attachments

2012-11-29 14:05:38

by Konrad Rzeszutek Wilk

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the xen-two tree

On Thu, Nov 29, 2012 at 02:25:30PM +1100, Stephen Rothwell wrote:
> Hi Konrad,
>
> After merging the xen-two tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> arch/x86/xen/enlighten.c: In function 'init_hvm_pv_info':
> arch/x86/xen/enlighten.c:1617:16: warning: unused variable 'ebx' [-Wunused-variable]
> arch/x86/xen/enlighten.c:1617:11: warning: unused variable 'eax' [-Wunused-variable]
>
> Introduced by commit 9d02b43dee0d ("xen PVonHVM: use E820_Reserved area
> for shared_info").

Olaf, could you send me a patch to fix that pls?

>
> --
> Cheers,
> Stephen Rothwell [email protected]

2012-11-29 14:32:56

by Olaf Hering

[permalink] [raw]
Subject: [PATCH] xen PVonHVM: fix compile warning in init_hvm_pv_info

After merging the xen-two tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

arch/x86/xen/enlighten.c: In function 'init_hvm_pv_info':
arch/x86/xen/enlighten.c:1617:16: warning: unused variable 'ebx' [-Wunused-variable]
arch/x86/xen/enlighten.c:1617:11: warning: unused variable 'eax' [-Wunused-variable]

Introduced by commit 9d02b43dee0d ("xen PVonHVM: use E820_Reserved area
for shared_info").

Signed-off-by: Olaf Hering <[email protected]>
---
arch/x86/xen/enlighten.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
index a90c3bb..46156f5 100644
--- a/arch/x86/xen/enlighten.c
+++ b/arch/x86/xen/enlighten.c
@@ -1557,7 +1557,7 @@ static void __init xen_hvm_init_shared_info(void)

static void __init init_hvm_pv_info(void)
{
- uint32_t eax, ebx, ecx, edx, pages, msr, base;
+ uint32_t ecx, edx, pages, msr, base;
u64 pfn;

base = xen_cpuid_base();
--
1.8.0