From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55460) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VGWNh-000556-Du for qemu-devel@nongnu.org; Mon, 02 Sep 2013 11:49:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VGWNb-0008SU-EL for qemu-devel@nongnu.org; Mon, 02 Sep 2013 11:49:33 -0400 Received: from mx1.redhat.com ([209.132.183.28]:18335) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VGWNb-0008SM-6l for qemu-devel@nongnu.org; Mon, 02 Sep 2013 11:49:27 -0400 Message-ID: <1378136968.2640.38.camel@localhost.localdomain> From: Marcel Apfelbaum Date: Mon, 02 Sep 2013 18:49:28 +0300 In-Reply-To: References: <1378131189-25538-1-git-send-email-marcel.a@redhat.com> <1378132788.2640.21.camel@localhost.localdomain> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC 0/3] pci: complete master abort protocol List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Paolo Bonzini , Anthony Liguori , "Michael S. Tsirkin" , QEMU Developers , Andreas =?ISO-8859-1?Q?F=E4rber?= On Mon, 2013-09-02 at 15:43 +0100, Peter Maydell wrote: > On 2 September 2013 15:39, Marcel Apfelbaum wrote: > > On Mon, 2013-09-02 at 15:30 +0100, Peter Maydell wrote: > >> On 2 September 2013 15:13, Marcel Apfelbaum wrote: > >> > Note: The series is incomplete, for review only > >> > > >> > PCI spec requires that a transaction that has not been claimed > >> > by any PCI bus devices will be terminated by the initiator > >> > with "master abort". For read transactions -1(FFFFFFFF) is returned and > >> > writes are silently dropped. (already implemented in quemu) > >> > >> ...but only erroneously and by breaking a pile of other boards. > > :( > > Let's find a way to fix it... Please suggest > > We need to apply Jan's two patches which revert the breakage > and give us back 1.5 behaviour for unassigned memory. > Then you can work on this patchset and we can commit it > when it works OK. I am going to have a look on those patches, thanks Marcel > > -- PMM >