Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2021-02-05 Thread Nathan Chancellor
On Mon, Dec 07, 2020 at 09:46:46PM -0800, Fangrui Song wrote: > arm64 references the start address of .builtin_fw (__start_builtin_fw) > with a pair of R_AARCH64_ADR_PREL_PG_HI21/R_AARCH64_LDST64_ABS_LO12_NC > relocations. The compiler is allowed to emit the > R_AARCH64_LDST64_ABS_LO12_NC

Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2021-02-05 Thread Doug Anderson
Hi, On Fri, Jan 22, 2021 at 11:04 AM Doug Anderson wrote: > > Hi, > > On Tue, Jan 5, 2021 at 11:20 AM Nick Desaulniers > wrote: > > > > On Tue, Jan 5, 2021 at 9:45 AM Doug Anderson wrote: > > > > > > Hi, > > > > > > On Mon, Dec 7, 2020 at 9:49 PM Fangrui Song wrote: > > > > > > > > arm64

Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2021-01-22 Thread Doug Anderson
Hi, On Tue, Jan 5, 2021 at 11:20 AM Nick Desaulniers wrote: > > On Tue, Jan 5, 2021 at 9:45 AM Doug Anderson wrote: > > > > Hi, > > > > On Mon, Dec 7, 2020 at 9:49 PM Fangrui Song wrote: > > > > > > arm64 references the start address of .builtin_fw (__start_builtin_fw) > > > with a pair of

Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2021-01-05 Thread Nick Desaulniers
On Tue, Jan 5, 2021 at 9:45 AM Doug Anderson wrote: > > Hi, > > On Mon, Dec 7, 2020 at 9:49 PM Fangrui Song wrote: > > > > arm64 references the start address of .builtin_fw (__start_builtin_fw) > > with a pair of R_AARCH64_ADR_PREL_PG_HI21/R_AARCH64_LDST64_ABS_LO12_NC > > relocations. The

Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2021-01-05 Thread Doug Anderson
Hi, On Mon, Dec 7, 2020 at 9:49 PM Fangrui Song wrote: > > arm64 references the start address of .builtin_fw (__start_builtin_fw) > with a pair of R_AARCH64_ADR_PREL_PG_HI21/R_AARCH64_LDST64_ABS_LO12_NC > relocations. The compiler is allowed to emit the > R_AARCH64_LDST64_ABS_LO12_NC relocation

Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2020-12-08 Thread Nick Desaulniers
On Mon, Dec 7, 2020 at 9:46 PM Fangrui Song wrote: > > arm64 references the start address of .builtin_fw (__start_builtin_fw) > with a pair of R_AARCH64_ADR_PREL_PG_HI21/R_AARCH64_LDST64_ABS_LO12_NC > relocations. The compiler is allowed to emit the > R_AARCH64_LDST64_ABS_LO12_NC relocation

[PATCH v2] firmware_loader: Align .builtin_fw to 8

2020-12-07 Thread Fangrui Song
arm64 references the start address of .builtin_fw (__start_builtin_fw) with a pair of R_AARCH64_ADR_PREL_PG_HI21/R_AARCH64_LDST64_ABS_LO12_NC relocations. The compiler is allowed to emit the R_AARCH64_LDST64_ABS_LO12_NC relocation because struct builtin_fw in include/linux/firmware.h is 8-byte