From: Julien Grall <julien.grall@linaro.org>
To: ian.campbell@citrix.com
Cc: xen-devel@lists.xenproject.org,
Julien Grall <julien.grall@linaro.org>,
ian.jackson@eu.citrix.com, patches@linaro.org
Subject: Re: [PATCH v6 08/10] xen/arm: Add relinquish_p2m_mapping to remove reference on every mapped page
Date: Wed, 18 Dec 2013 16:48:36 +0000 [thread overview]
Message-ID: <52B1D1E4.6060701@linaro.org> (raw)
In-Reply-To: <1387297678-17762-9-git-send-email-julien.grall@linaro.org>
On 12/17/2013 04:27 PM, Julien Grall wrote:
> @@ -343,12 +353,35 @@ static int create_p2m_entries(struct domain *d,
>
> memset(&pte, 0x00, sizeof(pte));
> write_pte(&third[third_table_offset(addr)], pte);
> + count++;
> }
> break;
> }
>
> if ( flush )
> flush_tlb_all_local();
> +
> + /* Preempt every 2MiB (mapped) or 32 MiB (unmapped) - arbitrary */
> + if ( op == RELINQUISH && count >= 0x2000 )
> + {
> + if ( hypercall_preempt_check() )
> + {
> + p2m->next_gfn_to_relinquish = maddr >> PAGE_SHIFT;
Oops, I made a mistake in this patch.
It should be addr instead of maddr.
As it's already in staging, I will send a fix right now.
--
Julien Grall
next prev parent reply other threads:[~2013-12-18 16:48 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-17 16:27 [PATCH v6 00/10] xen/arm: Handle correctly foreign mapping Julien Grall
2013-12-17 16:27 ` [PATCH v6 01/10] xen/arm: Introduce steps in domain_relinquish_resource Julien Grall
2013-12-17 16:27 ` [PATCH v6 02/10] xen/arm: move mfn_to_p2m_entry in arch/arm/p2m.c Julien Grall
2013-12-17 16:27 ` [PATCH v6 03/10] xen/arm: Implement p2m_type_t as an enum Julien Grall
2013-12-17 16:27 ` [PATCH v6 04/10] xen/arm: Store p2m type in each page of the guest Julien Grall
2013-12-18 14:18 ` Ian Campbell
2013-12-17 16:27 ` [PATCH v6 05/10] xen/arm: p2m: Extend p2m_lookup parameters to retrieve the p2m type Julien Grall
2013-12-17 16:27 ` [PATCH v6 06/10] xen/arm: Retrieve p2m type in get_page_from_gfn Julien Grall
2013-12-17 16:27 ` [PATCH v6 07/10] xen/arm: Handle remove foreign mapping Julien Grall
2013-12-17 16:27 ` [PATCH v6 08/10] xen/arm: Add relinquish_p2m_mapping to remove reference on every mapped page Julien Grall
2013-12-17 16:36 ` Ian Campbell
2013-12-17 16:58 ` Julien Grall
2013-12-17 17:02 ` Ian Campbell
2013-12-18 13:21 ` Ian Campbell
2013-12-18 13:30 ` Julien Grall
2013-12-18 14:06 ` Julien Grall
2013-12-18 14:12 ` Ian Campbell
2013-12-18 14:20 ` Ian Campbell
2013-12-18 16:48 ` Julien Grall [this message]
2013-12-18 16:50 ` Ian Campbell
2013-12-17 16:27 ` [PATCH v6 09/10] xen/arm: Set foreign page type to p2m_map_foreign Julien Grall
2013-12-17 16:27 ` [PATCH v6 10/10] xen/arm: grant-table: Support read-only mapping Julien Grall
2013-12-18 14:41 ` [PATCH v6 00/10] xen/arm: Handle correctly foreign mapping Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52B1D1E4.6060701@linaro.org \
--to=julien.grall@linaro.org \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=patches@linaro.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.