From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46174) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bWyJm-0005hn-8H for qemu-devel@nongnu.org; Tue, 09 Aug 2016 00:07:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bWyJk-0000VU-40 for qemu-devel@nongnu.org; Tue, 09 Aug 2016 00:07:05 -0400 Message-ID: <1470715599.3015.36.camel@kernel.crashing.org> From: Benjamin Herrenschmidt Date: Tue, 09 Aug 2016 14:06:39 +1000 In-Reply-To: References: <1457789937-30923-1-git-send-email-hpoussin@reactos.org> <1470698237.3015.15.camel@kernel.crashing.org> <1470702562.3015.32.camel@kernel.crashing.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH] adb: change handler only when recognized List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: BALATON Zoltan Cc: =?ISO-8859-1?Q?Herv=E9?= Poussineau , qemu-devel@nongnu.org, Peter Maydell , qemu-ppc@nongnu.org, programmingkidx@gmail.com On Tue, 2016-08-09 at 03:31 +0200, BALATON Zoltan wrote: >=20 > > Because PowerBooks do (or rather a PMU-simulation of ADB) and MacOS > > doesn't care. If ADB is in the device-tree, it will use it. It makes > > things easier to support multiple combinations especially when > > "comparing" things for debug. > >=20 > > Additionally, USB doesn't work well in OpenBIOS at this point ;-) >=20 > In what way? Keyboard works. What else is needed? Doesnt' work for me half of the time, I haven't dug into why yet. > > Also, I have some evil plan to change the way ADB autopoll works in > > Qemu so that the devices signal the PMU when they want to talk. That > > will avoid having yet another 30-something HZ timer ticking in qemu, > > and in that regard will probably be more efficient (read: slows down > > the emulator less) than OHCI. >=20 > OK, this is a really nice thing and justifies having an ADB bus. (This=C2= =A0 > would also likely fix mouse problems seen by others.) Possibly ;-) The tracking in OS 9 at least is still done by a timer inside MacOS itself. With my latest ndrv it's running at 30Hz. I'm looking at maybe doing some kind of paravirt hack to make it adaptative on whether there is actual movement on the cursor, but that's for later. > > Why not ? >=20 > Just thought emulating all the additional details for Powerbook power=C2= =A0 > management might be difficult. But if you're willing to go there I won'= t=C2=A0 > stop you. :-) Obviously you understand this very well so maybe it's not= =C2=A0 > that difficult for you. Well, I wrote most of the corresponding Linux code so it's mostly a matte= r of swapping that knowledge back into my brain from lossy long term storag= e and finding all the MacOS 9 bugs along the way :-) > > >=20 > > > =C2=A0Here's an (untested) patch for > > > switching to using USB keyboard and mouse instead of ADB unless USB= is=C2=A0 > > > disabled. (The device tree should be changed accordingly in OpenBIO= S.) Not=C2=A0 > > > sure if this is helpful. > >=20 > > Don't bother just yet, as I said, I am reworking all of that code. I = will > > probably just create various -machine options so you can chose what b= its > > and pieces you want to put togeher, ie, CUDA, PMU, with or without AD= B, > > which machine model string to expose to the OS etc... >=20 > Cool, looking forward to it. Thanks for all the great stuff you did for= =C2=A0 > this and keep it up. Cheers, Ben.