Messages in this thread | | | Date | Mon, 27 Mar 2006 13:42:15 +0200 (CEST) | From | Bodo Eggert <> | Subject | Re: [PATCH] Move SG_GET_SCSI_ID from sg to scsi |
| |
On Sun, 26 Mar 2006, Linus Torvalds wrote: > On Sun, 26 Mar 2006, Bodo Eggert wrote:
> > Having a SCSI ID is a generic SCSI property > > No it's not.
With respect to sg vs. scsi, it is.
> Havign a SCSI ID is a f*cking idiotic thing to do. > > Only idiots like Joerg Schilling thinks that any such thing even _exists_. > It does not, never has, and never will.
For the host ID ACK, but some transports do support channel/ID/LUN. In /these/ cases having them exported is a sane thing to do. I don't even try to argue if it's better to use sysfs or ioctl.
> The way you reach a SCSI device is through the device filename, and trying > to use controller/channel/id/lun naming IS INSANE!
ACK, and having an address space of 340 * 10^36 will make it perfectly clear. One day a subsystem might hand out ID and LUN cookies instead of counters, and a certain table of sg numbers will stop fitting in memory. I'm just handing out the bigger gun with the automatic foot-finder.-)
</chasing the pink herring>
> Stop it now. We should kill that ioctl, not try to make it look like it is > sensible. It's not a sensible way to look up SCSI devices, and the fact > that some SCSI people think it is is doesn't make it so.
Schedule for removal?
As I understand, having a /dev/sd0815 corresponding to a /dev/sg4711 is the root of all evil and idiotic things, and moving all sg functions into the apropiate place is the sane thing to do.
This patch is a part of the process, and as soon as all sg functions are available using /dev/s[dtr]*, the corresponding sg devices should be deprecated and removed.
-- Those who desire to give up Freedom in order to gain Security, will not have, nor do they deserve, either one. -- T. Jefferson - 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/
| |