[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-ia64-devel] Problem with xen-unstable cset 7937 (grant tabletransfer fix)


  • To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
  • From: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>
  • Date: Mon, 21 Nov 2005 16:46:11 -0800
  • Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Tue, 22 Nov 2005 00:46:56 +0000
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AcXu5dDMVhwibZWIQTe58kXllEzD4gAFrQ0A
  • Thread-topic: [Xen-ia64-devel] Problem with xen-unstable cset 7937 (grant tabletransfer fix)

Keir, could you translate the asm statement in gnttab_transfer
into a cmpxchg (see include/asm-x86/system.h)?  Or is it doing
something that the macros can't do?

Also, I've attached a patch that shows the differences between
the x86 and ia64 versions.  If you think it would be possible
to merge most of these differences (and/or incorporate a
small number of ifdefs into common/grant_table.c), that
would be very helpful.

Thanks,
Dan 

> -----Original Message-----
> From: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx 
> [mailto:xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf 
> Of Magenheimer, Dan (HP Labs Fort Collins)
> Sent: Monday, November 21, 2005 2:52 PM
> To: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> Cc: Keir Fraser
> Subject: [Xen-ia64-devel] Problem with xen-unstable cset 7937 
> (grant tabletransfer fix)
> 
> It appears that a grant table fix Keir made (cset 7937 in
> xen-unstable) has broken Xen/ia64 domU boot.  I am getting:
> 
>       WARNING: g.e. still in use!
>       WARNING: leaking g.e. and page still in use!
> 
> a number of times,  leading to "unable to read superblock"
> and "Cannot open root device".
> 
> As noted in previous discussions, ia64 uses a separate file
> for grant_table.c, though there are more similarities than
> differences.  (This was a development convenience to avoid
> adding ifdefs.)  So changes to common/grant_table.c -- with
> corresponding usage changes in sparse/arch/xen/kernel/gnttab.c,
> which *is* a common file between Xen/x86 and Xen/ia64 -- are
> likely to break Xen/ia64.  Hopefully a fix will just be a matter
> of merging Keir's changes from xen/common/grant_table.c into
> xen/arch/ia64/xen/grant_table.c, but ideally the ia64 variant
> should be merged back into the common file, though it may
> take a few ifdefs.
> 
> Any volunteers?
> 
> Thanks,
> Dan
> 
> 
> _______________________________________________
> Xen-ia64-devel mailing list
> Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ia64-devel
> 

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.