[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops


  • To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
  • From: Frank Pan <frankpzh@xxxxxxxxx>
  • Date: Fri, 4 Mar 2011 10:09:11 +0800
  • Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, Jeremy Fitzhardinge <Jeremy.Fitzhardinge@xxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
  • Delivery-date: Thu, 03 Mar 2011 18:10:21 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type:content-transfer-encoding; b=aG0MBAwSVDcobjG9Ef87eljqitAbtK7v8hF9Uja5v3UBGC+1d8cGoKf7xMbA92lHBs cxrVacBQAqkQIjjGPrzm8I1vpGRmSmtp0+vvItsnuSknJQ7FegHg/6TGyW/xJ0snwUNT F0o/yDQwkToBORq2a6eAojpTU8i9THN/vR3N0=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Sorry I didn't mention it. I use jeremy's tree.

git clone git://git.kernel.org/pub/scm/linux/kernel/git/jeremy/xen.git
linux-2.6-xen

The kernel version is 2.6.32.27.

On Fri, Mar 4, 2011 at 2:29 AM, Konrad Rzeszutek Wilk
<konrad.wilk@xxxxxxxxxx> wrote:
> On Fri, Mar 04, 2011 at 12:46:20AM +0800, Frank Pan wrote:
>> I use pvops kernel for HVM domU, and I find that shutdown watcher is
>
> Which one? 2.6.32? stock 2.6.37? devel 2.6.38?
>> not enabled at all when I compile platform-pci module inside kernel.
>> The code related calls xen_setup_shutdown_event after xenbus_probe,
>> that makes setup_shutdown_watcher never called.
>>
>> The following patch fixes this by changing the order of xenbus_probe
>> and xen_setup_shutdown, which makes my PV-on-HVM domU responses well
>> for the "control/shutdown" entry.
>>
>> ---
>> Âlinux-2.6-xen/drivers/xen/platform-pci.c | Â Â2 +-
>> Â1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/linux-2.6-xen/drivers/xen/platform-pci.c
>> b/linux-2.6-xen/drivers/xen/platform-pci.c
>> index c01b5dd..f141cfe 100644
>> --- a/linux-2.6-xen/drivers/xen/platform-pci.c
>> +++ b/linux-2.6-xen/drivers/xen/platform-pci.c
>> @@ -162,10 +162,10 @@ static int __devinit platform_pci_init(struct
>> pci_dev *pdev,
>> Â Â Â ret = gnttab_init();
>> Â Â Â if (ret)
>> Â Â Â Â Â Â Â goto out;
>> - Â Â xenbus_probe(NULL);
>> Â Â Â ret = xen_setup_shutdown_event();
>> Â Â Â if (ret)
>> Â Â Â Â Â Â Â goto out;
>> + Â Â xenbus_probe(NULL);
>> Â Â Â return 0;
>>
>> Âout:
>> --
>> 1.7.0.4
>>
>> --
>> Frank Pan
>>
>> Computer Science and Technology
>> Tsinghua University
>
>> From 8d2402bd48ccc9eb5afe2aaa407d61ff715cb24c Mon Sep 17 00:00:00 2001
>> From: Frank Pan <frankpzh@xxxxxxxxx>
>> Date: Thu, 3 Mar 2011 16:36:17 +0000
>> Subject: [PATCH] Fix a bug that shutdown watcher is disabled on pvops
>>
>> ---
>> Âlinux-2.6-xen/drivers/xen/platform-pci.c | Â Â2 +-
>> Â1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/linux-2.6-xen/drivers/xen/platform-pci.c 
>> b/linux-2.6-xen/drivers/xen/platform-pci.c
>> index c01b5dd..f141cfe 100644
>> --- a/linux-2.6-xen/drivers/xen/platform-pci.c
>> +++ b/linux-2.6-xen/drivers/xen/platform-pci.c
>> @@ -162,10 +162,10 @@ static int __devinit platform_pci_init(struct pci_dev 
>> *pdev,
>> Â Â Â ret = gnttab_init();
>> Â Â Â if (ret)
>> Â Â Â Â Â Â Â goto out;
>> - Â Â xenbus_probe(NULL);
>> Â Â Â ret = xen_setup_shutdown_event();
>> Â Â Â if (ret)
>> Â Â Â Â Â Â Â goto out;
>> + Â Â xenbus_probe(NULL);
>> Â Â Â return 0;
>>
>> Âout:
>> --
>> 1.7.0.4
>>
>
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel
>
>



-- 
æéç, Frank Pan

Computer Science and Technology
Tsinghua University

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.