On 07/01/2022 20:07, Andy Shevchenko wrote:
> [...]
> + if (!buf)
> + return -EINVAL;
>
>
> I also implied that you don’t need this condition.
>
> + while ((func = strsep(&buf, ","))) {
> + addr = kallsyms_lookup_name(func);
>
>
>
> +
>
>
> Btw, this is redundant blank line.
>
>
> + if (!addr) {
Thanks Andy, just submitted a V4 that incorporates your suggestions.
Cheers,
Guilherme
_______________________________________________
kexec mailing list
kexec@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/kexec
