From patchwork Tue Mar 17 02:52:13 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nolan X-Patchwork-Id: 12529 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n2H32eFJ011783 for ; Tue, 17 Mar 2009 03:02:40 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752850AbZCQDCi (ORCPT ); Mon, 16 Mar 2009 23:02:38 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752998AbZCQDCi (ORCPT ); Mon, 16 Mar 2009 23:02:38 -0400 Received: from phong.sigbus.net ([65.49.35.42]:52031 "EHLO phong.sigbus.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752219AbZCQDCi (ORCPT ); Mon, 16 Mar 2009 23:02:38 -0400 X-Greylist: delayed 621 seconds by postgrey-1.27 at vger.kernel.org; Mon, 16 Mar 2009 23:02:38 EDT Received: from [192.168.0.3] (c-71-202-202-194.hsd1.ca.comcast.net [71.202.202.194]) by phong.sigbus.net (Postfix) with ESMTPSA id CF43195C0D3 for ; Mon, 16 Mar 2009 19:52:13 -0700 (PDT) Subject: [PATCH] fix extboot from boot with cache=off From: Nolan To: kvm@vger.kernel.org Date: Mon, 16 Mar 2009 19:52:13 -0700 Message-Id: <1237258333.15350.62.camel@voxel> Mime-Version: 1.0 X-Mailer: Evolution 2.24.3 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org Extboot submits requests with whatever buffer alignment the guest gave to the BIOS. This breaks with O_DIRECT disks, as they require 512 byte alignment. Most guest bootloaders sector align their requests out of paranoia, but the OpenBSD bootloader does not. This patch always copies. Since extboot is only used at boot time to load limited amounts of data, the overhead is not problematic. I also switched to using cpu_physical_memory_* instead of groveling around with phys_ram_base directly. Signed-off-by: Nolan Leake sigbus.net> --- qemu/hw/extboot.c | 35 ++++++++++++++++++++++++----------- 1 files changed, 24 insertions(+), 11 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/qemu/hw/extboot.c b/qemu/hw/extboot.c index ada0fdd..ea16b3e 100644 --- a/qemu/hw/extboot.c +++ b/qemu/hw/extboot.c @@ -77,19 +77,29 @@ static void extboot_write_cmd(void *opaque, uint32_t addr, uint32_t value) BlockDriverState *bs = opaque; int cylinders, heads, sectors, err; uint64_t nb_sectors; - - get_translated_chs(bs, &cylinders, &heads, §ors); + target_phys_addr_t pa; + int blen; + void *buf = NULL; if (cmd->type == 0x01 || cmd->type == 0x02) { - target_ulong pa = cmd->xfer.segment * 16 + cmd->xfer.segment; + pa = cmd->xfer.segment * 16 + cmd->xfer.offset; + blen = cmd->xfer.nb_sectors * 512; + buf = qemu_memalign(512, blen); + if (!buf) { + printf("qemu_memalign failed\n"); + return; + } /* possible buffer overflow */ - if ((pa + cmd->xfer.nb_sectors * 512) > phys_ram_size) + if ((pa + blen) > phys_ram_size) { + qemu_free(buf); return; + } } switch (cmd->type) { case 0x00: + get_translated_chs(bs, &cylinders, &heads, §ors); bdrv_get_geometry(bs, &nb_sectors); cmd->query_geometry.cylinders = cylinders; cmd->query_geometry.heads = heads; @@ -98,22 +108,25 @@ static void extboot_write_cmd(void *opaque, uint32_t addr, uint32_t value) cpu_physical_memory_set_dirty((value & 0xFFFF) << 4); break; case 0x01: - err = bdrv_read(bs, cmd->xfer.sector, phys_ram_base + - cmd->xfer.segment * 16 + cmd->xfer.offset, - cmd->xfer.nb_sectors); + err = bdrv_read(bs, cmd->xfer.sector, buf, cmd->xfer.nb_sectors); if (err) printf("Read failed\n"); + + cpu_physical_memory_write(pa, buf, blen); + break; case 0x02: - err = bdrv_write(bs, cmd->xfer.sector, phys_ram_base + - cmd->xfer.segment * 16 + cmd->xfer.offset, - cmd->xfer.nb_sectors); + cpu_physical_memory_read(pa, buf, blen); + + err = bdrv_write(bs, cmd->xfer.sector, buf, cmd->xfer.nb_sectors); if (err) printf("Write failed\n"); - cpu_physical_memory_set_dirty(cmd->xfer.segment * 16 + cmd->xfer.offset); break; } + + if (buf) + qemu_free(buf); } void extboot_init(BlockDriverState *bs, int cmd)