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 p
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
> begin
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 a
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 is
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,
clearin
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 (uns
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() functi
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 str
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 Shah
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
---
qemu-char.c
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
---
hw/virtio-console.c | 31 ++-
1 files changed, 14 insertions(+
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 a
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 i
Assign directly to the bdrv_flags variable instead of using
magic numbers before translating to the BDRV_O_* options.
Signed-off-by: Christoph Hellwig
Index: qemu/vl.c
===
--- qemu.orig/vl.c 2010-04-05 11:05:39.042010326 +0200
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 away
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
/boot/vmlinuz-2.6.32-4-s390x -initrd /boot/initrd.img-
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
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. Second
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
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'
>
> 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 muc
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 submitted
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
---
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
--- a/linux-user/elfload.c
+++ b/linux-
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
---
linux-user/elfload.c | 57 +-
1 files changed, 33 insertions(+),
Signed-off-by: Richard Henderson
---
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
--- a/linux-user/elfload.c
+++ b/linux-us
Moving some PPC AT_* constants from elfload.c at the same time.
Signed-off-by: Richard Henderson
---
elf.h| 44
linux-user/elfload.c |9 -
2 files changed, 44 insertions(+), 9 deletions(-)
diff --git a/elf.h b/elf.h
inde
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
---
linux-user/elfload.c | 119 --
1 files cha
Alpha is little-endian on Linux.
Signed-off-by: Richard Henderson
---
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
+++ b/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 doe
... Well, sortof. The Makefile bits are broken.
Patch to load the vdso into the running program to follow.
Signed-off-by: Richard Henderson
---
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
---
linux-user/elfload.c | 105 ---
linux-user/linuxlo
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) wit
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
---
linux-user/elfload.c | 29 -
1 f
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 load
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
---
linux-user/elfload.c | 269 ++
1 files chang
On Sat, 3 Apr 2010 18:32:01 -0300
Miguel Di Ciurcio Filho 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
> ---
>
> This is my initial contribution, aim
The 32 and 64-bit definitions were swapped in the ifdef.
Signed-off-by: Richard Henderson
---
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
--- a/target-sparc/cpu.h
+++
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 co
Also remove an assertion on start being in range. The values
here can come directly from the guest via a syscall, and so
very well may be out of range via plain bug or DoS attack.
Signed-off-by: Richard Henderson
---
exec.c |9 +
1 files changed, 1 insertions(+), 8 deletions(-)
dif
Make sure to properly handle len = 0 first.
Signed-off-by: Richard Henderson
---
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
+++ b/linux-user/mma
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
---
cpu-all.h | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git
Signed-off-by: Richard Henderson
---
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 flags)
assert(s
Signed-off-by: Richard Henderson
---
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 target_mmap(abi_
Also properly signal error for non-page aligned inputs
and zero sized outputs.
Signed-off-by: Richard Henderson
---
linux-user/mmap.c | 44
1 files changed, 24 insertions(+), 20 deletions(-)
diff --git a/linux-user/mmap.c b/linux-user/mmap.c
index
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: N
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 u
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 -
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
> ---
> linux-user/elfload.c | 269
> ++
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
> ---
> 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 sugg
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
Note: our device unplug methods don't need conversion work, because
they can't currently fail.
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
hw/qdev.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index 9fb817e..97e8d4a 100644
--- a/qerror.c
+++ b/qerror.c
@@ -121,6 +121,1
From: Markus Armbruster
Add suitable comments to help keerp them in order.
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qerror.c | 12
qerror.h |8 +---
2 files changed, 13 insertions(+), 7 deletions(-)
diff --git a/qerror.c b/qerror.c
index eaa1
From: Markus Armbruster
Fixes device_add to report duplicate ID properly in QMP, as
DuplicateId instead of UndefinedError.
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qemu-option.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/qemu-option.
From: Markus Armbruster
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
Signed-off-by: Luiz Capitulino
---
net.c | 38 ++
Some people might think that this event is emitted whenever the
time changes, be more specific.
Signed-off-by: Luiz Capitulino
---
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 a94e9b4..c084a47 100644
--
From: Markus Armbruster
Second argument is now "on" or "off" instead of "up" or "down".
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
net.c | 10 ++
qemu-monitor.hx |8
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/net.c
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
hw/qdev.c |8
hw/qdev.h |2 +-
qemu-monitor.hx |3 ++-
3 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
index 0612dc0..5ca126b 100644
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qemu-option.c | 17 +++--
1 files changed, 7 insertions(+), 10 deletions(-)
diff --git a/qemu-option.c b/qemu-option.c
index 12ce322..394c763 100644
--- a/qemu-option.c
+++ b/qemu-option.
From: Markus Armbruster
To make 'b' available for boolean argument.
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
monitor.c |8
qemu-monitor.hx |2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/monitor.c b/monitor.c
index 792c8c
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index 4520b0d..9fb817e 100644
--- a/qerror.c
+++ b/qerror.c
@@ -97,6 +97,10
From: Markus Armbruster
Both functions report errors nicely enough now, no need for additional
messages.
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
hw/pci-hotplug.c |2 --
net.c|2 --
qemu-config.c|1 -
vl.c |5 -
4 file
From: Markus Armbruster
This is a boolean value. Human monitor accepts "on" or "off".
Consistent with option parsing (see parse_option_bool()).
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
monitor.c | 31 +++
1 files changed, 31 insertions
From: Markus Armbruster
Monitor commands to go with -netdev.
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
net.c | 55 +++
net.h |2 ++
qemu-monitor.hx | 30 ++
3
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index 97e8d4a..8d885cd 100644
--- a/qerror.c
+++ b/qerror.c
@@ -69,6 +69,10
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
qemu-option.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/qemu-option.c b/qemu-option.c
index 394c763..1ffc497 100644
--- a/qemu-option.c
+++ b/qemu-option.c
@@ -877,8 +87
From: Kevin Wolf
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
Signed-off-by: Luiz Capitulino
---
hw/virtio-blk.c |6 +++---
1 files changed, 3 insertions(+), 3 dele
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 revi
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
---
monitor.c |3 ++-
qemu-monitor.hx |3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/mon
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
net.c |7 ---
net.h |2 +-
qemu-monitor.hx |3 ++-
3 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/net.c b/net.c
index e625614..d7d76eb 100644
--- a/ne
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Luiz Capitulino
---
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 5ca126b..d3bf0fa 100644
--- a/hw/qdev.c
+++ b/hw/qdev.c
@@ -207,7
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 c
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_si
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 n
From: Ryan Harper
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 the failure. This
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 th
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 h
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
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 "add
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:
Th
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: Remove
The proper logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson
---
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
--- a/target-i386/translate
Logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson
---
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
--- a/target-ppc/translate.c
+++ b/targe
Logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson
---
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
--- a/target-mips/translate.c
+++
Logging for -d cpu is done in generic code.
Signed-off-by: Richard Henderson
---
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
--- a/target-sh4/translate.c
+++ b/t
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 n
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
cpu_physical_m
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
Signed-off-by: OHMURA Kei
---
arch_init.c | 54 +-
1 files ch
Replaces byte-based phys_ram_dirty bitmap with three bit-based phys_ram_dirty
bitmap. On allocation, it sets all bits in the bitmap.
Signed-off-by: Yoshiaki Tamura
Signed-off-by: OHMURA Kei
---
exec.c | 32 +++-
1 files changed, 27 insertions(+), 5 deletions(-)
d
Signed-off-by: Yoshiaki Tamura
Signed-off-by: OHMURA Kei
---
cpu-all.h | 81 ++--
1 files changed, 67 insertions(+), 14 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index c409fad..0f5bfbe 100644
--- a/cpu-all.h
+++ b/cpu-all.h
@@ -891
Introduces cpu_physical_memory_get_dirty_range().
It checks the first row and puts dirty addr in the array.
If the first row is empty, it skips to the first non-dirty row
or the end addr, and put the length in the first entry of the array.
Signed-off-by: Yoshiaki Tamura
Signed-off-by: OHMURA Kei
Signed-off-by: Yoshiaki Tamura
Signed-off-by: OHMURA Kei
---
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
@@ -853,7 +853,6 @@ target_phys_addr_t cpu_get_phys_page_de
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
Signed-off-by: Yoshiaki Tamura
---
bswap.h |2 ++
kvm-all.c | 33 +++--
2 files ch
1 - 100 of 107 matches
Mail list logo
