Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752098Ab2JSGr5 (ORCPT ); Fri, 19 Oct 2012 02:47:57 -0400 Received: from smtp.ctxuk.citrix.com ([62.200.22.115]:39659 "EHLO SMTP.EU.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750783Ab2JSGr4 (ORCPT ); Fri, 19 Oct 2012 02:47:56 -0400 X-IronPort-AV: E=Sophos;i="4.80,610,1344211200"; d="scan'208";a="15268085" Message-ID: <1350629274.28188.48.camel@dagon.hellion.org.uk> Subject: Re: [Xen-devel] [PATCH V3 2/6]: PVH: use native irq, enable callback, use HVM ring ops, smp, ... From: Ian Campbell To: Mukesh Rathor CC: Konrad Rzeszutek Wilk , "Xen-devel@lists.xensource.com" , "linux-kernel@vger.kernel.org" Date: Fri, 19 Oct 2012 07:47:54 +0100 In-Reply-To: <20121018150359.794768d2@mantra.us.oracle.com> References: <20121017173005.73a03eec@mantra.us.oracle.com> <1350557077.2460.113.camel@zakaz.uk.xensource.com> <20121018150359.794768d2@mantra.us.oracle.com> Organization: Citrix Systems, Inc. Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.4.3-1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2487 Lines: 54 On Thu, 2012-10-18 at 23:03 +0100, Mukesh Rathor wrote: > On Thu, 18 Oct 2012 11:44:37 +0100 > Ian Campbell wrote: > > > On Thu, 2012-10-18 at 01:30 +0100, Mukesh Rathor wrote: > > > PVH: make gdt_frames[]/gdt_ents into a union with {gdtaddr, gdtsz}, > > > PVH only needs to send down gdtaddr and gdtsz. irq.c: PVH uses > > > native_irq_ops. vcpu hotplug is currently not available for PVH. > > > events.c: setup callback vector for PVH. smp.c: This pertains to > > > bringing up smp vcpus. PVH runs in ring 0, so syscalls are native. > > > Also, the vcpu context is send down via the hcall to be set in the > > > vmcs. gdtaddr and gdtsz are unionionized as PVH only needs to send > > > these two to be set in the vmcs. Finally, PVH ring ops uses HVM > > > paths for xenbus. > > > > > > Signed-off-by: Mukesh Rathor > > > --- > > > arch/x86/include/asm/xen/interface.h | 11 +++++- > > > arch/x86/xen/irq.c | 5 ++- > > > arch/x86/xen/p2m.c | 2 +- > > > arch/x86/xen/smp.c | 75 > > > ++++++++++++++++++++++------------ drivers/xen/cpu_hotplug.c > > > | 4 +- drivers/xen/events.c | 9 ++++- > > > drivers/xen/xenbus/xenbus_client.c | 3 +- > > > > This patch seems to have been horribly whitespace damaged. > > Hmm.. not sure what happened. Resending. See another email. > > > Have you seen "git send-email" ? It's very useful for avoiding this > > sort of thing and also takes a lot of the grunt work out of reposting > > a series. > > > It also chains the patches as replies to the introductory zero-th mail > > -- which is something I've been meaning to ask you to do for a while. > > It's useful because it joins the series together in a thread which > > makes it easier to keep track of in my INBOX. > > I prefer different thread-sets for different version. Otherwise too > many emails makes much harder to manage and figure. Besides, as > comments come in, path x of y, may contain different files/changes in > a subsequent version. I meant one thread per posting, not one thread for all all them. Which seems to be what you are saying you prefer but not what you are doing. Ian. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/