From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33764) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z5hPX-00042e-KO for qemu-devel@nongnu.org; Thu, 18 Jun 2015 17:31:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z5hPT-0007lx-By for qemu-devel@nongnu.org; Thu, 18 Jun 2015 17:31:47 -0400 Received: from mail-qg0-x235.google.com ([2607:f8b0:400d:c04::235]:35020) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z5hPT-0007jh-7T for qemu-devel@nongnu.org; Thu, 18 Jun 2015 17:31:43 -0400 Received: by qgeu36 with SMTP id u36so30686310qge.2 for ; Thu, 18 Jun 2015 14:31:42 -0700 (PDT) From: Don Slutz Message-ID: <558338BB.5080106@Gmail.com> Date: Thu, 18 Jun 2015 17:31:39 -0400 MIME-Version: 1.0 References: <1434642231-24608-1-git-send-email-pbonzini@redhat.com> In-Reply-To: <1434642231-24608-1-git-send-email-pbonzini@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] exec: fixes for access clamping List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , qemu-devel@nongnu.org Cc: hpoussin@reactos.org, aurelien@aurel32.net On 06/18/15 11:43, Paolo Bonzini wrote: > The first patch fixes the remaining problems with Peter Crosthwaite's > access clamping patch (which broke kvmvapic and hence Windows XP/2003). > The second patch actually puts the clamping to good use: by fixing > address_space_translate_internal, the MIPS rc4030 emulation does not need > anymore the address_space_rw hack. > > Please test! > My testing on vmport says that this change also fixed: Subject: [BUGFIX][PATCH v7 1/9] vmport: The io memory region needs to be at least a size of 4 Date: Fri, 12 Jun 2015 10:05:48 -0400 Message-Id: <1434117956-4929-2-git-send-email-dslutz@verizon.com> So as long as this change goes in, the patch "vmport: The io memory region" will be dropped. -Don Slutz > Paolo > > Paolo Bonzini (2): > exec: do not clamp accesses to MMIO regions > exec: clamp accesses against the MemoryRegionSection > > exec.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) >