2007-04-29 17:48:58

by Jiri Benc

[permalink] [raw]
Subject: "Stable" branch of mac80211

Hi,

I've created a new 'stable' branch in my tree
(git://git.kernel.org/pub/scm/linux/kernel/git/jbenc/mac80211.git). It
contains patches that strip mac80211 down to a stable version. The purpose
of this branch is to prepare mac80211 for vanilla inclusion.

Please note that this branch is expected to be rebased frequently and all
patches are subject to change.

Currently, it contains following patches:

mac80211: remove 802.11n parts
mac80211: remove WPA testing stuff
mac80211: remove hostapd ioctls
mac80211: remove nonstable iwpriv ioctls
mac80211: remove iwpriv ioctls with wext equivalent
mac80211: remove AP and WDS support


Jiri

--
Jiri Benc
SUSE Labs


2007-04-30 17:29:16

by Jiri Benc

[permalink] [raw]
Subject: Re: "Stable" branch of mac80211

On Mon, 30 Apr 2007 09:52:51 -0700, James Ketrenos wrote:
> I'm curious what the criteria was for removing code from 'stable'
>
> Are there known bugs/oops? Is it known to be broken, or ?
>
> For code we know we ultimately want to be redone, removed, or whatever,
> I can understand not wanting to merge the code.

AP and user space MLME mode don't have clarified user space interface.
We don't want to support the current mess (mgmt interface) forever. WDS
is tightly linked with AP.

802.11n code is not deeply reviewed by anyone and thus it's unclear
whether it will need to be changed (I already heard some objections) or
not.

> However we ultimately want AP, WDS, and 802.11n.

We ultimately want STA mode and - maybe - IBSS. Other things are just
nice to have. They will still stay in wireless-dev, of course.

> At least with 802.11n, I haven't seen anything indicating the code
> needs to be done any differently than it is currently being done.

Nobody uses it. I don't see a reason why it needs to go in now. I
really want those patches to be reviewed by someone before considering
them "stable". It can be me, but I need some time to study .11n draft,
or someone else experienced with different hw designs.

> I assume the entire new tree will be marked as 'experimental', so what
> is the advantage gained by pulling this code out now?

Once the code is in vanilla, user space interfaces are set in stone.
Regarding .11n, untested and unreviewed patches shouldn't go to vanilla.

Thanks,

Jiri

--
Jiri Benc
SUSE Labs

2007-04-30 17:12:32

by James Ketrenos

[permalink] [raw]
Subject: Re: "Stable" branch of mac80211

Jiri Benc wrote:
> Hi,
>
> I've created a new 'stable' branch in my tree
> (git://git.kernel.org/pub/scm/linux/kernel/git/jbenc/mac80211.git). It
> contains patches that strip mac80211 down to a stable version. The purpose
> of this branch is to prepare mac80211 for vanilla inclusion.
...
> Currently, it contains following patches:
>
> mac80211: remove 802.11n parts
> mac80211: remove WPA testing stuff
> mac80211: remove hostapd ioctls
> mac80211: remove nonstable iwpriv ioctls
> mac80211: remove iwpriv ioctls with wext equivalent
> mac80211: remove AP and WDS support

I'm curious what the criteria was for removing code from 'stable'

Are there known bugs/oops? Is it known to be broken, or ?

For code we know we ultimately want to be redone, removed, or whatever, I can understand not wanting to merge the code. However we ultimately want AP, WDS, and 802.11n. At least with 802.11n, I haven't seen anything indicating the code needs to be done any differently than it is currently being done.

I assume the entire new tree will be marked as 'experimental', so what is the advantage gained by pulling this code out now?

James

2007-04-29 19:55:33

by Jiri Benc

[permalink] [raw]
Subject: Re: "Stable" branch of mac80211

On Sun, 29 Apr 2007 14:14:14 -0400, Michael Wu wrote:
> wireless-dev has been updated recently. One of the conflicts can be resolved
> easily. There's also an API change which conflicts with the qdisc fixes
> because tcf_destroy is no longer exported. A patch for that is attached.

I folded the patch into the qdisc fix patch and rebased my tree (all
branches).

I also ensured the previously misapplied radiotap patch is now applied
correctly and dropped the "move radiotap removal code to right function"
patch.

Thanks,

Jiri

--
Jiri Benc
SUSE Labs

2007-04-30 19:22:13

by John W. Linville

[permalink] [raw]
Subject: Re: "Stable" branch of mac80211

On Mon, Apr 30, 2007 at 07:29:51PM +0200, Jiri Benc wrote:
> On Mon, 30 Apr 2007 09:52:51 -0700, James Ketrenos wrote:

> > I assume the entire new tree will be marked as 'experimental', so what
> > is the advantage gained by pulling this code out now?
>
> Once the code is in vanilla, user space interfaces are set in stone.
> Regarding .11n, untested and unreviewed patches shouldn't go to vanilla.

ACK
--
John W. Linville
[email protected]

2007-04-29 19:37:14

by Michael Wu

[permalink] [raw]
Subject: Re: "Stable" branch of mac80211

On Sunday 29 April 2007 13:48, Jiri Benc wrote:
> Please note that this branch is expected to be rebased frequently and all
> patches are subject to change.
>
wireless-dev has been updated recently. One of the conflicts can be resolved
easily. There's also an API change which conflicts with the qdisc fixes
because tcf_destroy is no longer exported. A patch for that is attached.

Once updated to the latest wireless-dev, it should be fairly easy to pull
together a patch for net-2.6 since wireless-dev should have all the major
networking API changes for 2.6.22. I think mac80211 can be pushed to net-2.6
after that. What do you think?

Thanks,
-Michael Wu


Attachments:
(No filename) (0.00 B)
(No filename) (189.00 B)
Download all attachments

2007-05-01 01:00:20

by Jiri Benc

[permalink] [raw]
Subject: Re: "Stable" branch of mac80211

On Sun, 29 Apr 2007 19:48:55 +0200, Jiri Benc wrote:
> I've created a new 'stable' branch in my tree
> (git://git.kernel.org/pub/scm/linux/kernel/git/jbenc/mac80211.git). It
> contains patches that strip mac80211 down to a stable version. The purpose
> of this branch is to prepare mac80211 for vanilla inclusion.

And preliminary patches targeted for net-2.6 are in
http://kernel.org/pub/linux/kernel/people/jbenc/mac80211/

Please review and test (for testing purposes, a huge patch with all
mac80211 drivers currently in wireless-dev can be found at
http://kernel.org/pub/linux/kernel/people/jbenc/mac80211-test/drivers.patch).

Thanks,

Jiri

--
Jiri Benc
SUSE Labs