Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751698AbdCMNCz (ORCPT ); Mon, 13 Mar 2017 09:02:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57214 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750802AbdCMNCp (ORCPT ); Mon, 13 Mar 2017 09:02:45 -0400 From: Vitaly Kuznetsov To: Boris Ostrovsky Cc: xen-devel@lists.xenproject.org, Juergen Gross , x86@kernel.org, Andrew Jones , linux-kernel@vger.kernel.org Subject: Re: [Xen-devel] [PATCH v2 00/21] x86/xen: untangle PV and PVHVM guest support code References: <20170302175357.8222-1-vkuznets@redhat.com> <7ce41369-7784-862a-de8d-9cb2ecef83e1@oracle.com> Date: Mon, 13 Mar 2017 14:02:42 +0100 In-Reply-To: <7ce41369-7784-862a-de8d-9cb2ecef83e1@oracle.com> (Boris Ostrovsky's message of "Sun, 5 Mar 2017 12:59:08 -0500") Message-ID: <8760jdgwi5.fsf@vitty.brq.redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Mon, 13 Mar 2017 13:02:45 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1315 Lines: 36 Boris Ostrovsky writes: > On 03/02/2017 12:53 PM, Vitaly Kuznetsov wrote: >> Changes since v1: >> - Patches 1,2 and 3 were split and reordered to avoid adding temporary >> #ifdefs [Juergen Gross] >> - Juergen's R-b added to what is now patches 14 and 15 (patches 4 and 5 >> in v1). Due to re-ordering there are some tiny diffrences but I opted >> for keeping the tag. >> >> Some patches are known to produce checkpatch.pl WARNINGS and a couple of >> ERRORs, I fixed a few (mostly in _hvm* code I split) and I refrained from >> fixing the rest to make it easier to review. I think that we may leave PV >> code as it is as sooner or later it will go away. >> >> Original description: >> >> I have a long-standing idea to separate PV and PVHVM code in kernel and >> introduce Kconfig options to make it possible to enable the required >> parts only breaking the current 'all or nothing' approach. > > I tried applying this to mainline and it failed at some point. Is > there a branch i can pull from? > Sorry for the delayed response, I was on vacation last week. Just pushed this series to github, you can pull it from https://github.com/vittyvk/linux.git (xen_pv_hvm_split_v2 branch) I'll be rebasing and addressing Juergen's and kbuild's comments this week. -- Vitaly