Re: [Qemu-devel] [PATCH] Compile ide/core only once

2010-03-26 Thread Aurelien Jarno
On Wed, Mar 24, 2010 at 10:53:19PM +0200, Blue Swirl wrote:
 Make win2k install hack unconditional as it is still restricted to
 x86 only in vl.c.
 
 Replace TARGET_PAGE_SIZE with 4096 because that figure is already
 used later.

This patch has been mangled by your mailer and does not apply correctly
without manual patching.

 Signed-off-by: Blue Swirl blauwir...@gmail.com
 ---
  Makefile.objs|1 +
  Makefile.target  |   11 ---
  default-configs/arm-softmmu.mak  |1 +
  default-configs/i386-softmmu.mak |1 +
  default-configs/mips-softmmu.mak |1 +
  default-configs/mips64-softmmu.mak   |1 +
  default-configs/mips64el-softmmu.mak |1 +
  default-configs/mipsel-softmmu.mak   |1 +
  default-configs/ppc-softmmu.mak  |1 +
  default-configs/ppc64-softmmu.mak|1 +
  default-configs/ppcemb-softmmu.mak   |1 +
  default-configs/sh4-softmmu.mak  |1 +
  default-configs/sh4eb-softmmu.mak|1 +
  default-configs/sparc64-softmmu.mak  |1 +
  default-configs/x86_64-softmmu.mak   |1 +
  hw/ide/core.c|   10 +++---
  vl.c |2 +-
  17 files changed, 22 insertions(+), 15 deletions(-)
 
 diff --git a/Makefile.objs b/Makefile.objs
 index 281f7a6..fe81f6c 100644
 --- a/Makefile.objs
 +++ b/Makefile.objs
 @@ -161,6 +161,7 @@ hw-obj-$(CONFIG_LAN9118) += lan9118.o
  hw-obj-$(CONFIG_NE2000_ISA) += ne2000-isa.o
 
  # IDE
 +hw-obj-$(CONFIG_IDE_CORE) += ide/core.o
  hw-obj-$(CONFIG_IDE_QDEV) += ide/qdev.o
  hw-obj-$(CONFIG_IDE_PCI) += ide/pci.o
  hw-obj-$(CONFIG_IDE_ISA) += ide/isa.o
 diff --git a/Makefile.target b/Makefile.target
 index eb4d010..a17de90 100644
 --- a/Makefile.target
 +++ b/Makefile.target
 @@ -181,8 +181,7 @@ obj-y += rtl8139.o
  obj-y += e1000.o
 
  # Hardware support
 -obj-i386-y = ide/core.o
 -obj-i386-y += pckbd.o dma.o
 +obj-i386-y = pckbd.o dma.o
  obj-i386-y += vga.o
  obj-i386-y += mc146818rtc.o i8259.o pc.o
  obj-i386-y += cirrus_vga.o apic.o ioapic.o acpi.o piix_pci.o
 @@ -191,7 +190,7 @@ obj-i386-y += device-hotplug.o pci-hotplug.o
 smbios.o wdt_ib700.o
  obj-i386-y += debugcon.o multiboot.o
 
  # shared objects
 -obj-ppc-y = ppc.o ide/core.o ide/macio.o
 +obj-ppc-y = ppc.o ide/macio.o
  obj-ppc-y += vga.o dma.o openpic.o
  # PREP target
  obj-ppc-y += pckbd.o i8259.o mc146818rtc.o
 @@ -215,7 +214,6 @@ obj-mips-y += mips_addr.o mips_timer.o mips_int.o
  obj-mips-y += dma.o vga.o i8259.o rc4030.o
  obj-mips-y += vga-isa-mm.o
  obj-mips-y += g364fb.o jazz_led.o dp8393x.o
 -obj-mips-y += ide/core.o
  obj-mips-y += gt64xxx.o pckbd.o mc146818rtc.o acpi.o ds1225y.o
  obj-mips-y += piix4.o cirrus_vga.o
  obj-mips-y += mipsnet.o
 @@ -248,7 +246,6 @@ obj-cris-y += pflash_cfi02.o
 
  ifeq ($(TARGET_ARCH), sparc64)
  obj-sparc-y = sun4u.o pckbd.o apb_pci.o
 -obj-sparc-y += ide/core.o
  obj-sparc-y += vga.o
  obj-sparc-y += mc146818rtc.o
  obj-sparc-y += cirrus_vga.o
 @@ -268,7 +265,7 @@ obj-arm-y += arm-semi.o
  obj-arm-y += pxa2xx.o pxa2xx_pic.o pxa2xx_gpio.o pxa2xx_timer.o pxa2xx_dma.o
  obj-arm-y += pxa2xx_lcd.o pxa2xx_mmci.o pxa2xx_pcmcia.o pxa2xx_keypad.o
  obj-arm-y += pflash_cfi01.o gumstix.o
 -obj-arm-y += zaurus.o ide/core.o ide/microdrive.o spitz.o tosa.o tc6393xb.o
 +obj-arm-y += zaurus.o ide/microdrive.o spitz.o tosa.o tc6393xb.o
  obj-arm-y += omap1.o omap_lcdc.o omap_dma.o omap_clk.o omap_mmc.o omap_i2c.o
  obj-arm-y += omap2.o omap_dss.o soc_dma.o
  obj-arm-y += omap_sx1.o palm.o tsc210x.o
 @@ -282,7 +279,7 @@ obj-arm-y += syborg_virtio.o
 
  obj-sh4-y = shix.o r2d.o sh7750.o sh7750_regnames.o tc58128.o
  obj-sh4-y += sh_timer.o sh_serial.o sh_intc.o sh_pci.o sm501.o
 -obj-sh4-y += ide/core.o ide/mmio.o
 +obj-sh4-y += ide/mmio.o
 
  obj-m68k-y = an5206.o mcf5206.o mcf_uart.o mcf_intc.o mcf5208.o mcf_fec.o
  obj-m68k-y += m68k-semi.o dummy_m68k.o
 diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak
 index 02ad192..ea878a4 100644
 --- a/default-configs/arm-softmmu.mak
 +++ b/default-configs/arm-softmmu.mak
 @@ -8,6 +8,7 @@ CONFIG_ECC=y
  CONFIG_SERIAL=y
  CONFIG_PTIMER=y
  CONFIG_SD=y
 +CONFIG_IDE_CORE=y
  CONFIG_MAX7310=y
  CONFIG_WM8750=y
  CONFIG_TWL92230=y
 diff --git a/default-configs/i386-softmmu.mak 
 b/default-configs/i386-softmmu.mak
 index 4dbf656..59eb670 100644
 --- a/default-configs/i386-softmmu.mak
 +++ b/default-configs/i386-softmmu.mak
 @@ -9,6 +9,7 @@ CONFIG_I8254=y
  CONFIG_PCSPK=y
  CONFIG_USB_UHCI=y
  CONFIG_FDC=y
 +CONFIG_IDE_CORE=y
  CONFIG_IDE_QDEV=y
  CONFIG_IDE_PCI=y
  CONFIG_IDE_ISA=y
 diff --git a/default-configs/mips-softmmu.mak 
 b/default-configs/mips-softmmu.mak
 index 345a093..cb48ed1 100644
 --- a/default-configs/mips-softmmu.mak
 +++ b/default-configs/mips-softmmu.mak
 @@ -10,6 +10,7 @@ CONFIG_I8254=y
  CONFIG_PCSPK=y
  CONFIG_USB_UHCI=y
  CONFIG_FDC=y
 +CONFIG_IDE_CORE=y
  CONFIG_IDE_QDEV=y
  CONFIG_IDE_PCI=y
  CONFIG_IDE_ISA=y
 diff --git 

Re: [Qemu-devel] [PATCH] Compile ide/core only once

2010-03-26 Thread Aurelien Jarno
On Wed, Mar 24, 2010 at 10:53:19PM +0200, Blue Swirl wrote:
 Make win2k install hack unconditional as it is still restricted to
 x86 only in vl.c.
 
 Replace TARGET_PAGE_SIZE with 4096 because that figure is already
 used later.

I don't really know this part of the code, but I would go for arbitrary
value removal, not the reverse.

 Signed-off-by: Blue Swirl blauwir...@gmail.com
 ---
  Makefile.objs|1 +
  Makefile.target  |   11 ---
  default-configs/arm-softmmu.mak  |1 +
  default-configs/i386-softmmu.mak |1 +
  default-configs/mips-softmmu.mak |1 +
  default-configs/mips64-softmmu.mak   |1 +
  default-configs/mips64el-softmmu.mak |1 +
  default-configs/mipsel-softmmu.mak   |1 +
  default-configs/ppc-softmmu.mak  |1 +
  default-configs/ppc64-softmmu.mak|1 +
  default-configs/ppcemb-softmmu.mak   |1 +
  default-configs/sh4-softmmu.mak  |1 +
  default-configs/sh4eb-softmmu.mak|1 +
  default-configs/sparc64-softmmu.mak  |1 +
  default-configs/x86_64-softmmu.mak   |1 +
  hw/ide/core.c|   10 +++---
  vl.c |2 +-
  17 files changed, 22 insertions(+), 15 deletions(-)
 
 diff --git a/Makefile.objs b/Makefile.objs
 index 281f7a6..fe81f6c 100644
 --- a/Makefile.objs
 +++ b/Makefile.objs
 @@ -161,6 +161,7 @@ hw-obj-$(CONFIG_LAN9118) += lan9118.o
  hw-obj-$(CONFIG_NE2000_ISA) += ne2000-isa.o
 
  # IDE
 +hw-obj-$(CONFIG_IDE_CORE) += ide/core.o
  hw-obj-$(CONFIG_IDE_QDEV) += ide/qdev.o
  hw-obj-$(CONFIG_IDE_PCI) += ide/pci.o
  hw-obj-$(CONFIG_IDE_ISA) += ide/isa.o
 diff --git a/Makefile.target b/Makefile.target
 index eb4d010..a17de90 100644
 --- a/Makefile.target
 +++ b/Makefile.target
 @@ -181,8 +181,7 @@ obj-y += rtl8139.o
  obj-y += e1000.o
 
  # Hardware support
 -obj-i386-y = ide/core.o
 -obj-i386-y += pckbd.o dma.o
 +obj-i386-y = pckbd.o dma.o
  obj-i386-y += vga.o
  obj-i386-y += mc146818rtc.o i8259.o pc.o
  obj-i386-y += cirrus_vga.o apic.o ioapic.o acpi.o piix_pci.o
 @@ -191,7 +190,7 @@ obj-i386-y += device-hotplug.o pci-hotplug.o
 smbios.o wdt_ib700.o
  obj-i386-y += debugcon.o multiboot.o
 
  # shared objects
 -obj-ppc-y = ppc.o ide/core.o ide/macio.o
 +obj-ppc-y = ppc.o ide/macio.o
  obj-ppc-y += vga.o dma.o openpic.o
  # PREP target
  obj-ppc-y += pckbd.o i8259.o mc146818rtc.o
 @@ -215,7 +214,6 @@ obj-mips-y += mips_addr.o mips_timer.o mips_int.o
  obj-mips-y += dma.o vga.o i8259.o rc4030.o
  obj-mips-y += vga-isa-mm.o
  obj-mips-y += g364fb.o jazz_led.o dp8393x.o
 -obj-mips-y += ide/core.o
  obj-mips-y += gt64xxx.o pckbd.o mc146818rtc.o acpi.o ds1225y.o
  obj-mips-y += piix4.o cirrus_vga.o
  obj-mips-y += mipsnet.o
 @@ -248,7 +246,6 @@ obj-cris-y += pflash_cfi02.o
 
  ifeq ($(TARGET_ARCH), sparc64)
  obj-sparc-y = sun4u.o pckbd.o apb_pci.o
 -obj-sparc-y += ide/core.o
  obj-sparc-y += vga.o
  obj-sparc-y += mc146818rtc.o
  obj-sparc-y += cirrus_vga.o
 @@ -268,7 +265,7 @@ obj-arm-y += arm-semi.o
  obj-arm-y += pxa2xx.o pxa2xx_pic.o pxa2xx_gpio.o pxa2xx_timer.o pxa2xx_dma.o
  obj-arm-y += pxa2xx_lcd.o pxa2xx_mmci.o pxa2xx_pcmcia.o pxa2xx_keypad.o
  obj-arm-y += pflash_cfi01.o gumstix.o
 -obj-arm-y += zaurus.o ide/core.o ide/microdrive.o spitz.o tosa.o tc6393xb.o
 +obj-arm-y += zaurus.o ide/microdrive.o spitz.o tosa.o tc6393xb.o
  obj-arm-y += omap1.o omap_lcdc.o omap_dma.o omap_clk.o omap_mmc.o omap_i2c.o
  obj-arm-y += omap2.o omap_dss.o soc_dma.o
  obj-arm-y += omap_sx1.o palm.o tsc210x.o
 @@ -282,7 +279,7 @@ obj-arm-y += syborg_virtio.o
 
  obj-sh4-y = shix.o r2d.o sh7750.o sh7750_regnames.o tc58128.o
  obj-sh4-y += sh_timer.o sh_serial.o sh_intc.o sh_pci.o sm501.o
 -obj-sh4-y += ide/core.o ide/mmio.o
 +obj-sh4-y += ide/mmio.o
 
  obj-m68k-y = an5206.o mcf5206.o mcf_uart.o mcf_intc.o mcf5208.o mcf_fec.o
  obj-m68k-y += m68k-semi.o dummy_m68k.o
 diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak
 index 02ad192..ea878a4 100644
 --- a/default-configs/arm-softmmu.mak
 +++ b/default-configs/arm-softmmu.mak
 @@ -8,6 +8,7 @@ CONFIG_ECC=y
  CONFIG_SERIAL=y
  CONFIG_PTIMER=y
  CONFIG_SD=y
 +CONFIG_IDE_CORE=y
  CONFIG_MAX7310=y
  CONFIG_WM8750=y
  CONFIG_TWL92230=y
 diff --git a/default-configs/i386-softmmu.mak 
 b/default-configs/i386-softmmu.mak
 index 4dbf656..59eb670 100644
 --- a/default-configs/i386-softmmu.mak
 +++ b/default-configs/i386-softmmu.mak
 @@ -9,6 +9,7 @@ CONFIG_I8254=y
  CONFIG_PCSPK=y
  CONFIG_USB_UHCI=y
  CONFIG_FDC=y
 +CONFIG_IDE_CORE=y
  CONFIG_IDE_QDEV=y
  CONFIG_IDE_PCI=y
  CONFIG_IDE_ISA=y
 diff --git a/default-configs/mips-softmmu.mak 
 b/default-configs/mips-softmmu.mak
 index 345a093..cb48ed1 100644
 --- a/default-configs/mips-softmmu.mak
 +++ b/default-configs/mips-softmmu.mak
 @@ -10,6 +10,7 @@ CONFIG_I8254=y
  CONFIG_PCSPK=y
  CONFIG_USB_UHCI=y
  CONFIG_FDC=y
 +CONFIG_IDE_CORE=y
  CONFIG_IDE_QDEV=y
  CONFIG_IDE_PCI=y
  CONFIG_IDE_ISA=y
 diff --git 

[Qemu-devel] [PATCH] Compile ide/core only once

2010-03-24 Thread Blue Swirl
Make win2k install hack unconditional as it is still restricted to
x86 only in vl.c.

Replace TARGET_PAGE_SIZE with 4096 because that figure is already
used later.

Signed-off-by: Blue Swirl blauwir...@gmail.com
---
 Makefile.objs|1 +
 Makefile.target  |   11 ---
 default-configs/arm-softmmu.mak  |1 +
 default-configs/i386-softmmu.mak |1 +
 default-configs/mips-softmmu.mak |1 +
 default-configs/mips64-softmmu.mak   |1 +
 default-configs/mips64el-softmmu.mak |1 +
 default-configs/mipsel-softmmu.mak   |1 +
 default-configs/ppc-softmmu.mak  |1 +
 default-configs/ppc64-softmmu.mak|1 +
 default-configs/ppcemb-softmmu.mak   |1 +
 default-configs/sh4-softmmu.mak  |1 +
 default-configs/sh4eb-softmmu.mak|1 +
 default-configs/sparc64-softmmu.mak  |1 +
 default-configs/x86_64-softmmu.mak   |1 +
 hw/ide/core.c|   10 +++---
 vl.c |2 +-
 17 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/Makefile.objs b/Makefile.objs
index 281f7a6..fe81f6c 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -161,6 +161,7 @@ hw-obj-$(CONFIG_LAN9118) += lan9118.o
 hw-obj-$(CONFIG_NE2000_ISA) += ne2000-isa.o

 # IDE
+hw-obj-$(CONFIG_IDE_CORE) += ide/core.o
 hw-obj-$(CONFIG_IDE_QDEV) += ide/qdev.o
 hw-obj-$(CONFIG_IDE_PCI) += ide/pci.o
 hw-obj-$(CONFIG_IDE_ISA) += ide/isa.o
diff --git a/Makefile.target b/Makefile.target
index eb4d010..a17de90 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -181,8 +181,7 @@ obj-y += rtl8139.o
 obj-y += e1000.o

 # Hardware support
-obj-i386-y = ide/core.o
-obj-i386-y += pckbd.o dma.o
+obj-i386-y = pckbd.o dma.o
 obj-i386-y += vga.o
 obj-i386-y += mc146818rtc.o i8259.o pc.o
 obj-i386-y += cirrus_vga.o apic.o ioapic.o acpi.o piix_pci.o
@@ -191,7 +190,7 @@ obj-i386-y += device-hotplug.o pci-hotplug.o
smbios.o wdt_ib700.o
 obj-i386-y += debugcon.o multiboot.o

 # shared objects
-obj-ppc-y = ppc.o ide/core.o ide/macio.o
+obj-ppc-y = ppc.o ide/macio.o
 obj-ppc-y += vga.o dma.o openpic.o
 # PREP target
 obj-ppc-y += pckbd.o i8259.o mc146818rtc.o
@@ -215,7 +214,6 @@ obj-mips-y += mips_addr.o mips_timer.o mips_int.o
 obj-mips-y += dma.o vga.o i8259.o rc4030.o
 obj-mips-y += vga-isa-mm.o
 obj-mips-y += g364fb.o jazz_led.o dp8393x.o
-obj-mips-y += ide/core.o
 obj-mips-y += gt64xxx.o pckbd.o mc146818rtc.o acpi.o ds1225y.o
 obj-mips-y += piix4.o cirrus_vga.o
 obj-mips-y += mipsnet.o
@@ -248,7 +246,6 @@ obj-cris-y += pflash_cfi02.o

 ifeq ($(TARGET_ARCH), sparc64)
 obj-sparc-y = sun4u.o pckbd.o apb_pci.o
-obj-sparc-y += ide/core.o
 obj-sparc-y += vga.o
 obj-sparc-y += mc146818rtc.o
 obj-sparc-y += cirrus_vga.o
@@ -268,7 +265,7 @@ obj-arm-y += arm-semi.o
 obj-arm-y += pxa2xx.o pxa2xx_pic.o pxa2xx_gpio.o pxa2xx_timer.o pxa2xx_dma.o
 obj-arm-y += pxa2xx_lcd.o pxa2xx_mmci.o pxa2xx_pcmcia.o pxa2xx_keypad.o
 obj-arm-y += pflash_cfi01.o gumstix.o
-obj-arm-y += zaurus.o ide/core.o ide/microdrive.o spitz.o tosa.o tc6393xb.o
+obj-arm-y += zaurus.o ide/microdrive.o spitz.o tosa.o tc6393xb.o
 obj-arm-y += omap1.o omap_lcdc.o omap_dma.o omap_clk.o omap_mmc.o omap_i2c.o
 obj-arm-y += omap2.o omap_dss.o soc_dma.o
 obj-arm-y += omap_sx1.o palm.o tsc210x.o
@@ -282,7 +279,7 @@ obj-arm-y += syborg_virtio.o

 obj-sh4-y = shix.o r2d.o sh7750.o sh7750_regnames.o tc58128.o
 obj-sh4-y += sh_timer.o sh_serial.o sh_intc.o sh_pci.o sm501.o
-obj-sh4-y += ide/core.o ide/mmio.o
+obj-sh4-y += ide/mmio.o

 obj-m68k-y = an5206.o mcf5206.o mcf_uart.o mcf_intc.o mcf5208.o mcf_fec.o
 obj-m68k-y += m68k-semi.o dummy_m68k.o
diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak
index 02ad192..ea878a4 100644
--- a/default-configs/arm-softmmu.mak
+++ b/default-configs/arm-softmmu.mak
@@ -8,6 +8,7 @@ CONFIG_ECC=y
 CONFIG_SERIAL=y
 CONFIG_PTIMER=y
 CONFIG_SD=y
+CONFIG_IDE_CORE=y
 CONFIG_MAX7310=y
 CONFIG_WM8750=y
 CONFIG_TWL92230=y
diff --git a/default-configs/i386-softmmu.mak b/default-configs/i386-softmmu.mak
index 4dbf656..59eb670 100644
--- a/default-configs/i386-softmmu.mak
+++ b/default-configs/i386-softmmu.mak
@@ -9,6 +9,7 @@ CONFIG_I8254=y
 CONFIG_PCSPK=y
 CONFIG_USB_UHCI=y
 CONFIG_FDC=y
+CONFIG_IDE_CORE=y
 CONFIG_IDE_QDEV=y
 CONFIG_IDE_PCI=y
 CONFIG_IDE_ISA=y
diff --git a/default-configs/mips-softmmu.mak b/default-configs/mips-softmmu.mak
index 345a093..cb48ed1 100644
--- a/default-configs/mips-softmmu.mak
+++ b/default-configs/mips-softmmu.mak
@@ -10,6 +10,7 @@ CONFIG_I8254=y
 CONFIG_PCSPK=y
 CONFIG_USB_UHCI=y
 CONFIG_FDC=y
+CONFIG_IDE_CORE=y
 CONFIG_IDE_QDEV=y
 CONFIG_IDE_PCI=y
 CONFIG_IDE_ISA=y
diff --git a/default-configs/mips64-softmmu.mak
b/default-configs/mips64-softmmu.mak
index 5900ee6..585d6bb 100644
--- a/default-configs/mips64-softmmu.mak
+++ b/default-configs/mips64-softmmu.mak
@@ -10,6 +10,7 @@ CONFIG_I8254=y
 CONFIG_PCSPK=y
 CONFIG_USB_UHCI=y
 CONFIG_FDC=y
+CONFIG_IDE_CORE=y
 CONFIG_IDE_QDEV=y