On Tue, Jul 02, 2019 at 02:47:12PM +0100, Peter Maydell wrote: > On Tue, 2 Jul 2019 at 10:17, Peter Maydell <peter.mayd...@linaro.org> wrote: > > > > On Fri, 28 Jun 2019 at 11:40, Gerd Hoffmann <kra...@redhat.com> wrote: > > > > > > The following changes since commit > > > 474f3938d79ab36b9231c9ad3b5a9314c2aeacde: > > > > > > Merge remote-tracking branch > > > 'remotes/amarkovic/tags/mips-queue-jun-21-2019' into staging (2019-06-21 > > > 15:40:50 +0100) > > > > > > are available in the Git repository at: > > > > > > git://git.kraxel.org/qemu tags/vga-20190628-pull-request > > > > > > for you to fetch changes up to 263807f4e85e94f2d4e988d61d2fc0319b1dd36b: > > > > > > ati-vga: switch to vgabios-ati.bin (2019-06-28 10:49:36 +0200) > > > > > > ---------------------------------------------------------------- > > > vga: ati fixes, add ati vgabios. > > > > > > ---------------------------------------------------------------- > > > > > > BALATON Zoltan (3): > > > i2c: Move bitbang_i2c.h to include/hw/i2c/ > > > ati-vga: Implement DDC and EDID info from monitor > > > ati-vga: Fixes to offset and pitch registers > > > > > > Gerd Hoffmann (3): > > > seabios: add config for ati vgabios > > > seabios: add ati vgabios binary > > > ati-vga: switch to vgabios-ati.bin > > > > > > > Applied, thanks. > > > > Please update the changelog at https://wiki.qemu.org/ChangeLog/4.1 > > for any user-visible changes. > > Hmm, the bitbang-i2c changes in this pull seem to have broken > patchew's asan build: > https://patchew.org/logs/20190628103957.9504-1-kra...@redhat.com/testing.asan/?type=message > > (probably because now we're using and testing the buggy > code in a way we weren't before)
Hmm. Yes, gpio_i2c_init() allocates memory (in inlined bitbang_i2c_init). There is no exit() function so the memory is never freed indeed. On the other hand i2c devices are not really hotpluggable, so maybe we only need to tag it that way? cheers, Gerd