From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Williamson Subject: RE: [Xen-ia64-devel] cset 8690 blk driver regression Date: Tue, 31 Jan 2006 16:08:46 -0700 Message-ID: <1138748926.13596.132.camel@localhost> References: <516F50407E01324991DD6D07B0531AD5980932@cacexc12.americas.cpqcorp.net> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <516F50407E01324991DD6D07B0531AD5980932@cacexc12.americas.cpqcorp.net> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Magenheimer, Dan (HP Labs Fort Collins)" Cc: xen-devel@lists.xensource.com, sos22@cam.ac.uk, xen-ia64-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On Tue, 2006-01-31 at 14:48 -0800, Magenheimer, Dan (HP Labs Fort Collins) wrote: > These were two separate bugs. Backing out > (or otherwise rewriting) that patch is still > necessary for domU to boot on ia64. > > A gnttab_shared_gpfn macro sounds good to me! > (assuming defined to the pre-patch lines for > ia64 and the post-patch lines for x86?) The easiest thing would be making __mfn_to_gpfn() be a pass through like below. It works for now, but I don't know what I'm breaking by doing this. Thanks, Alex diff -r 9ef5f0ec9df8 xen/include/asm-ia64/mm.h --- a/xen/include/asm-ia64/mm.h Tue Jan 31 17:39:35 2006 +++ b/xen/include/asm-ia64/mm.h Tue Jan 31 10:07:45 2006 @@ -422,8 +422,7 @@ * here. However if it's allocated by HV, we should access it directly */ -#define __mfn_to_gpfn(_d, mfn) \ - machine_to_phys_mapping[(mfn)] +#define __mfn_to_gpfn(_d, mfn) (mfn) #define __gpfn_to_mfn(_d, gpfn) \ __gpfn_to_mfn_foreign((_d), (gpfn)) -- Alex Williamson HP Linux & Open Source Lab