From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: Re: [PATCH] fix ia64 breakage with PHYSDEVOP_pirq_eoi_mfn (was Re: [PATCH 2/2] linux/x86: use shared page indicating the need for an EOI notification) Date: Wed, 03 Dec 2008 09:31:41 +0000 Message-ID: <4936600D.76E4.0078.0@novell.com> References: <492FCF0D.76E4.0078.0@novell.com> <20081203020720.GA15798%yamahata@valinux.co.jp> <49364A50.76E4.0078.0@novell.com> <20081203084425.GG15798%yamahata@valinux.co.jp> <4936584C.76E4.0078.0@novell.com> <20081203092034.GH15798%yamahata@valinux.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20081203092034.GH15798%yamahata@valinux.co.jp> 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: Isaku Yamahata Cc: xen-devel@lists.xensource.com, Keir Fraser List-Id: xen-devel@lists.xenproject.org >>> Isaku Yamahata 03.12.08 10:20 >>> >Yes, you're correct. In fact I had the patch which you suggested, >but I was hesitated to change the x86 implementation so that >I had changed it to use virt_to_bus() on x86. > > > >evtchn, physdev: fix pirq_eoi_mfn for IA64 support. > >On ia64, global variables aren't in identity mapping area (i.e. kaddr) >so that there is no relationship between its virtual address and >its physical address. Thus virt_to_bus() can't be applied to them. >So introduce arbitrary_virt_to_bus() to wrap arch dependent function >and make use of it. So you really need arbitrary_virt_to_bus() along with arbitrary_virt_to_machine()? Is that another ia64 specific need (i.e. do the two have different meanings there)? Jan