On Thu, 12 Jul 2018 14:48:23 -0700 dsah...@kernel.org wrote: > @@ -4388,6 +4388,13 @@ static int ip6_route_multipath_add(struct fib6_config > *cfg, > rt = NULL; > goto cleanup; > } > + if (!rt6_qualify_for_ecmp(rt)) { > + err = EINVAL;
Shouldn't this be -EINVAL, just for consistency? E.g. we might return a -ENOMEM from ip6_route_info_append(), etc. -- Stefano