From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59405) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uz8bp-0008CL-Cr for qemu-devel@nongnu.org; Tue, 16 Jul 2013 13:00:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Uz8bk-00081D-G1 for qemu-devel@nongnu.org; Tue, 16 Jul 2013 13:00:17 -0400 Received: from smtp1-g21.free.fr ([2a01:e0c:1:1599::10]:53970) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uz8bj-0007ce-T9 for qemu-devel@nongnu.org; Tue, 16 Jul 2013 13:00:12 -0400 Message-ID: <51E57C09.5000000@reactos.org> Date: Tue, 16 Jul 2013 18:59:53 +0200 From: =?ISO-8859-1?Q?Herv=E9_Poussineau?= MIME-Version: 1.0 References: <51C75FA6.6080903@reactos.org> <51C7E21A.9090005@web.de> <8A36D64D-0625-49E1-9E59-391DAEEBD1FC@suse.de> <51DEA91B.40903@suse.de> <51E16683.1040304@redhat.com> <51E24216.4050406@redhat.com> <874nbxqolf.fsf@codemonkey.ws> <51E4F3C1.5030201@web.de> <51E4F730.5070907@redhat.com> In-Reply-To: <51E4F730.5070907@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3 11/14] ioport: Switch dispatching to memory core layer List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Peter Maydell , Liu Ping Fan , Mark Cave-Ayland , qemu-devel , Alexander Graf , Blue Swirl , Jan Kiszka , Anthony Liguori , =?ISO-8859-1?Q?Andreas_F=E4rber?= , Aurelien Jarno Paolo Bonzini a =E9crit : > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 >=20 > Il 16/07/2013 09:18, Jan Kiszka ha scritto: >> Sorry for sending out invitations and then being late to this party >> - vacation. What is the status now? Do we have a short-term plan to >> avoid the regression or is this better solved by cleaning up the >> whole endianess thing? Is anyone actively on it, or should I take a >> drink, sit down and join the discussion? >=20 > Basically, we need testing. The current state of the tree is before > Herve's patch, which means PREP is (should be) broken. >=20 > Alexey posted a patch that reintroduces the DEVICE_LITTLE_ENDIAN and > removes the cpu_{in,out}{b,w,l} indirection. >=20 > http://permalink.gmane.org/gmane.comp.emulators.qemu/222345 >=20 > We need to test platforms that used a cpu_{in,out}{b,w,l} indirection > (MIPS, PPC, SPARC) with and without Alexey's patch. >=20 > The other occurrences of indirections are: >=20 > - - hw/isa/i82378.c: This is PREP. Again, removing the indirection > should be tested by Herve or Andreas on top of Alexey's patch. For i82378, I have a big patch for it, which rewrites large parts of the=20 emulation. Moreover, as i82378 is only used in PReP machine, this=20 indirection can be ignored for now. > - - hw/isa/isa_mmio: bamboo, g3beige and mac99 could be tested by Alex. > I don't know about MIPS. If anything is broken, the solution is to > replace isa_mmio_{setup,init} with an alias to get_system_io(). This > stops using isa_mmio altogether, so it can be done only on those > platforms where it's needed. For MIPS Jazz, it is currently broken in SCSI emulation when installing=20 Windows NT 4.0/MIPS. However, it seems quite a general problem on=20 Windows NT 4, as Neozeed also reported the same freeze when running=20 NT4/x86 on http://virtuallyfun.superglobalmegacorp.com/?p=3D3065 . > - - hw/pci-host/apb.c: this is SPARC. Perhaps Mark Cave-Ayland can tes= t > it and see if it is broken---again with and without Alexey's patch. > There is a small difference. This file uses DEVICE_NATIVE_ENDIAN and > does the byte swap itself in pci_apb_io{read,write}{b,w,l}. >=20 > There is also Alpha. It doesn't matter because it's little endian, > but anyway rth is removing the indirection. Herv=E9