diff options
| author | Jan Kiszka <jan.kiszka@siemens.com> | 2013-03-19 12:36:51 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-04-05 10:16:38 -0700 |
| commit | 0072625c351588b8fde9e6f46fb60ba2e521fb47 (patch) | |
| tree | 5ede7ff405954501874d02b14a6c1042f04d6b0d /include | |
| parent | 2e8e2c7847cc17a8135ad17869f5ba37207e2f89 (diff) | |
| download | linux-stable-0072625c351588b8fde9e6f46fb60ba2e521fb47.tar.gz linux-stable-0072625c351588b8fde9e6f46fb60ba2e521fb47.tar.bz2 linux-stable-0072625c351588b8fde9e6f46fb60ba2e521fb47.zip | |
KVM: x86: Prevent starting PIT timers in the absence of irqchip support
commit 0924ab2cfa98b1ece26c033d696651fd62896c69 upstream.
User space may create the PIT and forgets about setting up the irqchips.
In that case, firing PIT IRQs will crash the host:
BUG: unable to handle kernel NULL pointer dereference at 0000000000000128
IP: [<ffffffffa10f6280>] kvm_set_irq+0x30/0x170 [kvm]
...
Call Trace:
[<ffffffffa11228c1>] pit_do_work+0x51/0xd0 [kvm]
[<ffffffff81071431>] process_one_work+0x111/0x4d0
[<ffffffff81071bb2>] worker_thread+0x152/0x340
[<ffffffff81075c8e>] kthread+0x7e/0x90
[<ffffffff815a4474>] kernel_thread_helper+0x4/0x10
Prevent this by checking the irqchip mode before starting a timer. We
can't deny creating the PIT if the irqchips aren't set up yet as
current user land expects this order to work.
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions