Nir Soffer has posted comments on this change.

Change subject: block: simplify zeroImgVolumes using threads
......................................................................


Patch Set 3: Code-Review-1

Don't we have enough threads?!

What is wrong with the current code? This code is exactly what we should be 
using everywhere in vdsm instead of starting zillions of threads.

For example, we could monitor all storage domains using *one* threads, waiting 
on multiple dd processes.

-- 
To view, visit http://gerrit.ovirt.org/35630
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I665d863085842de8ff93c9aacf7c26277dfb031d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to