Re: [Xen-devel] [PATCH v2] x86/hvm: do not register hpet mmio during s3 cycle

2017-11-13 Thread Julien Grall
Hi Jan, On 11/09/2017 02:45 PM, Jan Beulich wrote: On 09.11.17 at 15:42, wrote: Hi, On 09/11/17 08:55, Jan Beulich wrote: On 08.11.17 at 20:46, wrote: Do it once at domain creation (hpet_init). Sleep -> Resume cycles will end up crashing

Re: [Xen-devel] [PATCH v2] x86/hvm: do not register hpet mmio during s3 cycle

2017-11-09 Thread Eric Chanudet
On 09/11/17 at 01:55am, Jan Beulich wrote: >Perhaps my prior reply was ambiguous: By "inlining" I meant literally >inlining it My apologies for the clumsiness, I can resend a v3 right away. Thanks, -- Eric Chanudet ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v2] x86/hvm: do not register hpet mmio during s3 cycle

2017-11-09 Thread Jan Beulich
>>> On 09.11.17 at 15:42, wrote: > Hi, > > On 09/11/17 08:55, Jan Beulich wrote: > On 08.11.17 at 20:46, wrote: >>> Do it once at domain creation (hpet_init). >>> >>> Sleep -> Resume cycles will end up crashing an HVM guest with hpet as >>>

Re: [Xen-devel] [PATCH v2] x86/hvm: do not register hpet mmio during s3 cycle

2017-11-09 Thread Julien Grall
Hi, On 09/11/17 08:55, Jan Beulich wrote: On 08.11.17 at 20:46, wrote: Do it once at domain creation (hpet_init). Sleep -> Resume cycles will end up crashing an HVM guest with hpet as the sequence during resume takes the path: -> hvm_s3_suspend -> hpet_reset

Re: [Xen-devel] [PATCH v2] x86/hvm: do not register hpet mmio during s3 cycle

2017-11-09 Thread Jan Beulich
>>> On 08.11.17 at 20:46, wrote: > Do it once at domain creation (hpet_init). > > Sleep -> Resume cycles will end up crashing an HVM guest with hpet as > the sequence during resume takes the path: > -> hvm_s3_suspend > -> hpet_reset > -> hpet_deinit > ->

[Xen-devel] [PATCH v2] x86/hvm: do not register hpet mmio during s3 cycle

2017-11-08 Thread Eric Chanudet
Do it once at domain creation (hpet_init). Sleep -> Resume cycles will end up crashing an HVM guest with hpet as the sequence during resume takes the path: -> hvm_s3_suspend -> hpet_reset -> hpet_deinit -> hpet_init -> register_mmio_handler -> hvm_next_io_handler