[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors |
Date: |
Thu, 26 Feb 2015 19:51:16 -0300 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Fri, Feb 27, 2015 at 07:44:29AM +0900, Peter Maydell wrote:
> On 27 February 2015 at 01:06, Eduardo Habkost <address@hidden> wrote:
> > On Thu, Feb 26, 2015 at 08:27:12AM +0900, Peter Maydell wrote:
> >> Rather than bailing out here, it would be better to
> >> propagate the failure out to the caller, which can
> >> then fail the fork syscall it's trying to emulate
> >> by returning a suitable error code to the guest.
> >
> > Makes sense to me, but I would prefer that to be done by somebody
> > faimilar with the *-user code as a follow-up. My intention was to simply
> > fix the bug so that QEMU won't crash.
>
> When can it actually fail? Have you seen it do this in practice?
> (Note that this is only called from fork, so we know that
> cpu_model must be valid, or we would have failed on startup
> trying to create the CPU for the initial thread.)
I have never seen it in practice, but in x86 it will fail if we try to
create more than 254 VCPUs.
--
Eduardo
- [Qemu-devel] [PULL 05/11] target-i386: Eliminate cpu_init() function, (continued)
- [Qemu-devel] [PULL 06/11] target-i386: Simplify error handling on cpu_x86_init_user(), Eduardo Habkost, 2015/02/25
- [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Eduardo Habkost, 2015/02/25
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Andreas Färber, 2015/02/25
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Peter Maydell, 2015/02/25
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Eduardo Habkost, 2015/02/26
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Peter Maydell, 2015/02/26
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors,
Eduardo Habkost <=
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Peter Maydell, 2015/02/26
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Eduardo Habkost, 2015/02/26
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Peter Maydell, 2015/02/26
- Re: [Qemu-devel] [PULL 08/11] linux-user: Check for cpu_init() errors, Eduardo Habkost, 2015/02/27
[Qemu-devel] [PULL 07/11] target-i386: Move CPUX86State.cpuid_apic_id to X86CPU.apic_id, Eduardo Habkost, 2015/02/25
[Qemu-devel] [PULL 09/11] target-i386: Set APIC ID using cpu_index on CONFIG_USER, Eduardo Habkost, 2015/02/25
[Qemu-devel] [PULL 10/11] target-i386: Require APIC ID to be explicitly set before CPU realize, Eduardo Habkost, 2015/02/25
[Qemu-devel] [PULL 11/11] target-i386: Move APIC ID compatibility code to pc.c, Eduardo Habkost, 2015/02/25