[Qemu-devel] Fixes for ia64 host

[Andreas Schwab]

This patch fixes some problems with qemu hosted on ia64:

- remove unaligned accesses in ia64_apply_fixes
- make sure that the static variables are not optimized away in GOTO_TB.

Andreas.

--- dyngen.h    8 May 2007 23:30:44 -0000       1.14
+++ dyngen.h    25 Jun 2007 18:15:39 -0000
@@ -392,7 +392,8 @@ static inline void ia64_apply_fixes (uin
        0x05, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x00, /* nop 0; brl IP */
        0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xc0
     };
-    uint8_t *gen_code_ptr = *gen_code_pp, *plt_start, *got_start, *vp;
+    uint8_t *gen_code_ptr = *gen_code_pp, *plt_start, *got_start;
+    uint64_t *vp;
     struct ia64_fixup *fixup;
     unsigned int offset = 0;
     struct fdesc {
@@ -429,12 +430,12 @@ static inline void ia64_apply_fixes (uin
     /* First, create the GOT: */
     for (fixup = ltoff_fixes; fixup; fixup = fixup->next) {
        /* first check if we already have this value in the GOT: */
-       for (vp = got_start; vp < gen_code_ptr; ++vp)
-           if (*(uint64_t *) vp == fixup->value)
+       for (vp = (uint64_t *) got_start; vp < (uint64_t *) gen_code_ptr; ++vp)
+           if (*vp == fixup->value)
                break;
-       if (vp == gen_code_ptr) {
+       if (vp == (uint64_t *) gen_code_ptr) {
            /* Nope, we need to put the value in the GOT: */
-           *(uint64_t *) vp = fixup->value;
+           *vp = fixup->value;
            gen_code_ptr += 8;
        }
        ia64_imm22(fixup->addr, (long) vp - gp);
--- exec-all.h  26 Mai 2007 14:58:26 +0200      1.57
+++ exec-all.h  25 Jun 2007 20:04:14 +0200      
@@ -346,8 +346,8 @@ do {\
    cache flushing, but slower because of indirect jump) */
 #define GOTO_TB(opname, tbparam, n)\
 do {\
-    static void __attribute__((unused)) *dummy ## n = &&dummy_label ## n;\
-    static void __attribute__((unused)) *__op_label ## n \
+    static void __attribute__((used)) *dummy ## n = &&dummy_label ## n;\
+    static void __attribute__((used)) *__op_label ## n \
         __asm__(ASM_OP_LABEL_NAME(n, opname)) = &&label ## n;\
     goto *(void *)(((TranslationBlock *)tbparam)->tb_next[n]);\
 label ## n: ;\

-- 
Andreas Schwab, SuSE Labs, address@hidden
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."