Xen 
 
Home About Xen.org Xen Xen Summit Wiki Mailing List Bug Tracker Xen Downloads
 
   
 

xen-devel

[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,

        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


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

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