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]
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,



Powered by blists - more mailing lists