Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Content-Type: text/plain;
        charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 11.1 \(3445.4.7\))
Subject: Re: [PATCH] KVM: VMX: re-add ple_gap module parameter
From:   Liran Alon <liran.alon@oracle.com>
In-Reply-To: <20181123120214.174457a4@doriath>
Date:   Fri, 23 Nov 2018 19:42:53 +0200
Cc:     Paolo Bonzini <pbonzini@redhat.com>, babu.moger@amd.com,
        kvm@vger.kernel.org, linux-kernel@vger.kernel.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <8049C3CD-E816-4270-A64F-0BB92BD56338@oracle.com>
References: <20181123120214.174457a4@doriath>
To:     Luiz Capitulino <lcapitulino@redhat.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


> On 23 Nov 2018, at 19:02, Luiz Capitulino <lcapitulino@redhat.com> =
wrote:
>=20
>=20
> Apparently, the ple_gap parameter was accidentally removed
> by commit c8e88717cfc6b36bedea22368d97667446318291. Add it
> back.
>=20
> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>

Weird that nobody noticed this when patch was applied=E2=80=A6 Thanks.
Reviewed-by: Liran Alon <liran.alon@oracle.com>

> ---
> arch/x86/kvm/vmx.c | 1 +
> 1 file changed, 1 insertion(+)
>=20
> diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
> index 4555077d69ce..be6f13f1c25f 100644
> --- a/arch/x86/kvm/vmx.c
> +++ b/arch/x86/kvm/vmx.c
> @@ -174,6 +174,7 @@ module_param_named(preemption_timer, =
enable_preemption_timer, bool, S_IRUGO);
>  * refer SDM volume 3b section 21.6.13 & 22.1.3.
>  */
> static unsigned int ple_gap =3D KVM_DEFAULT_PLE_GAP;
> +module_param(ple_gap, uint, 0444);
>=20
> static unsigned int ple_window =3D KVM_VMX_DEFAULT_PLE_WINDOW;
> module_param(ple_window, uint, 0444);
> --=20
> 2.17.2
>=20