68d0d67
Bugzilla: 971139
68d0d67
Upstream-status: Fedora mustard for now
68d0d67
Dave Jones 18e50ce
Disable watchdog on virtual machines.
Dave Jones 18e50ce
Dave Jones 18e50ce
For various reasons, VMs seem to trigger the soft lockup detector a lot,
Dave Jones 18e50ce
in cases where it's just not possible for a lockup to occur.
Dave Jones 18e50ce
(Example: https://bugzilla.redhat.com/show_bug.cgi?id=971139)
Dave Jones 18e50ce
Dave Jones 18e50ce
In some cases it seems that the host just never scheduled the app running
Dave Jones 18e50ce
the VM for a very long time (Could be the host was under heavy load).
Dave Jones 18e50ce
Dave Jones 18e50ce
Just disable the detector on VMs.
Dave Jones 18e50ce
Dave Jones 18e50ce
Signed-off-by: Dave Jones <davej@redhat.com>
Dave Jones 18e50ce
Dave Jones 18e50ce
diff --git a/kernel/watchdog.c b/kernel/watchdog.c
Kyle McMartin de50edb
index 1241d8c..b2dc4e4 100644
Dave Jones 18e50ce
--- a/kernel/watchdog.c
Dave Jones 18e50ce
+++ b/kernel/watchdog.c
Dave Jones 18e50ce
@@ -24,6 +24,7 @@
Dave Jones 18e50ce
 #include <linux/sysctl.h>
Dave Jones 18e50ce
 #include <linux/smpboot.h>
Dave Jones 18e50ce
 #include <linux/sched/rt.h>
Dave Jones 18e50ce
+#include <linux/dmi.h>
Dave Jones 18e50ce
 
Dave Jones 18e50ce
 #include <asm/irq_regs.h>
Dave Jones 18e50ce
 #include <linux/kvm_para.h>
Dave Jones 18e50ce
@@ -96,6 +97,32 @@ static int __init nosoftlockup_setup(char *str)
Dave Jones 18e50ce
 __setup("nosoftlockup", nosoftlockup_setup);
Dave Jones 18e50ce
 /*  */
Dave Jones 18e50ce
 
Dave Jones 18e50ce
+static int disable_watchdog(const struct dmi_system_id *d)
Dave Jones 18e50ce
+{
Dave Jones 18e50ce
+	printk(KERN_INFO "watchdog: disabled (inside virtual machine)\n");
Kyle McMartin de50edb
+	watchdog_user_enabled = 0;
Dave Jones 18e50ce
+	return 0;
Dave Jones 18e50ce
+}
Dave Jones 18e50ce
+
Dave Jones 18e50ce
+static const struct dmi_system_id watchdog_virt_dmi_table[] = {
Dave Jones 18e50ce
+	{
Dave Jones 18e50ce
+		.callback = disable_watchdog,
Dave Jones 18e50ce
+		.ident = "VMware",
Dave Jones 18e50ce
+		.matches = {
Dave Jones 18e50ce
+			DMI_MATCH(DMI_SYS_VENDOR, "VMware, Inc."),
Dave Jones 18e50ce
+		},
Dave Jones 18e50ce
+	},
Dave Jones 18e50ce
+	{
Dave Jones 18e50ce
+		.callback = disable_watchdog,
Dave Jones 18e50ce
+		.ident = "Bochs",
Dave Jones 18e50ce
+		.matches = {
4d2c9a2
+			DMI_MATCH(DMI_SYS_VENDOR, "Bochs"),
Dave Jones 18e50ce
+		},
Dave Jones 18e50ce
+	},
Dave Jones 18e50ce
+	{}
Dave Jones 18e50ce
+};
Dave Jones 18e50ce
+
Dave Jones 18e50ce
+
Dave Jones 18e50ce
 /*
Dave Jones 18e50ce
  * Hard-lockup warnings should be triggered after just a few seconds. Soft-
Dave Jones 18e50ce
  * lockups can have false positives under extreme conditions. So we generally
Kyle McMartin de50edb
@@ -551,6 +578,8 @@ int proc_dowatchdog(struct ctl_table *table, int write,
Dave Jones 18e50ce
 
Dave Jones 18e50ce
 void __init lockup_detector_init(void)
Dave Jones 18e50ce
 {
Dave Jones 18e50ce
+	dmi_check_system(watchdog_virt_dmi_table);
Dave Jones 18e50ce
+
Dave Jones 18e50ce
 	set_sample_period();
Kyle McMartin de50edb
 
Kyle McMartin de50edb
 #ifdef CONFIG_NO_HZ_FULL