Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-05-04 Thread Jan Beulich
>>> On 04.05.17 at 11:47, wrote: > On 14/02/17 10:51, Jan Beulich wrote: > On 07.02.17 at 00:32, wrote: >>> Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has >>> added a assertion that intack.vector is the highest priority vector. But >>> according to the osstest, th

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-05-04 Thread Julien Grall
Hi, On 14/02/17 10:51, Jan Beulich wrote: On 07.02.17 at 00:32, wrote: Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has added a assertion that intack.vector is the highest priority vector. But according to the osstest, the assertion failed sometimes. More discussion

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-14 Thread Julien Grall
Hi Jan, On 02/14/2017 10:51 AM, Jan Beulich wrote: On 07.02.17 at 00:32, wrote: Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has added a assertion that intack.vector is the highest priority vector. But according to the osstest, the assertion failed sometimes. More di

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-14 Thread Julien Grall
(Resending the e-mail due to SMTP issue) On 02/14/2017 12:12 PM, Julien Grall wrote: Hi Jan, On 02/14/2017 10:51 AM, Jan Beulich wrote: On 07.02.17 at 00:32, wrote: Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has added a assertion that intack.vector is the highest

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-14 Thread Jan Beulich
>>> On 07.02.17 at 00:32, wrote: > Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has > added a assertion that intack.vector is the highest priority vector. But > according to the osstest, the assertion failed sometimes. More discussion can > be found in the thread > (http

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-12 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Friday, February 10, 2017 5:36 PM > > >>> On 07.02.17 at 00:32, wrote: > > Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has > > added a assertion that intack.vector is the highest priority vector. But > > according

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-10 Thread Jan Beulich
>>> On 07.02.17 at 00:32, wrote: > Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has > added a assertion that intack.vector is the highest priority vector. But > according to the osstest, the assertion failed sometimes. More discussion can > be found in the thread > (http

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-10 Thread Jan Beulich
>>> On 08.02.17 at 08:49, wrote: > Curious how this issue was initially caught? Would same practice make > sure next fail catching our eye? I guess Andrew just happened to look at the logs of a spurious failure of some test. Jan ___ Xen-devel mailing

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-08 Thread Chao Gao
On Wed, Feb 08, 2017 at 04:06:53PM +0800, Tian, Kevin wrote: >> From: Gao, Chao >> Sent: Wednesday, February 08, 2017 8:12 AM >> >> On Tue, Feb 07, 2017 at 09:18:56AM -0700, Jan Beulich wrote: >> On 07.02.17 at 08:28, wrote: >> >> On Tue, Feb 07, 2017 at 06:46:16AM -0700, Jan Beulich wrote:

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-08 Thread Tian, Kevin
> From: Gao, Chao > Sent: Wednesday, February 08, 2017 8:12 AM > > On Tue, Feb 07, 2017 at 09:18:56AM -0700, Jan Beulich wrote: > On 07.02.17 at 08:28, wrote: > >> On Tue, Feb 07, 2017 at 06:46:16AM -0700, Jan Beulich wrote: > >> On 07.02.17 at 07:32, wrote: > On Tue, Feb 07, 2017

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Tian, Kevin
> From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: Tuesday, February 07, 2017 6:05 PM > > On Tue, Feb 07, 2017 at 02:51:54AM -0700, Jan Beulich wrote: > > >>> On 07.02.17 at 00:32, wrote: > > > Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") > > > has > > > added a asse

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Chao Gao
On Tue, Feb 07, 2017 at 09:18:56AM -0700, Jan Beulich wrote: On 07.02.17 at 08:28, wrote: >> On Tue, Feb 07, 2017 at 06:46:16AM -0700, Jan Beulich wrote: >> On 07.02.17 at 07:32, wrote: On Tue, Feb 07, 2017 at 03:04:32AM -0700, Jan Beulich wrote: On 07.02.17 at 07:48, wrot

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Jan Beulich
>>> On 07.02.17 at 08:28, wrote: > On Tue, Feb 07, 2017 at 06:46:16AM -0700, Jan Beulich wrote: > On 07.02.17 at 07:32, wrote: >>> On Tue, Feb 07, 2017 at 03:04:32AM -0700, Jan Beulich wrote: >>> On 07.02.17 at 07:48, wrote: > Some comment from QEMU/KVM code, in /arch/x86/kvm/lapic.c

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Chao Gao
On Tue, Feb 07, 2017 at 06:46:16AM -0700, Jan Beulich wrote: On 07.02.17 at 07:32, wrote: >> On Tue, Feb 07, 2017 at 03:04:32AM -0700, Jan Beulich wrote: >> On 07.02.17 at 07:48, wrote: Some comment from QEMU/KVM code, in /arch/x86/kvm/lapic.c, int kvm_lapic_find_highest_

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Jan Beulich
>>> On 07.02.17 at 07:32, wrote: > On Tue, Feb 07, 2017 at 03:04:32AM -0700, Jan Beulich wrote: > On 07.02.17 at 07:48, wrote: >>> Some comment from QEMU/KVM code, in /arch/x86/kvm/lapic.c, >>> >>> int kvm_lapic_find_highest_irr(struct kvm_vcpu *vcpu) >>> { >>> /* This may race with sett

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Chao Gao
On Tue, Feb 07, 2017 at 03:04:32AM -0700, Jan Beulich wrote: On 07.02.17 at 07:48, wrote: >> Some comment from QEMU/KVM code, in /arch/x86/kvm/lapic.c, >> >> int kvm_lapic_find_highest_irr(struct kvm_vcpu *vcpu) >> { >> /* This may race with setting of irr in __apic_accept_irq() and >>

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Jan Beulich
>>> On 07.02.17 at 07:48, wrote: > Some comment from QEMU/KVM code, in /arch/x86/kvm/lapic.c, > > int kvm_lapic_find_highest_irr(struct kvm_vcpu *vcpu) > { > /* This may race with setting of irr in __apic_accept_irq() and >* value returned may be wrong, but kvm_vcpu_kick() in __apic

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Wei Liu
On Tue, Feb 07, 2017 at 02:51:54AM -0700, Jan Beulich wrote: > >>> On 07.02.17 at 00:32, wrote: > > Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has > > added a assertion that intack.vector is the highest priority vector. But > > according to the osstest, the assertion f

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-07 Thread Jan Beulich
>>> On 07.02.17 at 00:32, wrote: > Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") has > added a assertion that intack.vector is the highest priority vector. But > according to the osstest, the assertion failed sometimes. More discussion can > be found in the thread > (http

Re: [Xen-devel] [PATCH] X86/vmx: Dump PIR and vIRR before ASSERT()

2017-02-06 Thread Xuquan (Quan Xu)
On February 07, 2017 7:33 AM, Chao Gao wrote: >Commit c7bdecae42 ("x86/apicv: fix RTC periodic timer and apicv issue") >has added a assertion that intack.vector is the highest priority vector. But >according to the osstest, the assertion failed sometimes. More discussion >can be found in the thread