On 20/02/2015 09:02, Pavel Dovgaluk wrote:
Is it possible to call this together with configure_icount (or even from
configure_icount)?
I think that it is the best place for replay_configure.
It sets replay_mode which may be checked while configuring the virtual
machine.
That's why
On Fri, Feb 13, 2015 at 05:08:56PM -0500, John Snow wrote:
+def check_incremental(self, target=None):
+if target is None:
+target = self.bitmaps[-1].last_target()
+self.assertTrue(iotests.compare_images(self.test_img, target))
+return True
+
+
+
On Fri, Feb 13, 2015 at 05:08:46PM -0500, John Snow wrote:
We add a bitmap merge operation to assist in error cases
where we wish to combine two bitmaps together.
This is algorithmically O(bits) provided HBITMAP_LEVELS remains
constant. For a full bitmap on a 64bit machine:
sum(bits/64^k,
On Fri, Feb 13, 2015 at 05:08:41PM -0500, John Snow wrote:
This series requires: [PATCH v3] blkdebug: fix once rule
Welcome to the incremental backup newsletter, where we discuss
exciting developments in non-redundant backup technology.
This issue is called the Max Reitz Fixup issue.
This
On Fri, Feb 20, 2015 at 11:00 AM, Laurent Desnogues
laurent.desnog...@gmail.com wrote:
On Thu, Feb 19, 2015 at 10:14 PM, Richard Henderson r...@twiddle.net wrote:
While doing the mechanics of a previous patch set converting
translators to use to TCGLabel pointers, I was reminded of
several
This patchset allows vhost_net to be used with legacy virtio when guest and
host have different endianness. It depends on the following kernel patchset:
https://lists.linuxfoundation.org/pipermail/virtualization/2015-February/029347.html
---
Cédric Le Goater (1):
vhost_net: re-enable when
When the guest and the host have a different endian order, the data
being accessed in the vring queues needs to be byteswapped if we run
in legacy mode.
This patch adds a flag to inform the vhost kernel backend about the
endianness to use.
This patch is based on previous work by Cédric Le
From: Cédric Le Goater c...@fr.ibm.com
revert 371df9f5e0f1 vhost-net: disable when cross-endian
Signed-off-by: Cédric Le Goater c...@fr.ibm.com
Signed-off-by: Greg Kurz gk...@linux.vnet.ibm.com
---
hw/net/vhost_net.c | 19 ---
1 file changed, 19 deletions(-)
diff --git
A build from the current master attached in gdb reveals
Program received signal SIGSEGV, Segmentation fault.
sdl_switch (dcl=0x7f4db26e4b20, new_surface=new_surface@entry=0x0) at
ui/sdl.c:128
128 PixelFormat pf = qemu_pixelformat_from_pixman(new_surface-format);
(gdb) bt
#0 sdl_switch
* Markus Armbruster (arm...@redhat.com) wrote:
I'd like Eric's opinion on on encoding configuration tuples as URIs
rather than JSON in QMP.
Old discussion; he's already R-b the previous version of this
patch where the only difference was pause/defer.
Dave
Dr. David Alan Gilbert (git)
On Thu, Feb 19, 2015 at 10:14 PM, Richard Henderson r...@twiddle.net wrote:
While doing the mechanics of a previous patch set converting
translators to use to TCGLabel pointers, I was reminded of
several outstanding OPTME comments in the aarch64 translator.
I had started with the csel change,
From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo
Bonzini
On 18/02/2015 12:57, Pavel Dovgalyuk wrote:
@@ -2751,6 +2757,7 @@ int main(int argc, char **argv, char **envp)
{
int i;
int snapshot, linux_boot;
+const char *icount_option = NULL;
* Markus Armbruster (arm...@redhat.com) wrote:
Dr. David Alan Gilbert (git) dgilb...@redhat.com writes:
From: Dr. David Alan Gilbert dgilb...@redhat.com
-incoming defer causes qemu to wait for an incoming migration
to be specified later. The monitor can be used to set migration
Dr. David Alan Gilbert (git) dgilb...@redhat.com writes:
From: Dr. David Alan Gilbert dgilb...@redhat.com
Document the various URI formats for -incoming, the previous
manpage and help text was wrong (out of date?)
Thanks a lot for updating the docs.
Signed-off-by: Dr. David Alan Gilbert
Actually in any version this can never work, as you call
sdl_switch(dcl,NULL);
in ui/sdl.c:552. So the dereferncing statement
new_surface-format
must SEGFAULT.
The obvious patch is very simple, of course, as just the statement below
line 128 asks if(new_surface). So pf should be
I'd like Eric's opinion on on encoding configuration tuples as URIs
rather than JSON in QMP.
Dr. David Alan Gilbert (git) dgilb...@redhat.com writes:
From: Dr. David Alan Gilbert dgilb...@redhat.com
Add migrate_incoming/migrate-incoming to start an incoming
migration.
Once a qemu has been
From what I can tell in the code, padstr() is only coded in the IDE section,
not SCSI/SATA?
On Feb 19, 2015, at 11:42 PM, Markus Armbruster arm...@redhat.com wrote:
John Snow js...@redhat.com mailto:js...@redhat.com writes:
On 02/19/2015 02:48 PM, Alan Latteri wrote:
I am virtualzing a
Yes, left pad. This is a strace of the authorization program via VirtualBox
data[96]=[\0\200\0\24ABCD1234\0\0\0\0\0\0\0\0…]
then the same thing using the serial setting no spaces in QEMU.
data[96]=[\0\200\0\024ABCD1234\0\0\0\0\0\0\0\0…]
The spaces matter here.
I tried
Alan Latteri a...@instinctual.tv writes:
Yes, left pad. This is a strace of the authorization program via VirtualBox
data[96]=[\0\200\0\24ABCD1234\0\0\0\0\0\0\0\0…]
then the same thing using the serial setting no spaces in QEMU.
data[96]=[\0\200\0\024ABCD1234
Dr. David Alan Gilbert dgilb...@redhat.com writes:
* Markus Armbruster (arm...@redhat.com) wrote:
Dr. David Alan Gilbert (git) dgilb...@redhat.com writes:
From: Dr. David Alan Gilbert dgilb...@redhat.com
-incoming defer causes qemu to wait for an incoming migration
to be specified
This will be used to size the TLB when more than 8 MMU modes are
used by the target. Limitations come from the limited size of
the immediate fields (which sometimes, as in the case of Aarch64,
extend to instructions that shift the immediate).
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
Dr. David Alan Gilbert dgilb...@redhat.com wrote:
* Markus Armbruster (arm...@redhat.com) wrote:
I'd like Eric's opinion on on encoding configuration tuples as URIs
rather than JSON in QMP.
Old discussion; he's already R-b the previous version of this
patch where the only difference was
Create VMStateDescription for MIPS CPU. The new structure contains exactly the
same fields as before, therefore leaving existing version_id.
Signed-off-by: Leon Alrae leon.al...@imgtec.com
---
target-mips/cpu-qom.h | 4 +
target-mips/cpu.c | 1 +
target-mips/cpu.h | 2 -
Hi,
This small patchset converts MIPS CPU to use VMStateDescription:
[1/2]: ports all the MIPS CPU fields to new VMStateDescription structure
[2/2]: adds MSACSR and uses post_load() to restore *fp_status from *CSR
registers. These changes include ported Maciej's updates made for old
Save MSACSR state. Also remove fp_status, msa_fp_status, hflags and restore
them in post_load() from the architectural registers.
Float exception flags are not present in vmstate. Information they carry
is used only by softfloat caller who translates them into MIPS FCSR.Cause,
FCSR.Flags and then
On 20.02.15 07:20, Richard Henderson wrote:
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-ppc/translate.c | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
---
Note that this is relative to an outstanding TCG patch set.
The full tree is
On 20/02/2015 13:36, Paolo Bonzini wrote:
On 20/02/2015 13:18, Peter Maydell wrote:
Why not just use the sips --out option to specify a different
output file? That way we automatically put the current icon
into the executable, and don't have to update a hand-created
qemu.rsrc file in git
Am 20.02.2015 um 14:27 schrieb Peter Lieven:
On 2015-02-19 at 10:47, Kevin Wolf wrote:
Am 18.02.2015 um 18:18 hat Max Reitz geschrieben:
[snip]
Actually, now that I'm looking at is_allocated_sectors_min(), if the
first sectors is not allocated, it will return false and thus both
the
On 19 February 2015 at 18:56, Paolo Bonzini pbonz...@redhat.com wrote:
On 18/02/2015 22:09, Programmingkid wrote:
+ # Take an image and make the image its own icon:
+ sips -i ../pc-bios/qemu-nsis.ico
+ # Extract the icon to its own resource file:
+ DeRez -only icns
PowerPC TCG flushes the TLB on every IR/DR change, which basically
means on every user-kernel context switch. Encode IR/DR in the
MMU index.
This brings the number of TLB flushes down from ~90 to ~5
for starting up the Debian installer, which is in line with x86
and gives a ~10%
At 8k per TLB (for 64-bit host or target), 8 or more modes
make the TLBs bigger than 64k, and some RISC TCG backends do
not like that. On the affected hosts, cut the TLB size in
half---there is still a measurable speedup on PPC with the
next patch.
Signed-off-by: Paolo Bonzini
Patches 1 and 2 enable support from more than 8 MMU modes in TCG (patch
1 is in the targets, patch 2 is in cpu-defs.h). The TLB size is reduced
proportionally on targets where that is necessary.
Patch 3 uses the new support in the PPC target.
Paolo
Paolo Bonzini (3):
tcg: add
On Thu, 19 Feb 2015, Leon Alrae wrote:
Surely these fp_status fields are simply implementation of the architectural
CSR registers?
IMO you shouldn't store things related to TCG state, but always how the
architecture represents it. That way you're free to change the TCG
On Thu, 19 Feb 2015, Leon Alrae wrote:
I think this deserves a better description as it is about the specific
case of an unaligned standard MIPS instruction fetch. Address Error
exceptions can also happen for other reasons: unaligned data accesses or
any accesses outside memory
On 20/02/2015 13:18, Peter Maydell wrote:
Why not just use the sips --out option to specify a different
output file? That way we automatically put the current icon
into the executable, and don't have to update a hand-created
qemu.rsrc file in git if we change the icon in future (and I
bet
On 20.02.15 13:45, Paolo Bonzini wrote:
PowerPC TCG flushes the TLB on every IR/DR change, which basically
means on every user-kernel context switch. Encode IR/DR in the
MMU index.
This brings the number of TLB flushes down from ~90 to ~5
for starting up the Debian installer,
On 05.02.15 10:34, Markus Armbruster wrote:
We call try_create_xics() to create a xics-kvm. If it fails, we
call it again to fall back to plain xics.
try_create_xics() uses qdev_init(). qdev_init()'s error handling has
an unwanted side effect: it calls qerror_report_err(), which prints
On 17.02.15 15:24, Michael Mueller wrote:
This patch implements the static part of the s390 cpu class definitions.
It defines s390 cpu models by means of virtual cpu ids (enum) which contain
information on the cpu generation, the machine class, the GA number and
the machine type. The cpu id
This adds the 'top' parameter to the 'block-stream' QMP command and
checks that its value is valid before passing it to stream_start().
Signed-off-by: Alberto Garcia be...@igalia.com
---
blockdev.c| 19 +++
hmp.c | 2 +-
Hi Richard,
On Thu, Feb 19, 2015 at 10:14 PM, Richard Henderson r...@twiddle.net wrote:
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-arm/translate-a64.c | 50
+++---
1 file changed, 29 insertions(+), 21 deletions(-)
diff --git
On 17.02.15 15:24, Michael Mueller wrote:
This patch implements the static part of the s390 cpu class definitions.
It defines s390 cpu models by means of virtual cpu ids (enum) which contain
information on the cpu generation, the machine class, the GA number and
the machine type. The cpu id
man gcc:
Warn about logical not used on the left hand side operand of a
comparison. This option does not warn if the RHS operand is of a
boolean type.
By preferring bool over int where sensible, but without modifying any
depending code, make GCC happy in cases like this,
qemu-img.c: In
On 2015-02-20 at 05:02, Stefan Hajnoczi wrote:
On Fri, Feb 13, 2015 at 05:08:56PM -0500, John Snow wrote:
+def check_incremental(self, target=None):
+if target is None:
+target = self.bitmaps[-1].last_target()
+
Am 19.02.2015 um 13:07 schrieb Kevin Wolf:
Am 18.02.2015 um 22:03 hat Peter Lieven geschrieben:
Am 18.02.2015 um 21:57 schrieb Peter Lieven:
This implements bdrv_co_get_block_status() for VHD images. This can
significantly speed up qemu-img convert operation because only with this
function
Signed-off-by: Alberto Garcia be...@igalia.com
---
docs/live-block-ops.txt | 28 +++-
1 file changed, 19 insertions(+), 9 deletions(-)
diff --git a/docs/live-block-ops.txt b/docs/live-block-ops.txt
index a257087..63530e8 100644
--- a/docs/live-block-ops.txt
+++
Hello,
I added support to the Block Stream API for streaming to an
intermediate layer.
I followed the proposed API from the wiki, which simply adds an
additional 'top' parameter to block-stream specifying the image that
data is written to:
This adds a new 'top' parameter to stream_start(), that specifies the
block device where the data will be written. The image is changed to
read-write mode during the streaming and back to read-only afterwards.
This also unblocks the stream operation in backing files.
Signed-off-by: Alberto
On 17.02.15 15:24, Michael Mueller wrote:
The patch implements routines to set and retrieve processor configuration
data and to retrieve machine configuration data. The machine related data
is used together with the cpu model facility lists to determine the list of
supported cpu models of
On 17.02.15 15:24, Michael Mueller wrote:
This patch implements the functions:
- s390_get_proceccor_props()
- s390_set_proceccor_props()
They can be used to request or retrieve processor related information from an
accelerator.
That information comprises the cpu identifier, the ICB
On 10.02.15 05:36, David Gibson wrote:
This is a reworked version of BenH's patch to set VGA framebuffer
endianess based on the endianness of the guest (as determined by the
H_SET_MODE hypercall).
I've split the patch into two parts the first adding the object
property to the VGA device
GCC is more strict now.
Radim Krčmář (2):
fix GCC 5.0.0 logical-not-parentheses warnings
milkymist-pfpu: fix GCC 5.0.0 aggressive-loop-optimizations warning
hw/misc/milkymist-pfpu.c | 18 --
hw/net/virtio-net.c | 2 +-
kvm-all.c| 2 +-
qemu-img.c
man gcc:
Warn if in a loop with constant number of iterations the compiler
detects undefined behavior in some statement during one or more of
the iterations.
Refactored the code a bit to avoid the GCC warning, in an objectionable
way,
hw/misc/milkymist-pfpu.c: In function ‘pfpu_write’:
2015-02-20 15:18+0100, Radim Krčmář:
diff --git a/hw/misc/milkymist-pfpu.c b/hw/misc/milkymist-pfpu.c
@@ -164,6 +164,13 @@ output_queue_advance(MilkymistPFPUState *s)
static int pfpu_decode_insn(MilkymistPFPUState *s)
{
uint32_t pc = s-regs[R_PC];
+
+if (pc MICROCODE_WORDS) {
On 16.02.15 14:29, Marcel Apfelbaum wrote:
On 02/16/2015 03:00 PM, Alexander Graf wrote:
On 16.02.15 10:54, Marcel Apfelbaum wrote:
From: Marcel Apfelbaum marce...@redhat.com
Instead of assuming it has only one bus, it
enumerates all the host bridges until it finds
the one with bus
On 20/02/2015 15:18, Radim Krčmář wrote:
man gcc:
Warn if in a loop with constant number of iterations the compiler
detects undefined behavior in some statement during one or more of
the iterations.
Refactored the code a bit to avoid the GCC warning, in an objectionable
way,
On 20/02/2015 15:18, Radim Krčmář wrote:
man gcc:
Warn about logical not used on the left hand side operand of a
comparison. This option does not warn if the RHS operand is of a
boolean type.
By preferring bool over int where sensible, but without modifying any
depending code,
On 20 February 2015 at 23:18, Radim Krčmář rkrc...@redhat.com wrote:
man gcc:
Warn if in a loop with constant number of iterations the compiler
detects undefined behavior in some statement during one or more of
the iterations.
Refactored the code a bit to avoid the GCC warning, in an
Am 2015-02-20 15:36, schrieb Paolo Bonzini:
On 20/02/2015 15:18, Radim Krčmář wrote:
man gcc:
Warn if in a loop with constant number of iterations the compiler
detects undefined behavior in some statement during one or more of
the iterations.
Refactored the code a bit to avoid the GCC
On 20/02/2015 15:52, Michael Walle wrote:
-i = 0;
-while (pfpu_decode_insn(s)) {
-/* decode at most MICROCODE_WORDS instructions */
-if (i++ = MICROCODE_WORDS) {
Isn't the fix just to say ++i instead of i++?
In the first run,
On Fri, 20 Feb 2015 14:54:23 +0100
Alexander Graf ag...@suse.de wrote:
+/* machine related properties */
+typedef struct S390CPUMachineProps {
+uint16_t class; /* machine class */
+uint16_t ga; /* availability number of machine */
+uint16_t order; /*
Signed-off-by: John Snow js...@redhat.com
---
docs/bitmaps.md | 253
1 file changed, 253 insertions(+)
create mode 100644 docs/bitmaps.md
diff --git a/docs/bitmaps.md b/docs/bitmaps.md
new file mode 100644
index 000..7cda146
---
On Feb 20, 2015, at 12:56 PM, Paolo Bonzini wrote:
On 20/02/2015 18:32, Programmingkid wrote:
Ok, so I'll apply v3 to my tree as soon as I get a Tested-by.
Please take a look into providing a .rsrc file with larger-sized
icons (I think you can add more than one to a single .rsrc file?)
We often don't need the BlockDriverState for functions
that operate on bitmaps. Remove it.
Signed-off-by: John Snow js...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
---
block.c | 13 ++---
block/backup.c| 2 +-
block/mirror.c| 26
For dirty-bitmap sync mode, the block job will iterate through the
given dirty bitmap to decide if a sector needs backup (backup all the
dirty clusters and skip clean ones), just as allocation conditions of
top sync mode.
Signed-off-by: Fam Zheng f...@redhat.com
Signed-off-by: John Snow
This returns the granularity (in bytes) of dirty bitmap,
which matches the QMP interface and the existing query
interface.
Signed-off-by: John Snow js...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
---
block.c | 8 ++--
include/block/block.h | 1 +
2 files changed, 7
Add bdrv_clear_dirty_bitmap and a matching QMP command,
qmp_block_dirty_bitmap_clear that enables a user to reset
the bitmap attached to a drive.
This allows us to reset a bitmap in the event of a full
drive backup.
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: John Snow
Signed-off-by: John Snow js...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
---
tests/qemu-iotests/112 | 89 ++
tests/qemu-iotests/112.out | 5 +++
tests/qemu-iotests/group | 1 +
3 files changed, 95 insertions(+)
create mode 100644
To test the failure case, we modify iotests.py to allow us to specify
that we'd like to allow failures when we wait for block job events.
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
tests/qemu-iotests/112| 57
A bitmap successor is an anonymous BdrvDirtyBitmap that is intended to
be created just prior to a sensitive operation (e.g. Incremental Backup)
that can either succeed or fail, but during the course of which we still
want a bitmap tracking writes.
On creating a successor, we freeze the parent
We treat this field with a variety of different types everywhere
in the code. Now it's just uint32_t.
Reviewed-by: Eric Blake ebl...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
block.c | 11 ++-
block/mirror.c
We add a bitmap merge operation to assist in error cases
where we wish to combine two bitmaps together.
This is algorithmically O(bits) provided HBITMAP_LEVELS remains
constant. For a full bitmap on a 64bit machine:
sum(bits/64^k, k, 0, HBITMAP_LEVELS) ~= 1.01587 * bits
We may be able to improve
The new command pair is added to manage a user created dirty bitmap. The
dirty bitmap's name is mandatory and must be unique for the same device,
but different devices can have bitmaps with the same names.
The granularity is an optional field. If it is not specified, we will
choose a default
Adds the disabled and frozen status booleans.
Signed-off-by: Fam Zheng f...@redhat.com
Signed-off-by: John Snow js...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
---
block.c | 2 ++
qapi/block-core.json | 7 ++-
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git
Signed-off-by: John Snow js...@redhat.com
Reviewed-by: Max Reitz mre...@redhat.com
---
block.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/block.c b/block.c
index a7f56f4..c1c11aa 100644
--- a/block.c
+++ b/block.c
@@ -60,11 +60,11 @@
* or enabled. A
This series requires: [PATCH v3] blkdebug: fix once rule
Salutations, weary traveler. Might I interest you in some
incremental backups? Only slightly used, just like new!
This patchset enables the in-memory part of the incremental backup
feature. Bitmap persistence and bitmap migration will be
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
tests/qemu-iotests/112 | 49 ++
tests/qemu-iotests/112.out | 4 ++--
2 files changed, 51 insertions(+), 2 deletions(-)
diff --git a/tests/qemu-iotests/112
Signed-off-by: John Snow js...@redhat.com
---
block.c| 20
include/block/block.h | 1 +
include/qemu/hbitmap.h | 10 ++
util/hbitmap.c | 47 +++
4 files changed, 78 insertions(+)
diff --git
This allows to put the dirty bitmap into a disabled state where it is
read only. A disabled bitmap will ignore any attempts to set or reset
any of its bits, but can otherwise be renamed, deleted, or re-enabled.
It will be used before backup or writing to persistent file.
Signed-off-by: Fam Zheng
This adds four qmp commands to transactions.
Users can stop a dirty bitmap, start backup of it, and start another
dirty bitmap atomically, so that the dirty bitmap is tracked
incrementally and we don't miss any write.
For starting a new incremental backup chain, users can also chain
together a
Reviewed-by: Max Reitz mre...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
tests/qemu-iotests/112 | 122 +
tests/qemu-iotests/112.out | 4 +-
2 files changed, 124 insertions(+), 2 deletions(-)
diff --git a/tests/qemu-iotests/112
On 02/19/2015 06:43 AM, Vladimir Sementsov-Ogievskiy wrote:
On 19.02.2015 02:45, John Snow wrote:
On 02/18/2015 09:00 AM, Vladimir Sementsov-Ogievskiy wrote:
Meta bitmap is a 'dirty bitmap' for the BdrvDirtyBitmap. It tracks
changes (set/unset) of this BdrvDirtyBitmap. It is needed for
On 02/20/2015 04:07 PM, John Snow wrote:
Signed-off-by: John Snow js...@redhat.com
---
docs/bitmaps.md | 253
1 file changed, 253 insertions(+)
create mode 100644 docs/bitmaps.md
diff --git a/docs/bitmaps.md b/docs/bitmaps.md
At present finish decoding the first TB block: _start. Always let y1 and
x1 pipe at the last which may content jump instructions.
The related disassembly code is below which is the same as the objdump:
y0: 500bfdb4 move r52, r54
y2: 02080760 ld r1, r54
y1:
The left padding is important and necessary to keep for my particular
application. This is broken in libvirt, but works fine with direct Qemu
invocation.
Thank you for the help.
Alan
On Feb 20, 2015, at 1:28 AM, Markus Armbruster arm...@redhat.com wrote:
Alan Latteri
On 02/17/2015 06:24 AM, Michael Mueller wrote:
+/**
+ * s390_test_facility - test if given facility bit is set facility list
+ * of given cpu class
+ * @class: address of cpu class to test
+ * @nr: bit number to test
+ *
+ * Returns: true in case it is set
+ *
On Fri, 20 Feb 2015 16:41:49 +0100
Andreas Färber afaer...@suse.de wrote:
Also a general comment: cpu-model/ is not an existing directory nor one
you add, so please use target-s390x: Add foo to S390CPU or so.
I will address this with v3, thanks a lot for the hint, I never saw this as
On 02/20/2015 05:02 AM, Stefan Hajnoczi wrote:
On Fri, Feb 13, 2015 at 05:08:56PM -0500, John Snow wrote:
+def check_incremental(self, target=None):
+if target is None:
+target = self.bitmaps[-1].last_target()
+
Am 2015-02-20 16:37, schrieb Radim Krčmář:
2015-02-20 23:40+0900, Peter Maydell:
On 20 February 2015 at 23:18, Radim Krčmář rkrc...@redhat.com wrote:
man gcc:
Warn if in a loop with constant number of iterations the compiler
detects undefined behavior in some statement during one or more
Future patches should address the out-of-bounds access that is left by
using proposed solution for [2/2].
Radim Krčmář (2):
fix GCC 5.0.0 logical-not-parentheses warnings
milkymist-pfpu: fix GCC 5.0.0 aggressive-loop-optimizations warning
hw/misc/milkymist-pfpu.c | 2 +-
hw/net/virtio-net.c
Ingo Krabbe's suggested change fixes the issue for me.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1412098
Title:
qemu crashes when ctrl-alt-u is pressed
Status in QEMU:
New
Bug description:
On Fri, 20 Feb 2015 15:03:30 +0100
Alexander Graf ag...@suse.de wrote:
- s390_get_proceccor_props()
- s390_set_proceccor_props()
They can be used to request or retrieve processor related information from
an accelerator.
That information comprises the cpu identifier, the ICB value
2015-02-20 15:55+0100, Paolo Bonzini:
On 20/02/2015 15:52, Michael Walle wrote:
-i = 0;
-while (pfpu_decode_insn(s)) {
-/* decode at most MICROCODE_WORDS instructions */
-if (i++ = MICROCODE_WORDS) {
Isn't the fix just to
On 02/20/2015 05:57 AM, Laurent Desnogues wrote:
The problem with this approach is that you'll always call the FP
compare which might result in FP flags corruption.
The ARMv8 manual clearly states that the FP compare should only be
called if the condition holds.
Ah, I hadn't considered
Am 10.02.2015 um 11:41 hat Kevin Wolf geschrieben:
Instead of using the same function for entering and exiting coroutines,
and hoping that it doesn't add any functionality that hurts with the
parameters used for exiting, we can just directly call into the real
task switch in
On Fri, 20 Feb 2015 14:55:32 +0100
Alexander Graf ag...@suse.de wrote:
/**
* S390CPUClass:
* @parent_realize: The parent class' realize handler.
@@ -52,6 +69,11 @@ typedef struct S390CPUClass {
void (*load_normal)(CPUState *cpu);
void (*cpu_reset)(CPUState *cpu);
On 02/20/2015 05:07 AM, Leon Alrae wrote:
Save MSACSR state. Also remove fp_status, msa_fp_status, hflags and restore
them in post_load() from the architectural registers.
Float exception flags are not present in vmstate. Information they carry
is used only by softfloat caller who translates
On 02/20/2015 01:18 AM, Markus Armbruster wrote:
I'd like Eric's opinion on on encoding configuration tuples as URIs
rather than JSON in QMP.
+{ 'command': 'migrate-incoming', 'data': {'uri': 'str' } }
+
# @xen-save-devices-state:
#
# Save the state of all devices to file. The RAM and
Am 20.02.2015 um 16:32 schrieb Michael Mueller:
On Fri, 20 Feb 2015 15:03:30 +0100
Alexander Graf ag...@suse.de wrote:
- s390_get_proceccor_props()
- s390_set_proceccor_props()
They can be used to request or retrieve processor related information from
an accelerator.
That information
2015-02-20 15:38+0100, Paolo Bonzini:
On 20/02/2015 15:18, Radim Krčmář wrote:
-if (!!n-vhost_started ==
+if ((!!n-vhost_started) ==
(virtio_net_started(n, status) !nc-peer-link_down)) {
Does it still break if you just swap the terms?
No (there is no logical not on
On Fri, 20 Feb 2015 16:22:20 +0100
Alexander Graf ag...@suse.de wrote:
Just make this uint64_t fac_list[2]. That way we don't have to track any
messy allocations.
It will be something like uint64_t
fac_list[S390_CPU_FAC_LIST_SIZE_UINT64] and in total 2KB
not just 16 bytes but I
1 - 100 of 183 matches
Mail list logo
