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
| ||
|
Date: Thu, 14 Oct 2021 09:45:16 -0700 From: Guenter Roeck <linux@...ck-us.net> To: Eugene Shalygin <eugene.shalygin@...il.com> Cc: Denis Pauk <pauk.denis@...il.com>, andy.shevchenko@...il.com, platform-driver-x86@...r.kernel.org, Tor Vic <torvic9@...lbox.org>, kernel test robot <lkp@...el.com>, Jean Delvare <jdelvare@...e.com>, Jonathan Corbet <corbet@....net>, linux-hwmon@...r.kernel.org, linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v6 1/2] hwmon: (asus_wmi_ec_sensors) Support B550 Asus WMI. On 10/14/21 7:36 AM, Eugene Shalygin wrote: >> The compiler aligns the data nicely anyway (it will just leave >> a 1-byte hole where needed), so the packing is really irrelevant. >> Apart of that, does the above suggest that some information/ >> code is now missing from the driver ? > > It is perhaps an unnecessary removal which will need to be undone > anyway when I (or others) add the south bridge voltage sensor. > Everything is OK with that change now, but since the size of the > address struct or union is 32 bits anyway, I would not remove that > field. > If the field is not needed now, it is better to not have it in the first place. "We'll need it later" is all but a guarantee for dead code, unless the patch using it is added as part of the same series. In other words: If there is other dead code in the patch, please remove it now. I don't want to find it when I have a closer look at it. If there is code which is supposed to be used later on, either introduce it when it is needed, or introduce it with the same patch series that uses it. Guenter
Powered by blists - more mailing lists