On Wed, Feb 03, 2016 at 01:36:00PM +0300, Sergei Shtylyov wrote: > On 2/3/2016 1:32 PM, Paul Burton wrote: > > >>>MIPS32 code uses rel-style relocs, and MIPS32r6 modules may include the > >>>R_MIPS_PC16 relocation. We thus need to support R_MIPS_PC16 rel-style > >>>relocations in order to load MIPS32r6 kernel modules. This patch adds > >>>such support, which is similar to the rela-style R_MIPS_PC16 support but > >> > >> R_MIPS_LO16, you mean? > > > >Hi Sergei, > > > >No, I mean it's similar to the R_MIPS_PC16 code in module-rela.c. That > >is, its rela-style equivalent (rather than rel-style as here). > > But you're *adding* R_MIPS_PC16, no? Yup, this patch is adding R_MIPS_PC16 to module.c & it's similar to R_MIPS_PC16 in module-rela.c. (Incidentally I think we could tidy up the duplication between the two files, but that can come later...) Thanks, Paul