On Tue, Apr 20, 2010 at 12:54 AM, jvrao wrote:
> Mohammed Gamal wrote:
>> On Tue, Apr 13, 2010 at 9:08 PM, jvrao wrote:
>>> jvrao wrote:
Alexander Graf wrote:
> On 12.04.2010, at 13:58, Jamie Lokier wrote:
>
>> Mohammed Gamal wrote:
>>> On Mon, Apr 12, 2010 at 12:29 AM, Jamie
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
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
---
cpu-all.h |4 +++
exec.c| 67
Replaces byte-based phys_ram_dirty bitmap with four (MASTER, VGA, CODE,
MIGRATION) bit-based phys_ram_dirty bitmap. On allocation, it sets all bits in
the bitmap. It uses ffs() to convert DIRTY_FLAG to DIRTY_IDX.
Modifies wrapper functions for byte-based phys_ram_dirty bitmap to bit-based
phys_r
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
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
Hi,
I noticed that this patch never got applied, but I'm not sure if it
works. It seems like it works for forks, but not for pthreads. I
tried applying it to the current git head and it seems to change the
behavior from dumping the registers to dying silently for code with
pthreads, but does wor
Michael S. Tsirkin wrote:
> I took a stub at documenting CMD and FLUSH request types in virtio
> block. Christoph, could you look over this please?
>
> I note that the interface seems full of warts to me,
> this might be a first step to cleaning them.
>
> One issue I struggled with especially is
On Mon, Apr 19, 2010 at 08:19:55AM +0200, Gerhard Wiesinger wrote:
> Kevin O'Connor wrote:
> >The SeaBIOS log would really help. This can be done by adding:
> >
> >-chardev stdio,id=seabios -device isa-debugcon,iobase=0x402,chardev=seabios
> >
> >to the qemu command line.
> OK, I made some researc
Thank you for the explanation of this code.
Qemu has a command named singlestep, which reduces the translated code
block to be only one instruction.
This new patch flushes TBs both when singlestep is on and off.
Signed-off-by: Jun Koi
diff --git a/monitor.c b/monitor.c
index 5659991..2b2005b 1
Hi,
Does the linux-user mode in QEMU support running multithreaded code?
I am most interested in running SPARC binaries, but I tested with
x86_64 and it seems like some system calls needed for pthreads are not
included. I've attached a very simple program that uses pthreads.
When I run it normall
On Monday 19 April 2010 18:30:44 Chris Wright wrote:
> Please send in any agenda items you are interested in covering.
0.12.4?
>
> thanks,
> -chris
>
Please send in any agenda items you are interested in covering.
thanks,
-chris
Mohammed Gamal wrote:
> On Tue, Apr 13, 2010 at 9:08 PM, jvrao wrote:
>> jvrao wrote:
>>> Alexander Graf wrote:
On 12.04.2010, at 13:58, Jamie Lokier wrote:
> Mohammed Gamal wrote:
>> On Mon, Apr 12, 2010 at 12:29 AM, Jamie Lokier
>> wrote:
>>> Javier Guerra Giraldez wr
> I find the way we calculate the dummy field in CPUTLBEntry funny. What
> is the point of having:
>
>((-sizeof(target_ulong) * 3) & (sizeof(unsigned long) -
> 1))
>
> in its size? Why shouldnt it be more simple, like below?
>
> uint8_t dummy[(1 << CPU_TLB_ENTRY_BITS) -
On Fri, Feb 19, 2010 at 12:22:20AM +0200, Michael S. Tsirkin wrote:
> I took a stub at documenting CMD and FLUSH request types in virtio
> block.
Any comments?
David S. Ahern wrote:
> Hi Jan:
>
> The attached addresses the spinning with the usb net device. Now I can
> enable the device and ethtool shows a link:
>
> # ifconfig usb0 up
> # ethtool usb0
> Settings for usb0:
> Current message level: 0x0007 (7)
> Link detected: yes
>
> Thoug
2010/4/19 Blue Swirl :
> On 4/19/10, Artyom Tarasenko wrote:
>> is -single-step option supposed to work under qemu-system-sparc? Or is
>> it only for non-pipelined CPUs?
>
> I've never used it.
is there another way to reduce a translation block? -icount 1 doesn't
make a difference.
>> I get pr
From: Thomas Monjalon
Using GCC-4.2.4-1ubuntu4, there is a warning:
microblaze-dis.c:792: warning: unused variable 'fprintf'
Indeed, fprintf() is shadowed by a custom redefinition but is not used because
of FORTIFY_SOURCE option which replace calls to fprintf() by fprintf_chk().
It is f
>>> On 4/14/2010 at 01:24 AM, Gerd Hoffmann wrote:
> Hi,
>
>> When specifying ide devices using -device, the cmos information
>> which the bios depends on is not written. This patch generalizes
>> the cmos hd data setting for the existing code path and adds the
>> ability to call that code o
On 4/19/10, Artyom Tarasenko wrote:
> is -single-step option supposed to work under qemu-system-sparc? Or is
> it only for non-pipelined CPUs?
I've never used it.
> I get pretty strange results: some addresses seem to be executed
> twice, and then qemu hangs on the last nop:
One possibility
Hi folks,
I've implemented a functional test for the cpu_set feature some time
ago. I was going through the patch queue and realized the patch needed
some respin, so I did it. I decided to try and see what is the state of
the feature, considering last time I tried it was not working
(segfaulting)
On 04/19/2010 11:05 AM, malc wrote:
> Apparently i'm not the only one who misinterpreted this bit of bswap
> documentation. How about:
>
> diff --git a/tcg/README b/tcg/README
> index 68d27ff..5b39a38 100644
> --- a/tcg/README
> +++ b/tcg/README
> @@ -269,7 +269,7 @@ ext32u_i64 t0, t1
> * bswap16
Hello,
Finally I got QEMU-KVM to work but video performance under DOS is very
low (QEMU 0.12.3 stable and QEMU GIT master branch is fast, QEMU KVM is
slow)
I'm measuring 2 performance critical video performance parameters:
1.) INT 10h, function AX=4F05h (set same window/set window/get window)
attached is a tested 0.12.3-version of an old patch
http://xenon.stanford.edu/~eswierk/misc/qemu-linuxbios/qemu-piix-ram-size.patch
that together with
http://www.mail-archive.com/linuxb...@linuxbios.org/msg02390.html
(which is already in coreboot trunk) allows coreboot BIOS to autodetect the
amoun
On Wed, 14 Apr 2010, Gerhard Wiesinger wrote:
On Wed, 14 Apr 2010, Jamie Lokier wrote:
Gerhard Wiesinger wrote:
It is a non public, proprietary application which uses the Ergo Computing
286 DOS Extender. I guess some other application which use the same DOS
extender have the same problem. So
When removing usb on guest via usb_del monitor command, qemu does
not return the control to the host, the only solution user have is to
unplug/plug the device again in order to get the device back to the host
Signed-off-by: Shahar Havivi
---
hw/usb-bus.c |4
hw/usb.h |1 +
usb-l
On Sat, 10 Apr 2010, Gerhard Wiesinger wrote:
Hello,
Booting DOS 6.22 with NT4 Bootloader works fine. With enabled KVM it boots,
but hangs at counting down the boot loader selection menu (30s, 29s, hang).
Booting Knoppix 6.2 from CD wwith KVM orks well.
Versions (0.12.3):
x86_64-softmmu: OK
On Sun, Apr 18, 2010 at 11:52:16PM +0200, Juergen Lock wrote:
> On Sun, Apr 18, 2010 at 11:38:34PM +0200, Aurelien Jarno wrote:
> > On Sun, Apr 18, 2010 at 11:16:46PM +0200, Juergen Lock wrote:
> > > On Sun, Apr 18, 2010 at 10:52:22PM +0200, Aurelien Jarno wrote:
> > > > On Sun, Apr 11, 2010 at 06:
Hi,
I'm trying to use a 64-bit BAR for my shared memory device. In simply
changing the memory type in pci_register_bar() to
PCI_BASE_ADDRESS_MEM_TYPE_64 I get an unusual physical address for
that BAR (and my driver crashes in pci_ioremap).
from lspci:
00:04.0 RAM memory: Qumranet, Inc. Device 1
Artyom Tarasenko schrieb:
> is -single-step option supposed to work under qemu-system-sparc? Or is
> it only for non-pipelined CPUs?
>
> I get pretty strange results: some addresses seem to be executed
> twice, and then qemu hangs on the last nop:
>
> --
> IN:
> 0x: b 0xb788
> -
Kevin Wolf schrieb:
> Am 19.04.2010 03:23, schrieb Jamie Lokier:
>> Benjamin Drung wrote:
>>> - /* SDL does not send the key up event, so we generate it */
>> Was the original comment just plain wrong?
>>
>>> - kbd_put_keycode(keycode);
>>> - kbd_put_keycode(keycode | 0x80);
>>> + if (ev->type == S
On Mon, 19 Apr 2010, Artyom Tarasenko wrote:
> is -single-step option supposed to work under qemu-system-sparc? Or is
> it only for non-pipelined CPUs?
Eh? Perhaps you meant that it's only for CPUs without delay slots?
>
> I get pretty strange results: some addresses seem to be executed
> twice
On Mon, 19 Apr 2010, Richard Henderson wrote:
> On 04/18/2010 05:13 PM, Aurelien Jarno wrote:
> > On Tue, Apr 13, 2010 at 04:33:59PM -0700, Richard Henderson wrote:
> >> Define OPC_BSWAP. Factor opcode emission to separate functions.
> >> Use bswap+shift to implement 16-bit swap instead of a rolw
The BlockDriver bdrv_getlength function is called from the I/O code path
when checking that the request falls within the device. Unfortunately
this involves an lseek system call in the raw protocol; every read or
write request will incur this lseek cost.
Jan Kiszka identified this issue and its
is -single-step option supposed to work under qemu-system-sparc? Or is
it only for non-pipelined CPUs?
I get pretty strange results: some addresses seem to be executed
twice, and then qemu hangs on the last nop:
--
IN:
0x: b 0xb788
--
IN:
0x0004: rd %psr, %
2010/4/19 Avi Kivity :
> On 04/19/2010 02:52 PM, Yoshiaki Tamura wrote:
>>
>> Avi Kivity wrote:
>>>
>>> On 04/19/2010 02:31 PM, Yoshiaki Tamura wrote:
>>> typedef struct RAMBlock {
>>> @@ -2825,10 +2825,16 @@ ram_addr_t qemu_ram_alloc(ram_addr_t size)
>>> new_block->next = ram_bloc
Am 19.04.2010 16:26, schrieb Stefan Hajnoczi:
> On Mon, Apr 19, 2010 at 3:10 PM, Kevin Wolf wrote:
>>> @@ -416,9 +417,7 @@ static int bdrv_open_common(BlockDriverState *bs, const
>>> char *filename,
>>> }
>>>
>>> bs->keep_read_only = bs->read_only = !(open_flags & BDRV_O_RDWR);
>>> -
On Mon, Apr 19, 2010 at 3:10 PM, Kevin Wolf wrote:
>> @@ -416,9 +417,7 @@ static int bdrv_open_common(BlockDriverState *bs, const
>> char *filename,
>> }
>>
>> bs->keep_read_only = bs->read_only = !(open_flags & BDRV_O_RDWR);
>> - if (drv->bdrv_getlength) {
>> - bs->total_sect
malc a écrit :
> On Sun, 18 Apr 2010, Aurelien Jarno wrote:
>
>> On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote:
>>> As suggested by Andreas F?rber, here is a cumulative patch that syncs
>>> OSS_GETVERSION handling with head by merging the following commits:
>>>
>>> 1. oss: issue OSS
Am 19.04.2010 14:34, schrieb Stefan Hajnoczi:
> The BlockDriver bdrv_getlength function is called from the I/O code path
> when checking that the request falls within the device. Unfortunately
> this involves an lseek system call in the raw protocol; every read or
> write request will incur this l
On 04/18/2010 05:13 PM, Aurelien Jarno wrote:
> On Tue, Apr 13, 2010 at 04:33:59PM -0700, Richard Henderson wrote:
>> Define OPC_BSWAP. Factor opcode emission to separate functions.
>> Use bswap+shift to implement 16-bit swap instead of a rolw; this
>> gets the proper zero-extension required by IN
Looks good.
Stefan
qcow_create2 assumes that the new image will only need one cluster for its
refcount table initially. Obviously that's not true any more when the image is
big enough (exact value depends on the cluster size).
This patch calculates the refcount table size dynamically.
Signed-off-by: Kevin Wolf
---
On Mon, Apr 19, 2010 at 12:11:46PM +0600, Taimoor Mirza wrote:
> Program received signal SIGSEGV, Segmentation fault.
> [Switching to Thread -1208740160 (LWP 1912)]
> gdb_set_stop_cpu (env=0x9ef36a0) at
> /usr/QEMU_Learning/QEMU_Src/qemu-0.12.3/gdbstub.c:2114
> 2114gdbserver_state->c_cpu =
On Sun, 18 Apr 2010 23:58:43 +0200
Aurelien Jarno wrote:
> On Mon, Apr 05, 2010 at 05:33:41PM -0300, Luiz Capitulino wrote:
> > 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 g
VMDK is doing interesting things when it needs to open a backing file. This
patch changes that part to look more like in other drivers. The nice side
effect is that the file name isn't needed any more in the open function.
Signed-off-by: Kevin Wolf
---
v2:
- Removed vmdk_parent_close()
block/vm
The BlockDriver bdrv_getlength function is called from the I/O code path
when checking that the request falls within the device. Unfortunately
this involves an lseek system call in the raw protocol; every read or
write request will incur this lseek cost.
Jan Kiszka identified this issue and its
This patch combines the lseek+read/write calls to use pread/pwrite
instead. This will result in fewer system calls and is already used by
AIO.
Thanks to Jan Kiszka for identifying excessive
lseek and Christoph Hellwig for confirming that this
approach should work.
Signed-off-by: Stefan Hajnocz
Stefan Hajnoczi wrote:
> From: Stefan Hajnoczi
>
> The tests use bash language features including 'local' and 'let' which
> aren't supported by /bin/sh on systems that use a conservative shell
> like dash. This patch changes the interpreter to /bin/bash.
Minor nit: Dash does support 'local'.
-
On 04/19/2010 02:52 PM, Yoshiaki Tamura wrote:
Avi Kivity wrote:
On 04/19/2010 02:31 PM, Yoshiaki Tamura wrote:
typedef struct RAMBlock {
@@ -2825,10 +2825,16 @@ ram_addr_t qemu_ram_alloc(ram_addr_t size)
new_block->next = ram_blocks;
ram_blocks = new_block;
- phys_ram_dirty = qemu_realloc(p
Avi Kivity wrote:
On 04/19/2010 02:31 PM, Yoshiaki Tamura wrote:
typedef struct RAMBlock {
@@ -2825,10 +2825,16 @@ ram_addr_t qemu_ram_alloc(ram_addr_t size)
new_block->next = ram_blocks;
ram_blocks = new_block;
- phys_ram_dirty = qemu_realloc(phys_ram_dirty,
- (last_ram_offset + size)>> TARG
On 04/19/2010 02:31 PM, Yoshiaki Tamura wrote:
typedef struct RAMBlock {
@@ -2825,10 +2825,16 @@ ram_addr_t qemu_ram_alloc(ram_addr_t size)
new_block->next = ram_blocks;
ram_blocks = new_block;
- phys_ram_dirty = qemu_realloc(phys_ram_dirty,
- (last_ram_offset + size)>> TARGET_PAGE_BITS);
- m
2010/4/19 Yoshiaki Tamura :
> Avi Kivity wrote:
>>
>> On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
>>>
>>> Replaces byte-based phys_ram_dirty bitmap with four bit-based
>>> phys_ram_dirty
>>> bitmap. On allocation, it sets all bits in the bitmap.
>>>
>>> Signed-off-by: Yoshiaki Tamura
>>> ---
>>>
From: Stefan Hajnoczi
The tests use bash language features including 'local' and 'let' which
aren't supported by /bin/sh on systems that use a conservative shell
like dash. This patch changes the interpreter to /bin/bash.
Signed-off-by: Stefan Hajnoczi
---
001|2 +-
002
Gerhard Wiesinger wrote:
> On Tue, 13 Apr 2010, Jan Kiszka wrote:
>
>> Mulyadi Santosa wrote:
>>> Hi Jamie...
>>>
>>> On Mon, Apr 12, 2010 at 19:07, Jamie Lokier wrote:
There are various -no-kvm-XXX options to try:
-no-kvm-irqchip disable KVM kernel mode PIC/IOAPIC/LAPIC
-no-k
Avi Kivity wrote:
On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
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
p
Avi Kivity wrote:
On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
Replaces byte-based phys_ram_dirty bitmap with four bit-based
phys_ram_dirty
bitmap. On allocation, it sets all bits in the bitmap.
Signed-off-by: Yoshiaki Tamura
---
exec.c | 16 +++-
qemu-common.h | 3 +++
2 files chan
Avi Kivity wrote:
On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
It uses ffs() to convert DIRTY_FLAG to DIRTY_IDX.
Signed-off-by: Yoshiaki Tamura
---
cpu-all.h | 30 ++
1 files changed, 26 insertions(+), 4 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index f8bfa66
On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
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
On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
Replaces byte-based phys_ram_dirty bitmap with four bit-based phys_ram_dirty
bitmap. On allocation, it sets all bits in the bitmap.
Signed-off-by: Yoshiaki Tamura
---
exec.c| 16 +++-
qemu-common.h |3 +++
2 files chan
On 04/19/2010 12:43 PM, Yoshiaki Tamura wrote:
It uses ffs() to convert DIRTY_FLAG to DIRTY_IDX.
Signed-off-by: Yoshiaki Tamura
---
cpu-all.h | 30 ++
1 files changed, 26 insertions(+), 4 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index f8bfa66..8c2d678 100
Replaces byte-based phys_ram_dirty bitmap with four bit-based phys_ram_dirty
bitmap. On allocation, it sets all bits in the bitmap.
Signed-off-by: Yoshiaki Tamura
---
exec.c| 16 +++-
qemu-common.h |3 +++
2 files changed, 14 insertions(+), 5 deletions(-)
diff --git a
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
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
---
cpu-all.h |4 +++
exec.c| 67
It uses ffs() to convert DIRTY_FLAG to DIRTY_IDX.
Signed-off-by: Yoshiaki Tamura
---
cpu-all.h | 30 ++
1 files changed, 26 insertions(+), 4 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index f8bfa66..8c2d678 100644
--- a/cpu-all.h
+++ b/cpu-all.h
@@ -37,6 +37,9
MASTER works as a buffer, and upon get_diry() or get_dirty_flags(), it calls
cpu_physical_memory_sync_master() to update VGA and MIGRATION.
Signed-off-by: Yoshiaki Tamura
---
cpu-all.h | 96 +++-
1 files changed, 81 insertions(+), 15 dele
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
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
Am 17.04.2010 11:49, schrieb Stefan Hajnoczi:
> It is safer to set backing_hd to NULL after deleting it so that any use
> after deletion is obvious during development. Happy segfaulting!
>
> This patch should be applied after Kevin Wolf's "vmdk: Convert to
> bdrv_open" so that vmdk does not segfa
Am 19.04.2010 03:23, schrieb Jamie Lokier:
> Benjamin Drung wrote:
>> -/* SDL does not send the key up event, so we generate it */
>
> Was the original comment just plain wrong?
>
>> -kbd_put_keycode(keycode);
>> -kbd_put_keycode(keycode | 0x80);
>> +if (ev->type =
On Mon, 19 Apr 2010, Aurelien Jarno wrote:
> malc a ?crit :
> > On Sun, 18 Apr 2010, Aurelien Jarno wrote:
> >
> >> On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote:
> >>> As suggested by Andreas F?rber, here is a cumulative patch that syncs
> >>> OSS_GETVERSION handling with head by
Am 17.04.2010 11:39, schrieb Stefan Hajnoczi:
> On Fri, Apr 16, 2010 at 8:59 PM, Kevin Wolf wrote:
>> diff --git a/block/vmdk.c b/block/vmdk.c
>> index 781518a..27b6360 100644
>> --- a/block/vmdk.c
>> +++ b/block/vmdk.c
>> @@ -835,14 +828,12 @@ static void vmdk_close(BlockDriverState *bs)
>> q
On Sun, 18 Apr 2010, Aurelien Jarno wrote:
> On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote:
> > As suggested by Andreas F?rber, here is a cumulative patch that syncs
> > OSS_GETVERSION handling with head by merging the following commits:
> >
> > 1. oss: issue OSS_GETVERSION ioctl o
74 matches
Mail list logo
