lkml.org 
[lkml]   [1996]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: modules 2.1.8 fix

your patch seems to fix a part of it. instead of complaining about
uninitialised modules when trying to insmod isofs, it now segfaults.
However I am able to insmod ppp, sound and slhc modules just fine. I am
not able to insmod any of the filesystem modules such as fat, vfat, msdos
and isofs...

--Jauder

On Mon, 11 Nov 1996, Jacques Gelinas wrote:

> On Mon, 11 Nov 1996, Heiko Eissfeldt wrote:
>
> > --- insmod/load_elf.c~ Mon Nov 4 05:56:58 1996
> > +++ insmod/load_elf.c Mon Nov 11 14:40:22 1996
> > @@ -366,7 +366,7 @@
> > }
> >
> > /* JEJB: zero the bss (now it's actually allocated) */
> > - memset(secref[bss_seg], 0, bss_size);
> > + memset(secref[bss_seg], bss_size, 0);
> >
> > for (spnt = sections, i = 0; i < epnt->e_shnum; ++i, ++spnt) {
> > #ifdef __i386__
>
> Am I missing something ? The prototype of memset is
>
> memset (addr,character,nb_char);
>
> Your line is equivalent to a nop. Again, am I missing something.
>
> Some people have reported a bug about this. The bug is that some module
> lack a bss. Here is the official patch I intend to include in the next
> release of modules-2.1.8
>
> *** modules-2.1.8/insmod/load_elf.c Sun Nov 3 23:56:58 1996
> --- modules-2.1.8-1/insmod/load_elf.c Mon Nov 11 07:22:55 1996
> ***************
> *** 366,372 ****
> }
>
> /* JEJB: zero the bss (now it's actually allocated) */
> ! memset(secref[bss_seg], 0, bss_size);
>
> for (spnt = sections, i = 0; i < epnt->e_shnum; ++i, ++spnt) {
> #ifdef __i386__
> --- 366,372 ----
> }
>
> /* JEJB: zero the bss (now it's actually allocated) */
> ! if (bss_size > 0) memset(secref[bss_seg], 0, bss_size);
>
> for (spnt = sections, i = 0; i < epnt->e_shnum; ++i, ++spnt) {
> #ifdef __i386__
> ***************
> *** 622,628 ****
> }
>
> loaded = (char *)(((int)loaded + 3) & ~3);
> ! bss_size = loaded - secref[bss_seg];
> progsize = codesize = loaded - textseg;
> aout_flag = 0; /* i.e.: if it's not a.out, it _has_ to be ELF... */
> if (defsym(strncmp, "_GLOBAL_OFFSET_TABLE_", loaded - textseg, N_BSS | N_EXT, TRANSIENT))
> --- 622,630 ----
> }
>
> loaded = (char *)(((int)loaded + 3) & ~3);
> ! if (bss_seg != -1){
> ! bss_size = loaded - secref[bss_seg];
> ! }
> progsize = codesize = loaded - textseg;
> aout_flag = 0; /* i.e.: if it's not a.out, it _has_ to be ELF... */
> if (defsym(strncmp, "_GLOBAL_OFFSET_TABLE_", loaded - textseg, N_BSS | N_EXT, TRANSIENT))
>
>
> --------------------------------------------------------
> Jacques Gelinas (jacques@solucorp.qc.ca)
> Linuxconf: The ultimate administration system for Linux.
> see http://www.solucorp.qc.ca/linuxconf
>
>
>
>


.sig under construction


\
 
 \ /
  Last update: 2005-03-22 13:38    [W:0.099 / U:1.260 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site