From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH] fix rename: xenstore not fully updated Date: Mon, 17 Nov 2014 15:53:14 +0000 Message-ID: <1416239594.5466.23.camel@citrix.com> References: <1416215987-21571-1-git-send-email-cyliu@suse.com> <21610.6141.943750.141913@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <21610.6141.943750.141913@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Jackson Cc: wei.liu2@citrix.com, Chunyan Liu , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Mon, 2014-11-17 at 15:45 +0000, Ian Jackson wrote: > > + /* update backend /local/domain/0/backend///.../domain */ > > Um, what on earth creates that ? > > Worse, what reads it ? > > I think that putting this information in the backend directory is > entirely wrong. I concluded the same thing. Chunyan, I think it would be better to simply remove the code which adds this domain field under the backend dir in the first place, instead of adding all this complex code to try and keep it up to date. Can you do that in the next iteration please? Thanks, Ian.