Re: [Xen-devel] [PATCH] fix rename: xenstore not fully updated

["Chun Yan Liu" <cyliu@xxxxxxxx>]

>>> On 11/17/2014 at 11:53 PM, in message <1416239594.5466.23.camel@xxxxxxxxxx>,
Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote: 
> On Mon, 2014-11-17 at 15:45 +0000, Ian Jackson wrote: 
> > > +    /* update backend 
> > > /local/domain/0/backend/<device>/<domid>/.../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? 

Sure. I also doubt why the domain field is added to the backend dir, and only
under some device types, like vkbd, console, etc. under vif and vbd, there is
no such filed. If no one uses that field, I'll remove that from backend dir.

- Chunyan

>  
> Thanks, 
>  
> Ian. 
>  
>  
>  



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel