* [wl-testing/master] build regression in brcmfmac
@ 2018-05-09 21:12 Arend van Spriel
2018-05-09 21:22 ` Johannes Berg
0 siblings, 1 reply; 6+ messages in thread
From: Arend van Spriel @ 2018-05-09 21:12 UTC (permalink / raw)
To: Johannes Berg; +Cc: linux-wireless
Since yesterday I get the following build error on wl-testing/master:
CC [M] drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.o
drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c: In function
‘brcmf_notify_connect_status_ap’:
drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:5530:1:
error: the frame size of 1592 bytes is larger than 1024 bytes
[-Werror=frame-larger-than=]
}
^
In the function brcmf_notify_connect_status_ap() I do have following on
the stack:
static s32
brcmf_notify_connect_status_ap(struct brcmf_cfg80211_info *cfg,
struct net_device *ndev,
const struct brcmf_event_msg *e, void *data)
{
static int generation;
u32 event = e->event_code;
u32 reason = e->reason;
struct station_info sinfo;
I guess the struct station_info grew bigger by these commits:
52539ca cfg80211: Expose TXQ stats and parameters to userspace +816
81d5439 cfg80211: average ack rssi support for data frames +1
There are a number of other (cfg80211-based) drivers doing the same
thing, ie. mwifiex, ath6kl, qtnfmac.
Regards,
Arend
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [wl-testing/master] build regression in brcmfmac
2018-05-09 21:12 [wl-testing/master] build regression in brcmfmac Arend van Spriel
@ 2018-05-09 21:22 ` Johannes Berg
2018-05-09 21:27 ` Toke Høiland-Jørgensen
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2018-05-09 21:22 UTC (permalink / raw)
To: Arend van Spriel; +Cc: linux-wireless
On Wed, 2018-05-09 at 23:12 +0200, Arend van Spriel wrote:
> Since yesterday I get the following build error on wl-testing/master:
>
> CC [M] drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.o
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c: In function
> ‘brcmf_notify_connect_status_ap’:
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:5530:1:
> error: the frame size of 1592 bytes is larger than 1024 bytes
> [-Werror=frame-larger-than=]
> }
> ^
>
> In the function brcmf_notify_connect_status_ap() I do have following on
> the stack:
>
> static s32
> brcmf_notify_connect_status_ap(struct brcmf_cfg80211_info *cfg,
> struct net_device *ndev,
> const struct brcmf_event_msg *e, void *data)
> {
> static int generation;
> u32 event = e->event_code;
> u32 reason = e->reason;
> struct station_info sinfo;
>
> I guess the struct station_info grew bigger by these commits:
>
> 52539ca cfg80211: Expose TXQ stats and parameters to userspace +816
> 81d5439 cfg80211: average ack rssi support for data frames +1
>
> There are a number of other (cfg80211-based) drivers doing the same
> thing, ie. mwifiex, ath6kl, qtnfmac.
Yeah. Toke said this morning he was working on fixes.
johannes
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [wl-testing/master] build regression in brcmfmac
2018-05-09 21:22 ` Johannes Berg
@ 2018-05-09 21:27 ` Toke Høiland-Jørgensen
2018-05-09 21:30 ` Johannes Berg
0 siblings, 1 reply; 6+ messages in thread
From: Toke Høiland-Jørgensen @ 2018-05-09 21:27 UTC (permalink / raw)
To: Johannes Berg, Arend van Spriel; +Cc: linux-wireless
Johannes Berg <johannes@sipsolutions.net> writes:
> On Wed, 2018-05-09 at 23:12 +0200, Arend van Spriel wrote:
>> Since yesterday I get the following build error on wl-testing/master:
>>=20
>> CC [M] drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.o
>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c: In function=
=20
>> =E2=80=98brcmf_notify_connect_status_ap=E2=80=99:
>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:5530:1:=20
>> error: the frame size of 1592 bytes is larger than 1024 bytes=20
>> [-Werror=3Dframe-larger-than=3D]
>> }
>> ^
>>=20
>> In the function brcmf_notify_connect_status_ap() I do have following on=
=20
>> the stack:
>>=20
>> static s32
>> brcmf_notify_connect_status_ap(struct brcmf_cfg80211_info *cfg,
>> struct net_device *ndev,
>> const struct brcmf_event_msg *e, void *d=
ata)
>> {
>> static int generation;
>> u32 event =3D e->event_code;
>> u32 reason =3D e->reason;
>> struct station_info sinfo;
>>=20
>> I guess the struct station_info grew bigger by these commits:
>>=20
>> 52539ca cfg80211: Expose TXQ stats and parameters to userspace +816
>> 81d5439 cfg80211: average ack rssi support for data frames +1
>>=20
>> There are a number of other (cfg80211-based) drivers doing the same=20
>> thing, ie. mwifiex, ath6kl, qtnfmac.
>
> Yeah. Toke said this morning he was working on fixes.
That would be these:
https://patchwork.kernel.org/patch/10389333/
https://patchwork.kernel.org/patch/10389331/
https://patchwork.kernel.org/patch/10389335/
-Toke
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [wl-testing/master] build regression in brcmfmac
2018-05-09 21:27 ` Toke Høiland-Jørgensen
@ 2018-05-09 21:30 ` Johannes Berg
2018-05-09 21:36 ` Toke Høiland-Jørgensen
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2018-05-09 21:30 UTC (permalink / raw)
To: Toke Høiland-Jørgensen, Arend van Spriel; +Cc: linux-wireless
On Wed, 2018-05-09 at 23:27 +0200, Toke Høiland-Jørgensen wrote:
> That would be these:
>
> https://patchwork.kernel.org/patch/10389333/
> https://patchwork.kernel.org/patch/10389331/
> https://patchwork.kernel.org/patch/10389335/
>
D'oh, how did I miss that?!
Your fixes line is slightly wrong, it should be
Fixes: 12-digitid ("subject")
but I'm happy to fix that when I apply it.
johannes
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [wl-testing/master] build regression in brcmfmac
2018-05-09 21:30 ` Johannes Berg
@ 2018-05-09 21:36 ` Toke Høiland-Jørgensen
2018-05-09 21:37 ` Johannes Berg
0 siblings, 1 reply; 6+ messages in thread
From: Toke Høiland-Jørgensen @ 2018-05-09 21:36 UTC (permalink / raw)
To: Johannes Berg, Arend van Spriel; +Cc: linux-wireless
Johannes Berg <johannes@sipsolutions.net> writes:
> On Wed, 2018-05-09 at 23:27 +0200, Toke H=C3=B8iland-J=C3=B8rgensen wrote:
>
>> That would be these:
>>=20
>> https://patchwork.kernel.org/patch/10389333/
>> https://patchwork.kernel.org/patch/10389331/
>> https://patchwork.kernel.org/patch/10389335/
>>=20
>
> D'oh, how did I miss that?!
>
> Your fixes line is slightly wrong, it should be
>
> Fixes: 12-digitid ("subject")
You mean it's missing the parenthesis?
-Toke
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [wl-testing/master] build regression in brcmfmac
2018-05-09 21:36 ` Toke Høiland-Jørgensen
@ 2018-05-09 21:37 ` Johannes Berg
0 siblings, 0 replies; 6+ messages in thread
From: Johannes Berg @ 2018-05-09 21:37 UTC (permalink / raw)
To: Toke Høiland-Jørgensen, Arend van Spriel; +Cc: linux-wireless
On Wed, 2018-05-09 at 23:36 +0200, Toke Høiland-Jørgensen wrote:
> Johannes Berg <johannes@sipsolutions.net> writes:
>
> > On Wed, 2018-05-09 at 23:27 +0200, Toke Høiland-Jørgensen wrote:
> >
> > > That would be these:
> > >
> > > https://patchwork.kernel.org/patch/10389333/
> > > https://patchwork.kernel.org/patch/10389331/
> > > https://patchwork.kernel.org/patch/10389335/
> > >
> >
> > D'oh, how did I miss that?!
> >
> > Your fixes line is slightly wrong, it should be
> >
> > Fixes: 12-digitid ("subject")
>
> You mean it's missing the parenthesis?
Right, and the quotes.
johannes
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-05-09 21:37 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-09 21:12 [wl-testing/master] build regression in brcmfmac Arend van Spriel
2018-05-09 21:22 ` Johannes Berg
2018-05-09 21:27 ` Toke Høiland-Jørgensen
2018-05-09 21:30 ` Johannes Berg
2018-05-09 21:36 ` Toke Høiland-Jørgensen
2018-05-09 21:37 ` Johannes Berg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).