lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20221121135555.1227271-1-vladimir.oltean@nxp.com>
Date:   Mon, 21 Nov 2022 15:55:38 +0200
From:   Vladimir Oltean <vladimir.oltean@....com>
To:     netdev@...r.kernel.org
Cc:     Andrew Lunn <andrew@...n.ch>,
        Florian Fainelli <f.fainelli@...il.com>,
        Vladimir Oltean <olteanv@...il.com>,
        "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>
Subject: [PATCH net-next 00/17] Remove dsa_priv.h

After working on the "Autoload DSA tagging driver when dynamically
changing protocol" series:
https://patchwork.kernel.org/project/netdevbpf/cover/20221115011847.2843127-1-vladimir.oltean@nxp.com/

it became clear to me that the situation with DSA headers is a bit
messy, and I put the tagging protocol driver macros in a pretty random
temporary spot in dsa_priv.h.

Now is the time to make the net/dsa/ folder a bit more organized, and to
make tagging protocol driver modules include just headers they're going
to use.

Another thing is the merging and cleanup of dsa.c and dsa2.c. Before,
dsa.c had 589 lines and dsa2.c had 1817 lines. Now, the combined dsa.c
has 1749 lines, the rest went to some other places.

Sorry for the set size, I know the rules, but since this is basically
code movement for the most part, I thought more patches are better.

Vladimir Oltean (17):
  net: dsa: unexport dsa_dev_to_net_device()
  net: dsa: modularize DSA_TAG_PROTO_NONE
  net: dsa: move bulk of devlink code to devlink.{c,h}
  net: dsa: if ds->setup is true, ds->devlink is always non-NULL
  net: dsa: move rest of devlink setup/teardown to devlink.c
  net: dsa: move headers exported by port.c to port.h
  net: dsa: move headers exported by master.c to master.h
  net: dsa: move headers exported by slave.c to slave.h
  net: dsa: move tagging protocol code to tag.{c,h}
  net: dsa: move headers exported by switch.c to switch.h
  net: dsa: move dsa_tree_notify() and dsa_broadcast() to switch.c
  net: dsa: move notifier definitions to switch.h
  net: dsa: merge dsa.c into dsa2.c
  net: dsa: rename dsa2.c back into dsa.c and create its header
  net: dsa: move definitions from dsa_priv.h to slave.c
  net: dsa: move tag_8021q headers to their proper place
  net: dsa: kill off dsa_priv.h

 drivers/net/dsa/Kconfig           |    2 +
 drivers/net/dsa/b53/Kconfig       |    1 +
 drivers/net/dsa/microchip/Kconfig |    1 +
 include/linux/dsa/8021q.h         |   31 +-
 include/net/dsa.h                 |    3 +-
 net/dsa/Kconfig                   |    6 +
 net/dsa/Makefile                  |    4 +-
 net/dsa/devlink.c                 |  391 +++++++
 net/dsa/devlink.h                 |   16 +
 net/dsa/dsa.c                     | 1754 +++++++++++++++++++++++-----
 net/dsa/dsa.h                     |   40 +
 net/dsa/dsa2.c                    | 1817 -----------------------------
 net/dsa/dsa_priv.h                |  663 -----------
 net/dsa/master.c                  |   10 +-
 net/dsa/master.h                  |   19 +
 net/dsa/netlink.c                 |    3 +-
 net/dsa/netlink.h                 |    8 +
 net/dsa/port.c                    |    6 +-
 net/dsa/port.h                    |  114 ++
 net/dsa/slave.c                   |   49 +-
 net/dsa/slave.h                   |   69 ++
 net/dsa/switch.c                  |   53 +-
 net/dsa/switch.h                  |  120 ++
 net/dsa/tag.c                     |  243 ++++
 net/dsa/tag.h                     |  310 +++++
 net/dsa/tag_8021q.c               |   19 +-
 net/dsa/tag_8021q.h               |   27 +
 net/dsa/tag_ar9331.c              |    2 +-
 net/dsa/tag_brcm.c                |    2 +-
 net/dsa/tag_dsa.c                 |    2 +-
 net/dsa/tag_gswip.c               |    2 +-
 net/dsa/tag_hellcreek.c           |    2 +-
 net/dsa/tag_ksz.c                 |    3 +-
 net/dsa/tag_lan9303.c             |    2 +-
 net/dsa/tag_mtk.c                 |    2 +-
 net/dsa/tag_none.c                |   30 +
 net/dsa/tag_ocelot.c              |    3 +-
 net/dsa/tag_ocelot_8021q.c        |    4 +-
 net/dsa/tag_qca.c                 |    2 +-
 net/dsa/tag_rtl4_a.c              |    2 +-
 net/dsa/tag_rtl8_4.c              |    2 +-
 net/dsa/tag_rzn1_a5psw.c          |    2 +-
 net/dsa/tag_sja1105.c             |    4 +-
 net/dsa/tag_trailer.c             |    2 +-
 net/dsa/tag_xrs700x.c             |    2 +-
 45 files changed, 3016 insertions(+), 2833 deletions(-)
 create mode 100644 net/dsa/devlink.c
 create mode 100644 net/dsa/devlink.h
 create mode 100644 net/dsa/dsa.h
 delete mode 100644 net/dsa/dsa2.c
 delete mode 100644 net/dsa/dsa_priv.h
 create mode 100644 net/dsa/master.h
 create mode 100644 net/dsa/netlink.h
 create mode 100644 net/dsa/port.h
 create mode 100644 net/dsa/slave.h
 create mode 100644 net/dsa/switch.h
 create mode 100644 net/dsa/tag.c
 create mode 100644 net/dsa/tag.h
 create mode 100644 net/dsa/tag_8021q.h
 create mode 100644 net/dsa/tag_none.c

-- 
2.34.1

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ