Re: [PATCH] os-posix: Allow 'chroot' via '-run-with' and deprecate the old '-chroot' option

[Michael Tokarev]

30.06.2023 18:01, Thomas Huth wrote:
> We recently introduced "-run-with" for options that influence the
> runtime behavior of QEMU. This option has the big advantage that it
> can group related options (so that it is easier for the users to spot
> them) and that the options become introspectable via QMP this way.
> So let's start moving more switches into this option group, starting
> with "-chroot" now.
...
> +static QemuOptsList qemu_run_with_opts = {
> +    .name = "run-with",
> +    .head = QTAILQ_HEAD_INITIALIZER(qemu_run_with_opts.head),
> +    .desc = {
> +#if defined(CONFIG_LINUX)
> +        {
> +            .name = "async-teardown",
> +            .type = QEMU_OPT_BOOL,
> +        },
> +#endif
> +        {
> +            .name = "chroot",
> +            .type = QEMU_OPT_STRING,
> +        },
> +        { /* end of list */ }
> +    },
> +};
> +
> +static void register_teardown(void)
> +{
> +    qemu_add_opts(&qemu_run_with_opts);
> +}
> +opts_init(register_teardown);

Hmm.. Is it still register_teardown? :)

Other than that,

Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>

/mjt