[tkreuzer] 53711: [HAL/APIC] - Fix some stuff that gcc complains about. It doesn't like to use static data in non-static inline functions for some unknown reason. - Implement support for level trig...

Author: tkreuzer
Date: Thu Sep 15 11:34:21 2011
New Revision: 53711

URL: http://svn.reactos.org/svn/reactos?rev=53711&view=rev
Log:
[HAL/APIC]
- Fix some stuff that gcc complains about. It doesn't like to use static data in
non-static inline functions for some unknown reason.
- Implement support for level triggered interrupts, when using lazy irql

Modified:
    trunk/reactos/hal/halx86/apic/apic.c
    trunk/reactos/hal/halx86/apic/tsc.c

Modified: trunk/reactos/hal/halx86/apic/apic.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/hal/halx86/apic/apic.c?rev…
==============================================================================
--- trunk/reactos/hal/halx86/apic/apic.c [iso-8859-1] (original)
+++ trunk/reactos/hal/halx86/apic/apic.c [iso-8859-1] Thu Sep 15 11:34:21 2011
@@ -28,7 +28,7 @@
 UCHAR HalpVectorToIndex[256];
 
 #ifndef _M_AMD64
-static const UCHAR
+const UCHAR
 HalpIRQLtoTPR[32] =
 {
     0x00, /*  0 PASSIVE_LEVEL */
@@ -65,7 +65,7 @@
     0xff, /* 31 HIGH_LEVEL */
 };
 
-static const KIRQL
+const KIRQL
 HalVectorToIRQL[16] =
 {
        0, /* 00 PASSIVE_LEVEL */
@@ -132,7 +132,7 @@
 
 VOID
 FORCEINLINE
-ApicRequestInterrupt(IN UCHAR Vector)
+ApicRequestInterrupt(IN UCHAR Vector, UCHAR TriggerMode)
 {
     APIC_COMMAND_REGISTER CommandRegister;
 
@@ -140,7 +140,7 @@
     CommandRegister.Long0 = 0;
     CommandRegister.Vector = Vector;
     CommandRegister.MessageType = APIC_MT_Fixed;
-    CommandRegister.TriggerMode = APIC_TGM_Edge;
+    CommandRegister.TriggerMode = TriggerMode;
     CommandRegister.DestinationShortHand = APIC_DSH_Self;
 
     /* Write the low dword to send the interrupt */
@@ -653,7 +653,7 @@
 HalRequestSoftwareInterrupt(IN KIRQL Irql)
 {
     /* Convert irql to vector and request an interrupt */
-    ApicRequestInterrupt(IrqlToTpr(Irql));
+    ApicRequestInterrupt(IrqlToTpr(Irql), APIC_TGM_Edge);
 }
 
 VOID
@@ -700,15 +700,17 @@
         ReDirReg.DeliveryMode = APIC_MT_LowestPriority;
         ReDirReg.DestinationMode = APIC_DM_Logical;
         ReDirReg.Destination = 0;
-        ReDirReg.TriggerMode = 1 - InterruptMode;
     }
 
     /* Check if the destination is logical */
-    if (ReDirReg.DestinationMode = APIC_DM_Logical)
+    if (ReDirReg.DestinationMode == APIC_DM_Logical)
     {
         /* Set the bit for this cpu */
         ReDirReg.Destination |= ApicLogicalId(Prcb->Number);
     }
+
+    /* Set the trigger mode */
+    ReDirReg.TriggerMode = 1 - InterruptMode;
 
     /* Now unmask it */
     ReDirReg.Mask = FALSE;
@@ -758,6 +760,9 @@
     /* Check if this interrupt is allowed */
     if (CurrentIrql >= Irql)
     {
+        IOAPIC_REDIRECTION_REGISTER RedirReg;
+        UCHAR Index;
+
         /* It is not, set the real Irql in the TPR! */
         ApicWrite(APIC_TPR, IrqlToTpr(CurrentIrql));
 
@@ -767,9 +772,23 @@
         /* End this interrupt */
         ApicSendEOI();
 
-        // FIXME: level interrupts
-        /* Re-request the interrupt to be handled later */
-        ApicRequestInterrupt(Vector);
+        /* Get the irq for this vector */
+        Index = HalpVectorToIndex[Vector];
+
+        /* Check if its valid */
+        if (Index != 0xff)
+        {
+            /* Read the I/O redirection entry */
+            RedirReg = ApicReadIORedirectionEntry(Index);
+
+            /* Re-request the interrupt to be handled later */
+            ApicRequestInterrupt(Vector, (UCHAR)RedirReg.TriggerMode);
+       }
+       else
+       {
+            /* Re-request the interrupt to be handled later */
+            ApicRequestInterrupt(Vector, APIC_TGM_Edge);
+       }
 
         /* Pretend it was a spurious interrupt */
         return FALSE;

Modified: trunk/reactos/hal/halx86/apic/tsc.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/hal/halx86/apic/tsc.c?rev=…
==============================================================================
--- trunk/reactos/hal/halx86/apic/tsc.c [iso-8859-1] (original)
+++ trunk/reactos/hal/halx86/apic/tsc.c [iso-8859-1] Thu Sep 15 11:34:21 2011
@@ -14,7 +14,7 @@
 
 #include "tsc.h"
 
-LARGE_INTEGER HalpCpuClockFrequency = {INITIAL_STALL_COUNT * 1000000};
+LARGE_INTEGER HalpCpuClockFrequency = {{INITIAL_STALL_COUNT * 1000000}};
 
 UCHAR TscCalibrationPhase;
 ULONG64 TscCalibrationArray[NUM_SAMPLES];