On 04/05/2010 12:53 AM, Paul Brook wrote:
Surprising as there are ~10 descriptors being
polled, so ~1200 polls per second. Maybe epoll will help here.
I'm not sure where you get 1200 from. select will be called once per host
wakeup. i.e. if the USB controller is enabled then 1k times
On Wed, Mar 31, 2010 at 02:46:16PM +0200, Kevin Wolf wrote:
What is known today as bdrv_open2 becomes the new bdrv_open. All remaining
callers of the old function are converted to the new one. In some places they
even know the right format, so they should have used bdrv_open2 from the
On Wed, Mar 31, 2010 at 03:12:56PM +0200, Kevin Wolf wrote:
If I can have my pony, your patch is better than current behaviour :)
Feel free to post a v2 that implements your version. :-)
No real need for that, it can also be done as a separate path on top
of this one.
Note that there's also
On Sat, 3 Apr 2010, Paul Brook wrote:
This is my first contribution to QEMU: I've written a virtual piece of
hardware (e.g. ports) that the mouse pointer can be used as an absolute
pointing device even with non USB devices, e.g. with DOS and doesn't need
any high memory usage USB driver. This
Hi.
Could someone pick this patch set?
I have another patch set which is based on this.
If there are any issues, I'm happy to fix them.
Thanks,
Yoshi
Avi Kivity wrote:
On 03/23/2010 09:39 AM, Yoshiaki Tamura wrote:
Before replacing byte-based dirty bitmap with bit-based dirty bitmap,
On Mon, 5 Apr 2010, Paul Brook wrote:
I think I got all the tcg backend bits right, but can't test most of them.
Please test and report any problems.
Historically the qemu tlb addend field was used for both RAM and IO
accesses,
so needed to be able to hold both host addresses (unsigned
Hello,
(This series is based on the previous series that I've sent out for
virtio-serial).
This patch series adds fixes for the chardev interface to let callers
of qemu_chr_write() know how many bytes were sent and how many are
unsent in case of errors.
In case of EAGAIN, the unix_write()
There might be cases where a few bytes would have been sent out to char
devices and some not. Currently the return values from qemu_chr_write()
to char devs are only -1, indicating an error, or the complete length
of the string passed.
Make 'len' a pointer instead, and indicate how much of the
When the other end of a chardev connection isn't picking up data as
fast as we're sending, we just used to keep spinning in a tight loop
till all the data was sent out.
Polling for POLLOUT indefinitely gives the other end a chance to catch
up and also saves us CPU cycles.
Signed-off-by: Amit
If the chardev we're writing to is nonblocking, just report -EAGAIN to
the caller so that the caller can take any further action as it may see
fit.
Modify poll call for polling for a timeout of 10ms instead of waiting
indefinitely for POLLOUT to get set.
Signed-off-by: Amit Shah
The initialisation for generic ports and console ports is similar.
Factor out the parts that are the same in a different function that can
be called from each of the initfns.
Signed-off-by: Amit Shah amit.s...@redhat.com
---
hw/virtio-console.c | 31 ++-
1 files
If the char device we're connected to is overwhelmed with data and it
can't accept any more, signal to the virtio-serial-bus to stop sending
us more data till we tell otherwise.
If the current buffer being processed hasn't been completely written out
to the char device, we have to keep it around
Hello,
d6f4ade214a9f74dca9495b83a24ff9c113e4f9a: disentangle tcg and deadline
calculation
introduces following regression(s):
100% cpu utilization when QEMU is invoked like:
qemu -S -s ...
ditto when gdb takes control over the session via gdb-stub
(i.e. the breakpoint is hit or C-c is pressed
Assign directly to the bdrv_flags variable instead of using
magic numbers before translating to the BDRV_O_* options.
Signed-off-by: Christoph Hellwig h...@lst.de
Index: qemu/vl.c
===
--- qemu.orig/vl.c 2010-04-05
Split up the raw_getlength into separate generic, solaris and BSD
versions to reduce the ifdef maze a bit. The BSD variant still
is a complete maze, but to clean it up properly we'd need some
people using the BSD variants to figure out what code is used
for what variant.
Signed-off-by: Christoph
Hi
116348def2bb446d972bdc2f44bd77ff631f85de fails on s390 with the
following error:
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm -kernel
/boot/vmlinuz-2.6.32-4-s390x
| qemu-system-s390x: No 'PCI' bus found for device 'virtio-serial-pci'
After patching the pci virtio drivers
On (Mon) Apr 05 2010 [16:34:13], Bastian Blank wrote:
Hi
116348def2bb446d972bdc2f44bd77ff631f85de fails on s390 with the
following error:
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm -kernel
/boot/vmlinuz-2.6.32-4-s390x
| qemu-system-s390x: No 'PCI' bus found for device
BDRV_O_FILE is only used to communicate between bdrv_file_open and bdrv_open.
It affects two things: first bdrv_open only searches for protocols using
find_protocol instead of all image formats and host drivers. We can easily
move that to the caller and pass the found driver to bdrv_open.
On (Mon) Apr 05 2010 [16:45:21], Bastian Blank wrote:
Hi
116348def2bb446d972bdc2f44bd77ff631f85de works on s390/kvm, but on startup
parts of the console output is lost. The used kernel is a standard Debian
distribution kernel.
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm
On Mon, Apr 05, 2010 at 08:14:00PM +0530, Amit Shah wrote:
On (Mon) Apr 05 2010 [16:34:13], Bastian Blank wrote:
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm -kernel
/boot/vmlinuz-2.6.32-4-s390x
| qemu-system-s390x: No 'PCI' bus found for device 'virtio-serial-pci'
I'm not
There might be cases where a few bytes would have been sent out to char
devices and some not. Currently the return values from qemu_chr_write()
to char devs are only -1, indicating an error, or the complete length
of the string passed.
Make 'len' a pointer instead, and indicate how much of
To do this in any clean sort of way, I rewrote a substantial
portion of elfload.c. Doing otherwise would have wound up
with 3 entirely separate bits of code to load an elf image.
The first patch seems out of order, since I did this work on
a branch that already contained some previously
I caught padzero not properly initializing the .bss segment
on a statically linked Alpha program. Rather than a minimal
patch, replace the gross code with a single mmap+memset.
Share more code between load_elf_interp and load_elf_binary.
Legally, an ELF program need not have just a single .bss;
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 594 +-
1 files changed, 301 insertions(+), 293 deletions(-)
diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index ead22fc..ab741fd 100644
---
Validate more fields of the elf header. Extract those checks
into two common functions to be used in both load_elf_interp
and load_elf_binary.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 57 +-
1 files changed,
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 201 --
1 files changed, 193 insertions(+), 8 deletions(-)
diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index 100efdc..7b854e2 100644
---
Moving some PPC AT_* constants from elfload.c at the same time.
Signed-off-by: Richard Henderson r...@twiddle.net
---
elf.h| 44
linux-user/elfload.c |9 -
2 files changed, 44 insertions(+), 9 deletions(-)
diff --git
Remove ifdefs from code by defining empty inline functions
when byte swapping isn't needed. Push loops over swapping
arrays of structures into the swapping functions.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 119
Alpha is little-endian on Linux.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index 962f9ba..5814702 100644
--- a/linux-user/elfload.c
+++
At the bottom of the a.out support was the unimplemented load_aout_interp
function. There were other portions of the support that didn't look
right; when I went to look in the Linux kernel for clarification, I found
that the support for such interpreters has been removed from binfmt_elf.
There
... Well, sortof. The Makefile bits are broken.
Patch to load the vdso into the running program to follow.
Signed-off-by: Richard Henderson r...@twiddle.net
---
Makefile |3 +-
pc-bios/Makefile |5 ++
pc-bios/vdso-linux-x64.S | 102
Define BPRM_BUF_SIZE to 4k and read that amount initially. If the
data we want from the binary is in this buffer, use it instead of
reading from the file again.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 105 ---
Moving toward a single copy of the elf binary loading code.
Fill in the details of the loaded image into a struct image_info.
Adjust create_elf_tables to read from such structures instead
of from a collection of passed arguments. Don't return error
values from load_elf_interp; always exit(-1)
There are no supported stack-grows-up targets. We were putting
the guard page at the highest address, i.e. the bottom of the stack.
Use the maximum of host and guest page size for the guard size.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 29
First, adjust load_symbols to accept a load_bias parameter. At the same
time, read the entire section header table in one go, use pread instead
f lseek+read for the symbol and string tables, and properly free
allocated structures on error exit paths.
Second, adjust load_elf_interp to compute
This requires moving the PT_INTERP extraction and GUEST_BASE
handling into load_elf_image. Key this off a non-null pointer
argument to receive the interpreter name.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c | 269
On Sat, 3 Apr 2010 18:32:01 -0300
Miguel Di Ciurcio Filho miguel.fi...@gmail.com wrote:
Each device is represented by a QDict. The returned QObject is a QList
of all devices.
This commit should not change user output.
Signed-off-by: Miguel Di Ciurcio Filho miguel.fi...@gmail.com
---
The 32 and 64-bit definitions were swapped in the ifdef.
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-sparc/cpu.h | 12 ++--
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/target-sparc/cpu.h b/target-sparc/cpu.h
index 580f4d4..0e7f390 100644
---
The first patch in the series fixes a major think-o in the sparc port.
The 64 and 32-bit constants were reversed. Fixing these are required
to avoid a build error in later patches.
For the actual problem, introduce a guest_start_len_valid macro similar
to the existing h2g_valid macro, where we
Make sure to properly handle len = 0 first.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/mmap.c | 13 -
1 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/linux-user/mmap.c b/linux-user/mmap.c
index 46923c7..f4d44a8 100644
--- a/linux-user/mmap.c
+++
To be used by userspace emulation to verify that the memory
range defined by [start, start+len) is valid for the guest,
taking into account TARGET_VIRT_ADDR_SPACE_BITS.
Signed-off-by: Richard Henderson r...@twiddle.net
---
cpu-all.h | 10 ++
1 files changed, 10 insertions(+), 0
Signed-off-by: Richard Henderson r...@twiddle.net
---
exec.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/exec.c b/exec.c
index 33854e1..d69194c 100644
--- a/exec.c
+++ b/exec.c
@@ -2461,6 +2461,9 @@ int page_check_range(target_ulong start, target_ulong
len, int
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/mmap.c |8 ++--
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/linux-user/mmap.c b/linux-user/mmap.c
index 463679d..085030b 100644
--- a/linux-user/mmap.c
+++ b/linux-user/mmap.c
@@ -494,12 +494,8 @@ abi_long
On 05.04.2010, at 17:30, Bastian Blank wrote:
On Mon, Apr 05, 2010 at 08:14:00PM +0530, Amit Shah wrote:
On (Mon) Apr 05 2010 [16:34:13], Bastian Blank wrote:
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm -kernel
/boot/vmlinuz-2.6.32-4-s390x
| qemu-system-s390x: No 'PCI' bus
Michael S. Tsirkin schrieb:
On Thu, Apr 01, 2010 at 11:56:25PM +0200, Stefan Weil wrote:
u_int64_t raises compiler error messages:
CC libhw32/virtio.o
/qemu/ar7/hw/virtio.c: In function ‘virtio_queue_get_avail_size’:
/qemu/ar7/hw/virtio.c:776: error: ‘u_int64_t’ undeclared (first use
in
On 05.04.2010, at 16:45, Bastian Blank wrote:
Hi
116348def2bb446d972bdc2f44bd77ff631f85de works on s390/kvm, but on startup
parts of the console output is lost. The used kernel is a standard Debian
distribution kernel.
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm -kernel
On Mon, 5 Apr 2010, Richard Henderson wrote:
This requires moving the PT_INTERP extraction and GUEST_BASE
handling into load_elf_image. Key this off a non-null pointer
argument to receive the interpreter name.
Signed-off-by: Richard Henderson r...@twiddle.net
---
linux-user/elfload.c |
On Mon, 5 Apr 2010, Richard Henderson wrote:
To be used by userspace emulation to verify that the memory
range defined by [start, start+len) is valid for the guest,
taking into account TARGET_VIRT_ADDR_SPACE_BITS.
Signed-off-by: Richard Henderson r...@twiddle.net
---
cpu-all.h | 10
On 04/05/2010 11:15 AM, malc wrote:
Please do not use double leading underscore.
In contrast to the existing use in:
#define h2g_valid(x) ({ \
unsigned long __guest = (unsigned long)(x) - GUEST_BASE; \
What do you suggest instead? Trailing underscores?
r~
On Mon, 5 Apr 2010, Richard Henderson wrote:
On 04/05/2010 11:15 AM, malc wrote:
Please do not use double leading underscore.
In contrast to the existing use in:
#define h2g_valid(x) ({ \
unsigned long __guest = (unsigned long)(x) - GUEST_BASE; \
What do you suggest instead?
On 04/05/2010 11:14 AM, malc wrote:
+}
+interp_name = malloc(eppnt-p_filesz);
malloc can fail
+
+if (eppnt-p_offset + eppnt-p_filesz = BPRM_BUF_SIZE) {
+memcpy(interp_name, bprm_buf + eppnt-p_offset,
+
From: Markus Armbruster arm...@redhat.com
Note: our device unplug methods don't need conversion work, because
they can't currently fail.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
hw/qdev.c |3 +--
1 files changed, 1
From: Markus Armbruster arm...@redhat.com
Fixes device_add to report duplicate ID properly in QMP, as
DuplicateId instead of UndefinedError.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
qemu-option.c |4 ++--
1 files changed, 2
From: Markus Armbruster arm...@redhat.com
The conversion is shallow: client type init() methods aren't
converted. Converting them is a big job for relatively little
practical benefit, so leave it for later.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino
Some people might think that this event is emitted whenever the
time changes, be more specific.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
QMP/qmp-events.txt |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/QMP/qmp-events.txt b/QMP/qmp-events.txt
index
From: Markus Armbruster arm...@redhat.com
Second argument is now on or off instead of up or down.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
net.c | 10 ++
qemu-monitor.hx |8
2 files changed, 6
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
hw/qdev.c |8
hw/qdev.h |2 +-
qemu-monitor.hx |3 ++-
3 files changed, 7 insertions(+), 6 deletions(-)
diff
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
qemu-option.c | 17 +++--
1 files changed, 7 insertions(+), 10 deletions(-)
diff --git a/qemu-option.c b/qemu-option.c
index
From: Markus Armbruster arm...@redhat.com
To make 'b' available for boolean argument.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
monitor.c |8
qemu-monitor.hx |2 +-
2 files changed, 5 insertions(+), 5
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index
From: Markus Armbruster arm...@redhat.com
Both functions report errors nicely enough now, no need for additional
messages.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
hw/pci-hotplug.c |2 --
net.c|2 --
From: Markus Armbruster arm...@redhat.com
This is a boolean value. Human monitor accepts on or off.
Consistent with option parsing (see parse_option_bool()).
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
monitor.c | 31
From: Markus Armbruster arm...@redhat.com
Monitor commands to go with -netdev.
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
net.c | 55 +++
net.h |2 ++
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
qemu-option.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/qemu-option.c b/qemu-option.c
index 394c763..1ffc497
From: Kevin Wolf kw...@redhat.com
virtio_blk_req_complete frees the request, so we can't access it any more when
calling bdrv_mon_event. Use the pointer that was copied earlier.
Signed-off-by: Kevin Wolf kw...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
hw/virtio-blk.c
Hi Anthony,
I was going to nag you to apply patches from the list, but as I'm maintaining
a QMP/Monitor queue to test them, I thought it would be a good idea to nag you
in the form of a git pull :)
All the patches in there have been posted to the list already and, to make it
convenient to
Trivial, as it never fails, doesn't have output nor return
any data.
Note that it's also available under QMP, as kvm-autotest
needs this.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
monitor.c |3 ++-
qemu-monitor.hx |3 ++-
2 files changed, 4 insertions(+), 2
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
net.c |7 ---
net.h |2 +-
qemu-monitor.hx |3 ++-
3 files changed, 7 insertions(+), 5 deletions(-)
diff
From: Markus Armbruster arm...@redhat.com
Signed-off-by: Markus Armbruster arm...@redhat.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
hw/qdev.c |2 +-
monitor.c |6 --
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
index
Howdy,
I've been thinking a bit further on the whole issue around libvirt and why the
situation as is isn't satisfying. I came to the following points that currently
hurt building ease of use for KVM:
1) Brand
This is one of the major issues we have ourselves when it comes to appliances.
We
On Mon, Apr 05, 2010 at 08:01:16PM +0200, Stefan Weil wrote:
Michael S. Tsirkin schrieb:
On Thu, Apr 01, 2010 at 11:56:25PM +0200, Stefan Weil wrote:
u_int64_t raises compiler error messages:
CC libhw32/virtio.o
/qemu/ar7/hw/virtio.c: In function ‘virtio_queue_get_avail_size’:
On 05.04.2010, at 14:41, malc wrote:
On Mon, 5 Apr 2010, Paul Brook wrote:
I think I got all the tcg backend bits right, but can't test most of them.
Please test and report any problems.
Historically the qemu tlb addend field was used for both RAM and IO
accesses,
so needed to be
From: Ryan Harper ry...@us.ibm.com
Currently when using the change command to switch the file in the cd drive
the command doesn't complain if the file doesn't exit or can't be opened
and the drive keeps the existing image. This patch adds a qerror_report
call to print a message out indicating
Hi,
I think the correct way to get the full instruction trace on a MIPS emulated
processor is:
-Disabling the tb cache: I did this by modifying the tb_find_slow() and
tb_find_fast() functions to ever go to not_found label where the code is
translated with no cache searches.
-Loggin the
On 04/06/2010 12:11 AM, Alexander Graf wrote:
Howdy,
I've been thinking a bit further on the whole issue around libvirt and why the
situation as is isn't satisfying. I came to the following points that currently
hurt building ease of use for KVM:
1) Brand
This is one of the major issues we
On 06.04.2010, at 00:14, Avi Kivity wrote:
On 04/06/2010 12:11 AM, Alexander Graf wrote:
Howdy,
I've been thinking a bit further on the whole issue around libvirt and why
the situation as is isn't satisfying. I came to the following points that
currently hurt building ease of use for
On 04/05/2010 03:09 PM, Boris Cámara wrote:
I think the correct way to get the full instruction trace on a MIPS
emulated processor is:
-singlestep -d exec
That gives you the address of each instruction executed.
I'm not sure what else you want than this, as you havn't said.
r~
On Mon, 5 Apr 2010, Alexander Graf wrote:
On 05.04.2010, at 14:41, malc wrote:
On Mon, 5 Apr 2010, Paul Brook wrote:
I think I got all the tcg backend bits right, but can't test most of them.
Please test and report any problems.
Historically the qemu tlb addend field was used
The Problem:
CONFIG_USER_ONLY kinda sorta tries to manage the distinction between
qemu memory and guest memory. This can be seen in the PAGE_RESERVED
frobbing and qemu_malloc etc. However, it doesn't handle random malloc
calls eg from libc itself or other libraries in use.
Possible solutions:
Generic code handles -d cpu in cpu_exec, amidst the horde of ifdefs.
These duplicates simply double the amount of output produced with no
extra information emitted.
Maintainers for mips, ppc and sh4 cc'd.
r~
Richard Henderson (4):
target-i386: Remove duplicate CPU log.
target-mips:
The proper logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-i386/translate.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/target-i386/translate.c b/target-i386/translate.c
index 28d9940..66d8e75 100644
---
Logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-ppc/translate.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index 0af7e4f..3a6146a 100644
---
Logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-mips/translate.c |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/target-mips/translate.c b/target-mips/translate.c
index 0ade3bd..b796b4f 100644
---
Logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-sh4/translate.c |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/target-sh4/translate.c b/target-sh4/translate.c
index bff3188..391b762 100644
---
On Mon, Apr 05, 2010 at 03:45:23PM -0700, Richard Henderson wrote:
The Problem:
CONFIG_USER_ONLY kinda sorta tries to manage the distinction between
qemu memory and guest memory. This can be seen in the PAGE_RESERVED
frobbing and qemu_malloc etc. However, it doesn't handle random malloc
I need the PC and the executated instruction to extract a complete trace of the
execution in a file.
I found since the translation block is cached the -d in_asm option with not
output a complete execution trace. Only when the TB is cached is logged.
What does -singlestep means?
thanks,
On Mon, 5 Apr 2010, Richard Henderson wrote:
Generic code handles -d cpu in cpu_exec, amidst the horde of ifdefs.
These duplicates simply double the amount of output produced with no
extra information emitted.
Maintainers for mips, ppc and sh4 cc'd.
Fine by me, though i'm maintaining none
The dirty and non-dirty pages are checked one by one. When most of the memory
is not dirty, checking the dirty and non-dirty pages by multiple page size
should be much faster than checking them one by one. We introduced bit-based
phys_ram_dirty for VGA, CODE, MIGRATION, MASTER, and
Modifies ram_save_block() and ram_save_remaining() to use
cpu_physical_memory_get_dirty_range() to check multiple dirty and non-dirty
pages at once.
Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp
Signed-off-by: OHMURA Kei ohmura@lab.ntt.co.jp
---
arch_init.c | 54
Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp
Signed-off-by: OHMURA Kei ohmura@lab.ntt.co.jp
---
cpu-all.h | 81 ++--
1 files changed, 67 insertions(+), 14 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index
Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp
Signed-off-by: OHMURA Kei ohmura@lab.ntt.co.jp
---
cpu-all.h | 10 --
1 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index f8bfa66..c409fad 100644
--- a/cpu-all.h
+++ b/cpu-all.h
@@
Modifies kvm_physical_sync_dirty_bitmap to use
cpu_physical_memory_set_dirty_range() to update the row of
the bit-based phys_ram_dirty bitmap at once.
Signed-off-by: OHMURA Kei ohmura@lab.ntt.co.jp
Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp
---
bswap.h |2 ++
I think the correct way to get the full instruction trace on a MIPS
emulated processor is:
the way you describe is slow because you are constantly re-generating the
TBs. The best way to do this is to add your instrumentation to the TBs.
I have code that does that for a recent version of
On (Mon) Apr 05 2010 [17:33:38], Paul Brook wrote:
There might be cases where a few bytes would have been sent out to char
devices and some not. Currently the return values from qemu_chr_write()
to char devs are only -1, indicating an error, or the complete length
of the string passed.
On (Mon) Apr 05 2010 [19:58:35], Alexander Graf wrote:
On 05.04.2010, at 17:30, Bastian Blank wrote:
On Mon, Apr 05, 2010 at 08:14:00PM +0530, Amit Shah wrote:
On (Mon) Apr 05 2010 [16:34:13], Bastian Blank wrote:
| $ ./s390x-softmmu/qemu-system-s390x -nographic -enable-kvm -kernel
My question was not an april fools' day joke. You are developers and answer to
this question may seem obvious to you, but it is not obvious to me. Please
answer. Thanks
Hi,
I always thought that the stop command provided by the monitor
interface would pause the VM completely, but it doesnt seem so?
I checked this by issuing the stop command on my VM, and noted its
clock. Few minutes later, I resumed the VM (with cont command). The
clock is immediately updated
Please send in any agenda items you are interested in covering.
thanks,
-chris
99 matches
Mail list logo