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

Re: [Xen-devel] [PATCH 2 of 2] Added a hook for clearing ptes when a vma is freed by the kernel. Adapted from gntdev


  • To: "Isaku Yamahata" <yamahata@xxxxxxxxxxxxx>
  • From: "Geoffrey Lefebvre" <geoffrey@xxxxxxxxx>
  • Date: Wed, 23 May 2007 11:01:28 -0700
  • Cc: Jake.Wires@xxxxxxxxxxxxx, andrew.warfield@xxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Wed, 23 May 2007 10:59:45 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references:x-google-sender-auth; b=s8O58LPWtAaoK7koR+gkWLixxDhQb31hR8MkSjr2GB77C63yCY6gAHCL3nbClGMKoSJQ/A1T0BSZhnX1wRF7YaxyRh/gjjjVKLAkgyBonbrmyGN3XD6+ABVLmo4j3Ue89s7j2byTJ+YdabyFxMXfaRnkhjj1DHee1M3s98M71O4=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Hi,

Thanks for the comment. I will update the patch.

geoffrey

On 5/23/07, Isaku Yamahata <yamahata@xxxxxxxxxxxxx> wrote:
Hi.

On Tue, May 22, 2007 at 07:17:56AM -0700, Geoffrey Lefebvre wrote:
> +static pte_t blktap_clear_pte(struct vm_area_struct *vma,
> +                           unsigned long uvaddr,
> +                           pte_t *ptep, int is_fullmm)
   [ snip ]
> +     if (khandle->user != INVALID_GRANT_HANDLE) {
> +
> +             BUG_ON(xen_feature(XENFEAT_auto_translated_physmap));
> +
> +             gnttab_set_unmap_op(&unmap[count], virt_to_machine(ptep),
> +                                 GNTMAP_host_map
> +                                 | GNTMAP_application_map
> +                                 | GNTMAP_contains_pte,
> +                                 khandle->user);
> +             count++;
> +     }

'else pte_clear_full(...)' is necessary like gntdev for
auto translated physmap mode.

--
yamahata


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


 


Rackspace

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