yep, having up to 3 seconds worth of stale system data is too much time. Also, I think the builtins can just validate themselves - for example the kmod builtin can trivially call kmod_validate_resource(ctx) every time it's asked to load a module, and I think the other builtins that implement .validate probably also can self-validate too, and remove the udev_builtin_validate() call entirely.
I'll take another look next week to see if it can be expanded to cover all the builtins that do validation, and submit something upstream for discussion.
yep, having up to 3 seconds worth of stale system data is too much time. Also, I think the builtins can just validate themselves - for example the kmod builtin can trivially call kmod_validate_ resource( ctx) every time it's asked to load a module, and I think the other builtins that implement .validate probably also can self-validate too, and remove the udev_builtin_ validate( ) call entirely.
I have a small draft patch just to the kmod builtin that I'm testing: /code.launchpad .net/~ddstreet/ +git/systemd/ +ref/sf149300
https:/
I'll take another look next week to see if it can be expanded to cover all the builtins that do validation, and submit something upstream for discussion.