From: Vitaly Kuznetsov <vkuznets@redhat.com>
To: Jim Mattson <jmattson@google.com>
Cc: kvm list <kvm@vger.kernel.org>,
        "the arch\/x86 maintainers" <x86@kernel.org>,
        Paolo Bonzini <pbonzini@redhat.com>,
        Radim =?utf-8?B?S3LEjW3DocWZ?= <rkrcmar@redhat.com>,
        "K. Y. Srinivasan" <kys@microsoft.com>,
        Haiyang Zhang <haiyangz@microsoft.com>,
        Stephen Hemminger <sthemmin@microsoft.com>,
        "Michael Kelley \(EOSG\)" <Michael.H.Kelley@microsoft.com>,
        Mohammed Gamal <mmorsy@redhat.com>, Cathy Avery <cavery@redhat.com>,
        Bandan Das <bsd@redhat.com>, Roman Kagan <rkagan@virtuozzo.com>,
        LKML <linux-kernel@vger.kernel.org>, devel@linuxdriverproject.org
Subject: Re: [PATCH RFC 2/7] KVM: nVMX: modify vmcs12 fields to match Hyper-V enlightened VMCS
References: <20171218171742.5765-1-vkuznets@redhat.com>
        <20171218171742.5765-3-vkuznets@redhat.com>
        <CALMp9eT2j3bVnLn1_=x6LiVR=3Akxkt8y0We4rpNOpfgMxCyRg@mail.gmail.com>
        <CALMp9eThLdEUEcVOuyARP59_tv9pu5keGw4N5TCZ7MXjh4OiWg@mail.gmail.com>
Date: Tue, 19 Dec 2017 13:25:11 +0100
In-Reply-To: <CALMp9eThLdEUEcVOuyARP59_tv9pu5keGw4N5TCZ7MXjh4OiWg@mail.gmail.com>
        (Jim Mattson's message of "Mon, 18 Dec 2017 13:28:37 -0800")
Message-ID: <87zi6elxaw.fsf@vitty.brq.redhat.com>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1051
Lines: 26

Jim Mattson <jmattson@google.com> writes:

> At this point in time, I don't think you can just blithely change the
> virtual VMCS layout and revision number. Existing VMs using the old
> layout and revision number must continue to work on versions of kvm
> past this point. You could tie the layout and revision number changes
> to KVM_CAP_HYPERV_ENLIGHTENED_VMCS if you like, but kvm must be able
> to continue to service VMs using the previous layout and revision
> number in perpetuity.
>

I see what you mean. In case we need to keep migration of nested
workloads working between KVMs of different versions we can't (ever)
touch vmcs12.

The way to go in this case, I think, is to create a completely separate
enlightened_vmcs12 struct and use it when appropriate. We can't possibly
support migrating workloads which use enlightened VMCS to an old KVM
which doesn't support it.

P.S. "If there are changes in this struct, VMCS12_REVISION must be
changed." comment needs to be replaced with "Don't even think about
changing this" :-)

-- 
  Vitaly