Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread Maciej W. Rozycki
On Wed, 3 Feb 2016, Paul Burton wrote: > > Hmm, this looks like a fatal error condition to me, the module won't > > load. Why `pr_warn' rather than `pr_err' then? Likewise in the other > > file. > > To me fatality implies death, and nothing dies here. The module isn't > loaded but that's

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread Paul Burton
On Wed, Feb 03, 2016 at 12:24:38PM +, Maciej W. Rozycki wrote: > On Wed, 3 Feb 2016, Paul Burton wrote: > > > --- a/arch/mips/kernel/module-rela.c > > +++ b/arch/mips/kernel/module-rela.c > > @@ -134,9 +135,21 @@ int apply_relocate_add(Elf_Shdr *sechdrs, const char > > *strtab, > >

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread Maciej W. Rozycki
On Wed, 3 Feb 2016, Paul Burton wrote: > --- a/arch/mips/kernel/module-rela.c > +++ b/arch/mips/kernel/module-rela.c > @@ -134,9 +135,21 @@ int apply_relocate_add(Elf_Shdr *sechdrs, const char > *strtab, > return -ENOENT; > } > > - v =

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread James Hogan
On Wed, Feb 03, 2016 at 03:44:41AM +, Paul Burton wrote: > When an unsupported reloc is encountered in a module, we currently > blindly branch to whatever would be at its entry in the reloc handler > function pointer arrays. This may be NULL, or if the unsupported reloc > has a type greater

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread James Hogan
On Wed, Feb 03, 2016 at 03:44:41AM +, Paul Burton wrote: > When an unsupported reloc is encountered in a module, we currently > blindly branch to whatever would be at its entry in the reloc handler > function pointer arrays. This may be NULL, or if the unsupported reloc > has a type greater

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread Maciej W. Rozycki
On Wed, 3 Feb 2016, Paul Burton wrote: > --- a/arch/mips/kernel/module-rela.c > +++ b/arch/mips/kernel/module-rela.c > @@ -134,9 +135,21 @@ int apply_relocate_add(Elf_Shdr *sechdrs, const char > *strtab, > return -ENOENT; > } > > - v =

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread Paul Burton
On Wed, Feb 03, 2016 at 12:24:38PM +, Maciej W. Rozycki wrote: > On Wed, 3 Feb 2016, Paul Burton wrote: > > > --- a/arch/mips/kernel/module-rela.c > > +++ b/arch/mips/kernel/module-rela.c > > @@ -134,9 +135,21 @@ int apply_relocate_add(Elf_Shdr *sechdrs, const char > > *strtab, > >

Re: [PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-03 Thread Maciej W. Rozycki
On Wed, 3 Feb 2016, Paul Burton wrote: > > Hmm, this looks like a fatal error condition to me, the module won't > > load. Why `pr_warn' rather than `pr_err' then? Likewise in the other > > file. > > To me fatality implies death, and nothing dies here. The module isn't > loaded but that's

[PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-02 Thread Paul Burton
When an unsupported reloc is encountered in a module, we currently blindly branch to whatever would be at its entry in the reloc handler function pointer arrays. This may be NULL, or if the unsupported reloc has a type greater than that of the supported reloc with the highest type then we'll

[PATCH 1/5] MIPS: Bail on unsupported module relocs

2016-02-02 Thread Paul Burton
When an unsupported reloc is encountered in a module, we currently blindly branch to whatever would be at its entry in the reloc handler function pointer arrays. This may be NULL, or if the unsupported reloc has a type greater than that of the supported reloc with the highest type then we'll