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: Fri, 26 Oct 2018 15:24:44 -0700 From: Bart Van Assche <bvanassche@....org> To: Nick Desaulniers <ndesaulniers@...gle.com>, Linus Torvalds <torvalds@...ux-foundation.org> Cc: ooo@...ctrozaur.com, Nathan Chancellor <natechancellor@...il.com>, "James E.J. Bottomley" <jejb@...ux.vnet.ibm.com>, "Martin K. Petersen" <martin.petersen@...cle.com>, linux-scsi@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>, hch@...radead.org Subject: Re: [PATCH] libosd: Remove ignored __weak attribute On Fri, 2018-10-26 at 15:07 -0700, Nick Desaulniers wrote: > I don't disagree with you, I just don't see how what you state can be > reconciled with Linus' response in > https://lkml.org/lkml/2018/10/27/44. Those two viewpoints seem > incompatible to me, but maybe there's a nuance I'm missing? I don't think there is any disagreement nor that there are any conflicting viewpoints. As explained in previous e-mails it is unlikely that anyone is using these kernel drivers and as far as I know Linus is OK with removing unused kernel drivers. > Nathan and I are just pointing out a small fix to eliminate a small > warning, deleting all this code does kind of feels like "throwing out > the baby with the bath water." A nuclear option for what would be a > small change otherwise. Maybe it's good to discuss the EOL for > exofs/osd, but can we please decouple that conversation from the small > change Nathan and I are proposing? Removing a kernel driver is not a nuclear option. You may not be aware of this but it happens all the time. From a maintainer point of view it is a very sensible action because there are people who keep submitting cleanup patches for kernel drivers they do not use themselves. Every individual patch needs some attention and hence causes some work for a kernel maintainer. Removing kernel drivers that are not used helps to reduce the workload of a maintainer and hence is a rational action. Additionally, if anyone would ever complain about removal of a kernel driver, it can be brought back by reverting the commit through which it has been removed. Martin, please reply if you see this differently. Bart.
Powered by blists - more mailing lists