Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 10 Feb 2000 11:11:38 -0800 (PST)
From:      Doug Ambrisko <ambrisko@whistle.com>
To:        Soren Schmidt <sos@freebsd.dk>
Cc:        imp@village.org, freebsd-mobile@FreeBSD.ORG
Subject:   Re: Update to pccard ata changes (removal support)
Message-ID:  <200002101911.LAA91194@whistle.com>
In-Reply-To: <200002101840.TAA39117@freebsd.dk> from Soren Schmidt at "Feb 10, 2000 07:40:04 pm"

next in thread | previous in thread | raw e-mail | index | archive | help
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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200002101911.LAA91194>