From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43773) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X5fjm-00071t-4q for qemu-devel@nongnu.org; Fri, 11 Jul 2014 14:40:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X5fjf-0004yV-GE for qemu-devel@nongnu.org; Fri, 11 Jul 2014 14:40:02 -0400 Received: from mx1.redhat.com ([209.132.183.28]:18232) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X5fjf-0004xy-7F for qemu-devel@nongnu.org; Fri, 11 Jul 2014 14:39:55 -0400 Date: Fri, 11 Jul 2014 21:42:04 +0300 From: "Michael S. Tsirkin" Message-ID: <20140711184204.GB3643@redhat.com> References: <20140708140447.7314.87628.stgit@3820> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140708140447.7314.87628.stgit@3820> Subject: Re: [Qemu-devel] [PATCH 0/3] vhost-user fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Nikolay Nikolaev Cc: pbonzini@redhat.com, snabb-devel@googlegroups.com, qemu-devel@nongnu.org, tech@virtualopensystems.com On Tue, Jul 08, 2014 at 05:05:29PM +0300, Nikolay Nikolaev wrote: > The latest vhost-user changes changed the VHOST_SET_MEM_TABLE handling. > Now the memory regions are mapped from dev->mem. The BIOS is registered > at address 0xfffc0000 which is out of memory boundaries for guests with > less than 4G RAM. Calling qemu_get_ram_fd with this address causes abort() > in qemu_get_ram_block with "Bad ram offset". > > To prevent this situation we introduce a new function to check if the address > maps to any RAMBlock - qemu_is_ram_block. This is used in VHOST_SET_MEM_TABLE > handling to revent the aborting call to qemu_get_ram_fd. > > The related vhost-user qtest is also updated to reflect the changes in > vhost-user message structures. Ugh this is not good :(. The above change was merged very late, and now we are seeing fall-out. I guess we'll have to do something like this, bt mkes one wonder what will this change break, in turn. Besides, you are not tagging patches either RFC or 2.1, and you really must. Pls do it next time. And please find more people to Cc using scripts/get_maintainer.pl. Paolo, maybe you could look at patch 1? Thanks! > --- > > Nikolay Nikolaev (3): > Add qemu_is_ram_block > vhost-user: Fix VHOST_SET_MEM_TABLE processing > qtest: Adapt vhost-user-test to latehs vhost-user changes > > > exec.c | 15 +++++++++++++++ > hw/virtio/vhost-user.c | 4 ++++ > include/exec/ram_addr.h | 1 + > tests/vhost-user-test.c | 11 +++++++++-- > 4 files changed, 29 insertions(+), 2 deletions(-) > > -- > Signature