Peter V. Saveliev has posted comments on this change.
Change subject: BZ#785717 caps: cpuCores should report only physical cores ......................................................................
Patch Set 1: Verified; Looks good to me, but someone else must approve
(1 inline comment)
code works, code snippet checked.
.................................................... File vdsm/caps.py Line 76: return len(set( (p['core id'], p['physical id']) for p should it return only *physical* cores number, not logical cores?
ok.
(btw, in bugzilla I've got «You are not authorized to access bug #785717»
-- To view, visit http://gerrit.ovirt.org/1737 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I7b28d0f7864a351778c56048cb1233ed58807528 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Peter V. Saveliev peet@redhat.com