All PC machines now use the multiboot_dma.bin binary,
we can remove the non-DMA version (multiboot.bin).

Suggested-by: Thomas Huth <th...@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
---
 pc-bios/optionrom/optionrom.h     |   4 -
 hw/i386/pc.c                      |   1 -
 pc-bios/meson.build               |   1 -
 pc-bios/multiboot.bin             | Bin 1024 -> 0 bytes
 pc-bios/optionrom/Makefile        |   2 +-
 pc-bios/optionrom/multiboot.S     | 232 -----------------------------
 pc-bios/optionrom/multiboot_dma.S | 234 +++++++++++++++++++++++++++++-
 7 files changed, 233 insertions(+), 241 deletions(-)
 delete mode 100644 pc-bios/multiboot.bin
 delete mode 100644 pc-bios/optionrom/multiboot.S

diff --git a/pc-bios/optionrom/optionrom.h b/pc-bios/optionrom/optionrom.h
index 7bcdf0eeb24..2e6e2493f83 100644
--- a/pc-bios/optionrom/optionrom.h
+++ b/pc-bios/optionrom/optionrom.h
@@ -117,16 +117,12 @@
  *
  * Clobbers: %eax, %edx, %es, %ecx, %edi and adresses %esp-20 to %esp
  */
-#ifdef USE_FW_CFG_DMA
 #define read_fw_blob_dma(var)                           \
         read_fw         var ## _SIZE;                   \
         mov             %eax, %ecx;                     \
         read_fw         var ## _ADDR;                   \
         mov             %eax, %edi ;                    \
         read_fw_dma     var ## _DATA, %ecx, %edi
-#else
-#define read_fw_blob_dma(var) read_fw_blob(var)
-#endif
 
 #define read_fw_blob_pre(var)                           \
         read_fw         var ## _SIZE;                   \
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 49632b69d29..524d2fd98e8 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -657,7 +657,6 @@ void xen_load_linux(PCMachineState *pcms)
         assert(!strcmp(option_rom[i].name, "linuxboot.bin") ||
                !strcmp(option_rom[i].name, "linuxboot_dma.bin") ||
                !strcmp(option_rom[i].name, "pvh.bin") ||
-               !strcmp(option_rom[i].name, "multiboot.bin") ||
                !strcmp(option_rom[i].name, "multiboot_dma.bin"));
         rom_add_option(option_rom[i].name, option_rom[i].bootindex);
     }
diff --git a/pc-bios/meson.build b/pc-bios/meson.build
index 79bb2e18008..f2d4dc416a4 100644
--- a/pc-bios/meson.build
+++ b/pc-bios/meson.build
@@ -62,7 +62,6 @@ blobs = [
   'efi-e1000e.rom',
   'efi-vmxnet3.rom',
   'qemu-nsis.bmp',
-  'multiboot.bin',
   'multiboot_dma.bin',
   'linuxboot.bin',
   'linuxboot_dma.bin',
diff --git a/pc-bios/multiboot.bin b/pc-bios/multiboot.bin
deleted file mode 100644
index 
e772713c95749bee82c20002b50ec6d05b2d4987..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 1024
zcmeHFF-Tic6utlZQ$OjD#Hxcx2u0GNQv6GySOkZR(ulaX<>%N!Y#>cWhY}nf36J7X
zN(%*X6NHY>xcqO11dG^02a8L@B~ihln|%1*|7(haWa`)l@80w7;U4Ziyv0rZ8{K-w
zX(Ib3tLZ)RgZ}w1ei{~QEqQq9q1J-iHc<Nk_t=0qf%XfPZVHw2DY#ujc2P|}*P%6X
z5J{pPlX4<yP&N5pXK;s@UhB~&!E&UdqEwGZF6xQMIcu9YL#zeSRCoLGt{Nh+08t>}
z{m%E-b32A??+@XljSYirtWObwngi<ymS1Ta*dFGMp;8@=_3Z52!v09{UU~`QnFsB_
z7BiEC)uZxH%SW9kPWCicN{_iUjmk<~D^H|R%@|m9%43Woc(Pkeq%n{&8ND5Z*tPt#
zJua+xXAQhN4K(1MMs0{u_6sp>l#NmvPZ7KC<lsLdQgMFC@A6POvMoDMgW=M+K;T<o
zTkpnNq6x*`vM0CGE>t40l-CQ}*)y<y--c)(x}o&1TMzwX+ToGS@VER4zR&s70fl+(
qI)9<-H_-!n#lLJmGq*^~<$US&%R-@)$`@YPx#A6#|L`9<;9UXPG71m?

diff --git a/pc-bios/optionrom/Makefile b/pc-bios/optionrom/Makefile
index 30d07026c79..1183ef88922 100644
--- a/pc-bios/optionrom/Makefile
+++ b/pc-bios/optionrom/Makefile
@@ -2,7 +2,7 @@ include config.mak
 SRC_DIR := $(TOPSRC_DIR)/pc-bios/optionrom
 VPATH = $(SRC_DIR)
 
-all: multiboot.bin multiboot_dma.bin linuxboot.bin linuxboot_dma.bin 
kvmvapic.bin pvh.bin
+all: multiboot_dma.bin linuxboot.bin linuxboot_dma.bin kvmvapic.bin pvh.bin
 # Dummy command so that make thinks it has done something
        @true
 
diff --git a/pc-bios/optionrom/multiboot.S b/pc-bios/optionrom/multiboot.S
deleted file mode 100644
index 181a4b03a3f..00000000000
--- a/pc-bios/optionrom/multiboot.S
+++ /dev/null
@@ -1,232 +0,0 @@
-/*
- * Multiboot Option ROM
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, see <http://www.gnu.org/licenses/>.
- *
- * Copyright Novell Inc, 2009
- *   Authors: Alexander Graf <ag...@suse.de>
- */
-
-#include "optionrom.h"
-
-#define BOOT_ROM_PRODUCT "multiboot loader"
-
-#define MULTIBOOT_MAGIC                0x2badb002
-
-#define GS_PROT_JUMP           0
-#define GS_GDT_DESC            6
-
-
-BOOT_ROM_START
-
-run_multiboot:
-
-       cli
-       cld
-
-       mov             %cs, %eax
-       shl             $0x4, %eax
-
-       /* set up a long jump descriptor that is PC relative */
-
-       /* move stack memory to %gs */
-       mov             %ss, %ecx
-       shl             $0x4, %ecx
-       mov             %esp, %ebx
-       add             %ebx, %ecx
-       sub             $0x20, %ecx
-       sub             $0x30, %esp
-       shr             $0x4, %ecx
-       mov             %cx, %gs
-
-       /* now push the indirect jump descriptor there */
-       mov             (prot_jump), %ebx
-       add             %eax, %ebx
-       movl            %ebx, %gs:GS_PROT_JUMP
-       mov             $8, %bx
-       movw            %bx, %gs:GS_PROT_JUMP + 4
-
-       /* fix the gdt descriptor to be PC relative */
-       movw            (gdt_desc), %bx
-       movw            %bx, %gs:GS_GDT_DESC
-       movl            (gdt_desc+2), %ebx
-       add             %eax, %ebx
-       movl            %ebx, %gs:GS_GDT_DESC + 2
-
-       xor             %eax, %eax
-       mov             %eax, %es
-
-       /* Read the bootinfo struct into RAM */
-       read_fw_blob_dma(FW_CFG_INITRD)
-
-       /* FS = bootinfo_struct */
-       read_fw         FW_CFG_INITRD_ADDR
-       shr             $4, %eax
-       mov             %ax, %fs
-
-       /* Account for the EBDA in the multiboot structure's e801
-        * map.
-        */
-       int             $0x12
-       cwtl
-       movl            %eax, %fs:4
-
-       /* ES = mmap_addr */
-       mov             %fs:48, %eax
-       shr             $4, %eax
-       mov             %ax, %es
-
-       /* Initialize multiboot mmap structs using int 0x15(e820) */
-       xor             %ebx, %ebx
-       /* Start storing mmap data at %es:0 */
-       xor             %edi, %edi
-
-mmap_loop:
-       /* The multiboot entry size has offset -4, so leave some space */
-       add             $4, %di
-       /* entry size (mmap struct) & max buffer size (int15) */
-       movl            $20, %ecx
-       /* e820 */
-       movl            $0x0000e820, %eax
-       /* 'SMAP' magic */
-       movl            $0x534d4150, %edx
-       int             $0x15
-
-mmap_check_entry:
-       /* Error or last entry already done? */
-       jb              mmap_done
-
-mmap_store_entry:
-       /* store entry size */
-       /* old as(1) doesn't like this insn so emit the bytes instead:
-       movl            %ecx, %es:-4(%edi)
-       */
-       .dc.b           0x26,0x67,0x66,0x89,0x4f,0xfc
-
-       /* %edi += entry_size, store as mbs_mmap_length */
-       add             %ecx, %edi
-       movw            %di, %fs:0x2c
-
-       /* Continuation value 0 means last entry */
-       test            %ebx, %ebx
-       jnz             mmap_loop
-
-mmap_done:
-       /* Calculate upper_mem field: The amount of memory between 1 MB and
-          the first upper memory hole. Get it from the mmap. */
-       xor             %di, %di
-       mov             $0x100000, %edx
-upper_mem_entry:
-       cmp             %fs:0x2c, %di
-       je              upper_mem_done
-       add             $4, %di
-
-       /* Skip if type != 1 */
-       cmpl            $1, %es:16(%di)
-       jne             upper_mem_next
-
-       /* Skip if > 4 GB */
-       movl            %es:4(%di), %eax
-       test            %eax, %eax
-       jnz             upper_mem_next
-
-       /* Check for contiguous extension (base <= %edx < base + length) */
-       movl            %es:(%di), %eax
-       cmp             %eax, %edx
-       jb              upper_mem_next
-       addl            %es:8(%di), %eax
-       cmp             %eax, %edx
-       jae             upper_mem_next
-
-       /* If so, update %edx, and restart the search (mmap isn't ordered) */
-       mov             %eax, %edx
-       xor             %di, %di
-       jmp             upper_mem_entry
-
-upper_mem_next:
-       addl            %es:-4(%di), %edi
-       jmp             upper_mem_entry
-
-upper_mem_done:
-       sub             $0x100000, %edx
-       shr             $10, %edx
-       mov             %edx, %fs:0x8
-
-real_to_prot:
-       /* Load the GDT before going into protected mode */
-lgdt:
-       data32 lgdt     %gs:GS_GDT_DESC
-
-       /* get us to protected mode now */
-       movl            $1, %eax
-       movl            %eax, %cr0
-
-       /* the LJMP sets CS for us and gets us to 32-bit */
-ljmp:
-       data32 ljmp     *%gs:GS_PROT_JUMP
-
-prot_mode:
-.code32
-
-       /* initialize all other segments */
-       movl            $0x10, %eax
-       movl            %eax, %ss
-       movl            %eax, %ds
-       movl            %eax, %es
-       movl            %eax, %fs
-       movl            %eax, %gs
-
-       /* Read the kernel and modules into RAM */
-       read_fw_blob_dma(FW_CFG_KERNEL)
-
-       /* Jump off to the kernel */
-       read_fw         FW_CFG_KERNEL_ENTRY
-       mov             %eax, %ecx
-
-       /* EBX contains a pointer to the bootinfo struct */
-       read_fw         FW_CFG_INITRD_ADDR
-       movl            %eax, %ebx
-
-       /* EAX has to contain the magic */
-       movl            $MULTIBOOT_MAGIC, %eax
-ljmp2:
-       jmp             *%ecx
-
-/* Variables */
-.align 4, 0
-prot_jump:     .long prot_mode
-               .short 8
-
-.align 4, 0
-gdt:
-       /* 0x00 */
-.byte  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00
-
-       /* 0x08: code segment (base=0, limit=0xfffff, type=32bit code 
exec/read, DPL=0, 4k) */
-.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x9a, 0xcf, 0x00
-
-       /* 0x10: data segment (base=0, limit=0xfffff, type=32bit data 
read/write, DPL=0, 4k) */
-.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x92, 0xcf, 0x00
-
-       /* 0x18: code segment (base=0, limit=0x0ffff, type=16bit code 
exec/read/conf, DPL=0, 1b) */
-.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x9e, 0x00, 0x00
-
-       /* 0x20: data segment (base=0, limit=0x0ffff, type=16bit data 
read/write, DPL=0, 1b) */
-.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x92, 0x00, 0x00
-
-gdt_desc:
-.short (5 * 8) - 1
-.long  gdt
-
-BOOT_ROM_END
diff --git a/pc-bios/optionrom/multiboot_dma.S 
b/pc-bios/optionrom/multiboot_dma.S
index d809af3e23f..181a4b03a3f 100644
--- a/pc-bios/optionrom/multiboot_dma.S
+++ b/pc-bios/optionrom/multiboot_dma.S
@@ -1,2 +1,232 @@
-#define USE_FW_CFG_DMA 1
-#include "multiboot.S"
+/*
+ * Multiboot Option ROM
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, see <http://www.gnu.org/licenses/>.
+ *
+ * Copyright Novell Inc, 2009
+ *   Authors: Alexander Graf <ag...@suse.de>
+ */
+
+#include "optionrom.h"
+
+#define BOOT_ROM_PRODUCT "multiboot loader"
+
+#define MULTIBOOT_MAGIC                0x2badb002
+
+#define GS_PROT_JUMP           0
+#define GS_GDT_DESC            6
+
+
+BOOT_ROM_START
+
+run_multiboot:
+
+       cli
+       cld
+
+       mov             %cs, %eax
+       shl             $0x4, %eax
+
+       /* set up a long jump descriptor that is PC relative */
+
+       /* move stack memory to %gs */
+       mov             %ss, %ecx
+       shl             $0x4, %ecx
+       mov             %esp, %ebx
+       add             %ebx, %ecx
+       sub             $0x20, %ecx
+       sub             $0x30, %esp
+       shr             $0x4, %ecx
+       mov             %cx, %gs
+
+       /* now push the indirect jump descriptor there */
+       mov             (prot_jump), %ebx
+       add             %eax, %ebx
+       movl            %ebx, %gs:GS_PROT_JUMP
+       mov             $8, %bx
+       movw            %bx, %gs:GS_PROT_JUMP + 4
+
+       /* fix the gdt descriptor to be PC relative */
+       movw            (gdt_desc), %bx
+       movw            %bx, %gs:GS_GDT_DESC
+       movl            (gdt_desc+2), %ebx
+       add             %eax, %ebx
+       movl            %ebx, %gs:GS_GDT_DESC + 2
+
+       xor             %eax, %eax
+       mov             %eax, %es
+
+       /* Read the bootinfo struct into RAM */
+       read_fw_blob_dma(FW_CFG_INITRD)
+
+       /* FS = bootinfo_struct */
+       read_fw         FW_CFG_INITRD_ADDR
+       shr             $4, %eax
+       mov             %ax, %fs
+
+       /* Account for the EBDA in the multiboot structure's e801
+        * map.
+        */
+       int             $0x12
+       cwtl
+       movl            %eax, %fs:4
+
+       /* ES = mmap_addr */
+       mov             %fs:48, %eax
+       shr             $4, %eax
+       mov             %ax, %es
+
+       /* Initialize multiboot mmap structs using int 0x15(e820) */
+       xor             %ebx, %ebx
+       /* Start storing mmap data at %es:0 */
+       xor             %edi, %edi
+
+mmap_loop:
+       /* The multiboot entry size has offset -4, so leave some space */
+       add             $4, %di
+       /* entry size (mmap struct) & max buffer size (int15) */
+       movl            $20, %ecx
+       /* e820 */
+       movl            $0x0000e820, %eax
+       /* 'SMAP' magic */
+       movl            $0x534d4150, %edx
+       int             $0x15
+
+mmap_check_entry:
+       /* Error or last entry already done? */
+       jb              mmap_done
+
+mmap_store_entry:
+       /* store entry size */
+       /* old as(1) doesn't like this insn so emit the bytes instead:
+       movl            %ecx, %es:-4(%edi)
+       */
+       .dc.b           0x26,0x67,0x66,0x89,0x4f,0xfc
+
+       /* %edi += entry_size, store as mbs_mmap_length */
+       add             %ecx, %edi
+       movw            %di, %fs:0x2c
+
+       /* Continuation value 0 means last entry */
+       test            %ebx, %ebx
+       jnz             mmap_loop
+
+mmap_done:
+       /* Calculate upper_mem field: The amount of memory between 1 MB and
+          the first upper memory hole. Get it from the mmap. */
+       xor             %di, %di
+       mov             $0x100000, %edx
+upper_mem_entry:
+       cmp             %fs:0x2c, %di
+       je              upper_mem_done
+       add             $4, %di
+
+       /* Skip if type != 1 */
+       cmpl            $1, %es:16(%di)
+       jne             upper_mem_next
+
+       /* Skip if > 4 GB */
+       movl            %es:4(%di), %eax
+       test            %eax, %eax
+       jnz             upper_mem_next
+
+       /* Check for contiguous extension (base <= %edx < base + length) */
+       movl            %es:(%di), %eax
+       cmp             %eax, %edx
+       jb              upper_mem_next
+       addl            %es:8(%di), %eax
+       cmp             %eax, %edx
+       jae             upper_mem_next
+
+       /* If so, update %edx, and restart the search (mmap isn't ordered) */
+       mov             %eax, %edx
+       xor             %di, %di
+       jmp             upper_mem_entry
+
+upper_mem_next:
+       addl            %es:-4(%di), %edi
+       jmp             upper_mem_entry
+
+upper_mem_done:
+       sub             $0x100000, %edx
+       shr             $10, %edx
+       mov             %edx, %fs:0x8
+
+real_to_prot:
+       /* Load the GDT before going into protected mode */
+lgdt:
+       data32 lgdt     %gs:GS_GDT_DESC
+
+       /* get us to protected mode now */
+       movl            $1, %eax
+       movl            %eax, %cr0
+
+       /* the LJMP sets CS for us and gets us to 32-bit */
+ljmp:
+       data32 ljmp     *%gs:GS_PROT_JUMP
+
+prot_mode:
+.code32
+
+       /* initialize all other segments */
+       movl            $0x10, %eax
+       movl            %eax, %ss
+       movl            %eax, %ds
+       movl            %eax, %es
+       movl            %eax, %fs
+       movl            %eax, %gs
+
+       /* Read the kernel and modules into RAM */
+       read_fw_blob_dma(FW_CFG_KERNEL)
+
+       /* Jump off to the kernel */
+       read_fw         FW_CFG_KERNEL_ENTRY
+       mov             %eax, %ecx
+
+       /* EBX contains a pointer to the bootinfo struct */
+       read_fw         FW_CFG_INITRD_ADDR
+       movl            %eax, %ebx
+
+       /* EAX has to contain the magic */
+       movl            $MULTIBOOT_MAGIC, %eax
+ljmp2:
+       jmp             *%ecx
+
+/* Variables */
+.align 4, 0
+prot_jump:     .long prot_mode
+               .short 8
+
+.align 4, 0
+gdt:
+       /* 0x00 */
+.byte  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00
+
+       /* 0x08: code segment (base=0, limit=0xfffff, type=32bit code 
exec/read, DPL=0, 4k) */
+.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x9a, 0xcf, 0x00
+
+       /* 0x10: data segment (base=0, limit=0xfffff, type=32bit data 
read/write, DPL=0, 4k) */
+.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x92, 0xcf, 0x00
+
+       /* 0x18: code segment (base=0, limit=0x0ffff, type=16bit code 
exec/read/conf, DPL=0, 1b) */
+.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x9e, 0x00, 0x00
+
+       /* 0x20: data segment (base=0, limit=0x0ffff, type=16bit data 
read/write, DPL=0, 1b) */
+.byte  0xff, 0xff, 0x00, 0x00, 0x00, 0x92, 0x00, 0x00
+
+gdt_desc:
+.short (5 * 8) - 1
+.long  gdt
+
+BOOT_ROM_END
-- 
2.47.1


Reply via email to