* Re: [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h [not found] <200510290000.j9T00Bqd001135@hera.kernel.org> @ 2005-10-31 2:42 ` Dave Jones 2005-10-31 3:38 ` Grant Coady 0 siblings, 1 reply; 6+ messages in thread From: Dave Jones @ 2005-10-31 2:42 UTC (permalink / raw) To: Linux Kernel Mailing List On Fri, Oct 28, 2005 at 05:00:11PM -0700, Linux Kernel wrote: > tree 68609a74c3bc43e510f58f9c808a0a74e9d23452 > parent 4153812fc10ea91cb1a7b6ea4f4337dd211c1ef7 > author Grant Coady <gcoady@gmail.com> Thu, 29 Sep 2005 11:06:40 +1000 > committer Greg Kroah-Hartman <gregkh@suse.de> Sat, 29 Oct 2005 05:36:59 -0700 > > [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h > > pci_ids.h cleanup: removed non-referenced symbols, compile tested > with 'make allmodconfig' > > Signed-off-by: Grant Coady <gcoady@gmail.com> > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> This patch is removing some PCI idents from drivers that are currently marked BROKEN on some/all architectures. It seems counterproductive to create even more work to get those drivers fixed. Especially in the case of for eg, the advansys scsi driver, which actually works for some people, even though it isn't updated to use modern scsi layer interfaces. Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h 2005-10-31 2:42 ` [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h Dave Jones @ 2005-10-31 3:38 ` Grant Coady 2005-10-31 4:13 ` Dave Jones 2005-10-31 15:45 ` Alan Cox 0 siblings, 2 replies; 6+ messages in thread From: Grant Coady @ 2005-10-31 3:38 UTC (permalink / raw) To: Dave Jones; +Cc: Linux Kernel Mailing List Dave Jones wrote: > On Fri, Oct 28, 2005 at 05:00:11PM -0700, Linux Kernel wrote: > > tree 68609a74c3bc43e510f58f9c808a0a74e9d23452 > > parent 4153812fc10ea91cb1a7b6ea4f4337dd211c1ef7 > > author Grant Coady <gcoady@gmail.com> Thu, 29 Sep 2005 11:06:40 +1000 > > committer Greg Kroah-Hartman <gregkh@suse.de> Sat, 29 Oct 2005 05:36:59 -0700 > > > > [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h > > > > pci_ids.h cleanup: removed non-referenced symbols, compile tested > > with 'make allmodconfig' > > > > Signed-off-by: Grant Coady <gcoady@gmail.com> > > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> > > This patch is removing some PCI idents from drivers that are currently > marked BROKEN on some/all architectures. It seems counterproductive > to create even more work to get those drivers fixed. Nobody cares, the drivers are dying of bit-rot :) > Especially in the case of for eg, the advansys scsi driver, which > actually works for some people, even though it isn't updated to use > modern scsi layer interfaces. Any positive suggestions? How many years does a driver remain broken before it gets removed? These drivers don't compile cleanly thus are not in use, no? Perhaps a set of patches scheduling removal is in order. Grant. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h 2005-10-31 3:38 ` Grant Coady @ 2005-10-31 4:13 ` Dave Jones 2005-10-31 5:26 ` Grant Coady 2005-10-31 15:45 ` Alan Cox 1 sibling, 1 reply; 6+ messages in thread From: Dave Jones @ 2005-10-31 4:13 UTC (permalink / raw) To: Grant Coady; +Cc: Linux Kernel Mailing List On Mon, Oct 31, 2005 at 02:38:13PM +1100, Grant Coady wrote: > > This patch is removing some PCI idents from drivers that are currently > > marked BROKEN on some/all architectures. It seems counterproductive > > to create even more work to get those drivers fixed. > > Nobody cares, the drivers are dying of bit-rot :) Remove the BROKEN, and it builds, and runs just fine on most systems. (Or it least it did, until this intentional breakage occured). > > Especially in the case of for eg, the advansys scsi driver, which > > actually works for some people, even though it isn't updated to use > > modern scsi layer interfaces. > > Any positive suggestions? Yes. Don't remove symbols that are referenced by code in the rest of the tree (even if it isn't buildable). It's not as though leaving those symbols there breaks anything, or even bloats the kernel. > How many years does a driver remain broken before it gets removed? These > drivers don't compile cleanly thus are not in use, no? Perhaps a set of > patches scheduling removal is in order. At least 2 distros are carrying patches removing the BROKEN attribute on the advansys Kconfig for some architectures. The users of those kernels using their advansys controllers without any issue at all. Even if this were not the case, randomly removing bits of a driver so that it has no chance of working isn't how we schedule removal. Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h 2005-10-31 4:13 ` Dave Jones @ 2005-10-31 5:26 ` Grant Coady 2005-10-31 5:56 ` Dave Jones 0 siblings, 1 reply; 6+ messages in thread From: Grant Coady @ 2005-10-31 5:26 UTC (permalink / raw) To: Dave Jones; +Cc: Linux Kernel Mailing List Dave Jones wrote: > At least 2 distros are carrying patches removing the BROKEN attribute > on the advansys Kconfig for some architectures. The users of those kernels > using their advansys controllers without any issue at all. So why are your driver patches not in mainline then? Grant. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h 2005-10-31 5:26 ` Grant Coady @ 2005-10-31 5:56 ` Dave Jones 0 siblings, 0 replies; 6+ messages in thread From: Dave Jones @ 2005-10-31 5:56 UTC (permalink / raw) To: Grant Coady; +Cc: Linux Kernel Mailing List On Mon, Oct 31, 2005 at 04:26:06PM +1100, Grant Coady wrote: > Dave Jones wrote: > > At least 2 distros are carrying patches removing the BROKEN attribute > > on the advansys Kconfig for some architectures. The users of those kernels > > using their advansys controllers without any issue at all. > > So why are your driver patches not in mainline then? Because it serves as a reminder that it needs fixing. Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h 2005-10-31 3:38 ` Grant Coady 2005-10-31 4:13 ` Dave Jones @ 2005-10-31 15:45 ` Alan Cox 1 sibling, 0 replies; 6+ messages in thread From: Alan Cox @ 2005-10-31 15:45 UTC (permalink / raw) To: Grant Coady; +Cc: Dave Jones, Linux Kernel Mailing List On Llu, 2005-10-31 at 14:38 +1100, Grant Coady wrote: > > This patch is removing some PCI idents from drivers that are currently > > marked BROKEN on some/all architectures. It seems counterproductive > > to create even more work to get those drivers fixed. > > Nobody cares, the drivers are dying of bit-rot :) If you look at the -mm tree you will see various drivers are getting fixed eventually. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2005-10-31 15:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <200510290000.j9T00Bqd001135@hera.kernel.org>
2005-10-31 2:42 ` [PATCH] pci_ids: remove non-referenced symbols from pci_ids.h Dave Jones
2005-10-31 3:38 ` Grant Coady
2005-10-31 4:13 ` Dave Jones
2005-10-31 5:26 ` Grant Coady
2005-10-31 5:56 ` Dave Jones
2005-10-31 15:45 ` Alan Cox
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox