Changeset 7a0359b in mainline for kernel/arch/ia32/include/interrupt.h

Timestamp:

2010-07-02T15:42:19Z (14 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

bbfdf62

Parents:

e3ee9b9

Message:

improve kernel function tracing

• add support for more generic kernel sources
• replace attribute((no_instrument_function)) with NO_TRACE macro (shorter and for future compatibility with different compilers)
• to be on the safe side, do not instrument most of the inline and static functions (plus some specific non-static functions)

collateral code cleanup (no change in functionality)

File:

• kernel/arch/ia32/include/interrupt.h (modified) (3 diffs)

kernel/arch/ia32/include/interrupt.h

@@ -38 +38 @@
 #include <typedefs.h>
 #include <arch/pm.h>
+#include <trace.h>
 
-#define IVT_ITEMS       IDT_ITEMS
-#define IVT_FIRST       0
+#define IVT_ITEMS  IDT_ITEMS
+#define IVT_FIRST  0
 
-#define EXC_COUNT       32
-#define IRQ_COUNT       16
+#define EXC_COUNT  32
+#define IRQ_COUNT  16
 
-#define IVT_EXCBASE     0
-#define IVT_IRQBASE     (IVT_EXCBASE + EXC_COUNT)
-#define IVT_FREEBASE    (IVT_IRQBASE + IRQ_COUNT)
+#define IVT_EXCBASE   0
+#define IVT_IRQBASE   (IVT_EXCBASE + EXC_COUNT)
+#define IVT_FREEBASE  (IVT_IRQBASE + IRQ_COUNT)
 
-#define IRQ_CLK         0
-#define IRQ_KBD         1
-#define IRQ_PIC1        2
-#define IRQ_PIC_SPUR    7
-#define IRQ_MOUSE       12
-#define IRQ_DP8390      9
+#define IRQ_CLK       0
+#define IRQ_KBD       1
+#define IRQ_PIC1      2
+#define IRQ_PIC_SPUR  7
+#define IRQ_MOUSE     12
+#define IRQ_DP8390    9
 
-/* this one must have four least significant bits set to ones */
-#define VECTOR_APIC_SPUR        (IVT_ITEMS - 1)
+/* This one must have four least significant bits set to ones */
+#define VECTOR_APIC_SPUR  (IVT_ITEMS - 1)
 
 #if (((VECTOR_APIC_SPUR + 1) % 16) || VECTOR_APIC_SPUR >= IVT_ITEMS)
@@ -63 +64 @@
 #endif
 
-#define VECTOR_DEBUG                    1
-#define VECTOR_CLK                      (IVT_IRQBASE + IRQ_CLK)
-#define VECTOR_PIC_SPUR                 (IVT_IRQBASE + IRQ_PIC_SPUR)
-#define VECTOR_SYSCALL                  IVT_FREEBASE
-#define VECTOR_TLB_SHOOTDOWN_IPI        (IVT_FREEBASE + 1)
-#define VECTOR_DEBUG_IPI                (IVT_FREEBASE + 2)
+#define VECTOR_DEBUG              1
+#define VECTOR_CLK                (IVT_IRQBASE + IRQ_CLK)
+#define VECTOR_PIC_SPUR           (IVT_IRQBASE + IRQ_PIC_SPUR)
+#define VECTOR_SYSCALL            IVT_FREEBASE
+#define VECTOR_TLB_SHOOTDOWN_IPI  (IVT_FREEBASE + 1)
+#define VECTOR_DEBUG_IPI          (IVT_FREEBASE + 2)
 
 typedef struct istate {
@@ -79 +80 @@
         uint32_t ebp;
         
-        uint32_t ebp_frame;     /* imitation of frame pointer linkage */
-        uint32_t eip_frame;     /* imitation of return address linkage */
-
+        uint32_t ebp_frame;  /* imitation of frame pointer linkage */
+        uint32_t eip_frame;  /* imitation of return address linkage */
+        
         uint32_t gs;
         uint32_t fs;
         uint32_t es;
         uint32_t ds;
-
-        uint32_t error_word;    /* real or fake error word */
+        
+        uint32_t error_word;  /* real or fake error word */
         uint32_t eip;
         uint32_t cs;
         uint32_t eflags;
-        uint32_t esp;           /* only if istate_t is from uspace */
-        uint32_t ss;            /* only if istate_t is from uspace */
+        uint32_t esp;         /* only if istate_t is from uspace */
+        uint32_t ss;          /* only if istate_t is from uspace */
 } istate_t;
 
 /** Return true if exception happened while in userspace */
-static inline int istate_from_uspace(istate_t *istate)
+NO_TRACE static inline int istate_from_uspace(istate_t *istate)
 {
         return !(istate->eip & 0x80000000);
 }
 
-static inline void istate_set_retaddr(istate_t *istate, uintptr_t retaddr)
+NO_TRACE static inline void istate_set_retaddr(istate_t *istate,
+    uintptr_t retaddr)
 {
         istate->eip = retaddr;
 }
 
-static inline unative_t istate_get_pc(istate_t *istate)
+NO_TRACE static inline unative_t istate_get_pc(istate_t *istate)
 {
         return istate->eip;
 }
 
-static inline unative_t istate_get_fp(istate_t *istate)
+NO_TRACE static inline unative_t istate_get_fp(istate_t *istate)
 {
         return istate->ebp;
 }
 
-extern void (* disable_irqs_function)(uint16_t irqmask);
-extern void (* enable_irqs_function)(uint16_t irqmask);
+extern void (* disable_irqs_function)(uint16_t);
+extern void (* enable_irqs_function)(uint16_t);
 extern void (* eoi_function)(void);
 
 extern void interrupt_init(void);
-extern void trap_virtual_enable_irqs(uint16_t irqmask);
-extern void trap_virtual_disable_irqs(uint16_t irqmask);
+extern void trap_virtual_enable_irqs(uint16_t);
+extern void trap_virtual_disable_irqs(uint16_t);
 
 #endif