Add TYPE_RESETTABLE_INTERFACE to interfaces. Register callbacks for the reset phases. Add trace points for logging and debugging. No functional change, that will come in followup patches.
Signed-off-by: Gerd Hoffmann <[email protected]> Reviewed-by: Ani Sinha <[email protected]> Reviewed-by: Stefano Garzarella <[email protected]> --- include/system/igvm-cfg.h | 1 + include/system/igvm-internal.h | 1 + backends/igvm-cfg.c | 35 +++++++++++++++++++++++++++++++++- backends/trace-events | 5 +++++ 4 files changed, 41 insertions(+), 1 deletion(-) diff --git a/include/system/igvm-cfg.h b/include/system/igvm-cfg.h index 6c07f3084082..b0380133b9a0 100644 --- a/include/system/igvm-cfg.h +++ b/include/system/igvm-cfg.h @@ -14,6 +14,7 @@ #include "qemu/typedefs.h" #include "qom/object.h" +#include "hw/resettable.h" typedef struct IgvmCfgClass { ObjectClass parent_class; diff --git a/include/system/igvm-internal.h b/include/system/igvm-internal.h index 475a29bbf3d7..e0452080b127 100644 --- a/include/system/igvm-internal.h +++ b/include/system/igvm-internal.h @@ -21,6 +21,7 @@ struct IgvmCfg { * format. */ char *filename; + ResettableState reset_state; }; #endif diff --git a/backends/igvm-cfg.c b/backends/igvm-cfg.c index 001c4dc93346..e0df3eaa8efd 100644 --- a/backends/igvm-cfg.c +++ b/backends/igvm-cfg.c @@ -17,6 +17,8 @@ #include "system/reset.h" #include "qom/object_interfaces.h" +#include "trace.h" + static char *get_igvm(Object *obj, Error **errp) { IgvmCfg *igvm = IGVM_CFG(obj); @@ -30,24 +32,55 @@ static void set_igvm(Object *obj, const char *value, Error **errp) igvm->filename = g_strdup(value); } +static ResettableState *igvm_reset_state(Object *obj) +{ + IgvmCfg *igvm = IGVM_CFG(obj); + return &igvm->reset_state; +} + +static void igvm_reset_enter(Object *obj, ResetType type) +{ + trace_igvm_reset_enter(type); +} + +static void igvm_reset_hold(Object *obj, ResetType type) +{ + trace_igvm_reset_hold(type); +} + +static void igvm_reset_exit(Object *obj, ResetType type) +{ + trace_igvm_reset_exit(type); +} + OBJECT_DEFINE_TYPE_WITH_INTERFACES(IgvmCfg, igvm_cfg, IGVM_CFG, OBJECT, - { TYPE_USER_CREATABLE }, { NULL }) + { TYPE_USER_CREATABLE }, + { TYPE_RESETTABLE_INTERFACE }, + { NULL }) static void igvm_cfg_class_init(ObjectClass *oc, const void *data) { IgvmCfgClass *igvmc = IGVM_CFG_CLASS(oc); + ResettableClass *rc = RESETTABLE_CLASS(oc); object_class_property_add_str(oc, "file", get_igvm, set_igvm); object_class_property_set_description(oc, "file", "Set the IGVM filename to use"); igvmc->process = qigvm_process_file; + + rc->get_state = igvm_reset_state; + rc->phases.enter = igvm_reset_enter; + rc->phases.hold = igvm_reset_hold; + rc->phases.exit = igvm_reset_exit; } static void igvm_cfg_init(Object *obj) { + qemu_register_resettable(obj); } static void igvm_cfg_finalize(Object *obj) { + qemu_unregister_resettable(obj); } diff --git a/backends/trace-events b/backends/trace-events index 56132d3fd22b..45ac46dc2454 100644 --- a/backends/trace-events +++ b/backends/trace-events @@ -21,3 +21,8 @@ iommufd_backend_free_id(int iommufd, uint32_t id, int ret) " iommufd=%d id=%d (% iommufd_backend_set_dirty(int iommufd, uint32_t hwpt_id, bool start, int ret) " iommufd=%d hwpt=%u enable=%d (%d)" iommufd_backend_get_dirty_bitmap(int iommufd, uint32_t hwpt_id, uint64_t iova, uint64_t size, uint64_t page_size, int ret) " iommufd=%d hwpt=%u iova=0x%"PRIx64" size=0x%"PRIx64" page_size=0x%"PRIx64" (%d)" iommufd_backend_invalidate_cache(int iommufd, uint32_t id, uint32_t data_type, uint32_t entry_len, uint32_t entry_num, uint32_t done_num, uint64_t data_ptr, int ret) " iommufd=%d id=%u data_type=%u entry_len=%u entry_num=%u done_num=%u data_ptr=0x%"PRIx64" (%d)" + +# igvm-cfg.c +igvm_reset_enter(int type) "type=%u" +igvm_reset_hold(int type) "type=%u" +igvm_reset_exit(int type) "type=%u" -- 2.52.0
