2003-05-22 06:35:07

by jurriaan

[permalink] [raw]
Subject: bk15 build fails with modular apm (undefined reference to MODULE_OWNER)

gcc -Wp,-MD,init/.version.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
-pipe -mpreferred-stack-boundary=2 -march=athlon -Iinclude/asm-i386/mach-default -nostdinc -iwithprefix include -DKBUILD_BASENAME=
version -DKBUILD_MODNAME=version -c -o init/version.o init/version.c
ld -m elf_i386 -r -o init/built-in.o init/main.o init/version.o init/mounts.o init/initramfs.o
ld -m elf_i386 -T arch/i386/vmlinux.lds.s arch/i386/kernel/head.o arch/i386/kernel/init_task.o init/built-in.o --start-gro
up usr/built-in.o arch/i386/kernel/built-in.o arch/i386/mm/built-in.o arch/i386/mach-default/built-in.o kernel/built-in.o mm/bu
ilt-in.o fs/built-in.o ipc/built-in.o security/built-in.o crypto/built-in.o lib/lib.a arch/i386/lib/lib.a drivers/built-in.o
sound/built-in.o arch/i386/pci/built-in.o net/built-in.o --end-group -o .tmp_vmlinux1
arch/i386/kernel/built-in.o(.init.text+0x5744): In function `apm_init':
: undefined reference to `SET_MODULE_OWNER'
make: *** [.tmp_vmlinux1] Error 1
:

Is SET_MODULE_OWNER needed or not?
Browsing the source it seems it is defined as a nop in some places
(include/linux/netdevice.h) which is where it is mostly used...

Jurriaan
--
Don't ever trust the needle
It lies
Queensryche - Operation Mindcrime
Debian (Unstable) GNU/Linux 2.5.69-bk11 4104 bogomips load av: 0.29 0.48 0.31


2003-05-22 07:28:54

by jurriaan

[permalink] [raw]
Subject: Re: bk15 build fails with modular apm (undefined reference to MODULE_OWNER)

From: Jurriaan <[email protected]>
Date: Thu, May 22, 2003 at 08:47:46AM +0200
> gcc -Wp,-MD,init/.version.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
> -pipe -mpreferred-stack-boundary=2 -march=athlon -Iinclude/asm-i386/mach-default -nostdinc -iwithprefix include -DKBUILD_BASENAME=
> version -DKBUILD_MODNAME=version -c -o init/version.o init/version.c
> ld -m elf_i386 -r -o init/built-in.o init/main.o init/version.o init/mounts.o init/initramfs.o
> ld -m elf_i386 -T arch/i386/vmlinux.lds.s arch/i386/kernel/head.o arch/i386/kernel/init_task.o init/built-in.o --start-gro
> up usr/built-in.o arch/i386/kernel/built-in.o arch/i386/mm/built-in.o arch/i386/mach-default/built-in.o kernel/built-in.o mm/bu
> ilt-in.o fs/built-in.o ipc/built-in.o security/built-in.o crypto/built-in.o lib/lib.a arch/i386/lib/lib.a drivers/built-in.o
> sound/built-in.o arch/i386/pci/built-in.o net/built-in.o --end-group -o .tmp_vmlinux1
> arch/i386/kernel/built-in.o(.init.text+0x5744): In function `apm_init':
> : undefined reference to `SET_MODULE_OWNER'
> make: *** [.tmp_vmlinux1] Error 1
> :
>
> Is SET_MODULE_OWNER needed or not?
> Browsing the source it seems it is defined as a nop in some places
> (include/linux/netdevice.h) which is where it is mostly used...
>
Sorry for not being clear the first time:

this is _not_ with apm modular, but with CONFIG_APM=y.

Jurriaan
--
In the middle of a good time
Truth gave me her icy kiss
Look around, you must be joking
Oysterband - All that way for this
Debian (Unstable) GNU/Linux 2.5.69-bk11 4104 bogomips load av: 0.26 0.21 0.22

2003-05-22 08:24:48

by ismail donmez

[permalink] [raw]
Subject: Re: Re: bk15 build fails with modular apm (undefined reference to MODULE_OWNER)

Hi,

Adding
#define SET_MODULE_OWNER(apm_proc) do { } while (0)
to arch/i386/kernel/apm.c fixes this.

-----Original Message-----
From: Jurriaan <[email protected]>
To: [email protected]
Date: Thu, 22 May 2003 09:41:40 +0200
Subject: Re: bk15 build fails with modular apm (undefined reference to MODULE_OWNER)

From: Jurriaan <[email protected]>
Date: Thu, May 22, 2003 at 08:47:46AM +0200
> gcc -Wp,-MD,init/.version.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
> -pipe -mpreferred-stack-boundary=2 -march=athlon -Iinclude/asm-i386/mach-default -nostdinc -iwithprefix include -DKBUILD_BASENAME=
> version -DKBUILD_MODNAME=version -c -o init/version.o init/version.c
> ld -m elf_i386 -r -o init/built-in.o init/main.o init/version.o init/mounts.o init/initramfs.o
> ld -m elf_i386 -T arch/i386/vmlinux.lds.s arch/i386/kernel/head.o arch/i386/kernel/init_task.o init/built-in.o --start-gro
> up usr/built-in.o arch/i386/kernel/built-in.o arch/i386/mm/built-in.o arch/i386/mach-default/built-in.o kernel/built-in.o mm/bu
> ilt-in.o fs/built-in.o ipc/built-in.o security/built-in.o crypto/built-in.o lib/lib.a arch/i386/lib/lib.a drivers/built-in.o
> sound/built-in.o arch/i386/pci/built-in.o net/built-in.o --end-group -o .tmp_vmlinux1
> arch/i386/kernel/built-in.o(.init.text+0x5744): In function `apm_init':
> : undefined reference to `SET_MODULE_OWNER'
> make: *** [.tmp_vmlinux1] Error 1
> :
>
> Is SET_MODULE_OWNER needed or not?
> Browsing the source it seems it is defined as a nop in some places
> (include/linux/netdevice.h) which is where it is mostly used...
>
Sorry for not being clear the first time:

this is _not_ with apm modular, but with CONFIG_APM=y.

Jurriaan
--
In the middle of a good time
Truth gave me her icy kiss
Look around, you must be joking
Oysterband - All that way for this
Debian (Unstable) GNU/Linux 2.5.69-bk11 4104 bogomips load av: 0.26 0.21 0.22

2003-05-22 09:49:23

by YOSHIFUJI Hideaki

[permalink] [raw]
Subject: Re: bk15 build fails with modular apm (undefined reference to MODULE_OWNER)

In article <[email protected]> (at Thu, 22 May 2003 02:33:39 -0600), "ismail donmez" <[email protected]> says:

> Adding
> #define SET_MODULE_OWNER(apm_proc) do { } while (0)
> to arch/i386/kernel/apm.c fixes this.

:

> > arch/i386/kernel/built-in.o(.init.text+0x5744): In function `apm_init':
> > : undefined reference to `SET_MODULE_OWNER'
> > make: *** [.tmp_vmlinux1] Error 1
> > :
> >
> > Is SET_MODULE_OWNER needed or not?

I believe we should set apm_proc->owner here.

Index: linux25-LINUS/arch/i386/kernel/apm.c
===================================================================
RCS file: /cvsroot/usagi/usagi-backport/linux25/arch/i386/kernel/apm.c,v
retrieving revision 1.1.1.12
diff -u -r1.1.1.12 apm.c
--- linux25-LINUS/arch/i386/kernel/apm.c 15 May 2003 07:52:06 -0000 1.1.1.12
+++ linux25-LINUS/arch/i386/kernel/apm.c 22 May 2003 09:22:04 -0000
@@ -2013,7 +2013,7 @@

apm_proc = create_proc_info_entry("apm", 0, NULL, apm_get_info);
if (apm_proc)
- SET_MODULE_OWNER(apm_proc);
+ apm_proc->owner = THIS_MODULE;

kernel_thread(apm, NULL, CLONE_FS | CLONE_FILES | CLONE_SIGHAND | SIGCHLD);


--
Hideaki YOSHIFUJI @ USAGI Project <[email protected]>
GPG FP: 9022 65EB 1ECF 3AD1 0BDF 80D8 4807 F894 E062 0EEA