Re: [tip:tools/kvm] kvm tools: Add conditional compilation of symbolresolving

[Cyrill Gorcunov]

On 05/22/2011 03:00 PM, Ingo Molnar wrote:
> 
> * Ingo Molnar <mingo@xxxxxxx> wrote:
> 
>>
>> * tip-bot for Cyrill Gorcunov <gorcunov@xxxxxxxxx> wrote:
>>
>>> diff --git a/tools/perf/feature-tests.mak b/tools/kvm/config/feature-tests.mak
>>> similarity index 83%
>>> copy from tools/perf/feature-tests.mak
>>> copy to tools/kvm/config/feature-tests.mak
>>
>> Btw, now that we have feature-tests.mak it would be nice to populate the checks 
>> for the various assumptions.
>>
>> One i recently ran into on a new box where i tried to install tools/kvm was:
>>
>> In file included from /usr/include/features.h:387:0,
>>                  from /usr/include/unistd.h:26,
>>                  from include/kvm/util.h:12,
>>                  from bios/e820.c:5:
>> /usr/include/gnu/stubs.h:7:27: fatal error: gnu/stubs-32.h: No such file or directory
>> compilation terminated.
>> make: *** [bios/bios-rom.bin] Error 1
>>
>> that's a dependency on glibc-dev[el].
> 
> Another detail: on 64-bit hosts the dependency is on gibc-dev[el].i686, i.e. 
> the 32-bit package.
> 
> Would it be simple to remove this dependency? It's not typically installed by 
> default on distros and it would be nice to make most of the kvm code build by 
> default almost everywhere.
> 
> Thanks,
> 
> 	Ingo

  I'll take a look if we really need it (note we need to compile 16bit code for
bios blob so it might eventually be a problem ;)

-- 
    Cyrill
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/