Roy Golan has posted comments on this change. Change subject: refine calculation of cpu topology ......................................................................
Patch Set 9: (1 comment) http://gerrit.ovirt.org/#/c/30896/9/vdsm/rpc/vdsmapi-schema.json File vdsm/rpc/vdsmapi-schema.json: Line 3415: '*cpuShares': 'str', '*custom': 'StringMap', '*devices': ['VmDevice'], Line 3416: 'display': 'VmDisplayType', '*hypervEnable': 'bool', Line 3417: 'kvmEnable': 'bool', 'memSize': 'uint', Line 3418: 'nice': 'int', 'smp': 'uint', '*smpCoresPerSocket': 'uint', Line 3419: '*maxNumberOfSockets': 'int', > but add it after smpThreadsPerCore to keep the description order. I'm confused. where to add it? Line 3420: '*smpThreadsPerCore': 'uint', 'timeOffset': 'uint', Line 3421: 'transparentHugePages': 'bool', 'vmId': 'UUID', 'vmName': 'str', Line 3422: 'vmType': 'VmType', '*launchPaused': 'bool', Line 3423: 'numaTune': 'NumaTune', 'guestNumaNodes': ['GuestNumaNode']}} -- To view, visit http://gerrit.ovirt.org/30896 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0d71dfe98b67440c084b2a7bf8b292f6b9c3ae19 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Roy Golan <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
