| Date | Tue, 07 May 2002 14:16:04 +0100 | From | Anton Altaparmakov <> | Subject | Re: [PATCH] 2.5.14 IDE 57 |
| |
At 12:27 07/05/02, Martin Dalecki wrote: >Tue May 7 02:37:49 CEST 2002 ide-clean-57 > >Nuke /proc/ide. For explanations why, please see the frustrated comments >in the previous change log.
This is a big mistake IMO.
Nuking the ability to change settings, fair enough, but only if alternative interface is provided for userspace to tweak everything, otherwise provide the interface before you remove the existing one. (There may be already another interface, I don't know...I am sure someone will tell me if there is!)
Removing the information provided by /proc/ide is very bad! It is very useful to diagnose one's ide setup, to see what the host is configured as, what all settings are set to, etc. This is the first place I look to check whether the interfaces are configured as I expect them to be and in case of problems, this is again the first place I look.
What alternatives are you going to present to give all the information that /proc/ide gives? If the answer is none IMHO your patch is not acceptable...
Best regards,
Anton
-- "I've not lost my mind. It's backed up on tape somewhere." - Unknown -- Anton Altaparmakov <aia21 at cantab.net> (replace at with @) Linux NTFS Maintainer / IRC: #ntfs on irc.openprojects.net WWW: http://linux-ntfs.sf.net/ & http://www-stu.christs.cam.ac.uk/~aia21/
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|