* [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops @ 2011-03-03 16:46 Frank Pan 2011-03-03 18:29 ` Konrad Rzeszutek Wilk 0 siblings, 1 reply; 8+ messages in thread From: Frank Pan @ 2011-03-03 16:46 UTC (permalink / raw) To: xen-devel, Stefano Stabellini, Jeremy Fitzhardinge, Ian Campbell [-- Attachment #1: Type: text/plain, Size: 1089 bytes --] I use pvops kernel for HVM domU, and I find that shutdown watcher is 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 [-- Attachment #2: 0001-Fix-a-bug-that-shutdown-watcher-is-disabled-on-pvops.patch --] [-- Type: text/x-patch, Size: 811 bytes --] From 8d2402bd48ccc9eb5afe2aaa407d61ff715cb24c Mon Sep 17 00:00:00 2001 From: Frank Pan <frankpzh@gmail.com> 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 [-- Attachment #3: Type: text/plain, Size: 138 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-03 16:46 [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops Frank Pan @ 2011-03-03 18:29 ` Konrad Rzeszutek Wilk 2011-03-04 2:09 ` Frank Pan ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Konrad Rzeszutek Wilk @ 2011-03-03 18:29 UTC (permalink / raw) To: Frank Pan Cc: Ian Campbell, xen-devel, Jeremy Fitzhardinge, Stefano Stabellini 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@gmail.com> > 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@lists.xensource.com > http://lists.xensource.com/xen-devel ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-03 18:29 ` Konrad Rzeszutek Wilk @ 2011-03-04 2:09 ` Frank Pan 2011-03-04 8:41 ` Ian Campbell 2011-03-04 2:39 ` Frank Pan 2011-03-04 2:57 ` Frank Pan 2 siblings, 1 reply; 8+ messages in thread From: Frank Pan @ 2011-03-04 2:09 UTC (permalink / raw) To: Konrad Rzeszutek Wilk Cc: Ian Campbell, xen-devel, Jeremy Fitzhardinge, Stefano Stabellini 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@oracle.com> 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@gmail.com> >> 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@lists.xensource.com >> http://lists.xensource.com/xen-devel > > -- 潘震皓, Frank Pan Computer Science and Technology Tsinghua University ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-04 2:09 ` Frank Pan @ 2011-03-04 8:41 ` Ian Campbell 2011-03-04 9:29 ` Frank Pan 0 siblings, 1 reply; 8+ messages in thread From: Ian Campbell @ 2011-03-04 8:41 UTC (permalink / raw) To: Frank Pan Cc: xen-devel@lists.xensource.com, Stefano Stabellini, Jeremy Fitzhardinge, Konrad Rzeszutek Wilk On Fri, 2011-03-04 at 02:09 +0000, Frank Pan wrote: > 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. Is the issue you are seeing perhaps due to this patch being missing from that tree? commit a947f0f8f7012a5e8689a9cff7209ec6964ec154 Author: Stefano Stabellini <stefano.stabellini@eu.citrix.com> Date: Mon Oct 4 16:10:06 2010 +0100 xen: do not set xenstored_ready before xenbus_probe on hvm Register_xenstore_notifier should guarantee that the caller gets notified even if xenstore is already up. Therefore we revert "do not notify callers from register_xenstore_notifier" and set xenstored_read at the right time for PV on HVM guests too. In fact in case of PV on HVM guests xenstored is ready only after the platform pci driver has completed the initialization, so do not set xenstored_ready before the call to xenbus_probe(). This patch fixes a shutdown_event watcher registration bug that causes "xm shutdown" not to work properly. Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> Acked-by: Jeremy Fitzhardinge <jeremy@goop.org> If possible we should prefer to backport patches from newer kernels rather than diverge in the 2.6.32 branch. Stefano's "2.6.38-rc6-pvhvm" branch also contains this which looks related: commit 702d4eb9b3de4398ab99cf0a4e799e552c7ab756 Author: Stefano Stabellini <stefano.stabellini@eu.citrix.com> Date: Thu Dec 2 17:54:50 2010 +0000 xen: no need to delay xen_setup_shutdown_event for hvm guests anymore Now that xenstore_ready is used correctly for PV on HVM guests too, we don't need to delay the initialization of xen_setup_shutdown_event anymore. Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> Acked-by: Jeremy Fitzhardinge <jeremy@goop.org> Ian. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-04 8:41 ` Ian Campbell @ 2011-03-04 9:29 ` Frank Pan 2011-03-04 11:51 ` Ian Campbell 0 siblings, 1 reply; 8+ messages in thread From: Frank Pan @ 2011-03-04 9:29 UTC (permalink / raw) To: Ian Campbell Cc: xen-devel@lists.xensource.com, Stefano Stabellini, Jeremy Fitzhardinge, Konrad Rzeszutek Wilk [-- Attachment #1: Type: text/plain, Size: 2554 bytes --] > Is the issue you are seeing perhaps due to this patch being missing from > that tree? > > commit a947f0f8f7012a5e8689a9cff7209ec6964ec154 > Author: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > Date: Mon Oct 4 16:10:06 2010 +0100 > > xen: do not set xenstored_ready before xenbus_probe on hvm > Stefano's "2.6.38-rc6-pvhvm" branch also contains this which looks > related: > > commit 702d4eb9b3de4398ab99cf0a4e799e552c7ab756 > Author: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > Date: Thu Dec 2 17:54:50 2010 +0000 > > xen: no need to delay xen_setup_shutdown_event for hvm guests anymore FP: Yes, both commits seem like a solution of the issue I met. > If possible we should prefer to backport patches from newer kernels > rather than diverge in the 2.6.32 branch. FP: The former commit seems ok. The latter one is more like a cleanup. Here is a patch can be applied on xen/next-2.6.32: --- linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c b/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c index 3a83ba2..f368db9 100644 --- a/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c +++ b/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c @@ -646,7 +646,10 @@ int register_xenstore_notifier(struct notifier_block *nb) { int ret = 0; - blocking_notifier_chain_register(&xenstore_chain, nb); + if (xenstored_ready > 0) + ret = nb->notifier_call(nb, 0, NULL); + else + blocking_notifier_chain_register(&xenstore_chain, nb); return ret; } @@ -660,7 +663,7 @@ EXPORT_SYMBOL_GPL(unregister_xenstore_notifier); void xenbus_probe(struct work_struct *unused) { - BUG_ON((xenstored_ready <= 0)); + xenstored_ready = 1; /* Notify others that xenstore is up */ blocking_notifier_call_chain(&xenstore_chain, 0, NULL); @@ -722,7 +725,6 @@ static int __init xenbus_init(void) xen_store_interface = mfn_to_virt(xen_store_mfn); } else { - xenstored_ready = 1; if (xen_hvm_domain()) { uint64_t v = 0; err = hvm_get_parameter(HVM_PARAM_STORE_EVTCHN, &v); @@ -738,6 +740,7 @@ static int __init xenbus_init(void) xen_store_evtchn = xen_start_info->store_evtchn; xen_store_mfn = xen_start_info->store_mfn; xen_store_interface = mfn_to_virt(xen_store_mfn); + xenstored_ready = 1; } } -- 1.7.0.4 -- 潘震皓, Frank Pan Computer Science and Technology Tsinghua University [-- Attachment #2: 0001-xen-do-not-set-xenstored_ready-before-xenbus_probe-o.patch --] [-- Type: text/x-patch, Size: 1749 bytes --] From 1a37285b5d8fdbc5ef180318bdf42f4d561cbc06 Mon Sep 17 00:00:00 2001 From: Frank Pan <frankpzh@gmail.com> Date: Fri, 4 Mar 2011 17:20:04 +0800 Subject: [PATCH] xen: do not set xenstored_ready before xenbus_probe on hvm backport of commit a947f0f8f7012a5e8689a9cff7209ec6964ec154 --- linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c b/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c index 3a83ba2..f368db9 100644 --- a/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c +++ b/linux-2.6-xen/drivers/xen/xenbus/xenbus_probe.c @@ -646,7 +646,10 @@ int register_xenstore_notifier(struct notifier_block *nb) { int ret = 0; - blocking_notifier_chain_register(&xenstore_chain, nb); + if (xenstored_ready > 0) + ret = nb->notifier_call(nb, 0, NULL); + else + blocking_notifier_chain_register(&xenstore_chain, nb); return ret; } @@ -660,7 +663,7 @@ EXPORT_SYMBOL_GPL(unregister_xenstore_notifier); void xenbus_probe(struct work_struct *unused) { - BUG_ON((xenstored_ready <= 0)); + xenstored_ready = 1; /* Notify others that xenstore is up */ blocking_notifier_call_chain(&xenstore_chain, 0, NULL); @@ -722,7 +725,6 @@ static int __init xenbus_init(void) xen_store_interface = mfn_to_virt(xen_store_mfn); } else { - xenstored_ready = 1; if (xen_hvm_domain()) { uint64_t v = 0; err = hvm_get_parameter(HVM_PARAM_STORE_EVTCHN, &v); @@ -738,6 +740,7 @@ static int __init xenbus_init(void) xen_store_evtchn = xen_start_info->store_evtchn; xen_store_mfn = xen_start_info->store_mfn; xen_store_interface = mfn_to_virt(xen_store_mfn); + xenstored_ready = 1; } } -- 1.7.0.4 [-- Attachment #3: Type: text/plain, Size: 138 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-04 9:29 ` Frank Pan @ 2011-03-04 11:51 ` Ian Campbell 0 siblings, 0 replies; 8+ messages in thread From: Ian Campbell @ 2011-03-04 11:51 UTC (permalink / raw) To: Frank Pan, Jeremy Fitzhardinge Cc: xen-devel@lists.xensource.com, Stefano Stabellini, Jeremy Fitzhardinge, Konrad Rzeszutek Wilk On Fri, 2011-03-04 at 09:29 +0000, Frank Pan wrote: > > Is the issue you are seeing perhaps due to this patch being missing from > > that tree? > > > > commit a947f0f8f7012a5e8689a9cff7209ec6964ec154 > > Author: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > > Date: Mon Oct 4 16:10:06 2010 +0100 > > > > xen: do not set xenstored_ready before xenbus_probe on hvm > > > Stefano's "2.6.38-rc6-pvhvm" branch also contains this which looks > > related: > > > > commit 702d4eb9b3de4398ab99cf0a4e799e552c7ab756 > > Author: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > > Date: Thu Dec 2 17:54:50 2010 +0000 > > > > xen: no need to delay xen_setup_shutdown_event for hvm guests anymore > > FP: Yes, both commits seem like a solution of the issue I met. > > > If possible we should prefer to backport patches from newer kernels > > rather than diverge in the 2.6.32 branch. > > FP: The former commit seems ok. The latter one is more like a cleanup. > Here is a patch can be applied on xen/next-2.6.32: Thanks, but we should just cherry pick the original. I actually pulled this patch into the Debian Squeeze (2.6.32.x) kernel a while ago and it hasn't caused any problems TTBOMK. I kept forgetting to forward it on. Pull request below includes this path and a second one which I also pulled into the Debian kernel. Jeremy, please can you pull. Ian. The following changes since commit 892d2f052e979cf1916647c752b94cf62ec1c6dc: Jeremy Fitzhardinge (1): Merge commit 'v2.6.32.28' into xen/next-2.6.32 are available in the git repository at: git://xenbits.xen.org/people/ianc/linux-2.6.git pvhvm-2.6.32 Stefano Stabellini (2): xen: do not set xenstored_ready before xenbus_probe on hvm xen: do not initialize PV timers on HVM if !xen_have_vector_callback arch/x86/xen/time.c | 5 +++-- drivers/xen/xenbus/xenbus_probe.c | 8 ++++++-- 2 files changed, 9 insertions(+), 4 deletions(-) ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-03 18:29 ` Konrad Rzeszutek Wilk 2011-03-04 2:09 ` Frank Pan @ 2011-03-04 2:39 ` Frank Pan 2011-03-04 2:57 ` Frank Pan 2 siblings, 0 replies; 8+ messages in thread From: Frank Pan @ 2011-03-04 2:39 UTC (permalink / raw) To: Konrad Rzeszutek Wilk Cc: Ian Campbell, xen-devel, Jeremy Fitzhardinge, Stefano Stabellini > Which one? 2.6.32? stock 2.6.37? devel 2.6.38? I used branch xen/next-2.6.32 for testing. A quick code check shows this issue should also exist on: xen/next-2.6.37 xen/next-2.6.38 xen/stable xen/stable-2.6.32.x But I need tests on them. -- Frank Pan Computer Science and Technology Tsinghua University ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops 2011-03-03 18:29 ` Konrad Rzeszutek Wilk 2011-03-04 2:09 ` Frank Pan 2011-03-04 2:39 ` Frank Pan @ 2011-03-04 2:57 ` Frank Pan 2 siblings, 0 replies; 8+ messages in thread From: Frank Pan @ 2011-03-04 2:57 UTC (permalink / raw) To: Konrad Rzeszutek Wilk Cc: Ian Campbell, xen-devel, Jeremy Fitzhardinge, Stefano Stabellini Branch devel/next-2.6.38 on your tree has no such issue. In this branch, setup_shutdown_watcher will be called inside xen_setup_shutdown_event.(on PV-on-HVM) On Fri, Mar 4, 2011 at 2:29 AM, Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> 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@gmail.com> >> 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@lists.xensource.com >> http://lists.xensource.com/xen-devel > > -- 潘震皓, Frank Pan Computer Science and Technology Tsinghua University ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-03-04 11:51 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-03-03 16:46 [PATCH pvops] Fix a bug that shutdown watcher is disabled on pvops Frank Pan 2011-03-03 18:29 ` Konrad Rzeszutek Wilk 2011-03-04 2:09 ` Frank Pan 2011-03-04 8:41 ` Ian Campbell 2011-03-04 9:29 ` Frank Pan 2011-03-04 11:51 ` Ian Campbell 2011-03-04 2:39 ` Frank Pan 2011-03-04 2:57 ` Frank Pan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).