Tejun, On 11/12/06, Tejun Heo <htejun@xxxxxxxxx> wrote:
piix_init_one() allocates host private data which should be freed by piix_host_stop(). ich_pata_ops wasn't converted to piix_host_stop() while merging, leaking 4 bytes on driver detach. Fix it.
I tried your patch last night but the leak is still reported. I need to investigate further and put some printk's in the piix_host_stop function to check whether the freeing really takes place. What I can't follow is where the ata_port_info.private_data (port_info[] or ppinfo[]) in piix_init_one gets transfered to ata_host.private_data (the "host" argument) that piix_host_stop tries to free. -- Catalin - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
