cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrija Panic <andrija.pa...@gmail.com>
Subject Re: jira 8535 opinion please
Date Wed, 05 Oct 2016 16:46:18 GMT
BTW, based on the code, I guess this only support host OS being
centos/redhat 6.5+ and 7+ ? We need to expand this code to cover Ubuntu
14.04 as well I guess.

cheers

On 5 October 2016 at 18:44, Andrija Panic <andrija.panic@gmail.com> wrote:

> Hi Gabriel,
>
> I'm personally not familiar with any code :) but that is the reaason my
> colegues are in CC here.
>
> What I believe I miss here (please correct me if I'm wrong) is the same
> hyperV relaxed feature for also "Windows 7" OS type (although not that
> important) but also for "Windows PV" OS type, since we run our customers
> VMs with virtio, and we need to choose "Windows PV" to actually use virtio
> paravirtalization on KVM side.
>
> Thanks a lot for the specific code, I believe this is a good starting
> point for us - and please feel free to correct me in my opinions.
>
> Thanks again,
> Andrija
>
> On 5 October 2016 at 14:59, Gabriel Beims Bräscher <gabrascher@gmail.com>
> wrote:
>
>> Hi Andrija,
>>
>> Seems that this issue has been forgotten for a while, thanks for bringing
>> it back.
>>
>> I did a fast review on the code (current Apache CloudStack master) and
>> found something that might be useful.
>> The method
>> com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.c
>> reateVMFromSpec(VirtualMachineTO)
>> has some commented lines when adding features (lines 1947-1953), those
>> lines were supposed to set the HyperV relaxed feature in case of Windows
>> Server 2008.
>>
>>             /*
>>              * if (vmTO.getOs().contains("Windows Server 2008") &&
>> hostOsVersion != null && ((hostOsVersion.first() == 6 &&
>> hostOsVersion.second() >= 5) || (hostOsVersion.first() >= 7))) {
>>              *    LibvirtVMDef.HyperVEnlightenmentFeatureDef hyv = new
>> LibvirtVMDef.HyperVEnlightenmentFeatureDef();
>>              *    hyv.setRelaxed(true);
>>              *    features.addHyperVFeature(hyv);
>>              * }
>>              */
>>
>>
>> Would it be a starting point? Are you familiar with this code?
>>
>> Cheers,
>> Gabriel.
>>
>> 2016-10-04 5:52 GMT-03:00 Andrija Panic <andrija.panic@gmail.com>:
>>
>> > Hi,
>> >
>> > may I please get anyones opinion on this:
>> > https://issues.apache.org/jira/browse/CLOUDSTACK-8353
>> >
>> > We woul like to solve it internally and comit to comunity - just need
>> some
>> > opinions or advices.
>> >
>> > "There is a bug in KVM that causes a BSOD for Windows 2008 R2 and 7 or
>> > earlier. fix was added in libvirt 1.1.1 The fix requires enabling the
>> > "hv_relaxed" option for the affected VMs.
>> > "
>> >
>> > thanks
>> >
>> > Andrija Panić
>> >
>>
>
>
>
> --
>
> Andrija Panić
>



-- 

Andrija Panić

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message