Re: [PATCH] mac80211: rewrite fragmentation code

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Herbert Xu <herbert@...>
Cc: Johannes Berg <johannes@...>, <linux-wireless@...>, <netdev@...>, Ron Rindjunsky <ron.rindjunsky@...>, Tomas Winkler <tomasw@...>, Ivo van Doorn <ivdoorn@...>, Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@...>
Date: Thursday, May 8, 2008 - 9:13 am

On Thursday 08 May 2008 15:08:46 Herbert Xu wrote:

Ok, why is nobody actually answering my _question_? :)
So I'll try to answer it by myself: the hard_start_xmit callback
is actually supposed to return "void", but it returns "int" just
for historical reasons. So there is no way to fail in a TX handler.
If a failure happens, the driver must free and drop the skb.
How likely errors are is a completely different story. One could argue
that errors are unlikely for any part of the kernel, except for very
sone special cases.

So does that also apply to the wireless stack? Should we change drivers
to always return 0. later we can change the function prototype to return void.

-- 
Greetings Michael.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[RFC/RFT 3/4] mac80211: use GSO for fragmentation, Johannes Berg, (Wed Apr 30, 8:40 am)
Re: [RFC/RFT 3/4] mac80211: use GSO for fragmentation, Johannes Berg, (Wed May 7, 4:50 am)
[PATCH] mac80211: rewrite fragmentation code, Johannes Berg, (Wed May 7, 7:22 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Wed May 7, 7:41 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Johannes Berg, (Wed May 7, 7:52 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Wed May 7, 9:05 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Johannes Berg, (Wed May 7, 3:19 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, Michael Buesch, (Wed May 7, 9:48 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Wed May 7, 11:22 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, Michael Buesch, (Thu May 8, 9:00 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Thu May 8, 9:08 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Michael Buesch, (Thu May 8, 9:13 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Thu May 8, 9:32 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Michael Buesch, (Thu May 8, 9:15 am)
Re: [PATCH] mac80211: rewrite fragmentation code, David Miller, (Wed May 7, 11:26 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, Rusty Russell, (Thu May 15, 10:01 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, David Miller, (Fri May 16, 12:58 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Rusty Russell, (Fri May 16, 6:32 am)
Re: [PATCH] mac80211: rewrite fragmentation code, David Miller, (Fri May 16, 3:40 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, Rusty Russell, (Sun May 18, 11:08 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, David Miller, (Mon May 19, 3:03 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Fri May 16, 8:15 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Johannes Berg, (Fri May 16, 6:38 am)
Re: [PATCH] mac80211: rewrite fragmentation code, Herbert Xu, (Thu May 15, 11:28 pm)
Re: [PATCH] mac80211: rewrite fragmentation code, Johannes Berg, (Thu May 8, 5:00 am)