From patchwork Mon Aug 5 14:39:51 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Raghavendra K T X-Patchwork-Id: 2838718 Return-Path: X-Original-To: patchwork-kvm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 6C3B29F485 for ; Mon, 5 Aug 2013 14:34:22 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id A095F201E4 for ; Mon, 5 Aug 2013 14:34:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 731AA201E3 for ; Mon, 5 Aug 2013 14:34:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753796Ab3HEOd7 (ORCPT ); Mon, 5 Aug 2013 10:33:59 -0400 Received: from e23smtp01.au.ibm.com ([202.81.31.143]:47565 "EHLO e23smtp01.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753804Ab3HEOd5 (ORCPT ); Mon, 5 Aug 2013 10:33:57 -0400 Received: from /spool/local by e23smtp01.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 6 Aug 2013 00:23:57 +1000 Received: from d23dlp02.au.ibm.com (202.81.31.213) by e23smtp01.au.ibm.com (202.81.31.207) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Tue, 6 Aug 2013 00:23:56 +1000 Received: from d23relay03.au.ibm.com (d23relay03.au.ibm.com [9.190.235.21]) by d23dlp02.au.ibm.com (Postfix) with ESMTP id 3A4EB2BB0054; Tue, 6 Aug 2013 00:33:47 +1000 (EST) Received: from d23av04.au.ibm.com (d23av04.au.ibm.com [9.190.235.139]) by d23relay03.au.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r75EXasj9961848; Tue, 6 Aug 2013 00:33:36 +1000 Received: from d23av04.au.ibm.com (loopback [127.0.0.1]) by d23av04.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id r75EXguM014738; Tue, 6 Aug 2013 00:33:46 +1000 Received: from [9.77.125.37] ([9.77.125.37]) by d23av04.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id r75EXVOh014428; Tue, 6 Aug 2013 00:33:32 +1000 Message-ID: <51FFB937.6050003@linux.vnet.ibm.com> Date: Mon, 05 Aug 2013 20:09:51 +0530 From: Raghavendra K T Organization: IBM User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 MIME-Version: 1.0 To: Paolo Bonzini , Gleb Natapov CC: Ingo Molnar , mingo@redhat.com, x86@kernel.org, tglx@linutronix.de, jeremy@goop.org, konrad wilk , hpa@zytor.com, linux-doc@vger.kernel.org, habanero@linux.vnet.ibm.com, xen-devel@lists.xensource.com, peterz@infradead.org, mtosatti@redhat.com, stefano stabellini , andi@firstfloor.org, attilio rao , ouyang@cs.pitt.edu, gregkh@suse.de, agraf@suse.de, chegu vinod , torvalds@linux-foundation.org, avi kivity , kvm@vger.kernel.org, linux-kernel@vger.kernel.org, riel@redhat.com, drjones@redhat.com, virtualization@lists.linux-foundation.org, srivatsa vaddagiri Subject: Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor References: <51F0ED31.3040200@linux.vnet.ibm.com> <20130731062440.GK28372@redhat.com> <51FA1087.9080908@linux.vnet.ibm.com> <20130802092539.GB28327@gmail.com> <20130802095406.GB30072@redhat.com> <20130805094603.GA29303@gmail.com> <20130805095901.GL2258@redhat.com> <20130805135222.GA32429@gmail.com> <1952910281.9619371.1375711538505.JavaMail.root@redhat.com> In-Reply-To: <1952910281.9619371.1375711538505.JavaMail.root@redhat.com> X-Content-Scanned: Fidelis XPS MAILER x-cbid: 13080514-1618-0000-0000-00000466BF69 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org X-Spam-Status: No, score=-5.2 required=5.0 tests=BAYES_00,KHOP_BIG_TO_CC, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On 08/05/2013 07:35 PM, Paolo Bonzini wrote: >>>> I guess you'd want to carry this in the KVM tree or so - maybe in a >>>> separate branch because it changes Xen as well? >>> >>> It changes KVM host and guest side, XEN and common x86 spinlock code. I >>> think it would be best to merge common x86 spinlock bits and guest side >>> KVM/XEN bits through tip tree and host KVM part will go through KVM >>> tree. If this is OK with you, Ingo, and XEN folks Raghavendra can send >>> two separate patch series one for the tip and one for KVM host side. >> >> Sure, that's fine - if the initial series works fine in isolation as well >> (i.e. won't break anything). > > It would be a big problem if it didn't! Raghavendra, please send the > two separate series as Gleb explained above. > Yes. Sure. The patches have been split in that way. Only thing I am thinking is about KVM_FEATURE_PV_UNHALT, and KVM_HC_KICK_CPU definition in the below hunk, that is needed by guest as well. may be this header file change can be a separate patch so that duplicate can be handled easily during merge? I do testing of all combination after splitting and post. the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/x86/include/uapi/asm/kvm_para.h b/arch/x86/include/uapi/asm/kvm_para.h index 06fdbd9..94dc8ca 100644 --- a/arch/x86/include/uapi/asm/kvm_para.h +++ b/arch/x86/include/uapi/asm/kvm_para.h @@ -23,6 +23,7 @@ #define KVM_FEATURE_ASYNC_PF 4 #define KVM_FEATURE_STEAL_TIME 5 #define KVM_FEATURE_PV_EOI 6 +#define KVM_FEATURE_PV_UNHALT 7 diff --git a/include/uapi/linux/kvm_para.h b/include/uapi/linux/kvm_para.h index cea2c5c..2841f86 100644 --- a/include/uapi/linux/kvm_para.h +++ b/include/uapi/linux/kvm_para.h @@ -19,6 +19,7 @@ #define KVM_HC_MMU_OP 2 #define KVM_HC_FEATURES 3 #define KVM_HC_PPC_MAP_MAGIC_PAGE 4 +#define KVM_HC_KICK_CPU 5 -- To unsubscribe from this list: send the line "unsubscribe kvm" in