Re: [Qemu-ppc] [PATCH] Changes default cdrom options to work better with Mac OS X.

[Programmingkid]

On Jun 23, 2013, at 12:38 PM, Alexander Graf wrote:

> 
> 
> Am 23.06.2013 um 14:37 schrieb Programmingkid <address@hidden>:
> 
>> 
>> On Jun 22, 2013, at 9:20 PM, Alexander Graf wrote:
>> 
>>> 
>>> On 23.06.2013, at 03:14, Programmingkid wrote:
>>> 
>>>> 
>>>> On Jun 22, 2013, at 9:08 PM, Alexander Graf wrote:
>>>> 
>>>>> 
>>>>> On 23.06.2013, at 03:04, Programmingkid wrote:
>>>>> 
>>>>>> This patch makes the default cdrom options more compatible with Mac OS X.
>>>>>> 
>>>>>> With this patch I was finally able to see "Got boot device" printed by 
>>>>>> Darwin 8! This made the installer for Darwin 8 actually show up in QEMU. 
>>>>>> 
>>>>>> signed-off-by: John Arbuckle <address@hidden>
>>>>>> 
>>>>>> ---
>>>>>> vl.c |    2 +-
>>>>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>>>> 
>>>>>> diff --git a/vl.c b/vl.c
>>>>>> index a8dc73d..cb53656 100644
>>>>>> --- a/vl.c
>>>>>> +++ b/vl.c
>>>>>> @@ -3085,7 +3085,7 @@ int main(int argc, char **argv, char **envp)
>>>>>>            qemu_opts_set(qemu_find_opts("machine"), 0, "dtb", optarg);
>>>>>>            break;
>>>>>>        case QEMU_OPTION_cdrom:
>>>>>> -                drive_add(IF_DEFAULT, 2, optarg, CDROM_OPTS);
>>>>>> +                drive_add(IF_IDE, 0, optarg, CDROM_OPTS);
>>>>> 
>>>>> Consider me puzzled. Why is IF_DEFAULT and different from IF_IDE on 
>>>>> g3beige?
>>>>> 
>>>>> 
>>>>> Alex
>>>> 
>>>> 
>>>> Because IF_DEFAULT is equal to -1 and IF_IDE is equal to 0. Their 
>>>> definition is in blockdev.h.
>>> 
>>> So? The default block bus on g3beige is IDE, so IF_DEFAULT should behave 
>>> identical to IF_IDE.
>> 
>> Sounds like a good idea. 
>> 
>>> 
>>> The real change you did was to go from "2" to "0". So instead of attaching 
>>> the CD-ROM to hdc (on cmd646) you now attach it to hda (on mac-io).
>>> 
>>> So the real underlying problem is that we expose the second IDE channel on 
>>> a cmd646 IDE controller, not mac-io. I think we did this back then to 
>>> support booting at all, because the mac-io emulation was so broken that 
>>> booting from CD-ROM simply didn't work. I guess it's time to switch back ...
>> 
>> Yes, using mac-io is the best thing to do now. 
>> 
>> One of the last major problems with making Mac OS X boot in QEMU is with a 
>> function called IOSleep(). This is an XNU function that does not work 
>> correctly. Whenever a call to this function is made, the kernel pauses for a 
>> much longer time than it should. This will have to be fixed somehow.
> 
> This is probably because you have a local patch to override tbfreq, right? :)
> 
> Alex

I have no idea. Do you know the date of this patch or who made it?