On Wed, 16 Jul 2025, Jason Andryuk wrote:
> An error on one domain doesn't apply to others. Continue the loop over
> all domains if init_domain() fails.
>
> Signed-off-by: Jason Andryuk <jason.andr...@amd.com>
> ---
> tools/helpers/init-dom0less.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tools/helpers/init-dom0less.c b/tools/helpers/init-dom0less.c
> index 4336e0f418..bd213c5ccd 100644
> --- a/tools/helpers/init-dom0less.c
> +++ b/tools/helpers/init-dom0less.c
> @@ -377,7 +377,6 @@ int main(int argc, char **argv)
> rc = init_domain(xsh, xch, xfh, &info[i]);
> if (rc < 0) {
> fprintf(stderr, "init_domain failed.\n");
> - goto out;
init_domain doesn't always return < 0 on error
That said, this change is still good.
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
