From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754829AbdKAQPL (ORCPT ); Wed, 1 Nov 2017 12:15:11 -0400 Received: from esa4.dell-outbound.iphmx.com ([68.232.149.214]:38246 "EHLO esa4.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751682AbdKAQPJ (ORCPT ); Wed, 1 Nov 2017 12:15:09 -0400 From: X-LoopCount0: from 10.166.132.152 X-IronPort-AV: E=Sophos;i="5.44,329,1505797200"; d="scan'208";a="81913481" X-DLP: DLP_GlobalPCIDSS To: CC: , , , , , , , , , , Subject: RE: [PATCH v11 14/15] platform/x86: dell-smbios-wmi: introduce userspace interface Thread-Topic: [PATCH v11 14/15] platform/x86: dell-smbios-wmi: introduce userspace interface Thread-Index: AQHTUqg1+PzMnZFz9EyHTJeQyg2XQqL/svxg Date: Wed, 1 Nov 2017 16:15:06 +0000 Message-ID: References: In-Reply-To: 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="utf-8" 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 base64 to 8bit by nfs id vA1GFHJp011528 > > +/* This structure may be modified by the firmware when we enter > > + * system management mode through SMM, hence the volatiles > > + */ > > +struct calling_interface_buffer { > > + __u16 class; > > Hey Mario, > > I just realized that there is a slight problem calling this identifier > "class" for userspace that happens to be written in C++ that #includes > this header as class is a reversed word. > Perhaps we could rename it to class_ or something otherwise userspace > has to shim with a C object and extern it with new identifiers to the > C++ component, which is awkward. Edward, Ah darn, I wish we would have caught this earlier in the series development, but it's better to do fix it now rather than after the uapi has been included in a kernel. Darren, I think I should adjust the series for this change to s/class/cmd_class; s/select/cmd_select/ . As ->class is used in many patches in the series, it will require be better to resend as v12 rather than a single patch that changes this one thing. I'll send it later this afternoon. Thanks, From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subject: RE: [PATCH v11 14/15] platform/x86: dell-smbios-wmi: introduce userspace interface Date: Wed, 1 Nov 2017 16:15:06 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from esa4.dell-outbound.iphmx.com ([68.232.149.214]:38246 "EHLO esa4.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751682AbdKAQPJ (ORCPT ); Wed, 1 Nov 2017 12:15:09 -0400 In-Reply-To: Content-Language: en-US Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: quasisec@google.com Cc: dvhart@infradead.org, andy.shevchenko@gmail.com, linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, luto@kernel.org, pali.rohar@gmail.com, rjw@rjwysocki.net, mjg59@google.com, hch@lst.de, greg@kroah.com, gnomes@lxorguk.ukuu.org.uk PiA+ICsvKiBUaGlzIHN0cnVjdHVyZSBtYXkgYmUgbW9kaWZpZWQgYnkgdGhlIGZpcm13YXJlIHdo ZW4gd2UgZW50ZXINCj4gPiArICogc3lzdGVtIG1hbmFnZW1lbnQgbW9kZSB0aHJvdWdoIFNNTSwg aGVuY2UgdGhlIHZvbGF0aWxlcw0KPiA+ICsgKi8NCj4gPiArc3RydWN0IGNhbGxpbmdfaW50ZXJm YWNlX2J1ZmZlciB7DQo+ID4gKyAgICAgICBfX3UxNiBjbGFzczsNCj4gDQo+IEhleSBNYXJpbywN Cj4gDQo+IEkganVzdCByZWFsaXplZCB0aGF0IHRoZXJlIGlzIGEgc2xpZ2h0IHByb2JsZW0gY2Fs bGluZyB0aGlzIGlkZW50aWZpZXINCj4gImNsYXNzIiBmb3IgdXNlcnNwYWNlIHRoYXQgaGFwcGVu cyB0byBiZSB3cml0dGVuIGluIEMrKyB0aGF0ICNpbmNsdWRlcw0KPiB0aGlzIGhlYWRlciBhcyBj bGFzcyBpcyBhIHJldmVyc2VkIHdvcmQuDQo+IFBlcmhhcHMgd2UgY291bGQgcmVuYW1lIGl0IHRv IGNsYXNzXyBvciBzb21ldGhpbmcgb3RoZXJ3aXNlIHVzZXJzcGFjZQ0KPiBoYXMgdG8gc2hpbSB3 aXRoIGEgQyBvYmplY3QgYW5kIGV4dGVybiBpdCB3aXRoIG5ldyBpZGVudGlmaWVycyB0byB0aGUN Cj4gQysrIGNvbXBvbmVudCwgd2hpY2ggaXMgYXdrd2FyZC4NCg0KRWR3YXJkLA0KDQpBaCBkYXJu LCBJIHdpc2ggd2Ugd291bGQgaGF2ZSBjYXVnaHQgdGhpcyBlYXJsaWVyIGluIHRoZSBzZXJpZXMg ZGV2ZWxvcG1lbnQsDQpidXQgaXQncyBiZXR0ZXIgdG8gZG8gZml4IGl0IG5vdyByYXRoZXIgdGhh biBhZnRlciB0aGUgdWFwaSBoYXMgYmVlbiBpbmNsdWRlZCBpbg0KYSBrZXJuZWwuDQoNCkRhcnJl biwNCg0KSSB0aGluayBJIHNob3VsZCBhZGp1c3QgdGhlIHNlcmllcyBmb3IgdGhpcyBjaGFuZ2Ug dG8NCnMvY2xhc3MvY21kX2NsYXNzOyBzL3NlbGVjdC9jbWRfc2VsZWN0Lw0KLg0KQXMgLT5jbGFz cyBpcyB1c2VkIGluIG1hbnkgcGF0Y2hlcyBpbiB0aGUgc2VyaWVzLCBpdCB3aWxsIHJlcXVpcmUg YmUgYmV0dGVyIHRvDQpyZXNlbmQgYXMgdjEyIHJhdGhlciB0aGFuIGEgc2luZ2xlIHBhdGNoIHRo YXQgY2hhbmdlcyB0aGlzIG9uZSB0aGluZy4NCg0KSSdsbCBzZW5kIGl0IGxhdGVyIHRoaXMgYWZ0 ZXJub29uLg0KDQpUaGFua3MsDQoNCg0KDQo=