Re: [GIT PULL request] ELF rewrite part 2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On 5/11/09, Andreas Robinson <andr345@xxxxxxxxx> wrote:
> Update: I recompiled and got Valgrind working.
> Your "runtests --valgrind" patch ... not yet.

Ok.  It worked for me today.

Damn, I meant to say that you probably need to use "git apply" not
"patch", because it creates symlinks.  That might be your problem.

> sudo valgrind build/depmod
> returns errors like this one, not the one you see:
>
> ==19659== Invalid read of size 4
> ==19659==    at 0x401BBE: output_symbols (depmod.c:755)
> ==19659==    by 0x4041ED: main (depmod.c:1364)
> ==19659==  Address 0x53616ac is 284 bytes inside a block of size 287
> alloc'd
> ==19659==    at 0x4C2794E: malloc (vg_replace_malloc.c:207)
> ==19659==    by 0x403186: grab_module (depmod.c:269)
> ==19659==    by 0x403529: do_module (depmod.c:539)
> ==19659==    by 0x4034BB: grab_dir (depmod.c:581)
> ==19659==    by 0x403459: grab_dir (depmod.c:594)
> ==19659==    by 0x403459: grab_dir (depmod.c:594)
> ==19659==    by 0x403459: grab_dir (depmod.c:594)
> ==19659==    by 0x404414: main (depmod.c:616)
>
> It seems related to the new structs fields (module_tables and elf_file)
> in struct module. Is sizeof() returning the wrong size?!
>
> Anyway, I'll keep looking.

Ok... I'll investigate my end, it probably depends on the modules
which are used.

I didn't see those errors, but I have seen similar ones in the past.
They should go away if you do

valgrind --suppressions tests/vg-suppressions

or (I think this only works on 32-bit)

valgrind --partial-loads-ok=yes

Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Home]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Video Projectors]     [PDAs]     [Free Online Dating]     [Hacking TiVo]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]     [Big List of Linux Books]     [16.7MP]

Add to Google Powered by Linux