Return-path: Received: from mail-gw0-f46.google.com ([74.125.83.46]:52840 "EHLO mail-gw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751771Ab1EBUBG (ORCPT ); Mon, 2 May 2011 16:01:06 -0400 MIME-Version: 1.0 In-Reply-To: References: <1304303082.2833.159.camel@localhost> Date: Mon, 2 May 2011 21:01:04 +0100 Message-ID: (sfid-20110502_220109_565383_4BADC101) Subject: Re: Frequent spurious tx_timeouts for libertas From: Daniel Drake To: Ben Hutchings Cc: netdev@vger.kernel.org, libertas-dev@lists.infradead.org, linux-wireless Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 2 May 2011 20:59, Daniel Drake wrote: > dev->trans_start is unconditionally initialized by dev_activate() in > sch_generic.c: Also, many drivers (including libertas) set it in their tx_timeout handlers: dev->trans_start = jiffies; /* prevent tx timeout */ I don't understand why (the TX timeout has already occurred, it can't be prevented at that stage). Daniel