The behaviour of the ROUND_UP macro with negative numbers isn't obvious.
It happens to do the right thing in this please, but better avoid it.
Suggested-by: Laszlo Ersek
Signed-off-by: Kevin Wolf
---
block.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block.c b/block
Am 07.02.2014 um 15:47 hat Olaf Hering geschrieben:
> bdrv_acct_done was called unconditional. But in case the ioreq has no
> segments there is no matching bdrv_acct_start call. This could lead to
> bogus accounting values.
>
> Found by code inspection.
>
> Signed-off-by: Olaf Hering
> ---
> v2:
On 02/07/2014 09:12 AM, Kevin Wolf wrote:
> The behaviour of the ROUND_UP macro with negative numbers isn't obvious.
> It happens to do the right thing in this please, but better avoid it.
s/please/place/
>
> Suggested-by: Laszlo Ersek
> Signed-off-by: Kevin Wolf
> ---
> block.c | 4 ++--
> 1
On 07/02/14 14:47, Paolo Bonzini wrote:
Commit 360e607 (address_space_translate: do not cross page boundaries,
2014-01-30) broke MMIO accesses in cases where the section is shorter
than the full register width. This can happen for example with the
Bochs DISPI registers, which are 16 bits wide b
Hi,
Looks good, ack series.
Regards,
Hans
On 02/06/2014 12:42 PM, Gerd Hoffmann wrote:
Frameid specifies frames not microframes, so we
need to shift it to get the microframe index.
Signed-off-by: Gerd Hoffmann
---
hw/usb/hcd-xhci.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(
On 4 February 2014 08:31, Stefan Hajnoczi wrote:
> On Mon, Feb 03, 2014 at 04:01:00PM +, Peter Maydell wrote:
>> On 3 February 2014 15:20, Stefan Hajnoczi wrote:
>> > This pull request has been rebased onto qemu.git/master and retested.
>> >
>> > The series fixes init_socket() failures when r
On Fri, Feb 07, 2014 at 08:57:35PM +0530, Bharata B Rao wrote:
> On Fri, Feb 07, 2014 at 03:22:29PM +0100, Benoît Canet wrote:
> > Le Friday 07 Feb 2014 à 09:14:50 (+0530), Bharata B Rao a écrit :
> > > On Wed, Feb 05, 2014 at 08:25:36PM +0100, Benoît Canet wrote:
> > > > Le Tuesday 04 Feb 2014 à 1
From: Thomas Huth
The SET-ARCHITECTURE handler in QEMU caused an program interruption.
This is wrong according to the "Principles of Operations" specification
(since SIGP should never cause a program interrupt) and was likely only
introduced for debugging purposes. Since we handle SET-ARCHITECTUR
From: Thomas Huth
The s390_cpu_initial_reset() function had two deficiencies: First, it
used an ioctl for the destination CPU, and this ioctl could block
nearly forever, as long as the destination CPU was running in the SIE
loop. Second, it also cleared the general purpose registers - something
i
From: Thomas Huth
In the SCLP handler function, the condition code register must
only be set if no exception occured.
Signed-off-by: Thomas Huth
Reviewed-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
target-s390x/kvm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
dif
As a followup to commit 5f04c14a10fa7f259bc0808f35a0beda49f7821e
(s390-sclp: Define New SCLP Codes) we should mask the sclp command
not only in base sclp, but also in the event facility.
Based on an initial patch from Ralf Hoppe.
Signed-off-by: Christian Borntraeger
---
hw/s390x/event-facility.
From: Heinz Graalfs
init/exit functionality of abstract SCLPEvent class is now exploiting
realize/unrealize.
Signed-off-by: Heinz Graalfs
Reviewed-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
hw/s390x/event-facility.c | 27 ++-
1 file changed, 18 inserti
From: Jens Freimann
This patch implements a floating-interrupt controller device (flic)
which interacts with the s390 flic kvm_device.
Signed-off-by: Jens Freimann
Reviewed-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
default-configs/s390x-softmmu.mak | 1 +
hw/intc/Makefile.
From: Dominik Dingel
S390 can also use async page faults, to enhance guest scheduling.
In case of live migration we want to disable the feature let all
pending request finish.
Signed-off-by: Dominik Dingel
Signed-off-by: Jens Freimann
Signed-off-by: Christian Borntraeger
---
hw/intc/s390_fli
From: Heinz Graalfs
Add support for live migration using VMStateDescription.
Fold this into final patch adding Live Migration code for
SCLP Event Facility.
Signed-off-by: Heinz Graalfs
Reviewed-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
hw/s390x/event-facility.c | 12
From: Thomas Huth
The SIGP order CPU RESET was still missing in the list of our
supported handler. This patch now adds a simple implementation,
by using the cpu_reset() function that is already available in
target-s390x/cpu.c.
Signed-off-by: Thomas Huth
Reviewed-by: Christian Borntraeger
Revie
From: Thomas Huth
The handler for diag 500 did not check whether the requested function
was in the supported range, so illegal values could crash QEMU in the
worst case.
Signed-off-by: Thomas Huth
Reviewed-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
hw/s390x/s390-virtio-hcall.
From: Thomas Huth
The pointer to the SCCB should not be limited to 32 bits only.
In contrast to this, the command word parameter is only 32 bits
(the upper 32 bits should be ignored).
Signed-off-by: Thomas Huth
Reviewed-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
hw/s390x/sclp
From: Heinz Graalfs
Code restructure in order to simplify class hierarchy
- remove S390SCLPDevice abstract base class
and move function pointers into new SCLPEventFacilityClass
- implement SCLPEventFacility as SysBusDevice
- use define constants for instance creation strings
The follow
From: Thomas Huth
So far, the DIAG 500 hypervisor call was only setting -EINVAL in
R2 when a guest tried to call this function with an illegal subcode.
This patch now changes the behavior so that a specification exception
is thrown instead, since this is the common behavior of other DIAG
function
From: Heinz Graalfs
Do some renaming to shorten some identifiers and to emphasize sclp.
Signed-off-by: Heinz Graalfs
Acked-by: Cornelia Huck
Signed-off-by: Christian Borntraeger
---
hw/s390x/event-facility.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git
upstream commit id tbd
Signed-off-by: Christian Borntraeger
---
linux-headers/asm-arm/kvm.h| 28 ++
linux-headers/asm-arm64/kvm.h | 21 -
linux-headers/asm-s390/kvm.h | 19
linux-headers/asm-x86/hyperv.h | 16 --
linux-headers/linux/kvm.h
From: Thomas Huth
If the 51 most significant bits of the SCCB address are zero or equal to
the prefix, we should throw an specification exception, too.
Also moved the check for privileged mode to sclp_service_call() to have
all program checks in one place now.
Signed-off-by: Thomas Huth
Reviewe
Am 2013-09-04 11:04, schrieb Andreas Färber:
On part 11, rth said for alpha that using CPUAlphaState in DisasContext
was bad
and that the relevant fields should be copied into DisasContext
instead; a few
targets still have an env field though and use it for register access
and
cpu_abort(), so I
Hi Alex,
Maik Broemme wrote:
> Hi Alex,
>
> Alex Williamson wrote:
> > On Thu, 2014-02-06 at 01:25 +0100, Maik Broemme wrote:
> > > Hi Alex,
> > >
> > > Maik Broemme wrote:
> > > > > > > > Another minor issue is that the R9 290X is not reset during
> > > > > > > > shutdown of
> > > > > > > >
Am 07.02.2014 16:48, schrieb Stefan Hajnoczi:
> On Wed, Feb 05, 2014 at 06:01:13PM +0100, Andreas Färber wrote:
>> +#include "libqtest.h"
>> +
>> +#include
>> +#include
>> +#include "qemu/osdep.h"
>
> Please put includes before "user.h" includes. That way we
> can be sure our macros don't mess
I've bisected a problem with system emulation and SMP kernels using
per cpu timers to this commit. I can reproduce this problem on ARM
emulation with both ARM generic timers (only in 1.7.0) and ARM MPCore
timers. Using a single broadcast timer in the guest kernel works fine.
My host is ubuntu 13.1
From: Claudio Fontana
this is general polishing for tcg/aarch64, removing the inline magic
insn numbers, and putting up enums for intruction classes and subclasses
in their stead.
Signed-off-by: Claudio Fontana
Tested-by: Claudio Fontana
---
tcg/aarch64/tcg-target.c | 528
Successfully installed NextStep in bochs, but not qemu.
Having same problem with OpenStep 4.2 in qemu.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1276879
Title:
lots of dma command 10, 14 not s
I would be overjoyed if someone could point me to a description of the behavior
of qemu when generating the ACPI "power button press" event for the guest. And a
further reference to a description of the standard that qemu adheres to (I
presume) would be icing on the cake.
cheers,
Rob Urban
On 02
On Fri, 2014-02-07 at 01:22 +0100, Maik Broemme wrote:
> Interesting is the diff between 1st and 2nd boot, so if I do the lspci
> prior to the booting. The only difference between 1st start and 2nd
> start are:
>
> --- 001-lspci.290x.before.1st.log 2014-02-07 01:13:41.498827928 +0100
> +++ 004
* Stefan Priebe (s.pri...@profihost.ag) wrote:
> anything i could try or debug? to help to find the problem?
I think the most useful would be to see if the problem is
a new problem in the 1.7 you're using or has existed
for a while; depending on the machine type you used, it might
be possible to
Am 07.02.2014 21:02, schrieb Dr. David Alan Gilbert:
* Stefan Priebe (s.pri...@profihost.ag) wrote:
anything i could try or debug? to help to find the problem?
I think the most useful would be to see if the problem is
a new problem in the 1.7 you're using or has existed
for a while; depending
Hi Yan,
So recently I've been doing some KVM guest performance comparisons
between the scsi-mq prototype using virtio-scsi + vhost-scsi, and
Windows Server 2012 with vioscsi.sys (virtio-win-0.1-74.iso) +
vhost-scsi using PCIe flash backend devices.
I've noticed that small block random performance
Hi Alex,
Alex Williamson wrote:
> On Fri, 2014-02-07 at 01:22 +0100, Maik Broemme wrote:
> > Interesting is the diff between 1st and 2nd boot, so if I do the lspci
> > prior to the booting. The only difference between 1st start and 2nd
> > start are:
> >
> > --- 001-lspci.290x.before.1st.log 2
here is my next bunch of patches for s390x/kvm for 2.0.
The tree is also available at
git://github.com/borntraeger/qemu.git s390-next
It still depends on a not-yet pulled kvm tree
( git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
tags/kvm-s390-20140130)
which Paolo will probabl
anything i could try or debug? to help to find the problem?
Stefan
Am 07.02.2014 14:45, schrieb Stefan Priebe - Profihost AG:
it's always the same "pattern" there are too many 0 instead of X.
only seen:
read:0x ... expected:0x
or
read:0x ... e
Kevin Wolf (3):
block: Fix memory leaks in bdrv_co_do_pwritev()
block: bdrv_aligned_pwritev: Assert overlap range
block: Don't call ROUND_UP with negative values
block.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
--
1.8.1.4
On 02/05/2014 11:22 PM, Gaurav Sharma wrote:
> I need some help regarding the flow of generating a TCG op and then
> subsequent translation of any op into target instrn.
That's most of tcg/tcg.c and tcg/foo/tcg-target.c.
tcg_gen_code is the main entry point for beginning the compilation
of the pr
Peter,
Am 02.09.2013 13:41, schrieb Michael S. Tsirkin:
> On Mon, Jul 29, 2013 at 04:03:56AM +0200, Andreas Färber wrote:
>> Hello,
>>
>> Based on a comment from mst, this mini-series proposes to change semantics of
>> VMStateDescription registration to be more similar to those of static
>> prope
On 7 February 2014 21:28, Andreas Färber wrote:
> I have found a single such case: armv7m_nvic overrides arm_gic_common
> with a completely different vmsd. How can we fix that? :)
We can deduce from this that nobody's been using migration
with the Cortex-M3, since it would obviously be totally bu
http://lists.nongnu.org/archive/html/qemu-devel/2014-02/msg00889.html
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1276879
Title:
lots of dma command 10, 14 not supported
Status in QEMU:
New
B
Am 07.02.2014 um 17:27 hat Eric Blake geschrieben:
> On 02/07/2014 09:12 AM, Kevin Wolf wrote:
> > The behaviour of the ROUND_UP macro with negative numbers isn't obvious.
> > It happens to do the right thing in this please, but better avoid it.
>
> s/please/place/
Indeed... Thanks, fixed it loca
Implement the 'long' operations in the vector x indexed
element category.
Signed-off-by: Peter Maydell
---
target-arm/translate-a64.c | 144 +++--
1 file changed, 139 insertions(+), 5 deletions(-)
diff --git a/target-arm/translate-a64.c b/target-arm/trans
This is the fourth set of A64 Neon support patches. This set
provides complete implementations of the categories:
* scalar x indexed
* vector x indexed
* scalar three-different
and fills in all the previously missing instructions in:
* vector 3-reg-same
* scalar 3-reg-same
It includes adding
The ARMv8 instruction set includes a fused floating point
reciprocal square root step instruction which demands an
"(x * y + z) / 2" fused operation. Support this by adding
a flag to the softfloat muladd operations which requests
that the result is halved before rounding.
Signed-off-by: Peter Mayd
From: Alex Bennée
This adds all forms of the SIMD floating point and set instructions:
FCM(GT|GE|EQ|LE|LT)
Most of the heavy lifting is done by either the existing neon helpers or
some new helpers for the 64bit double cases. Most of the code paths are
common although the 2misc versions are a
Implement the SIMD scalar indexed instructions. The encoding
here is nearly identical to the vector indexed grouping, so
we combine the two.
Signed-off-by: Peter Maydell
---
target-arm/translate-a64.c | 115 -
1 file changed, 82 insertions(+), 33 delet
From: Alex Bennée
Add support for the floating-point pairwise operations
FADDP, FMAXP, FMAXNMP, FMINP and FMINNMP. To do this we use the
code which was previously handling only integer pairwise operations,
and push the integer-specific decode and handling of unallocated
cases up one level in the
Implement the remaining instructions in the SIMD 3-reg-same
and scalar-3-reg-same groups: FMULX, FRECPS, FRSQRTS, FACGE,
FACGT, FMLA and FMLS.
Signed-off-by: Peter Maydell
---
target-arm/helper-a64.c| 60 ++
target-arm/helper-a64.h| 4
ta
Implement the scalar three different instruction group:
it only has three instructions in it.
Signed-off-by: Peter Maydell
---
target-arm/translate-a64.c | 95 +-
1 file changed, 94 insertions(+), 1 deletion(-)
diff --git a/target-arm/translate-a64.c
Implement all the SIMD vector x indexed element instructions
in the subcategory which are not 'long' ops.
Signed-off-by: Peter Maydell
---
target-arm/helper-a64.c| 26 +
target-arm/helper-a64.h| 2 +
target-arm/translate-a64.c | 245 -
3
Michal,
commit 34b9c07a3 (microblaze: Disable stack protection from bootloader) results
in the following qemu crash in 3.14-rc1.
/opt/buildbot/bin/qemu-system-microblaze -M petalogix-s3adsp1800 -kernel
arch/microblaze/boot/linux.bin -no-reboot -append "console=ttyUL0,115200 "
-nographic
qemu: f
On 02/07/14 22:45, Kevin Wolf wrote:
> Am 07.02.2014 um 17:27 hat Eric Blake geschrieben:
>> On 02/07/2014 09:12 AM, Kevin Wolf wrote:
>>> The behaviour of the ROUND_UP macro with negative numbers isn't obvious.
>>> It happens to do the right thing in this please, but better avoid it.
>>
>> s/pleas
On 02/04/2014 08:21 AM, XliN wrote:
> Good day. Very little speed drives Virtio. Drivers are the latest
> guest on the system "Windows server 2008". Host system centos 6.5.
What latest? There are a few different places to get drivers (Fedora
site, RHEL subscription, build yourself, etc). From the
Well, despite those very real error messages I had, its gone now with a
full msys reinstall. Same dependencies for 32 bit. I just decided to
try the 64 bit since to me it seemed like you were saying that would be
more likely to work. As part of installing the 64 bit, I decided to
incur more pain
So, Zoltan pointed to two patches. I applied the first
(v8-06-10-i8259-fix.),
but the second patches a file that is not in version 1.7.0.
With the one patch, I am now able to get past the step where it crapped out
before. The dma messages are still there, but it seems that they can be
igno
On Fri, Feb 07, 2014 at 03:17:31PM -0800, Guenter Roeck wrote:
> Michal,
>
> commit 34b9c07a3 (microblaze: Disable stack protection from bootloader)
> results
> in the following qemu crash in 3.14-rc1.
>
> /opt/buildbot/bin/qemu-system-microblaze -M petalogix-s3adsp1800 -kernel
> arch/microblaz
can anyone provide further review to this patch please ?
Thank you,
Xin
On Mon, Feb 3, 2014 at 9:50 AM, Eric Blake wrote:
> On 02/03/2014 08:18 AM, Xin Tong wrote:
>> This patch adds a victim TLB to the QEMU system mode TLB.
>>
>> Changes from PATCH v3.
>> 1. updated measurements - performance i
Reviewed-by: Eric Johnson
From: qemu-devel-bounces+eric.johnson=imgtec@nongnu.org
[qemu-devel-bounces+eric.johnson=imgtec@nongnu.org] on behalf of Petar
Jovanovic [petar.jovano...@rt-rk.com]
Sent: Friday, January 24, 2014 8:18 AM
To: qemu-devel@no
Reviewed-by: Eric Johnson
From: qemu-devel-bounces+eric.johnson=imgtec@nongnu.org
[qemu-devel-bounces+eric.johnson=imgtec@nongnu.org] on behalf of Petar
Jovanovic [petar.jovano...@rt-rk.com]
Sent: Friday, January 24, 2014 8:18 AM
To: qemu-devel@no
Reviewed-by: Eric Johnson
From: qemu-devel-bounces+eric.johnson=imgtec@nongnu.org
[qemu-devel-bounces+eric.johnson=imgtec@nongnu.org] on behalf of Petar
Jovanovic [petar.jovano...@rt-rk.com]
Sent: Friday, January 24, 2014 8:18 AM
To: qemu-devel@no
Reviewed-by: Eric Johnson
From: qemu-devel-bounces+eric.johnson=imgtec@nongnu.org
[qemu-devel-bounces+eric.johnson=imgtec@nongnu.org] on behalf of Petar
Jovanovic [petar.jovano...@rt-rk.com]
Sent: Friday, January 24, 2014 8:18 AM
To: qemu-devel@no
On 02/07/2014 06:31 PM, Edgar E. Iglesias wrote:
On Fri, Feb 07, 2014 at 03:17:31PM -0800, Guenter Roeck wrote:
Michal,
commit 34b9c07a3 (microblaze: Disable stack protection from bootloader) results
in the following qemu crash in 3.14-rc1.
/opt/buildbot/bin/qemu-system-microblaze -M petalogix
Makefile.target includes rule.mak and unnested common-obj-y, then prefix
them with '../', this will ignore object specific QEMU_CFLAGS in subdir
Makefile.objs:
$(obj)/curl.o: QEMU_CFLAGS += $(CURL_CFLAGS)
Because $(obj) here is './block', instead of '../block'. This doesn't
hurt compiling bec
Adds extract-libs in LINK to expand any "per object libs", the syntax to define
such a libs options is like:
foo.o-libs := $(CURL_LIBS)
in block/Makefile.objs.
Similarly,
foo.o-cflags := $(FOO_CFLAGS)
is also supported.
"foo.o" must be listed in a nested var (e.g. common-obj-y
v19: [11/11] New to v18.
Fam Zheng (10):
util: Split out qemu_exec_dir from os_find_datadir
rules.mak: fix $(obj) to a real relative path
rules.mak: allow per object cflags and libs
block: use per-object cflags and libs
rules.mak: introduce DSO rules
module: implement module loading
This patch adds loading, stamp checking and initialization of modules.
The init function of dynamic module is no longer directly called as
__attribute__((constructor)) in static linked version, it is called
only after passed the checking of presense of stamp symbol:
qemu_stamp_$RELEASEHASH
w
While -mdynamic-no-pic can speed up the code somewhat, it is only used
on the legacy PowerPC Mac OS X, and I am not sure if anyone is still
testing that. Disabling PIC can cause problems when enabling modules,
so do not do that.
Signed-off-by: Paolo Bonzini
---
configure | 2 --
1 file changed,
$(common-obj-m) will include $(block-obj-m), like $(common-obj-y) does
for $(block-obj-y).
Signed-off-by: Fam Zheng
Signed-off-by: Paolo Bonzini
---
Makefile| 4 +++-
Makefile.objs | 2 ++
Makefile.target | 6 +-
3 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/Make
Install all the modules to ${MODDIR}.
Signed-off-by: Fam Zheng
Signed-off-by: Paolo Bonzini
---
Makefile | 6 ++
1 file changed, 6 insertions(+)
diff --git a/Makefile b/Makefile
index 07d1ed7..57d83a3 100644
--- a/Makefile
+++ b/Makefile
@@ -368,6 +368,12 @@ install-datadir install-localst
Add necessary rules and flags for shared object generation.
The new rules introduced here are:
1) %.o in $(common-obj-m) is compiled to %.o, then linked to %.so.
2) %.mo in $(common-obj-m) is the placeholder for %.so for pattern
matching in Makefile. It's linked to "-shared" with all its dependen
This can be reused by module loading routines.
Signed-off-by: Fam Zheng
Signed-off-by: Paolo Bonzini
---
include/qemu/osdep.h | 4
os-posix.c | 40 ++--
os-win32.c | 19 +--
util/oslib-posix.c | 45 +
This adds parameter "argv0" in calling path from main() to
module_call_init(). So that module loader knows the location of
executable.
Suggested-by: Paolo Bonzini
Signed-off-by: Fam Zheng
---
block.c| 8
bsd-user/main.c| 2 +-
include/block/blo
The converted block drivers are:
curl
iscsi
rbd
ssh
glusterfs
Signed-off-by: Fam Zheng
Signed-off-by: Paolo Bonzini
---
configure | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/configure b/configure
index 7aff69e..a30ec76 100755
--- a/configu
No longer adds flags and libs for them to global variables, instead
create config-host.mak variables like FOO_CFLAGS and FOO_LIBS, which is
used as per object cflags and libs.
This removes unwanted dependencies from libcacard.
Signed-off-by: Fam Zheng
[Split from Fam's patch to enable modules. -
101 - 176 of 176 matches
Mail list logo