[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 21/21] RISC-V Build Infrastructure
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v1 21/21] RISC-V Build Infrastructure |
Date: |
Wed, 3 Jan 2018 17:23:46 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
On 01/02/2018 06:44 PM, Michael Clark wrote:
> This adds RISC-V into the build system enabling the following targets:
>
> - riscv32-softmmu
> - riscv64-softmmu
> - riscv32-linux-user
> - riscv64-linux-user
>
> This adds defaults configs for RISC-V, enables the build for the RISC-V
> CPU core, hardware, and Linux User Emulation. The 'qemu-binfmt-conf.sh'
> script is updated to add the RISC-V ELF magic.
>
> Expected checkpatch errors for consistency reasons:
>
> ERROR: line over 90 characters
> FILE: scripts/qemu-binfmt-conf.sh
> Signed-off-by: Michael Clark <address@hidden>
> ---
> +++ b/qapi-schema.json
> @@ -413,7 +413,7 @@
> # Since: 2.6
> ##
> { 'enum': 'CpuInfoArch',
> - 'data': ['x86', 'sparc', 'ppc', 'mips', 'tricore', 'other' ] }
> + 'data': ['x86', 'sparc', 'ppc', 'mips', 'tricore', 'riscv', 'other' ] }
Missing documentation that riscv was added in 2.12 (see QKeyCode in
qapi/ui.json for an enum that serves as an example of documenting
changes over time).
>
> ##
> +# @CpuInfoRISCV:
> +#
> +# Additional information about a virtual RISCV CPU
> +#
> +# @pc: the instruction pointer
> +#
> +# Since 2.8
2.12, actually.
> +##
> +{ 'struct': 'CpuInfoRISCV', 'data': { 'pc': 'int' } }
Should this be 'uint64' or other specific type, rather than the generic
'int' (which happens to be 64 bits, but signed)? Other architectures
use 'int' because of history, but we could use this chance to improve
things if desired.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v1 16/21] RISC-V VirtIO Machine, (continued)
- [Qemu-devel] [PATCH v1 16/21] RISC-V VirtIO Machine, Michael Clark, 2018/01/02
- [Qemu-devel] [PATCH v1 17/21] SiFive RISC-V UART Device, Michael Clark, 2018/01/02
- [Qemu-devel] [PATCH v1 19/21] SiFive Freedom E300 RISC-V Machine, Michael Clark, 2018/01/02
- [Qemu-devel] [PATCH v1 20/21] SiFive Freedom U500 RISC-V Machine, Michael Clark, 2018/01/02
- [Qemu-devel] [PATCH v1 21/21] RISC-V Build Infrastructure, Michael Clark, 2018/01/02
- Re: [Qemu-devel] [PATCH v1 21/21] RISC-V Build Infrastructure,
Eric Blake <=
- Re: [Qemu-devel] [PATCH v1 21/21] RISC-V Build Infrastructure, Antony Pavlov, 2018/01/04
Re: [Qemu-devel] [PATCH v1 00/21] RISC-V QEMU Port Submission v1, no-reply, 2018/01/02