[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/15] Openrisc: add a simulation board
From: |
Jia Liu |
Subject: |
Re: [Qemu-devel] [PATCH 10/15] Openrisc: add a simulation board |
Date: |
Fri, 25 May 2012 10:31:01 +0800 |
On Thu, May 24, 2012 at 3:17 AM, Blue Swirl <address@hidden> wrote:
> On Wed, May 23, 2012 at 7:54 AM, Jia Liu <address@hidden> wrote:
>> Hi Blue,
>>
>> On Sat, May 19, 2012 at 3:51 PM, Blue Swirl <address@hidden> wrote:
>>> On Thu, May 17, 2012 at 8:35 AM, Jia Liu <address@hidden> wrote:
>>>> add a simulation board for openrisc.
>>>>
>>>> Signed-off-by: Jia Liu <address@hidden>
>>>> ---
>>>> Makefile.target | 1 +
>>>> hw/openrisc_sim.c | 140
>>>> +++++++++++++++++++++++++++++++++++++++++++++++++++++
>>>> 2 files changed, 141 insertions(+)
>>>> create mode 100644 hw/openrisc_sim.c
>>>>
>>>> diff --git a/Makefile.target b/Makefile.target
>>>> index 6b1d2d0..433dd3c 100644
>>>> --- a/Makefile.target
>>>> +++ b/Makefile.target
>>>> @@ -391,6 +391,7 @@ obj-xtensa-y += core-dc233c.o
>>>> obj-xtensa-y += core-fsf.o
>>>>
>>>> obj-openrisc-y += openrisc_pic.o
>>>> +obj-openrisc-y += openrisc_sim.o
>>>> obj-openrisc-y += openrisc_timer.o
>>>>
>>>> main.o: QEMU_CFLAGS+=$(GPROF_CFLAGS)
>>>> diff --git a/hw/openrisc_sim.c b/hw/openrisc_sim.c
>>>> new file mode 100644
>>>> index 0000000..869a02e
>>>> --- /dev/null
>>>> +++ b/hw/openrisc_sim.c
>>>> @@ -0,0 +1,140 @@
>>>> +/*
>>>> + * Openrisc simulator for use as an ISS.
>>>> + *
>>>> + * Copyright (c) 2011-2012 Jia Liu <address@hidden>
>>>> + * Feng Gao <address@hidden>
>>>> + *
>>>> + * This library is free software; you can redistribute it and/or
>>>> + * modify it under the terms of the GNU Library General Public
>>>> + * License as published by the Free Software Foundation; either
>>>> + * version 2 of the License, or (at your option) any later version.
>>>> + *
>>>> + * This library is distributed in the hope that it will be useful,
>>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>>>> + * Library General Public License for more details.
>>>> + *
>>>> + * You should have received a copy of the GNU Library General Public
>>>> + * License along with this library; if not, write to the Free Software
>>>> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA
>>>> 02110-1301 USA
>>>> + */
>>>> +
>>>> +#include "hw.h"
>>>> +#include "fdc.h"
>>>> +#include "net.h"
>>>> +#include "openrisc_cpudev.h"
>>>> +#include "boards.h"
>>>> +#include "pci.h"
>>>> +#include "elf.h"
>>>> +#include "smbus.h"
>>>> +#include "memory.h"
>>>> +#include "pc.h"
>>>> +#include "pci.h"
>>>> +#include "sysbus.h"
>>>> +#include "flash.h"
>>>> +#include "loader.h"
>>>> +#include "exec-memory.h"
>>>> +#include "sysemu.h"
>>>> +#include "isa.h"
>>>> +#include "mc146818rtc.h"
>>>
>>> There's no rtc, so this shouldn't be needed. Please trim other includes as
>>> well.
>>>
>>
>> deleted, thanks.
>>
>>>> +#include "blockdev.h"
>>>> +#include "qemu-log.h"
>>>> +
>>>> +#define KERNEL_LOAD_ADDR 0x100
>>>> +
>>>> +static uint64_t translate_phys_addr(void *env, uint64_t addr)
>>>> +{
>>>> + return cpu_get_phys_page_debug(env, addr);
>>>
>>> There's no virtual to physical mapping during CPU init, I suppose. Then just
>>> return addr;
>>>
>>
>> this func is useless, deleted, thanks.
>>
>>>> +}
>>>> +
>>>> +static void main_cpu_reset(void *opaque)
>>>> +{
>>>> + CPUOPENRISCState *env = opaque;
>>>> + openrisc_reset(env);
>>>> +}
>>>> +
>>>> +static void openrisc_sim_init(ram_addr_t ram_size,
>>>> + const char *boot_device,
>>>> + const char *kernel_filename,
>>>> + const char *kernel_cmdline,
>>>> + const char *initrd_filename,
>>>> + const char *cpu_model)
>>>> +{
>>>> + CPUOPENRISCState *env;
>>>> + MemoryRegion *ram = g_new(MemoryRegion, 1);
>>>> + int kernel_size;
>>>
>>> target_ulong, target_phys_addr_t or uint64_t?
>>>
>>
>> we need it int here, for load error, it return -1.
>
> Well, you could use for example (uint64_t)-1 in the comparison. My
> point with this change is to prepare for a 64 bit architecture while
> int could be 32 bits. Kernels larger than 4G are still a bit rare...
>
thanks, fixed.
>>
>>>> + uint64_t elf_entry;
>>>> + target_phys_addr_t entry;
>>>
>>> This is not really used.
>>
>> it is used by somewhere.
>
> It's written to in a few places, but there are no readers.
>
>>
>>>
>>>> + qemu_irq *i8259;
>>>> + ISABus *isa_bus;
>>>> +
>>>> + if (!cpu_model) {
>>>> + cpu_model = "or1200";
>>>> + }
>>>> + env = cpu_init(cpu_model);
>>>> + if (!env) {
>>>> + fprintf(stderr, "Unable to find CPU definition!\n");
>>>> + exit(1);
>>>> + }
>>>> +
>>>> + qemu_register_reset(main_cpu_reset, env);
>>>> + main_cpu_reset(env);
>>>> +
>>>> + memory_region_init_ram(ram, "openrisc.ram", ram_size);
>>>> + memory_region_add_subregion(get_system_memory(), 0, ram);
>>>> +
>>>> + if (kernel_filename) {
>>>
>>> Please add && !qtest_enabled().
>>>
>>
>> Thanks, added.
>>
>>>> + kernel_size = load_elf(kernel_filename, translate_phys_addr, env,
>>>> + &elf_entry, NULL, NULL, 1, ELF_MACHINE, 1);
>>>> + entry = elf_entry;
>>>> + if (kernel_size < 0) {
>>>> + kernel_size = load_uimage(kernel_filename, &entry, NULL,
>>>> NULL);
>>>> + }
>>>> + if (kernel_size < 0) {
>>>> + kernel_size = load_image_targphys(kernel_filename,
>>>> + KERNEL_LOAD_ADDR,
>>>> + ram_size -
>>>> KERNEL_LOAD_ADDR);
>>>> + entry = KERNEL_LOAD_ADDR;
>>>> + }
>>>> + if (kernel_size < 0) {
>>>> + fprintf(stderr, "qemu: could not load kernel '%s'\n",
>>>> + kernel_filename);
>>>> + exit(1);
>>>> + }
>>>> +
>>>> + if (kernel_size > 0) {
>>>> + env->pc = elf_entry;
>>>> + }
>>>> + } else {
>>>> + entry = 0;
>>>> + }
>>>> +
>>>> + cpu_openrisc_pic_init(env);
>>>> + cpu_openrisc_clock_init(env);
>>>> +
>>>> + isa_bus = isa_bus_new(NULL, get_system_io());
>>>> + i8259 = i8259_init(isa_bus, env->irq[3]);
>>>> + isa_bus_irqs(isa_bus, i8259);
>>>> +
>>>> + serial_mm_init(get_system_memory(), 0x90000000, 0,
>>>> + env->irq[2], 115200, serial_hds[0],
>>>> DEVICE_NATIVE_ENDIAN);
>>>> +
>>>> + if (nd_table[0].vlan) {
>>>> + isa_ne2000_init(isa_bus, 0x92000000, 4, &nd_table[0]);
>>>> + }
>>>> +}
>>>> +
>>>> +static QEMUMachine openrisc_sim_machine = {
>>>> + .name = "or32-sim",
>>>> + .desc = "or32 simulation",
>>>> + .init = openrisc_sim_init,
>>>> + .max_cpus = 1,
>>>> + .is_default = 1,
>>>> +};
>>>> +
>>>> +static void openrisc_sim_machine_init(void)
>>>> +{
>>>> + qemu_register_machine(&openrisc_sim_machine);
>>>> +}
>>>> +
>>>> +machine_init(openrisc_sim_machine_init);
>>>> --
>>>> 1.7.9.5
>>>>
>>>>
>>
>>
>> Regards,
>> Jia.
- [Qemu-devel] [PATCH 12/15] Openrisc: add gdb stub support, (continued)
- [Qemu-devel] [PATCH 12/15] Openrisc: add gdb stub support, Jia Liu, 2012/05/17
- [Qemu-devel] [PATCH 13/15] Openrisc: add linux syscall, signal and termbits, Jia Liu, 2012/05/17
- [Qemu-devel] [PATCH 14/15] Openrisc: add linux user support, Jia Liu, 2012/05/17
- [Qemu-devel] [PATCH 15/15] Openrisc: add testcases, Jia Liu, 2012/05/17
- [Qemu-devel] [PATCH 10/15] Openrisc: add a simulation board, Jia Liu, 2012/05/17