From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41216) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XUdE3-0006ao-38 for qemu-devel@nongnu.org; Thu, 18 Sep 2014 11:02:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XUdDw-0002G5-FQ for qemu-devel@nongnu.org; Thu, 18 Sep 2014 11:02:27 -0400 Message-ID: <1411052527.1199.60.camel@ul30vt.home> From: Alex Williamson Date: Thu, 18 Sep 2014 09:02:07 -0600 In-Reply-To: <541AE43B.5060106@ozlabs.ru> References: <1410262487-24079-1-git-send-email-aik@ozlabs.ru> <1410470418.2982.354.camel@ul30vt.home> <54124C99.8040504@ozlabs.ru> <541AE43B.5060106@ozlabs.ru> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] vfio: Another try to fix ROM BAR endianness List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, Nikunj A Dadhania , Alexander Graf On Thu, 2014-09-18 at 23:55 +1000, Alexey Kardashevskiy wrote: > On 09/12/2014 11:30 AM, Alexey Kardashevskiy wrote: > > On 09/12/2014 07:20 AM, Alex Williamson wrote: > >> On Tue, 2014-09-09 at 21:34 +1000, Alexey Kardashevskiy wrote: > >>> I did some tests on LE/BE guest/host combinations and came up with > >>> these 2 patches. Please comment. Thanks. > >> > >> Have you tried BE guest/LE host? > > > > Yes. > > Double checked, LE-BE host-guest work in any combination. > > > >> IIRC, the g3beige tcg guest on x86 > >> host used to work with a vfio-pci NIC. The ROM is of course typically > >> useless in this mode, but it would be nice to make sure we haven't > >> broken BAR access for that combination. Do we have any better BE tcg > >> targets with PCI support these days? Thanks, > > > > pseries? PPC64 TCG can do VFIO. > > And checked this too, VFIO works in BE TCG PPC64 guest running on PPC64 LE > host. Thanks for the additional testing. Greg had some comments on patch 1/2, perhaps you could address them or update the comments. Thanks, Alex