commit cd70202a4b596be44f10e2ca775db4658472b83c results in performance loss

Bug #1781312 reported by Yi-y-yang on 2018-07-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
High
Yi-y-yang
Trunk
Fix Committed
High
Yi-y-yang
OpenContrail
New
Undecided
Unassigned

Bug Description

After commit 4db65eb46bacc78d264ac6239b665720cb93b3b8 is applied, the below commit will have very big side effect in performance, I can see about 16% performance boost for Tx-only test after I revert it.

commit cd70202a4b596be44f10e2ca775db4658472b83c
Author: Yi Yang <email address hidden>
Date: Fri Feb 23 11:18:32 2018 +0800

    Optimize VM Rx by batch processing

    Closes-bug: #1748080

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

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

Review in progress for https://review.opencontrail.org/44538
Submitter: Yuvaraja Mariappan

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

commit 7bb10b4b68ca912f41c12ce84557e397b7f6b804
Author: Yi Yang <email address hidden>
Date: Thu Jul 12 09:33:10 2018 +0800

Revert "Optimize VM Rx by batch processing"

This reverts commit cd70202a4b596be44f10e2ca775db4658472b83c.

Change-Id: Ifa3328342330f4d000d8391f91279921c44667dc
Closes-Bug: #1781312
Signed-off-by: Yi Yang <email address hidden>

OpenContrail Admin (ci-admin-f) wrote :

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

commit eb52ad2f66f014556e3d36e6c4c67bfa4475e875
Author: Yi Yang <email address hidden>
Date: Thu Jul 12 09:33:10 2018 +0800

Revert "Optimize VM Rx by batch processing"

This reverts commit cd70202a4b596be44f10e2ca775db4658472b83c.

Change-Id: Ifa3328342330f4d000d8391f91279921c44667dc
Closes-Bug: #1781312
Signed-off-by: Yi Yang <email address hidden>

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

Other bug subscribers