[coreboot-gerrit] Change in coreboot[master]: cpu/x86/64bit: Generate static page tables from an assembly file

[Arthur Heymans (Code Review)]

Attention is currently required from: Raul Rangel, Patrick Rudolph.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63725 )

Change subject: cpu/x86/64bit: Generate static page tables from an assembly file
......................................................................

Patch Set 3:

(1 comment)

File src/cpu/x86/64bit/pt.S:

https://review.coreboot.org/c/coreboot/+/63725/comment/d9d9df5a_6ebafb50 
PS3, Line 27: .quad _GEN_PAGE(0x200000 * 0)
> > > Can you use a macro to reduce the duplication?
> > >
> > > https://sourceware.org/binutils/docs/as/Macro.html
> >
> > I'll try.
>
> Sadly 100 is the max amount of nested macros allowed. Let's see if it's possible to improve that...

The max is hardcoded. We can patch binutils but then it becomes impossible to compile coreboot with older and external toolchains. I guess this is the best we can do...