Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread Alexander Graf
On 02.10.2010, at 07:38, John Clark wrote: Hello, I found I had to make a few minor changes to the MMU code for the PowerPC 40x emulation to get NetBSD to run on a virtual PowerPC 405 core with qemu-system-ppcemb. The 'tlbre' instruction was not working, and permission checking for a TLB

Re: [Qemu-devel] [PATCH v2] es1370: Fix compiler warnings for debug code

2010-10-02 Thread Stefan Weil
Am 01.10.2010 20:22, schrieb Markus Armbruster: Stefan Weil w...@mail.berlios.de writes: The patch fixes these gcc warnings: ./hw/es1370.c: In function ‘es1370_update_voices’: ./hw/es1370.c:411: error: format ‘%d’ expects type ‘int’, but argument 3 has type ‘size_t’ ./hw/es1370.c: In

[Qemu-devel] [PATCH v3] es1370: Fix compiler warnings for debug code

2010-10-02 Thread Stefan Weil
The patch fixes these gcc warnings: ./hw/es1370.c: In function ‘es1370_update_voices’: ./hw/es1370.c:411: error: format ‘%d’ expects type ‘int’, but argument 3 has type ‘size_t’ ./hw/es1370.c: In function ‘es1370_writel’: ./hw/es1370.c:579: error: format ‘%d’ expects type ‘int’, but argument 3

[Qemu-devel] [PATCH 1/2] target-cris: Use %td for ptrdiff_t arguments in debug message

2010-10-02 Thread Stefan Weil
According to ISO/IEC 9899:1999 7.19.6.1, the correct length modifier for ptrdiff_t is 't', not 'z'. Cc: Blue Swirl blauwir...@gmail.com Cc: Markus Armbruster arm...@redhat.com Signed-off-by: Stefan Weil w...@mail.berlios.de --- target-cris/translate.c |2 +- 1 files changed, 1 insertions(+),

[Qemu-devel] [PATCH 2/2] target-microblaze: Use %td for ptrdiff_t arguments in debug message

2010-10-02 Thread Stefan Weil
According to ISO/IEC 9899:1999 7.19.6.1, the correct length modifier for ptrdiff_t is 't', not 'z'. Cc: Blue Swirl blauwir...@gmail.com Cc: Markus Armbruster arm...@redhat.com Signed-off-by: Stefan Weil w...@mail.berlios.de --- target-microblaze/translate.c |2 +- 1 files changed, 1

Re: [Qemu-devel] [PATCH 0/2] powerpc: Add a virtex5 ml507 refdesign board (v2)

2010-10-02 Thread Stefan Weil
Am 30.09.2010 19:12, schrieb Edgar E. Iglesias: On Thu, Sep 30, 2010 at 03:15:54PM +0200, Alexander Graf wrote: Edgar E. Iglesias wrote: v2: * Address review comments from A. Graf. Edgar E. Iglesias (2): powerpc: Add a ppc-440x5 Xilinx model powerpc: Add a virtex5 ml507 refdesign board

Re: [Qemu-devel] [PATCH 0/2] powerpc: Add a virtex5 ml507 refdesign board (v2)

2010-10-02 Thread Alexander Graf
On 02.10.2010, at 13:02, Stefan Weil wrote: Am 30.09.2010 19:12, schrieb Edgar E. Iglesias: On Thu, Sep 30, 2010 at 03:15:54PM +0200, Alexander Graf wrote: Edgar E. Iglesias wrote: v2: * Address review comments from A. Graf. Edgar E. Iglesias (2): powerpc: Add a ppc-440x5 Xilinx model

Re: [Qemu-devel] [PATCH 2/2] target-microblaze: Use %td for ptrdiff_t arguments in debug message

2010-10-02 Thread Edgar E. Iglesias
On Sat, Oct 02, 2010 at 12:41:04PM +0200, Stefan Weil wrote: According to ISO/IEC 9899:1999 7.19.6.1, the correct length modifier for ptrdiff_t is 't', not 'z'. Cc: Blue Swirl blauwir...@gmail.com Cc: Markus Armbruster arm...@redhat.com Signed-off-by: Stefan Weil w...@mail.berlios.de

Re: [Qemu-devel] [PATCH 0/2] powerpc: Add a virtex5 ml507 refdesign board (v2)

2010-10-02 Thread Edgar E. Iglesias
On Sat, Oct 02, 2010 at 01:02:50PM +0200, Stefan Weil wrote: Am 30.09.2010 19:12, schrieb Edgar E. Iglesias: On Thu, Sep 30, 2010 at 03:15:54PM +0200, Alexander Graf wrote: Edgar E. Iglesias wrote: v2: * Address review comments from A. Graf. Edgar E. Iglesias (2): powerpc: Add a

[Qemu-devel] Re: [PATCH v3] es1370: Fix compiler warnings for debug code

2010-10-02 Thread malc
On Sat, 2 Oct 2010, Stefan Weil wrote: The patch fixes these gcc warnings: ./hw/es1370.c: In function ?es1370_update_voices?: ./hw/es1370.c:411: error: format ?%d? expects type ?int?, but argument 3 has type ?size_t? ./hw/es1370.c: In function ?es1370_writel?: ./hw/es1370.c:579: error:

Re: [Qemu-devel] Re: Bad ram offset?

2010-10-02 Thread Blue Swirl
On Thu, Sep 30, 2010 at 8:36 PM, Artyom Tarasenko atar4q...@gmail.com wrote: On Wed, Sep 29, 2010 at 6:09 PM, Blue Swirl blauwir...@gmail.com wrote: On Tue, Sep 28, 2010 at 8:42 PM, Andreas Färber andreas.faer...@web.de wrote: Am 28.09.2010 um 22:24 schrieb Blue Swirl: On Tue, Sep 28, 2010

Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread John Clark
/* Check from TLB entry */ -/* XXX: there is a problem here or in the TLB fill code... */ +/* There is no longer a need to force PAGE_EXEC permission here */ +/* because of the tlb-attr fix in helper_4xx_tlbwe_lo() */ I guess that comment is

Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread Alexander Graf
On 02.10.2010, at 18:49, John Clark wrote: /* Check from TLB entry */ -/* XXX: there is a problem here or in the TLB fill code... */ +/* There is no longer a need to force PAGE_EXEC permission here */ +/* because of the tlb-attr fix in

Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread John Clark
-target_ulong helper_4xx_tlbre_hi (target_ulong entry) +target_ulong helper_4xx_tlbre_lo (target_ulong entry) Huh? To summarize, 'tlbre' has two forms: one to retrieve the high bits of a TLB entry (TLBHI), and one to retrieve the low bits (TLBLO) of a TLB entry. This code had the TLBLO

Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread Edgar E. Iglesias
On Sat, Oct 02, 2010 at 06:55:36PM +0200, Alexander Graf wrote: On 02.10.2010, at 18:49, John Clark wrote: /* Check from TLB entry */ -/* XXX: there is a problem here or in the TLB fill code... */ +/* There is no longer a need to force PAGE_EXEC

Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread Alexander Graf
On 02.10.2010, at 19:06, Edgar E. Iglesias wrote: On Sat, Oct 02, 2010 at 06:55:36PM +0200, Alexander Graf wrote: On 02.10.2010, at 18:49, John Clark wrote: /* Check from TLB entry */ -/* XXX: there is a problem here or in the TLB fill code... */ +/*

Re: [Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-02 Thread John Clark
Hi, Alex: I think you've missed the part of the patch that renames the _lo - _hi. As John says, qemu had the hi/lo parts reversed when reading 4xx TLB regs. Except for the comments and the define, the patch looks good to me. John, please also add a Signed-off-by line. Cheers Ok, I

[Qemu-devel] [Bug 638955] Re: emulated netcards don't work with recent sunos kernel

2010-10-02 Thread daniel pecka
well, i did some more investigations and here come a results .. this patch http://patchwork.ozlabs.org/patch/65137/raw/ solves problem partially .. NICs are working with that but after a deeper look, connection is lost when the netstack is flooded with higher traffic .. i can connect with

[Qemu-devel] qemu-kvm 0.12.5 virtio-net page allocation error

2010-10-02 Thread Peter Lieven
Hi, running 0.12.5 with a Ubuntu LTS 10.04.1 64-bit kernel I see the following error after a few days with severe load (constant 100-200mbps input). # uname -a Linux ubuntu-newsfeed 2.6.32-24-server #43-Ubuntu SMP Thu Sep 16 16:05:42 UTC 2010 x86_64 GNU/Linux Oct 2 10:09:35 ubuntu-newsfeed