* iw coalesce show - disabled at init?
@ 2015-08-12 18:36 Kobi Cohen-Arazi
2015-08-12 21:08 ` Johannes Berg
0 siblings, 1 reply; 5+ messages in thread
From: Kobi Cohen-Arazi @ 2015-08-12 18:36 UTC (permalink / raw)
To: linux-wireless
Hi,
When loading a wifi driver that supports set_coalesce and running "iw
show" command:
$ iw ... coalesce show
It would always show coalesce as disabled.
I think the reason for this is that there are no coalesce rules cached
in cfg/netlink layers yet. (empty set of rules).
Is my understanding correct? if yes, Is this by design?
Ideally, at startup, 80211 should query the driver for the current
coalesce rules and use it as the initial state.
Thank you,
- Kobi
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: iw coalesce show - disabled at init?
2015-08-12 18:36 iw coalesce show - disabled at init? Kobi Cohen-Arazi
@ 2015-08-12 21:08 ` Johannes Berg
2015-08-12 21:13 ` Kobi Cohen-Arazi
0 siblings, 1 reply; 5+ messages in thread
From: Johannes Berg @ 2015-08-12 21:08 UTC (permalink / raw)
To: Kobi Cohen-Arazi, linux-wireless
On Wed, 2015-08-12 at 11:36 -0700, Kobi Cohen-Arazi wrote:
> Hi,
>
> When loading a wifi driver that supports set_coalesce and running "iw
> show" command:
> $ iw ... coalesce show
> It would always show coalesce as disabled.
> I think the reason for this is that there are no coalesce rules
> cached
> in cfg/netlink layers yet. (empty set of rules).
>
> Is my understanding correct? if yes, Is this by design?
> Ideally, at startup, 80211 should query the driver for the current
> coalesce rules and use it as the initial state.
>
Typically devices/drivers shouldn't *have* any such rules at startup,
so the point is moot. If the driver/device you're using behaves
differently, arguably that's a bug. Marvell?
johannes
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: iw coalesce show - disabled at init?
2015-08-12 21:08 ` Johannes Berg
@ 2015-08-12 21:13 ` Kobi Cohen-Arazi
2015-08-12 21:28 ` Johannes Berg
0 siblings, 1 reply; 5+ messages in thread
From: Kobi Cohen-Arazi @ 2015-08-12 21:13 UTC (permalink / raw)
To: Johannes Berg; +Cc: linux-wireless
Hi Johannes,
On Wed, Aug 12, 2015 at 2:08 PM, Johannes Berg
<johannes@sipsolutions.net> wrote:
> On Wed, 2015-08-12 at 11:36 -0700, Kobi Cohen-Arazi wrote:
>> Hi,
>>
>> When loading a wifi driver that supports set_coalesce and running "iw
>> show" command:
>> $ iw ... coalesce show
>> It would always show coalesce as disabled.
>> I think the reason for this is that there are no coalesce rules
>> cached
>> in cfg/netlink layers yet. (empty set of rules).
>>
>> Is my understanding correct? if yes, Is this by design?
>> Ideally, at startup, 80211 should query the driver for the current
>> coalesce rules and use it as the initial state.
>>
>
> Typically devices/drivers shouldn't *have* any such rules at startup,
> so the point is moot. If the driver/device you're using behaves
> differently, arguably that's a bug. Marvell?
Nope :) not Marvell.
So what you're saying is that at startup, drivers should have coalesce
_disabled_ (i.e. no rules for coalesce) ?
- Kobi
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: iw coalesce show - disabled at init?
2015-08-12 21:13 ` Kobi Cohen-Arazi
@ 2015-08-12 21:28 ` Johannes Berg
2015-08-14 18:05 ` Kobi Cohen-Arazi
0 siblings, 1 reply; 5+ messages in thread
From: Johannes Berg @ 2015-08-12 21:28 UTC (permalink / raw)
To: Kobi Cohen-Arazi; +Cc: linux-wireless
On Wed, 2015-08-12 at 14:13 -0700, Kobi Cohen-Arazi wrote:
>
> So what you're saying is that at startup, drivers should have
> coalesce _disabled_ (i.e. no rules for coalesce) ?
>
I think so, yes. It's a service that should be actively enabled, with
user-configured rules. Mechanism vs. policy, and all that ...
johannes
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: iw coalesce show - disabled at init?
2015-08-12 21:28 ` Johannes Berg
@ 2015-08-14 18:05 ` Kobi Cohen-Arazi
0 siblings, 0 replies; 5+ messages in thread
From: Kobi Cohen-Arazi @ 2015-08-14 18:05 UTC (permalink / raw)
To: Johannes Berg; +Cc: linux-wireless
On Wed, Aug 12, 2015 at 2:28 PM, Johannes Berg
<johannes@sipsolutions.net> wrote:
> On Wed, 2015-08-12 at 14:13 -0700, Kobi Cohen-Arazi wrote:
>>
>> So what you're saying is that at startup, drivers should have
>> coalesce _disabled_ (i.e. no rules for coalesce) ?
>>
>
> I think so, yes. It's a service that should be actively enabled, with
> user-configured rules. Mechanism vs. policy, and all that ...
I think that in most (if not all) NICs, this specific service is
enabled by default.
Kobi
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-08-14 18:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-12 18:36 iw coalesce show - disabled at init? Kobi Cohen-Arazi
2015-08-12 21:08 ` Johannes Berg
2015-08-12 21:13 ` Kobi Cohen-Arazi
2015-08-12 21:28 ` Johannes Berg
2015-08-14 18:05 ` Kobi Cohen-Arazi
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).