From mboxrd@z Thu Jan 1 00:00:00 1970 From: Suravee Suthikulpanit Subject: Re: [PART1 RFC v4 01/11] KVM: x86: Misc LAPIC changes to expose helper functions Date: Tue, 19 Apr 2016 02:57:12 +0700 Message-ID: <57153C18.6020104@amd.com> References: <1460017232-17429-1-git-send-email-Suravee.Suthikulpanit@amd.com> <1460017232-17429-2-git-send-email-Suravee.Suthikulpanit@amd.com> <20160411203449.GA32480@potion.brq.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: , , , , , , , , To: =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= Return-path: Received: from mail-by2on0062.outbound.protection.outlook.com ([207.46.100.62]:58784 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752113AbcDRT5X (ORCPT ); Mon, 18 Apr 2016 15:57:23 -0400 In-Reply-To: <20160411203449.GA32480@potion.brq.redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Radim, On 04/12/2016 03:34 AM, Radim Kr=C4=8Dm=C3=A1=C5=99 wrote: > 2016-04-07 03:20-0500, Suravee Suthikulpanit: >> Exporting LAPIC utility functions and macros for re-use in SVM code. >> >> Signed-off-by: Suravee Suthikulpanit >> --- > > Adding kvm_lapic_set_reg looks weird when KVM has kvm_apic_get_reg, b= ut > the structure they accept is called kvm_lapic, so I vote to rename ge= t, > > Reviewed-by: Radim Kr=C4=8Dm=C3=A1=C5=99 > Sure. Within the same patch, I can rename kvm_apic_get_reg to=20 kvm_lapic_get_reg to be consistent. Thanks, Suravee