lkml.org 
[lkml]   [2021]   [May]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[RFC PATCH v5 10/16] watchdog/hardlockup/hpet: Adjust timer expiration on the number of monitored groups
Date
The CPUs monitored by the HPET hardlockup detector are partitioned into
groups based on their location in a given package. The HPET timer only
targets a single CPU per package (or group of packages) at expiration. The
rest of the CPUs in the group are monitored via an IPI issued by the
handling CPU.

Each monitored CPU must be checked for hardlockups every watch_thresh
seconds. This also means that each group of CPUs must be monitored in the
same interval. Therefore, the HPET timer expiration is determined by the
watch_thresh divided by the number of groups to monitor.

Add a new member, hpet_hld_data::ticks_per_group represents the number of
times the HPET timer must tick before interrupting the handling CPU. Derive
this value from the frequency of the HPET timer and the number of groups of
CPUs.

Furthermore, update the timer expiration whenever there is a change
in the number of monitored CPUs. Namely, when enabling or disabling the
hardlockup detector in a given CPU.

Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Ashok Raj <ashok.raj@intel.com>
Cc: Andi Kleen <andi.kleen@intel.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: "Ravi V. Shankar" <ravi.v.shankar@intel.com>
Cc: x86@kernel.org
Signed-off-by: Ricardo Neri <ricardo.neri-calderon@linux.intel.com>
---
Changes since v4:
* Reworked computation for the number of package groups instead of the
number of CPUs.
* Renamed local variable temp as ticks in update_ticks_per_group().
(Andi)

Changes since v3:
* None

Changes since v2:
* Since the round-robin mechanism set the affinity of the HPET timer
interrupt is in use again, it also becomes necessary to adjust
the timer expiration.

Changes since v1:
* Dropped this patch as there was no need to readjust the timer
expiration when the HPET timer only targets a single CPU.
---
arch/x86/include/asm/hpet.h | 3 ++
arch/x86/kernel/watchdog_hld_hpet.c | 47 +++++++++++++++++++++++++++--
2 files changed, 48 insertions(+), 2 deletions(-)

diff --git a/arch/x86/include/asm/hpet.h b/arch/x86/include/asm/hpet.h
index bb76f54effe4..738fcf256b14 100644
--- a/arch/x86/include/asm/hpet.h
+++ b/arch/x86/include/asm/hpet.h
@@ -102,6 +102,8 @@ extern void hpet_unregister_irq_handler(rtc_irq_handler handler);
* @has_periodic: The HPET channel supports periodic mode
* @channel: HPET channel assigned to the detector
* @ticks_per_second: Frequency of the HPET timer
+ * @ticks_per_group: HPET ticks per group that must elapse before
+ * the timer expires
* @irq: IRQ number assigned to the HPET channel
* @handling_cpu: CPU handling the HPET interrupt
* @pkgs_per_group: Number of physical packages in a group of CPUs
@@ -123,6 +125,7 @@ struct hpet_hld_data {
bool has_periodic;
u32 channel;
u64 ticks_per_second;
+ u64 ticks_per_group;
int irq;
u32 handling_cpu;
u32 pkgs_per_group;
diff --git a/arch/x86/kernel/watchdog_hld_hpet.c b/arch/x86/kernel/watchdog_hld_hpet.c
index 04b354a35e68..bf3ee354907f 100644
--- a/arch/x86/kernel/watchdog_hld_hpet.c
+++ b/arch/x86/kernel/watchdog_hld_hpet.c
@@ -48,18 +48,26 @@ static void kick_timer(struct hpet_hld_data *hdata, bool force)
* are able to update the comparator before the counter reaches such new
* value.
*
+ * Each CPU must be monitored every watch_thresh seconds. In order to
+ * keep the HPET channel interrupt under 1 per second, CPUs are targeted
+ * by groups. Each group is target separately.
+ *
+ * ticks_per_group = watch_thresh * ticks_per_second / nr_groups
+ *
+ * as computed in update_ticks_per_group().
+ *
* Let it wrap around if needed.
*/

count = hpet_readl(HPET_COUNTER);
- new_compare = count + watchdog_thresh * hdata->ticks_per_second;
+ new_compare = count + watchdog_thresh * hdata->ticks_per_group;

if (!hdata->has_periodic) {
hpet_writel(new_compare, HPET_Tn_CMP(hdata->channel));
return;
}

- period = watchdog_thresh * hdata->ticks_per_second;
+ period = watchdog_thresh * hdata->ticks_per_group;
hpet_set_comparator_periodic(hdata->channel, (u32)new_compare,
(u32)period);
}
@@ -411,6 +419,27 @@ static int setup_hpet_irq(struct hpet_hld_data *hdata)
return ret;
}

+/**
+ * update_ticks_per_group() - Update the number of HPET ticks CPU group
+ * @hdata: struct with the timer's the ticks-per-second and CPU mask
+ *
+ * From the overall ticks-per-second of the timer, compute the number of ticks
+ * after which the timer should expire to monitor each CPU every watch_thresh
+ * seconds. The monitored CPUs have been partitioned into groups, and the HPET
+ * channel targets one group at a time.
+ */
+static void update_ticks_per_group(struct hpet_hld_data *hdata)
+{
+ u64 ticks = hdata->ticks_per_second;
+
+ /* Only update if there are CPUs to monitor. */
+ if (!hdata->nr_groups)
+ return;
+
+ do_div(ticks, hdata->nr_groups);
+ hdata->ticks_per_group = ticks;
+}
+
/**
* hardlockup_detector_hpet_enable() - Enable the hardlockup detector
* @cpu: CPU Index in which the watchdog will be enabled.
@@ -424,6 +453,7 @@ void hardlockup_detector_hpet_enable(unsigned int cpu)
cpumask_set_cpu(cpu, hld_data->monitored_cpumask);

setup_cpu_groups(hld_data);
+ update_ticks_per_group(hld_data);

update_ipi_target_cpumask(hld_data);

@@ -436,7 +466,14 @@ void hardlockup_detector_hpet_enable(unsigned int cpu)
update_msi_destid(hld_data);
kick_timer(hld_data, true);
enable_timer(hld_data);
+ return;
}
+
+ /*
+ * Kick timer in case the number of monitored CPUs requires a change in
+ * the timer period.
+ */
+ kick_timer(hld_data, hld_data->has_periodic);
}

/**
@@ -469,9 +506,15 @@ void hardlockup_detector_hpet_disable(unsigned int cpu)
update_msi_destid(hld_data);

setup_cpu_groups(hld_data);
+ update_ticks_per_group(hld_data);

update_ipi_target_cpumask(hld_data);

+ /*
+ * Kick timer in case the number of monitored CPUs requires a change in
+ * the timer period.
+ */
+ kick_timer(hld_data, hld_data->has_periodic);
enable_timer(hld_data);
}

--
2.17.1
\
 
 \ /
  Last update: 2021-05-04 21:39    [W:0.096 / U:0.276 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site