Changeset 33540 in vbox for trunk/src/recompiler

Timestamp:

Oct 28, 2010 9:27:05 AM (15 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

67140

Message:

*: spelling fixes, thanks Timeless!

Location:

trunk/src/recompiler

Files:

• Makefile.kmk (modified) (1 diff)
• Sun/e_powl-amd64.S (modified) (1 diff)
• Sun/testmath.c (modified) (2 diffs)
• VBoxREMWrapper.cpp (modified) (3 diffs)
• VBoxRecompiler.c (modified) (7 diffs)
• a.out.h (modified) (1 diff)
• cpu-all.h (modified) (2 diffs)
• cpu-defs.h (modified) (1 diff)
• cpu-exec.c (modified) (3 diffs)
• dyngen-exec.h (modified) (1 diff)
• elf.h (modified) (1 diff)
• exec-all.h (modified) (1 diff)
• exec.c (modified) (6 diffs)
• hostregs_helper.h (modified) (2 diffs)
• qemu-lock.h (modified) (1 diff)
• softmmu_template.h (modified) (1 diff)
• target-i386/cpu.h (modified) (2 diffs)
• target-i386/exec.h (modified) (1 diff)
• target-i386/op_helper.c (modified) (6 diffs)
• target-i386/translate.c (modified) (2 diffs)
• tcg/README (modified) (1 diff)
• tcg/i386/tcg-target.c (modified) (1 diff)
• tcg/tcg.c (modified) (3 diffs)
• tcg/tcg.h (modified) (2 diffs)
• tests/qruncom.c (modified) (1 diff)
• tests/test-i386.c (modified) (1 diff)
• vl.h (modified) (2 diffs)

trunk/src/recompiler/Makefile.kmk

@@ -38 +38 @@
 #
 # This is extended by one of the VBoxREM* modules below.  Currently, this
-# isn't done by inheritance because of some obscure bug wrt inherting from
+# isn't done by inheritance because of some obscure bug wrt inheriting from
 # unused targets that I'm not going to fix now.
 #

trunk/src/recompiler/Sun/e_powl-amd64.S

@@ -1 +1 @@
 /* ix87 specific implementation of pow function.
-   Copyright (C) 1996, 1997, 1998, 1999, 2001, 2004 Free Software Founda
+   Copyright (C) 1996, 1997, 1998, 1999, 2001, 2004 Free Software Foundation
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <[email protected]>, 1996.

trunk/src/recompiler/Sun/testmath.c

@@ -278 +278 @@
 
 
-    /* Only works in extended precision, while double precesion is defaul on BSD (including Darwin) */
+    /* Only works in extended precision, while double precision is default on BSD (including Darwin) */
     set_cw(0x37f);
     CHECK(rintl(1.0L), 1.0);
@@ -419 +419 @@
 static inline void fpop(void)
 {
-    env->fptags[env->fpstt] = 1; /* invvalidate stack entry */
+    env->fptags[env->fpstt] = 1; /* invalidate stack entry */
     env->fpstt = (env->fpstt + 1) & 7;
 }

trunk/src/recompiler/VBoxREMWrapper.cpp

@@ -28 +28 @@
  * On WIN64 we're not aware of any GCC port which can emit code using the MSC
  * calling convention. So, we're in for some real fun here. The choice is between
- * porting GCC to AMD64 WIN64 and comming up with some kind of wrapper around
+ * porting GCC to AMD64 WIN64 and coming up with some kind of wrapper around
  * either the win32 build or the 64-bit linux build.
  *
@@ -112 +112 @@
  *
  * Direction flag is preserved as cleared.
- * The stack must be aligned on a 16-byte boundrary before the 'call/jmp' instruction.
+ * The stack must be aligned on a 16-byte boundary before the 'call/jmp' instruction.
  *
  *
@@ -140 +140 @@
  *
  * Dunno what the direction flag is...
- * The stack must be aligned on a 16-byte boundrary before the 'call/jmp' instruction.
+ * The stack must be aligned on a 16-byte boundary before the 'call/jmp' instruction.
  *
  *

trunk/src/recompiler/VBoxRecompiler.c

@@ -766 +766 @@
 
     /*
-     * Stop ignoring ignornable notifications.
+     * Stop ignoring ignorable notifications.
      */
     ASMAtomicDecU32(&pVM->rem.s.cIgnoreAll);
@@ -1296 +1296 @@
      * or 32 bits protected mode ring 0 code
      *
-     * The tests are ordered by the likelyhood of being true during normal execution.
+     * The tests are ordered by the likelihood of being true during normal execution.
      */
     if (fFlags & (HF_TF_MASK | HF_INHIBIT_IRQ_MASK))
@@ -1694 +1694 @@
 
 /**
- * Record trap occurance
+ * Record trap occurrence
  *
  * @returns VBox status code
@@ -1793 +1793 @@
  * @remark  The caller has to check for important FFs before calling REMR3Run. REMR3State will
  *          no do this since the majority of the callers don't want any unnecessary of events
- *          pending that would immediatly interrupt execution.
+ *          pending that would immediately interrupt execution.
  */
 REMR3DECL(int)  REMR3State(PVM pVM, PVMCPU pVCpu)
@@ -3790 +3790 @@
                 DISAS_PRINTF("disas error\n");
                 cbInstr = 1;
-#ifdef RT_ARCH_AMD64 /** @todo remove when DISInstr starts supporing 64-bit code. */
+#ifdef RT_ARCH_AMD64 /** @todo remove when DISInstr starts supporting 64-bit code. */
                 break;
 #endif
@@ -3971 +3971 @@
  * @param   pVCpuDst        The target cpu for this notification.
  *                          TM will not broadcast pending timer events, but use
- *                          a decidated EMT for them. So, only interrupt REM
+ *                          a dedicated EMT for them. So, only interrupt REM
  *                          execution if the given CPU is executing in REM.
  * @thread  Any.
@@ -4499 +4499 @@
  *
  * @param   rc      VBox error code.
- * @param   pszTip  Hint about why/when this happend.
+ * @param   pszTip  Hint about why/when this happened.
  */
 void remAbort(int rc, const char *pszTip)

trunk/src/recompiler/a.out.h

@@ -298 +298 @@
 
 #define IMAGE_SCN_LNK_NRELOC_OVFL            0x01000000  /* Section contains extended relocations. */
-#define IMAGE_SCN_MEM_NOT_CACHED             0x04000000  /* Section is not cachable.               */
+#define IMAGE_SCN_MEM_NOT_CACHED             0x04000000  /* Section is not cacheable.              */
 #define IMAGE_SCN_MEM_NOT_PAGED              0x08000000  /* Section is not pageable.               */
 #define IMAGE_SCN_MEM_SHARED                 0x10000000  /* Section is shareable.                  */

trunk/src/recompiler/cpu-all.h

@@ -1088 +1088 @@
 #define CPU_INTERRUPT_HALT   0x20 /* CPU halt wanted */
 #define CPU_INTERRUPT_SMI    0x40 /* (x86 only) SMI interrupt pending */
-#define CPU_INTERRUPT_DEBUG  0x80 /* Debug event occured.  */
+#define CPU_INTERRUPT_DEBUG  0x80 /* Debug event occurred.  */
 #define CPU_INTERRUPT_VIRQ   0x100 /* virtual interrupt pending.  */
 #define CPU_INTERRUPT_NMI    0x200 /* NMI pending. */
@@ -1193 +1193 @@
 /* MMIO pages are identified by a combination of an IO device index and
    3 flags.  The ROMD code stores the page ram offset in iotlb entry,
-   so only a limited number of ids are avaiable.  */
+   so only a limited number of ids are available.  */
 
 #define IO_MEM_SHIFT       3

trunk/src/recompiler/cpu-defs.h

@@ -130 +130 @@
     target_ulong addr_code;
       /* Addend to virtual address to get physical address.  IO accesses
-       use the correcponding iotlb value.  */
+       use the corresponding iotlb value.  */
 #if TARGET_PHYS_ADDR_BITS == 64
     /* on i386 Linux make sure it is aligned */

trunk/src/recompiler/cpu-exec.c

@@ -350 +350 @@
 #endif
                 {
-                    /** @todo: reconscille with what QEMU really does */
+                    /** @todo: reconcile with what QEMU really does */
 
                     /* Single instruction exec request, we execute it and return (one way or the other).
@@ -803 +803 @@
                        CPU tries to execute code at the magic address.
                        This will cause the magic PC value to be pushed to
-                       the stack if an interrupt occured at the wrong time.
+                       the stack if an interrupt occurred at the wrong time.
                        We avoid this by disabling interrupts when
                        pc contains a magic address.  */
@@ -850 +850 @@
                     }
 #endif
-                   /* Don't use the cached interupt_request value,
+                   /* Don't use the cached interrupt_request value,
                       do_interrupt may have updated the EXITTB flag. */
                     if (env->interrupt_request & CPU_INTERRUPT_EXITTB) {

trunk/src/recompiler/dyngen-exec.h

@@ -60 +60 @@
 
 /* if Solaris/__sun__, don't typedef int8_t, as it will be typedef'd
-   prior to this and will cause an error in compliation, conflicting
+   prior to this and will cause an error in compilation, conflicting
    with /usr/include/sys/int_types.h, line 75 */
 #ifndef __sun__

trunk/src/recompiler/elf.h

@@ -639 +639 @@
 #define EFA_PARISC_2_0              0x0214 /* PA-RISC 2.0 big-endian.  */
 
-/* Additional section indeces.  */
-
-#define SHN_PARISC_ANSI_COMMON  0xff00     /* Section for tenatively declared
+/* Additional section indices.  */
+
+#define SHN_PARISC_ANSI_COMMON  0xff00     /* Section for tentatively declared
                                               symbols in ANSI C.  */
 #define SHN_PARISC_HUGE_COMMON  0xff01     /* Common blocks in huge model.  */

trunk/src/recompiler/exec-all.h

@@ -60 +60 @@
 
 /* Maximum size a TCG op can expand to.  This is complicated because a
-   single op may require several host instructions and regirster reloads.
+   single op may require several host instructions and register reloads.
    For now take a wild guess at 128 bytes, which should allow at least
    a couple of fixup instructions per argument.  */

trunk/src/recompiler/exec.c

@@ -524 +524 @@
 
 #if defined(CONFIG_USER_ONLY)
-/* Currently it is not recommanded to allocate big chunks of data in
+/* Currently it is not recommended to allocate big chunks of data in
    user mode. It will change when a dedicated libc will be used */
 #define USE_STATIC_CODE_GEN_BUFFER
@@ -556 +556 @@
         code_gen_buffer_size = DEFAULT_CODE_GEN_BUFFER_SIZE;
 #else
-        /* XXX: needs ajustments */
+        /* XXX: needs adjustments */
         code_gen_buffer_size = (unsigned long)(phys_ram_size / 4);
 #endif
@@ -1697 +1697 @@
     if (env->singlestep_enabled != enabled) {
         env->singlestep_enabled = enabled;
-        /* must flush all the translated code to avoid inconsistancies */
+        /* must flush all the translated code to avoid inconsistencies */
         /* XXX: only flush what is necessary */
         tb_flush(env);
@@ -2393 +2393 @@
 #if 0
 /* called from signal handler: invalidate the code and unprotect the
-   page. Return TRUE if the fault was succesfully handled. */
+   page. Return TRUE if the fault was successfully handled. */
 int page_unprotect(target_ulong addr, unsigned long pc, void *puc)
 {
@@ -2514 +2514 @@
 
 /* modify the flags of a page and invalidate the code if
-   necessary. The flag PAGE_WRITE_ORG is positionned automatically
+   necessary. The flag PAGE_WRITE_ORG is positioned automatically
    depending on PAGE_WRITE */
 void page_set_flags(target_ulong start, target_ulong end, int flags)
@@ -2580 +2580 @@
 
 /* called from signal handler: invalidate the code and unprotect the
-   page. Return TRUE if the fault was succesfully handled. */
+   page. Return TRUE if the fault was successfully handled. */
 int page_unprotect(target_ulong address, unsigned long pc, void *puc)
 {

trunk/src/recompiler/hostregs_helper.h

@@ -1 +1 @@
 /*
- *  Save/restore host registrs.
+ *  Save/restore host registers.
  *
  *  Copyright (c) 2007 CodeSourcery
@@ -28 +28 @@
  */
 
-/* The GCC global register vairable extension is used to reserve some
+/* The GCC global register variable extension is used to reserve some
    host registers for use by dyngen.  However only the core parts of the
    translation engine are compiled with these settings.  We must manually

trunk/src/recompiler/qemu-lock.h

@@ -30 +30 @@
    pthread mutexes, and non-NPTL userspace isn't threadsafe anyway.
    In either case a spinlock is probably the wrong kind of lock.
-   Spinlocks are only good if you know annother CPU has the lock and is
+   Spinlocks are only good if you know another CPU has the lock and is
    likely to release it soon.  In environments where you have more threads
    than physical CPUs (the extreme case being a single CPU host) a spinlock

trunk/src/recompiler/softmmu_template.h

@@ -114 +114 @@
 /* Load helpers invoked from generated code, and TCG makes an assumption
    that valid value takes the whole register, why gcc after 4.3 may
-   use only lower part of register for smaller types. So force promoution. */
+   use only lower part of register for smaller types. So force promotion. */
 DATA_TYPE_PROMOTED REGPARM
 glue(glue(__ld, SUFFIX), MMUSUFFIX)(target_ulong addr,

trunk/src/recompiler/target-i386/cpu.h

@@ -136 +136 @@
 #define ID_MASK                 0x00200000
 
-/* hidden flags - used internally by qemu to represent additionnal cpu
+/* hidden flags - used internally by qemu to represent additional cpu
    states. Only the CPL, INHIBIT_IRQ, SMM and SVMI are not redundant. We avoid
    using the IOPL_MASK, TF_MASK and VM_MASK bit position to ease oring
@@ -254 +254 @@
 #define MSR_IA32_APICBASE_BASE          (0xfffff<<12)
 
-#ifndef MSR_IA32_SYSENTER_CS /* VBox x86.h klugde */
+#ifndef MSR_IA32_SYSENTER_CS /* VBox x86.h kludge */
 #define MSR_IA32_SYSENTER_CS            0x174
 #define MSR_IA32_SYSENTER_ESP           0x175

trunk/src/recompiler/target-i386/exec.h

@@ -365 +365 @@
 #endif
 {
-    env->fptags[env->fpstt] = 1; /* invvalidate stack entry */
+    env->fptags[env->fpstt] = 1; /* invalidate stack entry */
     env->fpstt = (env->fpstt + 1) & 7;
 }

trunk/src/recompiler/target-i386/op_helper.c

@@ -1030 +1030 @@
      /*
      * We must clear VIP/VIF too on interrupt entry, as otherwise FreeBSD
-     * gets confused by seeingingly changed EFLAGS. See #3491 and
+     * gets confused by seemingly changed EFLAGS. See #3491 and
      * public bug #2341.
      */
@@ -1306 +1306 @@
     /*
      * We must clear VIP/VIF too on interrupt entry, as otherwise FreeBSD
-     * gets confused by seeingingly changed EFLAGS. See #3491 and
+     * gets confused by seemingly changed EFLAGS. See #3491 and
      * public bug #2341.
      */
@@ -5611 +5611 @@
  * Reads byte from virtual address in guest memory area.
  * XXX: is it working for any addresses? swapped out pages?
- * @returns readed data byte.
+ * @returns read data byte.
  * @param   env1    CPU environment.
  * @param   pvAddr  GC Virtual address.
@@ -5628 +5628 @@
  * Reads byte from virtual address in guest memory area.
  * XXX: is it working for any addresses? swapped out pages?
- * @returns readed data byte.
+ * @returns read data byte.
  * @param   env1    CPU environment.
  * @param   pvAddr  GC Virtual address.
@@ -5645 +5645 @@
  * Reads byte from virtual address in guest memory area.
  * XXX: is it working for any addresses? swapped out pages?
- * @returns readed data byte.
+ * @returns read data byte.
  * @param   env1    CPU environment.
  * @param   pvAddr  GC Virtual address.
@@ -5662 +5662 @@
  * Writes byte to virtual address in guest memory area.
  * XXX: is it working for any addresses? swapped out pages?
- * @returns readed data byte.
+ * @returns read data byte.
  * @param   env1    CPU environment.
  * @param   pvAddr  GC Virtual address.

trunk/src/recompiler/target-i386/translate.c

@@ -1346 +1346 @@
 
 /* generate a conditional jump to label 'l1' according to jump opcode
-   value 'b'. In the fast case, T0 is guaranted not to be used. */
+   value 'b'. In the fast case, T0 is guaranteed not to be used. */
 #ifndef VBOX
 static inline void gen_jcc1(DisasContext *s, int cc_op, int b, int l1)
@@ -2764 +2764 @@
     } else {
         /* slow case: it is more efficient not to generate a jump,
-           although it is questionnable whether this optimization is
+           although it is questionable whether this optimization is
            worth to */
         inv = b & 1;

trunk/src/recompiler/tcg/README

@@ -425 +425 @@
 
 - Don't hesitate to use helpers for complicated or seldom used target
-  intructions. There is little performance advantage in using TCG to
+  instructions. There is little performance advantage in using TCG to
   implement target instructions taking more than about twenty TCG
   instructions.

trunk/src/recompiler/tcg/i386/tcg-target.c

@@ -571 +571 @@
     int useReg2 = ((index & 3) == 3);
 
-    /** @todo:  should we make phys addess accessors fastcalls - probably not a big deal */
+    /** @todo:  should we make phys address accessors fastcalls - probably not a big deal */
     /* out parameter (address), note that phys address is always 64-bit */
     AssertMsg(sizeof(RTGCPHYS) == 8, ("Physical address must be 64-bits, update caller\n"));

trunk/src/recompiler/tcg/tcg.c

@@ -47 +47 @@
 #include "qemu-common.h"
 
-/* Note: the long term plan is to reduce the dependancies on the QEMU
+/* Note: the long term plan is to reduce the dependencies on the QEMU
    CPU definitions. Currently they are used for qemu_ld/st
    instructions */
@@ -61 +61 @@
 /*
  * Liveness analysis doesn't work well with 32-bit hosts and 64-bit targets,
- * second element of the register pair to store 64-bit value is consedered
+ * second element of the register pair to store 64-bit value is considered
  * dead, it seems.
  * @todo: fix it in compiler
@@ -1465 +1465 @@
 }
 
-/* save globals to their cannonical location and assume they can be
+/* save globals to their canonical location and assume they can be
    modified be the following code. 'allocated_regs' is used in case a
    temporary registers needs to be allocated to store a constant. */

trunk/src/recompiler/tcg/tcg.h

@@ -114 +114 @@
 typedef tcg_target_ulong TCGArg;
 
-/* Define a type and accessor macros for varables.  Using a struct is
+/* Define a type and accessor macros for variables.  Using a struct is
    nice because it gives some level of type safely.  Ideally the compiler
    be able to see through all this.  However in practice this is not true,
@@ -198 +198 @@
     unsigned int mem_coherent:1;
     unsigned int mem_allocated:1;
-    unsigned int temp_local:1; /* If true, the temp is saved accross
+    unsigned int temp_local:1; /* If true, the temp is saved across
                                   basic blocks. Otherwise, it is not
-                                  preserved accross basic blocks. */
+                                  preserved across basic blocks. */
     unsigned int temp_allocated:1; /* never used for code gen */
     /* index of next free temp of same base type, -1 if end */

trunk/src/recompiler/tests/qruncom.c

@@ -275 +275 @@
     env->regs[R_EDI] = 0xfffe;
 
-    /* inform the emulator of the mmaped memory */
+    /* inform the emulator of the mapped memory */
     page_set_flags(0x00000000, 0x110000, 
                    PAGE_WRITE | PAGE_READ | PAGE_EXEC | PAGE_VALID);

trunk/src/recompiler/tests/test-i386.c

@@ -2210 +2210 @@
 
 /* Force %xmm0 usage to avoid the case where both register index are 0
-   to test intruction decoding more extensively */
+   to test instruction decoding more extensively */
 #define CVT_OP_XMM2MMX(op)\
 {\

trunk/src/recompiler/vl.h

@@ -573 +573 @@
     /* the following fields are informative. They are not needed for
        the consistency of the snapshot */
-    char name[256]; /* user choosen name */
+    char name[256]; /* user chosen name */
     uint32_t vm_state_size; /* VM state info size */
     uint32_t date_sec; /* UTC date of the snapshot */
@@ -1285 +1285 @@
 
 int32_t scsi_send_command(SCSIDevice *s, uint32_t tag, uint8_t *buf, int lun);
-/* SCSI data transfers are asynchrnonous.  However, unlike the block IO
+/* SCSI data transfers are asynchronous.  However, unlike the block IO
    layer the completion routine may be called directly by
    scsi_{read,write}_data.  */