riscv: Make sure local hart's icache is synced in pmap_sync_icache
ClosedPublic
Actions

Authored by jrtc27 on Apr 6 2020, 8:59 PM.

Tags

None

Referenced Files

	F133071798: D24317.id70282.diff
	Wed, Oct 22, 5:23 PM

	F133071794: D24317.id.diff
	Wed, Oct 22, 5:23 PM

	F133071790: D24317.id70272.diff
	Wed, Oct 22, 5:23 PM

	F133027537: D24317.diff
	Wed, Oct 22, 5:16 AM

	Unknown Object (File)
	Tue, Oct 21, 6:37 AM

	Unknown Object (File)
	Sun, Oct 19, 2:12 PM

	Unknown Object (File)
	Sat, Oct 18, 11:16 PM

	Unknown Object (File)
	Sep 5 2025, 1:56 AM

View All 54 Files

Subscribers

imp

Details

Reviewers

jhb
markj
br

Commits

rS359672: riscv: Make sure local hart's icache is synced in pmap_sync_icache

Summary

The only way to flush the local hart's icache is with a FENCE.I (or an
equivalent SBI call); a normal FENCE is insufficient and, for the
single-hart case, unnecessary.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 30325
Build 28093: arc lint + arc unit

Event Timeline

jrtc27 created this revision.Apr 6 2020, 8:59 PM

Harbormaster completed remote builds in B30325: Diff 70272.Apr 6 2020, 8:59 PM

I broke this in rS339367 when I switched to using sbi_remove_fence_i as I had cleared the current CPU from the mask before passing that mask to SBI.

This revision is now accepted and ready to land.Apr 6 2020, 9:42 PM

markj accepted this revision.Apr 6 2020, 9:48 PM

Closed by commit rS359672: riscv: Make sure local hart's icache is synced in pmap_sync_icache (authored by jrtc27). · Explain WhyApr 6 2020, 10:31 PM

This revision was automatically updated to reflect the committed changes.

jrtc27 added a commit: rS359672: riscv: Make sure local hart's icache is synced in pmap_sync_icache.

Herald added a subscriber: imp. · View Herald TranscriptApr 6 2020, 10:31 PM