fix: remove unwanted interrupts during disk init and probing process.

[lunaix-os.git] / lunaix-os / hal / ahci / ahci.c

diff --git a/lunaix-os/hal/ahci/ahci.c b/lunaix-os/hal/ahci/ahci.c
index 133513b049e096d74542c7f2ce9a20cff4cf4d17..ee521561da66fcdca484778002f4143fa1eb4a83 100644 (file)
--- a/lunaix-os/hal/ahci/ahci.c
+++ b/lunaix-os/hal/ahci/ahci.c
@@ -160,8 +160,6 @@ ahci_init()
         // 需要通过全部置位去清空这些寄存器（相当的奇怪……）
         port_regs[HBA_RPxSERR] = -1;
 
         // 需要通过全部置位去清空这些寄存器（相当的奇怪……）
         port_regs[HBA_RPxSERR] = -1;
 

-        port_regs[HBA_RPxIE] |= (HBA_PxINTR_D2HR);
-

         hba.ports[i] = port;
 
         if (!HBA_RPxSSTS_IF(port->ssts)) {
         hba.ports[i] = port;
 
         if (!HBA_RPxSSTS_IF(port->ssts)) {


@@ -186,7 +184,8 @@ char sata_ifs[][20] = { "Not detected",
 void
 __ahci_hba_isr(isr_param param)
 {
 void
 __ahci_hba_isr(isr_param param)
 {

-    // TODO: hba interrupt
+    // TODO: clear the interrupt status
+    // TODO: I/O-operation scheduler should be here

     kprintf(KDEBUG "HBA INTR\n");
 }
 
     kprintf(KDEBUG "HBA INTR\n");
 }
 


@@ -306,6 +305,9 @@ ahci_init_device(struct hba_port* port)
     struct hba_cmdt* cmd_table;
     struct hba_cmdh* cmd_header;
 
     struct hba_cmdt* cmd_table;
     struct hba_cmdh* cmd_header;
 

+    // mask DHR interrupt
+    port->regs[HBA_RPxIE] &= ~HBA_PxINTR_DHR;
+

     // 确保端口是空闲的
     wait_until(!(port->regs[HBA_RPxTFD] & (HBA_PxTFD_BSY)));
 
     // 确保端口是空闲的
     wait_until(!(port->regs[HBA_RPxTFD] & (HBA_PxTFD_BSY)));
 


@@ -398,6 +400,9 @@ ahci_init_device(struct hba_port* port)
     scsi_parse_capacity(port->device, (uint32_t*)data_in);
 
 done:
     scsi_parse_capacity(port->device, (uint32_t*)data_in);
 
 done:

+    // reset interrupt status and unmask D2HR interrupt
+    port->regs[HBA_RPxIS] = -1;
+    port->regs[HBA_RPxIE] |= HBA_PxINTR_DHR;

     achi_register_ops(port);
 
     vfree_dma(data_in);
     achi_register_ops(port);
 
     vfree_dma(data_in);


@@ -406,6 +411,8 @@ done:
     return 1;
 
 fail:
     return 1;
 
 fail:

+    port->regs[HBA_RPxIS] = -1;
+    port->regs[HBA_RPxIE] |= HBA_PxINTR_DHR;

     vfree_dma(data_in);
     vfree_dma(cmd_table);
 
     vfree_dma(data_in);
     vfree_dma(cmd_table);