<div dir="ltr">Hello, Shlomi.<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 25, 2014 at 7:07 PM, Shlomi Sasson <span dir="ltr"><<a href="mailto:shlomis@mellanox.com" target="_blank">shlomis@mellanox.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div lang="EN-US" link="blue" vlink="purple"><div>
<p class="MsoNormal">I want to share with the community the following challenge:<u></u><u></u></p>
<p class="MsoNormal">Currently, Vendors who have their iSCSI driver, and want to add RDMA transport (iSER), cannot leverage their existing plug-in which inherit from iSCSI<u></u><u></u></p>
<p class="MsoNormal">And must modify their driver or create an additional plug-in driver which inherit from iSER, and copy the exact same code.<u></u><u></u></p>
<p class="MsoNormal"><span style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal">Instead I believe a simpler approach is to add a new attribute to ISCSIDriver to support other iSCSI transports besides TCP, which will allow minimal changes to support iSER.<u></u><u></u></p>
<p class="MsoNormal">The existing ISERDriver code will be removed, this will eliminate significant code and class duplication, and will work with all the iSCSI vendors who supports both TCP and RDMA without the need to modify their plug-in drivers.</p>

<p class="MsoNormal"><u></u></p></div></div></blockquote></div><br>I remember Ann working on <a href="https://review.openstack.org/#/c/45393">https://review.openstack.org/#/c/45393</a> and it has landed since.</div><div class="gmail_extra">

<br></div><div class="gmail_extra">That change leaves ISERDriver just for backward compatibility and allows ISCSIDriver and any its descendant to use iscsi_helper='iseradm' to provide iSER usage.</div><div class="gmail_extra">

<br></div><div class="gmail_extra" style>Aren't those changes enough for this? What else is needed here?</div><div class="gmail_extra"><div><br></div>-- <br><br><div>Kind regards, Yuriy.</div>
</div></div>