Why does Keyboard interrupt is not working after initialization of bootstrap and application processor?

[gamingjam60]

I am using this two function to start bootstrap and ap along with apic_timer and keyboard but keyboard is not working

Code: Select all

void start_bootstrap_cpu_core() {
    uint32_t bsp_lapic_id = smp_request.response->bsp_lapic_id;
    asm volatile("cli");
    init_acpi();

    init_bootstrap_gdt_tss(bsp_lapic_id);
    init_bootstrap_interrupt(bsp_lapic_id);

    init_tsc();

    init_apic_interrupt();
    
    init_apic_timer(100);

    ioapic_route_irq(1, 0, 0x21); // Route IRQ 1 to LAPIC ID 0 with vector 0x20
    initKeyboard();

    asm volatile("sti");
    printf("Bootstrap CPU initialized...\n\n");
}

Code: Select all

void start_secondary_cpu_cores(int start_id, int end_id) {
    for (int core = start_id; core < smp_request.response->cpu_count; core++) {
        struct limine_smp_info *smp_info = smp_request.response->cpus[core];

        // Set function to execute on the AP
        smp_info->goto_address = (limine_goto_address) target_cpu_task;

        // passing sm_info as argument which will accept as input by target_cpu_task
        smp_info->extra_argument = (uint64_t)smp_info;
        
        // Short delay to let APs start (only for debugging)
        for (volatile int i = 0; i < 1000000; i++);

        printf("Initializing CPU %d (LAPIC ID %d)...\n\n", core, smp_info->lapic_id);
    }
}

Is this ioapic_route_irq correct?

Code: Select all

void ioapic_route_irq(uint8_t irq, uint8_t apic_id, uint8_t vector) {
    // Calculate the redirection table register index offset
    uint32_t index_low  = 0x10 + irq * 2; // Low dword index
    uint32_t index_high = index_low + 1;   // High dword index

    // Write to IOAPIC redirection table registers using the IOAPIC base address
    mmio_write(IOAPIC_BASE + index_high * 4, (apic_id << 24));
    mmio_write(IOAPIC_BASE + index_low * 4, vector);
}

The IOAPIC_BASE found from madt and LAPIC_BASE are showing same?

acpi.c
cpu.c
apic.c
keyboard
whole project

[Octocontrabass]

Code: Select all

        // Set function to execute on the AP
        smp_info->goto_address = (limine_goto_address) target_cpu_task;

        // passing sm_info as argument which will accept as input by target_cpu_task
        smp_info->extra_argument = (uint64_t)smp_info;

You need to set extra_argument before you set goto_address.

Is this ioapic_route_irq correct?

I don't see where you specify the polarity or trigger mode. I don't see where you handle the interrupt source overrides from the MADT.

The IOAPIC_BASE found from madt and LAPIC_BASE are showing same?

They can't be the same. Something must be wrong if you see the same base address for a local APIC and an IOAPIC.