From mboxrd@z Thu Jan  1 00:00:00 1970
From: Kai Huang <kai.huang@linux.intel.com>
Subject: Re: [PATCH] x86/EPT: defer enabling of A/D maintenance
 until PML get enabled
Date: Wed, 14 Oct 2015 09:19:29 +0800
Message-ID: <561DADA1.3050303@linux.intel.com>
References: <56096DE002000078000A63EE@prv-mh.provo.novell.com>
	<CAB+V_5MgNcF3r0mUG+BspLs0uiYdwe1hsDr6Qc01qbY-x_LNKQ@mail.gmail.com>
	<560BCD6B02000078000A6FF0@prv-mh.provo.novell.com>
	<CAB+V_5My4exzzwP=+LHifGKkwsGU9uxAk60rdN_DtBvUc70B8A@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta5.messagelabs.com ([195.245.231.135])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <kai.huang@linux.intel.com>) id 1ZmAmW-0002NA-O6
	for xen-devel@lists.xenproject.org; Wed, 14 Oct 2015 01:23:04 +0000
In-Reply-To: <CAB+V_5My4exzzwP=+LHifGKkwsGU9uxAk60rdN_DtBvUc70B8A@mail.gmail.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Kai Huang <kaih.linux@gmail.com>, Jan Beulich <JBeulich@suse.com>
Cc: Andrew Cooper <andrew.cooper3@citrix.com>, Kevin Tian <kevin.tian@intel.com>, Jun Nakajima <jun.nakajima@intel.com>, xen-devel <xen-devel@lists.xenproject.org>
List-Id: xen-devel@lists.xenproject.org

Hi Jan,

Our QA tested this patch but this patch broke PML. Neither GUI display 
(video ram tracking also uses PML) nor live migration works. I'll 
investigate what's wrong and get back to you.

Thanks,
-Kai

On 09/30/2015 08:45 PM, Kai Huang wrote:
> On Wed, Sep 30, 2015 at 5:54 PM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>> On 30.09.15 at 10:58, <kaih.linux@gmail.com> wrote:
>>> Good to me, if you have tested it. Sorry I cannot test it as I am
>>> taking vacation  until Oct.8.
>> Note how I asked for help with testing ...
>>
>>> On Mon, Sep 28, 2015 at 10:42 PM, Jan Beulich <JBeulich@suse.com> wrote:
>>>> There's no point in enabling the extra feature for every domain when
>>>> we're not meaning to use it (yet). Just setting the flag should be
>>>> sufficient - the domain is required to be paused for PML enabling
>>>> anyway, i.e. hardware will pick up the new setting the next time
>>>> each vCPU of the guest gets scheduled.
>>>>
>>>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>>>> Cc: Kai Huang <kai.huang@linux.intel.com>
>>>> ---
>>>> VT-x maintainers, Kai: Me lacking the hardware to test this, may I ask
>>>> for your help here?
>> ... here. This patch can certainly wait until you get back from
>> vacation.
> Thanks. I'll test it or ask someone has machine to test it after I get back.
>
> Thanks,
> -Kai
>> Jan
>>
>
>