Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-17 Thread David Gibson
On Fri, Jan 15, 2016 at 05:46:10PM +, Mark Cave-Ayland wrote: > On 12/01/16 02:44, David Gibson wrote: > > >>> In other words, isn't this just skipping the decrementer interrupts at > >>> the qemu level rather than the guest level? > >>> > >>> It seems that instead we should be reconstructing

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-15 Thread Mark Cave-Ayland
On 12/01/16 02:44, David Gibson wrote: >>> In other words, isn't this just skipping the decrementer interrupts at >>> the qemu level rather than the guest level? >>> >>> It seems that instead we should be reconstructing the decrementer on >>> the destination based on an offset from the timebase.

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-11 Thread David Gibson
On Mon, Jan 11, 2016 at 07:43:54AM +, Mark Cave-Ayland wrote: > On 11/01/16 04:55, David Gibson wrote: > > > On Mon, Jan 11, 2016 at 12:18:31PM +1100, Alexey Kardashevskiy wrote: > >> On 01/09/2016 01:21 AM, Mark Cave-Ayland wrote: > >>> On 08/01/16 02:47, Alexey Kardashevskiy wrote: > >>> >

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-10 Thread Mark Cave-Ayland
On 11/01/16 04:55, David Gibson wrote: > On Mon, Jan 11, 2016 at 12:18:31PM +1100, Alexey Kardashevskiy wrote: >> On 01/09/2016 01:21 AM, Mark Cave-Ayland wrote: >>> On 08/01/16 02:47, Alexey Kardashevskiy wrote: >>> On 01/07/2016 05:22 AM, Mark Cave-Ayland wrote: > During local testing

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-10 Thread David Gibson
On Mon, Jan 11, 2016 at 12:18:31PM +1100, Alexey Kardashevskiy wrote: > On 01/09/2016 01:21 AM, Mark Cave-Ayland wrote: > >On 08/01/16 02:47, Alexey Kardashevskiy wrote: > > > >>On 01/07/2016 05:22 AM, Mark Cave-Ayland wrote: > >>>During local testing with TCG, intermittent errors were found when

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-10 Thread Alexey Kardashevskiy
On 01/09/2016 01:21 AM, Mark Cave-Ayland wrote: On 08/01/16 02:47, Alexey Kardashevskiy wrote: On 01/07/2016 05:22 AM, Mark Cave-Ayland wrote: During local testing with TCG, intermittent errors were found when trying to migrate Darwin OS images. The underlying cause was that Darwin resets

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-08 Thread Mark Cave-Ayland
On 08/01/16 02:47, Alexey Kardashevskiy wrote: > On 01/07/2016 05:22 AM, Mark Cave-Ayland wrote: >> During local testing with TCG, intermittent errors were found when >> trying to >> migrate Darwin OS images. >> >> The underlying cause was that Darwin resets the decrementer value to >> fairly >>

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-07 Thread Alexey Kardashevskiy
On 01/07/2016 05:22 AM, Mark Cave-Ayland wrote: During local testing with TCG, intermittent errors were found when trying to migrate Darwin OS images. The underlying cause was that Darwin resets the decrementer value to fairly small values on each interrupt. cpu_ppc_set_tb_clk() sets the

[Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-06 Thread Mark Cave-Ayland
During local testing with TCG, intermittent errors were found when trying to migrate Darwin OS images. The underlying cause was that Darwin resets the decrementer value to fairly small values on each interrupt. cpu_ppc_set_tb_clk() sets the default value of the decrementer to 0x during