All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@linaro.org>
To: Thomas Leonard <talex5@gmail.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Julien Grall <julien.grall@citrix.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [PATCH v3 0/3] map grant refs at pfn = mfn
Date: Fri, 01 Aug 2014 17:21:21 +0100	[thread overview]
Message-ID: <53DBBE81.3070902@linaro.org> (raw)
In-Reply-To: <CAG4opy_d43YhCQZZctbUxrwN1e=9URRy5SjAd5QE5_saHMoQjQ@mail.gmail.com>



On 01/08/14 17:16, Thomas Leonard wrote:
> On 1 August 2014 16:13, Stefano Stabellini
> <stefano.stabellini@eu.citrix.com> wrote:
>> On Fri, 1 Aug 2014, Thomas Leonard wrote:
>>> On 24/07/14 14:30, Stefano Stabellini wrote:
>>>> Hi all,
>>>> this patch series introduces a second p2m mapping of grant reference on
>>>> ARM at guest physical address == machine address of the grant ref.  It
>>>> is safe because dom0 is already mapped 1:1. We export
>>>> XENFEAT_grant_map_identity to signal the guest that this second p2m
>>>> mapping is
>>>> available.
>>>>
>>>> One reason for wanting the second p2m mapping is to avoid tracking mfn
>>>> to pfn mappings in the guest kernel. Since the same mfn can be granted
>>>> multiple times to the backend, finding the right pfn corresponding to a
>>>> given mfn can be difficult and expensive. Providing a second mapping at
>>>> a known address allow the kernel to access the page without knowing the
>>>> pfn.
>>>
>>> Is there a version of these patches for Xen 4.4 that I can test? The
>>> restriction on duplicate pages is causing trouble for networking on Mirage too
>>> (http://roscidus.com/blog/blog/2014/07/28/my-first-unikernel/#tcp-retransmissions).
>>
>> The backport is non-trivial because
>> 552710b388630dfa461932940a998e120c42277d is missing from 4.4,
>> nonetheless it wasn't too hard to port:
>>
>>    git://xenbits.xen.org/people/sstabellini/xen-unstable.git grant_map_identity_4.4
>
> Thanks. I merged it with the stable-4.4 branch (as that has some
> useful fixes too), but it crashed for me when I started my Mirage
> guest:

You have to drop the BUG_ON line 729.

Regards,

-- 
Julien Grall

  reply	other threads:[~2014-08-01 16:21 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-24 13:30 [PATCH v3 0/3] map grant refs at pfn = mfn Stefano Stabellini
2014-07-24 13:31 ` [PATCH v3 1/3] xen/x86: introduce is_domain_direct_mapped(d) as (0) on x86 Stefano Stabellini
2014-07-24 13:41   ` Julien Grall
2014-07-24 16:51     ` Stefano Stabellini
2014-07-24 13:31 ` [PATCH v3 2/3] xen: introduce arch_grant_(un)map_page_identity Stefano Stabellini
2014-07-24 13:42   ` Jan Beulich
2014-07-24 16:55     ` Stefano Stabellini
2014-07-24 13:44   ` Julien Grall
2014-07-24 16:57     ` Stefano Stabellini
2014-07-24 16:59       ` Julien Grall
2014-07-24 13:31 ` [PATCH v3 3/3] xen/arm: introduce XENFEAT_grant_map_identity Stefano Stabellini
2014-07-24 13:41   ` Jan Beulich
2014-07-24 14:10     ` Stefano Stabellini
2014-07-24 14:43       ` Jan Beulich
2014-07-24 14:47         ` Julien Grall
2014-07-24 15:02           ` Stefano Stabellini
2014-07-24 15:17             ` Julien Grall
2014-07-24 17:15               ` Stefano Stabellini
2014-07-24 13:50   ` Julien Grall
2014-07-24 14:14     ` Stefano Stabellini
2014-07-24 14:51       ` Julien Grall
2014-08-01 12:35 ` [PATCH v3 0/3] map grant refs at pfn = mfn Thomas Leonard
2014-08-01 12:37 ` Thomas Leonard
2014-08-01 15:13   ` Stefano Stabellini
2014-08-01 16:16     ` Thomas Leonard
2014-08-01 16:21       ` Julien Grall [this message]
2014-08-01 16:25         ` Stefano Stabellini
2014-08-01 16:56           ` Thomas Leonard
2014-08-01 17:01             ` Stefano Stabellini
2014-08-01 17:04               ` Thomas Leonard
2014-08-01 17:16                 ` Stefano Stabellini
2014-08-01 18:12                   ` Thomas Leonard
2014-08-06 11:22                     ` Thomas Leonard
2014-08-06 13:35                       ` Stefano Stabellini
2014-08-06 13:39                         ` Thomas Leonard
2014-08-06 13:46                           ` Stefano Stabellini
2014-08-06 14:04                             ` Thomas Leonard
2014-08-06 14:14                               ` Stefano Stabellini
2014-08-06 14:19                                 ` Thomas Leonard
2014-08-06 14:27                                   ` Stefano Stabellini
2014-08-06 14:53                                     ` Thomas Leonard
2014-08-06 15:59                                       ` Wei Liu
2014-08-06 17:50                                         ` Thomas Leonard
2014-08-06 20:46                                           ` Wei Liu
2014-08-07  7:59                                             ` Thomas Leonard
2014-08-07 10:40                                               ` Wei Liu
2014-08-07 11:19                                                 ` Thomas Leonard
2014-08-06 14:24                                 ` Thomas Leonard
2014-08-06 14:11                           ` Wei Liu

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=53DBBE81.3070902@linaro.org \
    --to=julien.grall@linaro.org \
    --cc=Ian.Campbell@citrix.com \
    --cc=julien.grall@citrix.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=talex5@gmail.com \
    --cc=xen-devel@lists.xensource.com \
    /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.