From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Lutomirski Subject: Re: [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest Date: Tue, 26 Jan 2016 10:46:34 -0800 Message-ID: References: <1453498558-6028-1-git-send-email-boris.ostrovsky@oracle.com> <1453498558-6028-5-git-send-email-boris.ostrovsky@oracle.com> <20160122233218.GA20964@wotan.suse.de> <56A6488F.5030909@oracle.com> <20160125211203.GF20964@wotan.suse.de> <56A691F0.6020404@zytor.com> <56A6A178.9020200@oracle.com> <20160126183425.GH20964@wotan.suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1aO8dk-0000oD-0w for xen-devel@lists.xenproject.org; Tue, 26 Jan 2016 18:46:56 +0000 Received: by mail-lb0-f182.google.com with SMTP id dx2so6123125lbd.3 for ; Tue, 26 Jan 2016 10:46:54 -0800 (PST) In-Reply-To: <20160126183425.GH20964@wotan.suse.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Luis R. Rodriguez" Cc: Juergen Gross , Jeremy Fitzhardinge , Rusty Russell , "linux-kernel@vger.kernel.org" , David Vrabel , "H. Peter Anvin" , "xen-devel@lists.xenproject.org" , Boris Ostrovsky , Borislav Petkov , =?UTF-8?Q?Roger_Pau_Monn=C3=A9?= List-Id: xen-devel@lists.xenproject.org On Tue, Jan 26, 2016 at 10:34 AM, Luis R. Rodriguez wrote: > On Mon, Jan 25, 2016 at 05:28:08PM -0500, Boris Ostrovsky wrote: >> On 01/25/2016 04:21 PM, H. Peter Anvin wrote: >> >On 01/25/16 13:12, Luis R. Rodriguez wrote: >> >>>Perhaps, but someone would still have to set hardware_subarch. And >> >>>it's hvmlite_bootparams() that does it. >> >>No, Xen would do it as well, essentially all of hvmlite_bootparams() could be >> >>done in Xen. >> >> >> >Or a stub code. >> >> This patch in fact is the stub for Xen HVMlite guests, after we are >> done with it we jump to bare-metal startup code (i.e startup_32|64) > > Right the point is the stub need not be in Linux, I'll explain in the other > thread where I provided more details on the different known approaches. > ISTM if the Xen ABI-specified entry point has a different convention than the Linux native entry, then the stub should live in Linux. It would be just a couple if lines of code, right? The issue that caused headaches in the past isn't that there's code that's executed only on native, it's that there are whole big functions that are executed only on native for no good reason and that aren't clearly marked. If we had native_start_kernel and xen_start_kernel, and they both called very quickly in to common_start_kernel, it would be very clear what's going on. --Andy