summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* vectorize is_dup_pagePaolo Bonzini2012-01-131-6/+22
| | | | | | | | is_dup_page is already proceeding in 32-bit chunks. Changing it to 16 bytes using Altivec or SSE is easy. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* virtio-blk: refuse SG_IO requests with scsi=offPaolo Bonzini2012-01-131-0/+6
| | | | | | | | | | | | | | | QEMU does have a "scsi" option (to be used like -device virtio-blk-pci,drive=foo,scsi=off). However, it only masks the feature bit, and does not reject the command if a malicious guest disregards the feature bits and issues a request. Without this patch, using scsi=off does not protect you from CVE-2011-4127. Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* vmstate: extract declarations out of hw/hw.hPaolo Bonzini2012-01-1310-891/+957
| | | | | Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* ptimer: move declarations to ptimer.hPaolo Bonzini2012-01-1318-13/+43
| | | | | | Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* rtc: clear non-PF bits when reinjecting on ackPaolo Bonzini2012-01-131-3/+3
| | | | | | | | | | | When an rtc interrupt is reinjected immediately after being acked, other interrupts should not be reinjected, so do clear their bits. Also, if the periodic interrupts have been disabled before acking, do not reinject, as the guest might get very confused! Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* rtc: raise PF bit when the periodic timer triggers but PIE=0Paolo Bonzini2012-01-131-1/+2
| | | | | Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* rtc: raise AF bit when the alarm is encountered but AIE=0Paolo Bonzini2012-01-131-9/+10
| | | | | Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* rtc: fix 12-hour modePaolo Bonzini2012-01-131-4/+7
| | | | | | | Hours in 12-hour mode are in the 1-12 range, not 0-11. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* qdev: fix device_del by refactoring reference countingAnthony Liguori2012-01-131-6/+39
| | | | | | | | | Commit 8eb0283 broken device_del by having too overzealous reference counting checks. Move the reference count checks to qdev_free(), make sure to remove the parent link on free, and decrement the reference count on property removal. Reported-by: Markus Armbruster <armbru@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* Merge remote-tracking branch 'stefanha/trivial-patches' into HEADAnthony Liguori2012-01-1315-29/+28
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stefanha/trivial-patches: bt-host: add missing break statement virtfs-proxy-helper: Add missing printf format attribute virtfs-proxy-helper: Clean include files virtfs-proxy-helper: Fix compilation on newer systems hmp: Fix freeing of PciInfoList Add 'fall through' comments to case statements without break omap_dss: correct chip[1] index in RFBI_READ/RFBI_STATUS vnc: fix no-lock-key-sync strncmp() length vvfat: avoid leaking file descriptor in commit_one_file() Spelling fixes in comments and documentation tcg-arm: fix a typo in comments configure: Modify detection of supported warning options
| * bt-host: add missing break statementStefan Hajnoczi2012-01-131-0/+1
| | | | | | | | | | | | | | | | | | The switch statement in bt_host_read() is missing a break in one case. Andrzej Zaborowski <andrew.zaborowski@intel.com> confirmed that this is not an intentional fall-through. Reviewed-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * virtfs-proxy-helper: Add missing printf format attributeStefan Weil2012-01-131-1/+1
| | | | | | | | | | | | | | | | Every function with printf like arguments must have it (see file HACKING), so add it. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * virtfs-proxy-helper: Clean include filesStefan Weil2012-01-131-13/+2
| | | | | | | | | | | | | | | | | | The common standard include files are already included via qemu-common.h, and for the socket related include files there is qemu_socket.h, so the code can be reduced by some lines. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * virtfs-proxy-helper: Fix compilation on newer systemsStefan Weil2012-01-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | Include file attr/xattr.h is not available on "newer" systems (for example Fedora 12 or Debian Squeeze). See comments in qemu-xattr.h for more information. This file handles the system dependencies automatically. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * hmp: Fix freeing of PciInfoListStefan Berger2012-01-131-4/+4
| | | | | | | | | | | | | | | | Remember the original PciInfoList in info_list and use the info variable to traverse the list. Signed-off-by: Stefan Berger <stefanb@linux.vnet.ibm.com> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * Add 'fall through' comments to case statements without breakStefan Weil2012-01-133-0/+6
| | | | | | | | | | | | | | | | | | | | | | These comments are used by static code analysis tools and in code reviews to avoid false warnings because of missing break statements. The case statements handled here were reported by coverity. Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * omap_dss: correct chip[1] index in RFBI_READ/RFBI_STATUSStefan Hajnoczi2012-01-131-2/+2
| | | | | | | | | | | | | | | | | | | | The RFBI_READ/RFBI_STATUS code incorrectly uses chip[0] when it should be using chip[1]. Andrzej Zaborowski <balrog@zabor.org> confirmed this bug since I don't know this code well. Reported-by: Dr David Alan Gilbert <davidagilbert@uk.ibm.com> Reviewed-by: Andrzej Zaborowski <andrew.zaborowski@intel.com> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * vnc: fix no-lock-key-sync strncmp() lengthStefan Hajnoczi2012-01-131-1/+1
| | | | | | | | | | | | | | | | | | The no-lock-key-sync option is being parsed incorrectly because of an outdated strcmp() length value. Use the correct length so that invalid option names do not match. Reported-by: Dr David Alan Gilbert <davidagilbert@uk.ibm.com> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * vvfat: avoid leaking file descriptor in commit_one_file()Stefan Hajnoczi2012-01-131-0/+3
| | | | | | | | | | Reported-by: Dr David Alan Gilbert <davidagilbert@uk.ibm.com> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * Spelling fixes in comments and documentationStefan Weil2012-01-133-4/+4
| | | | | | | | | | | | | | Codespell detected these new spelling issues. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * tcg-arm: fix a typo in commentsAurelien Jarno2012-01-132-2/+2
| | | | | | | | | | | | | | | | | | ARM still doesn't support 16GB buffers in 32-bit modes, replace the 16GB by 16MB in the comment. Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| * configure: Modify detection of supported warning optionsStefan Weil2012-01-131-1/+1
| | | | | | | | | | | | | | | | Reversing the order of the warning options and -Werror is important when clang is used instead of gcc. It changes nothing for gcc. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
* | target-i386: fix compilation with --enable-debug-tcgAurelien Jarno2012-01-131-1/+2
|/ | | | | | | | Commit 2355c16e74ffa4d14e7fc2b4a23b055565ac0221 introduced a new ldmxcsr helper taking an i32 argument, but the helper is actually passed a long. Fix that by truncating the long to i32. Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
* cris: Update paths to match the move of tests/crisEdgar E. Iglesias2012-01-131-5/+5
| | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* check-qstring: remove check.h includeAnthony Liguori2012-01-121-1/+1
| | | | | Reported-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* MAINTAINERS: update tracing repo git URLStefan Hajnoczi2012-01-121-1/+1
| | | | | | | I have moved git hosting services. Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* Support for UDP unicast network backendBenjamin2012-01-123-3/+82
| | | | | Signed-off-by: Benjamin MARSILI <mlspirat42@gmail.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* cleanup, save a syscallLai Jiangshan2012-01-121-3/+1
| | | | | Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* cleanup, Remove duplicated codeLai Jiangshan2012-01-121-8/+0
| | | | | | | | These two blocks of code are exactly the same, remove one. Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com> Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* test: eliminate libcheck tests and have make check use gtesterAnthony Liguori2012-01-123-40/+8
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qjson: enable disabled testsAnthony Liguori2012-01-121-5/+0
| | | | | | gtest does the right thing here so there's no need to comment these tests out. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qjson: convert to gtestAnthony Liguori2012-01-122-198/+136
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qlist: convert to gtestAnthony Liguori2012-01-122-52/+25
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qstring: convert to gtestAnthony Liguori2012-01-122-53/+26
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qint: convert to gtestAnthony Liguori2012-01-122-49/+23
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qfloat: convert to gtestAnthony Liguori2012-01-122-37/+14
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* check-qdict: convert to gtestAnthony Liguori2012-01-122-137/+113
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* build: split unit test builds to a separate makefile fragmentAnthony Liguori2012-01-122-35/+39
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* tests: mv tests/* -> tests/tcgAnthony Liguori2012-01-12227-5/+5
| | | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* vga: optimize ppm_save() divisionsAvi Kivity2012-01-121-6/+4
| | | | | | | | | ppm_save() spends upwards of 50% of its time doing divisions. Replace them with shifts. Reviewed-by: Alon Levy <alevy@redhat.com> Signed-off-by: Avi Kivity <avi@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* Fix qapi code generation fixAvi Kivity2012-01-124-23/+15
| | | | | | | | | | | | | | The fixes to qapi code generation had multiple bugs: - the Null class used to drop output was missing some methods - in some scripts it was never instantiated, leading to a None return, which is missing even more methods - the --source and --header options were swapped Luckily, all those bugs were hidden by a makefile bug which caused the old behaviour (with the race) to be invoked. Signed-off-by: Avi Kivity <avi@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* etraxfs-dma: Model metadata and eopLars Persson2012-01-123-10/+33
| | | | | | | | - Send EOP flags to the out channels. - Send data descriptor metadata to the out channels. Signed-off-by: Lars Persson <larper@axis.com> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* microblaze: Break the tb at memory barriersEdgar E. Iglesias2012-01-121-1/+11
| | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* microblaze: Emulate the hw stackprotectorEdgar E. Iglesias2012-01-124-0/+48
| | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* microblaze: Add support for the clz insnEdgar E. Iglesias2012-01-123-0/+17
| | | | Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
* target-i386: fix SSE rounding and flush to zeroAurelien Jarno2012-01-113-11/+56
| | | | | | | | | | | | SSE rounding and flush to zero control has never been implemented. However given that softfloat-native was using a single state for FPU and SSE and given that glibc is setting both FPU and SSE state in fesetround(), this was working correctly up to the switch to softfloat. Fix that by adding an update_sse_status() function similar to update_fpu_status(), and callin git on write to mxcsr. Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
* target-i386: fix dpps and dppd SSE2 instructionsAurelien Jarno2012-01-111-14/+14
| | | | | | | | The helpers implemented dpps and dppd SSE instructions are not passing the correct argument types to the softfloat functions. While they do work anyway providing a correct behaviour, this patch fixes that. Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
* target-i386: fix round{pd,ps,sd,ss} SSE2 instructionsAurelien Jarno2012-01-111-8/+8
| | | | | | | | | | | | | roundps and roundss SSE2 instructions have been broken when switching target-i386 to softfloat. They use float64_round_to_int to convert a float32, and while the implicit conversion from float32 to float64 was correct for softfloat-native, it is not for pure softfloat. Fix that by using the correct registers and correct functions. Also fix roundpd and roundsd implementation at the same time, even if these functions are behaving correctly. Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
* target-i386: fix {min,max}{pd,ps,sd,ss} SSE2 instructionsAurelien Jarno2012-01-112-3/+7
| | | | | | | | | | | | | | | | | | | minpd, minps, minsd, minss and maxpd, maxps, maxsd, maxss SSE2 instructions have been broken when switching target-i386 to softfloat. It's not possible to use comparison instructions on float types anymore to softfloat, so use the floatXX_lt function instead, as the float_XX_min and float_XX_max functions can't be used due to the Intel specific behaviour. As it implements the correct NaNs behaviour, let's remove the corresponding entry from the TODO. It fixes GDM screen display on Debian Lenny. Thanks to Peter Maydell and Jason Wessel for their analysis of the problem. Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
* wm8750: Fix calculation of number of array elementsStefan Weil2012-01-101-1/+1
| | | | | | | | Coverity says that the division by sizeof(*s->rate) might be wrong. I think that coverity is right. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Andrzej Zaborowski <andrew.zaborowski@intel.com>
OpenPOWER on IntegriCloud