[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration bloc
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution |
Date: |
Thu, 10 Aug 2017 11:22:41 +0200 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, Aug 10, 2017 at 10:11:13AM +0100, Peter Maydell wrote:
> On 1 August 2017 at 10:41, Peter Maydell <address@hidden> wrote:
> > On 1 August 2017 at 10:35, KONRAD Frederic <address@hidden> wrote:
> >>
> >>
> >> On 08/01/2017 11:30 AM, Edgar E. Iglesias wrote:
> >>> At this stage, perhaps we should just register the blocker when this dev
> >>> realizes.
> >>>
> >>> If a request_ptr comes in during migration, the VM will fail either way...
> >
> >> Yes but this will breaks migration for the spips device everytime
> >> and not only when mmio-execution is used?
> >
> > This line of thought is why I ended up suggesting just disabling
> > the exec-in-place feature -- that way we just don't introduce
> > what would be a new-in-2.10 feature, rather than breaking something
> > that used to work in 2.9.
>
> OK, so what's the plan here? We have several options:
> * just disable exec-from-spips for 2.10 (I sent a patch for that)
> * disable exec-from-spips for 2.10 but with a device x-property
> to allow the user to turn it on again if they really want it
> * this patch or variants on it which try to only disable
> migration if exec-from-spips is actually used by the guest
> (I don't like these because of the awkward corner cases if
> migration and the guest using exec-from-spips happen at the
> same time)
>
> So my current view remains "we should just disable this feature
> for 2.10 and we can implement it properly with handling of
> migration for 2.11", unless somebody cares enough to implement
> the x-property thing within the next day or so.
Hi Peter,
I think the x-property sounds good.
Fred, would you like to send a patch for that?
Otherwise, I can do it later today.
Cheers,
Edgar
- [Qemu-devel] [PATCH for-2.10 0/2] mmio-execution and migration, KONRAD Frederic, 2017/08/01
- [Qemu-devel] [PATCH for-2.10 1/2] mmio-execution: warn the potential developer about migration, KONRAD Frederic, 2017/08/01
- [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, KONRAD Frederic, 2017/08/01
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, Peter Maydell, 2017/08/01
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, KONRAD Frederic, 2017/08/01
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, Edgar E. Iglesias, 2017/08/01
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, KONRAD Frederic, 2017/08/01
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, Peter Maydell, 2017/08/01
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, Peter Maydell, 2017/08/10
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution,
Edgar E. Iglesias <=
- Re: [Qemu-devel] [PATCH for-2.10 2/2] xilinx-spips: add a migration blocker when using mmio_execution, KONRAD Frederic, 2017/08/10