Date: Wed, 02 Nov 2005 10:20:21 -0800 From: Maksim Yevmenkin <maksim.yevmenkin@savvis.net> To: Brooks Davis <brooks@one-eyed-alien.net> Cc: freebsd-bluetooth@freebsd.org, Yar Tikhiy <yar@comp.chem.msu.su>, freebsd-rc@freebsd.org Subject: Re: [RFC] rc.d integration for the bluetooth subsystem Message-ID: <43690365.60909@savvis.net> In-Reply-To: <20051102161311.GA8499@odin.ac.hmc.edu> References: <4353DBBC.2000508@savvis.net> <43541F79.6040008@ebs.gr> <43554BCE.7090309@savvis.net> <4355FD0C.2090702@ebs.gr> <4356D12F.7000006@savvis.net> <43576A9D.1050209@ebs.gr> <4357CEA5.1000308@savvis.net> <4357D9E2.6010701@ebs.gr> <4367E346.4080106@savvis.net> <20051102111709.GD2465@comp.chem.msu.su> <20051102161311.GA8499@odin.ac.hmc.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
Brooks Davis wrote: > On Wed, Nov 02, 2005 at 02:17:09PM +0300, Yar Tikhiy wrote: > >>On Tue, Nov 01, 2005 at 01:51:02PM -0800, Maksim Yevmenkin wrote: >> >>>please find the first draft of bluetooth rc.d scripts located at >>> >>>http://people.freebsd.org/~emax/bluetooth-rc.diff.txt >>> >>>this patch adds >>> >>>1) /etc/rc.d/bluetooth script that will be used to start and stop >>>bluetooth devices. it will be called by devd(8) in response to device >>>arrival and departure events. the script also supports _optional_ per >>>device configuration. per device configuration is stored in >>>/etc/rc.conf.d/bluetooth.$dev file, where $dev is the driver name of the >>>device, i.e. ubt0, sio4, btccc1 >>> >>>2) /etc/rc.conf.d/bluetooth.device.sample script. this is just an >>>example. it should be copied to /etc/rc.conf.d/bluetooth.$dev and then >>>defaults can be adjusted. once again if there is no >>>/etc/rc.conf.d/bluetooth.$dev script then reasonable defaults will be used. >> >>My concern is about putting things not related directly to system >>startup driven by rc(8) in /etc/rc.d and /etc/rc.conf.d directories. >>Perhaps it would be better to still use rc.subr as a source of great >>subroutines, but place the bluetooth scripts and configs in their >>own directories -- rc.subr should support this. > > I don't disagree, but we've already got three scripts like this in > /etc/rc.d (dhclient, power_profile, and wpa_supplicant) so I don't think > it's a big deal. IMO, the conf files are find (though I don't like the this was another thing that i was worried about too :) however, as you pointed out, rc.d already has few 'nostart' scripts. keep in mind that even though /etc/rc.d/bluetooth has 'nostart' keyword it is still possible to execute it by hand, i.e. '/etc/rc.d/bluetooth restart ubt0' and it will work. this way you could restart bluetooth stack without unplugging the device. i imagine one might want to tweak config and the restart the stack. imo, /etc/rc.d is a good place for bluetooth script. > idea of a .sample in /etc/rc.conf.d). There is some argument for moving > the scripts to another directory though. I'm not sure what we'd call > it though. ok, let me re-phrase the question then do you think that having multiple config files under /etc/rc.conf.d is a good idea? do you think that other subsystem might benefit from similar (to bluetooth) config style or bluetooth will be the only subsystem that uses it? i'd really hate to introduce somewhat new config style just for bluetooth. i really do not want people whine about it and ask why they cant put things into /etc/rc.conf (where the rest of config is). freebsd is not linux. adding or changing things should produce benefits that would overweight potential complains from users, imo. thanks, max
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?43690365.60909>