Re: [PATCH v1 0/1] xilinx_zynq: Enable support for 4kiB MMU pages
On 27/11/2020 09:36, jan.som...@dlr.de wrote: -Original Message- From: Sebastian Huber Sent: Friday, November 27, 2020 9:27 AM To: Sommer, Jan;devel@rtems.org Subject: Re: [PATCH v1 0/1] xilinx_zynq: Enable support for 4kiB MMU pages On 26/11/2020 17:08, Jan Sommer wrote: This should enable the support for small MMU tables for the xilinx_zynq based BSPs and enable it by default. I would not enable it by default, since it makes the TLB cache less efficient and you more frequently have to access memory to get the MMU descriptors. You only need the 4KiB pages, if you do some extra stuff in your application. Ok, I copied the behavior from the realview BSP. Should I change it to default disabled there too? No, the realview_pbx_a9_qemu BSP is only used for testing and development on Qemu. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hub...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler Unsere Datenschutzerklärung finden Sie hier: https://embedded-brains.de/datenschutzerklaerung/ ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
RE: [PATCH v1 0/1] xilinx_zynq: Enable support for 4kiB MMU pages
> -Original Message- > From: Sebastian Huber > Sent: Friday, November 27, 2020 9:27 AM > To: Sommer, Jan ; devel@rtems.org > Subject: Re: [PATCH v1 0/1] xilinx_zynq: Enable support for 4kiB MMU pages > > On 26/11/2020 17:08, Jan Sommer wrote: > > > This should enable the support for small MMU tables for the > > xilinx_zynq based BSPs and enable it by default. > I would not enable it by default, since it makes the TLB cache less efficient > and you more frequently have to access memory to get the MMU > descriptors. You only need the 4KiB pages, if you do some extra stuff in your > application. Ok, I copied the behavior from the realview BSP. Should I change it to default disabled there too? Cheers, Jan > > > > Would this also be something to backport to RTEMS5? > > I could prepare a patch where it is not enabled by default, so that > > there is no difference for users of 5.1. > This would be fine from my side. > > -- > embedded brains GmbH > Herr Sebastian HUBER > Dornierstr. 4 > 82178 Puchheim > Germany > email: sebastian.hub...@embedded-brains.de > phone: +49-89-18 94 741 - 16 > fax: +49-89-18 94 741 - 08 > > Registergericht: Amtsgericht München > Registernummer: HRB 157899 > Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler > Unsere Datenschutzerklärung finden Sie hier: > https://embedded-brains.de/datenschutzerklaerung/ ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
Re: [PATCH v1 0/1] xilinx_zynq: Enable support for 4kiB MMU pages
On 26/11/2020 17:08, Jan Sommer wrote: This should enable the support for small MMU tables for the xilinx_zynq based BSPs and enable it by default. I would not enable it by default, since it makes the TLB cache less efficient and you more frequently have to access memory to get the MMU descriptors. You only need the 4KiB pages, if you do some extra stuff in your application. Would this also be something to backport to RTEMS5? I could prepare a patch where it is not enabled by default, so that there is no difference for users of 5.1. This would be fine from my side. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hub...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler Unsere Datenschutzerklärung finden Sie hier: https://embedded-brains.de/datenschutzerklaerung/ ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
[PATCH v1 0/1] xilinx_zynq: Enable support for 4kiB MMU pages
Hello, Thanks to the suggestions from Sebastian, here my first try to edit things in the new build system. This should enable the support for small MMU tables for the xilinx_zynq based BSPs and enable it by default. Would this also be something to backport to RTEMS5? I could prepare a patch where it is not enabled by default, so that there is no difference for users of 5.1. Ideally, the patch here https://lists.rtems.org/pipermail/devel/2020-November/063461.html is merged beforehand. Best regards, Jan Jan Sommer (1): bsp/xilinx_zynq: Enable support for 4kiB MMU pages c/src/lib/libbsp/arm/xilinx-zynq/configure.ac | 4 spec/build/bsps/arm/optmmusmallpages.yml | 10 ++ spec/build/bsps/arm/xilinx-zynq/grp.yml | 2 +- 3 files changed, 15 insertions(+), 1 deletion(-) -- 2.17.1 ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel