patch updated: It seems for me e1000_power_down_phy() should be added to rollbacke1000_power_up_phy(). I know, currently e1000_up() always return success. But this behaviour may be changed in future... Andrew, could you please use attached patch instead previous version? --- irq leak was found in 2.6.18-rc4 and e1000 7.2.7 from sourceforge: if e1000_up fails in e1000_open() we do not free allocated irq Signed-off-by: Vasily Averin Thank you, Vasily Averin SWsoft Virtuozzo/OpenVZ Linux kernel team