On Thu, Dec 03, 2015 at 12:02:27PM +0000, Catalin Marinas wrote: > On Wed, Nov 18, 2015 at 12:16:44AM +0300, Yury Norov wrote: > > From: Andrew Pinski <[email protected]> > > > > Reviewed-by: David Daney <[email protected]> > > > > Signed-off-by: Philipp Tomsich <[email protected]> > > Signed-off-by: Christoph Muellner <[email protected]> > > Signed-off-by: Yury Norov <[email protected]> > > Signed-off-by: Andrew Pinski <[email protected]> > > Please add a long description to every patch, even if it is trivial. > > And a nitpick: remove the empty lines between Reviewed-by and > Signed-off-by. These tags usually come in the same block. > > > +#else > > + > > +typedef elf_greg_t compat_elf_greg_t; > > +typedef elf_gregset_t compat_elf_gregset_t; > > +#define compat_a32_elf_check_arch(x) 0 > > +#define COMPAT_SET_PERSONALITY(ex) > > +#define COMPAT_ARCH_DLINFO > > + > > +#endif > > Do you need these definitions here? I guess they may be used later by > the ILP32 code paths but for A32 compat they don't make much sense (I > haven't made it to the end of the series yet). >
It should have been attached to patch "support core dump generation for ILP32". I'll fix. > -- > Catalin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
