<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">
<META content="MSHTML 6.00.2900.2963" name=GENERATOR></HEAD>
<BODY>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN class=554084715-27102006>Hi
Yossi,</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006></SPAN></FONT> </DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN class=554084715-27102006>Please
review the following patch.</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006></SPAN></FONT><FONT face=Arial color=#0000ff
size=2><SPAN class=554084715-27102006>It makes sense in case if IOC
provides more than one service entry.</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN class=554084715-27102006>From
SRP perspective</SPAN></FONT><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006> it is just another session (</SPAN></FONT><FONT
face=Arial color=#0000ff size=2><SPAN class=554084715-27102006> indexed by
target id value )</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006>to another target device
with given LUN.</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006></SPAN></FONT> </DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006>Thanks,</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006>Alex</SPAN></FONT></DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN
class=554084715-27102006></SPAN></FONT> </DIV>
<DIV><FONT face=Arial color=#0000ff size=2><SPAN class=554084715-27102006>Index:
srp_data_path.c<BR>===================================================================<BR>---
srp_data_path.c (revision 525)<BR>+++ srp_data_path.c (working
copy)<BR>@@ -57,7 +57,7 @@<BR> //#include
"srp_t_logout.h"<BR> <BR> // Final address is of the form
0b00ttttttllllllll<BR>-#define BUILD_SCSI_ADDRESS(target, lun)
((((uint64_t)(target & 0x3f)<<8)+(uint64_t)lun) <<
48)<BR>+#define BUILD_SCSI_ADDRESS(lun) ((uint64_t)lun <<
48)<BR> <BR> static inline<BR> void<BR>@@ -587,7 +587,7
@@<BR>
DBDF_NO_DATA_BUFFER_DESCRIPTOR_PRESENT,<BR>
0,<BR>
0,<BR>- BUILD_SCSI_ADDRESS(
p_srb->TargetId, p_srb->Lun
),<BR>+ BUILD_SCSI_ADDRESS(
p_srb->Lun ),<BR>
TAV_SIMPLE_TASK,<BR> 0
);<BR> <BR>@@ -899,7 +899,7 @@<BR> <BR> setup_srp_tsk_mgmt(
p_srp_tsk_mgmt,<BR> p_send_descriptor->tag,<BR>- BUILD_SCSI_ADDRESS(
p_srb->TargetId, p_srb->Lun
),<BR>+ BUILD_SCSI_ADDRESS( p_srb->Lun
),<BR> TMF_ABORT_TASK,<BR> iu_tag
);<BR> </SPAN></FONT></DIV></BODY></HTML>