Re: [Qemu-devel] [RFC] PowerPC Mac OS on Qemu

[Natalia Portillo]

Please,

a) separate the patch in pieces, I suggest you checking how other patches are 
split, there have been a lot of patches today to get insight.
b) use inline signed-by patch

Regards,
Natalia Portillo

El 09/08/2011, a las 20:46, William Hahne escribió:

> Hello,
> 
> I am a Google Summer of Code student working on getting PowerPC Mac OS 
> running on Qemu. While I am not finished I need to upstream what I currently 
> have before the end of Summer of Code. My patch is for OpenBIOS but I am 
> cross-posting to both Qemu and OpenBIOS mailing lists since it is useful to 
> get feedback from both.
> 
> One part of the patch I am particularly concerned about is the patch to 
> arch/ppc/qemu/init.c to provide a CPU and Timebase frequency. Qemu doesn't 
> report a CPU frequency and the reported Timebase frequency is too high and 
> causes the Mac OS scheduler to break. I hard-coded values that work but this 
> seems like an unacceptable long-term solution to me. A simple test for CPU 
> frequency might be the best solution here but seems a little redundant as Mac 
> OS later runs its own test, seemingly only relying on these values during 
> initialization (I am not 100% sure of this since it crashes before reaching 
> that point.) Feedback here would be especially appreciated.
> 
> Another thing to note is in drivers/adb_kbd.c. "get-key-map" which returns a 
> map of the current pressed keys on the keyboard just returns a dummy value. I 
> felt it was a waste of time making a full implementation when all it really 
> gets you is the ability to use keyboard shortcuts for verbose or single user 
> mode. 
> 
> Other than those specific concerns I welcome general feedback on the patch, 
> since as I said I am hoping to get it in before the end of Summer of Code.
> 
> Thanks,
> William Hahne
> <PowerPC_Mac_OS.patch>