![]() | ||||||||||
Messages in this thread |
Tejun, On 11/12/06, Tejun Heo <htejun@gmail.com> 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-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | |||||||||
| Last update: 2006-12-13 10:43 [from the cache] ©2003-2008 | ||||||||||