<div dir="ltr">Hi Jiri,<div><br></div><div>Sorry about the late response, only now I managed to push iSER into QEMU.</div><div>Because ISER is registered as a different protocol than iSCSI with the prefix of iser:// I want to add support for it in libvirt.</div><div><br></div><div>Libvirt code is pretty new for me, wondered if adding VIR_STORAGE_NET_PROTOCOL_ISER as another virStorageNetProtocol should be enough? </div><div>Of course I added ISER in all necessary switch-case in code also.</div><div><br></div><div>Thanks,</div><div>Roy</div><div> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 22, 2016 at 2:56 PM, Jiri Denemark <span dir="ltr"><<a href="mailto:jdenemar@redhat.com" target="_blank">jdenemar@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Mar 22, 2016 at 14:21:52 +0200, Roy Shterman wrote:<br>
> Correct me if I'm wrong but locked option is pinning all VM memory in host<br>
> RAM,<br>
><br>
> for example if I have a VM with 4G memory, and I want to run some QEMU code<br>
> which needs to pin 500M,<br>
><br>
> I will need to lock all 4G in host memory instead of locking only 500M.<br>
<br>
</span>So the question is which code wants to lock part of the memory, why, and<br>
if it's something that can be influenced by user.<br>
<br>
For example, we know that if you ask for all memory to by locked, we<br>
need to set the limit. The same applies when RDMA migration is started.<br>
On PPC we know some amount of memory will always need to be locked, we<br>
compute the amount and set the limit accordingly. We can't really expect<br>
user to have deep knowledge of QEMU and know what limit needs to be set<br>
when they use a specific device, QMP command, or whatever. So if the<br>
limit is something predictable and deterministic, we can automatically<br>
compute the amount of memory and use it when starting QEMU. Forcing<br>
users to set the limit when all memory needs to be locked is already bad<br>
enough that I don't think we should add a new option to explicitly set<br>
arbitrary lock limit.<br>
<br>
Jirka<br>
</blockquote></div><br></div>