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
Unknown Object (File)
Sun, Apr 20, 7:59 AM
Unknown Object (File)
Sun, Apr 20, 5:28 AM
Unknown Object (File)
Sat, Apr 19, 10:30 PM
Unknown Object (File)
Sat, Apr 19, 6:30 PM
Unknown Object (File)
Mon, Apr 14, 6:46 PM
Unknown Object (File)
Thu, Apr 10, 1:16 AM
Unknown Object (File)
Mar 11 2025, 5:47 AM
Unknown Object (File)
Mar 11 2025, 1:05 AM
View All 54 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.

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...