Dell dock MAC Address pass through doesn't work in Ubuntu

Bug #1579984 reported by Mario Limonciello
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Wishlist
Phidias
Xenial
Fix Released
Undecided
Phidias

Bug Description

Dell systems with Type-C ports support a feature called "MAC address" pass through.
This feature allows systems to set a persistent system MAC address to be used with the TB15, WD15 and Type-C ethernet adapter.

More information is available here:
http://www.dell.com/support/article/us/en/19/SLN301147/en

This function works by a system specific MAC address (referred to as an auxiliary mac address) being programmed in the factory. When the driver for the type-c ethernet (in dock or dongle) is loaded the MAC address is supposed to be overridden with this value.

To enable this in Linux a kernel patch is needed.
This has been submitted http://patchwork.ozlabs.org/patch/631759/

Revision history for this message
Brad Figg (brad-figg) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:

apport-collect 1579984

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Changed in linux (Ubuntu):
status: Incomplete → Confirmed
Revision history for this message
Sebastien Bacher (seb128) wrote :

thanks Mario, do you also plan to open upstream bugs for the userland changes required?

Changed in network-manager (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Mario Limonciello (superm1) wrote :

I've adjusted the description for the alternate approach in doing this entirely in the kernel. I've submitted a patch to LKML for this here.

https://patchwork.kernel.org/patch/9148319/

no longer affects: network-manager (Ubuntu)
no longer affects: systemd (Ubuntu)
description: updated
penalvch (penalvch)
Changed in linux (Ubuntu):
importance: Undecided → Wishlist
status: Confirmed → Triaged
description: updated
Revision history for this message
Mario Limonciello (superm1) wrote :

Upstream hasn't flat out rejected this patch, but they also haven't been openly accepting the patch.

As this is for enabling docks to properly work (as intended by OEM) in Ubuntu can this be carried as Ubuntu SAUCE? The latest version of the patch and discussion about it is available here:
http://patchwork.ozlabs.org/patch/631759/

It will only affect docks that have a special realtek chip with a special bit set indicating that they are docks and should take the firmware mac address.

Thanks,

Revision history for this message
Mario Limonciello (superm1) wrote :

David Miller applied this. Could it please be pulled into Ubuntu as sauce now?

http://patchwork.ozlabs.org/patch/647157/

penalvch (penalvch)
tags: added: cherry-pick
Changed in linux (Ubuntu):
assignee: nobody → Phidias (phidias-chiang)
Phidias (phidias-chiang)
Changed in linux (Ubuntu):
status: Triaged → In Progress
Revision history for this message
Mario Limonciello (superm1) wrote :
Revision history for this message
Tim Gardner (timg-tpi) wrote :
Changed in linux (Ubuntu Xenial):
assignee: nobody → Phidias (phidias-chiang)
status: New → In Progress
Revision history for this message
Tim Gardner (timg-tpi) wrote :

Merged for 4.8

Changed in linux (Ubuntu):
status: In Progress → Fix Released
Changed in linux (Ubuntu Xenial):
status: In Progress → Fix Committed
Revision history for this message
Stefan Bader (smb) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-xenial' to 'verification-done-xenial'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-xenial
Revision history for this message
Mario Limonciello (superm1) wrote :

Looks good to me:

$ dmesg | grep "Linux version\|r8152"
[ 0.000000] Linux version 4.4.0-36-generic (buildd@lcy01-01) (gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.2) ) #55-Ubuntu SMP Thu Aug 11 18:01:55 UTC 2016 (Ubuntu 4.4.0-36.55-generic 4.4.16)
[ 5.702293] usbcore: registered new interface driver r8152
[ 6.523384] r8152 4-1.2:1.0 (unnamed net_device) (uninitialized): Using pass-thru MAC addr 20:47:47:f5:6b:17
[ 6.523819] r8152 4-1.2:1.0 eth0: v1.08.2
[ 7.355705] r8152 4-1.2:1.0 enx204747f56b17: renamed from eth0

tags: added: verification-done-xenial
removed: verification-needed-xenial
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (13.4 KiB)

This bug was fixed in the package linux - 4.4.0-36.55

---------------
linux (4.4.0-36.55) xenial; urgency=low

  [ Stefan Bader ]

  * Release Tracking Bug
    - LP: #1612305

  * I2C touchpad does not work on AMD platform (LP: #1612006)
    - SAUCE: pinctrl/amd: Remove the default de-bounce time

  * CVE-2016-5696
    - tcp: make challenge acks less predictable

linux (4.4.0-35.54) xenial; urgency=low

  [ Stefan Bader ]

  * Release Tracking Bug
    - LP: #1611215

  * [i915_bpo] Sync with v4.7 (LP: #1609742)
    - SAUCE: i915_bpo: Sync with v4.7

  * s390/cio: fix reset of channel measurement block (LP: #1609415)
    - s390/cio: allow to reset channel measurement block

  * in Ubuntu16.10: Hit on Call traces and system goes down when transactional
    memory tests are running in 32TB Brazos system (LP: #1606786)
    - powerpc/tm: Avoid SLB faults in treclaim/trecheckpoint when RI=0
    - powerpc/tm: Fix stack pointer corruption in __tm_recheckpoint()

  * Power Menu does not display after press the Power Button (LP: #1609204)
    - intel-vbtn: new driver for Intel Virtual Button
    - [config] enable CONFIG_INTEL_VBTN=m

  * OptiPlex 7450 AIO hangs when rebooting (LP: #1608762)
    - x86/reboot: Add Dell Optiplex 7450 AIO reboot quirk

  * virtualbox+usb 3.0 breaks boot, -28 kernel works (LP: #1604058)
    - SAUCE: xhci: Fix soft lockup in xhci_pci_probe path when XHCI_STATE_HALTED

  * linux-kernel: Freeing IRQ from IRQ context (LP: #1597908)
    - block: defer timeouts to a workqueue

  * Tunnel offload indications not stripped from encapsulated packets, causing
    performance overhead (LP: #1602755)
    - tunnels: Remove encapsulation offloads on decap.

  * lm-sensors is throwing "ERROR: Can't get value of subfeature temp1_input:
    I/O error" for be2net driver (LP: #1607387)
    - be2net: perform temperature query in adapter regardless of its interface
      state

  * Dell dock MAC Address pass through doesn't work in Ubuntu (LP: #1579984)
    - r8152: Add support for setting pass through MAC address on RTL8153-AD

  * vmxnet3 LRO IPv6 performance issues (stalling TCP) (LP: #1605494)
    - Driver: Vmxnet3: set CHECKSUM_UNNECESSARY for IPv6 packets

  * ISST-LTE:pVM:monklp5:Ubuntu16.04.1:system crashed at
    lpfc_sli4_scmd_to_wqidx_distr (LP: #1597974)
    - SAUCE: lpfc: fix oops in lpfc_sli4_scmd_to_wqidx_distr() from
      lpfc_send_taskmgmt()

  * Backport cxlflash shutdown patch to Xenial SRU (LP: #1605405)
    - SAUCE: cxlflash: Verify problem state area is mapped before notifying
      shutdown

  * Xenial update to v4.4.16 stable release (LP: #1607404)
    - mac80211: fix fast_tx header alignment
    - mac80211: mesh: flush mesh paths unconditionally
    - mac80211_hwsim: Add missing check for HWSIM_ATTR_SIGNAL
    - mac80211: Fix mesh estab_plinks counting in STA removal case
    - EDAC, sb_edac: Fix rank lookup on Broadwell
    - IB/cm: Fix a recently introduced locking bug
    - IB/mlx4: Properly initialize GRH TClass and FlowLabel in AHs
    - powerpc/pseries: Fix IBM_ARCH_VEC_NRCORES_OFFSET since POWER8NVL was added
    - powerpc/tm: Always reclaim in start_thread() for exec() class syscalls
    - usb: dwc2: fix reg...

Changed in linux (Ubuntu Xenial):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.