All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels
@ 2015-06-09 16:26 Jean-Pierre TOSONI
  2015-09-15  9:57 ` Matthias May
  0 siblings, 1 reply; 4+ messages in thread
From: Jean-Pierre TOSONI @ 2015-06-09 16:26 UTC (permalink / raw
  To: linux-wireless

Allow the combination of 8 (AP + mesh) interfaces on radar channels

Tested with hostapd, 2 and 8 BSSIDs, 1 or 2 DFS channels, BW 20 MHz
TODO: test AP+mesh combination
TODO: test 40 MHz BW
---
This patch was done with quilt on Openwrt against compat-wireless
2014-11-04

Any feedback?

--- a/drivers/net/wireless/ath/ath9k/init.c
+++ b/drivers/net/wireless/ath/ath9k/init.c
@@ -723,6 +723,14 @@ static const struct ieee80211_iface_limi
 				 BIT(NL80211_IFTYPE_ADHOC) },
 };
 
+static const struct ieee80211_iface_limit if_dfs_limits_mvap[] = {
+	{ .max = 8,	.types =
+#ifdef CONFIG_MAC80211_MESH
+				 BIT(NL80211_IFTYPE_MESH_POINT) |
+#endif
+				 BIT(NL80211_IFTYPE_AP) },
+};
+
 static const struct ieee80211_iface_combination if_comb[] = {
 	{
 		.limits = if_limits,
@@ -746,6 +754,16 @@ static const struct ieee80211_iface_comb
 		.num_different_channels = 1,
 		.beacon_int_infra_match = true,
 		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
|
+					BIT(NL80211_CHAN_WIDTH_20) |
+					BIT(NL80211_CHAN_WIDTH_40),
+	},
+	{
+		.limits = if_dfs_limits_mvap,
+		.n_limits = ARRAY_SIZE(if_dfs_limits_mvap),
+		.max_interfaces = 8,
+		.num_different_channels = 1,
+		.beacon_int_infra_match = true,
+		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
|
 					BIT(NL80211_CHAN_WIDTH_20) |
 					BIT(NL80211_CHAN_WIDTH_40),
 	}
-- 
quilt


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels
  2015-06-09 16:26 [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels Jean-Pierre TOSONI
@ 2015-09-15  9:57 ` Matthias May
  2015-09-16 10:27     ` [ath9k-devel] " Jean-Pierre Tosoni
  0 siblings, 1 reply; 4+ messages in thread
From: Matthias May @ 2015-09-15  9:57 UTC (permalink / raw
  To: Jean-Pierre TOSONI, linux-wireless

On 09/06/15 18:26, Jean-Pierre TOSONI wrote:
> Allow the combination of 8 (AP + mesh) interfaces on radar channels
>
> Tested with hostapd, 2 and 8 BSSIDs, 1 or 2 DFS channels, BW 20 MHz
> TODO: test AP+mesh combination
> TODO: test 40 MHz BW
> ---
> This patch was done with quilt on Openwrt against compat-wireless
> 2014-11-04
>
> Any feedback?
>
> --- a/drivers/net/wireless/ath/ath9k/init.c
> +++ b/drivers/net/wireless/ath/ath9k/init.c
> @@ -723,6 +723,14 @@ static const struct ieee80211_iface_limi
>   				 BIT(NL80211_IFTYPE_ADHOC) },
>   };
>   
> +static const struct ieee80211_iface_limit if_dfs_limits_mvap[] = {
> +	{ .max = 8,	.types =
> +#ifdef CONFIG_MAC80211_MESH
> +				 BIT(NL80211_IFTYPE_MESH_POINT) |
> +#endif
> +				 BIT(NL80211_IFTYPE_AP) },
> +};
> +
>   static const struct ieee80211_iface_combination if_comb[] = {
>   	{
>   		.limits = if_limits,
> @@ -746,6 +754,16 @@ static const struct ieee80211_iface_comb
>   		.num_different_channels = 1,
>   		.beacon_int_infra_match = true,
>   		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
> |
> +					BIT(NL80211_CHAN_WIDTH_20) |
> +					BIT(NL80211_CHAN_WIDTH_40),
> +	},
> +	{
> +		.limits = if_dfs_limits_mvap,
> +		.n_limits = ARRAY_SIZE(if_dfs_limits_mvap),
> +		.max_interfaces = 8,
> +		.num_different_channels = 1,
> +		.beacon_int_infra_match = true,
> +		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
> |
>   					BIT(NL80211_CHAN_WIDTH_20) |
>   					BIT(NL80211_CHAN_WIDTH_40),
>   	}
Hi
This mail is in my list of "potentially interesting changes".
Do you plan to send this as patch at some point?
Or are you still testing something?

Best regards
Matthias

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels
  2015-09-15  9:57 ` Matthias May
@ 2015-09-16 10:27     ` Jean-Pierre Tosoni
  0 siblings, 0 replies; 4+ messages in thread
From: Jean-Pierre Tosoni @ 2015-09-16  8:01 UTC (permalink / raw
  To: 'Matthias May', linux-wireless; +Cc: ath9k-devel, ath9k-devel

Hi Matthias,

The patch is working for me, and I am using it, but I could not test
Much further since real-life radar situations are hard to produce.

Before DFS integration, combinations were less limited. I submitted
the patch as a RFC only, because I suspected the DFS people had a
hidden reason for limiting the combinations on radar channels.

However I received no comments.

Maybe I was on the wrong list. I will CC the ath9k maintainer now,
and try to submit a patch in a week or two.

Best regards
Jean-Pierre

> -----Message d'origine-----
> De : Matthias May [mailto:matthias.may@neratec.com]
> Envoyé : mardi 15 septembre 2015 11:57
> À : Jean-Pierre TOSONI; linux-wireless@vger.kernel.org
> Objet : Re: [RFC v2] [resend] ath9k: allow up to 8 access point vifs on
> DFS channels
> 
> On 09/06/15 18:26, Jean-Pierre TOSONI wrote:
> > Allow the combination of 8 (AP + mesh) interfaces on radar channels
> >
> > Tested with hostapd, 2 and 8 BSSIDs, 1 or 2 DFS channels, BW 20 MHz
> > TODO: test AP+mesh combination
> > TODO: test 40 MHz BW
> > ---
> > This patch was done with quilt on Openwrt against compat-wireless
> > 2014-11-04
> >
> > Any feedback?
> >
> > --- a/drivers/net/wireless/ath/ath9k/init.c
> > +++ b/drivers/net/wireless/ath/ath9k/init.c
> > @@ -723,6 +723,14 @@ static const struct ieee80211_iface_limi
> >   				 BIT(NL80211_IFTYPE_ADHOC) },
> >   };
> >
> > +static const struct ieee80211_iface_limit if_dfs_limits_mvap[] = {
> > +	{ .max = 8,	.types =
> > +#ifdef CONFIG_MAC80211_MESH
> > +				 BIT(NL80211_IFTYPE_MESH_POINT) | #endif
> > +				 BIT(NL80211_IFTYPE_AP) },
> > +};
> > +
> >   static const struct ieee80211_iface_combination if_comb[] = {
> >   	{
> >   		.limits = if_limits,
> > @@ -746,6 +754,16 @@ static const struct ieee80211_iface_comb
> >   		.num_different_channels = 1,
> >   		.beacon_int_infra_match = true,
> >   		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
> > |
> > +					BIT(NL80211_CHAN_WIDTH_20) |
> > +					BIT(NL80211_CHAN_WIDTH_40),
> > +	},
> > +	{
> > +		.limits = if_dfs_limits_mvap,
> > +		.n_limits = ARRAY_SIZE(if_dfs_limits_mvap),
> > +		.max_interfaces = 8,
> > +		.num_different_channels = 1,
> > +		.beacon_int_infra_match = true,
> > +		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
> > |
> >   					BIT(NL80211_CHAN_WIDTH_20) |
> >   					BIT(NL80211_CHAN_WIDTH_40),
> >   	}
> Hi
> This mail is in my list of "potentially interesting changes".
> Do you plan to send this as patch at some point?
> Or are you still testing something?
> 
> Best regards
> Matthias


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [ath9k-devel] [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels
@ 2015-09-16 10:27     ` Jean-Pierre Tosoni
  0 siblings, 0 replies; 4+ messages in thread
From: Jean-Pierre Tosoni @ 2015-09-16 10:27 UTC (permalink / raw
  To: ath9k-devel

Hi Matthias,

The patch is working for me, and I am using it, but I could not test
Much further since real-life radar situations are hard to produce.

Before DFS integration, combinations were less limited. I submitted
the patch as a RFC only, because I suspected the DFS people had a
hidden reason for limiting the combinations on radar channels.

However I received no comments.

Maybe I was on the wrong list. I will CC the ath9k maintainer now,
and try to submit a patch in a week or two.

Best regards
Jean-Pierre

> -----Message d'origine-----
> De?: Matthias May [mailto:matthias.may at neratec.com]
> Envoy??: mardi 15 septembre 2015 11:57
> ??: Jean-Pierre TOSONI; linux-wireless at vger.kernel.org
> Objet?: Re: [RFC v2] [resend] ath9k: allow up to 8 access point vifs on
> DFS channels
> 
> On 09/06/15 18:26, Jean-Pierre TOSONI wrote:
> > Allow the combination of 8 (AP + mesh) interfaces on radar channels
> >
> > Tested with hostapd, 2 and 8 BSSIDs, 1 or 2 DFS channels, BW 20 MHz
> > TODO: test AP+mesh combination
> > TODO: test 40 MHz BW
> > ---
> > This patch was done with quilt on Openwrt against compat-wireless
> > 2014-11-04
> >
> > Any feedback?
> >
> > --- a/drivers/net/wireless/ath/ath9k/init.c
> > +++ b/drivers/net/wireless/ath/ath9k/init.c
> > @@ -723,6 +723,14 @@ static const struct ieee80211_iface_limi
> >   				 BIT(NL80211_IFTYPE_ADHOC) },
> >   };
> >
> > +static const struct ieee80211_iface_limit if_dfs_limits_mvap[] = {
> > +	{ .max = 8,	.types =
> > +#ifdef CONFIG_MAC80211_MESH
> > +				 BIT(NL80211_IFTYPE_MESH_POINT) | #endif
> > +				 BIT(NL80211_IFTYPE_AP) },
> > +};
> > +
> >   static const struct ieee80211_iface_combination if_comb[] = {
> >   	{
> >   		.limits = if_limits,
> > @@ -746,6 +754,16 @@ static const struct ieee80211_iface_comb
> >   		.num_different_channels = 1,
> >   		.beacon_int_infra_match = true,
> >   		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
> > |
> > +					BIT(NL80211_CHAN_WIDTH_20) |
> > +					BIT(NL80211_CHAN_WIDTH_40),
> > +	},
> > +	{
> > +		.limits = if_dfs_limits_mvap,
> > +		.n_limits = ARRAY_SIZE(if_dfs_limits_mvap),
> > +		.max_interfaces = 8,
> > +		.num_different_channels = 1,
> > +		.beacon_int_infra_match = true,
> > +		.radar_detect_widths =	BIT(NL80211_CHAN_WIDTH_20_NOHT)
> > |
> >   					BIT(NL80211_CHAN_WIDTH_20) |
> >   					BIT(NL80211_CHAN_WIDTH_40),
> >   	}
> Hi
> This mail is in my list of "potentially interesting changes".
> Do you plan to send this as patch at some point?
> Or are you still testing something?
> 
> Best regards
> Matthias

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-09-16 10:27 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-09 16:26 [RFC v2] [resend] ath9k: allow up to 8 access point vifs on DFS channels Jean-Pierre TOSONI
2015-09-15  9:57 ` Matthias May
2015-09-16  8:01   ` Jean-Pierre Tosoni
2015-09-16 10:27     ` [ath9k-devel] " Jean-Pierre Tosoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.