| Message ID | 20211208141722.13646-1-shayd@nvidia.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | net/mlx5: Memory optimizations | expand |
On Wed, 2021-12-08 at 16:17 +0200, Shay Drory wrote: > This series provides knobs which will enable users to > minimize memory consumption of mlx5 Functions (PF/VF/SF). > mlx5 exposes two new generic devlink params for EQ size > configuration and uses devlink generic param max_macs. > > Patches summary: > - Patch-1 Introduce log_max_current_uc_list_wr_supported bit > - Patches-2-3 Provides I/O EQ size param which enables to save > up to 128KB. > - Patches-4-5 Provides event EQ size param which enables to save > up to 512KB. > - Patch-6 Clarify max_macs param. > - Patch-7 Provides max_macs param which enables to save up to 70KB > > In total, this series can save up to 700KB per Function. > > --- > changelog: > v2->v3: > - change type of EQ size param to u32 per Jiri suggestion. > - separate ifc changes to new patch > v1->v2: > - convert io_eq_size and event_eq_size from devlink_resources to > generic devlink_params Jakub are ok with this version ? I would like to take it to my trees.
On Mon, 13 Dec 2021 23:06:26 +0000 Saeed Mahameed wrote: > On Wed, 2021-12-08 at 16:17 +0200, Shay Drory wrote: > > This series provides knobs which will enable users to > > minimize memory consumption of mlx5 Functions (PF/VF/SF). > > mlx5 exposes two new generic devlink params for EQ size > > configuration and uses devlink generic param max_macs. > > > > Patches summary: > > - Patch-1 Introduce log_max_current_uc_list_wr_supported bit > > - Patches-2-3 Provides I/O EQ size param which enables to save > > up to 128KB. > > - Patches-4-5 Provides event EQ size param which enables to save > > up to 512KB. > > - Patch-6 Clarify max_macs param. > > - Patch-7 Provides max_macs param which enables to save up to 70KB > > > > In total, this series can save up to 700KB per Function. > > Jakub are ok with this version ? > I would like to take it to my trees. Yup, go for it. Sorry I didn't ack - I was going to apply it today but Dave reminded me that you probably want to take it thru your tree.
On Mon, 2021-12-13 at 17:01 -0800, Jakub Kicinski wrote: > On Mon, 13 Dec 2021 23:06:26 +0000 Saeed Mahameed wrote: > > On Wed, 2021-12-08 at 16:17 +0200, Shay Drory wrote: > > > This series provides knobs which will enable users to > > > minimize memory consumption of mlx5 Functions (PF/VF/SF). > > > mlx5 exposes two new generic devlink params for EQ size > > > configuration and uses devlink generic param max_macs. > > > > > > Patches summary: > > > - Patch-1 Introduce log_max_current_uc_list_wr_supported bit > > > - Patches-2-3 Provides I/O EQ size param which enables to save > > > up to 128KB. > > > - Patches-4-5 Provides event EQ size param which enables to save > > > up to 512KB. > > > - Patch-6 Clarify max_macs param. > > > - Patch-7 Provides max_macs param which enables to save up to > > > 70KB > > > > > > In total, this series can save up to 700KB per Function. > > > > Jakub are ok with this version ? > > I would like to take it to my trees. > > Yup, go for it. Sorry I didn't ack - I was going to apply it today > but > Dave reminded me that you probably want to take it thru your tree. Thanks ! applied. Will be sent in future PRs. First patch needs to go to mlx5-next shared branch with rdma. So this will be sent in two separate PRs.