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
