[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creat
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic |
Date: |
Wed, 07 Jan 2015 12:27:51 +0100 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 07.01.15 12:22, Paolo Bonzini wrote:
>
>
> On 07/01/2015 12:15, Alexander Graf wrote:
>>
>>
>> On 07.01.15 12:07, Paolo Bonzini wrote:
>>>
>>>
>>> On 07/01/2015 12:03, Marcel Apfelbaum wrote:
>>>>
>>>> While I agree it will be better if we place this in instance_init,
>>>> setting the machine_usb to defaults_enabled() there would be problematic
>>>> since it depends on
>>>> - papr_vga_init(phb->bus) for sparpr and
>>>
>>> That's effectively vga_interface_type == VGA_DEVICE ||
>>> vga_interface_type == VGA_STD.
>>>
>>>> - (PPC_INPUT(env) == PPC_FLAGS_INPUT_970) for mac99.
>>>> (The env itself is set in machine_init)
>>>
>>> Alex, why is auto-USB disabled for 6xx? Can it use vga_interface_type
>>> like spapr does?
>>
>> That one's a nasty hack. We basically have 2 different machine types
>> that we expose as a single type to the user: mac99. In reality there's a
>> 64bit mac99 and a 32bit mac99.
>>
>> 32bit mac99 can expose keyboard and mouse via a special apple bus. That
>> driver doesn't work with 64bit Linux guests though, so there we need USB.
>>
>> Thinking about it, maybe the best way forward would be to create 2
>> machine types out of these. Have a mac99 (32bit) and a mac99-g5 target
>> where the g5 target defaults to -cpu G5 and USB enabled.
>>
>> All of this is pretty frankenstein btw. What we would really want for a
>> G5 guest is something built around U3 or U4, not the U1 that -M mac99
>> exposes.
>
> Hmm, then I guess let's apply these patches and fix things up later?
Certainly works for me ;).
Alex
- Re: [Qemu-devel] [PATCH 1/6] hw/ppc: modified the condition for usb controllers to be created for some ppc machines, (continued)
- [Qemu-devel] [PATCH 3/6] hw/usb: simplified usb_enabled, Marcel Apfelbaum, 2015/01/06
- [Qemu-devel] [PATCH 4/6] hw/ppc/mac_newworld: QOMified mac99 machines, Marcel Apfelbaum, 2015/01/06
- [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Marcel Apfelbaum, 2015/01/06
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Paolo Bonzini, 2015/01/06
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Marcel Apfelbaum, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Paolo Bonzini, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Alexander Graf, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Marcel Apfelbaum, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Paolo Bonzini, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic,
Alexander Graf <=
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Paolo Bonzini, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Alexander Graf, 2015/01/07
- Re: [Qemu-devel] [PATCH 5/6] hw/ppc/spapr: simplify usb controller creation logic, Marcel Apfelbaum, 2015/01/07
[Qemu-devel] [PATCH 6/6] hw/ppc/mac_newworld: simplify usb controller creation logic, Marcel Apfelbaum, 2015/01/06
Re: [Qemu-devel] [PATCH 0/6] simplify usb enabling logic and fix a Qemu crash, Paolo Bonzini, 2015/01/06
Re: [Qemu-devel] [PATCH 0/6] simplify usb enabling logic and fix a Qemu crash, Stefan Hajnoczi, 2015/01/07
Re: [Qemu-devel] [PATCH 0/6] simplify usb enabling logic and fix a Qemu crash, Peter Maydell, 2015/01/08