On Mon, Feb 4, 2013 at 2:27 AM, liguang wrote:
> orginally, numa data was packed into an array,
> which was implicit and hard to maintain, we
> define a struct for this data, hope to be as
> clear as enough.
> also, we only pass cpumask of corresponding
> nodes to seabios, and leave the paring wor
On 02/04/13 18:51, Luiz Capitulino wrote:
> On Fri, 1 Feb 2013 18:38:18 +0100
> Laszlo Ersek wrote:
>
>>
>> Signed-off-by: Laszlo Ersek
>
> Splitting this as I suggested in the other patch would make review
> easier. I honestly got a bit lost while reviewing this one.
I think you cannot revie
On Mon, Feb 4, 2013 at 2:27 AM, liguang wrote:
> Signed-off-by: liguang
> ---
> hw/pc.c | 14 +++---
> 1 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/hw/pc.c b/hw/pc.c
> index 01d00f6..0ccd775 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -652,12 +652,20 @@ static voi
On 4 February 2013 17:29, Paolo Bonzini wrote:
> This reorganizes the over 600 files in the hw/ directory to multiple
> subdirectories: acpi/, audio/, block/, bt/, char/, display/, i2c/, input/,
> isa/, misc/, net/, scsi/, sd/, ssi/, timer/, virtio/, watchdog/, xen/.
> These are in addition to the
Il 04/02/2013 18:51, Fabien Chouteau ha scritto:
> On 02/04/2013 06:35 PM, Paolo Bonzini wrote:
>> Il 04/02/2013 16:07, Stefan Hajnoczi ha scritto:
> No idea why the windows version has been mixing g_poll() and select().
> I'd hope that this series kills select() for uniformity's sake, but
On 02/04/13 18:46, Luiz Capitulino wrote:
> On Fri, 1 Feb 2013 18:38:17 +0100
> Laszlo Ersek wrote:
>
>> Use an Error to communicate the "stop the search" message.
>>
>> Signed-off-by: Laszlo Ersek
>> ---
>> hw/qdev-monitor.c | 24
>> 1 files changed, 16 insertions(+
Paolo Bonzini writes:
> Il 04/02/2013 18:19, Markus Armbruster ha scritto:
>> +/* 2 Boundary condition test cases */
>> +/* 2.1 First possible sequence of a certain length */
>> +/* 2.1.5 5 bytes U+20 */
>> +{
>> +"\"\xF8\x88\x80\x80\x80\"",
>> +
From: Eduardo Habkost
Bit 10 of CPUID[8000_0001].EDX is not defined as an alias of
CPUID[1].EDX[10], so do not duplicate it on
kvm_arch_get_supported_cpuid().
Signed-off-by: Eduardo Habkost
Reviewed-By: Igor Mammedov
Reviewed-by: Don Slutz
Signed-off-by: Blue Swirl
(cherry picked from commit
Il 04/02/2013 18:57, Andreas Färber ha scritto:
>> > I haven't thought of a way to ensure patch 19 is safe, apart from trusting
>> > the poisoning mechanism. [...]
> I've wondered, is it possible it poison from the Makefile? Then we could
> forbid that the whole of common-obj-y uses CONFIG_USER_ONL
On Fri, 1 Feb 2013 18:38:12 +0100
Laszlo Ersek wrote:
> This series is the first in converting do_device_add() to qapi.
I've reviewed this, although it doesn't apply on master anymore and
then I was unable to try it.
Also, it's a good idea to CC qdev expert guys.
On Fri, 1 Feb 2013 18:38:19 +0100
Laszlo Ersek wrote:
>
> Signed-off-by: Laszlo Ersek
> ---
> hw/qdev-monitor.h |3 ++-
> hw/qdev-monitor.c | 29 +++--
> vl.c |9 +++--
> 3 files changed, 24 insertions(+), 17 deletions(-)
>
> diff --git a/hw
Am 04.02.2013 18:29, schrieb Paolo Bonzini:
> I haven't thought of a way to ensure patch 19 is safe, apart from trusting
> the poisoning mechanism. [...]
I've wondered, is it possible it poison from the Makefile? Then we could
forbid that the whole of common-obj-y uses CONFIG_USER_ONLY or
CONFIG_S
On Fri, 1 Feb 2013 18:38:18 +0100
Laszlo Ersek wrote:
>
> Signed-off-by: Laszlo Ersek
Splitting this as I suggested in the other patch would make review
easier. I honestly got a bit lost while reviewing this one.
> ---
> hw/qdev-monitor.c | 29 +++--
> 1 files chan
On 02/04/2013 06:35 PM, Paolo Bonzini wrote:
> Il 04/02/2013 16:07, Stefan Hajnoczi ha scritto:
No idea why the windows version has been mixing g_poll() and select().
I'd hope that this series kills select() for uniformity's sake, but the
00/10 subject and the commit msg for this pat
** Changed in: qemu (Debian)
Status: Confirmed => Fix Released
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1024275
Title:
bad iteraction between -daemonize and -nographic
Status in QEMU:
Signed-off-by: Paolo Bonzini
---
hw/vt82c686.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/hw/vt82c686.c b/hw/vt82c686.c
index 2d8e398..c2b1bfc 100644
--- a/hw/vt82c686.c
+++ b/hw/vt82c686.c
@@ -26,8 +26,6 @@
#include "qemu/timer.h"
#include "exec/address-spaces.h"
On Fri, 1 Feb 2013 18:38:17 +0100
Laszlo Ersek wrote:
> Use an Error to communicate the "stop the search" message.
>
> Signed-off-by: Laszlo Ersek
> ---
> hw/qdev-monitor.c | 24
> 1 files changed, 16 insertions(+), 8 deletions(-)
>
> diff --git a/hw/qdev-monitor.c
AioHandler already has a GPollFD so we can directly use its
events/revents.
Add the int pollfds_idx field to AioContext so we can map g_poll(3)
results back to AioHandlers.
Reuse aio_dispatch() to invoke handlers after g_poll(3).
Signed-off-by: Stefan Hajnoczi
---
aio-posix.c | 67
Il 04/02/2013 18:19, Markus Armbruster ha scritto:
> +/* 2 Boundary condition test cases */
> +/* 2.1 First possible sequence of a certain length */
> +/* 2.1.5 5 bytes U+20 */
> +{
> +"\"\xF8\x88\x80\x80\x80\"",
> +NULL,
aio-posix.c could not take advantage of G_IO_HUP and G_IO_ERR because
select(2) does not have equivalent events. Now that g_poll(3) is used
we can support G_IO_HUP and G_IO_ERR.
Signed-off-by: Stefan Hajnoczi
---
aio-posix.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
d
On 02/04/13 17:22, baban devkate wrote:
>
> When Working with OVMF+QEMU, What is the Max supported
> Buffer size (in Block I/O Protocol)?
Are you producing EFI_BLOCK_IO_PROTOCOL, or consuming it?
If producing (ie. implementing a driver), then why does it matter?
BufferSize should be an inte
Il 04/02/2013 16:07, Stefan Hajnoczi ha scritto:
>> > No idea why the windows version has been mixing g_poll() and select().
>> > I'd hope that this series kills select() for uniformity's sake, but the
>> > 00/10 subject and the commit msg for this patch indicate otherwise.
> This is why I CCed Fab
Signed-off-by: Paolo Bonzini
---
MAINTAINERS | 168 ---
1 files changed, 91 insertions(+), 77 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 21043e4..7d1a68e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -59,75 +59,90 @@ Alpha
Other devices are now moved out of hw/ARCH, adding new CONFIG_*
symbols that let them be included selectively in the emulators.
The devices however are still compiled in target-specific
directories.
Signed-off-by: Paolo Bonzini
---
Makefile.target|1 +
con
From: Gerd Hoffmann
usb-audio is a full speed (1.1) device,
but bcdUSB claims it is usb 2.0. Fix it.
Signed-off-by: Gerd Hoffmann
(cherry picked from commit 2bbd086c41a00dc4384727ec895a94890c688eb5)
Signed-off-by: Michael Tokarev
---
hw/usb/dev-audio.c |2 +-
1 file changed, 1 insertion
This moves devices that are board-specific and do not fit in
any more specific category to hw/ARCH directories. Devices
that can be moved to specific categories are left in hw/
for now. The next patch will take care of them, and remove
the strange hw/ARCH/../FILE indirection.
Signed-off-by: Paol
This simplifies the scripted execution of the next patch.
Signed-off-by: Paolo Bonzini
---
hw/ppc/e500-ccsr.h |2 +-
hw/ppc/e500plat.c |2 +-
hw/ppc/mpc8544ds.c |2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/ppc/e500-ccsr.h b/hw/ppc/e500-ccsr.h
index f20f51
Prepare the new directory structure.
Signed-off-by: Paolo Bonzini
---
hw/Makefile.objs | 29 ++---
1 files changed, 26 insertions(+), 3 deletions(-)
create mode 100644 hw/acpi/Makefile.objs
create mode 100644 hw/audio/Makefile.objs
create mode 100644 hw/bloc
The situation with device-hotplug.c is similar to qdev-monitor.c.
Add a stub for pci_drive_hot_add, so that it can be compiled once,
and move it out of hw/.
Signed-off-by: Paolo Bonzini
---
Makefile.objs |2 +-
hw/device-hotplug.c => device-hotplug.c | 13 ++--
fifo.c is generic code that can be easily unit tested. So it
belongs in libqemuutil.
Signed-off-by: Paolo Bonzini
---
hw/Makefile.objs |1 -
hw/xilinx_spi.c |2 +-
hw/xilinx_spips.c |2 +-
include/migration/vmstate.h |2 ++
Braille and msmouse support is in hw/, but it is not hardware.
Move it to the backends/ directory.
Signed-off-by: Paolo Bonzini
---
backends/Makefile.objs |4
{hw => backends}/baum.c|4 ++--
{hw => backends}/msmouse.c |2 +-
hw/Makefile.objs |
On Mon, 04 Feb 2013 13:52:32 +0100
Andreas Färber wrote:
> Am 04.02.2013 12:08, schrieb Igor Mammedov:
> > On Sat, 2 Feb 2013 01:37:07 +0100
> > Andreas Färber wrote:
> >
> >> Move x86_def_t definition to header and embed into X86CPUClass.
> >> Register types per built-in model definition.
> >
On Fri, 1 Feb 2013 18:38:15 +0100
Laszlo Ersek wrote:
>
> Signed-off-by: Laszlo Ersek
I usually split this kind of patch the following way:
1. add an Error ** argument to the function reporting the error. Callers
are changed to pass NULL for the new argument
2. Handling of the new err
On Mon, Feb 04, 2013 at 02:42:55PM +0100, Andreas Färber wrote:
> Am 27.01.2013 07:26, schrieb Andreas Färber:
> > Around r3361 (81fdc5f8d2d681da8d255baf0713144f8656bac9) env->debug1 used
> > to contain the address of an MMU fault. This is now written into
> > env->pregs[PR_EDA] instead.
> >
> > S
Test cases are scraped from Markus Kuhn's UTF-8 decoder capability and
stress test at
http://www.cl.cam.ac.uk/~mgk25/ucs/examples/UTF-8-test.txt
Unfortunately, both JSON parser and formatter misbehave right now.
This test expects current, incorrect results. They're all clearly
marked, and are to
From: Kevin Wolf
Without this, s->nsector can become negative and badness happens (trying
to malloc huge amount of memory and glib calls abort())
Signed-off-by: Kevin Wolf
Signed-off-by: Stefan Hajnoczi
(cherry picked from commit 038268e2e8087ee2fd8987a77ba580e15f14c147)
Signed-off-by: Michae
We will need to loop over AioHandlers calling ->io_read()/->io_write()
when aio_poll() is converted from select(2) to g_poll(2).
Luckily the code for this already exists, extract it into the new
aio_dispatch() function.
Two small changes:
* aio_poll() checks !node->deleted to avoid calling hand
On Mon, Feb 4, 2013 at 8:22 AM, baban devkate wrote:
> When Working with OVMF+QEMU, What is the Max supported Buffer size (in Block
> I/O Protocol)?
I don't think there should be a set limit, although the VM RAM size is
obviously 1 limit. :)
-Jordan
On Sun, 2013-02-03 at 14:10 +, Pandarathil, Vijaymohan R wrote:
> Add support for error containment when a VFIO device assigned to a KVM
> guest encounters an error. This is for PCIe devices/drivers that support AER
> functionality. When the host OS is notified of an error in a device either
>
On Sun, 2013-02-03 at 14:10 +, Pandarathil, Vijaymohan R wrote:
> - New VFIO_SET_IRQ ioctl option to pass the eventfd that is signaled
> when
> an error occurs in the vfio_pci_device
>
> - Register pci_error_handler for the vfio_pci driver
>
> - When the device en
On Sun, 2013-02-03 at 14:10 +, Pandarathil, Vijaymohan R wrote:
> - Create eventfd per vfio device assigned to a guest and register an
> event handler
>
> - This fd is passed to the vfio_pci driver through the SET_IRQ ioctl
>
> - When the device encounters an error
one remark below:
On 02/04/13 13:12, Stefan Hajnoczi wrote:
> Convert glib file descriptor polling from rfds/wfds/xfds to GPollFD.
>
> The Windows code still needs poll_fds[] and n_poll_fds but they can now
> become local variables.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> main-loop.c | 71
b0b873a07872f7ab7f66f259c73fb9dd42aa66a9 bumped the vmstate version and
introduced an old-style load function to handle migration from prior
(<= 1.2) versions.
The load function passes the top-level PIIX4PMState pointer to
vmstate_load_state() to handle nested structs for APMState and
pci_status,
> > [TODO]
> > Other functions below are used to output error messages in qemu.
> > - qerror_report() is called in many source codes.
>
> Not a problem, it ends up in qerror_print(), which calls error_report().
Thanks.
>
> > - fprintf() is called in vl.c.
>
> All over the place, not just vl.
On 01/18/2013 11:02 AM, Stefan Berger wrote:
This patch adds support for cancelling an executing TPM command.
In Linux for example a user can cancel a command through the TPM's
sysfs 'cancel' entry using
echo "1" > /sysfs/.../cancel
This patch propagates the cancellation to the host TPM's sysf
On Fri, 1 Feb 2013 18:38:14 +0100
Laszlo Ersek wrote:
>
> Signed-off-by: Laszlo Ersek
> ---
> hw/qdev-monitor.c |7 +--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/hw/qdev-monitor.c b/hw/qdev-monitor.c
> index 3ec9e49..32be5a2 100644
> --- a/hw/qdev-monitor.c
Hi All,When Working with OVMF+QEMU, What is the Max supported Buffer
size (in Block
I/O Protocol)?
Thanks,
Baban
On 02/04/13 16:07, Stefan Hajnoczi wrote:
> slirp uses OOB because it implements TCP. I don't think we can drop it.
I considered slirp for that reason, and I did grep the tree for OOB and
saw some hits. I didn't track it down but I thought slirp would use
lower level sockets (SOCK_RAW?) for the
On Mon, 04 Feb 2013 08:48:57 +0100
Markus Armbruster wrote:
> [Note cc: Luiz]
>
> Peter Maydell writes:
>
> > On 2 February 2013 21:37, Andreas Färber wrote:
> >> Am 02.02.2013 22:19, schrieb Peter Maydell:
> >>> It's OK and expected for visitors to return errors when presented with
> >>> the
> > Do we really want to add timestamps to error messages unconditionally?
> > I don't doubt it's useful in your scenario, but most of the time it's
> > just annoying clutter.
>
> Agreed, I think it should be an option.
OK. I will add the timestamp as an option.
> Also remember that management
From: Paolo Bonzini
When reverse connection is in use, there is no active VNC server
socket. Because of this, getsockopt(-1, ...) is attempted and
the following error is emitted:
$ socat TCP-LISTEN:5900,reuseaddr TCP-LISTEN:5901,reuseaddr &
$ x86_64-softmmu/qemu-system-x86_64 -vnc local
From: Dmitry Fleytman
Real HW always treats RX ring with RDH == RDT as empty.
Emulation is supposed to behave the same.
Reported-by: Chris Webb
Reported-by: Richard Davies
Signed-off-by: Dmitry Fleytman
Signed-off-by: Stefan Hajnoczi
(cherry picked from commit e5b8b0d4ba29fe1268ba049519a1b0c
Pass voltage sensor properties to the sysctl device. Since
these are daughterboard specific, we specify them via the
VEDBoardInfo structure.
Signed-off-by: Peter Maydell
---
hw/vexpress.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/hw/vexpress.c b/hw/vex
The SYS_CFGCTRL register consists of separate fields
for DCC, function, site, position and device, as well
as a read/write bit. Refactor the code handling SYS_CFGCTRL
writes to make it easier to add support for functions
like SYS_CFG_OSC which support multiple device fields.
We also pull the handli
Set the reset values for the VExpress daughterboard oscillators
via the new sysctl properties.
Signed-off-by: Peter Maydell
---
hw/vexpress.c | 32
1 file changed, 32 insertions(+)
diff --git a/hw/vexpress.c b/hw/vexpress.c
index 8653200..547379a 100644
--- a/
Vadim Rozenfeld writes:
> On Mon, 2013-02-04 at 14:22 +0200, Michael S. Tsirkin wrote:
>> On Mon, Feb 04, 2013 at 02:00:46PM +0200, Yan Vugenfirer wrote:
>> >
>> virtio spec is very explicit that revision never changes:
>> "The device must also have a Revision ID of 0 to match this
>> specificat
Pass the daughterboard-specific proc_id property to the code that
creates the sysctl device via the VEDBoardInfo struct, rather than
by having the daughterboard init function write to a uint32_t*
argument. This is a cleaner way to pass the info around, and
is in line with the way we are going to ha
Implement the SYS_CFG_OSC function. Since the idea of
programmable clock rates doesn't make much sense for QEMU,
we simply allow the clock rate to be read back as written.
The number and value of the daughterboard oscillators varies
between daughterboards, so we provide an array property to
allow t
This patchset fixes a number of warnings produced by Linux 3.8
on the vexpress boards resulting from newer kernels prodding
more of the SYS_CFG register operations.
Since some of the newly implemented functions expose voltage
and oscillator settings which vary in number and value between
vexpress
From: Joel Martin
If TIGHT_PNG is not enabled by the --enable-vnc-png configure flag
then do not report to the client that it is supported.
Also, since TIGHT_PNG is the same as the TIGHT encoding but with the
filter/copy replaced with PNG data, adding it to the supported
encodings list when it i
SYS_CFG_MUXFPGA allows the guest to select whether the
video output should come from the motherboard's LCD
controller or the daughterboard's one. Since QEMU doesn't
currently support selecting the video output like this,
implement as a no-op, so Linux doesn't complain about
the register not being i
Implement the SYS_CFG_VOLT registers which return the voltage
of various supplies on motherboard and daughterboard. Since
QEMU implements a perfectly stable power supply these registers
always return a constant value. The number and value of the
daughterboard voltages is dependent on the specific d
On 31 January 2013 19:50, Laurent Vivier wrote:
> The parameter "union semun" of semctl() is not a value
> but a pointer to the value.
Hi. For your next patch could you make sure you send it as
a fresh email rather than a followup to the previous version?
Anthony's patch-handling tools don't real
Convert this device from old-style qdev init to an instance_init
function. We don't need a realize function yet, though.
Signed-off-by: Peter Maydell
---
hw/arm_sysctl.c |7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/hw/arm_sysctl.c b/hw/arm_sysctl.c
index bb56238.
From: Peter Maydell
Clean up the mix of getting the RAM size from the global ram_size
and from the ram_size field in the arm_boot_info structure, so
that we always use the structure field.
Signed-off-by: Peter Maydell
Reviewed-by: Peter A. G. Crosthwaite
(cherry picked from commit 0b94438480c8
From: Kevin Shanahan
If readonly=on is given at device creation time, the ->readonly flag
needs to be set in the block driver state for this device so that
readonly-ness is preserved across media changes (qmp change command).
Similarly, to preserve the snapshot property requires ->open_flags to
b
On Mon, Feb 04, 2013 at 03:37:39PM +0100, Laszlo Ersek wrote:
> I assume this is complex because the pre-patch situation is overly
> complex as well:
>
> slirp -> fd_set
> iohandler -> fd_set
> unix
From: Don Slutz
The check using INT_MAX (2147483647) is wrong in this case.
Signed-off-by: Fred Oliveira
Signed-off-by: Don Slutz
Signed-off-by: Stefan Hajnoczi
(cherry picked from commit 2e84849aa2cc7f220d3b3668f5f7e3c57bb1b590)
Signed-off-by: Michael Tokarev
---
target-i386/cpu.c |2
Am 04.02.2013 14:44, schrieb Peter Maydell:
> Convert this device from old-style qdev init to an instance_init
> function. We don't need a realize function yet, though.
>
> Signed-off-by: Peter Maydell
> ---
> hw/arm_sysctl.c |7 +++
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
>
From: Paolo Bonzini
Sending more was possible if the buffer was large.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
Signed-off-by: Juan Quintela
(cherry picked from commit bde54c08b4854aceee3dee25121a2b835cb81166)
Conflicts (backport):
buffered_file.c
Signed-off-by: Michae
From: "陳韋任 (Wei-Ren Chen)"
While reading microMIPS decoding, I found a possible wrong opcode
encoding. According to [1] page 166, the bits 13..12 for MULTU is
0x01 rather than 0x00. Please review, thanks.
[1] MIPS Architecture for Programmers VolumeIV-e: The MIPS DSP
Application-Specific Ext
From: Hans de Goede
According to the spec we must raise an interrupt when one is requested
even for non active tds.
Linux depends on this, for bulk transfers it runs an inactivity timer
to work around a bug in early uhci revisions, when we take longer then
200 ms to process a packet, this timer
From: Max Filippov
Zero out tcg_ctx.gen_opc_instr_start for instructions representing the
last guest opcode in the TB.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Max Filippov
Signed-off-by: Blue Swirl
(cherry picked from commit 36f25d2537c40c6c47f4abee5d31a24863d1adf7)
Signed-off-by: Michael T
From: Alexey Kardashevskiy
QEMU allocates a map enough for 4k pages. However the system page size
can be 64K (for example on POWER) and the host kernel uses only a small
part of it as one big stores a dirty flag for 16 pages 4K each,
the hpratio variable stores this ratio and
the kvm_get_dirty_pa
Hi
Please send in any agenda topics you are interested in.
Later, Juan.
Now that all *_fill() and *_poll() functions use GPollFD we no longer
need rfds/wfds/xfds or pollfds_from_select()/pollfds_to_select().
>From now on everything uses GPollFD.
Signed-off-by: Stefan Hajnoczi
---
main-loop.c | 77 ++---
1 file
Stefan Hajnoczi writes:
> LVM volumes can be set read-only with "lvchange --permission r
> ". The device node permissions remain unchanged so the volume
> can still be opened O_RDWR. Actual writes will fail.
Are you sure it's just LVM? blockdev(8) suggests it's a more general
issue. Check ou
From: David Gibson
tcp_chr_connect(), unlike for example udp_chr_update_read_handler() does
not check if the fd it is using is valid (>= 0) before passing it to
qemu_set_fd_handler2(). If using e.g. a TCP serial port, which is not
initially connected, this can result in -1 being passed to FD_ISS
Fabio Fantoni writes:
> Il 04/02/2013 11:36, Markus Armbruster ha scritto:
>> Fabio Fantoni writes:
>>
>>> I tried to disable "default floppy" without use -nodefaults option
>>> that disable other things.
>>> I didn't found other parameters to do that in docs and code for now.
>>> Can someone te
On 02/01/2013 02:32 PM, Juan Quintela wrote:
Hi
migration expected_downtime calculation was removed on commit
e4ed1541ac9413eac494a03532e34beaf8a7d1c5.
We add the calculation back. Before doing the calculation we do:
- expected_downtime intial value is max_downtime. Much, much better
int
From: Avi Kivity
The memory core drops regions that are hidden by another region (for example,
during BAR sizing), but it doesn't do so correctly if the lower address of the
existing range is below the lower address of the new range.
Example (qemu-system-mips -M malta -kernel vmlinux-2.6.32-5-4k
From: Stefan Hajnoczi
Zero writes have cluster granularity in QED. Therefore they can only be
used to zero entire clusters.
If the zero write request leaves sectors untouched, zeroing the entire
cluster would obscure the backing file. Instead return -ENOTSUP, which
is handled by block.c:bdrv_c
Comments in-line.
On 02/04/13 13:12, Stefan Hajnoczi wrote:
> Use g_poll(3) instead of select(2). Well, this is kind of a cheat.
> It's true that we're now using g_poll(3) on POSIX hosts but the *_fill()
> and *_poll() functions are still using rfds/wfds/xfds.
>
> We've set the scene to start con
Rusty Russell writes:
> Anthony Liguori writes:
>> Michael Tokarev writes:
>>
>>> 03.02.2013 17:23, Yan Vugenfirer wrote:
>>>
> If it helps, mq changes the config size from 8 to 16 bytes. If the
> driver was making an assumption about an 8-byte config size, that's
> likely what the
From: Aurelien Jarno
OpenBIOS on sparc64 only support Standard VGA and not Cirrus VGA. Don't
build Cirrus VGA support so that it can't be selected.
This fixes the breakage introduced by commit f2898771.
Reported-by: Richard Henderson
Cc: Blue Swirl
Signed-off-by: Aurelien Jarno
Tested-by: Ri
Slirp uses rfds/wfds/xfds more extensively than other QEMU components.
The rarely-used out-of-band TCP data feature is used. That means we
need the full table of select(2) to g_poll(3) events:
rfds -> G_IO_IN | G_IO_HUP | G_IO_ERR
wfds -> G_IO_OUT | G_IO_ERR
xfds -> G_IO_PRI
I came up wit
Add support for declaring array properties for qdev devices.
These work by defining an initial static property 'len-arrayname'
which the user of the device should set to the desired size
of the array. When this property is set, memory is allocated
for the array elements, and dynamic properties "arr
From: Aurelien Jarno
The load/store slow path has been broken in e141ab52d:
- We need to move 4 registers for store functions and 3 registers for
load functions and not the reverse.
- According to the s390x calling convention the arguments of a function
should be zero extended. This means tha
From: Gerhard Wiesinger
Fixed a MAJOR BUG in VMDK files on file boundaries on reads
and ALSO ON WRITES WHICH MIGHT CORRUPT THE IMAGE AND DATA!!
Triggered for example with the following VMDK file (partly listed):
RW 4193792 FLAT "XP-W1-f001.vmdk" 0
RW 2097664 FLAT "XP-W1-f002.vmdk" 0
RW 41937
LVM volumes can be set read-only with "lvchange --permission r
". The device node permissions remain unchanged so the volume
can still be opened O_RDWR. Actual writes will fail.
This results in odd behavior for QEMU. bdrv_open() is supposed to fail
if a read-only image is being opened with BDRV
Am 04.02.2013 14:33, schrieb Fabien Chouteau:
> On 02/04/2013 01:24 PM, Markus Armbruster wrote:
>>
>> Good error reporting is hard. Knowledge about the error and its context
>> gets lost as you move up the call chain. Knowledge about how to report
>> errors gets lost as you move down.
>>
>
> Yo
Am 27.01.2013 07:26, schrieb Andreas Färber:
> Around r3361 (81fdc5f8d2d681da8d255baf0713144f8656bac9) env->debug1 used
> to contain the address of an MMU fault. This is now written into
> env->pregs[PR_EDA] instead.
>
> Signed-off-by: Andreas Färber
> ---
> target-cris/op_helper.c |2 +-
>
On Sat, 2 Feb 2013 01:37:07 +0100
Andreas Färber wrote:
> Move x86_def_t definition to header and embed into X86CPUClass.
> Register types per built-in model definition.
>
> Move version initialization from x86_cpudef_setup() to class_init.
>
> Inline cpu_x86_register() into the X86CPU initfn.
From: Gerd Hoffmann
Don't try to be clever and skip displaysurface reinitialization in case
the size hasn't changed. Other parameters might have changed
nevertheless, for example depth or stride, resulting in rendering being
broken then.
Trigger: boot linux guest with vesafb, start X11, make su
From: Alexander Graf
Device tree properties need to be specified in big endian. Fix the
bamboo memory size property accordingly.
Signed-off-by: Alexander Graf
CC: qemu-sta...@nongnu.org
(cherry picked from commit 5232fa59b17b45c04bd24e0d38224964816bf391)
Signed-off-by: Michael Tokarev
---
hw
From: Gerd Hoffmann
Commit 59310659073d85745854f2f10c4292555c5a1c51 is incomplete,
we'll arrive in the scsi command complete callback in CSW state
and must handle that case correctly.
Signed-off-by: Gerd Hoffmann
(cherry picked from commit 54414218d78c9d043417b27bb29bd0334b4e3cb5)
Signed-off-b
Il 04/02/2013 11:36, Markus Armbruster ha scritto:
Fabio Fantoni writes:
I tried to disable "default floppy" without use -nodefaults option
that disable other things.
I didn't found other parameters to do that in docs and code for now.
Can someone tell me if there is another way to disable def
Convert glib file descriptor polling from rfds/wfds/xfds to GPollFD.
The Windows code still needs poll_fds[] and n_poll_fds but they can now
become local variables.
Signed-off-by: Stefan Hajnoczi
---
main-loop.c | 71 +++--
1 file changed,
On 02/04/2013 01:24 PM, Markus Armbruster wrote:
> Fabien Chouteau writes:
>
>> On 02/04/2013 11:34 AM, Markus Armbruster wrote:
>>
>> Why? The caller doesn't know the difference between Windows/Linux
>> implementation. And the error handling would have to be duplicated.
>
> The function's (impl
From: Jason Baron
The Advanced Error Interrupt Message Number (bits 31:27 of the Root
Error Status Register) is updated when the number of msi messages assigned to a
device changes. Migration of windows 7 on q35 chipset failed because the check
in get_pci_config_device() fails due to cmask being
101 - 200 of 277 matches
Mail list logo
