Hi Szymon, > This file is intended to be included from device init.rc. > --- > android/Android.mk | 16 +++++++++++++++- > android/Makefile.am | 1 + > android/init.bluetooth.rc | 27 +++++++++++++++++++++++++++ > 3 files changed, 43 insertions(+), 1 deletion(-) > create mode 100644 android/init.bluetooth.rc > > diff --git a/android/Android.mk b/android/Android.mk > index ae52ab4..f13e703 100644 > --- a/android/Android.mk > +++ b/android/Android.mk > @@ -109,7 +109,7 @@ LOCAL_MODULE := bluetooth.default > LOCAL_MODULE_PATH := $(TARGET_OUT_SHARED_LIBRARIES)/hw > LOCAL_MODULE_TAGS := optional > LOCAL_MODULE_CLASS := SHARED_LIBRARIES > -LOCAL_REQUIRED_MODULES := bluetoothd bluetoothd-snoop > +LOCAL_REQUIRED_MODULES := bluetoothd bluetoothd-snoop init.bluetooth.rc > > include $(BUILD_SHARED_LIBRARY) > > @@ -263,3 +263,17 @@ LOCAL_MODULE_TAGS := optional > LOCAL_MODULE := bluetoothd-snoop > > include $(BUILD_EXECUTABLE) > + > +# > +# init.bleutooth.rc > +# > + > +include $(CLEAR_VARS) > + > +LOCAL_MODULE := init.bluetooth.rc > +LOCAL_MODULE_CLASS := ETC > +LOCAL_SRC_FILES := $(LOCAL_MODULE) > +LOCAL_MODULE_TAGS := optional > +LOCAL_MODULE_PATH := $(TARGET_ROOT_OUT) > + > +include $(BUILD_PREBUILT) > diff --git a/android/Makefile.am b/android/Makefile.am > index 7806f79..b205019 100644 > --- a/android/Makefile.am > +++ b/android/Makefile.am > @@ -134,6 +134,7 @@ android_audio_a2dp_default_la_LDFLAGS = $(AM_LDFLAGS) -module -avoid-version \ > endif > > EXTRA_DIST += android/Android.mk android/hal-ipc-api.txt android/README \ > + android/init.bluetooth.rc \ > android/pics-gap.txt android/pics-hid.txt \ > android/pics-pan.txt android/pics-did.txt \ > android/pics-opp.txt android/pics-pbap.txt \ > diff --git a/android/init.bluetooth.rc b/android/init.bluetooth.rc > new file mode 100644 > index 0000000..e2be9e8 > --- /dev/null > +++ b/android/init.bluetooth.rc > @@ -0,0 +1,27 @@ > +chown bluetooth bluetooth /data/misc/bluetooth > + > +chown bluetooth bluetooth /dev/uhid > + > +on property:bluetooth.start=bluetoothd > + start bluetoothd > + > +on property:bluetooth.stop=bluetoothd > + stop bluetoothd > + > +on property:bluetooth.start=bluetoothd-snoop > + start bluetoothd-snoop > + > +on property:bluetooth.stop=bluetoothd-snoop > + stop bluetoothd-snoop I think this is actually racy. The ctl.start and ctl.stop sound like they have special handling inside Android init. If you accidentally set bluetooth.start , then bad things happen. So I would propose doing bluetooth.daemon=on/off and bluetooth.snoop=on/off We just have to make sure that when bluetoothd or bluetoothd-snoop die unexpectedly that these properties get set back to off. > + > +service bluetoothd /system/bin/logwrapper /system/bin/bluetoothd > + class main > + group bluetooth net_admin > + disabled > + oneshot > + > +service bluetoothd-snoop /system/bin/bluetoothd-snoop > + class main > + group bluetooth net_admin > + disabled > + oneshot Does net_admin include CAP_NET_RAW actually or are we using some sort of patched module for this? Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
