> -----Original Message----- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 16 November 2015 17:31 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; Stefano Stabellini; Ian Campbell; Wei Liu > Subject: RE: [RFC PATCH 2/6] libxl: stop using libxl__xs_mkdir() for > ~/control/shutdown > > Paul Durrant writes ("RE: [RFC PATCH 2/6] libxl: stop using libxl__xs_mkdir() > for ~/control/shutdown"): > > [Ian Jackson:] > > > Maybe it would be easier to rename libxl__xs_mkdir to > > > libxl__xs_mknode ? (It's probably too late to rename XS_MKDIR.) > > > > There is still the need to set the path to an empty value though, which is > not implicitly done by the XS_MKDIR. > > Under what circumstances would this path not contain an empty value > after XS_MKDIR ? >
In this case I believe you are correct, but my feeling was that people reading the code would be lulled into a false sense of security that XS_MKDIR always did the right thing to initialize a new path. Paul > Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel