* [U-Boot] Question on Enabling hypervisor mode in u-boot
@ 2016-08-01 10:07 Keerthy
2016-08-01 17:41 ` Andreas Färber
2016-08-01 21:46 ` Alexander Graf
0 siblings, 2 replies; 8+ messages in thread
From: Keerthy @ 2016-08-01 10:07 UTC (permalink / raw)
To: u-boot
Hi Alexander,
I am trying to enable hypervisor in u-boot for DRA7(A15 based) family of
SoCs which does not have LPAE support yet.
Is it mandatory for LPAE to be enabled before enabling hypervisor for A15?
I was looking at commit:
commit d990f5c834f1b42293fb53e4fd7f3aa988184196
Author: Alexander Graf <agraf@suse.de>
Date: Wed Mar 16 15:41:21 2016 +0100
arm: Add support for HYP mode and LPAE page tables
and the is_hyp always returns 0 when LPAE config option is not enabled.
Just wanted to confirm if that is mandatory.
Best Regards,
Keerthy
^ permalink raw reply [flat|nested] 8+ messages in thread* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-01 10:07 [U-Boot] Question on Enabling hypervisor mode in u-boot Keerthy @ 2016-08-01 17:41 ` Andreas Färber 2016-08-02 2:19 ` Keerthy 2016-08-01 21:46 ` Alexander Graf 1 sibling, 1 reply; 8+ messages in thread From: Andreas Färber @ 2016-08-01 17:41 UTC (permalink / raw) To: u-boot Hi Keerthy, Am 01.08.2016 um 12:07 schrieb Keerthy: > I am trying to enable hypervisor in u-boot for DRA7(A15 based) family of > SoCs which does not have LPAE support yet. > > Is it mandatory for LPAE to be enabled before enabling hypervisor for A15? On the Linux kernel side you can't enable KVM without LPAE. I assume the same will hold true in U-Boot then. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany GF: Felix Imend?rffer, Jane Smithard, Graham Norton HRB 21284 (AG N?rnberg) ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-01 17:41 ` Andreas Färber @ 2016-08-02 2:19 ` Keerthy 0 siblings, 0 replies; 8+ messages in thread From: Keerthy @ 2016-08-02 2:19 UTC (permalink / raw) To: u-boot On Monday 01 August 2016 11:11 PM, Andreas F?rber wrote: > Hi Keerthy, > > Am 01.08.2016 um 12:07 schrieb Keerthy: >> I am trying to enable hypervisor in u-boot for DRA7(A15 based) family of >> SoCs which does not have LPAE support yet. >> >> Is it mandatory for LPAE to be enabled before enabling hypervisor for A15? > > On the Linux kernel side you can't enable KVM without LPAE. > I assume the same will hold true in U-Boot then. Hi Andreas, Okay. Thanks for the confirmations. I will try with LPAE config defined in u-boot also. Best Regards, Keerthy > > Regards, > Andreas > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-01 10:07 [U-Boot] Question on Enabling hypervisor mode in u-boot Keerthy 2016-08-01 17:41 ` Andreas Färber @ 2016-08-01 21:46 ` Alexander Graf 2016-08-02 1:54 ` Keerthy 1 sibling, 1 reply; 8+ messages in thread From: Alexander Graf @ 2016-08-01 21:46 UTC (permalink / raw) To: u-boot > On 01 Aug 2016, at 11:07, Keerthy <a0393675@ti.com> wrote: > > Hi Alexander, > > I am trying to enable hypervisor in u-boot for DRA7(A15 based) family of SoCs which does not have LPAE support yet. > > Is it mandatory for LPAE to be enabled before enabling hypervisor for A15? HYP mode shares the same page table layout as the LPAE one. I?m actually surprised you managed to configure an A15 without LPAE. Are you sure it doesn?t support it? Alex ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-01 21:46 ` Alexander Graf @ 2016-08-02 1:54 ` Keerthy 2016-08-02 11:59 ` Keerthy 0 siblings, 1 reply; 8+ messages in thread From: Keerthy @ 2016-08-02 1:54 UTC (permalink / raw) To: u-boot On Tuesday 02 August 2016 03:16 AM, Alexander Graf wrote: > >> On 01 Aug 2016, at 11:07, Keerthy <a0393675@ti.com> wrote: >> >> Hi Alexander, >> >> I am trying to enable hypervisor in u-boot for DRA7(A15 based) family of SoCs which does not have LPAE support yet. >> >> Is it mandatory for LPAE to be enabled before enabling hypervisor for A15? > > HYP mode shares the same page table layout as the LPAE one. I?m actually surprised you managed to configure an A15 without LPAE. Are you sure it doesn?t support it? I meant CONFIG_LPAE not enabled yet in our defconfig. I was trying to get hyp mode enabled and saw that enabling LPAE config seemed mandatory as per your commit. > > > Alex > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-02 1:54 ` Keerthy @ 2016-08-02 11:59 ` Keerthy 2016-08-04 5:21 ` Alexander Graf 0 siblings, 1 reply; 8+ messages in thread From: Keerthy @ 2016-08-02 11:59 UTC (permalink / raw) To: u-boot Hi Alex, On Tuesday 02 August 2016 07:24 AM, Keerthy wrote: > > > On Tuesday 02 August 2016 03:16 AM, Alexander Graf wrote: >> >>> On 01 Aug 2016, at 11:07, Keerthy <a0393675@ti.com> wrote: >>> >>> Hi Alexander, >>> >>> I am trying to enable hypervisor in u-boot for DRA7(A15 based) family >>> of SoCs which does not have LPAE support yet. >>> >>> Is it mandatory for LPAE to be enabled before enabling hypervisor for >>> A15? >> >> HYP mode shares the same page table layout as the LPAE one. I?m >> actually surprised you managed to configure an A15 without LPAE. Are >> you sure it doesn?t support it? > > I meant CONFIG_LPAE not enabled yet in our defconfig. I was trying to > get hyp mode enabled and saw that enabling LPAE config seemed mandatory > as per your commit. I am referring http://liris.cnrs.fr/~mmrissa/lib/exe/fetch.php?media=armv7-a-r-manual.pdf. Attrm[3:0] bits for DCACHE_WRITEALLOC, DCACHE_WRITEBACK, DCACHE_WRITETHROUGH definitions. DCACHE_WRITEBACK should have 0x3 << 2 to get 11RW for MAIRn.Attrm[3:0] encoding. Correct me if i am wrong of referring a wrong document. Regards, Keerthy > >> >> >> Alex >> > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-02 11:59 ` Keerthy @ 2016-08-04 5:21 ` Alexander Graf 2016-08-04 5:30 ` Keerthy 0 siblings, 1 reply; 8+ messages in thread From: Alexander Graf @ 2016-08-04 5:21 UTC (permalink / raw) To: u-boot > On 02 Aug 2016, at 12:59, Keerthy <a0393675@ti.com> wrote: > > Hi Alex, > > > On Tuesday 02 August 2016 07:24 AM, Keerthy wrote: >> >> >> On Tuesday 02 August 2016 03:16 AM, Alexander Graf wrote: >>> >>>> On 01 Aug 2016, at 11:07, Keerthy <a0393675@ti.com> wrote: >>>> >>>> Hi Alexander, >>>> >>>> I am trying to enable hypervisor in u-boot for DRA7(A15 based) family >>>> of SoCs which does not have LPAE support yet. >>>> >>>> Is it mandatory for LPAE to be enabled before enabling hypervisor for >>>> A15? >>> >>> HYP mode shares the same page table layout as the LPAE one. I?m >>> actually surprised you managed to configure an A15 without LPAE. Are >>> you sure it doesn?t support it? >> >> I meant CONFIG_LPAE not enabled yet in our defconfig. I was trying to >> get hyp mode enabled and saw that enabling LPAE config seemed mandatory >> as per your commit. > > I am referring http://liris.cnrs.fr/~mmrissa/lib/exe/fetch.php?media=armv7-a-r-manual.pdf. > > Attrm[3:0] bits for DCACHE_WRITEALLOC, DCACHE_WRITEBACK, DCACHE_WRITETHROUGH definitions. > > DCACHE_WRITEBACK should have 0x3 << 2 to get 11RW for MAIRn.Attrm[3:0] encoding. > > Correct me if i am wrong of referring a wrong document. I?m not quite sure I understand your issue correctly. If you want to run in HYP mode, the only page table format available is the ?long-descriptor? page table format, which is the same as the LPAE page table format. That?s the one that also puts the caching modes into MAIR registers instead of the PTEs. So if you want to enable page tables (which you need to enable in order to enable caching), you have to write page tables in the LPAE format if you?re running in HYP mode. That?s why the config option is mandatory. See sections B3.1.3 and B3.3 in the document you linked to: "the translation tables for the Non-secure PL2 stage 1 translations, and for the Non-secure PL1&0 stage 2 translations, must use the Long-descriptor translation table format." Alex ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] Question on Enabling hypervisor mode in u-boot 2016-08-04 5:21 ` Alexander Graf @ 2016-08-04 5:30 ` Keerthy 0 siblings, 0 replies; 8+ messages in thread From: Keerthy @ 2016-08-04 5:30 UTC (permalink / raw) To: u-boot On Thursday 04 August 2016 10:51 AM, Alexander Graf wrote: > >> On 02 Aug 2016, at 12:59, Keerthy <a0393675@ti.com> wrote: >> >> Hi Alex, >> >> >> On Tuesday 02 August 2016 07:24 AM, Keerthy wrote: >>> >>> >>> On Tuesday 02 August 2016 03:16 AM, Alexander Graf wrote: >>>> >>>>> On 01 Aug 2016, at 11:07, Keerthy <a0393675@ti.com> wrote: >>>>> >>>>> Hi Alexander, >>>>> >>>>> I am trying to enable hypervisor in u-boot for DRA7(A15 based) family >>>>> of SoCs which does not have LPAE support yet. >>>>> >>>>> Is it mandatory for LPAE to be enabled before enabling hypervisor for >>>>> A15? >>>> >>>> HYP mode shares the same page table layout as the LPAE one. I?m >>>> actually surprised you managed to configure an A15 without LPAE. Are >>>> you sure it doesn?t support it? >>> >>> I meant CONFIG_LPAE not enabled yet in our defconfig. I was trying to >>> get hyp mode enabled and saw that enabling LPAE config seemed mandatory >>> as per your commit. >> >> I am referring http://liris.cnrs.fr/~mmrissa/lib/exe/fetch.php?media=armv7-a-r-manual.pdf. >> >> Attrm[3:0] bits for DCACHE_WRITEALLOC, DCACHE_WRITEBACK, DCACHE_WRITETHROUGH definitions. >> >> DCACHE_WRITEBACK should have 0x3 << 2 to get 11RW for MAIRn.Attrm[3:0] encoding. >> >> Correct me if i am wrong of referring a wrong document. > > I?m not quite sure I understand your issue correctly. If you want to run in HYP mode, the only page table format available is the ?long-descriptor? page table format, which is the same as the LPAE page table format. That?s the one that also puts the caching modes into MAIR registers instead of the PTEs. > > So if you want to enable page tables (which you need to enable in order to enable caching), you have to write page tables in the LPAE format if you?re running in HYP mode. That?s why the config option is mandatory. > > See sections B3.1.3 and B3.3 in the document you linked to: > > "the translation tables for the Non-secure PL2 stage 1 translations, and for the Non-secure PL1&0 stage 2 translations, must use the Long-descriptor translation table format." Thanks for the confirmations and pointers. I will go through that part of ARM documentation. Best Regards, Keerthy > > > Alex > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-08-04 5:30 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-08-01 10:07 [U-Boot] Question on Enabling hypervisor mode in u-boot Keerthy 2016-08-01 17:41 ` Andreas Färber 2016-08-02 2:19 ` Keerthy 2016-08-01 21:46 ` Alexander Graf 2016-08-02 1:54 ` Keerthy 2016-08-02 11:59 ` Keerthy 2016-08-04 5:21 ` Alexander Graf 2016-08-04 5:30 ` Keerthy
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox