* 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