From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: Re: irq_guest_eoi_timer interaction with MSI Date: Tue, 25 Nov 2008 08:40:13 +0000 Message-ID: <492BC7FD.76E4.0078.0@novell.com> References: <492BC215.76E4.0078.0@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Content-Disposition: inline List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org >>> Keir Fraser 25.11.08 09:30 >>> >On 25/11/08 08:15, "Jan Beulich" wrote: > >>> The fixmap stuff is a bit ugly and I would just have done a >>> map_domain_page_global() for 32-bit Xen (good enough as far as I'm >>> concerned). I'm not dead set against your approach if you like it very = much, >>> though. >>=20 >> But just as map_domain_page(), map_domain_page_global() can't be used >> out of IRQ context... > >I would have kept the page mapped from when it was registered until = domain >death. Same as we do for guest-registered vcpu_info. Oh, I indeed didn't consider that option. The question is - does it scale? 2Mb doesn't seem all that much for a global pool, but otoh it's perhaps unlikely that 32-bit hypervisors would have to manage too large an amount of domains/vCPU-s. So yes, I'll go that route. Jan