Re: [ovs-dev] [PATCH net-next v7 1/1] net:openvswitch:reduce cpu_used_mask memory

2023-02-04 Thread Eddy Tao
Hi, Simon:     Thank you for the time on the review. and i looked into net folder and get various results 'net:', 'net: gre:', 'net: bridge:', 'net: thunderx', 'net: sock', 'net: genetlink', and there is also examples as you suggested like 'devlink:' similarly, in other folders i see

Re: [ovs-dev] [PATCH net-next v7 1/1] net:openvswitch:reduce cpu_used_mask memory

2023-02-04 Thread Simon Horman
On Fri, Feb 03, 2023 at 11:42:45PM +0800, Eddy Tao wrote: > Use actual CPU number instead of hardcoded value to decide the size > of 'cpu_used_mask' in 'struct sw_flow'. Below is the reason. > > 'struct cpumask cpu_used_mask' is embedded in struct sw_flow. > Its size is hardcoded to

[ovs-dev] [PATCH net-next v7 1/1] net:openvswitch:reduce cpu_used_mask memory

2023-02-03 Thread Eddy Tao
Use actual CPU number instead of hardcoded value to decide the size of 'cpu_used_mask' in 'struct sw_flow'. Below is the reason. 'struct cpumask cpu_used_mask' is embedded in struct sw_flow. Its size is hardcoded to CONFIG_NR_CPUS bits, which can be 8192 by default, it costs memory and slows down