I applied this patch as-is. We still need to figure out the issue around tskill.

On Tue, Jun 23, 2015 at 7:05 AM, Gurucharan Shetty <shet...@nicira.com> wrote:
> On Mon, Jun 22, 2015 at 5:21 PM, Alin Serdean
> <aserd...@cloudbasesolutions.com> wrote:
>> I could send another patch with the following:
>> -                        tskill $i
>> +                        taskkill //PID $i //F >/dev/null
>>
>> Alin.
>
> Your current patch is something that I think we should apply because
> from Windows' perspective it does a clean kill instead of the force
> kill.
>
> Do you know why your system does not have tskill? The code originally
> had taskkill instead of tskill. One thing I observed was that taskkill
> cannot kill deadlocked processes and unit tests would hang whenever
> processes deadlock. That is the reason I changed it to tskill.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to