bus="<bus filter option>"

On Thu, Apr 5, 2018 at 4:17 PM, Qi, Fuli <qi.f...@jp.fujitsu.com> wrote:
>> This seems to needlessly tie ndctl to systemd, it should be able to operate 
>> without
>> requiring systemd. I expect it would be straightforward to copy the 
>> configuration file
>> implementation from git.
> I have read the configuration file implementation of git, my understanding is 
> that git daemon does not
> have any options used to override default configuration.
> I want to confirm if the configuration file is only used for ndctl monitor.
> If yes, I do not think copy the configuration file implementation from git is 
> a good choice,
> because only getting keys and values from configuration file is needed for us 
> and the structure of
> configuration file implementation in git is too complexity.
> I prefer to borrow from udev[1], because the implementation in udev is 
> simpler and it seems ndctl also borrows a lot from udev.
> [1] https://git.kernel.org/pub/scm/linux/hotplug/udev.git/tree/src/libudev.c

Thank you for doing the due diligence on this investigation it is appreciated.

I think the simple udev approach is acceptable. Going back to the
proposed command line options of: --dimm-events, --namespace-events,
--region-events, --bus-events and device filter selectors (like the
ndctl list options) we can just have variables in the config file for
those, so:

dimm-events="<list of dimm events>"
namespace-events=="<list of namespace events>"
region-events=="<list of region events>"
bus-events="<list of bus events>"
dimm="<bus filter option>"
dimm="<dimm filter option>"
region="<region filter option>"
namespace="<namespace filter option>"

Linux-nvdimm mailing list

Reply via email to