Hei,

Am 03/03/2017 um 03:32 AM schrieb Yousong Zhou:
> This is the default behaviour with modprobe from kmod package [1] unless
> it is explicitly told that the module is to be loaded for --first-time
> 
>  [1] http://www.kernel.org/pub/linux/utils/kernel/kmod/
> 
> Fixes FS#433
> 
> Signed-off-by: Yousong Zhou <yszhou4t...@gmail.com>
> ---
>  kmodloader.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/kmodloader.c b/kmodloader.c
> index 892ddd8..ed8f833 100644
> --- a/kmodloader.c
> +++ b/kmodloader.c
> @@ -837,7 +837,7 @@ static int main_modprobe(int argc, char **argv)
>       if (m && m->state == LOADED) {
>               if (!quiet)
>                       ULOG_ERR("%s is already loaded\n", name);
> -             return -1;
> +             return 0;
>       } else if (!m) {
>               if (!quiet)
>                       ULOG_ERR("failed to find a module named %s\n", name);

Thanks for picking this up.

What about returning 1 in the "} else if (!m) {" case, this is
mobprobe's behaviour, afaik.

Greetz, yanosz


-- 
For those of you without hope, we have rooms with color TV,
cable and air conditioning

_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev

Reply via email to