Reviewed-by: Konstantin Kostiuk <kkost...@redhat.com>
On Thu, May 30, 2024 at 10:04 AM Philippe Mathieu-Daudé <phi...@linaro.org> wrote: > The 'blacklist' argument / config key are deprecated since commi > 582a098e6c ("qga: Replace 'blacklist' command line and config file > options by 'block-rpcs'"), time to remove them. > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > docs/about/deprecated.rst | 18 ------------------ > docs/about/removed-features.rst | 18 ++++++++++++++++++ > qga/main.c | 6 ------ > 3 files changed, 18 insertions(+), 24 deletions(-) > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst > index 40585ca7d5..a32e0e2564 100644 > --- a/docs/about/deprecated.rst > +++ b/docs/about/deprecated.rst > @@ -478,24 +478,6 @@ versions, aliases will point to newer CPU model > versions > depending on the machine type, so management software must > resolve CPU model aliases before starting a virtual machine. > > -QEMU guest agent > ----------------- > - > -``--blacklist`` command line option (since 7.2) > -''''''''''''''''''''''''''''''''''''''''''''''' > - > -``--blacklist`` has been replaced by ``--block-rpcs`` (which is a better > -wording for what this option does). The short form ``-b`` still stays > -the same and thus is the preferred way for scripts that should run with > -both, older and future versions of QEMU. > - > -``blacklist`` config file option (since 7.2) > -'''''''''''''''''''''''''''''''''''''''''''' > - > -The ``blacklist`` config file option has been renamed to ``block-rpcs`` > -(to be in sync with the renaming of the corresponding command line > -option). > - > Migration > --------- > > diff --git a/docs/about/removed-features.rst > b/docs/about/removed-features.rst > index fba0cfb0b0..deca066b21 100644 > --- a/docs/about/removed-features.rst > +++ b/docs/about/removed-features.rst > @@ -1129,4 +1129,22 @@ stable for some time and is now widely used. > The command line and feature set is very close to the removed > C implementation. > > +QEMU guest agent > +---------------- > + > +``--blacklist`` command line option (removed in 9.1) > +'''''''''''''''''''''''''''''''''''''''''''''''''''' > + > +``--blacklist`` has been replaced by ``--block-rpcs`` (which is a better > +wording for what this option does). The short form ``-b`` still stays > +the same and thus is the preferred way for scripts that should run with > +both, older and future versions of QEMU. > + > +``blacklist`` config file option (removed in 9.1) > +''''''''''''''''''''''''''''''''''''''''''''''''' > + > +The ``blacklist`` config file option has been renamed to ``block-rpcs`` > +(to be in sync with the renaming of the corresponding command line > +option). > + > .. _Intel discontinuance notification: > https://www.intel.com/content/www/us/en/content-details/781327/intel-is-discontinuing-ip-ordering-codes-listed-in-pdn2312-for-nios-ii-ip.html > diff --git a/qga/main.c b/qga/main.c > index bdf5344584..f4d5f15bb3 100644 > --- a/qga/main.c > +++ b/qga/main.c > @@ -1071,11 +1071,6 @@ static void config_load(GAConfig *config) > g_key_file_get_boolean(keyfile, "general", "retry-path", > &gerr); > } > > - if (g_key_file_has_key(keyfile, "general", "blacklist", NULL)) { > - g_warning("config using deprecated 'blacklist' key, should be > replaced" > - " with the 'block-rpcs' key."); > - blockrpcs_key = "blacklist"; > - } > if (g_key_file_has_key(keyfile, "general", blockrpcs_key, NULL)) { > config->bliststr = > g_key_file_get_string(keyfile, "general", blockrpcs_key, > &gerr); > @@ -1190,7 +1185,6 @@ static void config_parse(GAConfig *config, int argc, > char **argv) > { "path", 1, NULL, 'p' }, > { "daemonize", 0, NULL, 'd' }, > { "block-rpcs", 1, NULL, 'b' }, > - { "blacklist", 1, NULL, 'b' }, /* deprecated alias for > 'block-rpcs' */ > { "allow-rpcs", 1, NULL, 'a' }, > #ifdef _WIN32 > { "service", 1, NULL, 's' }, > -- > 2.41.0 > >