Page MenuHomeFreeBSD
Files F162408203

D58110.diff
No OneTemporary
Actions

D58110.diff
View Options

diff --git a/sys/x86/x86/local_apic.c b/sys/x86/x86/local_apic.c
--- a/sys/x86/x86/local_apic.c
+++ b/sys/x86/x86/local_apic.c
@@ -333,16 +333,6 @@
static void lapic_calibrate_initcount(struct lapic *la);
-/*
- * Calculate the max index of the present LVT entry from the value of
- * the LAPIC version register.
- */
-static int
-lapic_maxlvt(uint32_t version)
-{
- return ((version & APIC_VER_MAXLVT) >> MAXLVTSHIFT);
-}
-
/*
* Use __nosanitizethread to exempt the LAPIC I/O accessors from KCSan
* instrumentation. Otherwise, if x2APIC is not available, use of the global
@@ -390,6 +380,28 @@
}
}
+static inline uint32_t __pure2
+lapic_version(void)
+{
+ return (lapic_read32(LAPIC_VERSION));
+}
+
+/*
+ * Calculate the max index of the present LVT entry from the value of
+ * the LAPIC version register.
+ */
+static inline int __pure2
+lapic_version_maxlvt(uint32_t version)
+{
+ return ((version & APIC_VER_MAXLVT) >> MAXLVTSHIFT);
+}
+
+static inline int __pure2
+lapic_maxlvt(void)
+{
+ return (lapic_version_maxlvt(lapic_version()));
+}
+
#ifdef SMP
static uint64_t
lapic_read_icr_lo(void)
@@ -648,7 +660,7 @@
* It seems that at least some KVM versions report
* EOI_SUPPRESSION bit, but auto-EOI does not work.
*/
- ver = lapic_read32(LAPIC_VERSION);
+ ver = lapic_version();
if ((ver & APIC_VER_EOI_SUPPRESSION) != 0) {
lapic_eoi_suppression = 1;
if (vm_guest == VM_GUEST_KVM) {
@@ -755,7 +767,7 @@
if (cpu_vendor_id != CPU_VENDOR_AMD &&
cpu_vendor_id != CPU_VENDOR_HYGON)
return (0);
- version = lapic_read32(LAPIC_VERSION);
+ version = lapic_version();
if ((version & APIC_VER_AMD_EXT_SPACE) != 0)
return (lapic_read32(LAPIC_EXT_FEATURES));
else
@@ -780,14 +792,12 @@
void
lapic_dump(const char* str)
{
- uint32_t version;
- uint32_t maxlvt;
+ const uint32_t version = lapic_version();
+ const int maxlvt = lapic_version_maxlvt(version);
uint32_t extf;
int elvt_count;
int i;
- version = lapic_read32(LAPIC_VERSION);
- maxlvt = lapic_maxlvt(version);
printf("cpu%d %s:\n", PCPU_GET(cpuid), str);
printf(" ID: 0x%08x VER: 0x%08x LDR: 0x%08x DFR: 0x%08x",
lapic_read32(LAPIC_ID), version,
@@ -844,10 +854,8 @@
lapic_early_mask_vecs(void)
{
int elvt_count, lvts_count, i;
- uint32_t version;
- version = lapic_read32(LAPIC_VERSION);
- lvts_count = min(nitems(lvts), lapic_maxlvt(version) + 1);
+ lvts_count = min(nitems(lvts), lapic_maxlvt() + 1);
for (i = 0; i < lvts_count; i++)
lapic_early_mask_vec(&lvts[i]);
@@ -859,9 +867,9 @@
void
lapic_setup(int boot)
{
+ const uint32_t version = lapic_version();
+ const uint32_t maxlvt = lapic_version_maxlvt(version);
struct lapic *la;
- uint32_t version;
- uint32_t maxlvt;
register_t saveintr;
int elvt_count;
int i;
@@ -870,8 +878,6 @@
la = &lapics[lapic_id()];
KASSERT(la->la_present, ("missing APIC structure"));
- version = lapic_read32(LAPIC_VERSION);
- maxlvt = lapic_maxlvt(version);
/* Initialize the TPR to allow all interrupts. */
lapic_set_tpr(0);
@@ -1032,8 +1038,6 @@
int
lapic_enable_pcint(void)
{
- u_int32_t maxlvt;
-
#ifdef DEV_ATPIC
/* Fail if the local APIC is not present. */
if (!x2apic_mode && lapic_map == NULL)
@@ -1041,8 +1045,7 @@
#endif
/* Fail if the PMC LVT is not present. */
- maxlvt = lapic_maxlvt(lapic_read32(LAPIC_VERSION));
- if (maxlvt < APIC_LVT_PMC)
+ if (lapic_maxlvt() < APIC_LVT_PMC)
return (0);
if (refcount_acquire(&pcint_refcnt) > 0)
return (1);
@@ -1059,8 +1062,6 @@
void
lapic_disable_pcint(void)
{
- u_int32_t maxlvt;
-
#ifdef DEV_ATPIC
/* Fail if the local APIC is not present. */
if (!x2apic_mode && lapic_map == NULL)
@@ -1068,8 +1069,7 @@
#endif
/* Fail if the PMC LVT is not present. */
- maxlvt = lapic_maxlvt(lapic_read32(LAPIC_VERSION));
- if (maxlvt < APIC_LVT_PMC)
+ if (lapic_maxlvt() < APIC_LVT_PMC)
return;
if (!refcount_release(&pcint_refcnt))
return;
@@ -1987,19 +1987,19 @@
{
const struct lvt *l;
int elvt_count, lvts_count, i;
- uint32_t v, vr;
+ const uint32_t v = lapic_version();
+ const int maxlvt = lapic_version_maxlvt(v);
+ const uint32_t vr = lapic_read32(LAPIC_SVR);
db_printf("lapic ID = %d\n", lapic_id());
- v = lapic_read32(LAPIC_VERSION);
db_printf("version = %d.%d (%#x) \n", (v & APIC_VER_VERSION) >> 4,
v & 0xf, v);
- db_printf("max LVT = %d\n", lapic_maxlvt(v));
- vr = lapic_read32(LAPIC_SVR);
+ db_printf("max LVT = %d\n", maxlvt);
db_printf("SVR = %02x (%s)\n", vr & APIC_SVR_VECTOR,
vr & APIC_SVR_ENABLE ? "enabled" : "disabled");
db_printf("TPR = %02x\n", lapic_read32(LAPIC_TPR));
- lvts_count = min(nitems(lvts), lapic_maxlvt(v) + 1);
+ lvts_count = min(nitems(lvts), maxlvt + 1);
for (i = 0; i < lvts_count; i++) {
l = &lvts[i];
db_printf("LVT%d (reg %#x %-5s) = %#010x\n", i, l->lvt_reg,

File Metadata

Mime Type
text/plain
Expires
Mon, Jul 13, 11:28 PM (12 h, 12 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
34964404
Default Alt Text
D58110.diff (4 KB)

Event Timeline