Re: Re: [Xen-users] gcc 4.0.0 and xen-devel

[Vincent Hanquez <vincent.hanquez@cl.cam.ac.uk>]

On Thu, Jun 09, 2005 at 10:55:07AM -0300, Flavio Leitner wrote:
> 
> More info below where is failing.
> 
> On Thu, Jun 09, 2005 at 10:13:37AM -0300, Flavio Leitner wrote:
> > 
> > I checkout bktree from bk://xen.bkbits.net/xeno-unstable.bk 
> > and now I'm getting these messages:
> > 
> > make world or make dist
> > 
> > <snip>
> > gcc -nostdinc -fno-builtin -fno-common -fno-strict-aliasing -iwithprefix
> > include -Wall -Werror -Wno-pointer-arith -pipe
> > -I/home/fbl/xeno-unstable.bk/xen/include
> > -I/home/fbl/xeno-unstable.bk/xen/include/asm-x86/mach-generic
> > -I/home/fbl/xeno-unstable.bk/xen/include/asm-x86/mach-default -O3
> > -fomit-frame-pointer -msoft-float -m32 -march=i686 -DNDEBUG -c i8259.c
> > -o i8259.o
> > {standard input}: Assembler messages:
> > {standard input}:6: Error: suffix or operands invalid for `mov'
> > {standard input}:6: Error: suffix or operands invalid for `mov'
> > {standard input}:6: Error: suffix or operands invalid for `mov'
> > {standard input}:6: Error: suffix or operands invalid for `mov'
> > {standard input}:1296: Error: suffix or operands invalid for `mov'
> > {standard input}:1296: Error: suffix or operands invalid for `mov'
> > <snip>
> 
> Gcc 4.0.0
> Binutils 2.16.90.0.2

these binutils version are unstable CVS (not a good idea to have that IMHO) 

and as far as I remember they remove support for "movl %ds, ...".
now is must use: "mov %ds, ..."

does the following patch fix your problems ?

diff -Nru a/xen/include/asm-x86/x86_32/asm_defns.h b/xen/include/asm-x86/x86_32/asm_defns.h
--- a/xen/include/asm-x86/x86_32/asm_defns.h	2005-06-09 17:52:47 +01:00
+++ b/xen/include/asm-x86/x86_32/asm_defns.h	2005-06-09 17:52:47 +01:00
@@ -19,10 +19,10 @@
         "jmp 3f;"                                                       \
         "2:testb $3,"STR(UREGS_cs)"(%esp);"                             \
         "jz 1f;"                                                        \
-        "movl %ds,"STR(UREGS_ds)"(%esp);"                               \
-        "movl %es,"STR(UREGS_es)"(%esp);"                               \
-        "movl %fs,"STR(UREGS_fs)"(%esp);"                               \
-        "movl %gs,"STR(UREGS_gs)"(%esp);"                               \
+        "mov %ds,"STR(UREGS_ds)"(%esp);"                                \
+        "mov %es,"STR(UREGS_es)"(%esp);"                                \
+        "mov %fs,"STR(UREGS_fs)"(%esp);"                                \
+        "mov %gs,"STR(UREGS_gs)"(%esp);"                                \
         "3:"
 
 #define SAVE_ALL_NOSEGREGS(_reg)                \



-- 
Vincent Hanquez