This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] RE: [Xen-ia64-devel] cset 8690 blk driver regression

To: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>
Subject: [Xen-devel] RE: [Xen-ia64-devel] cset 8690 blk driver regression
From: Alex Williamson <alex.williamson@xxxxxx>
Date: Tue, 31 Jan 2006 16:08:46 -0700
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, sos22@xxxxxxxxx, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 31 Jan 2006 23:18:37 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <516F50407E01324991DD6D07B0531AD5980932@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: LOSL
References: <516F50407E01324991DD6D07B0531AD5980932@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
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,


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

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>