Compile pflash_cfi01 only once

Push TARGET_WORDS_BIGENDIAN dependency to board level.

Signed-off-by: Blue Swirl <[email protected]>

Compile pflash_cfi02 only once

Push TARGET_WORDS_BIGENDIAN dependency to board level.

Signed-off-by: Blue Swirl <[email protected]>

Compile acpi only once

Use qemu_irqs to trigger CMOS S3 and SMI events.

Avoid using kvm.h, which uses CPUState.

Signed-off-by: Blue Swirl <[email protected]>

Refactor target specific handling, compile vl.c only once

Move target specific functions and RAM handling to arch_init.c.

Add a flag to QEMUOptions structure to indicate for which
architectures the option is allowed, check the flag
in run time and remove conditional code in option handling.

Now that no target dependencies remain, compile vl.c only once
for all targets.

Signed-off-by: Blue Swirl <[email protected]>

Refactor CPUState handling out of vl.c

Signed-off-by: Blue Swirl <[email protected]>

Refactor a few architecture dependent pieces in vl.c

These will be moved later.

Signed-off-by: Blue Swirl <[email protected]>

Move KVM and Xen global flags to vl.c

Signed-off-by: Blue Swirl <[email protected]>

Move cpu_exec_init_all() declaration to qemu-common.h

Let cpu_exec_init_all() be called from non-CPU code.

Signed-off-by: Blue Swirl <[email protected]>

Allow various header files to be included from non-CPU code

Allow balloon.h, gdbstub.h and kvm.h to be included from
non-CPU code.

Signed-off-by: Blue Swirl <[email protected]>

Adjust debug handling

Signed-off-by: Blue Swirl <[email protected]>

tcg/mips: fix branch offset during retranslation

Branch offsets should only be overwritten during relocation, to support
partial retranslation.

Signed-off-by: Aurelien Jarno <[email protected]>

exec: remove dead code

Signed-off-by: Aurelien Jarno <[email protected]>

linux-user/ppc: use the Linux register layout

The dynamic linker converts the Linux layout to the AIX layout and is
reentrant so it won't do it a second time if it's already been
converted. In short it work just fine with either register layout.

OTOH, statically linked binaries expect a Linux layout.

Remove code converting the Linux layout to AIX layout so that all
binaries are presented the Linux Layout.

Signed-off-by: Rob Landley <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

qemu-io: fix aio help texts

Fix a few typos in the help texts for the various aio commands.

Signed-off-by: Christoph Hellwig <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg/arm: Replace qemu_ld32u (left over from previous commit)

Commit 86feb1c860dc38e9c89e787c5210e8191800385e
did not change all occurrences of INDEX_op_qemu_ld32u
for tcg/arm.

Please note that I could not test this patch
(I have currently no arm system available).

Cc: Richard Henderson <[email protected]>
Cc: Aurelien Jarno <[email protected]>
Signed-off-by: Stefan Weil <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

Fix driftfix option

Based on patch by Zachary Amsden.

Signed-off-by: Blue Swirl <[email protected]>

Fix BSD and win32 builds

  CC    net/tap-bsd.o
/src/qemu/net/tap-bsd.c: In function `tap_open':
/src/qemu/net/tap-bsd.c:93: warning: implicit declaration of function `error_report'

  CC    sparc-softmmu/../net/tap-win32.o
cc1: warnings being treated as errors
/src/qemu/target-sparc/../net/tap-win32.c: In function 'net_init_tap':
/src/qemu/target-sparc/../net/tap-win32.c:709: warning: implicit declaration of function 'error_report'

Signed-off-by: Blue Swirl <[email protected]>

Refactor numa mode setting

Signed-off-by: Blue Swirl <[email protected]>

pflash_cfi02: fix incorrect TARGET_FMT_lx/d use

Also use target_phys_addr_t for addresses.

Signed-off-by: Blue Swirl <[email protected]>

Compile vmware_vga only once

Signed-off-by: Blue Swirl <[email protected]>

tcg-mips: add guest base support

Signed-off-by: Aurelien Jarno <[email protected]>

tcg/mips: implement the not_i32 op the same way as gcc

Signed-off-by: Aurelien Jarno <[email protected]>

tcg-mips: implement nor

Signed-off-by: Aurelien Jarno <[email protected]>

be more specific in -mem-path error messages

Signed-Off-By: Michael Tokarev <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

Add a missing #include for FreeBSD hosts

Signed-off-by: Juergen Lock <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

error: Move qerror_report() from qemu-error.[ch] to qerror.[ch]

Signed-off-by: Markus Armbruster <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o

The location tracking interface is used by code shared with qemi-img,
qemu-nbd and qemu-io, so it needs to be available there.  Commit
827b0813 provides it in a rather hamfisted way: it adds a dummy
implementation to qemu-tool.c.

It's cleaner to provide the real thing, and put a few more dummy
monitor functions into qemu-tool.c.

Signed-off-by: Markus Armbruster <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

error: Make use of error_set_progname() optional

Signed-off-by: Markus Armbruster <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

error: Trim includes after "Infrastructure to track locations..."

Missed in commit 827b0813.

Signed-off-by: Markus Armbruster <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

error: Trim includes in qerror.c

Signed-off-by: Markus Armbruster <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

error: Trim includes after "Move qemu_error & friends..."

Missed in commit 2f792016.

Signed-off-by: Markus Armbruster <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

linux-user: Use RLIMIT_STACK for default stack size.

The current default stack limit of 512kB is far too small; a fair
number of gcc testsuite failures (for all guests) are directly
attributable to this.  Using the -s option in every invocation of
the emulator is annoying to say the least.

A reasonable compromise seems to be to honor the system rlimit.
At least on two Linux distributions, this is set to 8MB and 10MB
respectively.  If the system does not limit the stack, then we're
no worse off than before.

At the same time, rename the variable from x86_stack_size and
change the ultimate fallback size from 512kB to 8MB.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

target-arm: disable PAGE_EXEC for XN pages

Don't set PAGE_EXEC for XN pages, to avoid a bypass of XN protection
checking if the page is already in the TLB.

Signed-off-by: Rabin Vincent <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

fix race between timer firing vs. alarm_timer->pending = 0

The period for Win32 timers is very short and always the same
independent of dynticks, so it's possible that the timer fires
before qemu_run_all_timers has reset alarm_timer->pending to zero.
Reset alarm_timer->pending before rearming.

Signed-off-by: Paolo Bonzini <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

qemu-io: Fix return value handling of bdrv_open

bdrv_open may return -errno so we have to check
if the return value is '< 0', not '== -1'.

Signed-off-by: Ryota Ozaki <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

qemu-nbd: Fix invalid usage of the first argument of errx

errx takes the exit status of a process as the first
argument. Passing errno to it is wrong. Instead the
patch lets errx take EXIT_FAILURE.

Signed-off-by: Ryota Ozaki <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

qemu-nbd: Fix return value handling of bdrv_open

bdrv_open may return -errno so we have to check
if the return value is '< 0', not '== -1'.

Signed-off-by: Ryota Ozaki <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

balloon: Fix overflow when reporting actual memory size

Beginning with its introduction, the virtio balloon has had an overflow error
that causes 'info balloon' to misreport the actual memory size when the balloon
itself becomes larger than 4G.  Use a cast when converting dev->actual from
pages to kB to prevent overflows.

Before:
(qemu) info balloon
balloon: actual=5120
(qemu) balloon 1025
(qemu) info balloon
balloon: actual=1025
(qemu) balloon 1024
(qemu) info balloon
balloon: actual=5120

After:
(qemu) info balloon
balloon: actual=5120
(qemu) balloon 1025
(qemu) info balloon
balloon: actual=1025
(qemu) balloon 1024
(qemu) info balloon
balloon: actual=1024

Signed-off-by: Adam Litke <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

json-parser: Output the content of invalid keyword

When input some invalid word 'unknowcmd' through QMP port, qemu outputs
this error message:
  "parse error: invalid keyword `%s'"

This patch makes qemu output the content of invalid keyword, like:

  "parse error: invalid keyword `unknowcmd'"

Signed-off-by: Amos Kong <[email protected]>
Acked-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

raw-posix: Better error return values for hdev_create

Now that we output an error message according to the returned error code in
qemu-img, let's return the real error codes. "Input/output error" for
everything isn't helpful.

Signed-off-by: Kevin Wolf <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

qemu-options.hx: fix a typo

Signed-off-by: Aurelien Jarno <[email protected]>

Compile some MIPS devices only once

Move CPU specific declarations to a separate file.

Signed-off-by: Blue Swirl <[email protected]>

Fix build

Actually some systems don't define PAGE_SIZE. Fixes build breakage
by f7736b91c40a617e93505e32dcbd2cb56aad8a23.

Signed-off-by: Blue Swirl <[email protected]>

Compile ide/core only once

Make win2k install hack unconditional as it is still restricted to
x86 only in vl.c.

Replace TARGET_PAGE_SIZE and 4096 with PAGE_SIZE.

Signed-off-by: Blue Swirl <[email protected]>

target-s390: Don't compile in virtio-pci

As soon as virtio-pci.c gets compiled and used on S390 the internal qdev magic
gets confused and tries to give us PCI devices instead of S390 virtio devices.

Since we don't have PCI on S390, we can safely not compile virtio-pci at all.

In order to do this I added a new config option "CONFIG_VIRTIO_PCI" that I
enabled for every platform except S390. Thanks to this the change should be a
complete nop for every other platform.

Signed-off-by: Alexander Graf <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

target-ppc: generic PowerPC TBL

Time base SPRs TBL/TBU should be accessible in user/priv modes for reading
as specified in POWER ISA documentation. Therefore SPRs permissions were
changed in gen_tbl function.

Signed-off-by: Dmitry Ilyevsky <[email protected]>
Acked-by: Alexander Graf <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Disambiguate qemu_ld32u with 32-bit and 64-bit outputs.

Some targets (e.g. Alpha and MIPS64) need to keep 32-bit operands
sign-extended in 64-bit registers (regardless of the "real" sign
of the operand).  For that, we need to be able to distinguish
between a 32-bit load with a 32-bit result and a 32-bit load with
a given extension to a 64-bit result.  This distinction already
exists for the ld* loads, but not the qemu_ld* loads.

Reserve qemu_ld32u for 64-bit outputs and introduce qemu_ld32 for
32-bit outputs.  Adjust all code generators to match.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Allow target-specific implementation of NOR.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Allow target-specific implementation of NAND.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Allow target-specific implementation of EQV.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Use not_i32 to implement not_i64.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Change TCGType to an enumeration.

The TCGType name was already used consistently.  Changing it
to an enumeration instead of a set of defines aids debugging.

Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Use TCGCond where appropriate.

Use the TCGCond enumeration type in the brcond and setcond
related prototypes in tcg-op.h and each code generator.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg: Name the opcode enumeration.

Give the enumeration formed from tcg-opc.h a name: TCGOpcode.
Use that enumeration type instead of "int" whereever appropriate.

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

remove remaining occurrences AREG[1-9] and TCG_AREG[1-9]

Signed-off-by: Paolo Bonzini <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

linux-user: Add the syscall id for pselect6 on ARM

As this is now supported in newer linux kernels.

Signed-off-by: Michael Casadevall <[email protected]>
Signed-off-by: Riku Voipio <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

linux-user: add inotify_init1 syscall support

New syscall which gets actively used when you have a
fresh kernel.

Signed-off-by: Riku Voipio <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

target-arm: Fix handling of AL condition in IT instruction

Do not try to insert a conditional jump over next instruction when the
condition code is AL as this will trigger an internal error.

Signed-off-by: Johan Bengtsson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg-hppa: Fix 64-bit argument ordering

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

tcg-hppa: Fix const errors in hppa-dis.c

Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Aurelien Jarno <[email protected]>

Fix bsd-user broken by commit b5ec5ce0e39d6e7ea707d5604a5f6d567dfd2f48

Signed-off-by: Juergen Lock <[email protected]>
Signed-off-by: Blue Swirl <[email protected]>

Fix recent pxa270 serial breakage

This fixes a copy/paste bug introduced in commit
2d48377a8531de63ec1d0c4b9b1959dc4b78356c that pushed TARGET_WORDS_BIGENDIAN
dependency to board level.

Signed-off-by: Lars Munch <[email protected]>
Signed-off-by: Blue Swirl <[email protected]>

qemu-error: Avoid build warning.

  CC    qemu-error.o
cc1: warnings being treated as errors
/home/edgar/src/c/qemu/git/qemu/qemu-error.c: In function 'error_print_loc':
/home/edgar/src/c/qemu/git/qemu/qemu-error.c:191: error: format not a string literal and no format arguments
make: *** [qemu-error.o] Error 1

Signed-off-by: Edgar E. Iglesias <[email protected]>

pci_host: fix breakage

This has been broken by commit 952760bb7bce7fbfe0afcf04fee268745f297b87

Signed-off-by: Aurelien Jarno <[email protected]>

Compile virtio-pci only once

Signed-off-by: Blue Swirl <[email protected]>

Compile sound devices only once

Signed-off-by: Blue Swirl <[email protected]>

Fix Sparc64 build

952760bb7bce7fbfe0afcf04fee268745f297b87 missed one change.

Signed-off-by: Blue Swirl <[email protected]>

Compile pci_host only once

Convert pci_host_conf_register_mmio_noswap(x) to
pci_host_conf_register_mmio(x, 0).

Convert pci_host_conf_register_mmio(x) to
pci_host_conf_register_mmio(x, 1) for big endian hosts, all cases
happen to be BE.

Signed-off-by: Blue Swirl <[email protected]>

Compile pcie_host only once

Signed-off-by: Blue Swirl <[email protected]>

Compile pci only once

Move coalesced_mmio declarations to a more accessible location.

Signed-off-by: Blue Swirl <[email protected]>

Compile usb-ohci only once

Push TARGET_WORDS_BIGENDIAN dependency to board level.

Signed-off-by: Blue Swirl <[email protected]>

Compile serial only once

Push TARGET_WORDS_BIGENDIAN dependency to board level.

Signed-off-by: Blue Swirl <[email protected]>

Compile ne2000_isa only once

Signed-off-by: Blue Swirl <[email protected]>

Compile isa_mmio only once

Push TARGET_WORDS_BIGENDIAN dependency to board level.

Signed-off-by: Blue Swirl <[email protected]>

Compile most PCI network cards only once

Signed-off-by: Blue Swirl <[email protected]>

Compile qemu-error only once

Signed-off-by: Blue Swirl <[email protected]>

Compile async only once

Signed-off-by: Blue Swirl <[email protected]>

Compile fdc only once

Signed-off-by: Blue Swirl <[email protected]>

Compile most IDE devices only once

Signed-off-by: Blue Swirl <[email protected]>

Compile usb-uhci only once

Signed-off-by: Blue Swirl <[email protected]>

Compile vga-isa only once

Signed-off-by: Blue Swirl <[email protected]>

Compile pcspk only once

Signed-off-by: Blue Swirl <[email protected]>

Compile i8254 only once

Signed-off-by: Blue Swirl <[email protected]>

Compile parallel only once

Signed-off-by: Blue Swirl <[email protected]>

Compile vga-pci only once

Signed-off-by: Blue Swirl <[email protected]>

Move x86 specific PC declarations to a separate file

x86 definitions (especially CPUState uses) prevent many files from
being compiled within libhw.

Move x86 specific declarations (APIC stuff) to a separate file.

Signed-off-by: Blue Swirl <[email protected]>

Revert 'Fix build'

Partially revert 80a1ab598c6960e7a941e38c5d8638c532c585c3.

Signed-off-by: Blue Swirl <[email protected]>

Revert "Introduce a default qmp session"

This reverts commit 3290c4aac5b97bb1e3b2b28d94669f2c611ce84a.

Conflicts:

vl.c

Revert "qmp: don't make -qmp disable the default monitor"

This reverts commit d49f626ed00cecc90fb1ff88da9bdf11e57094d1.

Revert "tap: invoke downscript when we exit abnormally"

This reverts commit 8af8ce4d6116e3d46ad298ca8fe50d3b515b1aac.

Revert "Convert atexit users to exit_notifier"

This reverts commit d7234f4d7e373a708e1df9ab565a71b71b189025.

Conflicts:

hw/xen_machine_pv.c

This should have never been committed.

Signed-off-by: Anthony Liguori <[email protected]>

Revert "Add exit notifiers"

This reverts commit 3b6304f706ef7eebc0b3b3f3a5093ec75448ee19.

This was mistakenly committed.

Signed-off-by: Anthony Liguori <[email protected]>

Fix a typo

Signed-off-by: Blue Swirl <[email protected]>

Compile disassemblers only once

Signed-off-by: Blue Swirl <[email protected]>

tcg/arm: don't save/restore r7 in prologue/epilogue

There is no need to save r7, it is used to store the address
of the env structure and is not modified by GCC.

Signed-off-by: Aurelien Jarno <[email protected]>

tcg/arm: fix load/store definitions for 32-bit targets

Signed-off-by: Aurelien Jarno <[email protected]>

tcg: protect div2 in tcg/tcg-opc.h

Signed-off-by: Aurelien Jarno <[email protected]>

Fix a typo which broke Xen build

Signed-off-by: Blue Swirl <[email protected]>

Fix mingw32 build

mkdir() only takes path argument on mingw32:

  CC    i386-softmmu/vl.o
/src/qemu/vl.c: In function 'qmp_add_default':
/src/qemu/vl.c:3763: error: too many arguments to function 'mkdir'
/src/qemu/vl.c:3769: error: too many arguments to function 'mkdir'

Signed-off-by: Blue Swirl <[email protected]>

Fix build

CC    curses.o
cc1: warnings being treated as errors
/src/qemu/curses.c: In function 'curses_display_init':
/src/qemu/curses.c:341: error: initialization from incompatible pointer type

Signed-off-by: Blue Swirl <[email protected]>