From 69cca8ace4b7e510028fb035ceb026f49c74c6d5 Mon Sep 17 00:00:00 2001 From: sobomax Date: Wed, 27 Sep 2006 22:09:35 +0000 Subject: [PATCH] Extend comment explaining why code is conditional at !defined(SCHED_ULE). Suggested by: ru --- sys/amd64/amd64/local_apic.c | 4 ++++ sys/i386/i386/local_apic.c | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/sys/amd64/amd64/local_apic.c b/sys/amd64/amd64/local_apic.c index 740600392a23..3fb31b509b73 100644 --- a/sys/amd64/amd64/local_apic.c +++ b/sys/amd64/amd64/local_apic.c @@ -620,6 +620,10 @@ lapic_handle_timer(struct trapframe frame) * * No locking is necessary here, since even if we loose the race * when hlt_cpus_mask changes it is not a big deal, really. + * + * Don't do that for ULE, since ULE doesn't consider hlt_cpus_mask + * and unlike other schedulers it actually schedules threads to + * those CPUs. */ if ((hlt_cpus_mask & (1 << PCPU_GET(cpuid))) != 0) return; diff --git a/sys/i386/i386/local_apic.c b/sys/i386/i386/local_apic.c index 0b8d18c2df46..8c9b0858d4d3 100644 --- a/sys/i386/i386/local_apic.c +++ b/sys/i386/i386/local_apic.c @@ -622,6 +622,10 @@ lapic_handle_timer(struct trapframe frame) * * No locking is necessary here, since even if we loose the race * when hlt_cpus_mask changes it is not a big deal, really. + * + * Don't do that for ULE, since ULE doesn't consider hlt_cpus_mask + * and unlike other schedulers it actually schedules threads to + * those CPUs. */ if ((hlt_cpus_mask & (1 << PCPU_GET(cpuid))) != 0) return;