Re: [PATCH] Option to disable AMD C1E (allows dynticks to work)

[Mikhail Kshevetskiy]

Hello Eduard-Gabriel,

You write:

> +	  dynamic ticks to work. It's safe to enable this option even if
> +	  your system doesn't have an AMD CPU (there are no side-effects if
> +	  such a CPU isn't detected).

It's definitely not safe. There are the computers with totally broken
BIOSes, they setup C1e flags  for one core only. If you try to disable
C1e on such machine, you most likely kill the kernel.

>From my point of view, you patch should work in both directions, i.e.
1) user should be allowed to force disable C1e for all cores.
2) user should be allowed to force enable C1e for both cores.

and if the user don't use this flag, the C1e state should be untouched.

Mikhail Kshevetskiy
--
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/