All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] [PULL 15/22] hw/core/loader.c: Remove load_image()
Date: Fri, 14 Dec 2018 14:42:07 +0000	[thread overview]
Message-ID: <20181214144214.1260-16-peter.maydell@linaro.org> (raw)
In-Reply-To: <20181214144214.1260-1-peter.maydell@linaro.org>

The load_image() function is now no longer used anywhere, so
we can remove it completely. (Use load_image_size() or
g_file_get_contents() instead.)

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 20181130151712.2312-10-peter.maydell@linaro.org
---
 include/hw/loader.h |  1 -
 hw/core/loader.c    | 25 -------------------------
 2 files changed, 26 deletions(-)

diff --git a/include/hw/loader.h b/include/hw/loader.h
index 67a0af84ac3..3766559bc24 100644
--- a/include/hw/loader.h
+++ b/include/hw/loader.h
@@ -11,7 +11,6 @@
  * On error, errno is also set as appropriate.
  */
 int64_t get_image_size(const char *filename);
-int load_image(const char *filename, uint8_t *addr); /* deprecated */
 ssize_t load_image_size(const char *filename, void *addr, size_t size);
 
 /**load_image_targphys_as:
diff --git a/hw/core/loader.c b/hw/core/loader.c
index 66a616608af..fa41842280a 100644
--- a/hw/core/loader.c
+++ b/hw/core/loader.c
@@ -73,31 +73,6 @@ int64_t get_image_size(const char *filename)
     return size;
 }
 
-/* return the size or -1 if error */
-/* deprecated, because caller does not specify buffer size! */
-int load_image(const char *filename, uint8_t *addr)
-{
-    int fd, size;
-    fd = open(filename, O_RDONLY | O_BINARY);
-    if (fd < 0)
-        return -1;
-    size = lseek(fd, 0, SEEK_END);
-    if (size == -1) {
-        fprintf(stderr, "file %-20s: get size error: %s\n",
-                filename, strerror(errno));
-        close(fd);
-        return -1;
-    }
-
-    lseek(fd, 0, SEEK_SET);
-    if (read(fd, addr, size) != size) {
-        close(fd);
-        return -1;
-    }
-    close(fd);
-    return size;
-}
-
 /* return the size or -1 if error */
 ssize_t load_image_size(const char *filename, void *addr, size_t size)
 {
-- 
2.19.2

  parent reply	other threads:[~2018-12-14 14:42 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-14 14:41 [Qemu-devel] [PULL 00/22] misc queue Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 01/22] scripts/checkpatch.pl: Enforce multiline comment syntax Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 02/22] exec.c: Rename cpu_physical_memory_write_rom_internal() Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 03/22] Rename cpu_physical_memory_write_rom() to address_space_write_rom() Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 04/22] disas.c: Use address_space_read() to read memory Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 05/22] monitor: " Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 06/22] elf_ops.h: Use address_space_write() to write memory Peter Maydell
2018-12-14 14:41 ` [Qemu-devel] [PULL 07/22] hw/ppc/mac_newworld, mac_oldworld: Don't use load_image() Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 08/22] hw/ppc/ppc405_boards: " Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 09/22] hw/smbios/smbios.c: " Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 10/22] hw/pci/pci.c: " Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 11/22] hw/i386/pc.c: " Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 12/22] hw/i386/multiboot.c: " Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 13/22] hw/block/tc58128.c: " Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 14/22] device_tree.c: " Peter Maydell
2018-12-14 14:42 ` Peter Maydell [this message]
2018-12-14 14:42 ` [Qemu-devel] [PULL 16/22] include/hw/loader.h: Document load_image_size() Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 17/22] target/arm: Free name string in ARMCPRegInfo hashtable entries Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 18/22] hw/arm/mps2-tz.c: Free mscname string in make_dma() Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 19/22] hw/sd/sdhci: Don't leak memory region in sdhci_sysbus_realize() Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 20/22] tests/test-arm-mptimer: Don't leak string memory Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 21/22] target/arm: Create timers in realize, not init Peter Maydell
2018-12-14 14:42 ` [Qemu-devel] [PULL 22/22] virt: Fix broken indentation Peter Maydell
2018-12-16 21:48 ` [Qemu-devel] [PULL 00/22] misc queue Peter Maydell
2018-12-16 21:49 ` Peter Maydell
2018-12-23  7:58 ` no-reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181214144214.1260-16-peter.maydell@linaro.org \
    --to=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.