Hi,
Since updating to QEMU v1.0 I've had a problem with automatic mouse
grab. Whenever I move the mouse over the QEMU window it seems to make
the mouse cursor invisible, but the input device being emulated is a
touchscreen providing absolute coordinates so the software in the
emulator doesn't
Sirs,
I've been trying to debug a problem with Solaris 8 running on
sparc-softmmu. The syslog daemon in very unreliable (about 7 of 8 starts of
the syslog daemon end in a daemon hang - the daemon can be killed and
restarted manually).
*Background:* I looked at the syslogd.c code on the Oracle
Il 17/05/2012 02:24, Benjamin Herrenschmidt ha scritto:
Also, should I make the barrier conditional to kvm_enabled() ? IE. It's
pointless in full emulation and might actually be a performance hit on
something already quite slow...
Finally ... something like smp_mb() in qemu will turn into a
Il 18/05/2012 01:59, Zhang, Yang Z ha scritto:
Thanks for reviewing it! It's ok for me.
Will you pay effort to put it into 1.2? I will greatly appreciate it if you
can do it.:)
Yes, I can do that.
Paolo
On Fri, 2012-05-18 at 08:53 +0200, Paolo Bonzini wrote:
It depends on what semantics you attach to dma_mb. In my opinion,
having a separate barrier for DMA is wrong, because you want the same
semantics on all architectures.
The x86 requirements are roughly as follows:
1) it never needs
Il 18/05/2012 10:18, Benjamin Herrenschmidt ha scritto:
On Fri, 2012-05-18 at 08:53 +0200, Paolo Bonzini wrote:
It depends on what semantics you attach to dma_mb. In my opinion,
having a separate barrier for DMA is wrong, because you want the same
semantics on all architectures.
The x86
On Tue, May 15, 2012 at 11:21:13AM +0800, dunrong huang wrote:
Thanks for your reply.
As you say, for an input visitor we dont need to initialize the pointer.
visit_type_str in set_mac function and set_pci_devfn function is a input
visitor, it points to
qmp_input_type_str, if
Am 15.05.2012 13:08, schrieb Peter Maydell:
On 14 May 2012 18:31, Andreas Färber afaer...@suse.de wrote:
The board initializes only one CPU, so replace first_cpu with that CPU's
state to facilitate review of arm_load_kernel() signature change and to
avoid double casts then.
There was a
This breakage was introduced by the commit memory: make
phys_page_find() return an unadjusted.
Signed-off-by: TeLeMan gele...@gmail.com
---
exec.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/exec.c b/exec.c
index 0607c9b..ad99476 100644
--- a/exec.c
+++ b/exec.c
@@
On Thu, May 17, 2012 at 03:37:49PM +1000, Peter A. G. Crosthwaite wrote:
Implemented the swapb and swaph byte/halfword reversal instructions added
to microblaze v8.30
Signed-off-by: Peter A. G. Crosthwaite peter.crosthwa...@petalogix.com
Applied, thanks Peter
---
Hi Jia.
+ case 0x0009:
+ switch (op1) {
+ case 0x03: /*l.div*/
+ LOG_DIS(l.div r%d, r%d, r%d\n, rd, ra, rb);
+ if (rb != 0) {
+ tcg_gen_div_tl(cpu_R[rd], cpu_R[ra], cpu_R[rb]);
You also need to take care of integer overflow/division
On Thu, May 17, 2012 at 03:37:50PM +1000, Peter A. G. Crosthwaite wrote:
Signed-off-by: Peter A. G. Crosthwaite peter.crosthwa...@petalogix.com
---
target-microblaze/cpu.h |1 +
target-microblaze/helper.c|2 +
target-microblaze/translate.c | 51
On Thu, May 17, 2012 at 08:14:15AM -0600, Eric Blake wrote:
On 05/17/2012 07:42 AM, Stefan Hajnoczi wrote:
The -open-hook-fd approach allows QEMU to support file descriptor passing
without changing -drive. It also supports snapshot_blkdev and other
commands
By the way, How will it
On Thu, May 17, 2012 at 10:02:01PM +0800, Zhi Yong Wu wrote:
On Thu, May 17, 2012 at 9:42 PM, Stefan Hajnoczi
stefa...@linux.vnet.ibm.com wrote:
On Fri, May 04, 2012 at 11:28:47AM +0800, Zhi Yong Wu wrote:
On Tue, May 1, 2012 at 11:31 PM, Stefan Hajnoczi
stefa...@linux.vnet.ibm.com wrote:
Split IN_T into BSIZE and ITYPE, to avoid expansion if the OS has
defined macros for the intX_t and uintX_t types. The IN_T constant is
then defined in mixeng_template.h so it can be used by the
functions/macros on this header file.
This change has been tested successfully under Debian Linux and
The str allocated in visit_type_str was not freed.
The visit_type_str function is an input visitor(QMP/String/etc-to-native)
here, it will allocate memory for caller, so the caller is responsible for
freeing the memory.
Signed-off-by: dunrong huang riegama...@gmail.com
---
hw/qdev-properties.c
On Fri, 18 May 2012, Roger Pau Monne wrote:
Split IN_T into BSIZE and ITYPE, to avoid expansion if the OS has
defined macros for the intX_t and uintX_t types. The IN_T constant is
then defined in mixeng_template.h so it can be used by the
functions/macros on this header file.
This change
Hi Aurelien,
I found that when qemu-system-mips64el executed 'movz' with -M mips, it
would raise a reserved instruction exception.
The mips spec describes movz as below:
Mnemonic Instructio Defined in MIPS ISA
MOVZMove Conditional on Zero
Am 15.05.2012 13:14, schrieb Peter Maydell:
On 14 May 2012 18:31, Andreas Färber afaer...@suse.de wrote:
Following up on your remark about ugly naming as a consequence of my CPU
reset
patches and my OMAP field rename, here's a few more patches.
Patch 1 is spelling fixes in a comment and
On 18 May 2012 13:13, Andreas Färber afaer...@suse.de wrote:
Patch 1 is irrelevant to my refactorings so I'd ask you, Peter, to queue
it in case some other ARM patch for 1.1 comes along.
I would not at this point put patch 1 in for 1.1 even if there was
some other target-arm patch that caused
On 2 May 2012 18:12, Peter Maydell peter.mayd...@linaro.org wrote:
This patch series refactors the ARMv7M NVIC so it no longer
textually includes arm_gic.c Instead we have a single common
base class which has the state struct, and a subclass for
the GIC and another for the NVIC.
As well as
Am 02.05.2012 19:12, schrieb Peter Maydell:
Remove some NVIC ifdefs from the gic_state struct and its
state save/load functions. This means there are some fields
in it which are present for the NVIC but not used, but means
it always has the same layout and can be pulled out into a
common
Drop All rights reserved from all exynos-related file's licenses, I've sent
the same
patch a while ago. I think it'll be nice to fix license issues before next QEMU
release.
Igor Mitsyanko (1):
exynos4210: drop All rights reserved line from files license
hw/exynos4210.c |2 +-
It has been noted that All rights reserved statement conflicts with GPL,
remove it.
Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com
---
hw/exynos4210.c |2 +-
hw/exynos4210.h |2 +-
hw/exynos4210_combiner.c |3 +--
hw/exynos4210_fimd.c |3 +--
Am 02.05.2012 19:12, schrieb Peter Maydell:
Drop the special casing of NCPU=1 for the NVIC. This slightly
increases the amount of memory used by its state structure,
but removes some ifdeffery and means we can safely move the
GIC state into a common subclass structure.
Signed-off-by: Peter
On Mon, May 7, 2012 at 3:30 PM, Anthony Liguori anth...@codemonkey.ws wrote:
On 05/06/2012 09:39 AM, Avi Kivity wrote:
On 05/06/2012 05:35 PM, Anthony Liguori wrote:
So what's really the use case here? Would an IPMI - libvirt bridge get you
what you need? I really think that's the best path
On Thu, 17 May 2012 16:20:42 -0500
Anthony Liguori anth...@codemonkey.ws wrote:
Hmm, that's a good point, but my concern was that if we only emit
the event when the target is reached, what happens if the guest
gets very close to the target but never actually reaches it for
some reason.
malc wrote:
On Fri, 18 May 2012, Roger Pau Monne wrote:
Split IN_T into BSIZE and ITYPE, to avoid expansion if the OS has
defined macros for the intX_t and uintX_t types. The IN_T constant is
then defined in mixeng_template.h so it can be used by the
functions/macros on this header file.
This
On 18 May 2012 14:01, Andreas Färber afaer...@suse.de wrote:
Am 02.05.2012 19:12, schrieb Peter Maydell:
Drop the special casing of NCPU=1 for the NVIC. This slightly
increases the amount of memory used by its state structure,
but removes some ifdeffery and means we can safely move the
GIC
Am 18.05.2012 14:58, schrieb Igor Mitsyanko:
It has been noted that All rights reserved statement conflicts with GPL,
remove it.
Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com
---
[...]
diff --git a/hw/exynos4210_combiner.c b/hw/exynos4210_combiner.c
index 80af22c..a4a159a 100644
On 05/18/2012 05:26 PM, Andreas Färber wrote:
Am 18.05.2012 14:58, schrieb Igor Mitsyanko:
It has been noted that All rights reserved statement conflicts with GPL,
remove it.
Signed-off-by: Igor Mitsyankoi.mitsya...@samsung.com
---
[...]
diff --git a/hw/exynos4210_combiner.c
Suggest to change it to 2011-2012, according to real development frame.
Thanks,
Dmitry Solodkiy,
Emulator/Kernel PL, Mobile Group,
Moscow RD center, Samsung Electronics
// please use tnef utility to extract Outlook attachments on Linux
-Original Message-
From: Andreas
On 05/17/12 16:33, Luiz Capitulino wrote:
Note that set_option_parameter() callers still expect automatic error
reporting with QError, so set_option_parameter() calls
qerror_report_err() to keep the same semantics.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
qemu-option.c |
On 05/17/12 16:33, Luiz Capitulino wrote:
@@ -291,8 +290,10 @@ int set_option_parameter(QEMUOptionParameter *list,
const char *name,
break;
case OPT_SIZE:
-if (parse_option_size(name, value, list-value.n) == -1)
-return -1;
+
On 05/17/12 16:33, Luiz Capitulino wrote:
@@ -1060,21 +1060,18 @@ int qemu_opts_validate(QemuOpts *opts, const
QemuOptDesc *desc)
}
}
if (desc[i].name == NULL) {
-qerror_report(QERR_INVALID_PARAMETER, opt-name);
-return -1;
+
This is an alternative implementation of writethrough caching. By always
opening protocols in writethrough mode and doing flushes manually after
every write, it achieves two results: 1) it makes flipping the cache mode
extremely easy; 2) it lets formats control flushes during metadata updates
We want to make the formats handle their own flushes
autonomously, while keeping for guests the ability to use a writethrough
cache. Since formats will write metadata via bs-file, bdrv_co_do_writev
is the only place where we need to add a flush.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Enabling or disabling the write cache is done with the SET FEATURES
command. The command can be issued with sg_sat_set_features from
sg3-utils.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
hw/ide/core.c | 18 +++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff
Let qcow2 caches operate always in writeback mode. The block layer
adds flushes after every guest-initiated data write, and these will
also flush the qcow2 caches to disk.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
block/qcow2-cache.c| 25 ++---
comments in-line
On 05/17/12 16:33, Luiz Capitulino wrote:
do_device_add() and do_netdev_add() call qerror_report_err() to maintain
their QError semantics.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
hw/qdev-monitor.c |7 +--
net.c |5 -
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
block.c |5 +
block.h |1 +
2 files changed, 6 insertions(+)
diff --git a/block.c b/block.c
index e89b35a..47b4a43 100644
--- a/block.c
+++ b/block.c
@@ -2369,6 +2369,11 @@ int bdrv_enable_write_cache(BlockDriverState *bs)
Am 14.05.2012 21:03, schrieb Peter Maydell:
Initial infrastructure for data-driven registration of
coprocessor register implementations.
We still fall back to the old-style switch statements pending
complete conversion of all existing registers.
Signed-off-by: Peter Maydell
On Fri, 18 May 2012 16:03:30 +0200
Laszlo Ersek ler...@redhat.com wrote:
On 05/17/12 16:33, Luiz Capitulino wrote:
@@ -291,8 +290,10 @@ int set_option_parameter(QEMUOptionParameter *list,
const char *name,
break;
case OPT_SIZE:
-if (parse_option_size(name,
On Fri, 18 May 2012 16:43:16 +0200
Laszlo Ersek ler...@redhat.com wrote:
comments in-line
On 05/17/12 16:33, Luiz Capitulino wrote:
do_device_add() and do_netdev_add() call qerror_report_err() to maintain
their QError semantics.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
Am 18.05.2012 14:13, schrieb Andreas Färber:
Am 15.05.2012 13:14, schrieb Peter Maydell:
On 14 May 2012 18:31, Andreas Färber afaer...@suse.de wrote:
Following up on your remark about ugly naming as a consequence of my CPU
reset
patches and my OMAP field rename, here's a few more patches.
On 05/18/2012 08:08 AM, Stefan Hajnoczi wrote:
On Mon, May 7, 2012 at 3:30 PM, Anthony Liguorianth...@codemonkey.ws wrote:
On 05/06/2012 09:39 AM, Avi Kivity wrote:
On 05/06/2012 05:35 PM, Anthony Liguori wrote:
So what's really the use case here? Would an IPMI - libvirt bridge get you
Writing vm state uses bdrv_pwrite, so it will automatically get flushes
in writethrough mode. But doing a flush at the end in writeback mode
is probably a good idea anyway.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
Kind of unrelated, found by inspection.
savevm.c |2 +-
Because the guest will be able to flip enable_write_cache, the actual
state may not match what is used to open the new snapshot.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
block.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/block.c b/block.c
index b3d0054..e89b35a 100644
On 05/18/2012 08:08 AM, Stefan Hajnoczi wrote:
On Mon, May 7, 2012 at 3:30 PM, Anthony Liguorianth...@codemonkey.ws wrote:
On 05/06/2012 09:39 AM, Avi Kivity wrote:
On 05/06/2012 05:35 PM, Anthony Liguori wrote:
So what's really the use case here? Would an IPMI - libvirt bridge get you
Formats are entirely in charge of flushes for metadata writes. For
guest-initiated writes, a writethrough cache is faked in the block layer.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
block.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block.c b/block.c
On 05/17/12 16:33, Luiz Capitulino wrote:
@@ -1244,8 +1247,10 @@ int do_netdev_add(Monitor *mon, const QDict *qdict,
QObject **ret_data)
return -1;
}
-res = net_client_init(opts, 1);
+res = net_client_init(opts, 1, local_err);
if (res 0) {
+
Am 18.05.2012 15:36, schrieb Igor Mitsyanko:
On 05/18/2012 05:26 PM, Andreas Färber wrote:
Am 18.05.2012 14:58, schrieb Igor Mitsyanko:
It has been noted that All rights reserved statement conflicts with
GPL,
remove it.
Signed-off-by: Igor Mitsyankoi.mitsya...@samsung.com
---
[...]
diff
Comments in-line.
On 05/17/12 16:33, Luiz Capitulino wrote:
This is not a full QAPI conversion, but an intermediate step.
In essence, do_netdev_add() is split into three functions:
1. netdev_add(): performs the actual work. This function is fully
converted to Error (thus, it's
Needed for arm_load_kernel().
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/xilinx_zynq.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/xilinx_zynq.c b/hw/xilinx_zynq.c
index 7290c64..68349e2 100644
--- a/hw/xilinx_zynq.c
+++ b/hw/xilinx_zynq.c
@@
Hello Peter,
This series completes the post-reset cleanups. Please ack.
Regards,
Andreas
Cc: Peter Maydell peter.mayd...@linaro.org
Cc: Igor Mitsyanko i.mitsya...@samsung.com
v1 - v2:
* Drop typo patch - PMM does not want it in 1.1 and queued it in arm-devs.next.
* Many patches were
Pass it through to arm_pic_cpu_handler().
Signed-off-by: Andreas Färber afaer...@suse.de
Acked-by: Peter Maydell peter.mayd...@linaro.org
Acked-by: Igor Mitsyanko i.mitsya...@samsung.com (for exynos)
---
hw/arm-misc.h |2 +-
hw/arm_pic.c |8 +---
hw/armv7m.c |2 +-
In particular this simplifies the s-mpu-cpu-env expression again.
first_cpu and -next_cpu are expected to be QOM'ified later.
Signed-off-by: Andreas Färber afaer...@suse.de
Acked-by: Igor Mitsyanko i.mitsya...@samsung.com (for exynos)
---
hw/arm-misc.h |2 +-
hw/arm_boot.c |
On 05/17/12 16:33, Luiz Capitulino wrote:
diff --git a/hmp.c b/hmp.c
index 7a4e25f..2ce8cb9 100644
--- a/hmp.c
+++ b/hmp.c
@@ -990,3 +990,12 @@ void hmp_netdev_add(Monitor *mon, const QDict *qdict)
out:
hmp_handle_error(mon, err);
}
+
+void hmp_netdev_del(Monitor *mon, const
Hi all,
the current block job API is designed for streaming; one property of
streaming is that in case of an error it can be restarted from the point
where it was left.
In QEMU 1.2 I would like to add an implementation of mirroring (live
block copy) based on the block job API and on dirty-block
On Fri, 18 May 2012 17:51:01 +0200
Laszlo Ersek ler...@redhat.com wrote:
+void hmp_netdev_add(Monitor *mon, const QDict *qdict)
+{
+Error *err = NULL;
+QemuOpts *opts;
+
+opts = qemu_opts_from_qdict(qemu_find_opts(netdev), qdict, err);
I note we trust
Il 18/05/2012 19:41, Luiz Capitulino ha scritto:
Other than that I think we should have a better separation between front-ends
and back-ends in that code (we seem to call everything a type).
That makes sense if you consider the way -net originally worked with
VLANs...
Paolo
On Fri, 18 May 2012 18:15:34 +0200
Laszlo Ersek ler...@redhat.com wrote:
On 05/17/12 16:33, Luiz Capitulino wrote:
diff --git a/hmp.c b/hmp.c
index 7a4e25f..2ce8cb9 100644
--- a/hmp.c
+++ b/hmp.c
@@ -990,3 +990,12 @@ void hmp_netdev_add(Monitor *mon, const QDict *qdict)
out:
Hi Anthony, Michael,
While at it, could you pick up the attached patch too?
It has already been acked by Igor Kovalenko.
Thanks,
Fernando
Subject: [PATCH] rtl8139: honor RxOverflow flag in can_receive method
From: Fernando Luis Vazquez Cao ferna...@oss.ntt.co.jp
Some drivers (Linux' 8139too
Hello,
There seems to be a bug in qemu in the e1000 emulation, which triggers
an issue with the Linux driver.
What happens in Linux is the following:
- e1000_open
- e1000_configure
- e1000_setup_rctl
- enables E1000_RCTL_EN
- e1000_configure_rx
- sets RDT/RDH to 0
-
On Thu, 2012-05-17 at 17:20 +0800, Liu Ping Fan wrote:
Currently, the guest can not know the NUMA info of the vcpu, which
will
result in performance drawback.
This is the discovered and experiment by
Shirley Ma x...@us.ibm.com
Krishna Kumar krkum...@in.ibm.com
Tom
On Thu, May 17, 2012 at 11:04:55AM +0200, heli...@katamail.com wrote:
Hello,
I am testing Xen 4.2 (unstable) with Qemu 1.1rc2 and spice.
I have severe video performance problems on Windows 7 guest and X fails to
run on
Debian wheezy and Ubuntu Precise guest.
What version of drivers
Hi,
Zhi-zhou Zhang a écrit :
I found that when qemu-system-mips64el executed 'movz' with -M mips,
it would raise a reserved instruction exception.
The mips spec describes movz as below:
Mnemonic Instructio Defined in MIPS ISA
MOVZMove Conditional
Am 18.05.2012 13:14, schrieb dunrong huang:
The str allocated in visit_type_str was not freed.
The visit_type_str function is an input visitor(QMP/String/etc-to-native)
here, it will allocate memory for caller, so the caller is responsible for
freeing the memory.
Signed-off-by: dunrong
On Fri, 2012-05-18 at 10:57 +0200, Paolo Bonzini wrote:
I'm still tempted to add barriers in map and unmap as well in the case
where they don't bounce to provide consistent semantics here, ie, all
accesses done between the map and unmap are ordered vs all previous and
subsequent accesses.
The initial TLB entry is supposed to help us run the guest -kernel payload.
This means the guest needs to be able to access its own memory, the initrd
memory and the device tree.
So far we only statically reserved a TLB entry from [0;256M[. This patch
fixes it to span from [0;dt_end[, allowing
On 05/18/2012 06:02 PM, Alex Williamson wrote:
On Wed, 2012-05-16 at 09:29 -0400, Don Dutile wrote:
On 05/15/2012 05:09 PM, Alex Williamson wrote:
On Tue, 2012-05-15 at 13:56 -0600, Bjorn Helgaas wrote:
On Mon, May 14, 2012 at 4:49 PM, Alex Williamson
alex.william...@redhat.com wrote:
On
Hi Zhi-zhou Zhang,
You are correct. This should not be a reserved instruction exception on MIPS64
(nor should MOVN).
-Eric
From: qemu-devel-bounces+ericj=mips@nongnu.org
[mailto:qemu-devel-bounces+ericj=mips@nongnu.org] On Behalf Of Zhi-zhou
Zhang
Sent: Friday, May 18, 2012 4:39 AM
Hi EricZhang,
Nope, you two are wrong. Aurelien is right.
Let me show you the code.
1, qemu/hw/mips_r4k.c --forqemu-system-mips4el -M mipsboard.
void mips_r4k_init (...)
{
...
/* init CPUs */
if (cpu_model == NULL) {
#ifdef TARGET_MIPS64
cpu_model = R4000;
#else
On Fri, 2012-05-18 at 19:00 -0400, Don Dutile wrote:
On 05/18/2012 06:02 PM, Alex Williamson wrote:
On Wed, 2012-05-16 at 09:29 -0400, Don Dutile wrote:
On 05/15/2012 05:09 PM, Alex Williamson wrote:
On Tue, 2012-05-15 at 13:56 -0600, Bjorn Helgaas wrote:
On Mon, May 14, 2012 at 4:49 PM,
74 matches
Mail list logo