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] [PATCH] Add two arch hooks to gnttab_release_mappings()

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH] Add two arch hooks to gnttab_release_mappings()
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Thu, 15 Mar 2007 16:36:21 +0900
Cc: xen-ppc-devel@xxxxxxxxxxxxxxxxxxx, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 15 Mar 2007 00:35:39 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/
Add two arch hooks to gnttab_release_mappings().
The c/s 8409:b3edbeea3e79 changed gnttab_release_mappings() such that
it doesn't put page/type reference count because it is done implicitly
when page table is destructed.
It is x86 specific issue and it isn't the case for ia64 and powerpc.
Introduce two arch hook to address it.

Notes on powerpc.
I checked powerpc code very roughly and wasn't able to find any
code which does page/type reference count operation on grant mapped pages.
So I concluded that page is leaked. But I'm not sure.
Please correct me if I'm worng.


Attachment: 14378_c268103362bb_gnttab_release_mappings_put_page_and_type.patch
Description: Text Data

Xen-devel mailing list
<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] [PATCH] Add two arch hooks to gnttab_release_mappings(), Isaku Yamahata <=