On 7/2/22 09:37, Igor Mammedov wrote:
On Sat,  5 Feb 2022 13:45:26 +0100
Philippe Mathieu-Daudé <f4...@amsat.org> wrote:

Previously SGX-EPC objects were exposed in the QOM tree at a path

   /machine/unattached/device[nn]

where the 'nn' varies depending on what devices were already created.

With this change the SGX-EPC objects are now at

   /machine/sgx-epc[nn]

where the 'nn' of the first SGX-EPC object is always zero.

yet again, why it's necessary?

I'll defer that question to Yang & Daniel.


Reported-by: Yang Zhong <yang.zh...@intel.com>
Suggested-by: Paolo Bonzini <pbonz...@redhat.com>
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
  hw/i386/sgx.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/hw/i386/sgx.c b/hw/i386/sgx.c
index a2b318dd938..3ab2217ca43 100644
--- a/hw/i386/sgx.c
+++ b/hw/i386/sgx.c
@@ -304,6 +304,8 @@ void pc_machine_init_sgx_epc(PCMachineState *pcms)
      for (list = x86ms->sgx_epc_list; list; list = list->next) {
          obj = object_new("sgx-epc");
+ object_property_add_child(OBJECT(pcms), "sgx-epc[*]", OBJECT(obj));
+
          /* set the memdev link with memory backend */
          object_property_parse(obj, SGX_EPC_MEMDEV_PROP, list->value->memdev,
                                &error_fatal);



Reply via email to