Re: [PATCH] config: Add 'make kvmconfig'

[Arnaud Lacombe]

Hi,

On Thu, Oct 27, 2011 at 10:32 AM, Michal Marek <mmarek@xxxxxxx> wrote:
> On 27.10.2011 16:11, Pekka Enberg wrote:
>> On Thu, Oct 27, 2011 at 3:51 PM, Michal Marek <mmarek@xxxxxxx> wrote:
>>>> I like the feature - i hope it's expressed in the right Kbuild-ish
>>>> fashion.
>>>>
>>>> Acked-by: Ingo Molnar <mingo@xxxxxxx>
>>>
>>> I hope we won't now also be getting make nfsconfig, make paeconfig and
>>> whatnot :). Also, the lists of select statements might need updates from
>>> time to time, if new dependencies appear.
>>
>> The use case is to be able to take a .config posted in a bug report
>> and easily turn that into something that the KVM tool can boot.
>>
>> "kvmconfig" really represents a virtualized x86 architecture
>> configuration. Maybe the naming is off because it's not about
>> individual features like paeconfig or nfsconfig would be but rather
>> about turning on all the required options for virtualized x86 for the
>> KVM tool.
>
> OK, if it's just this single use case, then why not.
>
Still, it should not be a pretext to open the Pendora's box and have
15 different hack popping up in the future.

>> If you have better ideas how to do this, we're happy to change the
>> implementation.
>
> The implementation is not as important as the new user interface that
> people get used to. And I think the user interface is OK. Regarding
> implementation, expect that you'll have to add more select statements
> from time to time. But there is ATM no better solution to this.
>
what about providing hook for sourcing a given Makefile ? I can have a
look at that tonight.

 - Arnaud

> Michal
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
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/