[Crash-utility] Re: [PATCH v2] symbols: skip load .init.* sections if module was successfully initialized

> There might be address overlap of one module's .init.text symbols and > another module's .text symbols. As a result, gdb fails to translate the > address to symbol name correctly: > > crash> sym -m virtio_blk | grep MODULE > ffffffffc00a4000 MODULE START: virtio_blk > ffffffffc00a86ec MODULE END: virtio_blk > crash> gdb info address floppy_module_init > Symbol "floppy_module_init" is a function at address 0xffffffffc00a4131. > > Since the .init.* sections of a module had been freed by kernel if the > module was initialized successfully, there is no need to load the .init.* > sections data from "*.ko.debug" in gdb to create such an overlap. > lm->mod_init_module_ptr is used as a flag of whether module is freed. > > Good findings, Tao. The v2 looks good to me. So: Ack.