qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] SM501 emulation for R2D-SH4


From: Blue Swirl
Subject: Re: [Qemu-devel] [PATCH] SM501 emulation for R2D-SH4
Date: Tue, 4 Nov 2008 21:11:22 +0200

On 11/4/08, Shin-ichiro KAWASAKI <address@hidden> wrote:
> Thank you for reviewing!
>
>  I add the new version of patch to this mail, which reflects
>  you and andrzej's comments.  Some more comments on it will be
>  appreciated.  Especially advice on VRAM implementation with
>  normal RAM will be useful for me.
> > The display buffer should be allocated using machine definition
> .ram_require.
> >
>
>  I tried normal RAM implementation : "#define USE_NORMAL_RAM" in the patch
> will
>  turn the new implementation on.  But it causes system hang during SH-Linux
> boot up.
>  The initial VRAM clearance seems to cause the hang.
>  The reason might be my wrong implementation, or any MMU problem of SH4.
>
>  I did,
>  - added VRAM size to .ram_require
>  - invoked qemu_ram_alloc() with the VRAM size parameter,
>  - and invoked cpu_register_physical_memory(), passing the
> return value of
>   qemu_ram_alloc() as third parameter.
>
>  Is this sequence correct?

Yes, but the system RAM is not allocated using qemu_ram_alloc. Now
qemu_ram_alloc gives the same area (0 to VRAM_SIZE) the second time.

So the following:
    /* Allocate memory space */
    cpu_register_physical_memory(SDRAM_BASE, SDRAM_SIZE, 0);
should be changed to:
    ram_addr = qemu_ram_alloc(SDRAM_SIZE);
    cpu_register_physical_memory(SDRAM_BASE, SDRAM_SIZE, ram_addr);

Maybe that helps?




reply via email to

[Prev in Thread] Current Thread [Next in Thread]