* xl: Fix accidently waiting for domains to shutdown without --wait option
@ 2012-10-19 23:13 linux
2012-10-19 23:13 ` [PATCH 1/1] " linux
0 siblings, 1 reply; 4+ messages in thread
From: linux @ 2012-10-19 23:13 UTC (permalink / raw)
To: xen-devel; +Cc: Ian.Jackson, ian.campbell
xl: Fix accidently waiting for domains to shutdown without --wait option
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/1] xl: Fix accidently waiting for domains to shutdown without --wait option
2012-10-19 23:13 xl: Fix accidently waiting for domains to shutdown without --wait option linux
@ 2012-10-19 23:13 ` linux
2012-10-25 13:47 ` Sander Eikelenboom
2012-10-25 15:05 ` Ian Jackson
0 siblings, 2 replies; 4+ messages in thread
From: linux @ 2012-10-19 23:13 UTC (permalink / raw)
To: xen-devel; +Cc: Ian.Jackson, ian.campbell
From: Sander Eikelenboom <linux@eikelenboom>
Introduced by changeset 26091: "xl: Add --wait and --all to xl reboot."
Signed-off-by: Sander Eikelenboom <linux@eikelenboom>
---
tools/libxl/xl_cmdimpl.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
index dc80595..0e65f2b 100644
--- a/tools/libxl/xl_cmdimpl.c
+++ b/tools/libxl/xl_cmdimpl.c
@@ -3774,7 +3774,9 @@ static int main_shutdown_or_reboot(int reboot, int argc, char **argv)
fallback_trigger);
}
- wait_for_domain_deaths(deathws, nb_domain - 1 /* not dom 0 */);
+ if (wait_for_it)
+ wait_for_domain_deaths(deathws, nb_domain - 1 /* not dom 0 */);
+
libxl_dominfo_list_free(dominfo, nb_domain);
} else {
libxl_evgen_domain_death *deathw = NULL;
--
1.7.2.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] xl: Fix accidently waiting for domains to shutdown without --wait option
2012-10-19 23:13 ` [PATCH 1/1] " linux
@ 2012-10-25 13:47 ` Sander Eikelenboom
2012-10-25 15:05 ` Ian Jackson
1 sibling, 0 replies; 4+ messages in thread
From: Sander Eikelenboom @ 2012-10-25 13:47 UTC (permalink / raw)
To: linux; +Cc: Ian.Jackson, ian.campbell, xen-devel
Ping ?
Saturday, October 20, 2012, 1:13:04 AM, you wrote:
> From: Sander Eikelenboom <linux@eikelenboom>
> Introduced by changeset 26091: "xl: Add --wait and --all to xl reboot."
> Signed-off-by: Sander Eikelenboom <linux@eikelenboom>
> ---
> tools/libxl/xl_cmdimpl.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
> diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
> index dc80595..0e65f2b 100644
> --- a/tools/libxl/xl_cmdimpl.c
> +++ b/tools/libxl/xl_cmdimpl.c
> @@ -3774,7 +3774,9 @@ static int main_shutdown_or_reboot(int reboot, int argc, char **argv)
> fallback_trigger);
> }
>
> - wait_for_domain_deaths(deathws, nb_domain - 1 /* not dom 0 */);
> + if (wait_for_it)
> + wait_for_domain_deaths(deathws, nb_domain - 1 /* not dom 0 */);
> +
> libxl_dominfo_list_free(dominfo, nb_domain);
> } else {
> libxl_evgen_domain_death *deathw = NULL;
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] xl: Fix accidently waiting for domains to shutdown without --wait option
2012-10-19 23:13 ` [PATCH 1/1] " linux
2012-10-25 13:47 ` Sander Eikelenboom
@ 2012-10-25 15:05 ` Ian Jackson
1 sibling, 0 replies; 4+ messages in thread
From: Ian Jackson @ 2012-10-25 15:05 UTC (permalink / raw)
To: linux; +Cc: Ian.Jackson, ian.campbell, xen-devel
linux@eikelenboom.it writes ("[Xen-devel] [PATCH 1/1] xl: Fix accidently waiting for domains to shutdown without --wait option"):
> From: Sander Eikelenboom <linux@eikelenboom>
>
> Introduced by changeset 26091: "xl: Add --wait and --all to xl reboot."
>
> Signed-off-by: Sander Eikelenboom <linux@eikelenboom>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-10-25 15:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-19 23:13 xl: Fix accidently waiting for domains to shutdown without --wait option linux
2012-10-19 23:13 ` [PATCH 1/1] " linux
2012-10-25 13:47 ` Sander Eikelenboom
2012-10-25 15:05 ` Ian Jackson
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).