Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-05-08 Thread Gerd Hoffmann
On Fri, May 05, 2023 at 08:33:41AM -0600, Rebecca Cran wrote: > Gerd, > > > Does this series need rework following this discussion, or is it ready to > merge? I think we are good to go. The ASSERT suggested by Ard here ... > > > > So it would be better to ASSERT() on non-empty GOT, and ignore

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-05-05 Thread Rebecca Cran
Gerd, Does this series need rework following this discussion, or is it ready to merge? -- Rebecca Cran On 4/21/23 01:46, Marvin Häuser wrote: On 21. Apr 2023, at 09:21, Ard Biesheuvel wrote: On Fri, 21 Apr 2023 at 08:49, Gerd Hoffmann wrote: On Fri, Apr 21, 2023 at 06:01:11AM +00

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-21 Thread Gerd Hoffmann
> > Patch #1, adding a linker script assert as suggested by ard, then: > > > > * compile + test on my local workstation (fedora 37, gcc 12). > > * run CI > > * compile on some older distros: > > - rhel-8 (gcc 8) > > - ubuntu-18.04 (gcc 7) > > Did you include NOOPT? GCC5 specifies USING_LTO fo

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-21 Thread Marvin Häuser
> On 21. Apr 2023, at 09:21, Ard Biesheuvel wrote: > > On Fri, 21 Apr 2023 at 08:49, Gerd Hoffmann wrote: >> >>> On Fri, Apr 21, 2023 at 06:01:11AM +, Marvin Häuser wrote: >>> On 21. Apr 2023, at 06:45, Gerd Hoffmann wrote: Not needed any more on modern toolchains, they

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-21 Thread Ard Biesheuvel
On Fri, 21 Apr 2023 at 08:49, Gerd Hoffmann wrote: > > On Fri, Apr 21, 2023 at 06:01:11AM +, Marvin Häuser wrote: > > > > > On 21. Apr 2023, at 06:45, Gerd Hoffmann wrote: > > > > > > Not needed any more on modern toolchains, they are better > > > in not creating a GOT without this trick. >

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-20 Thread Marvin Häuser
> On 21. Apr 2023, at 08:49, Gerd Hoffmann wrote: > > On Fri, Apr 21, 2023 at 06:01:11AM +, Marvin Häuser wrote: >> On 21. Apr 2023, at 06:45, Gerd Hoffmann wrote: >>> >>> Not needed any more on modern toolchains, they are better >>> in not creating a GOT without this trick. >> >

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-20 Thread Gerd Hoffmann
On Fri, Apr 21, 2023 at 06:01:11AM +, Marvin Häuser wrote: > > > On 21. Apr 2023, at 06:45, Gerd Hoffmann wrote: > > > > Not needed any more on modern toolchains, they are better > > in not creating a GOT without this trick. > > Hi Gerd, > > Thanks! Just out of interest, how did you test

Re: [edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-20 Thread Marvin Häuser
> On 21. Apr 2023, at 06:45, Gerd Hoffmann wrote: > > Not needed any more on modern toolchains, they are better > in not creating a GOT without this trick. Hi Gerd, Thanks! Just out of interest, how did you test this and what were the results? Best regards, Marvin > > Signed-off-by: Gerd

[edk2-devel] [PATCH v5 02/10] MdePkg: don't set visibility to hidden

2023-04-20 Thread Gerd Hoffmann
Not needed any more on modern toolchains, they are better in not creating a GOT without this trick. Signed-off-by: Gerd Hoffmann --- MdePkg/Include/X64/ProcessorBind.h | 14 -- 1 file changed, 14 deletions(-) diff --git a/MdePkg/Include/X64/ProcessorBind.h b/MdePkg/Include/X64/Proc