On Tue Jul 2, 2024 at 4:23 PM AEST, Thomas Huth wrote:
> On 02/07/2024 00.23, BALATON Zoltan wrote:
> > On Mon, 1 Jul 2024, Michael S. Tsirkin wrote:
> >> On Thu, Jun 27, 2024 at 10:37:43PM +0900, Akihiko Odaki wrote:
> >>> Based-on:
> >>> <3ad18bc590ef28e1526e8053568086b453e7ffde.1718211878.git.quic_mathb...@quicinc.com>
> >>> ("[PATCH] cpu: fix memleak of 'halt_cond' and 'thread'")
> >>>
> >>> I saw various sanitizer errors when running check-qtest-ppc64. While
> >>> I could just turn off sanitizers, I decided to tackle them this time.
> >>>
> >>> Unfortunately, GLib does not free test data in some cases so some
> >>> sanitizer errors remain. All sanitizer errors will be gone with this
> >>> patch series combined with the following change for GLib:
> >>> https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4120
> >>>
> >>> Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com>
> >>
> >>
> >> Reviewed-by: Michael S. Tsirkin <m...@redhat.com>
> >>
> >> who's merging all this?
> >
> > Maybe needs to be split. Mark had an alternative for macio which was picked
> > up by Philippe if I'm not mistaken. I've sent an alternative for vt82c686
> > which is still discussed but could belong to Philippe as well. PPC parts
> > could be taken by the PPC maintainers if there were any active at the
> > moment
> > and I don't know who maintains tests normally or other misc areas.
>
> I can take the qtest patches through my tree.
> cpu: Free cpu_ases
> migration: Free removed SaveStateEntry
> memory: Do not create circular reference with subregion
> hw/virtio: Free vqs after vhost_dev_cleanup()
These are all core code, maybe Philippe, Peter, or Richard,
Fabiano for migration perhaps?
> hw/ide: Convert macio ide_irq into GPIO line
> hw/ide: Remove internal DMA qemu_irq
> hw/isa/vt82c686: Define a GPIO line between vt82c686 and i8259
I guess I'll let you wrangle these and go through Philippe then?
> spapr: Free stdout path
> ppc/vof: Fix unaligned FDT property access
These ppc ones I can take, they seem okay. Sorry for my recent
inactivity.
Thanks,
Nick
> tests/qtest: Use qtest_add_data_func_full()
> tests/qtest: Free unused QMP response
> tests/qtest: Free old machine variable name
> tests/qtest: Delete previous boot file
> tests/qtest: Free paths
> tests/qtest: Free GThread
