[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090703.201347.146036232.davem@davemloft.net>
Date: Fri, 03 Jul 2009 20:13:47 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: herbert@...dor.apana.org.au
Cc: rusty@...tcorp.com.au, patrick.ohly@...el.com,
netdev@...r.kernel.org, virtualization@...ts.linux-foundation.org,
divy@...lsio.com, rolandd@...co.com, xemul@...nvz.org,
dcbw@...hat.com, libertas-dev@...ts.infradead.org
Subject: Re: [PATCH 1/4] net: skb_orphan on dev_hard_start_xmit
From: Herbert Xu <herbert@...dor.apana.org.au>
Date: Sat, 4 Jul 2009 11:08:30 +0800
> On Fri, Jul 03, 2009 at 08:02:54PM -0700, David Miller wrote:
>>
>> In particular the case of handling a device without usable TX
>> completion event indications is still quite troublesome.
> e
> Which particular devices do you have in mind?
NIU
I basically can't defer interrupts because the chip supports
per-TX-desc interrupt indications but it lacks an "all TX queue sent"
event. So if, say, tell it to interrupt every 1/4 of the TX queue
then up to 1/4 of the queue can have packets "stuck" in there
if TX activity all of a sudden ceases.
The only thing I've come up with to be able to mitigate interrupts is
to use an hrtimer of some sort. But that's going to be hard to get
right, and who knows what kind of latencies will be introduced for TX
completion packet freeing unless I am very carefull.
And finally this belongs in generic code, not in the NIU driver,
whatever we come up with. Especially since my understanding is that
this is similar to what Rusty needs.
--
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