This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

PATCH : H8300 GDB simulator


Hi,

The CPU can access word data and longword data in memory, but word or longword data must begin at an even address. If an attempt is made to access word or longword data at an odd address, no address error occurs but the least significant bit of the address is regarded as 0, so the access starts at the preceding address. This is the case in actual H/W.

Now if an odd address is assigned to any word or long word memory pointer in H8 simulator and then if data at that address is accessed byte by byte then simulator was neither showing correct data (which is the case in actual H/W) not generating SIGBUS exception (which is the case in SH).

Patch below fixes this problem and after patch, simulator shows correct value as is the case in actual H/W.

ChangeLog/sim/h8300

2003-12-03  Anil Paranjpe <anilp1@kpitcummins.com>

        * compile.c : In all word and double word memory operations, 
			    address is forced to be always even.

Patch for /sim/h8300/compile.c

--- compile.c.orig	Fri Oct 17 18:15:56 2003
+++ compile.c	Wed Dec  3 22:04:22 2003
@@ -1265,20 +1265,19 @@
 
 #define GET_MEMORY_L(X) \
   ((X) < memory_size \
-   ? ((h8_get_memory (sd, (X)+0) << 24) | (h8_get_memory (sd, (X)+1) << 16)  \
-    | (h8_get_memory (sd, (X)+2) <<  8) | (h8_get_memory (sd, (X)+3) <<  0)) \
-   : ((h8_get_eightbit (sd, ((X)+0) & 0xff) << 24) \
-    | (h8_get_eightbit (sd, ((X)+1) & 0xff) << 16) \
-    | (h8_get_eightbit (sd, ((X)+2) & 0xff) <<  8) \
-    | (h8_get_eightbit (sd, ((X)+3) & 0xff) <<  0)))
+   ? ((h8_get_memory (sd, ((X) & 0xfffffffe)+0) << 24) | (h8_get_memory (sd, ((X) & 0xfffffffe)+1) << 16)  \
+    | (h8_get_memory (sd, ((X) & 0xfffffffe)+2) <<  8) | (h8_get_memory (sd, ((X) & 0xfffffffe)+3) <<  0)) \
+   : ((h8_get_eightbit (sd, (((X) & 0xfffffffe)+0) & 0xff) << 24) \
+    | (h8_get_eightbit (sd, (((X) & 0xfffffffe)+1) & 0xff) << 16) \
+    | (h8_get_eightbit (sd, (((X) & 0xfffffffe)+2) & 0xff) <<  8) \
+    | (h8_get_eightbit (sd, (((X) & 0xfffffffe)+3) & 0xff) <<  0)))
 
 #define GET_MEMORY_W(X) \
   ((X) < memory_size \
-   ? ((h8_get_memory   (sd, (X)+0) << 8) \
-    | (h8_get_memory   (sd, (X)+1) << 0)) \
-   : ((h8_get_eightbit (sd, ((X)+0) & 0xff) << 8) \
-    | (h8_get_eightbit (sd, ((X)+1) & 0xff) << 0)))
-
+   ? ((h8_get_memory   (sd, ((X) & 0xfffffffe)+0) << 8) \
+    | (h8_get_memory   (sd, ((X) & 0xfffffffe)+1) << 0)) \
+   : ((h8_get_eightbit (sd, (((X) & 0xfffffffe)+0) & 0xff) << 8) \
+    | (h8_get_eightbit (sd, (((X) & 0xfffffffe)+1) & 0xff) << 0)))
 
 #define GET_MEMORY_B(X) \
   ((X) < memory_size ? (h8_get_memory   (sd, (X))) \
@@ -1286,16 +1285,16 @@
 
 #define SET_MEMORY_L(X, Y)  \
 {  register unsigned char *_p; register int __y = (Y); \
-   _p = ((X) < memory_size ? h8_get_memory_buf   (sd) +  (X) : \
-                             h8_get_eightbit_buf (sd) + ((X) & 0xff)); \
+    _p = ((X) < memory_size ? h8_get_memory_buf   (sd) +  ((X) & 0xfffffffe) : \
+                             h8_get_eightbit_buf (sd) + (((X) & 0xfffffffe) & 0xff)); \
    _p[0] = __y >> 24; _p[1] = __y >> 16; \
    _p[2] = __y >>  8; _p[3] = __y >>  0; \
 }
 
 #define SET_MEMORY_W(X, Y) \
 {  register unsigned char *_p; register int __y = (Y); \
-   _p = ((X) < memory_size ? h8_get_memory_buf   (sd) +  (X) : \
-                             h8_get_eightbit_buf (sd) + ((X) & 0xff)); \
+   _p = ((X) < memory_size ? h8_get_memory_buf   (sd) +  ((X) & 0xfffffffe) : \
+                             h8_get_eightbit_buf (sd) + (((X) & 0xfffffffe) & 0xff)); \
    _p[0] = __y >> 8; _p[1] = __y; \
 }
 

Regards,
Anil Paranjpe


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]