From: Andrii Anisov <andrii.anisov@gmail.com>
To: Julien Grall <julien.grall@arm.com>, xen-devel@lists.xenproject.org
Cc: Oleksandr_Tyshchenko@epam.com, sstabellini@kernel.org,
Andrii_Anisov@epam.com
Subject: Re: [PATCH 13/20] xen/arm32: mm: Avoid to zero and clean cache for CPU0 domheap
Date: Mon, 6 May 2019 11:28:07 +0300 [thread overview]
Message-ID: <8a9dd31b-0063-41fe-7030-79d2d1626ba7@gmail.com> (raw)
In-Reply-To: <291f238e-16e8-a5d7-c0a5-5532fde69585@arm.com>
On 03.05.19 20:06, Julien Grall wrote:
> In C, uninitialized global variable will be zero by default. It is a bit of waste to allocate space in the binary for them. So the compiler will commonly put them in a section BSS that are going to be zeroed when at launch.
>
> On Arm32, this is always done in CPU0 at early boot. For Arm64, UEFI will do it for us, so we don't want to do it when using UEFI as we may override global
>
> The reason I chose to say "will always be zeroed beforehand" than specifically mention "BSS" is I wasn't entirely convinced the compiler will always put in BSS.
OK.
--
Sincerely,
Andrii Anisov.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
WARNING: multiple messages have this Message-ID (diff)
From: Andrii Anisov <andrii.anisov@gmail.com>
To: Julien Grall <julien.grall@arm.com>, xen-devel@lists.xenproject.org
Cc: Oleksandr_Tyshchenko@epam.com, sstabellini@kernel.org,
Andrii_Anisov@epam.com
Subject: Re: [Xen-devel] [PATCH 13/20] xen/arm32: mm: Avoid to zero and clean cache for CPU0 domheap
Date: Mon, 6 May 2019 11:28:07 +0300 [thread overview]
Message-ID: <8a9dd31b-0063-41fe-7030-79d2d1626ba7@gmail.com> (raw)
Message-ID: <20190506082807.k_y68-ytpYCbDMICnv_KPesGI2MpTiwVIVLXCeuQUio@z> (raw)
In-Reply-To: <291f238e-16e8-a5d7-c0a5-5532fde69585@arm.com>
On 03.05.19 20:06, Julien Grall wrote:
> In C, uninitialized global variable will be zero by default. It is a bit of waste to allocate space in the binary for them. So the compiler will commonly put them in a section BSS that are going to be zeroed when at launch.
>
> On Arm32, this is always done in CPU0 at early boot. For Arm64, UEFI will do it for us, so we don't want to do it when using UEFI as we may override global
>
> The reason I chose to say "will always be zeroed beforehand" than specifically mention "BSS" is I wasn't entirely convinced the compiler will always put in BSS.
OK.
--
Sincerely,
Andrii Anisov.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next prev parent reply other threads:[~2019-05-06 8:28 UTC|newest]
Thread overview: 118+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-22 16:49 [PATCH 00/20] xen/arm: Clean-up & fixes in boot/mm code Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-04-22 16:49 ` [PATCH 01/20] xen/const: Introduce _BITUL and _BITULL Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-04-25 12:15 ` Jan Beulich
2019-04-25 12:15 ` [Xen-devel] " Jan Beulich
2019-04-29 16:47 ` Julien Grall
2019-04-29 16:47 ` [Xen-devel] " Julien Grall
2019-04-30 6:57 ` Jan Beulich
2019-04-30 6:57 ` [Xen-devel] " Jan Beulich
2019-04-22 16:49 ` [PATCH 02/20] xen/arm: Rename SCTLR_* defines and remove unused one Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:56 ` Andrii Anisov
2019-05-03 15:56 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 03/20] xen/arm: processor: Use _BITUL instead of _AC(1, U) in SCTLR_ defines Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:56 ` Andrii Anisov
2019-05-03 15:56 ` [Xen-devel] " Andrii Anisov
2019-05-03 16:09 ` Julien Grall
2019-05-03 16:09 ` [Xen-devel] " Julien Grall
2019-05-03 16:12 ` Andrii Anisov
2019-05-03 16:12 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 04/20] xen/arm: Rework HSCTLR_BASE Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:56 ` Andrii Anisov
2019-05-03 15:56 ` [Xen-devel] " Andrii Anisov
2019-05-03 16:10 ` Julien Grall
2019-05-03 16:10 ` [Xen-devel] " Julien Grall
2019-05-03 16:17 ` Andrii Anisov
2019-05-03 16:17 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 05/20] xen/arm: Rework secondary_start prototype Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:56 ` Andrii Anisov
2019-05-03 15:56 ` [Xen-devel] " Andrii Anisov
2019-05-03 16:15 ` Julien Grall
2019-05-03 16:15 ` [Xen-devel] " Julien Grall
2019-04-22 16:49 ` [PATCH 06/20] xen/arm: Remove parameter cpuid from start_xen Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:56 ` Andrii Anisov
2019-05-03 15:56 ` [Xen-devel] " Andrii Anisov
2019-05-03 16:17 ` Julien Grall
2019-05-03 16:17 ` [Xen-devel] " Julien Grall
2019-05-03 16:19 ` Andrii Anisov
2019-05-03 16:19 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 07/20] xen/arm64: head: Remove unnecessary comment Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 08/20] xen/arm64: head: Move earlyprintk messages in .rodata.str Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 09/20] xen/arm64: head: Correctly report the HW CPU ID Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 10/20] xen/arm32: " Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 11/20] xen/arm32: head: Don't set MAIR0 and MAIR1 Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 12/20] xen/arm32: head: Always zero r3 before update a page-table entry Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-05-03 16:21 ` Julien Grall
2019-05-03 16:21 ` [Xen-devel] " Julien Grall
2019-04-22 16:49 ` [PATCH 13/20] xen/arm32: mm: Avoid to zero and clean cache for CPU0 domheap Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-05-03 17:06 ` Julien Grall
2019-05-03 17:06 ` [Xen-devel] " Julien Grall
2019-05-06 8:28 ` Andrii Anisov [this message]
2019-05-06 8:28 ` Andrii Anisov
2019-04-22 16:49 ` [PATCH 14/20] xen/arm32: mm: Avoid cleaning the cache for secondary CPUs page-tables Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:57 ` Andrii Anisov
2019-05-03 15:57 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 15/20] xen/arm: mm: Use the shorter version __aligned(PAGE_SIZE) to align page-tables Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:58 ` Andrii Anisov
2019-05-03 15:58 ` [Xen-devel] " Andrii Anisov
2019-05-03 17:09 ` Julien Grall
2019-05-03 17:09 ` [Xen-devel] " Julien Grall
2019-05-06 7:19 ` Andrii Anisov
2019-05-06 7:19 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 16/20] xen/arm: mm: Protect Xen page-table update with a spinlock Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:59 ` Andrii Anisov
2019-05-03 15:59 ` [Xen-devel] " Andrii Anisov
2019-05-03 17:19 ` Julien Grall
2019-05-03 17:19 ` [Xen-devel] " Julien Grall
2019-05-06 8:20 ` Andrii Anisov
2019-05-06 8:20 ` [Xen-devel] " Andrii Anisov
2019-05-06 16:54 ` Julien Grall
2019-05-06 16:54 ` [Xen-devel] " Julien Grall
2019-05-06 8:20 ` Andrii Anisov
2019-05-06 8:20 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 17/20] xen/arm: mm: Initialize page-tables earlier Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:59 ` Andrii Anisov
2019-05-03 15:59 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 18/20] xen/arm: mm: Check start is always before end in {destroy, modify}_xen_mappings Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:59 ` Andrii Anisov
2019-05-03 15:59 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 19/20] xen/arm: Pair call to set_fixmap with call to clear_fixmap in copy_from_paddr Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-05-03 15:59 ` Andrii Anisov
2019-05-03 15:59 ` [Xen-devel] " Andrii Anisov
2019-04-22 16:49 ` [PATCH 20/20] xen/arm: Allow cleaning the directory even when CONFIG_EARLY_PRINTK is set Julien Grall
2019-04-22 16:49 ` [Xen-devel] " Julien Grall
2019-04-24 15:14 ` Julien Grall
2019-04-24 15:14 ` [Xen-devel] " Julien Grall
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=8a9dd31b-0063-41fe-7030-79d2d1626ba7@gmail.com \
--to=andrii.anisov@gmail.com \
--cc=Andrii_Anisov@epam.com \
--cc=Oleksandr_Tyshchenko@epam.com \
--cc=julien.grall@arm.com \
--cc=sstabellini@kernel.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.