[Bug 1875927] Re: add support for phys_port_name attribute in Xenial/16.04LTS

[Dan Streetman]

I think this would change the device name for existing devices, right?
If so, that doesn't seem appropriate for an SRU, as it could break
existing users that expect their device names to be consistent...

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to systemd in Ubuntu.
https://bugs.launchpad.net/bugs/1875927

Title:
  add support for phys_port_name attribute in Xenial/16.04LTS

Status in systemd package in Ubuntu:
  Fix Released
Status in systemd source package in Xenial:
  In Progress

Bug description:
  [Impact]
  In Xenial/16.04LTS, one can't generate network interface name from "phys_port_name" attribute.

  "phys_port_name" indicates the interface physical port name within the
  NIC.

  [Test Case]

  Check that udev (systemd-udevd) provides the phys_port_name property
  Tests should be done on both kernel versions: v4.4 and v4.15

  [Regression Potential]

  Risk: Low
  * This piece of code is already in place in Bionic (systemd) and late.
  AFAICT, nothing has been reported since then with regards to this feature.

  * phys_port_name kernel support has been introduced in v4.1. Xenial
  supported kernel are : v4.4 and v4.15 (HWE).

  * If a regression arise, it will most likely be limited to the
  "Ethernet switch device driver model (switchdev)" reported by: rocker,
  mlxsw, broadcom, ...

  [Other informations]
  https://github.com/systemd/systemd/commit/4887b656c22af059d4e833de7b56544f24951184
  https://github.com/systemd/systemd/pull/4506

  [Original Description]

  It has been brought to my attention that systemd in Xenial/16.04LTS
  doesn't have support for phys_port_name[0] attribute.

  The support has been first introduced in systemd version "232" via:
  https://github.com/systemd/systemd/commit/4887b656c22af059d4e833de7b56544f24951184
  https://github.com/systemd/systemd/pull/4506

  Bionic and late have the necessary bits ( systemd >232), but not
  Xenial (229)[1]

  Support for "phys_port_name" has been first introduced in the kernel
  with v4.1[2]

  [0]
  - https://www.kernel.org/doc/Documentation/ABI/testing/sysfs-class-net
  - https://www.freedesktop.org/software/systemd/man/systemd.net-naming-scheme.html
  - https://www.kernel.org/doc/Documentation/networking/switchdev.txt

  [1]
  # git systemd/systemd
  git describe --contains 4887b656c22af059d4e833de7b56544f24951184
  v232~15

  # rmadison
   => systemd | 229-4ubuntu21.27 | xenial-updates
   systemd | 237-3ubuntu10.39 | bionic-updates
   systemd | 240-6ubuntu5.8   | disco-updates
   systemd | 242-7ubuntu3.7   | eoan-updates
   systemd | 245.4-4ubuntu3   | focal
   systemd | 245.4-4ubuntu3   | groovy

  [2]
  https://github.com/torvalds/linux/commit/db24a9044ee1

  $ git describe --contains db24a9044ee1
  v4.1-rc1

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1875927/+subscriptions