linux-kernel - RE: [PATCH v11 14/15] platform/x86: dell-smbios-wmi: introduce userspace interface

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <de1db88b582447e7a32a2825a656147b@ausx13mpc120.AMER.DELL.COM>
Date:   Wed, 1 Nov 2017 16:15:06 +0000
From:   <Mario.Limonciello@...l.com>
To:     <quasisec@...gle.com>
CC:     <dvhart@...radead.org>, <andy.shevchenko@...il.com>,
        <linux-kernel@...r.kernel.org>,
        <platform-driver-x86@...r.kernel.org>, <luto@...nel.org>,
        <pali.rohar@...il.com>, <rjw@...ysocki.net>, <mjg59@...gle.com>,
        <hch@....de>, <greg@...ah.com>, <gnomes@...rguk.ukuu.org.uk>
Subject: RE: [PATCH v11 14/15] platform/x86: dell-smbios-wmi: introduce
 userspace interface

> > +/* 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,