Re: [ovs-dev] [PATCH] datapath-windows: Hot add CPU support.

2016-04-04 Thread Sorin Vinturis
Hi Paul, Your observation is correct. On 'KeProcessorAddStartNotify' the processor is not active yet and 'KeQueryActiveProcessorCountEx' would not add it to the count. All per-cpu data structures must be allocated on 'KeProcessorAddStartNotify', when the processor is not yet active. On 'KeProc

Re: [ovs-dev] [PATCH] datapath-windows: Hot add CPU support.

2016-04-01 Thread Paul Boca
Hi Sorin! The only issue I see here is in OvsCpuChange. This function is called twice for each processor added to the system, once with KeProcessorAddStartNotify and second with KeProcessorAddCompleteNotify or KeProcessorAddFailureNotify. You make reallocation on StartNotify and AddFailureNotify