From commits-return-87420-archive-asf-public=cust-asf.ponee.io@cloudstack.apache.org Fri May 10 20:09:15 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id AC36B18061A for ; Fri, 10 May 2019 22:09:14 +0200 (CEST) Received: (qmail 41557 invoked by uid 500); 10 May 2019 20:09:14 -0000 Mailing-List: contact commits-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list commits@cloudstack.apache.org Received: (qmail 41545 invoked by uid 99); 10 May 2019 20:09:14 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 10 May 2019 20:09:14 +0000 From: GitBox To: commits@cloudstack.apache.org Subject: [GitHub] [cloudstack] svenvogel commented on issue #3319: Use virtio for platform emulator strings starting with Windows* Message-ID: <155751894901.23396.3064126631645503216.gitbox@gitbox.apache.org> Date: Fri, 10 May 2019 20:09:09 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit svenvogel commented on issue #3319: Use virtio for platform emulator strings starting with Windows* URL: https://github.com/apache/cloudstack/pull/3319#issuecomment-491415223 @skattoju4 @ustcweizhou @syed i will try to explain it. > @ustcweizhou the thinking behind always setting the root disk to ide was to be able to boot a windows vm regardless of whether or not virtio drivers were installed. That said setting all disks to virtio when selecting "Windows PV" when registering the template is one way to move forward. first i need to say is yes! lets me explain. how written in #3089 we have "NOT" only the problem with template! @DennisKonrad wrote: When we select "Windows Server 2016 (64-bit)" as OS Type for an **W2K16 cd image** the resulting VMs will not start with multiple disks attached. This is due to the usage of multiple IDE interfaces in the VM description XML. 1. we create a machine with "Windows Server 2016 (64-bit)" we have **one** disk with type root "ide". Bildschirmfoto 2019-05-10 um 21 50 40 2. after that we add **two more data disks** and they should be "NOT" ide because QEMU dont like this! When we use NFS the second and third disk will be a "virtio" disk automatically. this pr should "NOT" remove/break or add functionality. it should be correct the normal functionality for managed storage like NFS works and qemu it wants if you choose "Windows Server 201X..." as OS Type. its a bug they only will affect (at least what i know) managed storage. If any user wants "ONLY" virtio driver he can choose "other pv ..." OS Type. Bildschirmfoto 2019-05-10 um 21 58 11 there is no discussion about remove perfomance of ide or move all Microsoft Windows OS Types to virtio. it should clear for everybody here that virtio is faster than ide and yes we dont need for the root disk a driver. this is just as little the content of the discussion. i hope it is more clear now? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: users@infra.apache.org With regards, Apache Git Services