[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 08/10] hw/mips: malta: Add KVM support
From: |
James Hogan |
Subject: |
Re: [Qemu-devel] [PATCH v4 08/10] hw/mips: malta: Add KVM support |
Date: |
Thu, 20 Mar 2014 10:17:33 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 |
On 19/03/14 16:39, Paolo Bonzini wrote:
> Il 14/03/2014 13:47, James Hogan ha scritto:
>> int prom_index = 0;
>> + uint64_t (*xlate_to_phys) (void *opaque, uint64_t addr);
>> + uint64_t (*xlate_to_kseg0) (void *opaque, uint64_t addr);
>> +
>>
>> #ifdef TARGET_WORDS_BIGENDIAN
>> big_endian = 1;
>> @@ -781,7 +787,15 @@ static int64_t load_kernel (void)
>> big_endian = 0;
>> #endif
>>
>> - if (load_elf(loaderparams.kernel_filename,
>> cpu_mips_kseg0_to_phys, NULL,
>> + if (kvm_enabled()) {
>> + xlate_to_phys = cpu_mips_kvm_um_kseg0_to_phys;
>> + xlate_to_kseg0 = cpu_mips_kvm_um_phys_to_kseg0;
>> + } else {
>> + xlate_to_phys = cpu_mips_kseg0_to_phys;
>> + xlate_to_kseg0 = cpu_mips_phys_to_kseg0;
>> + }
>> +
>> + if (load_elf(loaderparams.kernel_filename, xlate_to_phys, NULL,
>> (uint64_t *)&kernel_entry, NULL, (uint64_t
>> *)&kernel_high,
>> big_endian, ELF_MACHINE, 1) < 0) {
>> fprintf(stderr, "qemu: could not load kernel '%s'\n",
>> @@ -820,7 +834,7 @@ static int64_t load_kernel (void)
>> prom_set(prom_buf, prom_index++, "%s",
>> loaderparams.kernel_filename);
>> if (initrd_size > 0) {
>> prom_set(prom_buf, prom_index++, "rd_start=0x%" PRIx64 "
>> rd_size=%li %s",
>> - cpu_mips_phys_to_kseg0(NULL, initrd_offset),
>> initrd_size,
>> + xlate_to_kseg0(NULL, initrd_offset), initrd_size,
>> loaderparams.kernel_cmdline);
>> } else {
>> prom_set(prom_buf, prom_index++, "%s",
>> loaderparams.kernel_cmdline);
>> @@ -829,12 +843,13 @@ static int64_t load_kernel (void)
>> prom_set(prom_buf, prom_index++, "memsize");
>> prom_set(prom_buf, prom_index++, "%i",
>> MIN(loaderparams.ram_size, 256 << 20));
>> +
>> prom_set(prom_buf, prom_index++, "modetty0");
>> prom_set(prom_buf, prom_index++, "38400n8r");
>> prom_set(prom_buf, prom_index++, NULL);
>>
>> rom_add_blob_fixed("prom", prom_buf, prom_size,
>> - cpu_mips_kseg0_to_phys(NULL, ENVP_ADDR));
>> + xlate_to_phys(NULL, ENVP_ADDR));
>>
>
> Why do you need the function pointers? Can
> cpu_mips_kseg0_to_phys/cpu_mips_phys_to_kseg0 just use kvm_enabled()?
Maybe because of its use as an argument to load_elf, which presumably
calls it multiple times. Although looking at glue(load_elf, SZ) I think
it's only used for each load section so I doubt it would make a
difference. I'll change it.
Cheers
James
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v4 02/10] hw/mips: Add API to convert KVM guest KSEG0 <-> GPA, (continued)
- [Qemu-devel] [PATCH v4 02/10] hw/mips: Add API to convert KVM guest KSEG0 <-> GPA, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 03/10] target-mips: get_physical_address: Add defines for segment bases, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 05/10] kvm: Allow arch to set sigmask length, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 04/10] target-mips: get_physical_address: Add KVM awareness, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 07/10] hw/mips: In KVM mode, inject IRQ2 (I/O) interupts via ioctls, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 08/10] hw/mips: malta: Add KVM support, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 09/10] target-mips: Enable KVM support in build system, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 10/10] MAINTAINERS: Add entry for MIPS KVM, James Hogan, 2014/03/14
- [Qemu-devel] [PATCH v4 06/10] target-mips: kvm: Add main KVM support for MIPS, James Hogan, 2014/03/14
- Re: [Qemu-devel] [PATCH v4 00/10] KVM Support for MIPS32 Processors, Peter Maydell, 2014/03/14
- Re: [Qemu-devel] [PATCH v4 00/10] KVM Support for MIPS32 Processors, Paolo Bonzini, 2014/03/19