Revert two patches I've pushed mistakenly

[PATCH 0/2] Revert two patches I've pushed mistakenly

Michal Privoznik posted 2 patches 2 years, 1 month ago

Download series mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1648452529.git.mprivozn@redhat.com

docs/formatdomain.rst                         |  4 --
src/conf/domain_conf.c                        | 17 -------
src/conf/domain_conf.h                        | 10 -----
src/conf/schemas/domaincommon.rng             |  8 ----
src/qemu/qemu_capabilities.c                  |  2 -
src/qemu/qemu_capabilities.h                  |  1 -
src/qemu/qemu_command.c                       | 11 -----
src/qemu/qemu_validate.c                      | 45 ++++++++-----------
.../caps_7.0.0.x86_64.replies                 |  4 --
.../caps_7.0.0.x86_64.xml                     |  1 -
.../cpu-tsc-clear-on-reset.args               | 32 -------------
.../cpu-tsc-clear-on-reset.x86_64-7.0.0.args  | 34 --------------
.../cpu-tsc-clear-on-reset.xml                | 35 ---------------
tests/qemuxml2argvtest.c                      |  2 -
14 files changed, 19 insertions(+), 187 deletions(-)
delete mode 100644 tests/qemuxml2argvdata/cpu-tsc-clear-on-reset.args
delete mode 100644 tests/qemuxml2argvdata/cpu-tsc-clear-on-reset.x86_64-7.0.0.args
delete mode 100644 tests/qemuxml2argvdata/cpu-tsc-clear-on-reset.xml

Expand all Fold all

[PATCH 0/2] Revert two patches I've pushed mistakenly

Posted by Michal Privoznik 2 years, 1 month ago

Firstly, QEMU part is not merged which alone should have prevented me
from pushing these patches.

Secondly, in the end QEMU is fixing the bug in a different manner that
does not require any libvirt work.

https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg06077.html

https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg06188.html

Michal Prívozník (2):
  Revert "domain: add tsc.on_reboot element"
  Revert "qemu: add support for tsc.on_reboot element"

 docs/formatdomain.rst                         |  4 --
 src/conf/domain_conf.c                        | 17 -------
 src/conf/domain_conf.h                        | 10 -----
 src/conf/schemas/domaincommon.rng             |  8 ----
 src/qemu/qemu_capabilities.c                  |  2 -
 src/qemu/qemu_capabilities.h                  |  1 -
 src/qemu/qemu_command.c                       | 11 -----
 src/qemu/qemu_validate.c                      | 45 ++++++++-----------
 .../caps_7.0.0.x86_64.replies                 |  4 --
 .../caps_7.0.0.x86_64.xml                     |  1 -
 .../cpu-tsc-clear-on-reset.args               | 32 -------------
 .../cpu-tsc-clear-on-reset.x86_64-7.0.0.args  | 34 --------------
 .../cpu-tsc-clear-on-reset.xml                | 35 ---------------
 tests/qemuxml2argvtest.c                      |  2 -
 14 files changed, 19 insertions(+), 187 deletions(-)
 delete mode 100644 tests/qemuxml2argvdata/cpu-tsc-clear-on-reset.args
 delete mode 100644 tests/qemuxml2argvdata/cpu-tsc-clear-on-reset.x86_64-7.0.0.args
 delete mode 100644 tests/qemuxml2argvdata/cpu-tsc-clear-on-reset.xml

-- 
2.34.1

Re: [PATCH 0/2] Revert two patches I've pushed mistakenly

Posted by Jiri Denemark 2 years, 1 month ago

On Mon, Mar 28, 2022 at 09:31:46 +0200, Michal Privoznik wrote:
> Firstly, QEMU part is not merged which alone should have prevented me
> from pushing these patches.
> 
> Secondly, in the end QEMU is fixing the bug in a different manner that
> does not require any libvirt work.
> 
> https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg06077.html
> 
> https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg06188.html

Reviewed-by: Jiri Denemark <jdenemar@redhat.com>