Re: [PATCH 2.6.18] WE-21 support (core API)

From
Michael Buesch <hidden>
Date
2006-09-01 18:57:28
Reply to
Jean Tourrilhes, 2006-09-01
On Friday 01 September 2006 18:35, Jean Tourrilhes wrote:
On Fri, Sep 01, 2006 at 08:54:00AM +0200, Johannes Berg wrote:
quoted
On Thu, 2006-08-31 at 10:12 -0700, Jean Tourrilhes wrote:
quoted
	And I strongly disagree with your disagrement ;-)
You're of course free to do that :) But let me explain.
	And my explanation is even more simple : let's not throw the
baby with the bathwater. If things are broken in WE, let's just fix
it. I've always worked with people working with me.
Fixing would require redesign of some parts, so incompatible ABI
changes. nl80211 actually _does_ fix it by rewriting the stuff.
	Note that one thing that worry me with your approach is
footprint. I've used various embedded devices over the years, such as
the Gumstix (4MB Flash), and this is why WE was optimised for
footprint.
Can you please explain in more detail, how WE + the WE-netlink wrapper
has lower footprint than this netlink-only layer?
quoted
Actually, no. But that's beside the point anyway. Actually, I think that
publishing WE over netlink was a mistake.
	I don't understand, in the last few years everybody was
clamoring for a Netlink interface, and now that it is done, people say
it is a stupid thing. Any specifics ?
quoted
The real
problem with WE is, as I previously said, the ill-defined semantics of
both the user-space API and the in-kernel API.
	I don't understand why you say it's ill defined, it 100%
documented in the iwconfig man page.
It is horribly documented.
There is one big union and one magic "extra" parameter.
You have to guess (or look at other implementations) to find
out which element of the union or even if and how to use the extra
parameter. That's a real pain.
And after you found out which element to use, you have to figure
out somehow how to actually use that element. That's nontrivial,
escpecially because some flags (that are not documented) may
magically change the whole semantics of the contents.

In my opinion this
"One function signature fits all" design used in WE is simply
broken by design. It leads to exactly this big union, the
magic extra field and all the other magic flags here and there.
It is in no way clear from looking at the function signature
what to do. But it should be. nl80211 is designed much better here.

-- 
Greetings Michael.
Thread (26 messages) 26 messages, 8 authors, last reply 2006-09-11
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help