* [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch
@ 2010-05-13 1:17 Bruno Randolf
2010-05-13 1:20 ` Luis R. Rodriguez
2010-05-13 11:48 ` Hauke Mehrtens
0 siblings, 2 replies; 5+ messages in thread
From: Bruno Randolf @ 2010-05-13 1:17 UTC (permalink / raw)
To: lrodriguez; +Cc: hauke, linux-wireless
i need this change to make it apply again
Signed-off-by: Bruno Randolf <br1@einfach.org>
---
patches/07-change-default-rate-alg.patch | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/patches/07-change-default-rate-alg.patch b/patches/07-change-default-rate-alg.patch
index f0ccbce..f73a4c5 100644
--- a/patches/07-change-default-rate-alg.patch
+++ b/patches/07-change-default-rate-alg.patch
@@ -10,9 +10,9 @@ and define CONFIG_COMPAT_MAC80211_RC_DEFAULT on config.mk.
Through the compat autoconf we then get it also defined there
at compilation time.
---- a/net/mac80211/rate.c
-+++ b/net/mac80211/rate.c
-@@ -23,7 +23,7 @@ struct rate_control_alg {
+--- a/net/mac80211/rate.c 2010-05-12 16:27:02.245747971 +0900
++++ b/net/mac80211/rate.c 2010-05-12 17:00:45.855748158 +0900
+@@ -23,7 +23,7 @@
static LIST_HEAD(rate_ctrl_algs);
static DEFINE_MUTEX(rate_ctrl_mutex);
@@ -21,7 +21,7 @@ at compilation time.
module_param(ieee80211_default_rc_algo, charp, 0644);
MODULE_PARM_DESC(ieee80211_default_rc_algo,
"Default rate control algorithm for mac80211 to use");
-@@ -119,8 +119,8 @@ ieee80211_rate_control_ops_get(const cha
+@@ -118,8 +118,8 @@
ops = ieee80211_try_rate_control_ops_get(ieee80211_default_rc_algo);
/* try built-in one if specific alg requested but not found */
@@ -29,6 +29,6 @@ at compilation time.
- ops = ieee80211_try_rate_control_ops_get(CONFIG_MAC80211_RC_DEFAULT);
+ if (!ops && strlen(CONFIG_COMPAT_MAC80211_RC_DEFAULT))
+ ops = ieee80211_try_rate_control_ops_get(CONFIG_COMPAT_MAC80211_RC_DEFAULT);
- kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
return ops;
+ }
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch
2010-05-13 1:17 [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch Bruno Randolf
@ 2010-05-13 1:20 ` Luis R. Rodriguez
2010-05-13 11:48 ` Hauke Mehrtens
1 sibling, 0 replies; 5+ messages in thread
From: Luis R. Rodriguez @ 2010-05-13 1:20 UTC (permalink / raw)
To: Bruno Randolf; +Cc: hauke, linux-wireless
On Wed, May 12, 2010 at 6:17 PM, Bruno Randolf <br1@einfach.org> wrote:
> i need this change to make it apply again
>
> Signed-off-by: Bruno Randolf <br1@einfach.org>
Applied and pushed out, thanks!
Luis
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch
2010-05-13 1:17 [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch Bruno Randolf
2010-05-13 1:20 ` Luis R. Rodriguez
@ 2010-05-13 11:48 ` Hauke Mehrtens
2010-05-14 1:19 ` Bruno Randolf
1 sibling, 1 reply; 5+ messages in thread
From: Hauke Mehrtens @ 2010-05-13 11:48 UTC (permalink / raw)
To: Bruno Randolf; +Cc: lrodriguez, linux-wireless
Am 13.05.2010 03:17, schrieb Bruno Randolf:
> i need this change to make it apply again
>
> Signed-off-by: Bruno Randolf <br1@einfach.org>
> ---
> patches/07-change-default-rate-alg.patch | 10 +++++-----
> 1 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/patches/07-change-default-rate-alg.patch b/patches/07-change-default-rate-alg.patch
> index f0ccbce..f73a4c5 100644
> --- a/patches/07-change-default-rate-alg.patch
> +++ b/patches/07-change-default-rate-alg.patch
> @@ -10,9 +10,9 @@ and define CONFIG_COMPAT_MAC80211_RC_DEFAULT on config.mk.
> Through the compat autoconf we then get it also defined there
> at compilation time.
>
> ---- a/net/mac80211/rate.c
> -+++ b/net/mac80211/rate.c
> -@@ -23,7 +23,7 @@ struct rate_control_alg {
> +--- a/net/mac80211/rate.c 2010-05-12 16:27:02.245747971 +0900
> ++++ b/net/mac80211/rate.c 2010-05-12 17:00:45.855748158 +0900
> +@@ -23,7 +23,7 @@
> static LIST_HEAD(rate_ctrl_algs);
> static DEFINE_MUTEX(rate_ctrl_mutex);
>
> @@ -21,7 +21,7 @@ at compilation time.
> module_param(ieee80211_default_rc_algo, charp, 0644);
> MODULE_PARM_DESC(ieee80211_default_rc_algo,
> "Default rate control algorithm for mac80211 to use");
> -@@ -119,8 +119,8 @@ ieee80211_rate_control_ops_get(const cha
> +@@ -118,8 +118,8 @@
> ops = ieee80211_try_rate_control_ops_get(ieee80211_default_rc_algo);
>
> /* try built-in one if specific alg requested but not found */
> @@ -29,6 +29,6 @@ at compilation time.
> - ops = ieee80211_try_rate_control_ops_get(CONFIG_MAC80211_RC_DEFAULT);
> + if (!ops && strlen(CONFIG_COMPAT_MAC80211_RC_DEFAULT))
> + ops = ieee80211_try_rate_control_ops_get(CONFIG_COMPAT_MAC80211_RC_DEFAULT);
> - kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
>
> return ops;
> + }
>
Hi Bruno,
For me it does not apply any more against linux-next with your patch,
without your patch it still applies.
In the linux-next tree there is this line
kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
it was introduced 2 days ago. In wireless-text this does not exist for now.
Could it be that you tried to generate compat-wireless out of
wireless-testing and not linux-next?
Please run "./scripts/admin-refresh.sh refresh" after changing a patch
to format it automatically in a common way.
Hauke
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch
2010-05-13 11:48 ` Hauke Mehrtens
@ 2010-05-14 1:19 ` Bruno Randolf
2010-05-14 1:23 ` Luis R. Rodriguez
0 siblings, 1 reply; 5+ messages in thread
From: Bruno Randolf @ 2010-05-14 1:19 UTC (permalink / raw)
To: Hauke Mehrtens; +Cc: lrodriguez, linux-wireless
On Thursday 13 May 2010 20:48:30 Hauke Mehrtens wrote:
> Am 13.05.2010 03:17, schrieb Bruno Randolf:
> > i need this change to make it apply again
> >
> > Signed-off-by: Bruno Randolf <br1@einfach.org>
> > ---
> >
> > patches/07-change-default-rate-alg.patch | 10 +++++-----
> > 1 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/patches/07-change-default-rate-alg.patch
> > b/patches/07-change-default-rate-alg.patch index f0ccbce..f73a4c5 100644
> > --- a/patches/07-change-default-rate-alg.patch
> > +++ b/patches/07-change-default-rate-alg.patch
> > @@ -10,9 +10,9 @@ and define CONFIG_COMPAT_MAC80211_RC_DEFAULT on
> > config.mk.
> >
> > Through the compat autoconf we then get it also defined there
> > at compilation time.
> >
> > ---- a/net/mac80211/rate.c
> > -+++ b/net/mac80211/rate.c
> > -@@ -23,7 +23,7 @@ struct rate_control_alg {
> > +--- a/net/mac80211/rate.c 2010-05-12 16:27:02.245747971 +0900
> > ++++ b/net/mac80211/rate.c 2010-05-12 17:00:45.855748158 +0900
> > +@@ -23,7 +23,7 @@
> >
> > static LIST_HEAD(rate_ctrl_algs);
> > static DEFINE_MUTEX(rate_ctrl_mutex);
> >
> > @@ -21,7 +21,7 @@ at compilation time.
> >
> > module_param(ieee80211_default_rc_algo, charp, 0644);
> > MODULE_PARM_DESC(ieee80211_default_rc_algo,
> >
> > "Default rate control algorithm for mac80211 to use");
> >
> > -@@ -119,8 +119,8 @@ ieee80211_rate_control_ops_get(const cha
> > +@@ -118,8 +118,8 @@
> >
> > ops =
ieee80211_try_rate_control_ops_get(ieee80211_default_rc_algo);
> >
> > /* try built-in one if specific alg requested but not found */
> >
> > @@ -29,6 +29,6 @@ at compilation time.
> >
> > - ops =
ieee80211_try_rate_control_ops_get(CONFIG_MAC80211_RC_DEFAULT);
> > + if (!ops && strlen(CONFIG_COMPAT_MAC80211_RC_DEFAULT))
> > + ops =
> > ieee80211_try_rate_control_ops_get(CONFIG_COMPAT_MAC80211_RC_DEFAULT);
> >
> > - kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
> >
> > return ops;
> >
> > + }
>
> Hi Bruno,
>
> For me it does not apply any more against linux-next with your patch,
> without your patch it still applies.
>
> In the linux-next tree there is this line
> kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
> it was introduced 2 days ago. In wireless-text this does not exist for now.
>
> Could it be that you tried to generate compat-wireless out of
> wireless-testing and not linux-next?
>
> Please run "./scripts/admin-refresh.sh refresh" after changing a patch
> to format it automatically in a common way.
yes, i used wireless-testing - sorry!
is compat-wireless meant to be used only with linux-next?
bruno
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch
2010-05-14 1:19 ` Bruno Randolf
@ 2010-05-14 1:23 ` Luis R. Rodriguez
0 siblings, 0 replies; 5+ messages in thread
From: Luis R. Rodriguez @ 2010-05-14 1:23 UTC (permalink / raw)
To: Bruno Randolf
Cc: Hauke Mehrtens, Luis Rodriguez, linux-wireless@vger.kernel.org
On Thu, May 13, 2010 at 06:19:38PM -0700, Bruno Randolf wrote:
> On Thursday 13 May 2010 20:48:30 Hauke Mehrtens wrote:
> > Am 13.05.2010 03:17, schrieb Bruno Randolf:
> > > i need this change to make it apply again
> > >
> > > Signed-off-by: Bruno Randolf <br1@einfach.org>
> > > ---
> > >
> > > patches/07-change-default-rate-alg.patch | 10 +++++-----
> > > 1 files changed, 5 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/patches/07-change-default-rate-alg.patch
> > > b/patches/07-change-default-rate-alg.patch index f0ccbce..f73a4c5 100644
> > > --- a/patches/07-change-default-rate-alg.patch
> > > +++ b/patches/07-change-default-rate-alg.patch
> > > @@ -10,9 +10,9 @@ and define CONFIG_COMPAT_MAC80211_RC_DEFAULT on
> > > config.mk.
> > >
> > > Through the compat autoconf we then get it also defined there
> > > at compilation time.
> > >
> > > ---- a/net/mac80211/rate.c
> > > -+++ b/net/mac80211/rate.c
> > > -@@ -23,7 +23,7 @@ struct rate_control_alg {
> > > +--- a/net/mac80211/rate.c 2010-05-12 16:27:02.245747971 +0900
> > > ++++ b/net/mac80211/rate.c 2010-05-12 17:00:45.855748158 +0900
> > > +@@ -23,7 +23,7 @@
> > >
> > > static LIST_HEAD(rate_ctrl_algs);
> > > static DEFINE_MUTEX(rate_ctrl_mutex);
> > >
> > > @@ -21,7 +21,7 @@ at compilation time.
> > >
> > > module_param(ieee80211_default_rc_algo, charp, 0644);
> > > MODULE_PARM_DESC(ieee80211_default_rc_algo,
> > >
> > > "Default rate control algorithm for mac80211 to use");
> > >
> > > -@@ -119,8 +119,8 @@ ieee80211_rate_control_ops_get(const cha
> > > +@@ -118,8 +118,8 @@
> > >
> > > ops =
> ieee80211_try_rate_control_ops_get(ieee80211_default_rc_algo);
> > >
> > > /* try built-in one if specific alg requested but not found */
> > >
> > > @@ -29,6 +29,6 @@ at compilation time.
> > >
> > > - ops =
> ieee80211_try_rate_control_ops_get(CONFIG_MAC80211_RC_DEFAULT);
> > > + if (!ops && strlen(CONFIG_COMPAT_MAC80211_RC_DEFAULT))
> > > + ops =
> > > ieee80211_try_rate_control_ops_get(CONFIG_COMPAT_MAC80211_RC_DEFAULT);
> > >
> > > - kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
> > >
> > > return ops;
> > >
> > > + }
> >
> > Hi Bruno,
> >
> > For me it does not apply any more against linux-next with your patch,
> > without your patch it still applies.
> >
> > In the linux-next tree there is this line
> > kparam_unblock_sysfs_write(ieee80211_default_rc_algo);
> > it was introduced 2 days ago. In wireless-text this does not exist for now.
> >
> > Could it be that you tried to generate compat-wireless out of
> > wireless-testing and not linux-next?
> >
> > Please run "./scripts/admin-refresh.sh refresh" after changing a patch
> > to format it automatically in a common way.
>
> yes, i used wireless-testing - sorry!
>
> is compat-wireless meant to be used only with linux-next?
Yeap
Luis
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-05-14 1:23 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-13 1:17 [PATCH] compat-wireless: fix 07-change-default-rate-alg.patch Bruno Randolf
2010-05-13 1:20 ` Luis R. Rodriguez
2010-05-13 11:48 ` Hauke Mehrtens
2010-05-14 1:19 ` Bruno Randolf
2010-05-14 1:23 ` Luis R. Rodriguez
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox