Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 08 Sep 2005 10:35:11 -0600
From:      Scott Long <scottl@samsco.org>
To:        Nate Lawson <nate@root.org>
Cc:        Perforce Change Reviews <perforce@FreeBSD.org>, Scott Long <scottl@FreeBSD.org>
Subject:   Re: PERFORCE change 83251 for review
Message-ID:  <4320683F.1060001@samsco.org>
In-Reply-To: <43206729.9020001@root.org>
References:  <200509080737.j887bmMo009640@repoman.freebsd.org> <43206729.9020001@root.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Nate Lawson wrote:
> Scott Long wrote:
> 
>> http://perforce.freebsd.org/chv.cgi?CH=83251
>>
>> Change 83251 by scottl@scottl-junior on 2005/09/08 07:37:47
>>
>>     Import Danny Braniss' iSCSI initiator stack.
>>
>> Affected files ...
>>
>> .. //depot/projects/iscsi/blurb#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/isc_cam.c#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/isc_sm.c#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/isc_soc.c#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/isc_subr.c#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/iscsi.c#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/iscsi.h#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/iscsi_subr.c#1 add
>> .. //depot/projects/iscsi/src/sys/dev/iscsi/iscsivar.h#1 add
>> .. //depot/projects/iscsi/src/sys/modules/iscsi_initiator/Makefile#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/Makefile#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/auth.c#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/config.c#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/config.sample#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/fsm.c#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/iscontrol.c#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/iscontrol.h#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/pdu.c#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/pdu.h#1 add
>> .. //depot/projects/iscsi/src/usr.sbin/iscontrol/textkey.h#1 add
>> .. //depot/projects/scottl-camlock/src/sys/cam/cam_xpt.c#17 edit
>>
>> Differences ...
>>
>> ==== //depot/projects/scottl-camlock/src/sys/cam/cam_xpt.c#17 
>> (text+ko) ====
>>
>> @@ -1527,7 +1527,7 @@
>>          s = splsoftcam();
>>          camq_resize(&device->drvq, device->drvq.array_size - 1);
>>  
>> -        device->generation++;
>> +        atomic_add_int(&device->generation, 1);
>>  
>>          SLIST_REMOVE(periph_head, periph, cam_periph, periph_links);
>>  
> 
> 
> And a locking change.  :)
> 
> Nice about iSCSI.  Will having iscontrol in usr.sbin prevent mounting 
> /usr as an iSCSI partition?  How hard is it to do iSCSI root mounts?
> 

The current incarnation of this code requires iscontrol to do discovery
and initial connection.  Without it, you don't even get a /dev device.
So it's really only suitable for secondary storge.  At some point we'll
probably look at putting discovery into the kernel.

Scott



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