qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] qdev: Use GList for global properties


From: Eduardo Habkost
Subject: Re: [Qemu-devel] [PATCH] qdev: Use GList for global properties
Date: Thu, 12 May 2016 09:26:30 -0300
User-agent: Mutt/1.5.24 (2015-08-30)

On Fri, Jan 29, 2016 at 12:15:12PM -0200, Eduardo Habkost wrote:
> On Thu, Jan 28, 2016 at 07:01:12PM +0200, Michael S. Tsirkin wrote:
> > On Thu, Jan 28, 2016 at 01:02:26PM -0200, Eduardo Habkost wrote:
> > > If the same GlobalProperty struct is registered twice, the list
> > > entry gets corrupted, making tqe_next points to itself, and
> > > qdev_prop_set_globals() gets stuck in a loop. The bug can be
> > > easily reproduced by running:
> > > 
> > >   $ qemu-system-x86_64 -rtc-td-hack -rtc-td-hack
> > > 
> > > Change global_props to use GList instead of queue.h, making the
> > > code simpler and able to deal with properties being registered
> > > twice.
> > > 
> > > Signed-off-by: Eduardo Habkost <address@hidden>
> > 
> > Reviewed-by: Michael S. Tsirkin <address@hidden>
> 
> Thanks! It seems we don't have a maintainer for hw/core/qdev*.
> Who should merge this?

I've just noticed that this didn't get included. Andreas, is it
OK if I merge this through the machine core tree?

-- 
Eduardo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]