From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933350AbcGLPEZ (ORCPT ); Tue, 12 Jul 2016 11:04:25 -0400 Received: from ausc60ps301.us.dell.com ([143.166.148.206]:59400 "EHLO ausc60ps301.us.dell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933220AbcGLPEW convert rfc822-to-8bit (ORCPT ); Tue, 12 Jul 2016 11:04:22 -0400 DomainKey-Signature: s=smtpout; d=dell.com; c=nofws; q=dns; h=X-LoopCount0:X-IronPort-AV:From:To:CC:Subject: Thread-Topic:Thread-Index:Date:Message-ID:References: In-Reply-To:Accept-Language:Content-Language: X-MS-Has-Attach:X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader:x-originating-ip: Content-Type:Content-Transfer-Encoding:MIME-Version: Return-Path; b=K9nll5Ic2i53TtFOj/+utrVo6bEJjDKDQuzuJmHCCqDVrKGQTdC+77C4 Rc/s4BBYUR+UqeA3LLqHR81apU2XZkb+1R/wbgu8i6Xse52ASfd99MVsQ i4nw2hhyuR07soIo62ELvyak201DtUUQvmpsfEcDPj835NagjcVC0Xv3c U=; X-LoopCount0: from 10.170.28.39 X-IronPort-AV: E=Sophos;i="5.28,352,1464670800"; d="scan'208";a="855246758" From: To: , CC: , , , Subject: RE: [PATCH v6 RESEND] r8152: Add support for setting pass through MAC address on RTL8153-AD Thread-Topic: [PATCH v6 RESEND] r8152: Add support for setting pass through MAC address on RTL8153-AD Thread-Index: AQHR29h6B4qewS7x5UizgWbxZsLpH6AUakUAgABIpwCAAC0m0A== Date: Tue, 12 Jul 2016 15:04:12 +0000 Message-ID: <5a304b784fe747aea4de6b69fb12a767@ausx13mpc124.AMER.DELL.COM> References: <1468285084-26557-1-git-send-email-mario_limonciello@dell.com> <20160711.194246.742914205682079488.davem@davemloft.net> <20160712090248.7b5633e5@sauron> In-Reply-To: <20160712090248.7b5633e5@sauron> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.208.89.195] Content-Type: text/plain; charset="iso-8859-2" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: Micha³ Pecio [mailto:michal.pecio@gmail.com] > Sent: Tuesday, July 12, 2016 2:03 AM > To: David Miller > Cc: Limonciello, Mario ; linux- > kernel@vger.kernel.org; netdev@vger.kernel.org; linux- > usb@vger.kernel.org; anthony.wong@canonical.com > Subject: Re: [PATCH v6 RESEND] r8152: Add support for setting pass through > MAC address on RTL8153-AD > > > From: Mario Limonciello > > Date: Mon, 11 Jul 2016 19:58:04 -0500 > > > > > The RTL8153-AD supports a persistent system specific MAC address. > > > This means a device plugged into two different systems with host > > > side support will show different (but persistent) MAC addresses. > > > > > > This information for the system's persistent MAC address is burned > > > in when the system HW is built and available under \_SB.AMAC in the > > > DSDT at runtime. > > > > > > This technology is currently implemented in the Dell TB15 and WD15 > > > Type-C docks. More information is available here: > > > http://www.dell.com/support/article/us/en/04/SLN301147 > > > > > > Signed-off-by: Mario Limonciello > > > > Applied. > > Hi, > > Isn't it possible that the same ACPI name could be used for other > vendor-specific extensions on other laptops and that r8152 will now > wreak havoc there? > > Regards, > MP This has been discussed a bit previously in earlier submissions. In short, this is an extreme corner case. Some changes were made to diminish potential impact. The ACPI code is resolved only when the specific variant of RTL8135 (-AD) has a bit set on the efuse. The patch also explicitly checks the return type and contents of the ACPI object and won't proceed if it's invalid. The Type-C devices that provide this are currently only sold by Dell. This of course may change one day if other OEM's add this feature, but it just shows that device scope is limited. For now the way this is implemented if Realtek does choose to work with another OEM on this feature, there should be no kernel code change necessary for interoperability of peripherals on other OEM machines. So in order to hit this hypothetical corner case today you would need to be using a real world type-C device something such as a Dell WD15 on another OEM's machine that has type-C and the exact same ACPI object name that does $BADSTUFF other than return a buffer. If that situation arises please alert me and I'll send a follow up patch that whitelists this to match DMI vendor of only Dell systems.