From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49820) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X4W1k-0000gF-QX for qemu-devel@nongnu.org; Tue, 08 Jul 2014 10:05:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X4W1c-0003pR-6S for qemu-devel@nongnu.org; Tue, 08 Jul 2014 10:05:48 -0400 Received: from mail-wg0-f45.google.com ([74.125.82.45]:53269) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X4W1b-0003pI-Sn for qemu-devel@nongnu.org; Tue, 08 Jul 2014 10:05:40 -0400 Received: by mail-wg0-f45.google.com with SMTP id z12so4258669wgg.16 for ; Tue, 08 Jul 2014 07:05:37 -0700 (PDT) From: Nikolay Nikolaev Date: Tue, 08 Jul 2014 17:05:29 +0300 Message-ID: <20140708140447.7314.87628.stgit@3820> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] [PATCH 0/3] vhost-user fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: snabb-devel@googlegroups.com, qemu-devel@nongnu.org, mst@redhat.com Cc: tech@virtualopensystems.com, n.nikolaev@virtualopensystems.com 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. --- 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