New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 923524 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Task



Sign in to add a comment

shill: remove Mac80211Monitor

Project Member Reported by briannorris@chromium.org, Jan 18 (4 days ago)

Issue description

shill's Mac80211Monitor was designed for ath9k, to work around some driver/firmware bugs:

https://chromium-review.googlesource.com/211706
https://bugs.chromium.org/p/chromium/issues/detail?id=392209

It relies on some out-of-tree patches to the wireless-3.4 / mac80211 stack, from here:

https://chromium-review.googlesource.com/211712

This is basically only valid on a handful of old platforms using Atheros chipsets (and ath9k).

This is a placeholder bug, to remind us to kill this module once these devices are EOL'd and this feature is no longer useful.
 

Comment 1 by briannorris@chromium.org, Jan 18 (4 days ago)

Description: Show this description
Project Member

Comment 2 by bugdroid, Today (68 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/d1c256b24f82877e251ad7bdc744dfbd75e6dab5

commit d1c256b24f82877e251ad7bdc744dfbd75e6dab5
Author: Brian Norris <briannorris@chromium.org>
Date: Wed Jan 23 05:05:13 2019

shill: mac80211_monitor: add TODO for removal

BUG=chromium:923524
TEST=none

Change-Id: Id6bb58596549fdd83d41184c150c2c40907ddd1c
Reviewed-on: https://chromium-review.googlesource.com/1422565
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>

[modify] https://crrev.com/d1c256b24f82877e251ad7bdc744dfbd75e6dab5/shill/wifi/mac80211_monitor.cc

Project Member

Comment 3 by bugdroid, Today (47 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/d1c256b24f82877e251ad7bdc744dfbd75e6dab5

commit d1c256b24f82877e251ad7bdc744dfbd75e6dab5
Author: Brian Norris <briannorris@chromium.org>
Date: Wed Jan 23 05:05:13 2019

shill: mac80211_monitor: add TODO for removal

BUG=chromium:923524
TEST=none

Change-Id: Id6bb58596549fdd83d41184c150c2c40907ddd1c
Reviewed-on: https://chromium-review.googlesource.com/1422565
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>

[modify] https://crrev.com/d1c256b24f82877e251ad7bdc744dfbd75e6dab5/shill/wifi/mac80211_monitor.cc

Project Member

Comment 4 by bugdroid, Today (25 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/d1c256b24f82877e251ad7bdc744dfbd75e6dab5

commit d1c256b24f82877e251ad7bdc744dfbd75e6dab5
Author: Brian Norris <briannorris@chromium.org>
Date: Wed Jan 23 05:05:13 2019

shill: mac80211_monitor: add TODO for removal

BUG=chromium:923524
TEST=none

Change-Id: Id6bb58596549fdd83d41184c150c2c40907ddd1c
Reviewed-on: https://chromium-review.googlesource.com/1422565
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>

[modify] https://crrev.com/d1c256b24f82877e251ad7bdc744dfbd75e6dab5/shill/wifi/mac80211_monitor.cc

Sign in to add a comment