* [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe
@ 2018-02-27 13:47 Ard Biesheuvel
2018-02-27 17:19 ` Ard Biesheuvel
0 siblings, 1 reply; 6+ messages in thread
From: Ard Biesheuvel @ 2018-02-27 13:47 UTC (permalink / raw)
To: edk2-devel; +Cc: leif.lindholm, Ard Biesheuvel
The Designware PCIe IP in the SynQuacer SoC needs a little help to
appear sane to the OS. Not only does it lack a true root port, and
therefore does not perform any filtering whatsoever of type 0 config
TLPs that are not intended for the link peer, it also has trouble
issuing 64-bit wide MMIO accesses, which are often used on MMIO BARs
with memory semantics (e.g., frame buffers).
So let's create a stage 2 mapping covering the entire physical address
space, and remap some ECAM regions and demote write combine attributes
to device/strongly ordered. This is not a water tight fix, but it does
work around the issues in the majority of cases.
(Note that the ECAM remapping can also be addressed in the SMMU mapping
of the PCIe IP exposed to the CPU, but this is currently under
development, and it does not hurt to have it in two places)
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
This is a followup to/replacement for '[RFC PATCH edk2-non-osi]
Platform/DeveloperBox: add prebuilt binary containing stage 2 page tables'
Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 1 +
Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 5 +-
Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile | 23 +++++
Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S | 88 ++++++++++++++++++++
Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf | 32 +++++++
5 files changed, 148 insertions(+), 1 deletion(-)
diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc
index 925ce36d278b..eb088524d2dd 100644
--- a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc
+++ b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc
@@ -580,6 +580,7 @@ [Components.common]
MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf
MdeModulePkg/Universal/EbcDxe/EbcDxe.inf
MdeModulePkg/Universal/Acpi/BootGraphicsResourceTableDxe/BootGraphicsResourceTableDxe.inf
+ Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
#
# eMMC support
diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.fdf b/Platform/Socionext/DeveloperBox/DeveloperBox.fdf
index edde1cfe3932..e4e4f13764cc 100644
--- a/Platform/Socionext/DeveloperBox/DeveloperBox.fdf
+++ b/Platform/Socionext/DeveloperBox/DeveloperBox.fdf
@@ -50,9 +50,12 @@ [FD.SPI_NOR_IMAGE]
#
################################################################################
-0x00000000|0x00080000
+0x00000000|0x00078000
FILE = Platform/Socionext/DeveloperBox/fip_all_arm_tf.bin
+0x00078000|0x00008000
+FILE = $(OUTPUT_DIRECTORY)/$(TARGET)_$(TOOL_CHAIN_TAG)/$(ARCH)/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables/OUTPUT/Stage2Tables.bin
+
0x00080000|0x00200000
gArmTokenSpaceGuid.PcdFvBaseAddress|gArmTokenSpaceGuid.PcdFvSize
FV = FVMAIN_COMPACT
diff --git a/Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile b/Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile
new file mode 100644
index 000000000000..a96ff6d2d01f
--- /dev/null
+++ b/Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile
@@ -0,0 +1,23 @@
+## @file
+#
+# Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
+#
+# This program and the accompanying materials
+# are licensed and made available under the terms and conditions of the BSD License
+# which accompanies this distribution. The full text of the license may be found at
+# http://opensource.org/licenses/bsd-license.php
+#
+# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+#
+##
+
+.PHONY: all
+all: $(OUTPUT_DIR)/Stage2Tables.bin
+
+$(OUTPUT_DIR)/Stage2Tables.bin: $(OUTPUT_DIR)/Stage2Tables.elf
+ $(OBJCOPY) $(OBJCOPY_FLAGS) $(^) $(@)
+
+$(OUTPUT_DIR)/Stage2Tables.elf: $(MODULE_DIR)/Stage2Tables.S
+ $(ASM) -o $(@) $(^) -nostdlib \
+ -Wl,-e,0x81f8000,--section-start=.rodata=0x81f8000
diff --git a/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S
new file mode 100644
index 000000000000..313ef3c56abc
--- /dev/null
+++ b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S
@@ -0,0 +1,88 @@
+/** @file
+ Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
+
+ This program and the accompanying materials are licensed and made available
+ under the terms and conditions of the BSD License which accompanies this
+ distribution. The full text of the license may be found at
+ http://opensource.org/licenses/bsd-license.php
+
+ THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+**/
+
+/*
+ * This file contains the assembler code to instantiate a set of stage 2
+ * translation tables that make the ECAM space of the Synopsys DesignWare
+ * PCIe root complexes appear sane to the OS.
+ * - ECAM 'shadows' caused by non TLP filtering root ports are eliminated
+ * - MMIO region are mapped with device attributes that supersede write combine
+ * attributes that the OS may attempt to use, and which is not supported by
+ * the SoC.
+ */
+
+#define TT_S2_CONT_SHIFT 52
+#define TT_S2_AF (0x1 << 10)
+#define TT_S2_SH_NON_SHAREABLE (0x0 << 8)
+#define TT_S2_AP_RW (0x3 << 6)
+#define TT_S2_MEMATTR_DEVICE_nGRE (0x2 << 2)
+#define TT_S2_MEMATTR_MEMORY_WB (0xf << 2)
+#define TT_S2_TABLE (0x3 << 0)
+#define TT_S2_L3_PAGE (0x1 << 1)
+#define TT_S2_VALID (0x1 << 0)
+
+ .altmacro
+ .macro for, start, count, do, arg2, arg3, arg4
+ .if \count == 1
+ \do \start, \arg2, \arg3, \arg4
+ .elseif \count > 1
+ for \start, %(\count / 2), \do, \arg2, \arg3, \arg4
+ for %(\start + \count / 2), %((\count + 1) / 2), \do, \arg2, \arg3, \arg4
+ .endif
+ .endm
+
+ .macro s2_dev_entry, base, shift=30, offset=0, cont=0
+ .quad ((\base << \shift) + \offset) | TT_S2_AF | TT_S2_AP_RW | \
+ TT_S2_SH_NON_SHAREABLE | TT_S2_MEMATTR_DEVICE_nGRE | \
+ TT_S2_VALID | (\cont << TT_S2_CONT_SHIFT)
+ .endm
+
+ .macro s2_mem_entry, base, shift=30, offset=0, cont=0
+ .quad ((\base << \shift) + \offset) | TT_S2_AF | TT_S2_AP_RW | \
+ TT_S2_SH_NON_SHAREABLE | TT_S2_MEMATTR_MEMORY_WB | \
+ TT_S2_VALID | (\cont << TT_S2_CONT_SHIFT)
+ .endm
+
+ .macro s2_l3_entry, base, offset=0, cont=0
+ .quad ((\base << 12) + \offset) | TT_S2_AF | TT_S2_AP_RW | \
+ TT_S2_SH_NON_SHAREABLE | TT_S2_MEMATTR_MEMORY_WB | \
+ TT_S2_L3_PAGE | TT_S2_VALID | (\cont << TT_S2_CONT_SHIFT)
+ .endm
+
+ .section ".rodata", "a", %progbits
+ /* level 1 */
+ s2_mem_entry 0 /* 0x0000_0000 - 0x3fff_ffff */
+ .quad 1f + TT_S2_TABLE /* 0x4000_0000 - 0x7fff_ffff */
+ for 2, 246, s2_mem_entry /* 0x8000_0000 - 0x3d_ffff_ffff */
+ for 248, 8, s2_dev_entry /* PCIe MMIO64 */
+ for 256, 768, s2_mem_entry /* 0x40_0000_0000 - 0xff_ffff_ffff */
+
+ /* level 2 */
+1:for 0, 256, s2_mem_entry, 21, 0x40000000, 1
+
+ .quad 2f + TT_S2_TABLE /* 0x6000_0000 -> RC #0 bus 0 */
+ for 1, 15, s2_mem_entry, 21, 0x60000000
+ for 0, 48, s2_mem_entry, 21, 0x62000000, 1
+ for 0, 64, s2_dev_entry, 21, 0x68000000, 1 /* PCIe MMIO32 */
+
+ .quad 3f + TT_S2_TABLE /* 0x7000_0000 -> RC #1 bus 0 */
+ for 1, 15, s2_mem_entry, 21, 0x70000000
+ for 0, 48, s2_mem_entry, 21, 0x72000000, 1
+ for 0, 64, s2_dev_entry, 21, 0x78000000, 1 /* PCIe MMIO32 */
+
+ /* level 3 */
+2:for 0, 8, s2_l3_entry, 0x60000000
+ for 0, 8, s2_l3_entry, 0x60010000 /* hide device #1 */
+ for 0, 496, s2_l3_entry, 0x60010000, 1
+3:for 0, 8, s2_l3_entry, 0x70000000
+ for 0, 8, s2_l3_entry, 0x70010000 /* hide device #1 */
+ for 0, 496, s2_l3_entry, 0x70010000, 1
diff --git a/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
new file mode 100644
index 000000000000..9bec659af444
--- /dev/null
+++ b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
@@ -0,0 +1,32 @@
+## @file
+#
+# Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
+#
+# This program and the accompanying materials
+# are licensed and made available under the terms and conditions of the BSD License
+# which accompanies this distribution. The full text of the license may be found at
+# http://opensource.org/licenses/bsd-license.php
+#
+# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+#
+##
+
+[Defines]
+ INF_VERSION = 0x0001001A
+ BASE_NAME = Stage2Tables
+ FILE_GUID = e11cbca5-8f82-41a7-8441-02b48acc13a7
+ MODULE_TYPE = USER_DEFINED
+ VERSION_STRING = 1.0
+ CUSTOM_MAKEFILE = GCC|GNUmakefile
+
+[Sources]
+ Stage2Tables.S
+
+[Packages]
+ MdePkg/MdePkg.dec
+ Silicon/Socionext/SynQuacer/SynQuacer.dec
+
+[BuildOptions]
+ *_*_*_OBJCOPY_PATH == objcopy
+ *_*_*_OBJCOPY_FLAGS == -I elf64-little -O binary -j .rodata
--
2.11.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe
2018-02-27 13:47 [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe Ard Biesheuvel
@ 2018-02-27 17:19 ` Ard Biesheuvel
2018-02-27 17:36 ` Leif Lindholm
0 siblings, 1 reply; 6+ messages in thread
From: Ard Biesheuvel @ 2018-02-27 17:19 UTC (permalink / raw)
To: edk2-devel@lists.01.org; +Cc: Leif Lindholm, Ard Biesheuvel
On 27 February 2018 at 13:47, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
> The Designware PCIe IP in the SynQuacer SoC needs a little help to
> appear sane to the OS. Not only does it lack a true root port, and
> therefore does not perform any filtering whatsoever of type 0 config
> TLPs that are not intended for the link peer, it also has trouble
> issuing 64-bit wide MMIO accesses, which are often used on MMIO BARs
> with memory semantics (e.g., frame buffers).
>
> So let's create a stage 2 mapping covering the entire physical address
> space, and remap some ECAM regions and demote write combine attributes
> to device/strongly ordered. This is not a water tight fix, but it does
> work around the issues in the majority of cases.
>
> (Note that the ECAM remapping can also be addressed in the SMMU mapping
> of the PCIe IP exposed to the CPU, but this is currently under
> development, and it does not hurt to have it in two places)
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
> This is a followup to/replacement for '[RFC PATCH edk2-non-osi]
> Platform/DeveloperBox: add prebuilt binary containing stage 2 page tables'
>
> Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 1 +
> Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 5 +-
> Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile | 23 +++++
> Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S | 88 ++++++++++++++++++++
> Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf | 32 +++++++
> 5 files changed, 148 insertions(+), 1 deletion(-)
>
...
> diff --git a/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
> new file mode 100644
> index 000000000000..9bec659af444
> --- /dev/null
> +++ b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
> @@ -0,0 +1,32 @@
> +## @file
> +#
> +# Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
> +#
> +# This program and the accompanying materials
> +# are licensed and made available under the terms and conditions of the BSD License
> +# which accompanies this distribution. The full text of the license may be found at
> +# http://opensource.org/licenses/bsd-license.php
> +#
> +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +#
> +##
> +
> +[Defines]
> + INF_VERSION = 0x0001001A
> + BASE_NAME = Stage2Tables
> + FILE_GUID = e11cbca5-8f82-41a7-8441-02b48acc13a7
> + MODULE_TYPE = USER_DEFINED
> + VERSION_STRING = 1.0
> + CUSTOM_MAKEFILE = GCC|GNUmakefile
> +
> +[Sources]
> + Stage2Tables.S
> +
> +[Packages]
> + MdePkg/MdePkg.dec
> + Silicon/Socionext/SynQuacer/SynQuacer.dec
> +
> +[BuildOptions]
> + *_*_*_OBJCOPY_PATH == objcopy
> + *_*_*_OBJCOPY_FLAGS == -I elf64-little -O binary -j .rodata
I can improve this and fix Clang at the same time by adding
*_*_*_ASM_FLAGS == -nostdlib
-Wl,-e,0x81f8000,--section-start=.rodata=0x81f8000
*_CLANG35_*_ASM_FLAGS = -no-integrated-as
*_CLANG38_*_ASM_FLAGS = -no-integrated-as
(and remove the linker arguments from the Makefile)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe
2018-02-27 17:19 ` Ard Biesheuvel
@ 2018-02-27 17:36 ` Leif Lindholm
2018-02-27 17:45 ` Ard Biesheuvel
0 siblings, 1 reply; 6+ messages in thread
From: Leif Lindholm @ 2018-02-27 17:36 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel@lists.01.org
On Tue, Feb 27, 2018 at 05:19:59PM +0000, Ard Biesheuvel wrote:
> On 27 February 2018 at 13:47, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
> > The Designware PCIe IP in the SynQuacer SoC needs a little help to
> > appear sane to the OS. Not only does it lack a true root port, and
> > therefore does not perform any filtering whatsoever of type 0 config
> > TLPs that are not intended for the link peer, it also has trouble
> > issuing 64-bit wide MMIO accesses, which are often used on MMIO BARs
> > with memory semantics (e.g., frame buffers).
> >
> > So let's create a stage 2 mapping covering the entire physical address
> > space, and remap some ECAM regions and demote write combine attributes
> > to device/strongly ordered. This is not a water tight fix, but it does
> > work around the issues in the majority of cases.
> >
> > (Note that the ECAM remapping can also be addressed in the SMMU mapping
> > of the PCIe IP exposed to the CPU, but this is currently under
> > development, and it does not hurt to have it in two places)
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > ---
> > This is a followup to/replacement for '[RFC PATCH edk2-non-osi]
> > Platform/DeveloperBox: add prebuilt binary containing stage 2 page tables'
> >
> > Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 1 +
> > Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 5 +-
> > Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile | 23 +++++
> > Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S | 88 ++++++++++++++++++++
> > Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf | 32 +++++++
> > 5 files changed, 148 insertions(+), 1 deletion(-)
> >
> ...
> > diff --git a/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
> > new file mode 100644
> > index 000000000000..9bec659af444
> > --- /dev/null
> > +++ b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
> > @@ -0,0 +1,32 @@
> > +## @file
> > +#
> > +# Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
> > +#
> > +# This program and the accompanying materials
> > +# are licensed and made available under the terms and conditions of the BSD License
> > +# which accompanies this distribution. The full text of the license may be found at
> > +# http://opensource.org/licenses/bsd-license.php
> > +#
> > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> > +#
> > +##
> > +
> > +[Defines]
> > + INF_VERSION = 0x0001001A
> > + BASE_NAME = Stage2Tables
> > + FILE_GUID = e11cbca5-8f82-41a7-8441-02b48acc13a7
> > + MODULE_TYPE = USER_DEFINED
> > + VERSION_STRING = 1.0
> > + CUSTOM_MAKEFILE = GCC|GNUmakefile
> > +
> > +[Sources]
> > + Stage2Tables.S
> > +
> > +[Packages]
> > + MdePkg/MdePkg.dec
> > + Silicon/Socionext/SynQuacer/SynQuacer.dec
> > +
> > +[BuildOptions]
> > + *_*_*_OBJCOPY_PATH == objcopy
> > + *_*_*_OBJCOPY_FLAGS == -I elf64-little -O binary -j .rodata
>
> I can improve this and fix Clang at the same time by adding
>
> *_*_*_ASM_FLAGS == -nostdlib
> -Wl,-e,0x81f8000,--section-start=.rodata=0x81f8000
> *_CLANG35_*_ASM_FLAGS = -no-integrated-as
> *_CLANG38_*_ASM_FLAGS = -no-integrated-as
>
> (and remove the linker arguments from the Makefile)
I can confirm this resolves the CLANG issue.
Could we do this with a CLANG_ALL_ASM_FLAGS, rather than listing each
new toolchain profile as they get added?
/
Leif
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe
2018-02-27 17:36 ` Leif Lindholm
@ 2018-02-27 17:45 ` Ard Biesheuvel
2018-02-27 18:10 ` Leif Lindholm
0 siblings, 1 reply; 6+ messages in thread
From: Ard Biesheuvel @ 2018-02-27 17:45 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel@lists.01.org
On 27 February 2018 at 17:36, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>
> On Tue, Feb 27, 2018 at 05:19:59PM +0000, Ard Biesheuvel wrote:
>> On 27 February 2018 at 13:47, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
>> > The Designware PCIe IP in the SynQuacer SoC needs a little help to
>> > appear sane to the OS. Not only does it lack a true root port, and
>> > therefore does not perform any filtering whatsoever of type 0 config
>> > TLPs that are not intended for the link peer, it also has trouble
>> > issuing 64-bit wide MMIO accesses, which are often used on MMIO BARs
>> > with memory semantics (e.g., frame buffers).
>> >
>> > So let's create a stage 2 mapping covering the entire physical address
>> > space, and remap some ECAM regions and demote write combine attributes
>> > to device/strongly ordered. This is not a water tight fix, but it does
>> > work around the issues in the majority of cases.
>> >
>> > (Note that the ECAM remapping can also be addressed in the SMMU mapping
>> > of the PCIe IP exposed to the CPU, but this is currently under
>> > development, and it does not hurt to have it in two places)
>> >
>> > Contributed-under: TianoCore Contribution Agreement 1.1
>> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> > ---
>> > This is a followup to/replacement for '[RFC PATCH edk2-non-osi]
>> > Platform/DeveloperBox: add prebuilt binary containing stage 2 page tables'
>> >
>> > Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 1 +
>> > Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 5 +-
>> > Silicon/Socionext/SynQuacer/Stage2Tables/GNUmakefile | 23 +++++
>> > Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.S | 88 ++++++++++++++++++++
>> > Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf | 32 +++++++
>> > 5 files changed, 148 insertions(+), 1 deletion(-)
>> >
>> ...
>> > diff --git a/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
>> > new file mode 100644
>> > index 000000000000..9bec659af444
>> > --- /dev/null
>> > +++ b/Silicon/Socionext/SynQuacer/Stage2Tables/Stage2Tables.inf
>> > @@ -0,0 +1,32 @@
>> > +## @file
>> > +#
>> > +# Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
>> > +#
>> > +# This program and the accompanying materials
>> > +# are licensed and made available under the terms and conditions of the BSD License
>> > +# which accompanies this distribution. The full text of the license may be found at
>> > +# http://opensource.org/licenses/bsd-license.php
>> > +#
>> > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
>> > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
>> > +#
>> > +##
>> > +
>> > +[Defines]
>> > + INF_VERSION = 0x0001001A
>> > + BASE_NAME = Stage2Tables
>> > + FILE_GUID = e11cbca5-8f82-41a7-8441-02b48acc13a7
>> > + MODULE_TYPE = USER_DEFINED
>> > + VERSION_STRING = 1.0
>> > + CUSTOM_MAKEFILE = GCC|GNUmakefile
>> > +
>> > +[Sources]
>> > + Stage2Tables.S
>> > +
>> > +[Packages]
>> > + MdePkg/MdePkg.dec
>> > + Silicon/Socionext/SynQuacer/SynQuacer.dec
>> > +
>> > +[BuildOptions]
>> > + *_*_*_OBJCOPY_PATH == objcopy
>> > + *_*_*_OBJCOPY_FLAGS == -I elf64-little -O binary -j .rodata
>>
>> I can improve this and fix Clang at the same time by adding
>>
>> *_*_*_ASM_FLAGS == -nostdlib
>> -Wl,-e,0x81f8000,--section-start=.rodata=0x81f8000
>> *_CLANG35_*_ASM_FLAGS = -no-integrated-as
>> *_CLANG38_*_ASM_FLAGS = -no-integrated-as
>>
>> (and remove the linker arguments from the Makefile)
>
> I can confirm this resolves the CLANG issue.
>
> Could we do this with a CLANG_ALL_ASM_FLAGS, rather than listing each
> new toolchain profile as they get added?
>
No, not really. CLANG3x is not a separate toolchain family, so we can
apply things to GCC+CLANG combined, or to certain versions
individually.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe
2018-02-27 17:45 ` Ard Biesheuvel
@ 2018-02-27 18:10 ` Leif Lindholm
2018-02-28 9:18 ` Ard Biesheuvel
0 siblings, 1 reply; 6+ messages in thread
From: Leif Lindholm @ 2018-02-27 18:10 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel@lists.01.org
On Tue, Feb 27, 2018 at 05:45:09PM +0000, Ard Biesheuvel wrote:
> >> > +[Sources]
> >> > + Stage2Tables.S
> >> > +
> >> > +[Packages]
> >> > + MdePkg/MdePkg.dec
> >> > + Silicon/Socionext/SynQuacer/SynQuacer.dec
> >> > +
> >> > +[BuildOptions]
> >> > + *_*_*_OBJCOPY_PATH == objcopy
> >> > + *_*_*_OBJCOPY_FLAGS == -I elf64-little -O binary -j .rodata
> >>
> >> I can improve this and fix Clang at the same time by adding
> >>
> >> *_*_*_ASM_FLAGS == -nostdlib
> >> -Wl,-e,0x81f8000,--section-start=.rodata=0x81f8000
> >> *_CLANG35_*_ASM_FLAGS = -no-integrated-as
> >> *_CLANG38_*_ASM_FLAGS = -no-integrated-as
> >>
> >> (and remove the linker arguments from the Makefile)
> >
> > I can confirm this resolves the CLANG issue.
> >
> > Could we do this with a CLANG_ALL_ASM_FLAGS, rather than listing each
> > new toolchain profile as they get added?
>
> No, not really. CLANG3x is not a separate toolchain family, so we can
> apply things to GCC+CLANG combined, or to certain versions
> individually.
Ah, I had managed to miss that bit up until now. Thanks!
In that case, with these latest changes:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
/
Leif
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe
2018-02-27 18:10 ` Leif Lindholm
@ 2018-02-28 9:18 ` Ard Biesheuvel
0 siblings, 0 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2018-02-28 9:18 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel@lists.01.org
On 27 February 2018 at 18:10, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Tue, Feb 27, 2018 at 05:45:09PM +0000, Ard Biesheuvel wrote:
>> >> > +[Sources]
>> >> > + Stage2Tables.S
>> >> > +
>> >> > +[Packages]
>> >> > + MdePkg/MdePkg.dec
>> >> > + Silicon/Socionext/SynQuacer/SynQuacer.dec
>> >> > +
>> >> > +[BuildOptions]
>> >> > + *_*_*_OBJCOPY_PATH == objcopy
>> >> > + *_*_*_OBJCOPY_FLAGS == -I elf64-little -O binary -j .rodata
>> >>
>> >> I can improve this and fix Clang at the same time by adding
>> >>
>> >> *_*_*_ASM_FLAGS == -nostdlib
>> >> -Wl,-e,0x81f8000,--section-start=.rodata=0x81f8000
>> >> *_CLANG35_*_ASM_FLAGS = -no-integrated-as
>> >> *_CLANG38_*_ASM_FLAGS = -no-integrated-as
>> >>
>> >> (and remove the linker arguments from the Makefile)
>> >
>> > I can confirm this resolves the CLANG issue.
>> >
>> > Could we do this with a CLANG_ALL_ASM_FLAGS, rather than listing each
>> > new toolchain profile as they get added?
>>
>> No, not really. CLANG3x is not a separate toolchain family, so we can
>> apply things to GCC+CLANG combined, or to certain versions
>> individually.
>
> Ah, I had managed to miss that bit up until now. Thanks!
>
> In that case, with these latest changes:
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>
Thanks
Pushed as f90743b8813518fd7111c272ea4a3483a94ed462
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-02-28 9:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-27 13:47 [PATCH edk2-platforms] Silicon/SynQuacer: add stage 2 override translation tables for PCIe Ard Biesheuvel
2018-02-27 17:19 ` Ard Biesheuvel
2018-02-27 17:36 ` Leif Lindholm
2018-02-27 17:45 ` Ard Biesheuvel
2018-02-27 18:10 ` Leif Lindholm
2018-02-28 9:18 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox