From patchwork Mon Aug 13 05:21:05 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Olivia Yin X-Patchwork-Id: 176862 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 035452C0097 for ; Mon, 13 Aug 2012 15:48:54 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751219Ab2HMFsu (ORCPT ); Mon, 13 Aug 2012 01:48:50 -0400 Received: from co1ehsobe003.messaging.microsoft.com ([216.32.180.186]:45139 "EHLO co1outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751226Ab2HMFst (ORCPT ); Mon, 13 Aug 2012 01:48:49 -0400 Received: from mail39-co1-R.bigfish.com (10.243.78.231) by CO1EHSOBE010.bigfish.com (10.243.66.73) with Microsoft SMTP Server id 14.1.225.23; Mon, 13 Aug 2012 05:48:48 +0000 Received: from mail39-co1 (localhost [127.0.0.1]) by mail39-co1-R.bigfish.com (Postfix) with ESMTP id CA4A91401B2; Mon, 13 Aug 2012 05:48:48 +0000 (UTC) X-Forefront-Antispam-Report: CIP:70.37.183.190; KIP:(null); UIP:(null); IPV:NLI; H:mail.freescale.net; RD:none; EFVD:NLI X-SpamScore: 0 X-BigFish: VS0(zzzz1202hzz8275bhz2dh2a8h668h839hd24he5bhf0ah107ah) Received: from mail39-co1 (localhost.localdomain [127.0.0.1]) by mail39-co1 (MessageSwitch) id 1344836926814999_20507; Mon, 13 Aug 2012 05:48:46 +0000 (UTC) Received: from CO1EHSMHS022.bigfish.com (unknown [10.243.78.240]) by mail39-co1.bigfish.com (Postfix) with ESMTP id BAFFA7C0044; Mon, 13 Aug 2012 05:48:46 +0000 (UTC) Received: from mail.freescale.net (70.37.183.190) by CO1EHSMHS022.bigfish.com (10.243.66.32) with Microsoft SMTP Server (TLS) id 14.1.225.23; Mon, 13 Aug 2012 05:48:46 +0000 Received: from az84smr01.freescale.net (10.64.34.197) by 039-SN1MMR1-004.039d.mgd.msft.net (10.84.1.14) with Microsoft SMTP Server (TLS) id 14.2.298.5; Mon, 13 Aug 2012 00:48:45 -0500 Received: from localhost.localdomain (rock.ap.freescale.net [10.193.20.106]) by az84smr01.freescale.net (8.14.3/8.14.0) with ESMTP id q7D5mebv002924; Sun, 12 Aug 2012 22:48:43 -0700 From: Olivia Yin To: , CC: Olivia Yin Subject: [PATCH 2/2] QEMU: register reset handlers to write images into memory Date: Mon, 13 Aug 2012 13:21:05 +0800 Message-ID: <1344835265-23499-2-git-send-email-hong-hua.yin@freescale.com> X-Mailer: git-send-email 1.6.4 In-Reply-To: <1344835265-23499-1-git-send-email-hong-hua.yin@freescale.com> References: <1344835265-23499-1-git-send-email-hong-hua.yin@freescale.com> MIME-Version: 1.0 X-OriginatorOrg: freescale.com Sender: kvm-ppc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm-ppc@vger.kernel.org Instead of add rom blobs, this patch just write them directly to memory. This patch registers reset handler uimage_reset() and image_file_reset() which load images into RAM during initial bootup and VM reset. v3: Use file_load() to sanity check image file. Move cpu_physical_memory_rw() out from uimage_physical_loader() so as not to write uimage twice. v2: Clean initrd_physical_loader(). Use cpu_physical_memory_rw() to write images into physical memory. Signed-off-by: Olivia Yin --- This patch is based on branch 'ppc-next' of Alex's upstream QEMU repo: http://repo.or.cz/r/qemu/agraf.git hw/loader.c | 81 +++++++++++++++++++++++++++++++++++++++++++---------------- 1 files changed, 59 insertions(+), 22 deletions(-) diff --git a/hw/loader.c b/hw/loader.c index 8475850..58fc902 100644 --- a/hw/loader.c +++ b/hw/loader.c @@ -56,6 +56,12 @@ static int roms_loaded; +typedef struct ImageFile ImageFile; +struct ImageFile { + char *name; + target_phys_addr_t addr; +}; + /* return the size or -1 if error */ int get_image_size(const char *filename) { @@ -115,6 +121,17 @@ err: g_free(*data); return -1; } + +static void image_file_reset(void *opaque) +{ + ImageFile *image = opaque; + int size; + uint8_t *data = NULL; + + size = file_load(image->name, &data); + cpu_physical_memory_rw(image->addr, data, size, 1); + g_free(data); +} /* read()-like version */ ssize_t read_targphys(const char *name, @@ -142,7 +159,12 @@ int load_image_targphys(const char *filename, return -1; } if (size > 0) { - rom_add_file_fixed(filename, addr, -1); + ImageFile *image; + image = g_malloc0(sizeof(*image)); + image->name = g_strdup(filename); + image->addr = addr; + + qemu_register_reset(image_file_reset, image); } return size; } @@ -463,15 +485,14 @@ static ssize_t gunzip(void *dst, size_t dstlen, uint8_t *src, return dstbytes; } -/* Load a U-Boot image. */ -int load_uimage(const char *filename, target_phys_addr_t *ep, - target_phys_addr_t *loadaddr, int *is_linux) +/* write uimage into memory */ +static int uimage_physical_loader(const char *filename, uint8_t **data, + target_phys_addr_t *loadaddr) { int fd; int size; uboot_image_header_t h; uboot_image_header_t *hdr = &h; - uint8_t *data = NULL; int ret = -1; fd = open(filename, O_RDONLY | O_BINARY); @@ -504,18 +525,9 @@ int load_uimage(const char *filename, target_phys_addr_t *ep, goto out; } - /* TODO: Check CPU type. */ - if (is_linux) { - if (hdr->ih_os == IH_OS_LINUX) - *is_linux = 1; - else - *is_linux = 0; - } - - *ep = hdr->ih_ep; - data = g_malloc(hdr->ih_size); + *data = g_malloc(hdr->ih_size); - if (read(fd, data, hdr->ih_size) != hdr->ih_size) { + if (read(fd, *data, hdr->ih_size) != hdr->ih_size) { fprintf(stderr, "Error reading file\n"); goto out; } @@ -525,11 +537,11 @@ int load_uimage(const char *filename, target_phys_addr_t *ep, size_t max_bytes; ssize_t bytes; - compressed_data = data; + compressed_data = *data; max_bytes = UBOOT_MAX_GUNZIP_BYTES; - data = g_malloc(max_bytes); + *data = g_malloc(max_bytes); - bytes = gunzip(data, max_bytes, compressed_data, hdr->ih_size); + bytes = gunzip(*data, max_bytes, compressed_data, hdr->ih_size); g_free(compressed_data); if (bytes < 0) { fprintf(stderr, "Unable to decompress gzipped image!\n"); @@ -538,7 +550,6 @@ int load_uimage(const char *filename, target_phys_addr_t *ep, hdr->ih_size = bytes; } - rom_add_blob_fixed(filename, data, hdr->ih_size, hdr->ih_load); if (loadaddr) *loadaddr = hdr->ih_load; @@ -546,12 +557,38 @@ int load_uimage(const char *filename, target_phys_addr_t *ep, ret = hdr->ih_size; out: - if (data) - g_free(data); close(fd); return ret; } +static void uimage_reset(void *opaque) +{ + ImageFile *image = opaque; + uint8_t *data = NULL; + int size; + + size = uimage_physical_loader(image->name, &data, &image->addr); + cpu_physical_memory_rw(image->addr, data, size, 1); + g_free(data); +} + +/* Load a U-Boot image. */ +int load_uimage(const char *filename, target_phys_addr_t *ep, + target_phys_addr_t *loadaddr, int *is_linux) +{ + int size; + ImageFile *image; + uint8_t *data = NULL; + + size= uimage_physical_loader(filename, &data, loadaddr); + g_free(data); + image = g_malloc0(sizeof(*image)); + image->name = g_strdup(filename); + image->addr = *loadaddr; + qemu_register_reset(uimage_reset, image); + return size; +} + /* * Functions for reboot-persistent memory regions. * - used for vga bios and option roms.