From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37325) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VGVMX-0005dl-Qq for qemu-devel@nongnu.org; Mon, 02 Sep 2013 10:44:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VGVMR-0003MV-Rs for qemu-devel@nongnu.org; Mon, 02 Sep 2013 10:44:17 -0400 Received: from mail-la0-f43.google.com ([209.85.215.43]:43632) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VGVMR-0003MJ-Kw for qemu-devel@nongnu.org; Mon, 02 Sep 2013 10:44:11 -0400 Received: by mail-la0-f43.google.com with SMTP id ep20so3670040lab.16 for ; Mon, 02 Sep 2013 07:44:10 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1378132788.2640.21.camel@localhost.localdomain> References: <1378131189-25538-1-git-send-email-marcel.a@redhat.com> <1378132788.2640.21.camel@localhost.localdomain> From: Peter Maydell Date: Mon, 2 Sep 2013 15:43:50 +0100 Message-ID: Content-Type: text/plain; charset=UTF-8 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: Marcel Apfelbaum Cc: Paolo Bonzini , Anthony Liguori , QEMU Developers , =?UTF-8?Q?Andreas_F=C3=A4rber?= , "Michael S. Tsirkin" 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. -- PMM