Commit c9cc3aaa authored by Lukas Wunner's avatar Lukas Wunner Committed by Ingo Molnar

thunderbolt: Use Device ROM retrieved from EFI

Macs with Thunderbolt 1 do not have a unit-specific DROM: The DROM is
empty with uid 0x1000000000000. (Apple started factory-burning a unit-
specific DROM with Thunderbolt 2.)

Instead, the NHI EFI driver supplies a DROM in a device property. Use
it if available. It's only available when booting with the efistub.
If it's not available, silently fall back to our hardcoded DROM.

The size of the DROM is always 256 bytes. The number is hardcoded into
the NHI EFI driver. This commit can deal with an arbitrary size however,
just in case they ever change that.

Background information: The EFI firmware volume contains ROM files for
the NHI, GMUX and several other chips as well as key material. This
strategy allows Apple to deploy ROM or key updates by simply publishing
an EFI firmware update on their website. Drivers do not access those
files directly but rather through a file server via EFI protocol
AC5E4829-A8FD-440B-AF33-9FFE013B12D8. Files are identified by GUID, the
NHI DROM has 339370BD-CFC6-4454-8EF7-704653120818.

The NHI EFI driver amends that file with a unit-specific uid. The uid
has 64 bit but its entropy is much lower: 24 bit represent the model,
24 bit are taken from a serial number, 16 bit are fixed. The NHI EFI
driver obtains the serial number via the DataHub protocol, copies it
into the DROM, calculates the CRC and submits the result as a device
property.

A modification is needed in the resume code where we currently read the
uid of all switches in the hierarchy to detect plug events that occurred
during sleep. On Thunderbolt 1 root switches this will now lead to a
mismatch between the uid of the empty DROM and the EFI DROM. Exempt the
root switch from this check: It's built in, so the uid should never
change. However we continue to *read* the uid of the root switch, this
seems like a good way to test its reachability after resume.

Tested-by: Lukas Wunner <lukas@wunner.de> [MacBookPro9,1]
Tested-by: Pierre Moreau <pierre.morrow@free.fr> [MacBookPro11,3]
Signed-off-by: default avatarLukas Wunner <lukas@wunner.de>
Signed-off-by: default avatarMatt Fleming <matt@codeblueprint.co.uk>
Acked-by: default avatarAndreas Noever <andreas.noever@gmail.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Pedro Vilaça <reverser@put.as>
Cc: Peter Jones <pjones@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-efi@vger.kernel.org
Link: http://lkml.kernel.org/r/20161112213237.8804-10-matt@codeblueprint.co.ukSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent 58c5475a
menuconfig THUNDERBOLT menuconfig THUNDERBOLT
tristate "Thunderbolt support for Apple devices" tristate "Thunderbolt support for Apple devices"
depends on PCI depends on PCI
select APPLE_PROPERTIES
select CRC32 select CRC32
help help
Cactus Ridge Thunderbolt Controller driver Cactus Ridge Thunderbolt Controller driver
......
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
*/ */
#include <linux/crc32.h> #include <linux/crc32.h>
#include <linux/property.h>
#include <linux/slab.h> #include <linux/slab.h>
#include "tb.h" #include "tb.h"
...@@ -359,6 +360,40 @@ static int tb_drom_parse_entries(struct tb_switch *sw) ...@@ -359,6 +360,40 @@ static int tb_drom_parse_entries(struct tb_switch *sw)
return 0; return 0;
} }
/**
* tb_drom_copy_efi - copy drom supplied by EFI to sw->drom if present
*/
static int tb_drom_copy_efi(struct tb_switch *sw, u16 *size)
{
struct device *dev = &sw->tb->nhi->pdev->dev;
int len, res;
len = device_property_read_u8_array(dev, "ThunderboltDROM", NULL, 0);
if (len < 0 || len < sizeof(struct tb_drom_header))
return -EINVAL;
sw->drom = kmalloc(len, GFP_KERNEL);
if (!sw->drom)
return -ENOMEM;
res = device_property_read_u8_array(dev, "ThunderboltDROM", sw->drom,
len);
if (res)
goto err;
*size = ((struct tb_drom_header *)sw->drom)->data_len +
TB_DROM_DATA_START;
if (*size > len)
goto err;
return 0;
err:
kfree(sw->drom);
sw->drom = NULL;
return -EINVAL;
}
/** /**
* tb_drom_read - copy drom to sw->drom and parse it * tb_drom_read - copy drom to sw->drom and parse it
*/ */
...@@ -373,6 +408,13 @@ int tb_drom_read(struct tb_switch *sw) ...@@ -373,6 +408,13 @@ int tb_drom_read(struct tb_switch *sw)
return 0; return 0;
if (tb_route(sw) == 0) { if (tb_route(sw) == 0) {
/*
* Apple's NHI EFI driver supplies a DROM for the root switch
* in a device property. Use it if available.
*/
if (tb_drom_copy_efi(sw, &size) == 0)
goto parse;
/* /*
* The root switch contains only a dummy drom (header only, * The root switch contains only a dummy drom (header only,
* no entries). Hardcode the configuration here. * no entries). Hardcode the configuration here.
...@@ -418,6 +460,7 @@ int tb_drom_read(struct tb_switch *sw) ...@@ -418,6 +460,7 @@ int tb_drom_read(struct tb_switch *sw)
if (res) if (res)
goto err; goto err;
parse:
header = (void *) sw->drom; header = (void *) sw->drom;
if (header->data_len + TB_DROM_DATA_START != size) { if (header->data_len + TB_DROM_DATA_START != size) {
......
...@@ -460,7 +460,7 @@ int tb_switch_resume(struct tb_switch *sw) ...@@ -460,7 +460,7 @@ int tb_switch_resume(struct tb_switch *sw)
tb_sw_warn(sw, "uid read failed\n"); tb_sw_warn(sw, "uid read failed\n");
return err; return err;
} }
if (sw->uid != uid) { if (sw != sw->tb->root_switch && sw->uid != uid) {
tb_sw_info(sw, tb_sw_info(sw,
"changed while suspended (uid %#llx -> %#llx)\n", "changed while suspended (uid %#llx -> %#llx)\n",
sw->uid, uid); sw->uid, uid);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment