Statistics
| Branch: | Revision:

root / hw @ e59fb374

# Date Author Comment
f22f5b07 10/05/2009 05:32 pm Juan Quintela

cirrus_vga: also assign gr0/1 when writting shadow_gr0/1

This is a regression/bug caused by previous vga_cleanup. This fixes
Ubuntu installer reported by:
Pierre Riteau

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

50af3246 10/05/2009 05:32 pm Juan Quintela

vga: move back dirty_log functions to vga.c

They are needed there on qemu-kvm.

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

84e2e3eb 10/05/2009 05:32 pm Juan Quintela

vmstate: remove const for put operations

In a later patch, we introduce pre_save() and post_save() functions.
The whole point of that operation is to change things in the state.
Without this patch, we have to remove the const qualifier in each
use with a cast...

e59fb374 10/05/2009 05:32 pm Juan Quintela

vmstate: add version_id argument to post_load

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

4e3db917 10/05/2009 05:29 pm Aurelien Jarno

Revert "eepro100: Remove unused device status entries"

This reverts commit 3031efabd0bb744126a53f32c5426580b5d394d5.

c4c270e2 10/04/2009 03:59 pm Stefan Weil

eepro100: Add more i825xx devices

The new devices added here are still not functional -
partially because some patches are still missing,
partially because I cannot test them. Nevertheless
they belong to the same family and will be supported
by this driver some day....

3031efab 10/04/2009 03:54 pm Stefan Weil

eepro100: Remove unused device status entries

A lot of entries are unused (they were added by copy + paste
from other drivers during development of eepro100.c).

Removing them from nic_save, nic_load makes any
old saved status incompatible, so a new version...

317b7e67 10/04/2009 02:24 pm Michael S. Tsirkin

hw/omap_dma: add matching {} in if 0

MULTI_REQ is never defined, so it doesn't matter much, but since
we have an if statement there, let's add {} to clarify what it
should do if it's uncommented, and indent the code properly.

Signed-off-by: Michael S. Tsirkin <>...

c227f099 10/02/2009 12:12 am Anthony Liguori

Revert "Get rid of _t suffix"

In the very least, a change like this requires discussion on the list.

The naming convention is goofy and it causes a massive merge problem. Something
like this must be presented on the list first so people can provide input...

99a0949b 10/01/2009 09:45 pm malc

Get rid of _t suffix

Some not so obvious bits, slirp and Xen were left alone for the time
being.

Signed-off-by: malc <>

084bd071 09/30/2009 09:56 pm Michael S. Tsirkin

escc: fix another coding style nit

Fix another place with =- to be "= ".
to avoid confusion with old-style "
="
(which we also have, and needs to be fixed).

Signed-off-by: Michael S. Tsirkin <>
Signed-off-by: Blue Swirl <>

a0d98a71 09/30/2009 09:45 pm Michael S. Tsirkin

escc: fix coding style nit

Put space between = and - assigning a negative number
to avoid confusion with old-style "-="
(which we also have, and needs to be fixed).

Signed-off-by: Michael S. Tsirkin <>
Signed-off-by: Blue Swirl <>

f90554ad 09/30/2009 09:45 pm Michael S. Tsirkin

omap_dma: fix unbalanced { in commented out code

Fix unbalanced {} in commented out code.

Signed-off-by: Michael S. Tsirkin <>
Signed-off-by: Blue Swirl <>

bdd7e1bc 09/30/2009 09:45 pm Michael S. Tsirkin

twl92230: fix old style increment/decrement usage

Modern compilers do not parse "=-" as decrement:
you must use "-=" for that. Same for "=+"/"+=".

Signed-off-by: Michael S. Tsirkin <>
Signed-off-by: Blue Swirl <>

067d01de 09/27/2009 12:48 am Stefan Weil

eepro100: Fix format strings in debug messages

size_t arguments need %zu instead of %d.

Signed-off-by: Stefan Weil <>
Signed-off-by: Aurelien Jarno <>

41cbc23c 09/27/2009 12:48 am Stefan Weil

eepro100: Replace sprintf by snprintf

Signed-off-by: Stefan Weil <>
Signed-off-by: Aurelien Jarno <>

fd9ae2ec 09/27/2009 12:48 am Stefan Weil

eepro100: Remove unused code

Signed-off-by: Stefan Weil <>
Signed-off-by: Aurelien Jarno <>

aac443e6 09/27/2009 12:48 am Stefan Weil

eepro100: Enhanced logging and comments

  • Use TRACE macro to allow different logging flags.
  • Add new debugging messages and clean existing ones.

Signed-off-by: Stefan Weil <>
Signed-off-by: Aurelien Jarno <>

3df04ac3 09/25/2009 10:57 pm Mark McLoughlin

Fix coding style issue

Replace:

if (-1  foo())

with:

if (foo()  -1)

While this coding style is not in direct contravention of our currently
ratified CODING_STYLE treaty, it could be argued that the Article 3 of
the European Convention on Human Rights (prohibiting torture and "inhuman...

38972938 09/25/2009 10:52 pm Juan Quintela

gcc wants 1st static and then const

Signed-off-by: Juan Quintela <>
Signed-off-by: Blue Swirl <>

86178a57 09/25/2009 10:51 pm Juan Quintela

static and inline should came before the type of the functions

Signed-off-by: Juan Quintela <>
Signed-off-by: Blue Swirl <>

2b321d69 09/25/2009 10:51 pm Juan Quintela

Use proper typedef syntax

Why this ever compiled is a mistery to me.

Signed-off-by: Juan Quintela <>
Signed-off-by: Blue Swirl <>

979ba184 09/25/2009 05:34 pm Stefan Weil

Fix spelling in comment

replace Convery -> Convert

Cc: Paul Brook <>
Signed-off-by: Stefan Weil <>
Signed-off-by: Aurelien Jarno <>

d89c682f 09/25/2009 05:31 pm Stefan Weil

Suppress some variants of English in comments

Replace surpress, supress by suppress.

Signed-off-by: Stefan Weil <>
Signed-off-by: Aurelien Jarno <>

16d55035 09/21/2009 11:21 pm Blue Swirl

Fix Sparse warning about invalid access past the end of 'mode'

Signed-off-by: Blue Swirl <>

284b08f1 09/21/2009 10:50 pm Blue Swirl

Fix Sparse warning about obsolete struct initializer

Signed-off-by: Blue Swirl <>

6f4fc367 09/21/2009 09:39 pm Blue Swirl

Add 'static' to please Sparse

Signed-off-by: Blue Swirl <>

b9d38e95 09/21/2009 09:11 pm Blue Swirl

Fix Sparse warnings about using plain integer as NULL pointer

Signed-off-by: Blue Swirl <>

bd390e64 09/20/2009 07:19 pm Blue Swirl

Compile ne2000 only once

Signed-off-by: Blue Swirl <>

afcea8cb 09/20/2009 07:05 pm Blue Swirl

ioports: remove unused env parameter and compile only once

The CPU state parameter is not used, remove it and adjust callers. Now we
can compile ioport.c once for all targets.

Signed-off-by: Blue Swirl <>

5e520a7d 09/20/2009 06:35 pm Blue Swirl

Compile msix only once

Get page size in device init.

Signed-off-by: Blue Swirl <>

ca20cf32 09/20/2009 05:58 pm Blue Swirl

Compile loader only once

Callers must pass ELF machine, byte swapping and symbol LSB clearing
information to ELF loader. A.out loader needs page size information, pass
that too as a parameter.

Extract prototypes to a separate file. Move loader.[ch] and elf_ops.h under hw....

9801c7b4 09/20/2009 11:02 am Blue Swirl

Compile wdt_i6300esb only once

Signed-off-by: Blue Swirl <>

cdd0935c 09/19/2009 08:59 pm Blue Swirl

PPC: make system bus parent of PCI bus

Signed-off-by: Blue Swirl <>

cc9952f3 09/19/2009 06:44 pm Blue Swirl

ESP: convert to VMState

Signed-off-by: Blue Swirl <>

4f4cc0ef 09/18/2009 01:04 pm malc

audio: use muldiv64 where it makes sense

Signed-off-by: malc <>

2d6ee8e7 09/16/2009 09:58 pm Juergen Lock

qemu serial: lost tx irqs (affecting FreeBSD's new uart(4) driver)

Well one problem seems to be the rx condition,
... if ((s->ier & UART_IER_RDI) && (s->lsr & UART_LSR_DR))
is not enough to trigger an irq, yet still causes the following
conditions not to be checked anymore at all....

e1a0e47f 09/16/2009 01:15 am Aurelien Jarno

escc: fix IRQ routing, broken by 6c319c82223a1766c5d64a20051e5c6ab7b53951

The logic of Zilog makes channel B the first device and channel A the
second one.

Signed-off-by: Aurelien Jarno <>

dea21e97 09/15/2009 11:05 pm Gerd Hoffmann

ide/isa: convert to qdev.

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Blue Swirl <>

feef3102 09/15/2009 11:04 pm Gerd Hoffmann

ide/pci: convert to qdev.

With this patch applied ide drives (when attached to a pci adapter) can
be created via -device, like this:

-drive if=none,id=mydisk,file=/path/to/disk.img
-device ide-drive,drive=mydisk,bus=ide.0,unit=0

Note that creating a master on ide1 doesn't work that way. That is a...

9a43dba0 09/15/2009 11:00 pm Gerd Hoffmann

ide/pci: fix indention

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Blue Swirl <>

da4d0419 09/15/2009 10:23 pm Gerd Hoffmann

ide/qdev: add ide bus.

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Blue Swirl <>

88804180 09/15/2009 10:23 pm Gerd Hoffmann

split away drive init from ide_init2()

This allows the ide bus being initialized without drives attached
and the drives being attached and initialization later on as
separate step.

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Blue Swirl <>

71077c1c 09/15/2009 10:23 pm Gerd Hoffmann

qdev/pci: add pci_create_noinit()

Like pci_create_simple() but doesn't call qdev_init(), so one can
set properties before initializing the device.

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Blue Swirl <>

ffabf037 09/15/2009 02:17 am Aurelien Jarno

mips malta: ensure that the serial ports are associated with a device

The serial ports should be present even if associated with a null device
as some firmware wants to initialize them.

Signed-off-by: Aurelien Jarno <>

387f4a5a 09/15/2009 02:16 am Aurelien Jarno

hw/serial: don't create a char device if none is specified

When creating null devices, there is no way to ensure the unicity of
the labels. Bail out with an error message instead.

Signed-off-by: Aurelien Jarno <>

b37fc148 09/14/2009 07:40 pm Gerd Hoffmann

unbreak ppc/prep

Changes: * added isa bus, hooked up to the system bus. Not sure this is correct,
but 'info pci' lists lists no pci-isa bridge in the machine ...). * switches the default cpu to one which actually works.

Signed-off-by: Gerd Hoffmann <>...

f80237d4 09/14/2009 06:33 pm Blue Swirl

Add an ISA bus version of m48t59

Many thanks to Gerd Hoffmann for finding and fixing a bug in the initial
version.

Signed-off-by: Blue Swirl <>

042eb37a 09/14/2009 12:16 pm Daniel Jacobowitz

Fix Linux task preemption on Versatile board

Recent versions of the Linux kernel will not preempt CPU-intensive
tasks unless the clock used by sched_clock() works. On -M versatilepb
that's the 24MHz timer in the system controller. It's a very simple
timer, so implement it....

fad8c772 09/14/2009 11:44 am Edgar E. Iglesias

pflash_cfi01: Correct debug build, no functional changes.

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

a6124103 09/14/2009 08:21 am malc

ne2000-isa: Do not free memory owned by qdev

Signed-off-by: malc <>

c5ff6d54 09/13/2009 11:32 am Blue Swirl

Sparc64: make system bus parent of PCI bus

Signed-off-by: Blue Swirl <>

075cd324 09/13/2009 11:32 am Blue Swirl

x86: move a declaration to header

Signed-off-by: Blue Swirl <>

675d6f82 09/13/2009 11:32 am Blue Swirl

x86: add 'const'

Signed-off-by: Blue Swirl <>

6a0b9cc9 09/12/2009 06:20 pm Reimar Döffinger

Make string arrays used to convert numbers to strings when DEBUG_EEPRO100 is enabled const.

Signed-off-by: Reimar Döffinger <>
Signed-off-by: Blue Swirl <>

d8becc35 09/12/2009 06:00 pm Jan Kiszka

Fix xen build after sys-queue renaming

Signed-off-by: Jan Kiszka <>
Signed-off-by: Blue Swirl <>

72cf2d4f 09/12/2009 10:36 am Blue Swirl

Fix sys-queue.h conflict for good

Problem: Our file sys-queue.h is a copy of the BSD file, but there are
some additions and it's not entirely compatible. Because of that, there have
been conflicts with system headers on BSD systems. Some hacks have been
introduced in the commits 15cc9235840a22c289edbe064a9b3c19c5f49896,...

435c247a 09/12/2009 01:50 am malc

gus: Do not manually free the state, qdev does it for us

Signed-off-by: malc <>

057fa65c 09/12/2009 01:50 am malc

audio: remove lsbindex/popcount in favour of host-utils's ctz32

Signed-off-by: malc <>

e6cb4d45 09/11/2009 07:10 pm Juan Quintela

vmstate: port hpet device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

747791f1 09/11/2009 07:10 pm Juan Quintela

vmstate: port serial device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

7e72abc3 09/11/2009 07:10 pm Juan Quintela

vmstate: port cirrus_vga device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

bcb58d9a 09/11/2009 07:10 pm Juan Quintela

vmstate: port vmmouse device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

3c619b59 09/11/2009 07:10 pm Juan Quintela

vmstate: port pckbd device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

80a04bbe 09/11/2009 07:10 pm Juan Quintela

vmstate: add uint64 array support

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

3e9e9888 09/11/2009 07:10 pm Juan Quintela

vmstate: port ioapic device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

67f0875e 09/11/2009 07:10 pm Juan Quintela

hpet: it is imposible that qemu_timer field is NULL at this point

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

7b5045c5 09/11/2009 07:10 pm Juan Quintela

vmstate: port dma device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

7d2edd40 09/11/2009 07:10 pm Juan Quintela

vmstate: port fw_cfg device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

77eea838 09/11/2009 07:10 pm Juan Quintela

vmstate: port i8259 device

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

80cd83e7 09/11/2009 07:10 pm Juan Quintela

vmstate: add support for uint8_t equal

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

d7a6c270 09/11/2009 07:10 pm Juan Quintela

vmstate: port fdc device

We can't move fifo back to an embeded array because it needs to be aligned

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

632cf073 09/11/2009 07:10 pm Juan Quintela

vmstate: add support for arrays of uint16_t

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

1eb7538b 09/11/2009 07:10 pm Juan Quintela

vmstate: add sensible arguments to vmstate_unregister()

vmsd alone is not enugh, because we can have several structs saved with the same description (vmsd).

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

752ff2fa 09/11/2009 07:10 pm Juan Quintela

vmstate: rename run_after_load() -> post_load()

This naming was used in kvm tree, and is easier to remember

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

fd4d52de 09/11/2009 07:10 pm Juan Quintela

vmstate: Add pre_load() hook

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

8fb0791d 09/11/2009 07:10 pm Juan Quintela

vmstate: Add pre/post_save() hooks

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

6ee093c9 09/11/2009 06:19 pm Juan Quintela

Unexport ticks_per_sec variable. Create get_ticks_per_sec() function

Signed-off-by: Juan Quintela <>
Signed-off-by: Anthony Liguori <>

06113719 09/11/2009 06:19 pm Gerd Hoffmann

qdev: add parser for chardev properties

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Anthony Liguori <>

f0457e8d 09/11/2009 06:19 pm Gerd Hoffmann

convert msmouse chardev to QemuOpts.

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Anthony Liguori <>

dc1c21e6 09/11/2009 06:19 pm Gerd Hoffmann

convert braille chardev to QemuOpts.

Signed-off-by: Gerd Hoffmann <>
Signed-off-by: Anthony Liguori <>

ed757e14 09/11/2009 06:19 pm Yan Vugenfirer

VirtIO: Fix QEMU crash during Windows PNP tests

Hello,

In some cases bus driver can deassert "bus master" bit in PCI command
register. The driver will no longer be able to update related registers in
the device. Eventually it will cause QEMU to exit in "virtqueue_num_heads"...

d03f09cc 09/11/2009 06:19 pm Markus Armbruster

qdev: Fix i6300 upcast

Use DO_UPCAST() instead of container_of() to go from PCIDevice to
I6300State. This ensures that PCIDevice is the first member of struct
I6300State.

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

711bf3d9 09/11/2009 06:19 pm Michael S. Tsirkin

qemu: make virtio-blk PCI compliant by default

commit bf011293faaa7f87e4de83185931e7411b794128 made virtio-blk-pci not
PCI-compliant, since it makes region 0 (which is an i/o region)
size > 256, and, since PCI 2.1, i/o regions are limited to 256 bytes size....

b8193adb 09/11/2009 06:19 pm Michael S. Tsirkin

qemu: init all queues to NO_VECTOR value

initialize vectors for all vqs to VIRTIO_NO_VECTOR rather than 0 which
is a valid vector. This fixes migration which happened before driver
was loaded.

Signed-off-by: Michael S. Tsirkin <>
Reported-by: Amit Shah <>...

e900a7b7 09/11/2009 06:19 pm Christoph Hellwig

block: add enable_write_cache flag

Add a enable_write_cache flag in the block driver state, and use it to
decide if we claim to have a volatile write cache that needs controlled
flushing from the guest. The flag is off if cache=writethrough is
defined because O_DSYNC guarantees that every write goes to stable...

b0484ae4 09/11/2009 06:19 pm Christoph Hellwig

ide: use bdrv_aio_flush

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

aa659be3 09/11/2009 06:19 pm Christoph Hellwig

virtio-blk: add volatile writecache feature

Add a new VIRTIO_BLK_F_WCACHE feature to virtio-blk to indicate that we have
a volatile write cache that needs controlled flushing. Implement a
VIRTIO_BLK_T_FLUSH operation to flush it.

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

1c3173b9 09/11/2009 06:18 pm Anthony Liguori

Revert "don't call cpu_sychronize_state from reset handlers"

This reverts commit 733318ea9c6d846a6a047b87619e7d9d6e9707d1.

Signed-off-by: Anthony Liguori <>

91553dcc 09/11/2009 06:18 pm Kevin Wolf

virtio-blk: Use bdrv_aio_multiwrite

It is quite common for virtio-blk to submit more than one write request in a
row to the qemu block layer. Use bdrv_aio_multiwrite to allow block drivers to
optimize its handling of the requests.

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

31226166 09/10/2009 06:59 pm malc

Fix formatting

a5e8e46b 09/10/2009 06:59 pm malc

Fix formatting, get rid of conf and fix description

41875be5 09/10/2009 06:59 pm malc

Fix formatting and and description field

9453c5bc 09/10/2009 06:48 pm Gerd Hoffmann

qdev/isa: convert ne2000

Also split the isa bits into a separate source file, so we don't drag in
a dependency for isa-bus.o for machines which want ne2k_pci only.

Signed-off-by: Gerd Hoffmann <>

11d23c35 09/10/2009 06:48 pm Gerd Hoffmann

qdev/isa: finish pckbd conversion

drop old init path and switch remaining users to
isa_create_simple().

Signed-off-by: Gerd Hoffmann <>

32e0c826 09/10/2009 06:48 pm Gerd Hoffmann

qdev/isa: convert real time clock

Signed-off-by: Gerd Hoffmann <>

f7b4f61f 09/10/2009 06:48 pm Gerd Hoffmann

qdev/isa: convert soundblaster

Signed-off-by: Gerd Hoffmann <>

f8ba7846 09/10/2009 06:48 pm Gerd Hoffmann

qdev/isa: convert cs4231a sound card

Signed-off-by: Gerd Hoffmann <>

9df34396 09/10/2009 06:48 pm Gerd Hoffmann

qdev/isa: convert gravis ultrasound

Signed-off-by: Gerd Hoffmann <>

924f6d72 09/10/2009 06:47 pm Gerd Hoffmann

qdev: add isa_create() function

Like isa_create_simple, but doesn't call qdev_init, so one can set
properties after creating and before initializing the device.

Signed-off-by: Gerd Hoffmann <>