Re: [PATCH] libosd: Remove ignored __weak attribute

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2018-10-26 at 15:07 -0700, Nick Desaulniers wrote:
+AD4 I don't disagree with you, I just don't see how what you state can be
+AD4 reconciled with Linus' response in
+AD4 https://lkml.org/lkml/2018/10/27/44.  Those two viewpoints seem
+AD4 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.

+AD4 Nathan and I are just pointing out a small fix to eliminate a small
+AD4 warning, deleting all this code does kind of feels like +ACI-throwing out
+AD4 the baby with the bath water.+ACI A nuclear option for what would be a
+AD4 small change otherwise.  Maybe it's good to discuss the EOL for
+AD4 exofs/osd, but can we please decouple that conversation from the small
+AD4 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.



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux