[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH for-2.6] spapr_drc: fix aborts during DRC-count ba
From: |
Bharata B Rao |
Subject: |
Re: [Qemu-ppc] [PATCH for-2.6] spapr_drc: fix aborts during DRC-count based hotplug |
Date: |
Tue, 26 Apr 2016 09:20:53 +0530 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
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>
Regards,
Bharata.