From owner-freebsd-mobile Thu Feb 10 11:12:24 2000 Delivered-To: freebsd-mobile@freebsd.org Received: from alpo.whistle.com (alpo.whistle.com [207.76.204.38]) by builder.freebsd.org (Postfix) with ESMTP id 69E8E44E2 for ; Thu, 10 Feb 2000 11:12:21 -0800 (PST) Received: from whistle.com (crab.whistle.com [207.76.205.112]) by alpo.whistle.com (8.9.1a/8.9.1) with ESMTP id LAA17779; Thu, 10 Feb 2000 11:12:01 -0800 (PST) Received: (from ambrisko@localhost) by whistle.com (8.9.3/8.9.1) id LAA91194; Thu, 10 Feb 2000 11:11:38 -0800 (PST) (envelope-from ambrisko) From: Doug Ambrisko Message-Id: <200002101911.LAA91194@whistle.com> Subject: Re: Update to pccard ata changes (removal support) In-Reply-To: <200002101840.TAA39117@freebsd.dk> from Soren Schmidt at "Feb 10, 2000 07:40:04 pm" To: Soren Schmidt Date: Thu, 10 Feb 2000 11:11:38 -0800 (PST) Cc: imp@village.org, freebsd-mobile@FreeBSD.ORG X-Mailer: ELM [version 2.4ME+ PL61 (25)] MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-freebsd-mobile@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org Soren Schmidt writes: [Charset ISO-8859-1 unsupported, filtering to ASCII...] | It seems Doug Ambrisko wrote: | > Caveats, the devices increment each time they are inserted and removed. | > Also I do not checks for mounted devices etc. yet. I'm not sure what | > the details are to deal with this (since as the card is being ripped out | > there won't be time to sync the device). So your machine might panic. | > I don't know and haven't tested this. I also need to look at the | > incrementing device issue. I thought it would be usefull to get this | > code out before I start to investigate that. | | ARGH! this will loose memory allocated in the subdrivers, also inc'ing | the device # are a no go, you will run out of softc's... | I have thought about this, and when I get a spare hour I'll implement | it... I agree but it atleast it has code for the controller to come and go and that part worked. A few people have asked for this feature so I thought I'd try it and get comments/help. So with this you don't have to reboot after swapping cards around. The major change was the newbus resources & id's need to be saved so it could be unregistered from newbus on detach. Also another problem is probably the incrementing lun's for the the controller. It should scan for the first available one in the ata_devices array after (0 or 1 since they seems to be special). | However find below my version of your previous patch set, please let | me know how it works (I have no way of testing it here)... | There are a few other cosmetic changes in there, this is part of | the next update, and not all was easily pulled out... I tested with my external CD-Rom & hard disk (with and with an internal IDE drive attached) and it worked fine. To save you time I could add the controller removal part to your patches so you have working code for that part? Thanks for looking at it, Doug A. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-mobile" in the body of the message