Francesco Romani has posted comments on this change.

Change subject: virt: add VmTracker to track domain traffic
......................................................................


Patch Set 16:

I understand the code's not brilliant and quite opaque actually (especially 
reserve_available() and friends).
We are in a hot path here, I'm afraid we'll have to sacrifice some readability 
on the altar of speed.
I'm very open to suggestions, however.

-- 
To view, visit https://gerrit.ovirt.org/44543
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iab735193ea7b2e177e146099843cc31706379e68
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to