Philippe Mathieu-Daudé <phi...@redhat.com> 于2019年10月7日周一 下午11:19写道:
> Document mm_fw_cfg_init() return value must be released > with g_free(). mm_fw_cfg_uninit() was never used, remove it. > > This partly reverts commit 0729d833d6d6: > "tests/libqos: Add mm_fw_cfg_uninit()" > > Reviewed-by: Laszlo Ersek <ler...@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > Reviewed-by: Li Qiang <liq...@gmail.com> > --- > tests/libqos/fw_cfg.c | 5 ----- > tests/libqos/fw_cfg.h | 10 +++++++++- > 2 files changed, 9 insertions(+), 6 deletions(-) > > diff --git a/tests/libqos/fw_cfg.c b/tests/libqos/fw_cfg.c > index 37c3f2cf4d..ddeec821db 100644 > --- a/tests/libqos/fw_cfg.c > +++ b/tests/libqos/fw_cfg.c > @@ -126,11 +126,6 @@ QFWCFG *mm_fw_cfg_init(QTestState *qts, uint64_t base) > return fw_cfg; > } > > -void mm_fw_cfg_uninit(QFWCFG *fw_cfg) > -{ > - g_free(fw_cfg); > -} > - > static void io_fw_cfg_select(QFWCFG *fw_cfg, uint16_t key) > { > qtest_outw(fw_cfg->qts, fw_cfg->base, key); > diff --git a/tests/libqos/fw_cfg.h b/tests/libqos/fw_cfg.h > index 15604040bd..3247fd4000 100644 > --- a/tests/libqos/fw_cfg.h > +++ b/tests/libqos/fw_cfg.h > @@ -34,8 +34,16 @@ uint64_t qfw_cfg_get_u64(QFWCFG *fw_cfg, uint16_t key); > size_t qfw_cfg_get_file(QFWCFG *fw_cfg, const char *filename, > void *data, size_t buflen); > > +/** > + * mm_fw_cfg_init(): > + * @qts: The #QTestState that will be referred to by the QFWCFG object. > + * @base: The MMIO base address of the fw_cfg device in the guest. > + * > + * Returns a newly allocated QFWCFG object which must be released > + * with a call to g_free() when no longer required. > + */ > QFWCFG *mm_fw_cfg_init(QTestState *qts, uint64_t base); > -void mm_fw_cfg_uninit(QFWCFG *fw_cfg); > + > /** > * io_fw_cfg_init(): > * @qts: The #QTestState that will be referred to by the QFWCFG object. > -- > 2.21.0 > >