From: Andrii Anisov <andrii.anisov@gmail.com>
To: Julien Grall <julien.grall@arm.com>, xen-devel@lists.xen.org
Cc: Matthew Daley <mattd@bugfuzz.com>, sstabellini@kernel.org
Subject: Re: [PATCH for-4.12 v2 2/2] xen/arm: Stop relocating Xen
Date: Fri, 14 Dec 2018 18:13:25 +0200 [thread overview]
Message-ID: <a3f012fa-0fcd-dd1a-e29a-dedc5d9993a6@gmail.com> (raw)
In-Reply-To: <20181214114455.5841-3-julien.grall@arm.com>
On 14.12.18 13:44, Julien Grall wrote:
> At the moment, Xen is relocated towards the end of the memory. While
> this has the advantage to free space in low memory, the code is not
> compliant with the break-before-make because it requires to switch
> between two sets of page-table. This is not entirely trivial to fix as
> it would require us to go through an identity mapping and disabling MMU.
>
> Furthermore, it looks like that some platform (such as the Hikey960)
> may not be able to bring-up secondary CPUs if the entry is too high.
>
> I don't believe the low memory is an issue because Xen is quite tiny
> (< 2MB). So the best solution is to stop relocating Xen. This has the
> advantage to simplify the code and should speed-up the boot as relocation
> is not necessary anymore.
>
> Note that the break-before-make issue is not fixed by this patch.
>
> Signed-off-by: Julien Grall <julien.grall@arm.com>
> Reported-by: Matthew Daley <mattd@bugfuzz.com>
> Tested-by: Matthew Daley <mattd@bugfuzz.com>
Tested-by: Andrii Anisov <andrii_anisov@epam.com>
Reviewed-by: Andrii Anisov <andrii_anisov@epam.com>
--
Sincerely,
Andrii Anisov.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
prev parent reply other threads:[~2018-12-14 16:13 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-14 11:44 [PATCH for-4.12 v2 0/2] xen/arm: mm: Boot fixes Julien Grall
2018-12-14 11:44 ` [PATCH for-4.12 v2 1/2] xen/arm: mm: Use pte_xen_addr when creating xen entries Julien Grall
2018-12-14 21:44 ` Stefano Stabellini
2018-12-14 11:44 ` [PATCH for-4.12 v2 2/2] xen/arm: Stop relocating Xen Julien Grall
2018-12-14 15:52 ` Andrii Anisov
2018-12-14 16:09 ` Andrii Anisov
2018-12-14 16:26 ` Julien Grall
2018-12-14 17:24 ` Andrii Anisov
2018-12-14 17:48 ` Julien Grall
2018-12-17 11:11 ` Julien Grall
2018-12-17 16:02 ` Andrii Anisov
2018-12-17 16:56 ` Julien Grall
2018-12-17 17:57 ` Stefano Stabellini
2018-12-17 18:03 ` Julien Grall
2018-12-17 13:14 ` Andrii Anisov
2018-12-17 13:38 ` Julien Grall
2018-12-17 15:55 ` Andrii Anisov
2018-12-17 17:02 ` Julien Grall
2018-12-17 17:34 ` Andrii Anisov
2018-12-18 12:09 ` Andrii Anisov
2018-12-18 13:13 ` Julien Grall
2018-12-14 17:46 ` Andrii Anisov
2018-12-14 18:04 ` Julien Grall
2018-12-17 15:54 ` Andrii Anisov
2018-12-17 17:03 ` Julien Grall
2018-12-14 16:13 ` Andrii Anisov [this message]
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=a3f012fa-0fcd-dd1a-e29a-dedc5d9993a6@gmail.com \
--to=andrii.anisov@gmail.com \
--cc=julien.grall@arm.com \
--cc=mattd@bugfuzz.com \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xen.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.