Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: ebiederm@xxxxxxxxxxxx (Eric W. Biederman)
- Date: Mon, 03 Nov 2008 08:13:07 -0800
Avi Kivity <avi@xxxxxxxxxx> writes:
Eduardo Habkost wrote:
We probably need to disable vmx on all CPUs, but emergency reboot skips
native_smp_send_stop() (where we could hook a virt_disable call in).
As relying on IPIs defeats the whole point of emergency_restart, a proper
fix will need to use NMIs like the kdump code does.
They should use the same code; they have a similar environment at entry and
reliability requirements for e_r are not greater than kdump's.
Just a sec.
I think we are confusing two issues here.
- Ordinary machine_restart which happens to call emergency_restart.
And is proceeded by machine_shutdown.
- And emergency_restart itself.
To some extent I would be a lot happier if Alt-sysrq-r did what
was necessary to get into a context where it can call machine_restart
or even better kernel_restart().
emergency_restart() is a nice idea but is broken by design.
That said. If we can turn off vmx on that one processor.
That should be enough for the cpu to triple fault and let
the BIOS do what it needs to do on that cpu i.e. outb(magic, 0x92)
and toggle a motherboard level reset?
If I read the earlier comments correctly the deep issue is
that going through ACPI to reset systems is less reliable than
doing it the classic way.
Eric
--
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/
- Follow-Ups:
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: Avi Kivity
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: Eduardo Habkost
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- References:
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: Ingo Molnar
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: Avi Kivity
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: Eduardo Habkost
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- From: Avi Kivity
- Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- Prev by Date: Re: bdi: register sysfs bdi device only once per queue
- Next by Date: [PATCH] jffs2: Fix race condition in jffs2_lzo_compress()
- Previous by thread: Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- Next by thread: Re: [PATCH] Use BIOS reboot on Toshiba Portege 4000
- Index(es):
Relevant Pages
|