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

Re: [Xen-users] DomU suspension/hibernation

  • To: Ian Campbell <Ian.Campbell@xxxxxxxxxx>, David Sutton <kantras@xxxxxxxxx>
  • From: Ian Murray <murrayie@xxxxxxxxxxx>
  • Date: Wed, 19 Jun 2013 13:57:50 +0100 (BST)
  • Cc: "xen-users@xxxxxxxxxxxxx" <xen-users@xxxxxxxxxxxxx>, Joshua Tuttle <jtuttle@xxxxxxxxx>
  • Delivery-date: Wed, 19 Jun 2013 12:58:33 +0000
  • Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=yahoo.co.uk; h=X-YMail-OSG:Received:X-Rocket-MIMEInfo:X-Mailer:References:Message-ID:Date:From:Reply-To:Subject:To:Cc:In-Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding; b=xXL7YoflyQnb/Rxw8OM5aj7vGj7hPUgXZlEzbhZyqSP5dpigyqe2a3oU7i++sITDG8cx61/XtjNZdsgw5VnFQcl8upYvuTrLvjySrrAv6DDsJnPMgXApyzpwDAf7LKAeernzpzMrnXKC4+a9pLP4QpOxNtt/MDADI0uHtgmiyKg=;
  • List-id: Xen user discussion <xen-users.lists.xen.org>

----- Original Message -----
> From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
> To: David Sutton <kantras@xxxxxxxxx>
> Cc: Ian Murray <murrayie@xxxxxxxxxxx>; "xen-users@xxxxxxxxxxxxx" 
> <xen-users@xxxxxxxxxxxxx>; Joshua Tuttle <jtuttle@xxxxxxxxx>
> Sent: Wednesday, 19 June 2013, 9:51
> Subject: Re: [Xen-users] DomU suspension/hibernation
> On Wed, 2013-06-19 at 03:04 -0500, David Sutton wrote:
>>  Thanks for the information. I decided to have another look over
>>  xendomains on xen-4.2.2 this evening, as I decided I wanted it to
>>  work. There were a few interesting things broken in there - for
>>  example, I can't see how the zombie testing code would work as neither
>>  formats of "xl list -l" give the state (and I couldn't see 
> how it
>>  would be trying to get that, even if it were)
>>  I managed to patch it so it was working for me. I rewrote most of the
>>  functions which had been using "xl list -l" to use
>>  "xenstore-list /local/domain" instead
> Thanks for doing this but I'm concerned that a change of this magnitude
> is too much for post rc5 of a release, assuming you were intending to
> propose this for 4.3 with a view to getting it backported to 4.2. We
> don't generally take stuff directly into 4.2.
> If someone could find a targeted fix to the JSON parsing then that has a
> much higher chance of acceptance, I think.
> If we are going to rewrite this code for 4.4 (which is something I think
> I would support!) then I think it might be better to do it as "xl
> autosave" and "xl autorestore" rather than continuing to try and 
> parse
> either JSON or SXP from a shell script (which is just a losing
> proposition really).
> Ian.


I think I have this sorted now. The JSON "{" that is used to signify the start 
of domain data is preceeded with 4 spaces in the xl list -l output. Add those 
four spaces into the regex (needs to be exact otherwise there will be more 
unwanted matches, I think) and put quotes around the variable in the grep 
statement(s) and it works for me... so a 5 line patch or so.

I will do more testing tonight and get a patch together if you are comfortable 
with the above in principle.

There is still an issue with xl list -l giving bogus domid's when using sxp 
format, but that's for another day. Maybe that is fixed in 4.3 already.


Ian M.

Xen-users mailing list



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