Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

Conversation

@zzxgzgz
Copy link

@zzxgzgz zzxgzgz commented Jan 31, 2022

That this PR does:

  1. Integrated with ACA's packet_in code path.
  2. Added throttling to packet_in sending.

Building and Usage:
Please refer to this link and this link

@xieus xieus requested a review from lfu-ps January 31, 2022 20:24
@xieus xieus added the enhancement New feature or request label Jan 31, 2022
Copy link

@xieus xieus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zzxgzgz Two comments:
(1) could you highlight the throttling part as this is the main payload;
(2) does Cbench have some CI workflow we could leverage so that we could avoid missing compile error?

@zzxgzgz
Copy link
Author

zzxgzgz commented Jan 31, 2022

@zzxgzgz Two comments: (1) could you highlight the throttling part as this is the main payload; (2) does Cbench have some CI workflow we could leverage so that we could avoid missing compile error?

  1. I updated the code in cbench.c so the throtlling part can be found when you search for the word throttling.
  2. They don't have any CICD that I'm aware of, that is an old repo and they stopped updating it about 6 years ago.

@xieus xieus merged commit e1edd91 into futurewei-cloud:master Jan 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants