Re: [Qemu-devel] [PATCH for-2.6] spapr_drc: fix aborts during DRC-count based hotplug

[David Gibson]

On Tue, Apr 26, 2016 at 09:20:53AM +0530, Bharata B Rao wrote:
> On Tue, Apr 26, 2016 at 11:18:25AM +1000, David Gibson wrote:
> > On Mon, Apr 25, 2016 at 05:24:25PM -0500, Michael Roth wrote:
> > > CPU/memory resources can be signalled en-masse via
> > > spapr_hotplug_req_add_by_count(), and when doing so, actually change
> > > the meaning of the 'drc' parameter passed to
> > > spapr_hotplug_req_event() to be a count rather than an index.
> > > 
> > > f40eb92 added a hook in spapr_hotplug_req_event() to record when a
> > > device had been 'signalled' to the guest, but that code assumes that
> > > drc is always an index. In cases where it's a count, such as memory
> > > hotplug, the DRC lookup will fail, leading to an assert.
> > > 
> > > Fix this by only explicitly setting the signalled state for cases where
> > > we are doing PCI hotplug.
> > > 
> > > For other resources types, since we cannot selectively track whether a
> > > resource has been signalled in cases where we signal attach as a count,
> > > set the 'signalled' state to true immediately upon making the
> > > resource available via drck->attach().
> > > 
> > > Reported-by: Bharata B Rao <address@hidden>
> > > Cc: Bharata B Rao <address@hidden>
> > > Cc: address@hidden
> > > Cc: address@hidden
> > > Signed-off-by: Michael Roth <address@hidden>
> > 
> > Applued to 2.6, I'll send a pull request today.
> 
> Checked that memory hotplug is working fine after this fix. So if it
> matters now,
> 
> Tested-by: Bharata B Rao <address@hidden>

Thanks, but I already sent the pull request.

-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson

signature.asc
Description: PGP signature