From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrey Smetanin Subject: Re: [PATCH v2 3/7] kvm/x86: Hyper-V unify stimer_start() and stimer_restart() Date: Thu, 7 Jan 2016 23:15:43 +0300 Message-ID: <568EC76F.1070309@virtuozzo.com> References: <1451316444-31766-1-git-send-email-asmetanin@virtuozzo.com> <1451316444-31766-4-git-send-email-asmetanin@virtuozzo.com> <568E9504.30900@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Cc: Gleb Natapov , Roman Kagan , "Denis V. Lunev" , To: Paolo Bonzini , Return-path: Received: from mx2.parallels.com ([199.115.105.18]:49516 "EHLO mx2.parallels.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752073AbcAGUPy (ORCPT ); Thu, 7 Jan 2016 15:15:54 -0500 In-Reply-To: <568E9504.30900@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 01/07/2016 07:40 PM, Paolo Bonzini wrote: > > > On 28/12/2015 16:27, Andrey Smetanin wrote: >> This will be used in future to start Hyper-V SynIC timer >> in several places by one logic in one function. >> >> Changes v2: >> * drop stimer->count == 0 check inside stimer_start() >> * comment stimer_start() assumptions > > Can you replace comments with WARN_ON_ONCE? Yes, I can, ETA 12 Jan since i'm on vacation. > > Paolo >