On Mon, Sep 21, 2009 at 11:33:12PM +0900, Ryota Ozaki wrote:
OK, fixed the two parts. Thanks for the review!
And the fixed patch is here!
ozaki-r
PS: git rebase works well ;-)
From abe363de43f5053a01593de6e634f654525cd8b4 Mon Sep 17 00:00:00 2001
From: Ryota Ozaki
On Thu, Sep 17, 2009 at 12:42:08AM +0900, Ryota Ozaki wrote:
+static int lxcDomainSuspend(virDomainPtr dom)
+{
+lxc_driver_t *driver = dom-conn-privateData;
+virDomainObjPtr vm;
+virDomainEventPtr event = NULL;
+int ret = -1;
+
+lxcDriverLock(driver);
+vm =
On Mon, Sep 21, 2009 at 9:05 PM, Daniel P. Berrange berra...@redhat.com wrote:
On Thu, Sep 17, 2009 at 12:42:08AM +0900, Ryota Ozaki wrote:
+static int lxcDomainSuspend(virDomainPtr dom)
+{
+ lxc_driver_t *driver = dom-conn-privateData;
+ virDomainObjPtr vm;
+ virDomainEventPtr
It looks relatively good, but one suggestion and two missing pieces:
Ryota Ozaki wrote:
snip
diff --git a/src/lxc_driver.c b/src/lxc_driver.c
index 0ec1e92..2399d98 100644
--- a/src/lxc_driver.c
+++ b/src/lxc_driver.c
@@ -1862,6 +1862,188 @@ static char *lxcGetHostname (virConnectPtr conn)
Hi Chris,
On Wed, Sep 16, 2009 at 8:15 PM, Chris Lalancette clala...@redhat.com wrote:
It looks relatively good, but one suggestion and two missing pieces:
Ryota Ozaki wrote:
snip
diff --git a/src/lxc_driver.c b/src/lxc_driver.c
index 0ec1e92..2399d98 100644
--- a/src/lxc_driver.c
+++
On Wed, Sep 16, 2009 at 9:41 PM, Ryota Ozaki ozaki.ry...@gmail.com wrote:
Hi Chris,
On Wed, Sep 16, 2009 at 8:15 PM, Chris Lalancette clala...@redhat.com wrote:
It looks relatively good, but one suggestion and two missing pieces:
(snip)
Suggestion: it might be better to sleep for shorter