[OpenWrt-Devel] [PATCH libubox] hotplug: add hotplug_call() helper
rafal at milecki.pl
Fri Dec 7 03:39:29 EST 2018
On 2018-12-07 04:41, Yousong Zhou wrote:
> On Thu, 6 Dec 2018 at 20:42, Rafał Miłecki <zajec5 at gmail.com> wrote:
>> From: Rafał Miłecki <rafal at milecki.pl>
>> This new function imlements common code needed to run /etc/hotplug.d/
>> listeners. It allows specifying subsystem and environment strings as
>> commonly used in the hotplug.d.
> hotplug-call is currently a OpenWrt-specific shell script from
> base-files. A library function's dependence on exact name and path of
> non-standard executable does not seem fit.
You're probably right, I guess the best solution would be to implement
hotplug-call script behavior directly in the libubox library.
> Hotplug events from the kernel is already being handled by
> hotplug.json script. Events from non-standard subsystem like "iface"
> as defined by netifd are generated and handled fine just by itself
> . As such, I cannot find enough other places needing this function
> to make it into libubox.
We could make netifd use that hotplug_call() helper, but now I noticed
that netifd simply uses setenv() after the fork(). That may be simpler
than building all that envp array.
I'm not sure now if it's better to work on:
1) Improved (native) hotplug_call() implementation in the libubox
2) Original idea of doing fork() in the fstools / block + setenv()
Implementing hotplug_call() was originally suggested by John, let's see
what's his opinion too.
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
More information about the openwrt-devel