From owner-freebsd-mobile Tue Jul 20 11:52:33 1999 Delivered-To: freebsd-mobile@freebsd.org Received: from ns.mt.sri.com (ns.mt.sri.com [206.127.79.91]) by hub.freebsd.org (Postfix) with ESMTP id 0082C15380 for ; Tue, 20 Jul 1999 11:52:29 -0700 (PDT) (envelope-from nate@mt.sri.com) Received: from mt.sri.com (rocky.mt.sri.com [206.127.76.100]) by ns.mt.sri.com (8.8.8/8.8.8) with SMTP id MAA19028; Tue, 20 Jul 1999 12:52:25 -0600 (MDT) (envelope-from nate@rocky.mt.sri.com) Received: by mt.sri.com (SMI-8.6/SMI-SVR4) id MAA20790; Tue, 20 Jul 1999 12:52:24 -0600 Date: Tue, 20 Jul 1999 12:52:24 -0600 Message-Id: <199907201852.MAA20790@mt.sri.com> From: Nate Williams MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit To: Warner Losh Cc: Nate Williams , hosokawa@itc.keio.ac.jp (HOSOKAWA Tatsumi), mobile@FreeBSD.ORG Subject: Re: usr.sbin/pccard/pccardd change for "cardio" and "cardmem" In-Reply-To: <199907201754.LAA87429@harmony.village.org> References: <199907201518.JAA19388@mt.sri.com> <199907201331.WAA22907@afs.ntc.mita.keio.ac.jp> <199907201754.LAA87429@harmony.village.org> X-Mailer: VM 6.34 under 19.16 "Lille" XEmacs Lucid Sender: owner-freebsd-mobile@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org > In message <199907201518.JAA19388@mt.sri.com> Nate Williams writes: > : Can you be more specific? When are these used? It has been my > : experience that too often 'broken cards' are in fact not broken, and the > : FreeBSD code is broken. Adding workarounds for (potentially) broken > : code is not a good fix. > > Assuming for the moment that pccardd's cis reading is somehow > inadequate, I think that this is a good enough fix. Fix for what? What is the fix for? Just adding keywords to fix 'buggy' cards is not a fix, it's (potentially) a hack to workaround bogus code in other parts of the system that might be easily fixed. Many of the 'fixes' in PAO were *easily* fixed with just a bit of time with the 'bad' cards. Making the already difficult to understand code bigger and more complex with fixes for broken existing code is just silly. Now, I'm not saying that this is the case, but in the past this was very often the case, so allow me a bit of conservatism in not allowing every new feature to fix 'broken code'. :) Nate ps. I agree that there are fixes in the patch, but the addition of the keywords are not fixes. In particule, the 'release' of IRQ is a fix, as I pointed out in the original email. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-mobile" in the body of the message