powerpc: question concerning insmod

[Denys Vlasenko]

On Sunday 13 July 2008 21:36, Harald Küthe wrote:
> />/ I looked into the source file indicated, and it seems like the variable 
> />/ ifile is defined in this section, which does not include a " || 
> />/ defined(__powerpc__)" condition:
> />/ 
> />/ #if defined(__arm__) || defined(__i386__) || defined(__mc68000__) \
> />/  || defined(__sh__) || defined(__s390__) || defined(__x86_64__)
> />/         struct arch_file *ifile = (struct arch_file *) f;
> />/ #endif
> /
> Does it work if you add || defined(__powerpc__) ?
> --
> vda
> 
> It works for me.

HE IS ALIVE! ;)

> Patch:
> --- busybox-1.11.1.orig/modutils/insmod.c.orig    2008-06-25 
> 14:51:18.000000000 +0200
> +++ busybox-1.11.1/modutils/insmod.c    2008-07-13 15:58:34.000000000 +0200
> @@ -839,7 +839,7 @@
>                 struct obj_symbol *sym,
>                 ElfW(RelM) *rel, ElfW(Addr) v)
> {
> -#if defined(__arm__) || defined(__i386__) || defined(__mc68000__) \
> +#if defined(__arm__) || defined(__i386__) || defined(__mc68000__) || 
> defined(__powerpc__) \
>  || defined(__sh__) || defined(__s390__) || defined(__x86_64__)
>     struct arch_file *ifile = (struct arch_file *) f;
> #endif

Applied, thanks!
--
vda