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

Oliver Sang
2023-05-16T05:47:19Z

hi Shakeel,

On Mon, May 15, 2023 at 12:50:31PM -0700, Shakeel Butt wrote:
> +Feng, Yin and Oliver
>
> >
> > > Thanks a lot Cathy for testing. Do you see any performance improvement for
> > > the memcached benchmark with the patch?
> >
> > Yep, absolutely :- ) RPS (with/without patch) = +1.74
>
> Thanks a lot Cathy.
>
> Feng/Yin/Oliver, can you please test the patch at [1] with other
> workloads used by the test robot? Basically I wanted to know if it has
> any positive or negative impact on other perf benchmarks.

is it possible for you to resend patch with Signed-off-by?
without it, test robot will regard the patch as informal, then it cannot feed
into auto test process.
and could you tell us the base of this patch? it will help us apply it
correctly.

on the other hand, due to resource restraint, we normally cannot support
this type of on-demand test upon a single patch, patch set, or a branch.
instead, we try to merge them into so-called hourly-kernels, then distribute
tests and auto-bisects to various platforms.
after we applying your patch and merging it to hourly-kernels sccussfully,
if it really causes some performance changes, the test robot could spot out
this patch as 'fbc' and we will send report to you. this could happen within
several weeks after applying.
but due to the complexity of whole process (also limited resourse, such like
we cannot run all tests on all platforms), we cannot guanrantee capture all
possible performance impacts of this patch. and it's hard for us to provide
a big picture like what's the general performance impact of this patch.
this maybe is not exactly what you want. is it ok for you?

>
> [1] https://lore.kernel.org/all/20230512171702.923725-1-shakeelb@google.com/
>
> Thanks in advance.