[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20110113.215020.69966117.davem@davemloft.net>
Date: Thu, 13 Jan 2011 21:50:20 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: romieu@...zoreil.com
Cc: netdev@...r.kernel.org, jarek@...o.eu.org, hayeswang@...ltek.com,
benh@...ian.org, torvalds@...ux-foundation.org
Subject: Re: [PATCH] r8169: keep firmware in memory.
From: Francois Romieu <romieu@...zoreil.com>
Date: Fri, 14 Jan 2011 00:07:53 +0100
> The firmware agent is not available during resume. Loading the firmware
> during open() (see eee3a96c6368f47df8df5bd4ed1843600652b337) is not
> enough.
>
> close() is run during resume through rtl8169_reset_task(), whence the
> mildly natural release of firmware in the driver removal method instead.
>
> It will help with http://bugs.debian.org/609538. It will not avoid
> the 60 seconds delay when:
> - there is no firmware
> - the driver is loaded and the device is not up before a suspend/resume
>
> Signed-off-by: Francois Romieu <romieu@...zoreil.com>
> Tested-by: Jarek KamiĆski <jarek@...o.eu.org>
> Cc: Hayes <hayeswang@...ltek.com>
> Cc: Ben Hutchings <benh@...ian.org>
Applied.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists