Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-28 Thread Saurabh Singh Sengar
On Sun, Jul 28, 2024 at 02:06:41PM +, Michael Kelley wrote: > From: Saurabh Singh Sengar Sent: Sunday, July > 28, 2024 2:18 AM > > > > On Sun, Jul 28, 2024 at 04:32:23AM +, Michael Kelley wrote: > > > From: Saurabh Sengar Sent: Wednesday, July > > > 24, > > 2024 10:26 PM > > > > > > >

RE: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-28 Thread Michael Kelley
From: Saurabh Singh Sengar Sent: Sunday, July 28, 2024 2:18 AM > > On Sun, Jul 28, 2024 at 04:32:23AM +, Michael Kelley wrote: > > From: Saurabh Sengar Sent: Wednesday, July 24, > 2024 10:26 PM > > > > > > Currently on a very large system with 1780 CPUs, hv_acpi_init takes > > > around 3 se

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-28 Thread Saurabh Singh Sengar
On Sun, Jul 28, 2024 at 04:32:23AM +, Michael Kelley wrote: > From: Saurabh Sengar Sent: Wednesday, July 24, > 2024 10:26 PM > > > > Currently on a very large system with 1780 CPUs, hv_acpi_init takes > > around 3 seconds to complete for all the CPUs. This is because of > > sequential synic

RE: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-27 Thread Michael Kelley
From: Saurabh Sengar Sent: Wednesday, July 24, 2024 10:26 PM > > Currently on a very large system with 1780 CPUs, hv_acpi_init takes > around 3 seconds to complete for all the CPUs. This is because of > sequential synic initialization for each CPU. > > Defer these tasks so that each CPU execute

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-27 Thread Srivatsa S. Bhat
: Thursday, July 25, 2024 8:35 AM > > > > Subject: Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks > > > > > > Without the patch, I think the current CPU uses IPIs to let the other > > > CPUs, one by one, run the function calls, and synchronously waits > > &g

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-26 Thread Saurabh Singh Sengar
On Fri, Jul 26, 2024 at 06:34:53AM +, Srivatsa S. Bhat wrote: > On Fri, Jul 26, 2024 at 12:01:33AM +, Dexuan Cui wrote: > > > From: Saurabh Singh Sengar > > > Sent: Thursday, July 25, 2024 8:35 AM > > > Subject: Re: [PATCH] Drivers: hv: vmbus: Deferring per

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-25 Thread Srivatsa S. Bhat
On Fri, Jul 26, 2024 at 12:01:33AM +, Dexuan Cui wrote: > > From: Saurabh Singh Sengar > > Sent: Thursday, July 25, 2024 8:35 AM > > Subject: Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks > > Without the patch, I think the current CPU uses IPIs to let the

RE: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-25 Thread Dexuan Cui
> From: Saurabh Singh Sengar > Sent: Thursday, July 25, 2024 10:27 PM > [...] > > > + ret = __cpuhp_setup_state_cpuslocked(CPUHP_AP_ONLINE_DYN, > > > "hyperv/vmbus:online", false, > > > + hv_synic_init, hv_synic_cleanup, > > > false); > > > + cpus_read_unlock()

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-25 Thread Saurabh Singh Sengar
On Fri, Jul 26, 2024 at 12:01:33AM +, Dexuan Cui wrote: > > From: Saurabh Singh Sengar > > Sent: Thursday, July 25, 2024 8:35 AM > > Subject: Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks > > Without the patch, I think the current CPU uses IPIs to let the

RE: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-25 Thread Dexuan Cui
> From: Saurabh Singh Sengar > Sent: Thursday, July 25, 2024 8:35 AM > Subject: Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks Without the patch, I think the current CPU uses IPIs to let the other CPUs, one by one, run the function calls, and synchronously waits for the funct

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-25 Thread Saurabh Singh Sengar
On Thu, Jul 25, 2024 at 08:23:21AM -0700, Nuno Das Neves wrote: > On 7/24/2024 10:26 PM, Saurabh Sengar wrote: > > Currently on a very large system with 1780 CPUs, hv_acpi_init takes > > around 3 seconds to complete for all the CPUs. This is because of > > sequential synic initialization for each C

Re: [PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-25 Thread Nuno Das Neves
On 7/24/2024 10:26 PM, Saurabh Sengar wrote: > Currently on a very large system with 1780 CPUs, hv_acpi_init takes > around 3 seconds to complete for all the CPUs. This is because of > sequential synic initialization for each CPU. > > Defer these tasks so that each CPU executes hv_acpi_init in par

[PATCH] Drivers: hv: vmbus: Deferring per cpu tasks

2024-07-24 Thread Saurabh Sengar
Currently on a very large system with 1780 CPUs, hv_acpi_init takes around 3 seconds to complete for all the CPUs. This is because of sequential synic initialization for each CPU. Defer these tasks so that each CPU executes hv_acpi_init in parallel to take full advantage of multiple CPUs. This so