From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vincent Hanquez Subject: Re: [PATCH] add a way to disable xen's udev script. Date: Fri, 17 Jun 2011 08:45:24 +0100 Message-ID: <4DFB0614.8090708@eu.citrix.com> References: <4DEFA993.1020803@eu.citrix.com> <1307554945.4176.47.camel@dagon.hellion.org.uk> <4DEFCC40.3000103@eu.citrix.com> <1307562567.4176.57.camel@dagon.hellion.org.uk> <4DEFDCB5.4020301@eu.citrix.com> <1307605345.775.758.camel@zakaz.uk.xensource.com> <4DF08BF4.3010904@eu.citrix.com> <1307611406.775.781.camel@zakaz.uk.xensource.com> <4DF09B50.1020401@eu.citrix.com> <1307689982.12738.34.camel@dagon.hellion.org.uk> <4DF5D128.4030208@eu.citrix.com> <1308058567.17937.92.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1308058567.17937.92.camel@zakaz.uk.xensource.com> 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: "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org On 06/14/2011 02:36 PM, Ian Campbell wrote: > On Mon, 2011-06-13 at 09:58 +0100, Vincent Hanquez wrote: > Oh, right, that makes sense, more so than the cfg file based scheme I > suggested (although eventually I expect that the will be needed too to > express the admin's wishes, that's not relevant here). Thanks for > explaining the actual usage. Right, I guess i should have started with that. it would have same us both some time. > I think we could make it opt-in for toolstacks which want it rather than > opt-out for those that don't, since only xend and xl want them (I > think???) that seems like a manageable change. The xencommons initscript > might work for touching the file, but perhaps the xend initscript and > something internal to libxl/xl (which doesn't have it's own initscript) > would be better. Only question is who removes it in the xl case. > > Since the file is automatically managed I don't think /etc is the right > place -- probably somewhere like /var/run (which handily gets it cleaned > up on reboot too). Yes. I don't mind too much about the details, /run (with a fallback to /var/run for old distrib) is probably better indeed. I'm not sure about xl, i had the idea that you have to switch away from xend when using xl by using an initscript. If that's not the case, then probably something internal would work. >> This feature has been discussed with toolstack people, patch has been >> send to maintainers already and it's about to be committed in one case >> (on two) provided it pass my tests i've left running over the weekend. > > Sounds interesting, do you have a link? I didn't see anything in the > xen-api archives, but perhaps it was elsewhere? (or is this a reference > some other toolstack?) No, unfortunately there's no record of the discussion since we had it directly. The udev module has been merged in the XCP tree [1]; The "only" thing left is switching the hotplug module to be using this module. In XCI the full patch is in (but I don't have any public link to it). [1] https://github.com/xen-org/xen-api-libs/commit/d6db355a4df93fafcf42cfb05df92dbae6a3b8f2 -- Vincent