Looks good.

Looks good

I also have a comment from jhb@:
@br typically we only route device interrupts to a single core since interrupt handlers generally assume they are not run concurrently (interrupt filters, ithreads will be single-threaded even if the interrupt is broadcast)
as Andy noted, you want to handle interrupt binding requests, but you also need to do something for the “default” case.
Not sure what INTRNG does, but on x86 we round-robin interrupts among CPUs when they are first setup.

In D21928#484096, @mhorne wrote:

In D21928#484089, @br wrote:

Why do we need to enable interrupts for all CPUs and not only current cpu?
where an interrupt will be signaled in that case?

PIC_ENABLE_INTR is called very seldomly, often only once while setting up the PIC during intr_setup_irq by the BSP. Since it is not called per-cpu, this function should handle enabling the interrupt for all active CPUs.
BBL sets all PLIC enable bits to 1 by default, whereas OpenSBI sets them to 0, which is how I noticed that APs weren't having their interrupts enabled.

looks good!

Why do we need to enable interrupts for all CPUs and not only current cpu?
where an interrupt will be signaled in that case?

Are these example of PLIC contexts?

interrupts-extended = <
				&cpu0_intc 0xffffffff
				&cpu1_intc 0xffffffff &cpu1_intc 9
				&cpu2_intc 0xffffffff &cpu2_intc 9
				&cpu3_intc 0xffffffff &cpu3_intc 9
				&cpu4_intc 0xffffffff &cpu4_intc 9>;

• Infer the tracing device based on compiler architecture (PT on amd64, Coresight on aarch64)
• Document all the pmctrace usage flags

I was looking at Linux driver before writing this one. Why don't they do the same ?

hart ID is recorded to pcpup->pc_hart. Could it be used ? i.e. PCPU_GET(hart) or pcpu_find(cpu)->pc_hart

Regenerate patch.

Regenerate patch.

Regenerate patch.

Regenerate patch.

Regenerate patch.

Regenerate patch.

Regenerate patch.

Regenerate patch.

Thanks!

I discussed with James Clarke jrtc27, it is fine

In D21439#469311, @andrew wrote:

Can you update the psci code to use the arm_smccc_* functions? We should only have one copy the functions to make smccc calls and you can consider psci to be based on it.

Reuse generic SMCCC for PSCI.

Check if device is opened before writing to it or closing it. This could happen if user process forks.

address Andy's comments

• Assert that hart < MAXCPU
• Rename cpu variable to hart

Fix bug: use CPU_FILL to fill up kernel pmap pm_active.
all_harts is not initialized yet in pmap_boostrap()

Use all_harts mask in release_aps()

1. Store hart in struct pcpu
2. Store harts not CPUIDs in pmap->pm_active

Fix !SMP build.