Contrail-vrouter can't handle core ID which is bigger than 63

Bug #1748068 reported by Yi-y-yang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Undecided
Unassigned
OpenContrail
New
Undecided
Unassigned

Bug Description

In skylake platform, we have two sockets, each socket has 28 cores, if we enable hyperthread, it totals 112 logical cores, current contrail-vrouter can't handle core id bigger than 63, but for AT&T use case, we should be able to assign logical core 111 to contrail-vrouter, but it is impossible now.

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/39555
Submitter: Yi Yang (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/39555
Committed: http://github.com/Juniper/contrail-vrouter/commit/bc26ab2cb6c4f21b57c268a4a213b00a814cd12f
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit bc26ab2cb6c4f21b57c268a4a213b00a814cd12f
Author: Yi Yang <email address hidden>
Date: Fri Feb 23 09:58:45 2018 +0800

Fix maximum core id limitation

Currently vrouter used uint64_t to save core mask, so it only
can support maximum core id 63, it can support 128 or more if we
use cpu_set_t to save core mask, this patch fixes this limitation
by using cpu_set_t instead of uint64_t.

Closes-bug: #1748068

Change-Id: Id2374d3e21bd5406c4e4a065e54f9e989f84b8a9
Signed-off-by: Yi Yang <email address hidden>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.