Re: [PATCH net-next 1/2] net: Keep sk->sk_forward_alloc as a proper size

Shakeel Butt
2023-05-09T18:17:56Z

On Tue, May 9, 2023 at 11:04 AM Chen, Tim C <tim.c.chen@intel.com> wrote:
>
> >>
> >> Run memcached with memtier_benchamrk to verify the optimization fix. 8
> >> server-client pairs are created with bridge network on localhost,
> >> server and client of the same pair share 28 logical CPUs.
> >>
> > >Results (Average for 5 run)
> > >RPS (with/without patch) +2.07x
> > >
>
> >Do you have regression data from any production workload? Please keep in mind that many times we (MM subsystem) accepts the regressions of microbenchmarks over complicated optimizations. So, if there is a real production regression, please be very explicit about it.
>
> Though memcached is actually used by people in production. So this isn't an unrealistic scenario.
>

Yes, memcached is used in production but I am not sure anyone runs 8
pairs of server and client on the same machine for production
workload. Anyways, we can discuss, if needed, about the practicality
of the benchmark after we have some impactful memcg optimizations.

> Tim