WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops
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
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-type:content-transfer-encoding; bh=YZh9yaW8OwGtEB5v8vLzbGc9H6xU/a9zmxnXioJCo5w=; b=GgvS0mmOnvAPir+5/NAJAR664CgwDxRMJzITQ/R8RM989s9uREaUhU+9DcMm+8het2 kR6bCwJ7pXAuTlPxvzpFIZJyA7eTw2p/rsFBk87XyUoUIpe+8+CmypiKOlBzLMXudOvM 0UbreC6wgxNe3tknk+nRPhcXYate4tGPbzbsQ=
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=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20110303182909.GA10812@xxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <AANLkTikUyYpVKG_LZHFLL-ziCfU-Oc3qM9uFCd7QH13Z@xxxxxxxxxxxxxx> <20110303182909.GA10812@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
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