On 05/18/2018 01:28 PM, Sergei Shtylyov wrote: > On 05/18/2018 02:37 PM, Pavel Tatashin wrote: > >> * parallelized this function, so drop lock for the >>> >>> Parallelizing? Else the sentence doesn't parse for me. :-) > > My comment hardly makes sense when you removed all the context...
Hi Sergei, Ah gotcha: + /* Suspend takes a long time, device_shutdown may be + * parallelized this function, so drop lock for the The comment should have been: ixgbe_close_suspend() takes a long time, and device_shutdown may parallelize ixgbe_shutdown(), therefore drop lock to allow concurrent execution of ixgbe_close_suspend(). Anyway, as I said, this patch should be dropped. Thank you, Pavel