Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting the size of bpf maps in Kmesh #318

Open
LiZhenCheng9527 opened this issue May 11, 2024 · 3 comments
Open

Setting the size of bpf maps in Kmesh #318

LiZhenCheng9527 opened this issue May 11, 2024 · 3 comments
Milestone

Comments

@LiZhenCheng9527
Copy link
Collaborator

What would you like to be added:
In Kmesh, we declare a number of bpf maps and set their size. However, the previous settings were very confusing and the sizes did not reasonably.

#define MAP_SIZE_OF_PER_LISTENER 64
#define MAP_SIZE_OF_PER_FILTER_CHAIN 4
#define MAP_SIZE_OF_PER_FILTER 4
#define MAP_SIZE_OF_PER_VIRTUAL_HOST 16
#define MAP_SIZE_OF_PER_ROUTE 8
#define MAP_SIZE_OF_PER_CLUSTER 32
#define MAP_SIZE_OF_PER_ENDPOINT 64
#define MAP_SIZE_OF_MANAGER 8192

We need to discuss the size of the bpf maps according to the actual usage story. Make the size setting more reasonable
Why is this needed:

@LiZhenCheng9527
Copy link
Collaborator Author

We can discuss this at the next community meeting

@hzxuzhonghu
Copy link
Member

And for worklaod mode, most are two small, while MAP_SIZE_OF_AUTH maybe too big

#define MAP_SIZE_OF_FRONTEND 100
#define MAP_SIZE_OF_SERVICE  100
#define MAP_SIZE_OF_ENDPOINT 1000
#define MAP_SIZE_OF_BACKEND  500
#define MAP_SIZE_OF_AUTH     8192
#define MAP_SIZE_OF_MANAGER  8192
#define MAP_SIZE_OF_DSTINFO  8192

@LiZhenCheng9527
Copy link
Collaborator Author

total service mesh:
services 5000
workloads100000

@hzxuzhonghu hzxuzhonghu added this to the v0.4 milestone Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants