From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756107AbdKCQaS (ORCPT ); Fri, 3 Nov 2017 12:30:18 -0400 Received: from esa3.dell-outbound.iphmx.com ([68.232.153.94]:62388 "EHLO esa3.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755883AbdKCQaP (ORCPT ); Fri, 3 Nov 2017 12:30:15 -0400 From: X-LoopCount0: from 10.166.132.169 X-IronPort-AV: E=Sophos;i="5.44,339,1505797200"; d="scan'208";a="1053135696" X-DLP: DLP_GlobalPCIDSS To: CC: , , , , , , , , , , Subject: RE: [PATCH v12 00/16] Introduce support for Dell SMBIOS over WMI Thread-Topic: [PATCH v12 00/16] Introduce support for Dell SMBIOS over WMI Thread-Index: AQHTVD3B8BjewxEJmE+y9FRs3FWqjaMC2ORg Date: Fri, 3 Nov 2017 16:30:13 +0000 Message-ID: <3ee790eeb8e242f2a064905074777674@ausx13mpc124.AMER.DELL.COM> References: <20171103005025.GA7971@fury> In-Reply-To: <20171103005025.GA7971@fury> 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.143.242.75] Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id vA3GUM7K024452 > -----Original Message----- > From: Darren Hart [mailto:dvhart@infradead.org] > Sent: Thursday, November 2, 2017 7:50 PM > To: Limonciello, Mario > Cc: Andy Shevchenko ; LKML kernel@vger.kernel.org>; platform-driver-x86@vger.kernel.org; Andy Lutomirski > ; quasisec@google.com; pali.rohar@gmail.com; > rjw@rjwysocki.net; mjg59@google.com; hch@lst.de; Greg KH ; > Alan Cox > Subject: Re: [PATCH v12 00/16] Introduce support for Dell SMBIOS over WMI > > On Wed, Nov 01, 2017 at 02:25:21PM -0500, Mario Limonciello wrote: > > The existing way that the dell-smbios helper module and associated > > other drivers (dell-laptop, dell-wmi) communicate with the platform > > really isn't secure. It requires creating a buffer in physical > > DMA32 memory space and passing that to the platform via SMM. > > > > Since the platform got a physical memory pointer, you've just got > > to trust that the platform has only modified (and accessed) memory > > within that buffer. > > > > Dell Platform designers recognize this security risk and offer a > > safer way to communicate with the platform over ACPI. This is > > in turn exposed via a WMI interface to the OS. > > > > When communicating over WMI-ACPI the communication doesn't occur > > with physical memory pointers. When the ASL is invoked, the fixed > > length ACPI buffer is copied to a small operating region. The ASL > > will invoke the SMI, and SMM will only have access to this operating > > region. When the ASL returns the buffer is copied back for the OS > > to process. > > > > This method of communication should also deprecate the usage of the > > dcdbas kernel module and software dependent upon it's interface. > > Instead offer a character device interface for communicating with this > > ASL method to allow userspace to use instead. > > > > To faciliate that this patch series introduces a generic way for WMI > > drivers to be able to create discoverable character devices with > > a predictable IOCTL interface through the WMI bus when desired. > > Requiring WMI drivers to explicitly ask for this functionality will > > act as an effective vendor whitelist to character device creation. > > > > Some of this work is the basis for what will be a proper interpreter > > of MOF in the kernel and controls for what drivers will be able to > > do with that MOF. > > > > NOTE: This patch series is intended to go on top of platform-drivers-x86 > > linux-next. > > > > For convenience the entire series including those is also available here: > > https://github.com/dell/linux/tree/wmi-smbios > > Queued for testing, thanks Mario. > > -- > Darren Hart > VMware Open Source Technology Center Thanks Darren. BTW Did you forget to push? I didn't see it at the testing branch: http://git.infradead.org/users/dvhart/linux-platform-drivers-x86.git/shortlog/refs/heads/testing Thanks,