Changes in kernel/arch/ia32/src/smp/apic.c [0f17bff:b2fa1204] in mainline

File:

• kernel/arch/ia32/src/smp/apic.c (modified) (3 diffs)

kernel/arch/ia32/src/smp/apic.c

@@ -72 +72 @@
  *
  */
-volatile uint32_t *l_apic = (uint32_t *) L_APIC_BASE;
-volatile uint32_t *io_apic = (uint32_t *) IO_APIC_BASE;
+volatile uint32_t *l_apic = (uint32_t *) UINT32_C(0xfee00000);
+volatile uint32_t *io_apic = (uint32_t *) UINT32_C(0xfec00000);
 
 uint32_t apic_id_mask = 0;
@@ -264 +264 @@
 }
 
-/* Waits for the destination cpu to accept the previous ipi. */
+#define DELIVS_PENDING_SILENT_RETRIES   4       
+
 static void l_apic_wait_for_delivery(void)
 {
         icr_t icr;
-        
+        unsigned retries = 0;
+
         do {
+                if (retries++ > DELIVS_PENDING_SILENT_RETRIES) {
+                        retries = 0;
+#ifdef CONFIG_DEBUG
+                        log(LF_ARCH, LVL_DEBUG, "IPI is pending.");
+#endif
+                        delay(20);
+                }
                 icr.lo = l_apic[ICRlo];
-        } while (icr.delivs != DELIVS_IDLE);
-}
-
-/** Send one CPU an IPI vector.
- *
- * @param apicid Physical APIC ID of the destination CPU.
+        } while (icr.delivs == DELIVS_PENDING);
+        
+}
+
+/** Send all CPUs excluding CPU IPI vector.
+ *
  * @param vector Interrupt vector to be sent.
  *
  * @return 0 on failure, 1 on success.
- */
-int l_apic_send_custom_ipi(uint8_t apicid, uint8_t vector)
+ *
+ */
+int l_apic_broadcast_custom_ipi(uint8_t vector)
 {
         icr_t icr;
-
-        /* Wait for a destination cpu to accept our previous ipi. */
-        l_apic_wait_for_delivery();
-        
-        icr.lo = l_apic[ICRlo];
-        icr.hi = l_apic[ICRhi];
-        
-        icr.delmod = DELMOD_FIXED;
-        icr.destmod = DESTMOD_PHYS;
-        icr.level = LEVEL_ASSERT;
-        icr.shorthand = SHORTHAND_NONE;
-        icr.trigger_mode = TRIGMOD_LEVEL;
-        icr.vector = vector;
-        icr.dest = apicid;
-
-        /* Send the IPI by writing to l_apic[ICRlo]. */
-        l_apic[ICRhi] = icr.hi;
-        l_apic[ICRlo] = icr.lo;
-        
-        return apic_poll_errors();
-}
-
-/** Send all CPUs excluding CPU IPI vector.
- *
- * @param vector Interrupt vector to be sent.
- *
- * @return 0 on failure, 1 on success.
- *
- */
-int l_apic_broadcast_custom_ipi(uint8_t vector)
-{
-        icr_t icr;
-
-        /* Wait for a destination cpu to accept our previous ipi. */
-        l_apic_wait_for_delivery();
         
         icr.lo = l_apic[ICRlo];
@@ -329 +304 @@
         
         l_apic[ICRlo] = icr.lo;
+
+        l_apic_wait_for_delivery();
         
         return apic_poll_errors();