On Mo, 2015-05-11 at 15:49 +0200, Paolo Bonzini wrote:
> From: Gerd Hoffmann
[ more verbose commit message for squashing in ]
Not all bits in SMRAM and ESMRAMC can be changed by the guest.
Add wmask defines accordingly and set them in mch_reset().
> Signed-off-by: Gerd Hoffmann
> Signed-off-by
On 12 May 2015 at 00:45, Programmingkid wrote:
> On May 10, 2015, at 6:19 PM, Peter Maydell wrote:
>> I've chosen to implement some of them by simply dropping the
>> backward-compatibility support for OSX 10.4. This is basically
>> a pragmatic decision since I don't think we can support ancient
>>
On Mo, 2015-05-11 at 15:49 +0200, Paolo Bonzini wrote:
> From: Gerd Hoffmann
[ more verbose commit message for squashing in ]
The cache bits in ESMRAMC are hardcoded to 1 (=disabled) according to
the q35 mch specs. Add and use a define with this default.
While being at it also update the SMRAM
From: Miroslav Rezanina
mips build fail with link error in case PARALLEL_CONFIG is disabled as
hw/mips/mips_jazz.c calls parallel_mm_init. Due to dependecies to content
of parallel.c we can't simply move it to hw/isa/isa-devices.c.
This patch adds stubs/parallel.c file that contains stub version
From: Miroslav Rezanina
Disabling CONFIG_PARALLEL cause removing parallel_hds_isa_init defined in
parallel.c. This function is called during initialization of some boards so
disabling CONFIG_PARALLEL cause build failure.
This patch moves parallel_hds_isa_init to hw/isa/isa-bus.c so it is include
From: Miroslav Rezanina
There's few uncoditional calls to functions in hw/char/parallel.c so
disabling CONFIG_PARALLEL can cause build failure. This series allow
building with CONFIG_PARALLEL disabled. In addition, it changes behavior
so the run of "qemu -parallel" will abort in case target expec
On Mon, 05/11 15:22, John Snow wrote:
>
>
> On 05/06/2015 12:52 AM, Fam Zheng wrote:
> > Unsetting dirty globally with discard is not very correct. The discard may
> > zero
> > out sectors (depending on can_write_zeroes_with_unmap), we should replicate
> > this change to destinition side to make
Test zero write in byte range 512~1024 for 4k alignment.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
---
tests/qemu-iotests/033 | 13 +
tests/qemu-iotests/033.out | 30 ++
2 files changed, 43 insertions(+)
diff --git a/tests/qemu-iotests/03
For zero write, callers pass in NULL qiov (qemu-io "write -z" or
scsi-disk "write same").
Commit fc3959e466 fixed bdrv_co_write_zeroes which is the common case
for this bug, but it still exists in bdrv_aio_write_zeroes. A simpler
fix would be in bdrv_co_do_pwritev which is the NULL dereference poi
An unaligned zero write causes NULL deferencing in bdrv_co_do_pwritev. That
path is reachable from bdrv_co_write_zeroes and bdrv_aio_write_zeroes.
You can easily trigger through the former with qemu-io, as the test case added
by 61815d6e0aa. For bdrv_aio_write_zeroes, in common cases there's alway
This reverts commit fc3959e4669a1c2149b91ccb05101cfc7ae1fc05.
The core write code already handles the case, so remove this
duplication.
Because commit 61007b316 moved the touched code from block.c to
block/io.c, the change is manually reverted.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajno
On 02/05/15 13:13, Denis V. Lunev wrote:
Functionality match with Linux. Patches 1 and 2 are very useful for
consistent backups of Windows guests.
Signed-off-by: Olga Krishtal
Signed-off-by: Denis V. Lunev
CC: Michael Roth
ping
The following sequence
int fd = open(argv[1], O_RDWR | O_CREAT | O_DIRECT, 0644);
for (i = 0; i < 10; i++)
write(fd, buf, 4096);
performs 5% better if buf is aligned to 4096 bytes.
The difference is quite reliable.
On the other hand we do not want at the moment to enforce
I have used the following program to test
#define _GNU_SOURCE
#include
#include
#include
#include
#include
#include
int main(int argc, char *argv[])
{
int fd = open(argv[1], O_RDWR | O_CREAT | O_DIRECT, 0644);
void *buf;
int i = 0, align = atoi(argv[2]);
do {
buf =
The patch introduces new concept: minimal memory alignment for bounce
buffers. Original so called "optimal" value is actually minimal required
value for aligment. It should be used for validation that the IOVec
is properly aligned and bounce buffer is not required.
Though, from the performance poi
This patch changes the way cpu_index is handed out to newly created
CPUs by tracking the allocted CPUs in a bitmap. More information and
the need for this patch is described in patch 2/3 of this series. These
generic changes are needed to support CPU hot plug/unplug on PowerPC.
An open question is
Move cpu_exec_init() call from instance_init to realize. This allows
any failures from cpu_exec_init() to be handled appropriately.
Also add corresponding cpu_exec_exit() call from unrealize.
Signed-off-by: Bharata B Rao
Reviewed-by: David Gibson
---
target-ppc/translate_init.c | 9 +++--
1
Currently CPUState.cpu_index is monotonically increasing and a newly
created CPU always gets the next higher index. The next available
index is calculated by counting the existing number of CPUs. This is
fine as long as we only add CPUs, but there are architectures which
are starting to support CPU
Add an Error argument to cpu_exec_init() to let users collect the
error. This is in preparation to change the CPU enumeration logic
in cpu_exec_init(). With the new enumeration logic, cpu_exec_init()
can fail if cpu_index values corresponding to max_cpus have already
been handed out.
Since all cur
From: Shannon Zhao
Add a gpio-keys node. This is used for Poweroff for the systems which
use DT not ACPI.
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/arm/virt.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/hw/arm/virt.c b/hw/arm/virt.c
index 354b020.
From: Shannon Zhao
Add power button device in ACPI DSDT table.
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/arm/virt-acpi-build.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
index e505c16..b582047 100
From: Shannon Zhao
Currently mach-virt model doesn't support powerdown request. Guest VM
doesn't react to system_powerdown from monitor console (or QMP) because
there is no communication mechanism for such requests. This patch registers
GPIO Pin 3 with powerdown notification. So guest VM can rece
From: Shannon Zhao
Add GPIO controller in ACPI DSDT table. It can be used
for gpio event.
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/arm/virt-acpi-build.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-b
From: Shannon Zhao
ACPI 5.0 supports GPIO-signaled ACPI Events. This can be used for
powerdown, hotplug evnets. Add a GPIO controller in machine virt,
to support powerdown, maybe can be used for cpu hotplug. And
here we use pl061.
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw
From: Shannon Zhao
Here GPIO pin 3 is used for Power Button, add _E03 in ACPI DSDT table.
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/arm/virt-acpi-build.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
i
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/acpi/aml-build.c | 60 +
include/hw/acpi/aml-build.h | 16
2 files changed, 76 insertions(+)
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.
From: Shannon Zhao
ACPI SPEC 5.0 defines GPIO-signaled ACPI Events for Hardware-reduced
platforms(like ARM). It uses GPIO pin to trigger an event to the guest.
For QEMU, here we add PL061 GPIO controller and use PIN 3 for
system_powerdown, reserving PIN 0, 1, 2 for PCI hotplug, CPU hotplug and
me
On May 10, 2015, at 2:45 PM, Peter Maydell wrote:
> On 15 January 2015 at 21:13, Programmingkid wrote:
>> Fixes keyboard mapping so right shift, right command, right option, right
>> control, keypad period, keypad '=', keypad enter, and F13 all work.
>>
>> Signed-off-by: John Arbuckle
>>
>>
On 05/07/2015 06:12 PM, Alexander Graf wrote:
> +static ExitStatus op_laa(DisasContext *s, DisasOps *o)
> +{
> +TCGv_i64 m2 = tcg_temp_new_i64();
> +
> +/* XXX should be atomic */
> +tcg_gen_qemu_ld32s(m2, o->in2, get_mem_index(s));
> +
> +/* Set r1 to the unmodified contents of m2
On 05/07/2015 06:12 PM, Alexander Graf wrote:
> I find it really hard to grasp what each field in the opcode list means.
> Slowly walking through its semantics myself, I figured I'd write a small
> summary at the top of the file to make life easier for me and whoever
> looks at the file next.
>
>
On Mon, 05/11 15:43, Stefan Hajnoczi wrote:
> On Tue, May 05, 2015 at 10:51:14AM +0800, Fam Zheng wrote:
>
> This function is complex. I had to draw a diagram to remember the
> relationships between the variables. It would be nice to split it if
> that can be done in a way that makes the code ni
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/audio/intel-hda.c | 2 +-
hw/char/virtio-serial-bus.c| 2 +-
hw/core/platform-bus.c | 2 +-
hw/dma/i82374.c| 2 +-
hw/i386/kvm/ioapic.c | 2 +-
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/arm/spitz.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/hw/arm/spitz.c b/hw/arm/spitz.c
index 5bf032a..fdb1010 100644
--- a/hw/arm/spitz.c
+++ b/hw/arm/spitz.c
@@ -1060,10 +1060,6 @@ static VMStateDes
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/gpio/zaurus.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/hw/gpio/zaurus.c b/hw/gpio/zaurus.c
index 24a7727..aa8ee5f 100644
--- a/hw/gpio/zaurus.c
+++ b/hw/gpio/zaurus.c
@@ -235,10 +235,6 @@ static co
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/virtio/virtio-balloon.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c
index 484c3c3..a3f5709 100644
--- a/hw/virtio/virtio-balloon.c
+++ b/hw/virt
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/virtio/virtio-pci.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index 5c173c4..47be00f 100644
--- a/hw/virtio/virtio-pci.c
+++ b/hw/virtio/virtio-pci.c
From: Shannon Zhao
Signed-off-by: Shannon Zhao
Signed-off-by: Shannon Zhao
---
hw/s390x/s390-virtio-bus.c | 15 ---
1 file changed, 15 deletions(-)
diff --git a/hw/s390x/s390-virtio-bus.c b/hw/s390x/s390-virtio-bus.c
index 1a72b56..273bd9c 100644
--- a/hw/s390x/s390-virtio-bus.c
+
From: Shannon Zhao
After [1]"s390-virtio: use common features" and [2]"virtio: Move host
features to backends" applied, there are some Property arrays which only
contain the list terminator DEFINE_PROP_END_OF_LIST. This patchset
removes those arrays and other meaningless Property arrays and also
Hi,
I'm still trying to understanding how disas_insn in
*target-i386/translate.c* works.
Currently I met "*case 0x140 ... 0x14f: /* cmov Gv, Ev */*",
I thought it just check the condition and execute the MOV according to the
condition.
However, in *gen_cmovcc1 *function, I have no idea where th
> -Original Message-
> From: xen-devel-boun...@lists.xen.org
> [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Stefano Stabellini
> Sent: Friday, May 08, 2015 1:26 AM
> To: Xu, Quan
> Cc: wei.l...@citrix.com; stef...@linux.vnet.ibm.com;
> stefano.stabell...@eu.citrix.com; qemu-devel
On Thu, May 7, 2015 at 2:34 AM, Michael Tokarev wrote:
> 07.05.2015 09:47, Michael Tokarev wrote:
>> 07.05.2015 09:12, Michael Tokarev wrote:
>>> 07.05.2015 04:11, G 3 wrote:
Did you boot Windows XP to the desktop? I have tested Windows 95, Windows
2000, and Windows XP. All of them fail
On Tue, 05/12 00:24, Andreas Färber wrote:
> Am 07.05.2015 um 08:55 schrieb Fam Zheng:
> > Because of the trick of process-archive-undefs, all .mo objects, even
> > with --enable-modules, are dependencies of executables.
> >
> > This breaks CFLAGS propogation because the compiling of module object
For me, I still stick to uint8_t, since all callers and callee always treat it
as uint8_t. It will make the code more clearer for readers.
> Date: Mon, 11 May 2015 15:06:48 -0700
> From: r...@twiddle.net
> To: xili_gchen_5...@hotmail.com; peter.mayd...@linaro.org; afaer...@suse.de;
> cmetc...@ez
On Mon, May 11, 2015 at 4:40 PM, Programmingkid
wrote:
>
> On May 11, 2015, at 7:11 PM, Peter Crosthwaite wrote:
>
>> On Mon, May 11, 2015 at 3:08 PM, Programmingkid
>> wrote:
>>> This patch series adds a machine menu to the Macintosh interface.
>>> Patch 1/3 adds the machine menu with pause and
On May 10, 2015, at 6:19 PM, Peter Maydell wrote:
> This patchset fixes a number of new compile warnings when building
> on OSX10.10 which were not present on 10.9, which are mostly fixes
> to avoid deprecated APIs.
>
> I've chosen to implement some of them by simply dropping the
> backward-comp
On May 11, 2015, at 7:11 PM, Peter Crosthwaite wrote:
> On Mon, May 11, 2015 at 3:08 PM, Programmingkid
> wrote:
>> This patch series adds a machine menu to the Macintosh interface.
>> Patch 1/3 adds the machine menu with pause and resume menu items.
>> Patch 2/3 adds menu items for all devices
** No longer affects: qemu (Ubuntu)
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1450881
Title:
qemu-system-sparc MUTEX_HELD assert and libC lock errors
Status in QEMU:
New
Bug description:
On Mon, May 11, 2015 at 3:08 PM, Programmingkid
wrote:
> This patch series adds a machine menu to the Macintosh interface.
> Patch 1/3 adds the machine menu with pause and resume menu items.
> Patch 2/3 adds menu items for all devices that are available.
> Patch 3/3 adds Reset and Power Down menu
Use a transaction to request an incremental backup across two drives.
Coerce one of the jobs to fail, and then re-run the transaction.
Verify that no bitmap data was lost due to the partial transaction
failure.
Signed-off-by: John Snow
Reviewed-by: Max Reitz
---
tests/qemu-iotests/124 | 12
The goal here is to add a new method to transactions that allows
developers to specify a callback that will get invoked only once
all jobs spawned by a transaction are completed, allowing developers
the chance to perform actions conditionally pending complete success,
partial failure, or complete f
From: Kashyap Chamarthy
Although the canonical source of reference for QMP commands is
qapi-schema.json, for consistency's sake, update qmp-commands.hx to
state the list of supported transactionable operations, namely:
drive-backup
blockdev-backup
blockdev-snapshot-internal-sync
This patch actually implements the transactional callback system
for the drive_backup action.
(1) We manually pick up a reference to the bitmap if present to allow
its cleanup to be delayed until after all drive_backup jobs launched
by the transaction have fully completed.
(2) We create a
If we want to get at the job after the life of the job,
we'll need a refcount for this object.
This may occur for example if we wish to inspect the actions
taken by a particular job after a transactional group of jobs
runs, and further actions are required.
Signed-off-by: John Snow
Reviewed-by:
We'd like to be able to specify the callback given to backup_start
manually in the case of transactions, so split apart qmp_drive_backup
into an implementation and a wrapper.
Switch drive_backup_prepare to use the new wrapper, but don't overload
the callback and closure yet.
Signed-off-by: John S
These structures are misnomers, somewhat.
(1) BlockTransactionState is not state for a transaction,
but is rather state for a single transaction action.
Rename it "BlkActionState" to be more accurate.
(2) The BdrvActionOps describes operations for the BlkActionState,
above. This name
Patch 1 adds basic support for add and clear transactions.
Patch 2 tests this basic support.
Patches 3-4 refactor transactions a little bit, to add clarity.
Patch 5 adds the framework for error scenarios where only
some jobs that were launched by a transaction complete successfully,
and we
Allow bitmap successors to carry reference counts.
We can in a later patch use this ability to clean up the dirty bitmap
according to both the individual job's success and the success of all
jobs in the transaction group.
The code for cleaning up a bitmap is also moved from backup_run to
backup_c
This adds two qmp commands to transactions.
block-dirty-bitmap-add allows you to create a bitmap simultaneously
alongside a new full backup to accomplish a clean synchronization
point.
block-dirty-bitmap-clear allows you to reset a bitmap back to as-if
it were new, which can also be used alongsid
Test simple usage cases for using transactions to create
and synchronize incremental backups.
Signed-off-by: John Snow
Reviewed-by: Max Reitz
Reviewed-by: Stefan Hajnoczi
---
tests/qemu-iotests/124 | 54 ++
tests/qemu-iotests/124.out | 4 ++--
2
Now that the structure formerly known as BlkTransactionState has been
renamed to something sensible (BlkActionState), re-introduce an actual
BlkTransactionState that actually manages state for the entire Transaction.
In the process, convert the old QSIMPLEQ list of actions into a QTAILQ,
to let us
On May 10, 2015, at 10:54 AM, Paolo Bonzini wrote:
>
>
> On 06/05/2015 18:40, Programmingkid wrote:
>> When I try to use the pcspk sound hardware, QEMU freezes and uses
>> 100% of the cpu time. This is the command I use:
>>
>> qemu-system-i386 -cdrom -soundhw pcspk
>>
>> This looks like a de
Am 07.05.2015 um 08:55 schrieb Fam Zheng:
> Because of the trick of process-archive-undefs, all .mo objects, even
> with --enable-modules, are dependencies of executables.
>
> This breaks CFLAGS propogation because the compiling of module object
> will happen too early before building for DSO.
>
Adds Reset and Power Down menu items to the Machine menu.
Signed-off-by: John Arbuckle
---
ui/cocoa.m | 24
1 files changed, 24 insertions(+), 0 deletions(-)
diff --git a/ui/cocoa.m b/ui/cocoa.m
index 5e558ea..2c4a61a 100644
--- a/ui/cocoa.m
+++ b/ui/cocoa.m
@@ -828
Adds all removable devices to the Machine menu as a Change and Eject menu item
pair.
ide-cd0 would have a "Change ide-cd0..." and "Eject ide-cd0" menu items.
Signed-off-by: John Arbuckle
---
ui/cocoa.m | 113 ++--
1 files changed, 110
Adds a Machine menu to the Macintosh interface that pause and resume menu items.
These items can either pause or resume execution of the guest operating system.
Signed-off-by: John Arbuckle
---
ui/cocoa.m | 79
1 files changed, 79 i
This patch series adds a machine menu to the Macintosh interface.
Patch 1/3 adds the machine menu with pause and resume menu items.
Patch 2/3 adds menu items for all devices that are available.
Patch 3/3 adds Reset and Power Down menu items.
On 05/11/2015 02:06 PM, Chen Gang wrote:
> On 5/12/15 00:01, Richard Henderson wrote:
>> On 05/10/2015 03:42 PM, Chen Gang wrote:
>>> -static __inline unsigned int
>>> +static inline uint8_t
>>> get_BFEnd_X0(tilegx_bundle_bits num)
>>
>> Do not change these casts to uint8_t. It's unnecessary chur
On May 7, 2015, at 5:34 AM, Michael Tokarev wrote:
> 07.05.2015 09:47, Michael Tokarev wrote:
>> 07.05.2015 09:12, Michael Tokarev wrote:
>>> 07.05.2015 04:11, G 3 wrote:
Did you boot Windows XP to the desktop? I have tested Windows 95, Windows
2000, and Windows XP. All of them fail to
Firstly, thank you very much for your response quickly!
On 5/12/15 00:55, Richard Henderson wrote:
> On 05/10/2015 03:45 PM, Chen Gang wrote:
>> +static void gen_cmpltsi(struct DisasContext *dc,
>> +uint8_t rdst, uint8_t rsrc, int8_t imm8)
>> +{
>> +qemu_log_mask(CPU_L
On 5/12/15 00:01, Richard Henderson wrote:
> On 05/10/2015 03:42 PM, Chen Gang wrote:
>> -static __inline unsigned int
>> +static inline uint8_t
>> get_BFEnd_X0(tilegx_bundle_bits num)
>
> Do not change these casts to uint8_t. It's unnecessary churn.
>
For me, it is enough to return uint8_t, a
The compat property was added by commit
9dbcca5aa13cb9ab40788ac4c56bc227d94ca920, and the pc-0.12 and older
machine-types were not changed because virtio-9p-pci was introduced on QEMU
0.13 (commit 9f10751365b26b13b8a9b67e0e90536ae3d282df). The only problem is
that this breaks the PC_COMPAT_* nestin
On 08.05.15 02:01, David Gibson wrote:
> On Fri, May 08, 2015 at 12:39:20AM +0200, Alexander Graf wrote:
>>
>>
>> On 07.05.15 07:33, David Gibson wrote:
>>> From: Michael Roth
>>>
>>> This device emulates a firmware abstraction used by pSeries guests to
>>> manage hotplug/dynamic-reconfiguration
The VGA and vmware-svga rombar compat properties were added by commit
281a26b15b4adcecb8604216738975abd754bea8, but only to pc-0.13 and
pc-0.12. This breaks the PC_COMPAT_* nesting pattern we currently
follow.
The new variables will now be inherited by pc-0.11 and older, but
pc-0.11 and pc-0.10 al
Move compat_props from pc-0.10 to the macro, to make it consistent with
the other machines.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_piix.c | 47 +--
1 file changed, 25 insertions(+), 22 deletions(-)
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_pi
Those properties were introduced by commit
3827cdb1c3aa17a792d1658161195b9d7173c26b. They were not duplicated into
pc-0.13 and older because 0.14 was the first QEMU version supporting
qxl. The only problem is that this breaks the PC_COMPAT_* nesting
pattern we currently use.
So, move the propertie
SPAPR_COMPAT_2_1 will need to include both HW_COMPAT_2_2 and
HW_COMPAT_2_1, so include HW_COMPAT_2_1 inside SPAPR_COMPAT_2_1 and
HW_COMPAT_2_2 inside SPAPR_COMPAT_2_2.
Signed-off-by: Eduardo Habkost
---
hw/ppc/spapr.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/pp
Don't add the pseries-2.3 machine yet, but define the corresponding
SPAPR_COMPAT macro to make sure both pseries-2.2 and pseries-2.1 will
inherit HW_COMPAT_2_3.
Signed-off-by: Eduardo Habkost
---
hw/ppc/spapr.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr
Now we can make everything consistent and define the macros even if they
are still empty.
Signed-off-by: Eduardo Habkost
---
include/hw/compat.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/hw/compat.h b/include/hw/compat.h
index c56a698..4a43466 100644
--- a/include/hw/comp
The current code setting ide-drive.ver and scsi-disk.ver on pc-0.11
breaks the PC_COMPAT_* nesting pattern we currently use.
As those variables are overwritten in pc-0.10 too, they can be inherited
by pc-0.10 with no side-effects at all.
Cc: Gerd Hoffmann
Signed-off-by: Eduardo Habkost
---
hw/
Changing the convention to include commas inside the macros will allow
macros containing empty lists to be defined and used without compilation
errors.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_piix.c| 62 ++--
hw/i386/pc_q35.c | 10 +++
Coding style change only.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_piix.c| 2 +-
include/hw/i386/pc.h | 12 ++--
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
index 1fe7bfb..b4ec3ce 100644
--- a/hw/i386/pc_piix.c
+++ b/hw/i
Once we start adding compat code for pc-2.3, the usage of HW_COMPAT_2_1
in pc-*-2.2 won't be enough, as it also has to include PC_COMPAT_2_3
inside it. To ensure that, define PC_COMPAT_2_3, PC_COMPAT_2_2, and
PC_COMPAT_2_1 macros.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_piix.c| 10
This series does some changes to make the macros more consistent. Now every pc
and pseries version has a corresponding compat macro defined.
Eduardo Habkost (13):
pc: Replace tab with spaces
hw: Move commas inside HW_COMPAT_2_1 macro
pc: Move commas inside PC_COMPAT_* macros
spapr: Move co
Changing the convention to include commas inside the macros will allow
macros containing empty lists to be defined and used without compilation
errors.
Signed-off-by: Eduardo Habkost
---
hw/ppc/spapr.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/ppc/spapr.c b/hw/
Changing the convention to include commas inside the macros will allow
macros containing empty lists to be defined and used without compilation
errors.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_piix.c| 2 +-
hw/i386/pc_q35.c | 2 +-
hw/ppc/spapr.c | 2 +-
include/hw/compat.h |
On 05/06/2015 12:52 AM, Fam Zheng wrote:
> Only poll the specific type of event we are interested in, to avoid
> stealing events that should be consumed by someone else.
>
> Suggested-by: John Snow
> Signed-off-by: Fam Zheng
> ---
> tests/qemu-iotests/iotests.py | 9 ++---
> 1 file change
On 08.05.15 08:24, Gerd Hoffmann wrote:
> On Fr, 2015-05-08 at 00:28 +0200, Alexander Graf wrote:
>>
>> On 05.05.15 11:43, Gerd Hoffmann wrote:
>>> Signed-off-by: Gerd Hoffmann
>>> Reviewed-by: Max Reitz
>>> ---
>>
>> [...]
>>
>>> +void surface_gl_create_texture(ConsoleGLState *gls,
>>> +
On 05/11/2015 03:24 AM, Paolo Bonzini wrote:
>
>
> On 11/05/2015 12:18, Andreas Färber wrote:
+int (*cpu_mmu_index)(CPUState *cpu);
+void (*cpu_get_tb_cpu_state)(CPUState *cpu,
+ void *pc, /* target_long * */
+
On Mon, May 11, 2015 at 09:26:08PM +0200, Michael S. Tsirkin wrote:
> On Mon, May 11, 2015 at 11:56:44AM -0700, Sean O. Stalley wrote:
> > PCI Enhanced Allocation is a new method of allocating MMIO & IO
> > resources for PCI devices & bridges. It can be used instead
> > of the traditional PCI metho
On 05/06/2015 12:52 AM, Fam Zheng wrote:
> This checks that the discard on mirror source that effectively zeroes
> data is also reflected by the data of target.
>
> Signed-off-by: Fam Zheng
> ---
> tests/qemu-iotests/131 | 59
> ++
> tests/qemu-
On 05/11/2015 02:42 PM, Paolo Bonzini wrote:
>
> On 11/05/2015 19:26, Andreas Färber wrote:
>> Another problem is that you're using object_new() in realize at all,
>> which means that it's too late for any management interface to tweak
>> properties on the new device. One possible solution would be
On 05/06/2015 12:52 AM, Fam Zheng wrote:
> Signed-off-by: Fam Zheng
> ---
> tests/qemu-iotests/041| 66
> ++-
> tests/qemu-iotests/iotests.py | 28 ++
> 2 files changed, 43 insertions(+), 51 deletions(-)
>
> diff --git a/tests/qe
On 11/05/2015 19:26, Andreas Färber wrote:
> Another problem is that you're using object_new() in realize at all,
> which means that it's too late for any management interface to tweak
> properties on the new device. One possible solution would be to create
> the object in a property setter, befo
On Mon, May 11, 2015 at 11:56:44AM -0700, Sean O. Stalley wrote:
> PCI Enhanced Allocation is a new method of allocating MMIO & IO
> resources for PCI devices & bridges. It can be used instead
> of the traditional PCI method of using BARs.
>
> EA entries are hardware-initialized to a fixed address
On 05/06/2015 12:52 AM, Fam Zheng wrote:
> Unsetting dirty globally with discard is not very correct. The discard may
> zero
> out sectors (depending on can_write_zeroes_with_unmap), we should replicate
> this change to destinition side to make sure that the guest sees the same
> data.
>
> Cal
On 05/06/2015 12:52 AM, Fam Zheng wrote:
> Using this function would always be wrong because a dirty bitmap must
> have a specific owner that consumes the dirty bits and calls
> bdrv_reset_dirty_bitmap().
>
Good point.
> Remove the unused function to avoid future misuse.
>
> Reviewed-by: Eric
PCI Enhanced Allocation is a new method of allocating MMIO & IO
resources for PCI devices & bridges. It can be used instead
of the traditional PCI method of using BARs.
EA entries are hardware-initialized to a fixed address.
Unlike BARs, regions described by EA are cannot be moved.
Because of this
For larger pieces of data that won't need to be debugged and
viewing the hex nibbles is unlikely to be useful, we can encode
data using base64 instead of encoding each byte as %02x, which
leads to some space savings and faster reads/writes.
For now, the default is left as hex nibbles in memwrite()
Instead of converting each byte one-at-a-time and then sending each byte
over the wire, use sprintf() to pre-compute all of the hex nibs into a
single buffer, then send the entire buffer all at once.
This gives a moderate speed boost to memread() and memwrite() functions.
Signed-off-by: John Snow
Previously, memset was just a frontend to write() and only
stupidly sent the pattern many times across the wire.
Let's not discuss who stupidly wrote it like that in the first place.
(Hint: It was me.)
Signed-off-by: John Snow
Message-id: 1430864578-22072-4-git-send-email-js...@redhat.com
---
q
1 - 100 of 390 matches
Mail list logo