From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shriram Rajagopalan Subject: Re: [PATCH 0 of 8] Remove static variables from xc_domain_{save, restore}.c Date: Tue, 24 May 2011 09:47:35 -0400 Message-ID: References: Reply-To: rshriram@cs.ubc.ca Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1795243115==" Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: Jim Fehlig , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org --===============1795243115== Content-Type: multipart/alternative; boundary=00032555b34ef75f9304a405d6b3 --00032555b34ef75f9304a405d6b3 Content-Type: text/plain; charset=ISO-8859-1 On Tue, May 24, 2011 at 5:14 AM, Ian Campbell wrote: > Otherwise users which deal with multiple domains need to do their own > locking and cannot save/migrate multiple domains in parallel (should > they want to). > > Also made a bunch of cleanup along the way, mainly to make it easier > to figure out what was going on with the twisty maze of macros > redefining functions as macros and redefining the macros etc. I'm sure > there is plenty more straightening out which could be done but I don't > have the stomach for it this morning. > > I ll do it!!.. I have been waiting for this. Thanks a lot for cleaning up this chaff! I was under the impression that this was some arcane legacy code that shouldnt be touched. One particular thing that I would like to do is to factor out the write functions (outbuf_*, noncached_write, ratewrite*, etc) into a separate file and make it sort of pluggable. (selfish :P) I wanted to introduce a patch that would overlap outbuf flush operation and guest memory copy operation instead of the current model . This might be helpful for both Remus and live migration of large domains. Shriram, does this have any impact on Remus? > > From a cursory look, it doesnt look like it would impact Remus. I ll test it in my setup and revert soon. shriram --00032555b34ef75f9304a405d6b3 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
On Tue, May 24, 2011 at 5:14 AM, Ian Campbell <ian.campbell= @citrix.com> wrote:
Otherwise users which deal with multiple domains need to do their own
locking and cannot save/migrate multiple domains in parallel (should
they want to).

Also made a bunch of cleanup along the way, mainly to make it easier
to figure out what was going on with the twisty maze of macros
redefining functions as macros and redefining the macros etc. I'm sure<= br> there is plenty more straightening out which could be done but I don't<= br> have the stomach for it this morning.

I ll do it!!.. I have been waiting for this. Thanks a= lot for cleaning up this chaff!
I was under the impression that this wa= s some arcane legacy code that shouldnt
be touched. One particular thing= that I would like to do is to factor out the write functions
(outbuf_*, noncached_write, ratewrite*, etc) into a separate file and make = it sort of pluggable.

(selfish :P) I wanted to introduce a patch tha= t would overlap outbuf flush operation
and guest memory copy operation i= nstead of the current model <flush,copy,flush,copy..>.
This might be helpful for both Remus and live migration of large domains. <= br>
Shriram, does this have any impact on Remus?

From a cursory look, it doesnt look like it would impact= Remus. I ll test it
in my setup and revert soon.

shriram
--00032555b34ef75f9304a405d6b3-- --===============1795243115== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel --===============1795243115==--