qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] QEMU 3.0 ?


From: Paolo Bonzini
Subject: Re: [Qemu-devel] QEMU 3.0 ?
Date: Thu, 23 Nov 2017 15:13:22 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0

On 23/11/2017 15:01, Thomas Huth wrote:
> On 23.11.2017 14:57, Peter Maydell wrote:
>> On 23 November 2017 at 13:51, Paolo Bonzini <address@hidden> wrote:
>>> On 23/11/2017 14:13, Peter Maydell wrote:
>>>> On 23 November 2017 at 13:02, Paolo Bonzini <address@hidden> wrote:
>>>>> In theory I don't like it either (and I hadn't thought about it until
>>>>> today).  In practice, qemu-kvm is not going away from
>>>>> blogs/scripts/tutorials in a decade, so we might as well embrace it...
>>>> Isn't this distro-specific? In ubuntu by default there isn't
>>>> any wrapper, and if you do install the optional 'qemu-kvm' package
>>>> the wrapper it provides is /usr/bin/kvm, not /usr/bin/qemu-kvm.
>>>
>>> Fedora also has no wrapper in the qemu-system-x86 package, and only
>>> "qemu-kvm" installs one.  In practice if you install the virtualization
>>> package group you get it.  What about Ubuntu?
>>
>> Well, I didn't have the qemu-kvm package installed until I
>> pulled it in to check the wrapper name.
>>
>> My point is more that if there's no consensus between distros
>> about what the wrapper script name should be then as upstream
>> if we provide a qemu-kvm then we might be helping Fedora/RedHat
>> but we're just increasing confusion for those distros that
>> used a different name
>
> We could simply check whether argv[0] ends in "kvm" ... that should work
> with both "kvm" and "qemu-kvm".

We could also have two completely different binaries, one of which adds
a file accel/kvm/default-accel.c:

   const char *default_accel = "kvm:tcg";

(likewise for hax and hvf).  For everyone else stubs/default-accel.c
provides:

   const char *default_accel = "tcg";

It shouldn't be hard to add it to Makefile.target.  Then Debian can
install our qemu-kvm binary as /usr/bin/kvm.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]