[Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer o

qemu-commits
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer o

From:	Richard Henderson
Subject:	[Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer of the 'Memory...
Date:	Wed, 13 Oct 2021 06:58:29 -0700
  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 60145485232cf1c4f3e9314ddb8a99d0473d785a
      
https://github.com/qemu/qemu/commit/60145485232cf1c4f3e9314ddb8a99d0473d785a
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Add myself as reviewer of the 'Memory API'

Having developed interest with the Memory API,
volunteer to review the patches.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Acked-by: David Hildenbrand <david@redhat.com>
Message-Id: <20211007091949.319404-1-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 8009806ec81ff2caa711c99e87ca394f9ba50565
      
https://github.com/qemu/qemu/commit/8009806ec81ff2caa711c99e87ca394f9ba50565
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M tests/Makefile.include

  Log Message:
  -----------
  tests: add missing dependency for check-block

qemu-iotests run qemu-storage-daemon, make sure it is up to date.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 097a5f19a357b6bc699a05ccd119bfa89ea409cc
      
https://github.com/qemu/qemu/commit/097a5f19a357b6bc699a05ccd119bfa89ea409cc
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M tests/Makefile.include

  Log Message:
  -----------
  build: fix "make check" without earlier "make"

"make check", if not preceded by "make", will not build the tools
needed by qemu-iotests.  This happens because qemu-iotests, aka
"make check-block", is not yet part of meson.build.

While at it, remove the reference to the now-dead QEMU_IOTESTS_HELPERS-y
variable.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: ecc00666a020368ec2c3712c40386dc7ea75a8a9
      
https://github.com/qemu/qemu/commit/ecc00666a020368ec2c3712c40386dc7ea75a8a9
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M tests/qemu-iotests/testrunner.py

  Log Message:
  -----------
  qemu-iotests: flush after every test

This makes it possible to see what is happening, even if the output of
"make check-block" is not sent to a tty (for example if it is sent to
grep or tee).

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 6bd17dccb63e152266cf7a7399931c9fe11bc8cf
      
https://github.com/qemu/qemu/commit/6bd17dccb63e152266cf7a7399931c9fe11bc8cf
  Author: Kacper Słomiński <kacper.slominski72@gmail.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M meson.build
    M util/compatfd.c

  Log Message:
  -----------
  util/compatfd.c: use libc signalfd wrapper instead of raw syscall

This allows the use of native signalfd instead of the sigtimedwait
based emulation on systems other than Linux.

Signed-off-by: Kacper Słomiński <kacper.slominski72@gmail.com>
Message-Id: <20210905011621.200785-1-kacper.slominski72@gmail.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: b50f6dc174cf391d725fca895d74c441b23e2e47
      
https://github.com/qemu/qemu/commit/b50f6dc174cf391d725fca895d74c441b23e2e47
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M qapi/misc-target.json

  Log Message:
  -----------
  qapi/misc-target: Wrap long 'SEV Attestation Report' long lines

Wrap long lines before 70 characters for legibility.

Suggested-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-2-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 993e26058d10756f5e206742c20b0cd03a9cb973
      
https://github.com/qemu/qemu/commit/993e26058d10756f5e206742c20b0cd03a9cb973
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M qapi/misc-target.json

  Log Message:
  -----------
  qapi/misc-target: Group SEV QAPI definitions

There is already a section with various SEV commands / types,
so move the SEV guest attestation together.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-3-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 02310f3a91470575a103857222d3a6e8acea338e
      
https://github.com/qemu/qemu/commit/02310f3a91470575a103857222d3a6e8acea338e
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/kvm/meson.build

  Log Message:
  -----------
  target/i386/kvm: Introduce i386_softmmu_kvm Meson source set

Introduce the i386_softmmu_kvm Meson source set to be able to
add features dependent on CONFIG_KVM.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-4-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 773ab6cb16b34272dc7eb8171824924c38dbeb90
      
https://github.com/qemu/qemu/commit/773ab6cb16b34272dc7eb8171824924c38dbeb90
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M accel/kvm/meson.build
    R accel/kvm/sev-stub.c
    M target/i386/kvm/meson.build
    A target/i386/kvm/sev-stub.c

  Log Message:
  -----------
  target/i386/kvm: Restrict SEV stubs to x86 architecture

SEV is x86-specific, no need to add its stub to other
architectures. Move the stub file to target/i386/kvm/.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-5-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 2c7233eb103adbf567778270b69950dd7a776c72
      
https://github.com/qemu/qemu/commit/2c7233eb103adbf567778270b69950dd7a776c72
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c
    M target/i386/sev.c

  Log Message:
  -----------
  target/i386/sev: Prefix QMP errors with 'SEV'

Multiple errors might be reported to the monitor,
better to prefix the SEV ones so we can distinct them.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20211007161716.453984-6-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 9f885cac701325ebcbf7e4393aa9b21b32ec3c12
      
https://github.com/qemu/qemu/commit/9f885cac701325ebcbf7e4393aa9b21b32ec3c12
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c

  Log Message:
  -----------
  target/i386/monitor: Return QMP error when SEV is not enabled for guest

If the management layer tries to inject a secret, it gets an empty
response in case the guest doesn't have SEV enabled, or the binary
is built without SEV:

  { "execute": "sev-inject-launch-secret",
    "arguments": { "packet-header": "mypkt", "secret": "mypass", "gpa": 
4294959104 }
  }
  {
      "return": {
      }
  }

Make it clearer by returning an error:

  { "execute": "sev-inject-launch-secret",
    "arguments": { "packet-header": "mypkt", "secret": "mypass", "gpa": 
4294959104 }
  }
  {
      "error": {
          "class": "GenericError",
          "desc": "SEV not enabled for guest"
      }
  }

Note: we will remove the sev_inject_launch_secret() stub in few commits,
      so we don't bother to add error_setg() there.

Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-7-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: f83aeeaeba1cf3e25d6292d9cc12064bfe2e78ab
      
https://github.com/qemu/qemu/commit/f83aeeaeba1cf3e25d6292d9cc12064bfe2e78ab
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/cpu.c

  Log Message:
  -----------
  target/i386/cpu: Add missing 'qapi/error.h' header

Commit 00b81053244 ("target-i386: Remove assert_no_error usage")
forgot to add the "qapi/error.h" for &error_abort, add it now.

Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-8-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 40cbafe05bb8f463d9dc096985ceff5d2535dc56
      
https://github.com/qemu/qemu/commit/40cbafe05bb8f463d9dc096985ceff5d2535dc56
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev-stub.c
    M target/i386/sev_i386.h

  Log Message:
  -----------
  target/i386/sev_i386.h: Remove unused headers

Declarations don't require these headers, remove them.

Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-9-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 4e7189e112966ac3a7bc884273a8b07291c3012e
      
https://github.com/qemu/qemu/commit/4e7189e112966ac3a7bc884273a8b07291c3012e
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev-stub.c
    M target/i386/sev.c
    M target/i386/sev_i386.h

  Log Message:
  -----------
  target/i386/sev: Remove sev_get_me_mask()

Unused dead code makes review harder, so remove it.

Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-10-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: d58d9f55a78a69730b4f1641f1c969d8585b970b
      
https://github.com/qemu/qemu/commit/d58d9f55a78a69730b4f1641f1c969d8585b970b
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev-stub.c

  Log Message:
  -----------
  target/i386/sev: Mark unreachable code with g_assert_not_reached()

The unique sev_encrypt_flash() invocation (in pc_system_flash_map)
is protected by the "if (sev_enabled())" check, so is not
reacheable.
Replace the abort() call in sev_es_save_reset_vector() by
g_assert_not_reached() which meaning is clearer.

Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-11-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: ed84ae720d0b28c50906412a903e598f75ee1d7c
      
https://github.com/qemu/qemu/commit/ed84ae720d0b28c50906412a903e598f75ee1d7c
  Author: Dr. David Alan Gilbert <dgilbert@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev.c

  Log Message:
  -----------
  target/i386/sev: sev_get_attestation_report use g_autofree

Removes a whole bunch of g_free's and a goto.

Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Reviewed-by: Brijesh Singh <brijesh.singh@amd.com>
Message-Id: <20210603113017.34922-1-dgilbert@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-12-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 2f573c415e8c84ec79aa6deb1d04fa9fccdebca7
      
https://github.com/qemu/qemu/commit/2f573c415e8c84ec79aa6deb1d04fa9fccdebca7
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev.c

  Log Message:
  -----------
  target/i386/sev: Use g_autofree in sev_launch_get_measure()

Use g_autofree to remove a pair of g_free/goto.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-13-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: e24b3332042b7372f4817694aec48650dcfa68a0
      
https://github.com/qemu/qemu/commit/e24b3332042b7372f4817694aec48650dcfa68a0
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/meson.build
    M target/i386/sev-stub.c
    A target/i386/sev-sysemu-stub.c

  Log Message:
  -----------
  target/i386/sev: Restrict SEV to system emulation

SEV is irrelevant on user emulation, so restrict it to sysemu.
Some stubs are still required because used in cpu.c by
x86_register_cpudef_types(), so move the sysemu specific stubs
to sev-sysemu-stub.c instead. This will allow us to simplify
monitor.c (which is not available in user emulation) in the
next commit.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-14-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 93777de3650e7db4b0434d63dd461505b85519f7
      
https://github.com/qemu/qemu/commit/93777de3650e7db4b0434d63dd461505b85519f7
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M hw/i386/x86.c
    M target/i386/cpu.c
    M target/i386/kvm/kvm.c
    M target/i386/monitor.c
    M target/i386/sev-stub.c
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c
    A target/i386/sev.h
    R target/i386/sev_i386.h

  Log Message:
  -----------
  target/i386/sev: Rename sev_i386.h -> sev.h

SEV is a x86 specific feature, and the "sev_i386.h" header
is already in target/i386/. Rename it as "sev.h" to simplify.

Patch created mechanically using:

  $ git mv target/i386/sev_i386.h target/i386/sev.h
  $ sed -i s/sev_i386.h/sev.h/ $(git grep -l sev_i386.h)

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20211007161716.453984-15-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: deae846f944a330e69109af8c807d6f2e66c95e6
      
https://github.com/qemu/qemu/commit/deae846f944a330e69109af8c807d6f2e66c95e6
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M hw/i386/pc_sysfw.c
    R include/sysemu/sev.h
    M target/i386/kvm/kvm.c
    M target/i386/kvm/sev-stub.c
    M target/i386/monitor.c
    M target/i386/sev.h

  Log Message:
  -----------
  target/i386/sev: Declare system-specific functions in 'sev.h'

"sysemu/sev.h" is only used from x86-specific files. Let's move it
to include/hw/i386, and merge it with target/i386/sev.h.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-16-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 02eacf31374152e6fb44554a4632f477d82dd9fc
      
https://github.com/qemu/qemu/commit/02eacf31374152e6fb44554a4632f477d82dd9fc
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/cpu.c
    M target/i386/meson.build
    R target/i386/sev-stub.c
    M target/i386/sev.h

  Log Message:
  -----------
  target/i386/sev: Remove stubs by using code elision

Only declare sev_enabled() and sev_es_enabled() when CONFIG_SEV is
set, to allow the compiler to elide unused code. Remove unnecessary
stubs.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20211007161716.453984-17-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 3208de1cd23937254a456b95ef16658b68821a13
      
https://github.com/qemu/qemu/commit/3208de1cd23937254a456b95ef16658b68821a13
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c
    M target/i386/sev.h

  Log Message:
  -----------
  target/i386/sev: Move qmp_query_sev_attestation_report() to sev.c

Move qmp_query_sev_attestation_report() from monitor.c to sev.c
and make sev_get_attestation_report() static. We don't need the
stub anymore, remove it.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-18-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 11a6ed0e77e8dd37ccea07575791b70c0410efea
      
https://github.com/qemu/qemu/commit/11a6ed0e77e8dd37ccea07575791b70c0410efea
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c

  Log Message:
  -----------
  target/i386/sev: Move qmp_sev_inject_launch_secret() to sev.c

Move qmp_sev_inject_launch_secret() from monitor.c to sev.c
and make sev_inject_launch_secret() static. We don't need the
stub anymore, remove it.

Previously with binaries built without SEV, management layer
was getting an empty response:

  { "execute": "sev-inject-launch-secret",
    "arguments": { "packet-header": "mypkt", "secret": "mypass", "gpa": 
4294959104 }
  }
  {
      "return": {
      }
  }

Now the response is explicit, mentioning the feature is disabled:

  { "execute": "sev-inject-launch-secret",
          "arguments": { "packet-header": "mypkt", "secret": "mypass", "gpa": 
4294959104 }
  }
  {
      "error": {
          "class": "GenericError",
          "desc": "this feature or command is not currently supported"
      }
  }

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-19-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 8371df29021241b43a7a9df6369fc70f649551a2
      
https://github.com/qemu/qemu/commit/8371df29021241b43a7a9df6369fc70f649551a2
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c
    M target/i386/sev.h

  Log Message:
  -----------
  target/i386/sev: Move qmp_query_sev_capabilities() to sev.c

Move qmp_query_sev_capabilities() from monitor.c to sev.c
and make sev_get_capabilities() static. We don't need the
stub anymore, remove it.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-20-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 0875a7038b8de0d79479a7d4662f0a9cdfa61940
      
https://github.com/qemu/qemu/commit/0875a7038b8de0d79479a7d4662f0a9cdfa61940
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c
    M target/i386/sev.h

  Log Message:
  -----------
  target/i386/sev: Move qmp_query_sev_launch_measure() to sev.c

Move qmp_query_sev_launch_measure() from monitor.c to sev.c
and make sev_get_launch_measurement() static. We don't need the
stub anymore, remove it.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-21-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: aa3950182f902a48f60d7040069b092eb5aaae21
      
https://github.com/qemu/qemu/commit/aa3950182f902a48f60d7040069b092eb5aaae21
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/monitor.c
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c
    M target/i386/sev.h

  Log Message:
  -----------
  target/i386/sev: Move qmp_query_sev() & hmp_info_sev() to sev.c

Move qmp_query_sev() & hmp_info_sev()() from monitor.c to sev.c
and make sev_get_info() static. We don't need the stub anymore,
remove it. Add a stub for hmp_info_sev().

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-22-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: cd35beb488c655a86da60d85949c026387187171
      
https://github.com/qemu/qemu/commit/cd35beb488c655a86da60d85949c026387187171
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M include/monitor/hmp-target.h
    M include/monitor/hmp.h
    M target/i386/sev-sysemu-stub.c
    M target/i386/sev.c

  Log Message:
  -----------
  monitor: Reduce hmp_info_sev() declaration

While being conditionally used for TARGET_I386 in hmp-commands-info.hx,
hmp_info_sev() is declared for all targets. Reduce its declaration
to target including "monitor/hmp-target.h". This is a minor cleanup.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-23-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 8c9e7f8c8a6de68be1e31a645f7d76855fc219ba
      
https://github.com/qemu/qemu/commit/8c9e7f8c8a6de68be1e31a645f7d76855fc219ba
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Cover SEV-related files with X86/KVM section

Complete the x86/KVM section with SEV-related files.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007161716.453984-24-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: f1279fc15bf4315fb8e103e6eabaee014ce1f914
      
https://github.com/qemu/qemu/commit/f1279fc15bf4315fb8e103e6eabaee014ce1f914
  Author: Thomas Huth <thuth@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M qapi/qom.json

  Log Message:
  -----------
  qapi: Make some ObjectTypes depend on the build settings

Some of the ObjectType entries already depend on CONFIG_* switches.
Some others also only make sense with certain configurations, but
are currently always listed in the ObjectType enum. Let's make them
depend on the correpsonding CONFIG_* switches, too, so that upper
layers (like libvirt) have a better way to determine which features
are available in QEMU.

Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20210928160232.432980-1-thuth@redhat.com>
[Do the same for MemoryBackendEpcProperties. - Paolo]
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 8e751e9c38e324737fd3d3aa0562f886313bba3c
      
https://github.com/qemu/qemu/commit/8e751e9c38e324737fd3d3aa0562f886313bba3c
  Author: Cole Robinson <crobinso@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M tests/tcg/x86_64/system/kernel.ld

  Log Message:
  -----------
  tests: tcg: Fix PVH test with binutils 2.36+

binutils started adding a .note.gnu.property ELF section which
makes the PVH test fail:

  TEST    hello on x86_64
qemu-system-x86_64: Error loading uncompressed kernel without PVH ELF Note

Discard .note.gnu* while keeping the PVH .note bits intact.

This also strips the build-id note, so drop the related comment.

Signed-off-by: Cole Robinson <crobinso@redhat.com>
Message-Id: 
<5ab2a54c262c61f64c22dbb49ade3e2db8a740bb.1633708346.git.crobinso@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: dec490db8b5c98d60814d07c23ddf70708a56d6e
      
https://github.com/qemu/qemu/commit/dec490db8b5c98d60814d07c23ddf70708a56d6e
  Author: Alexander Graf <agraf@csgraf.de>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M accel/hvf/hvf-accel-ops.c

  Log Message:
  -----------
  hvf: Determine slot count from struct layout

We can handle up to a static amount of memory slots, capped by the size of
an internal array.

Let's make sure that array size is the only source of truth for the number
of elements in that array.

Signed-off-by: Alexander Graf <agraf@csgraf.de>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20211008054616.43828-1-agraf@csgraf.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: a821186ec02ecf5ac4949ed42410bf2095bcea66
      
https://github.com/qemu/qemu/commit/a821186ec02ecf5ac4949ed42410bf2095bcea66
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Cover SGX documentation file with X86/KVM section

Cc: Yang Zhong <yang.zhong@intel.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007175612.496366-2-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 05fc8db720a2effad2fe523ab45c1ce5f8f32379
      
https://github.com/qemu/qemu/commit/05fc8db720a2effad2fe523ab45c1ce5f8f32379
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M hw/i386/sgx-stub.c
    M hw/i386/sgx.c
    M include/hw/i386/sgx-epc.h

  Log Message:
  -----------
  hw/i386/sgx: Have sgx_epc_get_section() return a boolean

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007175612.496366-3-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 021658566b71e138db46434849f0556bab3fc30b
      
https://github.com/qemu/qemu/commit/021658566b71e138db46434849f0556bab3fc30b
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M hw/i386/sgx-stub.c
    M hw/i386/sgx.c
    M include/hw/i386/sgx.h
    M target/i386/monitor.c

  Log Message:
  -----------
  hw/i386/sgx: Move qmp_query_sgx_capabilities() to hw/i386/sgx.c

Move qmp_query_sgx_capabilities() from target/i386/monitor.c to
hw/i386/sgx.c, removing the sgx_get_capabilities() indirection.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007175612.496366-4-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 6e81733e27a250af7507c3dd5ae945c473ab0d6b
      
https://github.com/qemu/qemu/commit/6e81733e27a250af7507c3dd5ae945c473ab0d6b
  Author: Philippe Mathieu-Daudé <philmd@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M hw/i386/sgx-stub.c
    M hw/i386/sgx.c
    R include/hw/i386/sgx.h
    M target/i386/monitor.c

  Log Message:
  -----------
  hw/i386/sgx: Move qmp_query_sgx() and hmp_info_sgx() to hw/i386/sgx.c

Move qmp_query_sgx() and hmp_info_sgx() from target/i386/monitor.c
to hw/i386/sgx.c, removing the sgx_get_info() indirection and the
"hw/i386/sgx.h" header.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20211007175612.496366-5-philmd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: b71803a74612cc73eee82681f942a083038642ba
      
https://github.com/qemu/qemu/commit/b71803a74612cc73eee82681f942a083038642ba
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M hw/misc/applesmc.c

  Log Message:
  -----------
  Revert "hw/misc: applesmc: use host osk as default on macs"

This reverts commit 93ddefbc3c909bb6c3b76086f1dfc8ad98dd3725.
The commit included code under the APSL 2.0, which is incompatible
with the GPL v2.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 69520261204b9a1769134ed24b92cc0fa8445951
      
https://github.com/qemu/qemu/commit/69520261204b9a1769134ed24b92cc0fa8445951
  Author: Markus Armbruster <armbru@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M softmmu/qdev-monitor.c

  Log Message:
  -----------
  monitor: Tidy up find_device_state()

Commit 6287d827d4 "monitor: allow device_del to accept QOM paths"
extended find_device_state() to accept QOM paths in addition to qdev
IDs.  This added a checked conversion to TYPE_DEVICE at the end, which
duplicates the check done for the qdev ID case earlier, except it sets
a *different* error: GenericError "ID is not a hotpluggable device"
when passed a QOM path, and DeviceNotFound "Device 'ID' not found"
when passed a qdev ID.  Fortunately, the latter won't happen as long
as we add only devices to /machine/peripheral/.

Earlier, commit b6cc36abb2 "qdev: device_del: Search for to be
unplugged device in 'peripheral' container" rewrote the lookup by qdev
ID to use QOM instead of qdev_find_recursive(), so it can handle
buss-less devices.  It does so by constructing an absolute QOM path.
Works, but object_resolve_path_component() is easier.  Switching to it
also gets rid of the unclean duplication described above.

While there, avoid converting to TYPE_DEVICE twice, first to check
whether it's possible, and then for real.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Damien Hedde <damien.hedde@greensocs.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20210916111707.84999-1-armbru@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: eb8257a261325988873e047872975b5eda03efb4
      
https://github.com/qemu/qemu/commit/eb8257a261325988873e047872975b5eda03efb4
  Author: Dov Murik <dovmurik@linux.ibm.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev.c

  Log Message:
  -----------
  target/i386/sev: Use local variable for kvm_sev_launch_start

The struct kvm_sev_launch_start has a constant and small size, and
therefore we can use a regular local variable for it instead of
allocating and freeing heap memory for it.

Signed-off-by: Dov Murik <dovmurik@linux.ibm.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20211011173026.2454294-2-dovmurik@linux.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 59e42d88b6851b238927cbe48505b1cef547ce3b
      
https://github.com/qemu/qemu/commit/59e42d88b6851b238927cbe48505b1cef547ce3b
  Author: Dov Murik <dovmurik@linux.ibm.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M target/i386/sev.c

  Log Message:
  -----------
  target/i386/sev: Use local variable for kvm_sev_launch_measure

The struct kvm_sev_launch_measure has a constant and small size, and
therefore we can use a regular local variable for it instead of
allocating and freeing heap memory for it.

Signed-off-by: Dov Murik <dovmurik@linux.ibm.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20211011173026.2454294-3-dovmurik@linux.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: eeecc2ede44b2a5f2551dfcebd561a3945d4c132
      
https://github.com/qemu/qemu/commit/eeecc2ede44b2a5f2551dfcebd561a3945d4c132
  Author: Paolo Bonzini <pbonzini@redhat.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M meson.build

  Log Message:
  -----------
  ebpf: really include it only in system emulators

eBPF libraries are being included in user emulators, which is useless and
also breaks --static compilation if a shared library for libbpf is
present in the system.

Reported-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


  Commit: 946de558354c99e1989621abe053f2ab87dc8de9
      
https://github.com/qemu/qemu/commit/946de558354c99e1989621abe053f2ab87dc8de9
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M MAINTAINERS
    M accel/hvf/hvf-accel-ops.c
    M accel/kvm/meson.build
    R accel/kvm/sev-stub.c
    M hw/i386/pc_sysfw.c
    M hw/i386/sgx-stub.c
    M hw/i386/sgx.c
    M hw/i386/x86.c
    M hw/misc/applesmc.c
    M include/hw/i386/sgx-epc.h
    R include/hw/i386/sgx.h
    M include/monitor/hmp-target.h
    M include/monitor/hmp.h
    R include/sysemu/sev.h
    M meson.build
    M qapi/misc-target.json
    M qapi/qom.json
    M softmmu/qdev-monitor.c
    M target/i386/cpu.c
    M target/i386/kvm/kvm.c
    M target/i386/kvm/meson.build
    A target/i386/kvm/sev-stub.c
    M target/i386/meson.build
    M target/i386/monitor.c
    R target/i386/sev-stub.c
    A target/i386/sev-sysemu-stub.c
    M target/i386/sev.c
    A target/i386/sev.h
    R target/i386/sev_i386.h
    M tests/Makefile.include
    M tests/qemu-iotests/testrunner.py
    M tests/tcg/x86_64/system/kernel.ld
    M util/compatfd.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging

* SEV and SGX cleanups (Philippe, Dov)
* bugfixes for "check-block"
* bugfix for static build
* ObjectOptions cleanups (Thomas)
* binutils fix for PVH (Cole)
* HVF cleanup (Alex)

# gpg: Signature made Wed 13 Oct 2021 01:47:56 AM PDT
# gpg:                using RSA key F13338574B662389866C7682BFFBD25F78C7AE83
# gpg:                issuer "pbonzini@redhat.com"
# gpg: Good signature from "Paolo Bonzini <bonzini@gnu.org>" [full]
# gpg:                 aka "Paolo Bonzini <pbonzini@redhat.com>" [full]

* remotes/bonzini/tags/for-upstream: (40 commits)
  ebpf: really include it only in system emulators
  target/i386/sev: Use local variable for kvm_sev_launch_measure
  target/i386/sev: Use local variable for kvm_sev_launch_start
  monitor: Tidy up find_device_state()
  Revert "hw/misc: applesmc: use host osk as default on macs"
  hw/i386/sgx: Move qmp_query_sgx() and hmp_info_sgx() to hw/i386/sgx.c
  hw/i386/sgx: Move qmp_query_sgx_capabilities() to hw/i386/sgx.c
  hw/i386/sgx: Have sgx_epc_get_section() return a boolean
  MAINTAINERS: Cover SGX documentation file with X86/KVM section
  hvf: Determine slot count from struct layout
  tests: tcg: Fix PVH test with binutils 2.36+
  qapi: Make some ObjectTypes depend on the build settings
  MAINTAINERS: Cover SEV-related files with X86/KVM section
  monitor: Reduce hmp_info_sev() declaration
  target/i386/sev: Move qmp_query_sev() & hmp_info_sev() to sev.c
  target/i386/sev: Move qmp_query_sev_launch_measure() to sev.c
  target/i386/sev: Move qmp_query_sev_capabilities() to sev.c
  target/i386/sev: Move qmp_sev_inject_launch_secret() to sev.c
  target/i386/sev: Move qmp_query_sev_attestation_report() to sev.c
  target/i386/sev: Remove stubs by using code elision
  ...

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Compare: https://github.com/qemu/qemu/compare/ee26ce674a93...946de558354c
[Prev in Thread]
Current Thread
[Next in Thread]
[Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer of the 'Memory..., Richard Henderson <=
- [Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer of the 'Memory..., Richard Henderson, 2021/10/13
Prev by Date: [Qemu-commits] [qemu/qemu] 025720: python/aqmp: add greeting property to QMPClient
Next by Date: [Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer of the 'Memory...
Previous by thread: [Qemu-commits] [qemu/qemu] 025720: python/aqmp: add greeting property to QMPClient
Next by thread: [Qemu-commits] [qemu/qemu] 601454: MAINTAINERS: Add myself as reviewer of the 'Memory...
Index(es):
- Date
- Thread