All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch] kvm/ia64 build fix
@ 2008-09-12 12:12 Jes Sorensen
  2008-09-12 15:29 ` Luck, Tony
  2008-09-17  6:16 ` Avi Kivity
  0 siblings, 2 replies; 3+ messages in thread
From: Jes Sorensen @ 2008-09-12 12:12 UTC (permalink / raw)
  To: kvm-ia64

[-- Attachment #1: Type: text/plain, Size: 96 bytes --]

Hi,

This is needed against Linus' current tree to avoid a struct redefine
error.

Cheers,
Jes


[-- Attachment #2: 0200-ia64-kvm-build-fix.patch --]
[-- Type: text/plain, Size: 817 bytes --]

Rely on struct fdisc from asm/elf.h to define the ia64 function pointer
layout - no need to duplicate locally.

Fixes build problem.

Signed-off-by: Jes Sorensen <jes@sgi.com>

---
 arch/ia64/kvm/kvm-ia64.c |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

Index: linux-2.6.git/arch/ia64/kvm/kvm-ia64.c
===================================================================
--- linux-2.6.git.orig/arch/ia64/kvm/kvm-ia64.c
+++ linux-2.6.git/arch/ia64/kvm/kvm-ia64.c
@@ -38,6 +38,7 @@
 #include <asm/cacheflush.h>
 #include <asm/div64.h>
 #include <asm/tlb.h>
+#include <asm/elf.h>
 
 #include "misc.h"
 #include "vti.h"
@@ -61,12 +62,6 @@
 	{ NULL }
 };
 
-
-struct fdesc{
-    unsigned long ip;
-    unsigned long gp;
-};
-
 static void kvm_flush_icache(unsigned long start, unsigned long len)
 {
 	int l;

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [patch] kvm/ia64 build fix
  2008-09-12 12:12 [patch] kvm/ia64 build fix Jes Sorensen
@ 2008-09-12 15:29 ` Luck, Tony
  2008-09-17  6:16 ` Avi Kivity
  1 sibling, 0 replies; 3+ messages in thread
From: Luck, Tony @ 2008-09-12 15:29 UTC (permalink / raw)
  To: kvm-ia64

> This is needed against Linus' current tree to avoid a struct redefine
> error.

Drat!  I noticed that kvm had cloned the definition of
struct fdesc when chatting with James on IRC about this,
but forgot to check a CONFIG_KVM build when testing his
patch.

Avi: Will you apply this and send to Linus?  If so
here's an:

Acked-by: Tony Luck <tony.luck@intel.com>

-Tony

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] kvm/ia64 build fix
  2008-09-12 12:12 [patch] kvm/ia64 build fix Jes Sorensen
  2008-09-12 15:29 ` Luck, Tony
@ 2008-09-17  6:16 ` Avi Kivity
  1 sibling, 0 replies; 3+ messages in thread
From: Avi Kivity @ 2008-09-17  6:16 UTC (permalink / raw)
  To: kvm-ia64

Jes Sorensen wrote:
> Hi,
>
> This is needed against Linus' current tree to avoid a struct redefine
> error.
>

Applied, thanks.  Will queue for 2.6.27.

-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-09-17  6:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-09-12 12:12 [patch] kvm/ia64 build fix Jes Sorensen
2008-09-12 15:29 ` Luck, Tony
2008-09-17  6:16 ` Avi Kivity

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.