Page MenuHomeFreeBSD

amd64 efirt: do not flush cache for runtime pages
ClosedPublic

Authored by kib on Oct 5 2021, 2:27 PM.
Tags
None
Referenced Files
F162266345: D32318.diff
Sat, Jul 11, 12:39 PM
Unknown Object (File)
Mon, Jul 6, 12:44 PM
Unknown Object (File)
Mon, Jun 29, 9:43 AM
Unknown Object (File)
Mon, Jun 29, 12:20 AM
Unknown Object (File)
Sun, Jun 28, 6:57 AM
Unknown Object (File)
Wed, Jun 24, 6:49 AM
Unknown Object (File)
Mon, Jun 22, 12:04 PM
Unknown Object (File)
Sat, Jun 20, 2:03 PM
Subscribers

Details

Summary
We actually do not know is it safe or not to flush cache for random
BAR/register page existing in the system.  It is well-known that for
instance LAPICs cannot tolerate cache flush.  As report indicates,
there are more such devices.

This issue typically affects AMD machines which do not report self-snoop,
causing real CLFLUSH invocation on the mapped pages.  Intels do self-snoop,
so this change should be nop for them, and unsafe devices, if any, are
already ignored.

Also:

apic: initialize lapic_paddr statically

The default value for LAPIC registers page physical address
is usually right. Having this value available early makes
pmap_force_invalidate_cache_range(), used on non-self-snoop machines,
avoid flushing LAPIC range for early calls.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable