Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-11-03 Thread Xuan Zhuo
On Wed, 30 Oct 2024 18:23:26 -0600, Caleb Sander Mateos wrote: > net_dim() is currently passed a struct dim_sample argument by value. > struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 > passes it on the stack. All callers have already initialized dim_sample > on the stack, s

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-11-03 Thread Caleb Sander
On Sun, Nov 3, 2024 at 12:21 PM Jakub Kicinski wrote: > > On Wed, 30 Oct 2024 18:23:26 -0600 Caleb Sander Mateos wrote: > > In a heavy TCP workload, mlx5e_handle_rx_dim() consumes 3% of CPU time, > > 94% of which is attributed to the first push instruction to copy > > dim_sample on the stack for t

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-11-03 Thread Jakub Kicinski
On Wed, 30 Oct 2024 18:23:26 -0600 Caleb Sander Mateos wrote: > In a heavy TCP workload, mlx5e_handle_rx_dim() consumes 3% of CPU time, > 94% of which is attributed to the first push instruction to copy > dim_sample on the stack for the call to net_dim(): Change itself looks fine, so we can apply,

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-11-01 Thread Vladimir Oltean
On Fri, Nov 01, 2024 at 10:54:47AM +0200, Louis Peens wrote: > Hi - sorry, I do still manage to mix up when to use signed-off-by and > reviewed-by. You use Signed-off-by when you submit a patch and Reviewed-by when you review it.

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-11-01 Thread Louis Peens
On Thu, Oct 31, 2024 at 10:19:55AM -0700, Caleb Sander wrote: > [Some people who received this message don't often get email from > csan...@purestorage.com. Learn why this is important at > https://aka.ms/LearnAboutSenderIdentification ] > > On Thu, Oct 31, 2024 at 5:49 AM Louis Peens wrote: >

RE: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-10-31 Thread Kiyanovski, Arthur
> -Original Message- > From: Caleb Sander Mateos > Sent: Wednesday, October 30, 2024 5:23 PM > > net_dim() is currently passed a struct dim_sample argument by value. > struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 passes > it > on the stack. All callers have alrea

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-10-31 Thread Louis Peens
On Wed, Oct 30, 2024 at 06:23:26PM -0600, Caleb Sander Mateos wrote: > net_dim() is currently passed a struct dim_sample argument by value. > struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 > passes it on the stack. All callers have already initialized dim_sample > on the stac

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-10-31 Thread Florian Fainelli
On 10/30/24 17:23, Caleb Sander Mateos wrote: net_dim() is currently passed a struct dim_sample argument by value. struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 passes it on the stack. All callers have already initialized dim_sample on the stack, so passing it by value req

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-10-31 Thread Caleb Sander
On Thu, Oct 31, 2024 at 5:49 AM Louis Peens wrote: > > On Wed, Oct 30, 2024 at 06:23:26PM -0600, Caleb Sander Mateos wrote: > > net_dim() is currently passed a struct dim_sample argument by value. > > struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 > > passes it on the stack.

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-10-31 Thread Nelson, Shannon
On 10/30/2024 5:23 PM, Caleb Sander Mateos wrote: net_dim() is currently passed a struct dim_sample argument by value. struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 passes it on the stack. All callers have already initialized dim_sample on the stack, so passing it by valu

Re: [resend PATCH 2/2] dim: pass dim_sample to net_dim() by reference

2024-10-31 Thread Vladimir Oltean
On Wed, Oct 30, 2024 at 06:23:26PM -0600, Caleb Sander Mateos wrote: > net_dim() is currently passed a struct dim_sample argument by value. > struct dim_sample is 24 bytes. Since this is greater 16 bytes, x86-64 > passes it on the stack. All callers have already initialized dim_sample > on the stac