From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37194) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UyzmW-0000H5-7h for qemu-devel@nongnu.org; Tue, 16 Jul 2013 03:34:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UyzmV-0001RT-18 for qemu-devel@nongnu.org; Tue, 16 Jul 2013 03:34:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:39677) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UyzmU-0001RO-Ni for qemu-devel@nongnu.org; Tue, 16 Jul 2013 03:34:42 -0400 Message-ID: <51E4F730.5070907@redhat.com> Date: Tue, 16 Jul 2013 09:33:04 +0200 From: Paolo Bonzini 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> In-Reply-To: <51E4F3C1.5030201@web.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit 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: Jan Kiszka Cc: Peter Maydell , Liu Ping Fan , Mark Cave-Ayland , qemu-devel , Alexander Graf , Blue Swirl , =?ISO-8859-1?Q?Herv=E9_Poussineau?= , Anthony Liguori , =?ISO-8859-1?Q?Andreas_F=E4rber?= , Aurelien Jarno -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 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? Basically, we need testing. The current state of the tree is before Herve's patch, which means PREP is (should be) broken. Alexey posted a patch that reintroduces the DEVICE_LITTLE_ENDIAN and removes the cpu_{in,out}{b,w,l} indirection. http://permalink.gmane.org/gmane.comp.emulators.qemu/222345 We need to test platforms that used a cpu_{in,out}{b,w,l} indirection (MIPS, PPC, SPARC) with and without Alexey's patch. The other occurrences of indirections are: - - hw/isa/i82378.c: This is PREP. Again, removing the indirection should be tested by Herve or Andreas on top of Alexey's patch. - - 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. - - hw/pci-host/apb.c: this is SPARC. Perhaps Mark Cave-Ayland can test 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}. There is also Alpha. It doesn't matter because it's little endian, but anyway rth is removing the indirection. Paolo -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJR5PcwAAoJEBvWZb6bTYbyAOMP/1y19WnUTxcA3+kIhQE1HU2R ildgJ2VV3kRaY4n1HeBuXOXEQzNxwppC0c0Fq/fQpYWhUxB7Y7UTtDVq9S997EUm SoU02UlD2+xnmgOSSTTrYNY20WBRt5/s3xkMjiQSPKItBGw1og7+72m/ydzjSPAh DSBG/mPs12zI3WcTzaydirGVX874pV2N76TSyP+T4DMkipGlgGs1JSOP+Ib/fAWw Hk/mIu+in7qQjv7DZ/0fJuk8cvbgafCGsKppQ5yYu2YKqkmsLhR6xPNeduWK2GMv eRLOoobSOlQxUL4konlvI74Gw/+bFtpjWcjybDfAKcVBkm8H/zpg8SIgbmlbJzoR Gx2bQJvj1PN9pJ/TZRo+1bcw4W7JXKN94s8CTWCL8eGDELB7BgNZFXrxnyfPrY5f 8ega3uZVtZ6SmtPMO7g5arQDfqdvplH/oQozK0muxXgfBzWwvF0Y/qDNRHY18yXC ZdPdEI2FmslOp8ZzhFW+VApfJXDADlC9xOV9rIJ0jqOewBdJlKrEkqJ8oedYGsPM am3fmRL3N/KJ1y59hkNEpGhF6yHS7eHJBVNYp0YakthKOKhZTcBfBryFs1Yo5zwt zvozbSEM0wXnL2ajSwWmU5drH59GdtmYdsIyOm52Etobiu002SZ1VeuCEQOORTTL qtqyWS4kFP/PTTm8KwV4 =RIcU -----END PGP SIGNATURE-----