Hi, all
I noticed there are several patchset has been sent out about IOMMU
emulation like
http://lists.gnu.org/archive/html/qemu-devel/2011-10/msg03764.html,
http://lists.gnu.org/archive/html/qemu-devel/2011-01/msg03196.html . Also I
noticed that PPC platform has IOMMU emulation support
On 11/19/2013 05:47 AM, Clemens Kolbitsch wrote:
> Getting around this is not trivial, so if it's a new issue, does anyone have
> an
> idea how to approach it most meaningfully? I was thinking about detecting this
> very specific case and marking the TB for re-translation only after the TB is
> ex
Ping.
r~
On 10/29/2013 03:30 AM, Richard Henderson wrote:
> Ping.
>
> r~
>
> On 10/17/2013 08:29 AM, Richard Henderson wrote:
>> Changes from v2:
>> * Rebase for trivial conflicts
>> * Fix codingstyle errors
>> * Drop the static fallback code
>> * Use getauxval for AT_EXECFD
>>
>>
>> r~
Ping?
r~
On 09/20/2013 05:58 AM, Richard Henderson wrote:
> We previously allocated 32-bits per temp for the next_free_temp entry.
> We now allocate 4 bits per temp across the 4 bitmaps.
>
> Using a linked list meant that if a translator is tweeked, resulting in
> temps being freed in a differen
Hi,
> I think it would be better if the HwOps calls all took a QemuConsole
> instead of the opaque structure. The hw implementations can dig their
> opaque structure out from there.
QemuConsole is private to ui/console.c though (and I prefer to keep it
this way). So we need either a helper f
The second patch is too big in size (over 10M), and rejected by list
moderator.
You can PULL from git branch:
https://github.com/famz/qemu.git 035-filter-qemu-io
On 2013年11月18日 15:21, Fam Zheng wrote:
Fam Zheng (2):
qemu-iotests: Filter qemu-io output in 025
qemu-iotests: Filter
On 2013年11月18日 23:38, Kevin Wolf wrote:
Am 18.11.2013 um 16:29 hat Stefan Hajnoczi geschrieben:
On Thu, Nov 14, 2013 at 10:24:04AM +0800, Fam Zheng wrote:
The default cache mode for drive options is changed to writethrough, and
overridable with "./check -c ".
Please make the default "writebac
The option sets cache mode used in the tests. "-nocache" is changed to
an alias to "-c none", and internally passes "-t none" to qemu-io.
Python scripts will make use of option this in the next commit.
Signed-off-by: Fam Zheng
---
tests/qemu-iotests/common | 18 --
1 file change
This will allow overriding cache mode from the "-c mode" option.
Signed-off-by: Fam Zheng
---
tests/qemu-iotests/iotests.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
index fb10ff4..c84a1a5 100644
--- a/test
The first patch adds "-c " option to ./check and convert "-nocache" to an
alias to "-c none". The mode is used in qemu-io.
The second patch modifies iotests.py to use the cache mode option in qemu drive
command line.
Fam Zheng (2):
qemu-iotests: Add "-c " option
qemu-iotests: Honour cache mod
So I've started banging my head against using QemuConsole as the
container for a single output, and have been left with the usual 10
ways to design things, but since I don't want to spend ages
implementing one way just to be told its unacceptable it would be good
to get some more up front design in
Save and restore the ARM KVM VGIC state from the kernel. We rely on
QEMU to marshal the GICState data structure and therefore simply
synchronize the kernel state with the QEMU emulated state in both
directions.
We take some care on the restore path to check the VGIC has been
configured with enoug
To make the code slightly cleaner to look at and make the save/restore
code easier to understand, introduce this function to set the priority of
interrupts.
Reviewed-by: Peter Maydell
Signed-off-by: Christoffer Dall
---
hw/intc/arm_gic.c | 15 ++-
hw/intc/gic_internal.h | 1 +
Add support for saving VMState of 2D arrays of uint32 values.
Reviewed-by: Peter Maydell
Signed-off-by: Christoffer Dall
---
include/migration/vmstate.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/migration/vmstate.h b/include/migration/vmstate.h
index 9d09e60..c44e3b5 100
The GICC_APRn registers are not currently supported by the ARM GIC v2.0
emulation. This patch adds the missing state. The state was previously
added in the main vgic save/restore patch, but moved to a separate patch
as suggested under review.
Note that we also change the number of APRs to use a
If software writes to the ISPENDR and sets the pending state of a
level-triggered interrupt, the falling edge of the hardware input must
not clear the pending state. Conversely, if software writes to the
ICPENDR, the pending state of a level-triggered interrupt should only be
cleared if the hardwa
Implement support to save/restore the ARM KVM VGIC state from the
kernel. The basic appraoch is to transfer state from the in-kernel VGIC
to the emulated arm-gic state representation and let the standard QEMU
vmstate save/restore handle saving the arm-gic state. Restore works by
reversing the pro
The pending flags for level-triggered interrupts should not be cleared
if the interrupt input signal remains asserted.
Signed-off-by: Christoffer Dall
Changelog[3]:
- New patch in the series
---
hw/intc/arm_gic.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/hw
TRIGGER can really mean mean anything (e.g. was it triggered, is it
level-triggered, is it edge-triggered, etc.). Rename to EDGE_TRIGGER to
make the code comprehensible without looking up the data structure.
Signed-off-by: Christoffer Dall
---
hw/intc/arm_gic.c| 12 ++--
hw/intc
Right now the arm gic emulation doesn't keep track of the source of an
SGI (which apparently Linux guests don't use, or they're fine with
assuming CPU 0 always).
Add the necessary matrix on the GICState structure and maintain the data
when setting and clearing the pending state of an IRQ.
Note th
Add a binary_point field to the gic emulation structure and support
setting/getting this register now when we have it. We don't actually
support interrupt grouping yet, oh well.
Signed-off-by: Christoffer Dall
Changelog [v3]:
- Treat writes for GIC prior to v2 as RAZ/WI.
Changelog [v2]:
- Re
Support creating the ARM vgic device through the device control API and
setting the base address for the distributor and cpu interfaces in KVM
VMs using this API.
Because the older KVM_CREATE_IRQCHIP interface needs the irq chip to be
created prior to creating the VCPUs, we first test if we can us
For some reason only edge-triggered or enabled level-triggered
interrupts would set the pending state of a raised IRQ. This is not in
compliance with the specs, which indicate that the pending state is
separate from the enabled state, which only controls if a pending
interrupt is actually forwarde
Update the KVM kernel headers to add support for the device control API
on ARM used to create in-kernel devices and set and get attributes on
these.
This is needed for VGIC save/restore with KVM ARM targets.
Headers are included from:
git://git.linaro.org/people/cdall/linux-kvm-arm.git vgic-migra
Introduces two simple functions:
int kvm_device_ioctl(int fd, int type, ...);
int kvm_create_device(KVMState *s, uint64_t type, bool test);
These functions wrap the basic ioctl-based interactions with KVM in a
way similar to other KVM ioctl wrappers.
Reviewed-by: Peter Maydell
Signed-off
Introduce kvm_arch_irqchip_create an arch-specific hook in preparation
for architecture-specific use of the device control API to create IRQ
chips.
Following patches will implement the ARM irqchip create method to prefer
the device control API over the older KVM_CREATE_IRQCHIP API.
Reviewed-by: P
This patch series adds generic support for issuing device control
related ioctls and supports creating the ARM KVM-accelerated VGIC using
the device control API while maintaining backwards compatibility for
older kernels.
This is an RFC patch set because it relies on kernel header changes that
are
** Attachment added: "Boot test with the old version"
https://bugs.launchpad.net/qemu/+bug/1252009/+attachment/3912215/+files/Old%20version%20test.asf
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs
** Attachment added: "Boot files"
https://bugs.launchpad.net/qemu/+bug/1252009/+attachment/3912216/+files/boot%20files.7z
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1252009
Title:
slow boot
** Attachment added: "Boot test with the new version"
https://bugs.launchpad.net/qemu/+bug/1252009/+attachment/3912214/+files/New%20version%20test.asf
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs
Signed-off-by: Alexey Kardashevskiy
---
hw/intc/xics.c | 24
hw/nvram/spapr_nvram.c | 16
hw/ppc/spapr_events.c | 6 +++---
hw/ppc/spapr_pci.c | 42 +-
hw/ppc/spapr_rtas.c| 34 +
This adds very basic handlers for ibm,get-system-parameter and
ibm,set-system-parameter RTAS calls.
The only parameter handled at the moment is
"platform-processor-diagnostics-run-mode" which is always disabled and
does not support changing. This is expected to make
"ppc64_cpu --run-mode=1" happy.
This reworks RTAS code to use predefined macros vs. numbers and
adds handlers for ibm,get-system-parameter and ibm,set-system-parameter.
Alexey Kardashevskiy (2):
spapr-rtas: replace constants with macros
spapr-rtas: add ibm,(get|set)-system-parameter
hw/ppc/spapr_rtas.c | 86 +++
On Tue, Oct 15, 2013 at 12:15:03PM +0100, Peter Maydell wrote:
> On 26 September 2013 22:03, Christoffer Dall
> wrote:
> > Save and restore the ARM KVM VGIC state from the kernel. We rely on
> > QEMU to marshal the GICState data structure and therefore simply
> > synchronize the kernel state with
Hello guys,
I'm using qemu-arm-static and binfmt_misc to have an ARM chroot on my
laptop. Although most programs run fine when I run a go program or the go
compiler itself it crashes. Digging into the issue I found that the problem
has to do with go trying to use all the signals. I also found a th
On Fri, Sep 27, 2013 at 09:11:18AM +0100, Alex Bennée wrote:
>
> christoffer.d...@linaro.org writes:
>
[...]
> > +
> > +static void translate_active(GICState *s, int irq, int cpu,
> > + uint32_t *field, bool to_kernel)
> > +{
> > +int cm = (irq < GIC_INTERNAL) ?
On Mon, Oct 14, 2013 at 04:36:24PM +0100, Peter Maydell wrote:
> On 26 September 2013 22:03, Christoffer Dall
> wrote:
> > Right now the arm gic emulation doesn't keep track of the source of an
> > SGI (which apparently Linux guests don't use, or they're fine with
> > assuming CPU 0 always).
> >
>
So far POWER7+ was a part of POWER7 family. However it has a different
PVR base value so in order to support PVR masks, it needs a separate
family class.
This adds a new family class, PVR base and mask values and moves
Power7+ v2.1 CPU to a new family. The class init function is copied
from the PO
On 11/19/2013 07:58 AM, Alexander Graf wrote:
>
> On 17.11.2013, at 22:09, Alexey Kardashevskiy wrote:
>
>> This adds very basic handlers for ibm,get-system-parameter and
>> ibm,set-system-parameter RTAS calls.
>>
>> The only parameter handled at the moment is
>> "platform-processor-diagnostics-
On 11/19/13 01:04, Igor Mammedov wrote:
> On Tue, 19 Nov 2013 00:34:31 +0100
> Laszlo Ersek wrote:
>
>> On 11/12/13 16:11, Laszlo Ersek wrote:
>>
>>> - Boot progress bar and console (efifb) continue to work (tested with
>>> both 1GB and 5GB guest sizes).
>>
>> Turns out one can't be diligent en
On Tue, 19 Nov 2013 00:34:31 +0100
Laszlo Ersek wrote:
> On 11/12/13 16:11, Laszlo Ersek wrote:
>
> > - Boot progress bar and console (efifb) continue to work (tested with
> > both 1GB and 5GB guest sizes).
>
> Turns out one can't be diligent enough.
>
> This patch causes (or exposes, depend
On 11/12/13 16:11, Laszlo Ersek wrote:
> - Boot progress bar and console (efifb) continue to work (tested with
> both 1GB and 5GB guest sizes).
Turns out one can't be diligent enough.
This patch causes (or exposes, dependent on your POV) breakage. It
breaks the cirrus video output for both RHE
On Mon, 2013-11-18 at 17:55 -0500, Vlad Yasevich wrote:
> On 11/18/2013 05:40 PM, Alex Williamson wrote:
> > On Mon, 2013-11-18 at 17:07 -0500, Vlad Yasevich wrote:
> >> On 11/18/2013 04:33 PM, Alex Williamson wrote:
> >>> On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> On 11/18/2013
On Mon, 2013-11-18 at 17:07 -0500, Vlad Yasevich wrote:
> On 11/18/2013 04:33 PM, Alex Williamson wrote:
> > On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> >> On 11/18/2013 03:33 PM, Alex Williamson wrote:
> >>> On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich wrote:
> On 11/18/2013
On 11/18/2013 05:40 PM, Alex Williamson wrote:
> On Mon, 2013-11-18 at 17:07 -0500, Vlad Yasevich wrote:
>> On 11/18/2013 04:33 PM, Alex Williamson wrote:
>>> On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
On 11/18/2013 03:33 PM, Alex Williamson wrote:
> On Mon, 2013-11-18 at 15:0
On Tue, Nov 19, 2013 at 06:49:40AM +1000, Richard Henderson wrote:
> hw/i386/acpi-build.c:294:5: error: implicit declaration of function
> ‘g_string_vprintf’ [-Werror=implicit-function-declaration]
> g_string_vprintf(s, format, args);
>
> Introduced in 2.14.
>
>
> hw/i386/acpi-build.c:427:5
On Mon, Nov 18, 2013 at 03:26:09PM -0700, Alex Williamson wrote:
> On Mon, 2013-11-18 at 23:47 +0200, Michael S. Tsirkin wrote:
> > On Mon, Nov 18, 2013 at 02:33:16PM -0700, Alex Williamson wrote:
> > > On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> > > > On 11/18/2013 03:33 PM, Alex Wil
On Mon, Nov 18, 2013 at 03:40:51PM -0700, Alex Williamson wrote:
> On Mon, 2013-11-18 at 17:07 -0500, Vlad Yasevich wrote:
> > On 11/18/2013 04:33 PM, Alex Williamson wrote:
> > > On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> > >> On 11/18/2013 03:33 PM, Alex Williamson wrote:
> > >>> O
On Mon, 2013-11-18 at 23:47 +0200, Michael S. Tsirkin wrote:
> On Mon, Nov 18, 2013 at 02:33:16PM -0700, Alex Williamson wrote:
> > On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> > > On 11/18/2013 03:33 PM, Alex Williamson wrote:
> > > > On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich w
Avoids a wasted cycle loading up small constants.
Simplify the code assuming the tcg optimizer is going to work
and don't expect the first operand of the add to be constant.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 20
1 file ch
There's no need to go through the full opcode-to-insn function call
to generate nops. This makes the source a bit more readable.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 251 +-
1 file changed, 127 ins
Using only indirect calls results in 3 bundles (one to load the
descriptor address), and 4 stop bits. By looking through the
descriptor to the constants, we can perform the call with 2
bundles and only 1 stop bit.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-targe
On 11/18/2013 04:33 PM, Alex Williamson wrote:
> On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
>> On 11/18/2013 03:33 PM, Alex Williamson wrote:
>>> On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich wrote:
On 11/18/2013 02:58 PM, Alex Williamson wrote:
> On Mon, 2013-11-18 at 21:4
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 35 ---
1 file changed, 16 insertions(+), 19 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 322955b..2f630f4 100644
--- a/tcg/ia64/tcg-target.c
++
Being able to "extend" from 64-bits (with a mov) simplifies
a few places where the conditional breaks the train of thought.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 54 +++
1 file changed, 24 insertions
We can subtract from more small constants that just 0 with one insn,
and we can add the negative for most small constants.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 25 +++--
1 file changed, 23 insertions(+), 2 deletions(-)
diff -
We can and/or/xor/andcm small constants, saving one cycle.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 64 +++
1 file changed, 34 insertions(+), 30 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 2f630f4..afcb1ad 100644
--- a/tcg/ia64/tcg-target.c
+++ b
When performing an operation with two input registers, we'd leave
the stop bit (and thus an extra cycle) that's only needed when one
or the other input is a constant.
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 17 ++---
1 file changed, 6 in
Since the move away from the global areg0, we're no longer globally
reserving areg0. Which means our use of R7 clobbers a call-saved
register. Shift areg0 into the windowed registers. Indeed, choose
the incoming parameter register that it comes to us by.
This requires moving the register holdin
There was a misconception that a stop bit is required between a compare
and the branch that uses the predicate set by the compare. This lead to
the usage of an extra bundle in which to perform the compare. The extra
bundle left room for constants to be loaded for use with the compare insn.
If we
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 63 +++
1 file changed, 28 insertions(+), 35 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 8168519..2d8e00c 100644
--- a/tcg/ia64
These are extracted from a larger patch set, including only the
subset that has been reviewed by Aurelien.
Patch 5 fixes a serious bug in the ia64 backend, so I'd like to
ensure this gets fixed for 1.7.
r~
The following changes since commit 5c5432e7d630592ddcc1876ac8a1505f8f14ef15:
Merge re
Acked-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 173 ++
1 file changed, 91 insertions(+), 82 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 0656d39..8db0ac1 100644
--- a/tcg/ia64
On Mon, Nov 18, 2013 at 02:33:16PM -0700, Alex Williamson wrote:
> On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> > On 11/18/2013 03:33 PM, Alex Williamson wrote:
> > > On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich wrote:
> > >> On 11/18/2013 02:58 PM, Alex Williamson wrote:
> > >>> O
On Mon, 2013-11-18 at 15:57 -0500, Vlad Yasevich wrote:
> On 11/18/2013 03:33 PM, Alex Williamson wrote:
> > On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich wrote:
> >> On 11/18/2013 02:58 PM, Alex Williamson wrote:
> >>> On Mon, 2013-11-18 at 21:47 +0200, Michael S. Tsirkin wrote:
> This rev
Please note that this patch fixes real bugs, namely wrong format strings.
As far as I could see those bugs are only in code branches which handle
rare error conditions, so maybe they only have a low priority.
Nevertheless they can be fixed in QEMU 1.7.
Do we need a pull request, or can it be appl
On 18.11.2013, at 03:55, Alexey Kardashevskiy wrote:
> On 11/12/2013 06:18 PM, Alexey Kardashevskiy wrote:
>> On 11/09/2013 11:20 AM, Alexey Kardashevskiy wrote:
>>> On 11/09/2013 03:59 AM, Andreas Färber wrote:
Am 08.11.2013 15:54, schrieb Alexey Kardashevskiy:
> On 11/09/2013 12:44 AM
On 17.11.2013, at 22:09, Alexey Kardashevskiy wrote:
> This adds very basic handlers for ibm,get-system-parameter and
> ibm,set-system-parameter RTAS calls.
>
> The only parameter handled at the moment is
> "platform-processor-diagnostics-run-mode" which is always disabled and
> does not suppor
On 11/18/2013 03:33 PM, Alex Williamson wrote:
> On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich wrote:
>> On 11/18/2013 02:58 PM, Alex Williamson wrote:
>>> On Mon, 2013-11-18 at 21:47 +0200, Michael S. Tsirkin wrote:
This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
Digging
hw/i386/acpi-build.c:294:5: error: implicit declaration of function
‘g_string_vprintf’ [-Werror=implicit-function-declaration]
g_string_vprintf(s, format, args);
Introduced in 2.14.
hw/i386/acpi-build.c:427:5: error: implicit declaration of function
‘g_array_get_element_size’ [-Werror=impli
On Mon, 2013-11-18 at 15:09 -0500, Vlad Yasevich wrote:
> On 11/18/2013 02:58 PM, Alex Williamson wrote:
> > On Mon, 2013-11-18 at 21:47 +0200, Michael S. Tsirkin wrote:
> >> This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
> >> Digging into hardware specs shows this does not
> >> actu
On 11/18/2013 02:58 PM, Alex Williamson wrote:
> On Mon, 2013-11-18 at 21:47 +0200, Michael S. Tsirkin wrote:
>> This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
>> Digging into hardware specs shows this does not
>> actually make QEMU behave more like hardware.
>> Let's stick to the tr
On Mon, 2013-11-18 at 21:47 +0200, Michael S. Tsirkin wrote:
> This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
> Digging into hardware specs shows this does not
> actually make QEMU behave more like hardware.
> Let's stick to the tried heuristic for 1.7 and
> possibly revisit for 1.8.
Am 18.11.2013 17:27, schrieb Anthony Liguori:
>
>
> On Nov 18, 2013 12:20 AM, "Peter Lieven" mailto:p...@kamp.de>>
> wrote:
> >
> > vnc_update_client currently scans the dirty bitmap of each client
> > bitwise which is a very costly operation if only few bits are dirty.
> > vnc_refresh_server_surf
Am 18.11.2013 17:11, schrieb Paolo Bonzini:
> Il 18/11/2013 16:37, Peter Lieven ha scritto:
>>>
>>> If I specify: -smp 2,sockets=1,cores=2,threads=1 to a Windows 2012 R2
>>> Server it crashes
>>> at boot time. -smp 2 works.
>> for Linux /proc/cpuinfo reveals no cpu layout information (sibliings,
>>
On 11/18/2013 11:55 PM, Michael Matz wrote:
>> > I think we need to either explicitly convert the tcg_shift to a
>> > TCGv_i32, or we need to use an open coded version of the rotr_i64 that
>> > inserts at (32 - n) instead of (64 - n)
>> >
>> > What do you think?
> I think converting tcg_shift mi
On 11/18/2013 02:47 PM, Michael S. Tsirkin wrote:
> This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
> Digging into hardware specs shows this does not
> actually make QEMU behave more like hardware.
> Let's stick to the tried heuristic for 1.7 and
> possibly revisit for 1.8.
>
> Repor
Hi guys,
a quick search on the archives did not reveal anything - if this is already
a known issue, please don't kick me ;)
We encountered an issue using TCG, where self-modifying code is overwriting
the executing instruction, causing different behavior in TCG and a real CPU.
Clearly, when code
This reverts commit cd5be5829c1ce87aa6b3a7806524fac07ac9a757.
Digging into hardware specs shows this does not
actually make QEMU behave more like hardware.
Let's stick to the tried heuristic for 1.7 and
possibly revisit for 1.8.
Reported-by: Vlad Yasevich
Cc: Amos Kong
Cc: Alex Williamson
---
On 11/18/2013 02:42 PM, Michael S. Tsirkin wrote:
> On Mon, Nov 18, 2013 at 12:33:20PM -0500, Vlad Yasevich wrote:
>> On 11/18/2013 10:02 AM, Michael S. Tsirkin wrote:
>>> On Tue, Nov 05, 2013 at 07:17:18PM +0800, Amos Kong wrote:
We currently just update the HMP NIC info when the last bit of
On Mon, Nov 18, 2013 at 12:33:20PM -0500, Vlad Yasevich wrote:
> On 11/18/2013 10:02 AM, Michael S. Tsirkin wrote:
> > On Tue, Nov 05, 2013 at 07:17:18PM +0800, Amos Kong wrote:
> >> We currently just update the HMP NIC info when the last bit of macaddr
> >> is written. This assumes that guest driv
On 11/18/2013 11:41 AM, Varad Gautam wrote:
> On Mon, Nov 18, 2013 at 10:31 PM, Eric Blake wrote:
>> Varad, as a new contributor, you are in a position to possibly help us:
>> what documentation pages did you read to learn where to post your
>> patches, so that we can try and modify those pages t
On Mon, Nov 18, 2013 at 10:31 PM, Eric Blake wrote:
> Often-times, this happens when a new contributor fails to realize that
> the list is moderated, but that we will moderate non-subscriber's
> messages through. What commonly happens is that a new contributor sends
> a message, sees nothing on t
On 11/18/2013 10:02 AM, Michael S. Tsirkin wrote:
> On Tue, Nov 05, 2013 at 07:17:18PM +0800, Amos Kong wrote:
>> We currently just update the HMP NIC info when the last bit of macaddr
>> is written. This assumes that guest driver will write all the macaddr
>> from bit 0 to bit 5 when it changes th
On 11/18/2013 09:45 AM, Andreas Färber wrote:
> Am 18.11.2013 16:59, schrieb Varad Gautam:
>>> To discuss the code you really should post the patch, I couldn't
>>> find any related code by your link.
>>
>> Hi! I had posted the patch onto the list.
>
> Unfortunately several copies of your mail arri
Am 18.11.2013 17:48, schrieb Stefan Hajnoczi:
> On Mon, Nov 18, 2013 at 05:36:34PM +0100, Andreas Färber wrote:
>> This avoids each test needing to add it to suppress windows popping up.
>>
>> Signed-off-by: Andreas Färber
>> ---
>> tests/boot-order-test.c | 2 +-
>> tests/endianness-test.c | 6 +
On Mon, Nov 18, 2013 at 05:36:34PM +0100, Andreas Färber wrote:
> This avoids each test needing to add it to suppress windows popping up.
>
> Signed-off-by: Andreas Färber
> ---
> tests/boot-order-test.c | 2 +-
> tests/endianness-test.c | 6 +++---
> tests/fw_cfg-test.c | 3 +--
> tests/hd-
Am 18.11.2013 16:59, schrieb Varad Gautam:
>> To discuss the code you really should post the patch, I couldn't
>> find any related code by your link.
>
> Hi! I had posted the patch onto the list.
Unfortunately several copies of your mail arrived on the list today, and
only one actually has the pa
@FanFan,
if you start such a vm and do 'ps -ef | grep kvm' should see the kvm
command line which is working for you.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/685096
Title:
USB Passthrough not
Il 18/11/2013 16:03, Stefan Hajnoczi ha scritto:
> As of today, there are still two vring implementations in
> hw/virtio/virtio.c and hw/virtio/dataplane/vring.c. This means it isn't
> clean and easy to integrate into a new device yet. Existing dataplane
> devices basically take advantage of the
This avoids each test needing to add it to suppress windows popping up.
Signed-off-by: Andreas Färber
---
tests/boot-order-test.c | 2 +-
tests/endianness-test.c | 6 +++---
tests/fw_cfg-test.c | 3 +--
tests/hd-geo-test.c | 2 +-
tests/i440fx-test.c | 2 +-
tests/libqtest.c|
On Nov 18, 2013 12:20 AM, "Peter Lieven" wrote:
>
> vnc_update_client currently scans the dirty bitmap of each client
> bitwise which is a very costly operation if only few bits are dirty.
> vnc_refresh_server_surface does almost the same.
> this patch optimizes both by utilizing the heavily optim
On Nov 18, 2013 7:05 AM, "Stefan Hajnoczi" wrote:
>
> On Mon, Nov 18, 2013 at 02:52:53PM +1000, Dave Airlie wrote:
> > So after talking to a few people at kvm forum I think the GPU code
> > should probably use the dataplane stuff from the outset,
> >
> > The main advantages I think this gives me i
Il 18/11/2013 16:37, Peter Lieven ha scritto:
>>
>>
>> If I specify: -smp 2,sockets=1,cores=2,threads=1 to a Windows 2012 R2
>> Server it crashes
>> at boot time. -smp 2 works.
> for Linux /proc/cpuinfo reveals no cpu layout information (sibliings,
> cores, threads etc.) with
> this patch applied a
Il 18/11/2013 13:48, Peter Lieven ha scritto:
> On 24.10.2013 11:14, Paolo Bonzini wrote:
>> Il 24/10/2013 08:21, Peter Lieven ha scritto:
>>> Additionally we memmap target memory so it is essentially
>>> zero initialized (except for e.g. option roms and bios which are loaded
>>> into target memory
Hi,
Am 15.11.2013 20:54, schrieb Kevin Wolf:
> Without it, you either get a window for a short time, or worse, test
> failures when 'make check' isn't run in an X session.
>
> Signed-off-by: Kevin Wolf
> ---
> tests/blockdev-test.c | 2 +-
> tests/qdev-monitor-test.c | 2 +-
> 2 files chang
Incomplete patch.
Usecase: `qemu-img convert` with -p now shows the write speed.
1. I'm calculating the speed using the time taken to run the for(;;)
at qemu-img.c:1477. I figured that every time this loop runs, n1
sectors are converted, and so I calculate the write_speed
accordingly. Is this cor
> To discuss the code you really should post the patch, I couldn't
> find any related code by your link.
Hi! I had posted the patch onto the list. I have also put it at [1] now.
Thanks.
Varad
[1] https://bugs.launchpad.net/qemu/+bug/603872
Am 18.11.2013 16:32, schrieb Amos Kong:
> object_get_canonical_path() returns a gchar*, it should be freeed by the
> caller.
>
Cc: qemu-sta...@nongnu.org
> Signed-off-by: Amos Kong
> ---
> v2: put gchar *path inside rxfilter_notify_enabled block
> ---
> hw/net/virtio-net.c | 8
> 1 fi
1 - 100 of 203 matches
Mail list logo