* [Qemu-devel] [PATCH] riscv: requires libfdt
@ 2018-04-19 10:51 KONRAD Frederic
2018-04-19 11:57 ` Bastian Koppelmann
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: KONRAD Frederic @ 2018-04-19 10:51 UTC (permalink / raw)
To: qemu-devel; +Cc: peter.maydell, mjc, palmer, sagark, kbastian, frederic.konrad
When compiling on a machine without libfdt installed the configure script
should try to get libfdt from the git or should die because otherwise
CONFIG_LIBFDT is not set and the build process end in an error in the link
phase.. eg:
hw/riscv/virt.o: In function `riscv_virt_board_init':
qemu/src/hw/riscv/virt.c:317: undefined reference to `qemu_fdt_setprop_cell'
qemu/src/hw/riscv/virt.c:319: undefined reference to `qemu_fdt_setprop_cell'
qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
collect2: error: ld returned 1 exit status
make[1]: *** [qemu-system-riscv64] Error 1
make: *** [subdir-riscv64-softmmu] Error 2
Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
---
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure
index 0a19b03..1587f08 100755
--- a/configure
+++ b/configure
@@ -3732,7 +3732,7 @@ fi
fdt_required=no
for target in $target_list; do
case $target in
- aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu)
+ aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu|riscv64-softmmu)
fdt_required=yes
;;
esac
--
1.8.3.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 10:51 [Qemu-devel] [PATCH] riscv: requires libfdt KONRAD Frederic
@ 2018-04-19 11:57 ` Bastian Koppelmann
2018-04-19 12:00 ` Paolo Bonzini
2018-04-19 14:29 ` KONRAD Frederic
2 siblings, 0 replies; 8+ messages in thread
From: Bastian Koppelmann @ 2018-04-19 11:57 UTC (permalink / raw)
To: KONRAD Frederic, qemu-devel; +Cc: peter.maydell, sagark, palmer, mjc
On 04/19/2018 12:51 PM, KONRAD Frederic wrote:
> When compiling on a machine without libfdt installed the configure script
> should try to get libfdt from the git or should die because otherwise
> CONFIG_LIBFDT is not set and the build process end in an error in the link
> phase.. eg:
>
> hw/riscv/virt.o: In function `riscv_virt_board_init':
> qemu/src/hw/riscv/virt.c:317: undefined reference to `qemu_fdt_setprop_cell'
> qemu/src/hw/riscv/virt.c:319: undefined reference to `qemu_fdt_setprop_cell'
> qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
> collect2: error: ld returned 1 exit status
> make[1]: *** [qemu-system-riscv64] Error 1
> make: *** [subdir-riscv64-softmmu] Error 2
>
> Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
> ---
Reviewed-by: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
Cheers,
Bastian
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 10:51 [Qemu-devel] [PATCH] riscv: requires libfdt KONRAD Frederic
2018-04-19 11:57 ` Bastian Koppelmann
@ 2018-04-19 12:00 ` Paolo Bonzini
2018-04-19 12:03 ` KONRAD Frederic
2018-04-19 12:11 ` Peter Maydell
2018-04-19 14:29 ` KONRAD Frederic
2 siblings, 2 replies; 8+ messages in thread
From: Paolo Bonzini @ 2018-04-19 12:00 UTC (permalink / raw)
To: KONRAD Frederic, qemu-devel; +Cc: peter.maydell, sagark, kbastian, palmer, mjc
On 19/04/2018 12:51, KONRAD Frederic wrote:
> When compiling on a machine without libfdt installed the configure script
> should try to get libfdt from the git or should die because otherwise
> CONFIG_LIBFDT is not set and the build process end in an error in the link
> phase.. eg:
>
> hw/riscv/virt.o: In function `riscv_virt_board_init':
> qemu/src/hw/riscv/virt.c:317: undefined reference to `qemu_fdt_setprop_cell'
> qemu/src/hw/riscv/virt.c:319: undefined reference to `qemu_fdt_setprop_cell'
> qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
> collect2: error: ld returned 1 exit status
> make[1]: *** [qemu-system-riscv64] Error 1
> make: *** [subdir-riscv64-softmmu] Error 2
>
> Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
> ---
> configure | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index 0a19b03..1587f08 100755
> --- a/configure
> +++ b/configure
> @@ -3732,7 +3732,7 @@ fi
> fdt_required=no
> for target in $target_list; do
> case $target in
> - aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu)
> + aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu|riscv64-softmmu)
> fdt_required=yes
> ;;
> esac
>
Maybe good for 2.12, even though it would make the final rc and final
release different?
Thanks,
Paolo
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 12:00 ` Paolo Bonzini
@ 2018-04-19 12:03 ` KONRAD Frederic
2018-04-19 12:11 ` Peter Maydell
1 sibling, 0 replies; 8+ messages in thread
From: KONRAD Frederic @ 2018-04-19 12:03 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: peter.maydell, sagark, kbastian, palmer, mjc
On 04/19/2018 02:00 PM, Paolo Bonzini wrote:
> On 19/04/2018 12:51, KONRAD Frederic wrote:
>> When compiling on a machine without libfdt installed the configure script
>> should try to get libfdt from the git or should die because otherwise
>> CONFIG_LIBFDT is not set and the build process end in an error in the link
>> phase.. eg:
>>
>> hw/riscv/virt.o: In function `riscv_virt_board_init':
>> qemu/src/hw/riscv/virt.c:317: undefined reference to `qemu_fdt_setprop_cell'
>> qemu/src/hw/riscv/virt.c:319: undefined reference to `qemu_fdt_setprop_cell'
>> qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
>> collect2: error: ld returned 1 exit status
>> make[1]: *** [qemu-system-riscv64] Error 1
>> make: *** [subdir-riscv64-softmmu] Error 2
>>
>> Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
>> ---
>> configure | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index 0a19b03..1587f08 100755
>> --- a/configure
>> +++ b/configure
>> @@ -3732,7 +3732,7 @@ fi
>> fdt_required=no
>> for target in $target_list; do
>> case $target in
>> - aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu)
>> + aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu|riscv64-softmmu)
>> fdt_required=yes
>> ;;
>> esac
>>
>
> Maybe good for 2.12, even though it would make the final rc and final
> release different?
>
> Thanks,
>
> Paolo
>
On my side it doesn't matter much if it is 2.12 or 2.13, up to
you guys.
Thanks,
Fred
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 12:00 ` Paolo Bonzini
2018-04-19 12:03 ` KONRAD Frederic
@ 2018-04-19 12:11 ` Peter Maydell
1 sibling, 0 replies; 8+ messages in thread
From: Peter Maydell @ 2018-04-19 12:11 UTC (permalink / raw)
To: Paolo Bonzini
Cc: KONRAD Frederic, QEMU Developers, Sagar Karandikar,
Bastian Koppelmann, Palmer Dabbelt, Michael Clark
On 19 April 2018 at 13:00, Paolo Bonzini <pbonzini@redhat.com> wrote:
> On 19/04/2018 12:51, KONRAD Frederic wrote:
>> --- a/configure
>> +++ b/configure
>> @@ -3732,7 +3732,7 @@ fi
>> fdt_required=no
>> for target in $target_list; do
>> case $target in
>> - aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu)
>> + aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu|riscv64-softmmu)
>> fdt_required=yes
>> ;;
>> esac
>>
>
> Maybe good for 2.12, even though it would make the final rc and final
> release different?
I strongly prefer not to have changes between final rc and actual
release, and I don't think this rises to the level of justifying
another rc, since there is an easy user workaround for it (pass
--enable-fdt to configure).
thanks
-- PMM
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 10:51 [Qemu-devel] [PATCH] riscv: requires libfdt KONRAD Frederic
2018-04-19 11:57 ` Bastian Koppelmann
2018-04-19 12:00 ` Paolo Bonzini
@ 2018-04-19 14:29 ` KONRAD Frederic
2018-04-19 16:56 ` Philippe Mathieu-Daudé
2 siblings, 1 reply; 8+ messages in thread
From: KONRAD Frederic @ 2018-04-19 14:29 UTC (permalink / raw)
To: qemu-devel; +Cc: peter.maydell, mjc, palmer, sagark, kbastian
On 04/19/2018 12:51 PM, KONRAD Frederic wrote:
> When compiling on a machine without libfdt installed the configure script
> should try to get libfdt from the git or should die because otherwise
> CONFIG_LIBFDT is not set and the build process end in an error in the link
> phase.. eg:
>
> hw/riscv/virt.o: In function `riscv_virt_board_init':
> qemu/src/hw/riscv/virt.c:317: undefined reference to `qemu_fdt_setprop_cell'
> qemu/src/hw/riscv/virt.c:319: undefined reference to `qemu_fdt_setprop_cell'
> qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
> collect2: error: ld returned 1 exit status
> make[1]: *** [qemu-system-riscv64] Error 1
> make: *** [subdir-riscv64-softmmu] Error 2
>
> Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
> ---
> configure | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index 0a19b03..1587f08 100755
> --- a/configure
> +++ b/configure
> @@ -3732,7 +3732,7 @@ fi
> fdt_required=no
> for target in $target_list; do
> case $target in
> - aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu)
> + aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu|riscv64-softmmu)
I just realized that riscv32 use it too so I'll correct with
riscv*-softmmu.
Cheers,
Fred
> fdt_required=yes
> ;;
> esac
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 14:29 ` KONRAD Frederic
@ 2018-04-19 16:56 ` Philippe Mathieu-Daudé
2018-04-19 23:37 ` Michael Clark
0 siblings, 1 reply; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2018-04-19 16:56 UTC (permalink / raw)
To: KONRAD Frederic, qemu-devel; +Cc: peter.maydell, mjc, palmer, sagark, kbastian
> On 04/19/2018 12:51 PM, KONRAD Frederic wrote:
>> When compiling on a machine without libfdt installed the configure script
>> should try to get libfdt from the git or should die because otherwise
>> CONFIG_LIBFDT is not set and the build process end in an error in the
>> link
>> phase.. eg:
>>
>> hw/riscv/virt.o: In function `riscv_virt_board_init':
>> qemu/src/hw/riscv/virt.c:317: undefined reference to
>> `qemu_fdt_setprop_cell'
>> qemu/src/hw/riscv/virt.c:319: undefined reference to
>> `qemu_fdt_setprop_cell'
>> qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
>> collect2: error: ld returned 1 exit status
>> make[1]: *** [qemu-system-riscv64] Error 1
>> make: *** [subdir-riscv64-softmmu] Error 2
>>
>> Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
>> ---
>> configure | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index 0a19b03..1587f08 100755
>> --- a/configure
>> +++ b/configure
>> @@ -3732,7 +3732,7 @@ fi
>> fdt_required=no
>> for target in $target_list; do
>> case $target in
>> -
>> aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu)
>>
>> +
>> aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-softmmu|mips64el-softmmu|riscv64-softmmu)
>>
>
> I just realized that riscv32 use it too so I'll correct with
> riscv*-softmmu.
Using "riscv*-softmmu":
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> fdt_required=yes
>> ;;
>> esac
>>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: requires libfdt
2018-04-19 16:56 ` Philippe Mathieu-Daudé
@ 2018-04-19 23:37 ` Michael Clark
0 siblings, 0 replies; 8+ messages in thread
From: Michael Clark @ 2018-04-19 23:37 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: KONRAD Frederic, QEMU Developers, Peter Maydell, Palmer Dabbelt,
Sagar Karandikar, Bastian Koppelmann
On Fri, Apr 20, 2018 at 4:56 AM, Philippe Mathieu-Daudé <f4bug@amsat.org>
wrote:
> > On 04/19/2018 12:51 PM, KONRAD Frederic wrote:
> >> When compiling on a machine without libfdt installed the configure
> script
> >> should try to get libfdt from the git or should die because otherwise
> >> CONFIG_LIBFDT is not set and the build process end in an error in the
> >> link
> >> phase.. eg:
> >>
> >> hw/riscv/virt.o: In function `riscv_virt_board_init':
> >> qemu/src/hw/riscv/virt.c:317: undefined reference to
> >> `qemu_fdt_setprop_cell'
> >> qemu/src/hw/riscv/virt.c:319: undefined reference to
> >> `qemu_fdt_setprop_cell'
> >> qemu/src/hw/riscv/virt.c:345: undefined reference to `qemu_fdt_dumpdtb'
> >> collect2: error: ld returned 1 exit status
> >> make[1]: *** [qemu-system-riscv64] Error 1
> >> make: *** [subdir-riscv64-softmmu] Error 2
> >>
> >> Signed-off-by: KONRAD Frederic <frederic.konrad@adacore.com>
> >> ---
> >> configure | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/configure b/configure
> >> index 0a19b03..1587f08 100755
> >> --- a/configure
> >> +++ b/configure
> >> @@ -3732,7 +3732,7 @@ fi
> >> fdt_required=no
> >> for target in $target_list; do
> >> case $target in
> >> -
> >> aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-
> softmmu|mips64el-softmmu)
> >>
> >> +
> >> aarch64*-softmmu|arm*-softmmu|ppc*-softmmu|microblaze*-
> softmmu|mips64el-softmmu|riscv64-softmmu)
> >>
> >
> > I just realized that riscv32 use it too so I'll correct with
> > riscv*-softmmu.
>
> Using "riscv*-softmmu":
>
Indeed.
We had this fix in our tree before one of the rebases against upstream:
https://github.com/riscv/riscv-qemu/commit/90cdfb86e81c54c1df42412b10b86fd83a6dee82
It must have somehow got dropped when we forward ported to QEMU master in
December 2017. My apologies.
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>
Reviewed-by: Michael Clark <mjc@sifive.com>
>> fdt_required=yes
> >> ;;
> >> esac
> >>
> >
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-04-19 23:37 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-19 10:51 [Qemu-devel] [PATCH] riscv: requires libfdt KONRAD Frederic
2018-04-19 11:57 ` Bastian Koppelmann
2018-04-19 12:00 ` Paolo Bonzini
2018-04-19 12:03 ` KONRAD Frederic
2018-04-19 12:11 ` Peter Maydell
2018-04-19 14:29 ` KONRAD Frederic
2018-04-19 16:56 ` Philippe Mathieu-Daudé
2018-04-19 23:37 ` Michael Clark
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.