From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH] x86/hvm: implement save/restore for posted interrupts Date: Fri, 18 Jul 2014 15:58:31 +0200 Message-ID: <20140718135831.GA10870@aepfle.de> References: <1404400159-27346-1-git-send-email-olaf@aepfle.de> <20140716142844.GA13820@aepfle.de> <20140717071101.GA3046@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Zhang, Yang Z" Cc: "Tian, Kevin" , Keir Fraser , "Nakajima, Jun" , "Dong, Eddie" , "xen-devel@lists.xen.org" , "Xu, Dongxiao" , Jan Beulich List-Id: xen-devel@lists.xenproject.org On Thu, Jul 17, Zhang, Yang Z wrote: > Olaf Hering wrote on 2014-07-17: > > I can probably do that. And I think initially I just changed > > lapic_save_regs, which was not enough. I think in the end the reason > > is that with posted irqs the code paths are different and just > > having IRR set is not enough. > It doesn't make sense. PIR is more like a tmp_IRR. All interrupts > pending in PIR will finally sync to IRR by hardware. In your case, you > already record it in IRR, it should be enough. The problem I am > seeing is that the eoi_exit_bitmap need to be setup correct after > migration, please try attached patch to see whether it solves your > problem: No, this patch does not work for me. Does it work for you? Olaf