[wpan-next,00/20] net: ieee802154: Support scanning/beaconing

Message

Miquel Raynal July 1, 2022, 2:30 p.m. UTC

Hello,

After a few exchanges about inter-PAN management with Alexander, it was
decided that most of this series would be dropped, because the kernel
should in the end not really care about keeping a local copy of the
discovered coordinators, this is userspace job.

So here is a "first" version of the scanning series which hopefully
meets the main requirements discussed the past days on the mailing
list. I know it is rather big, but there are a few very trivial patches
there, so here is how it is built:

* net: mac802154: Allow the creation of coordinator interfaces

  Beaconing must be reserved to coordinator interfaces, so we must
  support the creation of these interfaces in the mac layer.

* net: ieee802154: Advertize coordinators discovery
* net: ieee802154: Handle coordinators discovery
 
  Introduction of a user interface and then a cfg802154 interface for
  coordinators discovery and advertisement.

* net: ieee802154: Define frame types
* net: ieee802154: Add support for user scanning requests
* net: ieee802154: Define a beacon frame header
* net: mac802154: Prepare forcing specific symbol duration
* net: mac802154: Introduce a global device lock
* net: mac802154: Handle passive scanning

  User requests to scan and MAC handling of these requests.

* net: ieee802154: Add support for user beaconing requests
* net: mac802154: Handle basic beaconing

  User requests to send beacons and MAC handling of these requests.

* net: ieee802154: Add support for user active scan requests
* net: mac802154: Handle active scanning

  User requests to scan actively and MAC handling of these requests.

* net: ieee802154: Add support for allowing to answer BEACON_REQ
* net: mac802154: Handle received BEACON_REQ

  User requests to answer BEACON_RQ and MAC handling of these requests.

* net: ieee802154: Handle limited devices with only datagram support
* ieee802154: ca8210: Flag the driver as being limited

  This is a result of a previous review from Alexander, which pointed
  that the hardMAC ca8210 would not support the scanning operations, so
  it is flagged as limited.

* ieee802154: hwsim: Do not check the rtnl
* ieee802154: hwsim: Allow devices to be coordinators

  And finally these two patches are there to allow using hwsim to
  validate the series.

The corresponding userspace code will follow.

The series lacks support for forwarding association requests to the user
and waiting for feedback before accepting the device. This can be added
later on top of this work and is not necessary right now.

All of this is based on the initial work from David Girault and Romuald
Despres, they are often credited as Co-developpers.

Thanks,
Miquèl

David Girault (1):
  net: ieee802154: Trace the registration of new PANs

Miquel Raynal (19):
  net: mac802154: Allow the creation of coordinator interfaces
  net: ieee802154: Advertize coordinators discovery
  net: ieee802154: Handle coordinators discovery
  net: ieee802154: Define frame types
  net: ieee802154: Add support for user scanning requests
  net: ieee802154: Define a beacon frame header
  net: mac802154: Prepare forcing specific symbol duration
  net: mac802154: Introduce a global device lock
  net: mac802154: Handle passive scanning
  net: ieee802154: Add support for user beaconing requests
  net: mac802154: Handle basic beaconing
  net: ieee802154: Add support for user active scan requests
  net: mac802154: Handle active scanning
  net: ieee802154: Add support for allowing to answer BEACON_REQ
  net: mac802154: Handle received BEACON_REQ
  net: ieee802154: Handle limited devices with only datagram support
  ieee802154: ca8210: Flag the driver as being limited
  ieee802154: hwsim: Do not check the rtnl
  ieee802154: hwsim: Allow devices to be coordinators

 drivers/net/ieee802154/ca8210.c          |   3 +-
 drivers/net/ieee802154/mac802154_hwsim.c |   4 +-
 include/linux/ieee802154.h               |   7 +
 include/net/cfg802154.h                  |  97 ++++-
 include/net/ieee802154_netdev.h          |  89 +++++
 include/net/nl802154.h                   |  93 +++++
 net/ieee802154/Makefile                  |   2 +-
 net/ieee802154/core.c                    |   2 +
 net/ieee802154/header_ops.c              |  69 ++++
 net/ieee802154/nl802154.c                | 402 ++++++++++++++++++++
 net/ieee802154/nl802154.h                |   7 +
 net/ieee802154/pan.c                     | 114 ++++++
 net/ieee802154/rdev-ops.h                |  56 +++
 net/ieee802154/trace.h                   |  86 +++++
 net/mac802154/Makefile                   |   2 +-
 net/mac802154/cfg.c                      |  76 +++-
 net/mac802154/ieee802154_i.h             |  70 ++++
 net/mac802154/iface.c                    |  30 +-
 net/mac802154/main.c                     |  29 +-
 net/mac802154/rx.c                       | 113 +++++-
 net/mac802154/scan.c                     | 462 +++++++++++++++++++++++
 net/mac802154/tx.c                       |  12 +-
 22 files changed, 1793 insertions(+), 32 deletions(-)
 create mode 100644 net/ieee802154/pan.c
 create mode 100644 net/mac802154/scan.c

Comments

Alexander Aring July 4, 2022, 1:17 a.m. UTC | #1

Hi,

On Fri, Jul 1, 2022 at 10:36 AM Miquel Raynal <[e-mail]> wrote:
>
> Hello,
>
> After a few exchanges about inter-PAN management with Alexander, it was
> decided that most of this series would be dropped, because the kernel
> should in the end not really care about keeping a local copy of the
> discovered coordinators, this is userspace job.
>
> So here is a "first" version of the scanning series which hopefully
> meets the main requirements discussed the past days on the mailing
> list. I know it is rather big, but there are a few very trivial patches
> there, so here is how it is built:

I have probably more to say about this patch series, but I need more
days to look into it closely. So maybe we can first discuss here more
about things I comment on and have questions about it.

Thanks.

- Alex