summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* spice: simple displayGerd Hoffmann2010-09-215-1/+490
| | | | | | | | | With that patch applied you'll actually see the guests screen in the spice client. This does *not* bring qxl and full spice support though. This is basically the qxl vga mode made more generic, so it plays together with any qemu-emulated gfx card. You can display stdvga or cirrus via spice client. You can have both vnc and spice enabled and clients connected at the same time.
* spice: add mouseGerd Hoffmann2010-09-211-0/+52
| | | | | Open mouse channel. Now you can move the guests mouse pointer. No tablet / absolute positioning (yet) though.
* spice: add keyboardGerd Hoffmann2010-09-214-1/+89
| | | | | | Open keyboard channel. Now you can type into the spice client and the keyboard events are sent to your guest. You'll need some other display like vnc to actually see the guest responding to them though.
* spice: core bitsGerd Hoffmann2010-09-217-0/+282
| | | | | | Add -spice command line switch. Has support setting passwd and port for now. With this patch applied the spice client can successfully connect to qemu. You can't do anything useful yet though.
* add spice into the configure fileGerd Hoffmann2010-09-211-0/+35
|
* configure: add loggingGerd Hoffmann2010-09-211-2/+5
| | | | | Write compile commands and messages to config.log. Useful for debugging configure.
* add pflib: PixelFormat conversion library.Gerd Hoffmann2010-09-213-0/+234
|
* Use machine_init() to register virtfs config options.Gerd Hoffmann2010-09-212-5/+9
|
* Use display types for local display only.Gerd Hoffmann2010-09-212-12/+13
| | | | | | | This patch drops DT_VNC. The display types are only used to select select the local display (i.e. curses, sdl, coca, ...). Remote displays (for now only vnc, spice will follow) can be enabled independently.
* Merge remote branch 'mst/for_anthony' into stagingAnthony Liguori2010-09-202-1/+11
|\
| * e1000: Pad short frames to minimum size (60 bytes)Stefan Hajnoczi2010-09-201-0/+10
| | | | | | | | | | | | | | | | | | | | | | The OpenIndiana (Solaris) e1000g driver drops frames that are too long or too short. It expects to receive frames of at least the Ethernet minimum size. ARP requests in particular are small and will be dropped if they are not padded appropriately, preventing a Solaris VM from becoming visible on the network. Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
| * vhost: fix infinite loop on error pathMichael S. Tsirkin2010-09-141-1/+1
| | | | | | | | | | | | | | | | file.index is unsigned, hence 'while (--file.index >= 0)' will loop > forever. Change to while (file.index-- > 0). Reported-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
* | mingw: use ASLR, no-SEH and DEP if availableBlue Swirl2010-09-191-0/+9
| | | | | | | | | | | | | | | | | | | | If the linker supports the flags --dynamicbase, --no-seh, or --nxcompat, use them. Tested on Windows Vista: Process Explorer reports that ASLR and DEP are in use. No effect seen on Wine or Windows XP. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | cris: Fix watchdog resetsEdgar E. Iglesias2010-09-191-2/+3
| | | | | | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | cris: Fix TLB exec bit protectionEdgar E. Iglesias2010-09-182-2/+2
| | | | | | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | Use gcc warning flag -Wnested-externsBlue Swirl2010-09-184-3/+4
| | | | | | | | | | | | | | | | | | If the compiler supports the warning flag -Wnested-externs, use it. Avoid the only warning by moving the declaration of xml_builtin to a more proper place. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Use gcc warning flag -Wempty-bodyBlue Swirl2010-09-186-10/+14
| | | | | | | | | | | | | | | | If the compiler supports the warning flag -Wempty-body, use it. Adjust the code to avoid the warnings. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Use a few more gcc warning flagsBlue Swirl2010-09-181-1/+4
| | | | | | | | | | | | | | | | | | | | | | If the compiler supports the following warning flags, use them: -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wmissing-include-dirs Currently, these flags don't produce any warnings. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Use gcc warning flag -Wtype-limitsBlue Swirl2010-09-181-1/+1
| | | | | | | | | | | | If the compiler supports the warning flag -Wtype-limits, use it. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | pxa2xx: fix SSSR TFN logicBlue Swirl2010-09-181-4/+4
| | | | | | | | | | | | | | | | | | Fix SSSR TFN logic: TX FIFO is never filled, so it is always in underrun condition if SSP is enabled. This also avoids a gcc warning with -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | MIPS: fix yield handlingBlue Swirl2010-09-181-1/+3
| | | | | | | | | | | | | | | | | | The parameter for yield should be handled as a signed integer for the comparisons to have any effect. This also avoids a gcc warning with -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | PPC: Suppress gcc warnings with -Wtype-limitsBlue Swirl2010-09-181-25/+25
| | | | | | | | | | | | | | | | The hack added by c5b76b381081680633e2e0a91216507430409fb2 was not enough to avoid warnings with gcc flag -Wtype-limits. Add a new macro to fix both problems. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | blkdebug: fix enum comparisonBlue Swirl2010-09-181-3/+1Star
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The signedness of enum types depend on the compiler implementation. Therefore the check for negative values may or may not be meaningful. Fix by explicitly casting to a signed integer. Since the values are also checked earlier against event_names table, this is an internal error. Change the 'if' to 'assert'. This also avoids a warning with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | pxa2xx: remove useless checksBlue Swirl2010-09-181-1/+1
| | | | | | | | | | | | | | | | | | Remove checks which were made useless by r5849, 8da3ff180974732fc4272cb4433fef85c1822961. This also avoids a warning with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Use range_covers_byteBlue Swirl2010-09-182-8/+11
| | | | | | | | | | | | | | | | Use range_covers_byte() instead of comparisons. This avoids some warnings with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Introduce range.hBlue Swirl2010-09-187-31/+34
| | | | | | | | | | | | | | Extract range functions from pci.h. These will be used by later patches by non-PCI devices. Adjust current users. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | linux-user: improve flatload error checkingBlue Swirl2010-09-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | Because of the use of unsigned type, possible errors during load were ignored. Fix by using a signed type. This also avoids a warning with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | linux-user: fix types in a comparisonBlue Swirl2010-09-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | -1ul is unsigned long, which does not necessarily match abi_ulong type. Fix by using abi_long instead. This also avoids a warning with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | linux-user: fix socklen_t comparisonsBlue Swirl2010-09-181-7/+13
| | | | | | | | | | | | | | | | | | | | | | On many systems, socklen_t is defined as unsigned. This means that checks for negative values are not meaningful. Fix by explicitly casting to a signed integer. This also avoids some warnings with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Check for errors during BIOS or kernel loadBlue Swirl2010-09-184-13/+18
| | | | | | | | | | | | | | | | | | | | | | Because of the use of unsigned types, possible errors during BIOS or kernel load were ignored. Fix by using a signed type. This also avoids some warnings with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* | Make ARP replies at least 64 bytes longHervé Poussineau2010-09-171-1/+2
| | | | | | | | | | | | | | | | IEEE 802.3 standard requires Ethernet frames to be at least 64 bytes long. If it is not the case, they will be considered as runt frames, and may be ignored by netcard and/or OS Signed-off-by: Hervé Poussineau <hpoussin@reactos.org> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | Accept packets with TTL=1Hervé Poussineau2010-09-171-1/+1
| | | | | | | | | | | | | | | | Packets with TTL=1 may be directed to local network (DHCP/DNS servers for example), so don't discard them This is required by old versions of NetBSD which send DHCP DISCOVER packets with TTL=1 Signed-off-by: Hervé Poussineau <hpoussin@reactos.org> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | cris: Avoid spurios hw_abort on recursive bus faultsEdgar E. Iglesias2010-09-161-1/+7
| | | | | | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | PPC: Redesign interrupt trigger pathAlexander Graf2010-09-151-51/+29Star
| | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the Book3S spec, the interrupt context starts with an MSR value that is rather simple. If we leave out the HV case, it's almost always 0. To reflect this, let's redesign the way that MSR value gets calculated. Using this, we also squash the bug where MSR_POW can slip through into the interrupt handler MSR. Reported-by: Thomas Monjalon <thomas.monjalon@openwide.fr> Signed-off-by: Alexander Graf <agraf@suse.de> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | PPC: Enable hint bits for lwarx/ldarxAlexander Graf2010-09-151-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | The lwarx and ldarx instructions have a bit to give some hint to the CPU which is safe to ignore. We currently refuse to accept any instruction with that bit set, as it used to be declared MBZ. Let's remove the reserved bit and make the instruction work as expected. This fixes Linux boot for ppc64. Signed-off-by: Alexander Graf <agraf@suse.de> Reviewed-by: Andreas Färber <andreas.faerber@web.de> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | serial: Update parameters after loadMichal Novotny2010-09-151-0/+1
| | | | | | | | | | | | | | | | This is the patch to update serial port parameters after guest is already loaded. Signed-off-by: Michal Novotny <minovotn@redhat.com> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* | Remove wrong semicolon in macro definitionStefan Weil2010-09-142-3/+3
|/ | | | | | | | | Macros normally should not end with a semicolon, otherwise their usage results in two statements where only one statement was expected. Signed-off-by: Stefan Weil <weil@mail.berlios.de> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* serial: Wrap debug prints around a DPRINTF macroMichal Novotny2010-09-131-12/+13
| | | | | Signed-off-by: Michal Novotny <minovotn@redhat.com> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* ESP: fix ESP DMA access when DMA is not enabledBlue Swirl2010-09-115-18/+88
| | | | | | | | | | | | | | Sending ESP a command caused it to trigger DMA immediately even if DMA was not enabled at the DMA controller. Add a signal from DMA controller to ESP to tell ESP about changes in DMA enable bit. Also use the correct function for setting up GPIO outputs. This fixes NetBSD 1.6.1 through 3.0 boot. Thanks to Artyom Tarasenko for extensive debugging of the problem. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* powerpc: Avoid TLB related log spammingEdgar E. Iglesias2010-09-111-1/+0Star
| | | | | | Invalid TLB entries are normal and should not spam the log. Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* trace: fix a regex portability problemBlue Swirl2010-09-111-1/+1
| | | | | | | | | | The /bin/sh in Milax has problems with the regex: Error: invalid trace backend Please choose a supported trace backend. Fix it by escaping ')' like the regexes with '('. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* trace: fix a typoBlue Swirl2010-09-101-1/+1
| | | | | | There is no qemu_valloc() but qemu_vmalloc(). Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add rules for printf-like functionsBlue Swirl2010-09-101-0/+13
| | | | | | Add rules for printf-like functions, based on libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add string management rulesBlue Swirl2010-09-101-0/+24
| | | | | | Add string management rules, somewhat like libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add memory management rulesBlue Swirl2010-09-101-0/+14
| | | | | | Add memory management rules, somewhat like libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add C type rulesBlue Swirl2010-09-102-3/+68
| | | | | | | | | | Add C type rules, adapted from libvirt HACKING. Also include a description of special QEMU scalar types. Move typedef rule from CODING_STYLE rule 3 to HACKING rule 6 where it belongs. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add preprocessor rulesBlue Swirl2010-09-101-0/+6
| | | | | | | | | Add a new file, HACKING, in order to collect recurring issues with submitted patches. Start with preprocessor rules, adapted from libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* vmware_vga: Replace the few tab-indents with spaces (clean-up).Andrzej Zaborowski2010-09-101-9/+9
| | | | Use 4 spaces instead of 2, too.
* vmware_vga: Add checks to deal with non-atomic fifo writes.Andrzej Zaborowski2010-09-101-8/+56
| | | | | | | | | | | | Janne Huttunen noticed that the FIFO end pointer is updated by the guest after writing each word to the FIFO, at least the X.org driver which is open does this. This means that there's no way for the host to know if the guest is in the middle a write operation. Qemu thus needs to read the beginning of the command up to when it's able to tell how many words are expected for the given command. It will abort reading and rewind the FIFO if there aren't enough words yet, this should be relatively rare but it is suspected to have been the cause of the occasional FIFO overrun that killed the display.
* hw/omap: Fix default setup for OMAP UART devicesStefan Weil2010-09-104-8/+19
| | | | | | | | | | | | | | | | | | Character devices created by qemu_chr_open don't allow duplicate device names, so naming all UART devices "null" no longer works. Running "qemu-system-arm -M n800" (and some other machines) results in this error message: qemu-system-arm: Duplicate ID 'null' for chardev Can't create serial device, empty char device This is fixed by setting a default label "uart1", "uart2" or "uart3". Cc: Andrzej Zaborowski <andrew.zaborowski@intel.com> Signed-off-by: Stefan Weil <weil@mail.berlios.de>