lkml.org 
[lkml]   [2002]   [Oct]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] [RFC] Advanced TCA SCSI Disk Hotswap
Date
From
> The locking most definately needs to be added to the kernel.  I'm
> surprised the original patch didn't contain any locking, but then
> again, my first patch didn't either:)

When I first read the SCSI code many years ago, I found surprise wasn't
adequate and I was forced to resort to astonishment. It's being cleaned up
slowly.

Originally hot removal/insertion was the exception, so nobody tripped over the
locking issue. Now it's fast becoming the rule.

> This may be true, but most systems will only have at most 4-5 devices.
>
> Theres only so much room on PCI for FC devices :)

I have to think about other SCSI systems as well. Some IBM beasties have >
256 PCI slots. Infiniband is threatening direct bus-fibre attachment.

> In Advanced TCA (what spawned this work) a button is pressed to
> indicate hotswap removal which makes for easy detection of hotswap
> events. This is why there are kernel interfaces for removal and
> insertion (so a kernel driver can be written to detect the button
> press and remove the devices from the os data structures and then
> light a blue led indicating safe for removal).

OK, I understand what's going on now. It's no different from those hotplug
PCI busses where you press the button and a second or so later the LED goes
out and you can remove the card. 10ms sounds rather a short maximum time for
a technician to wait for a light to go out....I suppose Telco technicians are
rather impatient.

I really think you need to lengthen this interval. The kernel is moving
towards this type of hotplug infrastructure which you can easily leverage (or
even help build), but it's definitely going to be mainly in user space.

James



-
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/

\
 
 \ /
  Last update: 2005-03-22 13:30    [W:0.066 / U:3.564 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site