Statistics
| Branch: | Revision:

root @ 452efba6

# Date Author Comment
452efba6 01/24/2010 04:28 pm Blue Swirl

Sparc32: fix free-run timer

According to Sun4M System Architecture Manual chapter 5.3.2, a limit
of 0 will not generate interrupts.

Signed-off-by: Blue Swirl <>

07b064e9 01/24/2010 03:55 pm Jan Kiszka

Musicpal: Fix descriptor walk in eth_send

Commit 930c86820e introduced a regression to eth_send: eth_tx_desc_put
manipulates the host's tx descriptor copy before writing it back, but
two lines down the descriptor is evaluated again, leaving us with an
invalid next address if host and guest endianness differ. So this was...

64258229 01/23/2010 08:18 pm Jan Kiszka

Musicpal: Fix wm8750 I2C address

Commit b3a219883e uncovered that we attached the Wolfson with an I2C
address shifted left by one. Fixing this makes sound work again for
the Musicpal.

Signed-off-by: Jan Kiszka <>
Signed-off-by: malc <>

15e7c451 01/23/2010 10:11 am Artyom Tarasenko

sparc32 fix np dereference in do_unassigned_access

fix a potential null pointer dereference introduced in
commit 576c2cdc767ab9e2dc038fa4c99f22e53287a3de

Signed-off-by: Artyom Tarasenko <>
Signed-off-by: Blue Swirl <>

204c7a39 01/22/2010 08:39 pm Blue Swirl

Sparc64: fix PCI probe problems

Byte swap PCI config values.

Remove old bogus PCI config mechanism so that device 0:0.0 can be probed.
This requires OpenBIOS r667.

Signed-off-by: Blue Swirl <>

3e39789b 01/20/2010 06:12 pm Anthony Liguori

Revert "block: prevent multiwrite_merge from creating too large iovecs"

This reverts commit 0076bc0c1d93adcbc7f1af184e04902cf37e9ab8.

Kevin Wolf pointed out that this breaks the mingw32 build.

Signed-off-by: Anthony Liguori <>

a6a853c8 01/20/2010 06:09 pm Anthony Liguori

make: qemu-img depends on config-host.h

Fixes mingw32 build out of tree.

Signed-off-by: Anthony Liguori <>

dd17765b 01/20/2010 04:51 pm Gleb Natapov

reduce number of reinjects on ACK

Windows 7 BSODs under load with HAL_RTC_IRQF_WILL_NOT_CLEAR error.

It happens here:
hal!HalpRtcUnmaskClock:
8281b93a 8bff mov edi,edi
8281b93c 56 push esi
8281b93d 33f6 xor esi,esi...

0076bc0c 01/20/2010 04:51 pm Christoph Hellwig

block: prevent multiwrite_merge from creating too large iovecs

If we go over the maximum number of iovecs support by syscall we get
back EINVAL from the kernel which translate to I/O errors for the guest.

Signed-off-by: Christoph Hellwig <>
Signed-off-by: Anthony Liguori <>

1d44952f 01/20/2010 04:51 pm Christoph Hellwig

block: fix cache flushing in bdrv_commit

Signed-off-by: Anthony Liguori <>

bb61564c 01/20/2010 04:25 pm Amit Shah

virtio: Remove duplicate macro definition for max. virtqueues, bump up the max

VIRTIO_PCI_QUEUE_MAX is redefined in hw/virtio.c. Let's just keep it in
hw/virtio.h.

Also, bump up the value of the maximum allowed virtqueues to 64. This is
in preparation to allow multiple ports per virtio-console device....

98b19252 01/20/2010 04:25 pm Amit Shah

virtio-console: qdev conversion, new virtio-serial-bus

This commit converts the virtio-console device to create a new
virtio-serial bus that can host console and generic serial ports. The
file hosting this code is now called virtio-serial-bus.c.

The virtio console is now a very simple qdev device that sits on the...

6663a195 01/20/2010 04:25 pm Amit Shah

virtio-serial-bus: Maintain guest and host port open/close state

Via control channel messages, the guest can tell us whether a port got
opened or closed. Similarly, we can also indicate to the guest of host
port open/close events.

Signed-off-by: Amit Shah <>...

160600fd 01/20/2010 04:25 pm Amit Shah

virtio-serial-bus: Add a port 'name' property for port discovery in guests

The port 'id' or number is internal state between the guest kernel and
our bus implementation. This is invocation-dependent and isn't part of
the guest-host ABI.

To correcly enumerate and map ports between the host and the guest, the...

f146ec9a 01/20/2010 04:25 pm Amit Shah

virtio-serial-bus: Add ability to hot-unplug ports

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

b60c470b 01/20/2010 04:25 pm Amit Shah

virtio-serial: Add a 'virtserialport' device for generic serial port support

This commit adds a simple chardev-based serial port. Any data the guest
sends is forwarded to the chardev and vice-versa.

Sample uses for such a device can be obtaining info from the guest like...

4d3053a3 01/20/2010 04:25 pm Amit Shah

Move virtio-serial to Makefile.objs

There's nothing target-dependent in the virtio-serial code so allow it
to be compiled just once for all the targets.

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

a1829205 01/20/2010 04:25 pm Amit Shah

virtio-serial: Use MSI vectors for port virtqueues

This commit enables the use of MSI interrupts for virtqueue
notifications for ports. We use nr_ports + 1 (for control channel) msi
entries for the ports, as only the in_vq operations need an interrupt on...

c9c4b34e 01/20/2010 04:25 pm Luiz Capitulino

net: Make inet_strfamily() public

So that it can be used by other subsystems.

Signed-off-by: Luiz Capitulino <>
Signed-off-by: Anthony Liguori <>

c445321e 01/20/2010 04:25 pm Luiz Capitulino

net: inet_strfamily(): Better unknown family report

Returning "????" is a bit meaningless, let's call it "unknown".

Signed-off-by: Luiz Capitulino <>
Signed-off-by: Anthony Liguori <>

dc0d4efc 01/20/2010 04:25 pm Luiz Capitulino

vnc: Use inet_strfamily()

Signed-off-by: Luiz Capitulino <>
Signed-off-by: Anthony Liguori <>

23fabed1 01/20/2010 04:25 pm Luiz Capitulino

QMP: Fix asynchronous events delivery

Commit f039a563f200beee80cc10fd70b21ea396979dab introduces
a regression as monitor_protocol_event() will return in
the first user Monitor it finds in the QLIST_FOREACH()
loop.

The right thing to do is to only delivery an asynchronous...

3dbf2c7f 01/20/2010 04:25 pm Stefan Weil

Documentation: Add missing documentation for qdev related command line options

The command line options -device, -nodefaults, -readconfig,
-writeconfig had entries for command line help, but
documentation for texi and derived formats (man, html, info)
was missing....

b196b153 01/20/2010 04:25 pm Naphtali Sprei

Make CDROM a read-only drive

Signed-off-by: Naphtali Sprei <>
Signed-off-by: Anthony Liguori <>

f5edb014 01/20/2010 04:25 pm Naphtali Sprei

Clean-up a little bit the RW related bits of BDRV_O_FLAGS. BDRV_O_RDONLY gone (and so is BDRV_O_ACCESS). Default value for bdrv_flags (0/zero) is READ-ONLY. Need to explicitly request READ-WRITE.

Instead of using the field 'readonly' of the BlockDriverState struct for passing the request,...

3f3ed593 01/20/2010 04:25 pm Naphtali Sprei

Added drives' readonly option

Signed-off-by: Naphtali Sprei <>
Signed-off-by: Anthony Liguori <>

03cbdac7 01/20/2010 04:25 pm Naphtali Sprei

Disable fall-back to read-only when cannot open drive's file for read-write

Signed-off-by: Naphtali Sprei <>
Signed-off-by: Anthony Liguori <>

0d2ed46a 01/20/2010 12:31 am Luiz Capitulino

QMP: Introduce VNC_INITIALIZED event

It's emitted when a VNC client session is activated by QEMU,
client's information such as port, IP and auth ID (if the
session is authenticated) are provided.

Event example:

{ "event": "VNC_INITIALIZED",
"timestamp": {"seconds": 1263475302, "microseconds": 150772},...

beb6f0de 01/20/2010 12:31 am Kevin Wolf

Fix QEMU_WARN_UNUSED_RESULT

Since commit 747bbdf7 QEMU_WARN_UNUSED_RESULT is never defined as it is
conditional on a define from config-host.h which is included only later.
Include that file earlier to get the warnings back.

Reactivating it unfortunately leads to some warnings about unused qdev_init...

f039a563 01/20/2010 12:31 am Adam Litke

QMP: Emit asynchronous events on all QMP monitors

When using a control/QMP monitor in tandem with a regular monitor, asynchronous
messages can get lost depending on the order of the QEMU program arguments.
QEMU events issued by monitor_protocol_event() always go to cur_mon. If the...

47c06340 01/20/2010 12:31 am Gerd Hoffmann

ide: device version property

This patch adds a new property named 'ver' to ide-drive which allows to
specify the version which the virtual disk/cdrom should report to the
guest. By default this is the qemu version (i.e. 0.12). usage:

-drive if=none,id=disk,file=......
383b4d9b 01/20/2010 12:31 am Gerd Hoffmann

scsi: device version property

This patch adds a new property named 'ver' to scsi-disk which allows to
specify the version which the virtual disk/cdrom should report to the
guest. By default this is the qemu version (i.e. 0.12). usage:

-drive if=none,id=disk,file=......
374ef704 01/20/2010 12:31 am Gerd Hoffmann

pc: add driver version compat properties

This patch adds compat property entries for ide-disk.ver and
scsi-disk.ver to pc-0.10 and pc-0.11. With this patch applied
the scsi and ide disks report "0.10" and "0.11" as version when
you start qemu with "-M pc-0.10" or "-M pc-0.11"....

040f2fb2 01/20/2010 12:31 am Paolo Bonzini

fix wrong indentation

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

1c3569fe 01/20/2010 12:31 am Paolo Bonzini

clean up env->current_tb

There are three paths from the innermost while loop of cpu_exec
to the top of the outermost for loop. Two do not reset
env->current_tb. Fix this.

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

34dec6a5 01/20/2010 12:31 am Paolo Bonzini

add assertions about env->current_tb

By virtue of the previous patch env->current_tb will always be NULL at
the top of cpu_exec's outermost for loop, and at the end of the innermost
while loop.

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

6113d6d3 01/20/2010 12:31 am Paolo Bonzini

change while to if

The while loop will be executed exactly 0 or 1 times, depending on
env->exit_request.

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

f8fb2fa8 01/20/2010 12:31 am Paolo Bonzini

remove assertions

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

301d2908 01/20/2010 12:31 am Paolo Bonzini

avoid unreachable statement after break

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

3baf720e 01/20/2010 12:31 am Cole Robinson

raw-posix: Detect CDROM via ioctl on linux

Current CDROM detection is hardcoded based on source file name.
Make this smarter on linux by attempting a CDROM specific ioctl.

This makes '-cdrom /dev/sr0' succeed with no media present.

v2:
Give ioctl check higher priority than filename check....

2ebf7c4b 01/20/2010 12:31 am Cole Robinson

raw-posix: Detect legacy floppy via ioctl on linux

Current legacy floppy detection is hardcoded based on source file
name. Make this smarter on linux by attempting a floppy specific
ioctl.

v2:
Give ioctl check higher priority than filename check
s/IDE/legacy/...

3b9d7e66 01/20/2010 12:31 am Kusanagi Kouichi

char: Remove redundant qemu_chr_generic_open() call.

qemu_chr_open_fd() calls qemu_chr_generic_open(),
so qemu_chr_open_tty() doesn't need to call it.

Signed-off-by: Kusanagi Kouichi <>
Signed-off-by: Anthony Liguori <>

8950a950 01/20/2010 12:31 am Luiz Capitulino

VNC: Use 'enabled' key instead of 'status'

Currently the 'status' key is a string whose value can be
"disabled" or "enabled", change it to the QMP's standard
'enabled' key, which is a bool.

Note that 'status' in being dropped and this wouldn't be
allowed if QMP were stable....

a7789382 01/20/2010 12:31 am Luiz Capitulino

VNC: Make 'auth' key mandatory

There is no reason to have it as optional and the code
in the server and client gets slightly simpler if the
key is mandatory.

While there also do some cleanup on how the server info is
collected.

Signed-off-by: Luiz Capitulino <>...

76825067 01/20/2010 12:31 am Luiz Capitulino

VNC: Rename client's 'username' key

It's the SASL username, so it's better to call it 'sasl_username'
to be consistent.

Note that this change wouldn't be allowed if QMP were stable.

Signed-off-by: Luiz Capitulino <>
Signed-off-by: Anthony Liguori <>

5c7238c5 01/20/2010 12:31 am Luiz Capitulino

VNC: Add 'family' key

It contains the socket adress family name, like "ipv4" or
"ipv6".

This is useful for clients so that they can interpret the
'host' key reliably.

Signed-off-by: Luiz Capitulino <>
Signed-off-by: Anthony Liguori <>

4a80dba3 01/20/2010 12:31 am Luiz Capitulino

VNC: Cache client info at connection time

When a disconnection happens the client's socket on QEMU
side may become invalid, this way it won't be possible
to query it to get client information, which is going to
be needed by the future QMP VNC_DISCONNECTED event....

586153d9 01/20/2010 12:31 am Luiz Capitulino

QMP: Introduce VNC_CONNECTED event

It's emitted when a VNC client connects to QEMU, client's information
such as port and IP address are provided.

Note that this event is emitted right when the connection is
established. This means that it happens before authentication...

0d72f3d3 01/20/2010 12:31 am Luiz Capitulino

QMP: Introduce VNC_DISCONNECTED event

It's emitted when a VNC client disconnects from QEMU, client's
information such as port and IP address are provided.

Event example:

{ "event": "VNC_DISCONNECTED",
"timestamp": { "seconds": 1262976601, "microseconds": 975795 },...

7b8737de 01/20/2010 12:31 am Stefan Weil

eepro100: Fix multicast support

  • Handling of multicast list was missing.
  • Multicast all was missing.
  • Promiscuous mode for multicast frames was wrong.

This patch is a step to synchronize my maintainer version
of eepro100.c (git://repo.or.cz/qemu/ar7.git) with the...

4ef1a3d3 01/20/2010 12:31 am Igor V. Kovalenko

rtl8139: fix clang reporting unused assignment of VLAN tagging data

Currently we do not implement VLAN tagging for rtl8139(C+),
still data is read from ring buffer headers.

- augment unused assignment with TODO item
- cast txdw1 to void for now

Signed-off-by: Igor V. Kovalenko <>...

baee019f 01/20/2010 12:31 am Paolo Bonzini

remove dead code from target-i386/exec.h

These are unused since edea5f0 (no need to define global registers in
cpu-exec.c, 2008-05-10).

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

49a945a3 01/20/2010 12:31 am Paolo Bonzini

kill regs_to_env and env_to_regs

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

f3a52e50 01/20/2010 12:31 am Stefan Weil

eepro100: Restructure code (new function tx_command)

Handling of transmit commands is rather complex,
so about 80 lines of code were moved from function
action_command to the new function tx_command.

The two new values "tx" and "cb_address" in the
eepro100 status structure made this possible without...

1c39457a 01/17/2010 10:36 pm Blue Swirl

Sparc32: Update OpenBIOS image to r666

Fixes the boot problem introduced by r665 image in
694b9309462f07307d16f492961f01271f10c245.

Signed-off-by: Blue Swirl <>

43bb98bf 01/17/2010 06:51 pm Blue Swirl

Sparc: improve CPU register dump

Common: * Remove unnecessary 0x prefix * Print %y * Fix NZVC flag print order to match CPU bit order

Sparc64 specific: * Print registers without line wrapping * Print %f40-%f63 * Pretty print CCR flags * Print %fsr and %fprs in full precision...

0ddbc96e 01/17/2010 10:51 am Mike Frysinger

fix linux-user microblaze ELF_ARCH definition

Signed-off-by: Mike Frysinger <>
Signed-off-by: Edgar E. Iglesias <>

82584e21 01/17/2010 01:03 am malc

audio: include more information into audio_bug's output

Signed-off-by: malc <>

ff541499 01/16/2010 11:25 pm malc

Revert "sdlaudio: make it suck less"

This reverts commit 4839abe78fd466a3cf06faa7c362154afd5404f1.

The commit was badly broken, Gentoo has sdl as the default driver,
consequently 5 gentoo users have hit the breakage and were kind enough
to report, so thank you:...

787cfbc4 01/16/2010 11:06 am Blue Swirl

Sparc32: improve DMA controller IRQ debugging

Signed-off-by: Blue Swirl <>

dca47edd 01/16/2010 11:06 am Blue Swirl

ESP: improve IRQ debugging

Signed-off-by: Blue Swirl <>

10760f0f 01/16/2010 11:06 am Artyom Tarasenko

sparc32 do not clear interrupts when masking

Don't clear interrupts on disabling, because
  • Sun4M_SystemArchitecture_edited2.pdf doesn't describe
    that masking or un-masking IRQ shall clear pending ones.
  • Field tests also show that SPARCstation-20 doesn't...
694b9309 01/16/2010 11:05 am Blue Swirl

Update Sparc OpenBIOS images to r665

Signed-off-by: Blue Swirl <>

576c2cdc 01/15/2010 11:33 pm Artyom Tarasenko

sparc32 do_unassigned_access overhaul v2

According to pages 9-31 - 9-34 of "SuperSPARC & MultiCache Controller
User's Manual":

1. "A lower priority fault may not overwrite the
MFSR status of a higher priority fault."
2. The MFAR is overwritten according to the policy defined for the MFSR...

09aac126 01/15/2010 08:32 am Richard Henderson

tcg/x86_64: Avoid unnecessary REX.B prefixes.

The existing P_REXB internal opcode flag unconditionally emits
the REX prefix. Technically it's not needed if the register in
question is %al, %bl, %cl, %dl.

Eliding the prefix requires splitting the P_REXB flag into two,...

fccd2613 01/15/2010 01:06 am Edgar E. Iglesias

nand: Correct random data reads.

Random reading depends on having the last row/page latched and not beeing
clobbered between read and any following random reads.

Also, s->iolen must be updated when loading the io/data register with
randomly accessed flash data....

b88bc808 01/14/2010 09:01 pm Stefan Weil

Makefile: Fix message for missing configure

When make is called without a valid configuration,
it should tell the user what to do.

Revision 0e8c9214ba1d4128cf92442cd343bc3733478261
was a regression which resulted in a message
which was no longer user friendly...

b3d6b959 01/14/2010 09:00 pm Kazu Hirata

target-m68k: fix a typo in 'P' packet processing

Attached is a patch to fix a typo in 'P' packet processing for M68K.

Without this patch, QEMU fails to honor GDB's P packets from GDB
(writing to registers) for the address registers (A0 - A7).

The problem is because of an obvious typo. Notice that the second...

57169903 01/14/2010 07:16 pm Richard Henderson

tcg/x86_64: Special-case all 32-bit AND operands.

This avoids an unnecessary REX.W prefix when dealing with AND
operands that fit into a 32-bit quantity. The most common change
actually seen is movz[wb]q -> movz[wb]l.

Similarly, avoid REXW in ext{8,16}u_i64 tcg opcodes....

0a4e7cd2 01/14/2010 05:15 pm Shin-ichiro KAWASAKI

sh: sm501: Add hardware cursor feature

This patch adds hardware cursor feature to SM501 graphics chip emulation,
to make the graphic console more useful for QEMU SH4 users.

Signed-off-by: Shin-ichiro KAWASAKI <>
Signed-off-by: Aurelien Jarno <>

8b0ee8c5 01/14/2010 04:59 pm Vince Weaver

alpha: fix stat64 issue

The stat64/fstat64 syscalls are broken for alpha linux-user.

This is because Alpha, even though it is native 64-bits, has a stat64
syscall that is different than regular stat. This means that the
"TARGET_LONG_BITS==64" check in syscall.c isn't enough. Below is...

dcbc9a70 01/14/2010 03:42 pm Edgar E. Iglesias

ppc-40x: Correct ESR for zone protection faults.

Raise the zone protection fault in ESR for TLB faults caused by
zone protection bits.

Signed-off-by: Edgar E. Iglesias <>

ec5c3e48 01/14/2010 03:42 pm Edgar E. Iglesias

ppc-40x: Correct decoding of zone protection bits.

The 40x MMU has 15 zones in the ZPR register.

Signed-off-by: Edgar E. Iglesias <>

c5b9729a 01/14/2010 03:42 pm Edgar E. Iglesias

ppc-40x: Correct check for Endian swapping TLB entries.

Bailout on 40x TLB entries with endianess swapping only if the entry
is valid.

Signed-off-by: Edgar E. Iglesias <>

f25359cd 01/14/2010 03:41 pm Edgar E. Iglesias

ppc-40x: Get TLB attributes from TLBLO.

The ZSEL was incorrectly beeing decoded from TLBHI. Decode it from
TLBLO instead.

Signed-off-by: Edgar E. Iglesias <>

bb0300dc 01/14/2010 01:14 am Gleb Natapov

Add KVM paravirt cpuid leaf

Initialize KVM paravirt cpuid leaf and allow user to control guest
visible PV features through -cpu flag.

Signed-off-by: Gleb Natapov <>
Signed-off-by: Anthony Liguori <>

1cb1a66a 01/14/2010 01:14 am Milan Plzik

Qemu's internal TFTP server breaks lock-step-iness of TFTP

According to RFC 1350 and RFC 2347, TFTP server should answer RRQ by
either OACK or DATA packet. Qemu's internal TFTP server answers RRQ with
additional options by sending both OACK and DATA packet, thus breaking...

96560cb3 01/14/2010 01:14 am Markus Armbruster

docs: New qdev-device-use.txt

Signed-off-by: Markus Armbruster <>
Signed-off-by: Anthony Liguori <>

b783e409 01/14/2010 01:14 am Kevin Wolf

block: Introduce BDRV_O_NO_BACKING

If an image references a backing file that doesn't exist, qemu-img info fails
to open this image. Exactly in this case the info would be valuable, though:
the user might want to find out which file is missing.

This patch introduces a BDRV_O_NO_BACKING flag to ignore the backing file when...

756e6736 01/14/2010 01:14 am Kevin Wolf

block: Add bdrv_change_backing_file

Introduce the functions needed to change the backing file of an image. The
function is implemented for qcow2.

Signed-off-by: Kevin Wolf <>
Signed-off-by: Anthony Liguori <>

3e85c6fd 01/14/2010 01:14 am Kevin Wolf

qemu-img rebase

This adds a rebase subcommand to qemu-img which allows to change the backing
file of an image.

In default mode, both the current and the new backing file need to exist, and
after the rebase, the COW image is guaranteed to have the same guest visible...

347ed55c 01/14/2010 01:14 am Kevin Wolf

osdep.c: Fix accept4 fallback

Commit 3a03bfa5 added a fallback in case the Linux kernel running qemu is older
than the kernel of the build system. Unfortunately, v1 was committed instead of
v2, so the code has a bug that was revealed in the review (checking for the...

5cdc9b76 01/14/2010 01:14 am Amit Shah

vl.c: Remove dead assignment

clang-analyzer pointed out the value of 'sockets' is never reused.

Signed-off-by: Amit Shah <>
CC: Andre Przywara <>
Signed-off-by: Anthony Liguori <>

22c253d9 01/14/2010 01:14 am Amit Shah

virtio: net: remove dead assignment

clang-analyzer points out value assigned to 'len' is not used.

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

d7209371 01/14/2010 01:14 am Amit Shah

x86: translate.c: remove dead assignment

clang-analyzer points out a redundant assignment.

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

c4af120f 01/14/2010 01:14 am Amit Shah

hw/vga.c: remove dead assignment

clang-analyzer points out a redundant assignment.

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

f7dcfac0 01/14/2010 01:14 am Amit Shah

qcow2-refcount: remove dead assignment

clang-analyzer points out a redundant assignment.

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

2da69c8d 01/14/2010 01:14 am Amit Shah

json-parser: remove dead increment

clang-analyzer points out a redundant increment.

Signed-off-by: Amit Shah <>
Signed-off-by: Anthony Liguori <>

3f5075ae 01/14/2010 01:14 am Christoph Hellwig

block: flush backing_hd in the right place

The backing device is only modified from bdrv_commit. So instead of
flushing it every time bdrv_flush is called for the front-end device
only flush it after we're written data to it in bdrv_commit.

Signed-off-by: Christoph Hellwig <>...

f8012c13 01/14/2010 01:14 am Kevin Wolf

qcow/qcow2: implement bdrv_aio_flush

Now that we do not have to flush the backing device anymore implementing
the bdrv_aio_flush method for image formats is trivial.

[hch: forward ported to qemu mainline from a product tree]

Signed-off-by: Kevin Wolf <>...

4277906d 01/14/2010 01:14 am Christoph Hellwig

virtio-blk: remove dead variable in virtio_blk_handle_scsi

As pointed out by clang size is only ever written to, but never actually
used.

Signed-off-by: Christoph Hellwig <>
Signed-off-by: Anthony Liguori <>

f91672e5 01/14/2010 01:14 am Paolo Bonzini

use cross-prefix for pkgconfig

Since pkgconfig can give different output for different targets,
it should be tried with the cross-compilation prefix first.

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

67c274d3 01/14/2010 01:14 am Paolo Bonzini

fixes to the static compilation case for sdl

After the next commit, pkg-config could be used for the shared library
configuration case and sdl-config for static libraries. So I prepare
the test here by doing two changes:

at the same time I remove useless backslashes from the invocation of...

9316f803 01/14/2010 01:14 am Paolo Bonzini

use pkg-config for sdl whenever available

Together with the first patch this enables using the prefixed
pkg-config, thus picking up the correct flags for SDL.

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

4e2b0658 01/14/2010 01:14 am Paolo Bonzini

use pkg-config for libcurl whenever available

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

8f0056b7 01/14/2010 01:14 am Paolo Bonzini

move kbd/mouse handling to input.c

Move 200 lines out of vl.c already into common code that only needs to
be compiled once.

Signed-off-by: Paolo Bonzini <>
Signed-off-by: Anthony Liguori <>

884a0c76 01/13/2010 08:59 pm Blue Swirl

pcnet: remove dead nested assignment, spotted by clang

Although the value stored to 'addr' is used in the enclosing expression,
the value is never actually read from 'addr'.

Probably a typo.

Signed-off-by: Blue Swirl <>

f7e683b8 01/13/2010 08:58 pm Blue Swirl

Sparc32: remove dead increment, spotted by clang

Signed-off-by: Blue Swirl <>

f368a3ce 01/13/2010 08:55 pm Blue Swirl

Sparc32: remove a variable used only for debugging to avoid a clang warning

Signed-off-by: Blue Swirl <>

a9be79d6 01/13/2010 08:54 pm Blue Swirl

Sparc: comment out unused variable, spotted by clang

The variable is_annulled is referenced in a FIXME comment, so instead
of removing the variable, comment out the lines so that the FIXME
can be understood.

Signed-off-by: Blue Swirl <>