From patchwork Fri Jul 30 17:02:44 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tim Gardner X-Patchwork-Id: 60369 X-Patchwork-Delegate: leann.ogasawara@canonical.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from chlorine.canonical.com (chlorine.canonical.com [91.189.94.204]) by ozlabs.org (Postfix) with ESMTP id 7BDA1B70A5 for ; Sat, 31 Jul 2010 03:03:19 +1000 (EST) Received: from localhost ([127.0.0.1] helo=chlorine.canonical.com) by chlorine.canonical.com with esmtp (Exim 4.69) (envelope-from ) id 1OeszI-0004DQ-9I; Fri, 30 Jul 2010 18:03:12 +0100 Received: from mail.tpi.com ([70.99.223.143]) by chlorine.canonical.com with esmtp (Exim 4.69) (envelope-from ) id 1OeszF-0004CK-Mb for kernel-team@lists.ubuntu.com; Fri, 30 Jul 2010 18:03:10 +0100 Received: from [10.0.2.5] (unknown [10.0.2.5]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mail.tpi.com (Postfix) with ESMTP id 7BFCD259F20; Fri, 30 Jul 2010 10:03:56 -0700 (PDT) Message-ID: <4C5305B4.7040500@canonical.com> Date: Fri, 30 Jul 2010 11:02:44 -0600 From: Tim Gardner User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.11) Gecko/20100713 Thunderbird/3.0.6 MIME-Version: 1.0 To: Leann Ogasawara Subject: Re: [Maverick][pull request] Xen pv-ops for EC2 References: <4C52E065.4030006@canonical.com> In-Reply-To: <4C52E065.4030006@canonical.com> Cc: Kernel team list X-BeenThere: kernel-team@lists.ubuntu.com X-Mailman-Version: 2.1.9 Precedence: list Reply-To: tim.gardner@canonical.com List-Id: Kernel team discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kernel-team-bounces@lists.ubuntu.com Errors-To: kernel-team-bounces@lists.ubuntu.com On 07/30/2010 08:23 AM, John Johansen wrote: > The following request pull enables -virtual kernel to function as a pv-ops > kernel on EC2. This does not include the pv-on-HVM drivers which are required > for Amazon's Compute Cloud, will be in a separate request. > > The following changes since commit 8f2ff69dce18ed856a8d1b93176f768b47eeed86: > > UBUNTU: SAUCE: drm -- stop early access to drm devices (2010-07-29 11:41:40 -0700) > > are available in the git repository at: > git://kernel.ubuntu.com/jj/ubuntu-maverick.git pvops > > John Johansen (3): > UBUNTU: SAUCE: fix pv-ops for legacy Xen > UBUNTU: SAUCE: blkfront: default to sd devices > UBUNTU: [Config] Build in drivers required for Xen pv-ops > > arch/x86/xen/enlighten.c | 1 + > .../abi/2.6.35-12.17/amd64/virtual.modules | 2 -- > .../abi/2.6.35-12.17/i386/virtual.modules | 2 -- > debian.master/config/amd64/config.flavour.generic | 2 ++ > debian.master/config/amd64/config.flavour.server | 2 ++ > debian.master/config/amd64/config.flavour.virtual | 2 ++ > debian.master/config/armel/config.common.armel | 2 ++ > debian.master/config/config.common.ubuntu | 2 -- > debian.master/config/i386/config.flavour.generic | 2 ++ > .../config/i386/config.flavour.generic-pae | 2 ++ > debian.master/config/i386/config.flavour.virtual | 2 ++ > drivers/block/xen-blkfront.c | 2 +- > 12 files changed, 16 insertions(+), 7 deletions(-) > A couple of minor commit log edits, and a rerun of updateconfigs after changing the Xen config options. rtg diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index 1ea06f8..f7ff4c7 100644 --- a/arch/x86/xen/enlighten.c +++ b/arch/x86/xen/enlighten.c @@ -794,6 +794,7 @@ static void xen_write_cr4(unsigned long cr4) { cr4 &= ~X86_CR4_PGE; cr4 &= ~X86_CR4_PSE; + cr4 &= ~X86_CR4_OSXSAVE; native_write_cr4(cr4); } -- 1.7.0.4 From e4abd74d96f7a74bb74b47354b6d6434894778bb Mon Sep 17 00:00:00 2001 From: John Johansen Date: Wed, 28 Jul 2010 11:44:04 -0700 Subject: [PATCH 2/3] UBUNTU: SAUCE: blkfront: default to sd devices EC2 defaults to "/dev/sda" rather than "/dev/xvd". Once they finally switch then this patch should be dropped. Signed-off-by: John Johansen Signed-off-by: Tim Gardner --- drivers/block/xen-blkfront.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c index 82ed403..cda9b5a 100644 --- a/drivers/block/xen-blkfront.c +++ b/drivers/block/xen-blkfront.c @@ -119,7 +119,7 @@ static DEFINE_SPINLOCK(blkif_io_lock); #define VDEV_IS_EXTENDED(dev) ((dev)&(EXTENDED)) #define BLKIF_MINOR_EXT(dev) ((dev)&(~EXTENDED)) -#define DEV_NAME "xvd" /* name in /dev */ +#define DEV_NAME "sd" /* name in /dev */ static int get_id_from_freelist(struct blkfront_info *info) { -- 1.7.0.4 From 10f627556319f67875502f3e3ffd8463f1b94427 Mon Sep 17 00:00:00 2001 From: John Johansen Date: Fri, 30 Jul 2010 03:29:31 -0700 Subject: [PATCH 3/3] UBUNTU: [Config] Build in drivers required for Xen pv-ops Build in the Xen drivers so that we don't require a ramdisk on EC2 instance store images. Signed-off-by: John Johansen Signed-off-by: Tim Gardner --- .../abi/2.6.35-12.17/amd64/virtual.modules | 2 -- .../abi/2.6.35-12.17/i386/virtual.modules | 2 -- debian.master/config/amd64/config.flavour.generic | 2 ++ debian.master/config/amd64/config.flavour.server | 2 ++ debian.master/config/amd64/config.flavour.virtual | 2 ++ debian.master/config/armel/config.common.armel | 2 ++ debian.master/config/config.common.ubuntu | 2 -- .../config/i386/config.flavour.generic-pae | 2 ++ debian.master/config/i386/config.flavour.virtual | 2 ++ 9 files changed, 12 insertions(+), 6 deletions(-) diff --git a/debian.master/abi/2.6.35-12.17/amd64/virtual.modules b/debian.master/abi/2.6.35-12.17/amd64/virtual.modules index 37b8980..ff1393a 100644 --- a/debian.master/abi/2.6.35-12.17/amd64/virtual.modules +++ b/debian.master/abi/2.6.35-12.17/amd64/virtual.modules @@ -422,11 +422,9 @@ wanrouter wp512 x25 xcbc -xen-blkfront xen-fbfront xenfs xen-kbdfront -xen-netfront xfrm4_mode_beet xfrm4_mode_transport xfrm4_mode_tunnel diff --git a/debian.master/abi/2.6.35-12.17/i386/virtual.modules b/debian.master/abi/2.6.35-12.17/i386/virtual.modules index f4fc5a8..5b49085 100644 --- a/debian.master/abi/2.6.35-12.17/i386/virtual.modules +++ b/debian.master/abi/2.6.35-12.17/i386/virtual.modules @@ -420,11 +420,9 @@ wanrouter wp512 x25 xcbc -xen-blkfront xen-fbfront xenfs xen-kbdfront -xen-netfront xfrm4_mode_beet xfrm4_mode_transport xfrm4_mode_tunnel diff --git a/debian.master/config/amd64/config.flavour.generic b/debian.master/config/amd64/config.flavour.generic index 1dae2db..e25d3d6 100644 --- a/debian.master/config/amd64/config.flavour.generic +++ b/debian.master/config/amd64/config.flavour.generic @@ -14,3 +14,5 @@ CONFIG_VIRTIO_BLK=m CONFIG_VIRTIO_NET=m CONFIG_VIRTIO_PCI=m CONFIG_VIRTIO_RING=m +CONFIG_XEN_BLKDEV_FRONTEND=m +CONFIG_XEN_NETDEV_FRONTEND=m diff --git a/debian.master/config/amd64/config.flavour.server b/debian.master/config/amd64/config.flavour.server index d9bd9be..43c6c71 100644 --- a/debian.master/config/amd64/config.flavour.server +++ b/debian.master/config/amd64/config.flavour.server @@ -14,3 +14,5 @@ CONFIG_VIRTIO_BLK=y CONFIG_VIRTIO_NET=y CONFIG_VIRTIO_PCI=y CONFIG_VIRTIO_RING=y +CONFIG_XEN_BLKDEV_FRONTEND=m +CONFIG_XEN_NETDEV_FRONTEND=m diff --git a/debian.master/config/amd64/config.flavour.virtual b/debian.master/config/amd64/config.flavour.virtual index 442a3b1..54164a3 100644 --- a/debian.master/config/amd64/config.flavour.virtual +++ b/debian.master/config/amd64/config.flavour.virtual @@ -14,3 +14,5 @@ CONFIG_VIRTIO_BLK=y CONFIG_VIRTIO_NET=y CONFIG_VIRTIO_PCI=y CONFIG_VIRTIO_RING=y +CONFIG_XEN_BLKDEV_FRONTEND=y +CONFIG_XEN_NETDEV_FRONTEND=y diff --git a/debian.master/config/armel/config.common.armel b/debian.master/config/armel/config.common.armel index 4147c7b..0f0e053 100644 --- a/debian.master/config/armel/config.common.armel +++ b/debian.master/config/armel/config.common.armel @@ -127,4 +127,6 @@ CONFIG_VIRTIO_RING=m # CONFIG_WM8350_WATCHDOG is not set # CONFIG_X25_ASY is not set # CONFIG_X86_X2APIC is not set +CONFIG_XEN_BLKDEV_FRONTEND=m +CONFIG_XEN_NETDEV_FRONTEND=m CONFIG_ZONE_DMA_FLAG=0 diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu index 3127524..804967a 100644 --- a/debian.master/config/config.common.ubuntu +++ b/debian.master/config/config.common.ubuntu @@ -5006,14 +5006,12 @@ CONFIG_X86_XADD=y CONFIG_XEN=y CONFIG_XENFS=m CONFIG_XEN_BALLOON=y -CONFIG_XEN_BLKDEV_FRONTEND=m CONFIG_XEN_COMPAT_XENFS=y # CONFIG_XEN_DEBUG_FS is not set CONFIG_XEN_DEV_EVTCHN=m CONFIG_XEN_FBDEV_FRONTEND=m CONFIG_XEN_KBDDEV_FRONTEND=m CONFIG_XEN_MAX_DOMAIN_MEMORY=32 -CONFIG_XEN_NETDEV_FRONTEND=m CONFIG_XEN_SAVE_RESTORE=y CONFIG_XEN_SCRUB_PAGES=y CONFIG_XEN_SYS_HYPERVISOR=y diff --git a/debian.master/config/i386/config.flavour.generic-pae b/debian.master/config/i386/config.flavour.generic-pae index d227cdc..d14011f 100644 --- a/debian.master/config/i386/config.flavour.generic-pae +++ b/debian.master/config/i386/config.flavour.generic-pae @@ -13,3 +13,5 @@ CONFIG_VIRTIO_BLK=y CONFIG_VIRTIO_NET=y CONFIG_VIRTIO_PCI=y CONFIG_VIRTIO_RING=y +CONFIG_XEN_BLKDEV_FRONTEND=m +CONFIG_XEN_NETDEV_FRONTEND=m diff --git a/debian.master/config/i386/config.flavour.virtual b/debian.master/config/i386/config.flavour.virtual index b006b26..327b2f2 100644 --- a/debian.master/config/i386/config.flavour.virtual +++ b/debian.master/config/i386/config.flavour.virtual @@ -13,3 +13,5 @@ CONFIG_VIRTIO_BLK=y CONFIG_VIRTIO_NET=y CONFIG_VIRTIO_PCI=y CONFIG_VIRTIO_RING=y +CONFIG_XEN_BLKDEV_FRONTEND=y +CONFIG_XEN_NETDEV_FRONTEND=y