Page MenuHomeFreeBSD
Differential D43452

riscv: Introduce support for Incoming MSI Controller (IMSIC)
Needs ReviewPublic
Actions

Authored by himanshu_thechauhan.dev on Jan 14 2024, 6:40 AM.
Tags
Referenced Files
F168536598: D43452.id148351.diff
Fri, Aug 28, 9:57 PM
Unknown Object (File)
Wed, Aug 26, 10:25 AM
Unknown Object (File)
Wed, Aug 26, 9:31 AM
Unknown Object (File)
Mon, Aug 17, 3:29 AM
Unknown Object (File)
Wed, Aug 12, 11:41 PM
Unknown Object (File)
Wed, Aug 12, 8:05 PM
Unknown Object (File)
Wed, Aug 12, 2:27 PM
Unknown Object (File)
Mon, Aug 10, 7:10 AM
View All Files
Subscribers
imp
riscv

Details

Reviewers
mhorne
jrtc27
br
Summary

This patch adds support for IMSIC on RISCV architecture.
The following features are supported:

  • Interrupt delivery with aplic-imsic
  • Sending IPIs between HARTS without ecall
  • IPIs are sent IMSIC-to-IMSIC

Whats not supported:

  • MSI interrupts from devices.

MSI chosen for IPI is 1 as it has the highest priority.

IPIs over IMSIC can be disabled by "aia.imsic.ipi_disabled=1" kernel parameter.

Test Plan
  • Tested SMP boot with aplic-imsic and aplic-only.
  • Tested from 1 CPU to 16 CPUs

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

himanshu_thechauhan.dev created this revision.Jan 14 2024, 6:40 AM
Herald added subscribers: riscv, imp. · View Herald TranscriptJan 14 2024, 6:40 AM
himanshu_thechauhan.dev requested review of this revision.Jan 14 2024, 6:40 AM
himanshu_thechauhan.dev edited the test plan for this revision. (Show Details)Jan 14 2024, 8:35 AM
mhorne mentioned this in D35899: intrng: Allow alternative IPI PICs to be registered and used.Jan 22 2024, 4:51 PM
mmel mentioned this in D40474: intrng: call pic_init_secondary on all registered PICs.Jan 23 2024, 8:17 AM
br added a comment.Aug 13 2024, 1:29 PM

Hi Himanshu what is status of this? could you regenerate for the latest HEAD ?

himanshu_thechauhan.dev added a comment.Aug 14 2024, 6:39 AM
In D43452#1055467, @br wrote:

Hi Himanshu what is status of this? could you regenerate for the latest HEAD ?

Hi Ruslan,

This patch (and the latest one I have) depend on D40474 for the correct ordering of initialisation. Do we have any idea on how to close D40474?
I can generate the patch on current HEAD, no problem.

himanshu_thechauhan.dev retitled this revision from [RFC] riscv: Introduce support for Incoming MSI Controller (IMSIC) to riscv: Introduce support for Incoming MSI Controller (IMSIC).Dec 23 2024, 5:50 PM
himanshu_thechauhan.dev edited the summary of this revision. (Show Details)
himanshu_thechauhan.dev edited the test plan for this revision. (Show Details)
himanshu_thechauhan.dev updated this revision to Diff 148351.Dec 23 2024, 5:52 PM
  • Refactor aplic code into common init and aplic/imsic specific code.
  • Add support for IMSIC.
himanshu_thechauhan.dev added a comment.Jun 12 2025, 4:53 PM

Shall I send a new patch? No comments on this one.

br added a comment.Jan 30 2026, 4:38 PM

how hard it would be to implement MSI interface here? to support MSI interrupts from devices, which is a crucial functionality

sys/riscv/include/cpufunc.h
60

looks like these macroses used just once in entire code, so we can remove them?

sys/riscv/riscv/aplic.c
646

look like magic numbers all around

Revision Contents
Changeset List

PathSize
sys/
conf/
1 line
riscv/
include/
10 lines
32 lines
2 lines
riscv/
246 lines
928 lines

Diff 148351

View Options

sys/conf/files.riscv

Loading...
View Options

sys/riscv/include/cpufunc.h

Loading...
View Options

sys/riscv/include/imsic.h

Loading...
View Options

sys/riscv/include/intr.h

Loading...
View Options

sys/riscv/riscv/aplic.c

Loading...
View Options

sys/riscv/riscv/imsic.c

Loading...