Re: [PATCH v3] watchdog: add watchdog_cpumask sysctl to assist nohz

From: Peter Zijlstra
Date: Thu Apr 02 2015 - 14:06:53 EST


On Thu, Apr 02, 2015 at 01:39:28PM -0400, cmetcalf@xxxxxxxxxx wrote:
> @@ -431,6 +434,10 @@ static void watchdog_enable(unsigned int cpu)
> hrtimer_init(hrtimer, CLOCK_MONOTONIC, HRTIMER_MODE_REL);
> hrtimer->function = watchdog_timer_fn;
>
> + /* Exit if the cpu is not allowed for watchdog. */
> + if (!cpumask_test_cpu(cpu, watchdog_mask))
> + do_exit(0);
> +

Ick, that doesn't look right for smpboot threads.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/