[Buildroot] [PATCH 2/2] kvmtool: bump to f77d646ba0

Matthew Weber matthew.weber at rockwellcollins.com
Wed Dec 6 14:29:32 UTC 2017


Thomas,

On Wed, Dec 6, 2017 at 7:46 AM, Matthew Weber
<matthew.weber at rockwellcollins.com> wrote:
> Thomas,
>
> On Wed, Dec 6, 2017 at 1:59 AM, Thomas Petazzoni
> <thomas.petazzoni at free-electrons.com> wrote:
>>
>> Hello,
>>
>> On Tue, 5 Dec 2017 20:05:42 -0600, Matthew Weber wrote:
>>
>> > Gustavo, the latest kvmtool switched to doing the final link with gcc
>> > vs originally ld.  I believe this is the right fix.
>> >
>> >       3 --- a/package/kvmtool/kvmtool.mk
>> >       4 +++ b/package/kvmtool/kvmtool.mk
>> >       5 @@ -33,7 +33,7 @@ KVMTOOL_LICENSE_FILES = COPYING
>> >       6  # If more packages need this (unlikely) an ld wrapper might be a better
>> >       7  # solution, using gcc -dumpspecs information.
>> >       8  KVMTOOL_EXTRA_LDFLAGS = \
>> >       9 -       $(if $(BR2_x86_64),-m elf_x86_64)
>> >      10 +       $(if $(BR2_x86_64),-m64)
>>
>> If kvmtool is now building with gcc, do we need -m64 at all ?
>>
>
> That's true, gcc would already be assuming m64 based on the target's
> arch vs the linker wouldn't have been able to infer that.  I'll remove
> and kickoff a test.
>

Verified, test-pkg now passes and so does your test config.  Will send a v2.

Matt


More information about the buildroot mailing list