PATCH & call for help: Marking ISA only drivers
Subsystems (4)
Hallo,
This patch tries to make most ISA only drivers dependent on CONFIG_ISA.
The motivation is that it is a lot of work to get old drivers to compile meaningfully (at least without warnings, not even testing them) on x86-64 and a lot of them are obviously not 64bit safe. As it is very unlikely that x86-64 boxes will ever have ISA slots[1] one simple way for that is just removing the old ISA drivers from the configuration.
BTW I think CONFIG_ISA would be an useful configuration option for i386 too - at least most modern PCs do not come with ISA slots anymore.
Back to the patch:
I left drivers/net alone because it looked like it already used CONFIG_ISA properly. I went over drivers/scsi and some other drivers directories and just made those that looked like ISA only dependent on CONFIG_ISA. Note that I usually don't know much about these devices, I just took a quick look at the source of the driver. Of course I may have made mistakes during that, so if you see any driver wrongly marked please tell me.
You can test that on i386 by applying the patch, change the "define_bool CONFIG_ISA y" in arch/i386/config to n and run your favourite configuration tool and look for any ISA only drivers still appearing as configuration choices.
I left out drivers/media/video and drivers/input/joystick - if there are any ISA only devices in there please tell me.
Patch for 2.5.14
Thanks,
-Andi
[1] If someone later will be ever crazy enough to buy an x86-64 machine and put an ISA bridge into it they'll have to pay for that and fix all the ISA drivers they need ;)
arch/x86_64/config.in +9 -5
@@ -2,12 +2,15 @@ # For a description of the syntax of this configuration file, # see Documentation/kbuild/config-language.txt. # +# Note: ISA is disabled and will hopefully never be enabled. +# + mainmenu_name "Linux Kernel Configuration" define_bool CONFIG_X86_64 y define_bool CONFIG_X86 y -define_bool CONFIG_ISA y +define_bool CONFIG_ISA n define_bool CONFIG_SBUS n define_bool CONFIG_UID16 y
@@ -94,7 +99,7 @@ source drivers/mtd/Config.in source drivers/parport/Config.in -source drivers/pnp/Config.in +#source drivers/pnp/Config.in source drivers/block/Config.in
@@ -142,9 +147,10 @@ if [ "$CONFIG_NET" = "y" ]; then bool 'Network device support' CONFIG_NETDEVICES if [ "$CONFIG_NETDEVICES" = "y" ]; then source drivers/net/Config.in - if [ "$CONFIG_ATM" = "y" ]; then - source drivers/atm/Config.in - fi +# ATM seems to be largely 64bit unsafe and also unmaintained. +# if [ "$CONFIG_ATM" = "y" ]; then +# source drivers/atm/Config.in +# fi fi endmenu fi
Only in linux/arch/x86_64: config.in-o
drivers/char/Config.in +1 -1
@@ -39,7 +39,7 @@ if [ "$CONFIG_SERIAL_NONSTANDARD" = "y" if [ "$CONFIG_DIGIEPCA" = "n" ]; then tristate ' Digiboard PC/Xx Support' CONFIG_DIGI fi - tristate ' Hayes ESP serial port support' CONFIG_ESPSERIAL + dep_tristate ' Hayes ESP serial port support' CONFIG_ESPSERIAL CONFIG_ISA tristate ' Moxa Intellio support' CONFIG_MOXA_INTELLIO tristate ' Moxa SmartIO support' CONFIG_MOXA_SMARTIO if [ "$CONFIG_EXPERIMENTAL" = "y" ]; then
Only in linux/drivers/char: Config.in-o
Only in linux/drivers/char: conmakehash
drivers/media/radio/Config.in +4 -0
@@ -4,6 +4,7 @@ mainmenu_option next_comment comment 'Radio Adapters' +if [ "$CONFIG_ISA" = "y" ]; then dep_tristate ' ADS Cadet AM/FM Tuner' CONFIG_RADIO_CADET $CONFIG_VIDEO_DEV dep_tristate ' AIMSlab RadioTrack (aka RadioReveal) support' CONFIG_RADIO_RTRACK $CONFIG_VIDEO_DEV if [ "$CONFIG_RADIO_RTRACK" = "y" ]; then
@@ -21,9 +22,11 @@ dep_tristate ' GemTek Radio Card suppor if [ "$CONFIG_RADIO_GEMTEK" = "y" ]; then hex ' GemTek i/o port (0x20c, 0x30c, 0x24c or 0x34c)' CONFIG_RADIO_GEMTEK_PORT 34c fi +fi dep_tristate ' GemTek PCI Radio Card support' CONFIG_RADIO_GEMTEK_PCI $CONFIG_VIDEO_DEV $CONFIG_PCI dep_tristate ' Guillemot MAXI Radio FM 2000 radio' CONFIG_RADIO_MAXIRADIO $CONFIG_VIDEO_DEV dep_tristate ' Maestro on board radio' CONFIG_RADIO_MAESTRO $CONFIG_VIDEO_DEV +if [ "$CONFIG_ISA" = "y" ]; then dep_tristate ' miroSOUND PCM20 radio' CONFIG_RADIO_MIROPCM20 $CONFIG_VIDEO_DEV $CONFIG_SOUND_ACI_MIXER dep_tristate ' miroSOUND PCM20 radio RDS user interface (EXPERIMENTAL)' CONFIG_RADIO_MIROPCM20_RDS $CONFIG_RADIO_MIROPCM20 $CONFIG_EXPERIMENTAL dep_tristate ' SF16FMI Radio' CONFIG_RADIO_SF16FMI $CONFIG_VIDEO_DEV
@@ -48,6 +51,7 @@ fi dep_tristate ' Zoltrix Radio' CONFIG_RADIO_ZOLTRIX $CONFIG_VIDEO_DEV if [ "$CONFIG_RADIO_ZOLTRIX" = "y" ]; then hex ' ZOLTRIX I/O port (0x20c or 0x30c)' CONFIG_RADIO_ZOLTRIX_PORT 20c +fi fi endmenu
Only in linux/drivers/media/radio: Config.in-o
drivers/net/hamradio/Config.in +4 -2
@@ -4,8 +4,10 @@ dep_tristate 'Serial port KISS driver' C dep_tristate 'Serial port 6PACK driver' CONFIG_6PACK $CONFIG_AX25 dep_tristate 'BPQ Ethernet driver' CONFIG_BPQETHER $CONFIG_AX25 -dep_tristate 'High-speed (DMA) SCC driver for AX.25' CONFIG_DMASCC $CONFIG_AX25 -dep_tristate 'Z8530 SCC driver' CONFIG_SCC $CONFIG_AX25 +if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'High-speed (DMA) SCC driver for AX.25' CONFIG_DMASCC $CONFIG_AX25 + dep_tristate 'Z8530 SCC driver' CONFIG_SCC $CONFIG_AX25 +fi if [ "$CONFIG_SCC" != "n" ]; then bool ' additional delay for PA0HZP OptoSCC compatible boards' CONFIG_SCC_DELAY bool ' support for TRX that feedback the tx signal to rx' CONFIG_SCC_TRXECHO
Only in linux/drivers/net/hamradio: Config.in-o
drivers/net/wan/Config.in +2 -1
@@ -9,11 +9,12 @@ bool 'Wan interfaces support' CONFIG_WAN if [ "$CONFIG_WAN" = "y" ]; then # There is no way to detect a comtrol sv11 - force it modular for now. + if [ "$CONFIG_ISA" = "y" ]; then dep_tristate ' Comtrol Hostess SV-11 support' CONFIG_HOSTESS_SV11 m - # The COSA/SRP driver has not been tested as non-modular yet. dep_tristate ' COSA/SRP sync serial boards support' CONFIG_COSA m + fi # # COMX drivers
Only in linux/drivers/net/wan: Config.in-o
Only in linux/drivers/pci: gen-devlist
drivers/scsi/Config.in +51 -20
133 lines not on this page · raw message
Only in linux/drivers/scsi: Config.in-o
Good evening, Andi,
Quoted 1 line · Andi Kleen
This patch tries to make most ISA only drivers dependent on
CONFIG_ISA.
I did quite a bit of this work for CML2 - bus dependencies can be
found in the CML2 sources.
Cheers,
- Bill
---------------------------------------------------------------------------
"Do you smell something burning or is it me?"
-- Joan of Arc
--------------------------------------------------------------------------
William Stearns ([email protected]). Mason, Buildkernel, named2hosts,
and ipfwadm2ipchains are at: http://www.stearns.org
--------------------------------------------------------------------------
CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Andi KleenQuoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Btw, what happened to CML2?
I haven't seen any updates since about February and haven't almost certainly stumbled upon a post from ESR for quite long either.
T.
Quoted 2 lines
BTW I think
CONFIG_ISAwould be an useful configuration option for i386 too - at least most modern PCs do not come with ISA slots anymore.
ISA slots != ISA devices
And ISA stuff hanging off things like chipset glue busses is everywhere
Review
Quoted 2 lines
(quoted diff) +2
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Adaptec AHA152X/2825 support' CONFIG_SCSI_AHA152X $CONFIG_SCSI
2825 is not ISA bus
drivers/scsi/Config.in if [ "$CONFIG_SCSI_AIC7XXX" != "y" ]; th · line 74
Quoted 4 lines
drivers/scsi/Config.in +4
-0+# does not use pci dma and seems to be isa/onboard only for old machines +if [ "$CONFIG_X86_64" != "y" ]; then + dep_tristate 'AM53/79C974 PCI SCSI support' CONFIG_SCSI_AM53C974 $CONFIG_SCSI $CONFIG_PCI +fi
Thats PCI.
drivers/scsi/Config.in if [ "$CONFIG_MCA" = "y" ]; then · line 103
Quoted 3 lines
drivers/scsi/Config.in +3
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Generic NCR5380/53c400 SCSI support' CONFIG_SCSI_GENERIC_NCR5380 $CONFIG_SCSI + if [ "$CONFIG_SCSI_GENERIC_NCR5380" != "n" ]; then
This is used in multiple non ISA situations.
Quoted 3 lines
+if [ "$CONFIG_X86" = "y" -a "$CONFIG_ISA" = "y" ]; then
dep_tristate'UltraStor SCSI support'CONFIG_SCSI_ULTRASTOR$CONFIG_SCSI
fi
It makes an ugly mess of Config.in - I guess 2.5 is the time to do what Russell and Keith wanted and make undefined imply "N" in dep_*
Alan
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Andi KleenQuoted 3 lines · Tomas Szepe
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Btw, what happened to CML2?
The majority seemed to be of the opinion it was too overfeatured whilst lacking some of the more basic functionality desired from those who would end up using it on a daily basis in favour of eye candy for users who compile a kernel once per release.
Quoted 2 lines
I haven't seen any updates since about February and haven't almost certainly stumbled upon a post from ESR for quite long either.
Yes, signal has been quite high on Linux Kernel last few months.
Signature · 2 lines
| Dave Jones. http://www.codemonkey.org.uk | SuSE Labs
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Tomas SzepeQuoted 4 lines · Tomas Szepe
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Btw, what happened to CML2?
This is an ex-parrot
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Tomas SzepeQuoted 8 lines
[Alexander Viro <[email protected]>, May-09 2002, Thu, 16:23 -0400]
Quoted 4 lines
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Btw, what happened to CML2?
This is an ex-parrot
If you're referring to what I think you are (uhhh), then maybe "he's just resting"?
I mean, people certainly don't want to see a slug in its place.
:)
T.
Uz.ytkownik Andi Kleen napisa?:
Quoted 3 lines
Hallo,
This patch tries to make most ISA only drivers dependent on
CONFIG_ISA.
If only for the fact that it allows you to don't look at archaic hardware configuration options makes it a good idea I think. Bus is bus if we have CONFIG_PCI, we should have CONFIG_ISA as well.
Quoted 7 lines
The motivation is that it is a lot of work to get old drivers to compile meaningfully (at least without warnings, not even testing them) on x86-64 and a lot of them are obviously not 64bit safe. As it is very unlikely that x86-64 boxes will ever have ISA slots[1] one simple way for that is just removing the old ISA drivers from the configuration.
BTW I think
CONFIG_ISAwould be an useful configuration option for i386 too - at least most modern PCs do not come with ISA slots anymore.
Plase add mcd and mcdx - CD-ROM drivers. Both of them required an special "controller" card, which was indeed ISA based.
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Andi KleenQuoted 6 lines · Tomas Szepe
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Btw, what happened to CML2?
I haven't seen any updates since about February and haven't almost certainly stumbled upon a post from ESR for quite long either.
Eric was blasted away from the linux-kernel mailing list.
I understand that people tend to disagree about some subject every once in a while. But he certainly didn't deserve the treatment he got. Specially considering the enormous amount of work he put in it.
And I miss his signatures and the discussions about Aunt Tillie and stuff. :-)
Toon.
Signature · 4 lines
/"\ | \ / ASCII RIBBON CAMPAIGN | X AGAINST HTML MAIL | / \
Quoted 7 lines · Martin Dalecki
Uz.ytkownik Andi Kleen napisa?:
Quoted 3 lines
Hallo,
This patch tries to make most ISA only drivers dependent on
CONFIG_ISA.If only for the fact that it allows you to don't look at archaic hardware configuration options makes it a good idea I think.
And often the old drivers do not work anymore anyways...
Quoted 11 lines
Quoted 8 lines
The motivation is that it is a lot of work to get old drivers to compile meaningfully (at least without warnings, not even testing them) on x86-64 and a lot of them are obviously not 64bit safe. As it is very unlikely that x86-64 boxes will ever have ISA slots[1] one simple way for that is just removing the old ISA drivers from the configuration.
BTW I think
CONFIG_ISAwould be an useful configuration option for i386 too - at least most modern PCs do not come with ISA slots anymore.Plase add mcd and mcdx - CD-ROM drivers. Both of them required an special "controller" card, which was indeed ISA based.
I removed the complete drivers/cdrom directory for x86-64 now, as it seems to contain no PCI code at all. For i386 it can be made dependent on CONFIG_ISA.
Also I did this change for the IDE configuration (assuming that there are no boxes with no ISA but a VLB slot). I hope this covers all ISA/VLB only adapters for IDE.
You may want to incorporate it.
-Andi
drivers/ide/Config.in renamed from drivers/ide/Config.in-o +2 -1
@@ -131,7 +131,8 @@ EXT_DIRECT CONFIG_IDE_EXT_DIRECT" 8xx_PCCARD fi - bool ' Other IDE chipset support' CONFIG_IDE_CHIPSETS + # assume no ISA -> also no VLB + dep_bool ' Other ISA/VLB IDE chipset support' CONFIG_IDE_CHIPSETS CONFIG_ISA if [ "$CONFIG_IDE_CHIPSETS" = "y" ]; then comment 'Note: most of these also require special kernel boot parameters' bool ' ALI M14xx support' CONFIG_BLK_DEV_ALI14XX
Quoted 11 lines · Alan Cox
Quoted 2 lines
BTW I think
CONFIG_ISAwould be an useful configuration option for i386 too - at least most modern PCs do not come with ISA slots anymore.ISA slots != ISA devices
And ISA stuff hanging off things like chipset glue busses is everywhere
Review
Quoted 2 lines
(quoted diff) +2
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Adaptec AHA152X/2825 support' CONFIG_SCSI_AHA152X $CONFIG_SCSI2825 is not ISA bus
What then ?
Quoted 7 lines
drivers/scsi/Config.in if [ "$CONFIG_SCSI_AIC7XXX" != "y" ]; th · line 74
Quoted 5 lines
drivers/scsi/Config.in +4
-0+# does not use pci dma and seems to be isa/onboard only for old machines +if [ "$CONFIG_X86_64" != "y" ]; then + dep_tristate 'AM53/79C974 PCI SCSI support' CONFIG_SCSI_AM53C974 $CONFIG_SCSI $CONFIG_PCI +fiThats PCI.
But seems to be not 64bit safe and miss pci dma support (note the CONFIG_X86_64).
Quoted 6 lines
drivers/scsi/Config.in if [ "$CONFIG_MCA" = "y" ]; then · line 103
Quoted 4 lines
drivers/scsi/Config.in +3
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Generic NCR5380/53c400 SCSI support' CONFIG_SCSI_GENERIC_NCR5380 $CONFIG_SCSI + if [ "$CONFIG_SCSI_GENERIC_NCR5380" != "n" ]; thenThis is used in multiple non ISA situations.
Only on ancient motherboards (I remember having it on some really old EISA machine) and non PC devices, no ?
I'm hoping the non PC users will add their perspective architecture again.
-Andi
Quoted 6 lines
Quoted 4 lines
Review
Quoted 2 lines
(quoted diff) +2
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Adaptec AHA152X/2825 support' CONFIG_SCSI_AHA152X $CONFIG_SCSI2825 is not ISA bus
What then ?
Vesa local bus
Quoted 1 line
But seems to be not 64bit safe and miss pci dma support (note the
CONFIG_X86_64).
My fault
Quoted 8 lines
Quoted 5 lines
drivers/scsi/Config.in if [ "$CONFIG_MCA" = "y" ]; then · line 103
Quoted 3 lines
drivers/scsi/Config.in +3
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Generic NCR5380/53c400 SCSI support' CONFIG_SCSI_GENERIC_NCR5380 $CONFIG_SCSI + if [ "$CONFIG_SCSI_GENERIC_NCR5380" != "n" ]; thenThis is used in multiple non ISA situations.
Only on ancient motherboards (I remember having it on some really old EISA machine) and non PC devices, no ?
On just about anything. If you have some old (or new) random weird box then so long as you know the address this works. NCR5380 macrocells are still being used I'm afraid to say, and attached to pretty much any bus people can find.
Alan
Quoted 8 lines · Alan Cox
Quoted 6 lines
Quoted 4 lines
Review
Quoted 2 lines
(quoted diff) +2
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Adaptec AHA152X/2825 support' CONFIG_SCSI_AHA152X $CONFIG_SCSI2825 is not ISA bus
What then ?
Vesa local bus
Ok, I'm assuming that there are no boxes with no ISA slots but VLB slots. I guess that's safe. If someone really has a weird box were that is not true I guess they'll have to live with defining CONFIG_ISA.
In theory one could introduce an CONFIG_VLB, but I don't think it is worth it.
Quoted 6 lines
Quoted 6 lines
Quoted 5 lines
drivers/scsi/Config.in if [ "$CONFIG_MCA" = "y" ]; then · line 103
Quoted 3 lines
drivers/scsi/Config.in +3
-0+if [ "$CONFIG_ISA" = "y" ]; then + dep_tristate 'Generic NCR5380/53c400 SCSI support' CONFIG_SCSI_GENERIC_NCR5380 $CONFIG_SCSI + if [ "$CONFIG_SCSI_GENERIC_NCR5380" != "n" ]; thenThis is used in multiple non ISA situations.
Quoted 7 lines
Quoted 2 lines
Only on ancient motherboards (I remember having it on some really old EISA machine) and non PC devices, no ?
On just about anything. If you have some old (or new) random weird box then so long as you know the address this works. NCR5380 macrocells are still being used I'm afraid to say, and attached to pretty much any bus people can find.
Ok thanks, will change it.
-Andi
Quoted 5 lines
Ok, I'm assuming that there are no boxes with no ISA slots but VLB slots. I guess that's safe. If someone really has a weird box were that is not true I guess they'll have to live with defining
CONFIG_ISA.
In theory one could introduce anCONFIG_VLB, but I don't think it is worth it.
Agreed
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Andi KleenSome personal thoughts from a Linux user. Seems like several possibilities for Eric's attitude...
1. waiting for people to see the need for CML2
2. waiting for people who said they could do it better to put up
3. totally disgusted by the attitude and behavior on the list
4. all of the above
I never got CML2 to work correctly and found Eric to act helpful but to not really help. It surprised me that he knew little about how people (me) actually used kernel config and build scripts.
Free software has to attract people...developers can't force it. CML2 required powerful, knowledgeable kernel developers to change and they didn't want to.
----- Original Message -----
From: "Tomas Szepe" <[email protected]>
To: "William Stearns" <[email protected]>
Cc: "Andi Kleen" <[email protected]>; "ML-linux-kernel" <[email protected]>
Sent: Thursday, May 09, 2002 4:08 PM
Subject: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
Quoted 13 lines
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Btw, what happened to CML2?
I haven't seen any updates since about February and haven't almost certainly stumbled upon a post from ESR for quite long either.T.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Uz.ytkownik Alan Cox napisa?:
Quoted 6 lines
Quoted 3 lines
Ok, I'm assuming that there are no boxes with no ISA slots but VLB slots.
I guess that's safe. If someone really has a weird box were that is not true
I guess they'll have to live with definingCONFIG_ISA. In theory one could introduce anCONFIG_VLB, but I don't think it is worth it.Agreed
In esp. in face of the fact that VLB was in reality merely the 486 CPU to memmory bus hanging out of the system... so it didn't have it's own protocol different from inb outb - which logically corresponds to ISA.
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to Andi KleenQuoted 4 lines · On Thu, May 09, 2002 at 10:49:28PM +0200, [email protected] wrote:
Quoted 3 lines · Tomas Szepe
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Quoted 8 lines
Quoted 3 lines
Btw, what happened to CML2?
I haven't seen any updates since about February and haven't almost certainly stumbled upon a post from ESR for quite long either.Eric was blasted away from the linux-kernel mailing list.
I understand that people tend to disagree about some subject every once in a while. But he certainly didn't deserve the treatment he got. Specially considering the enormous amount of work he put in it.
Perhaps that will be a lesson to the self-styled "Hacker of social systems" that there is a reason he has two ears and only one mouth.
As a low-evel hacker he needed to elicit support from others to get his work recognised and included. He was unwilling to listen to advice and criticism from those who know a lot better than him, and his work was not of such obvious brilliance that people were prepared to accept it without change.
He was not prepared to make the changes folks wanted, so he took his ball and went away in a strop.
Quoted 1 line
And I miss his signatures and the discussions about Aunt Tillie and stuff. :-)
I certainly don't miss his signatures. I considered a lot of them to be offensively off-topic and needlessly provocative.
Sean
Re: CML2 [was Re: PATCH & call for help: Marking ISA only drivers]
In reply to [email protected]Quoted 4 lines · On Thu, 9 May 2002 22:49:28 +0200 [email protected] wrote:
Quoted 3 lines · Tomas Szepe
Quoted 2 lines
I did quite a bit of this work for CML2 - bus dependencies can be found in the CML2 sources.
Quoted 8 lines
Quoted 3 lines
Btw, what happened to CML2?
I haven't seen any updates since about February and haven't almost certainly stumbled upon a post from ESR for quite long either.Eric was blasted away from the linux-kernel mailing list.
I understand that people tend to disagree about some subject every once in a while. But he certainly didn't deserve the treatment he got. Specially considering the enormous amount of work he put in it.
Agreed. I did a lot of testing and some grunt-level coding on CML2, and I think it addressed a lot of problems that existed, and was getting steadily better. Sure, it had some flaky spots, but I really don't see all of the problems that people had with it. Even autoconfig had this tendency to do the Right Thing. I think it was just about ready to start getting some wide testing and use, when everyone abandoned it... And now nobody wants to use it, because of course patches don't play nice with it, and etc... Pretty sad.
Oh well.