[ovs-dev] [PATCH 0/7] Introduce high resolution sleep support.

Alin Serdean aserdean at cloudbasesolutions.com
Tue Nov 14 03:27:38 UTC 2017


> -----Original Message-----
> From: ovs-dev-bounces at openvswitch.org [mailto:ovs-dev-
> bounces at openvswitch.org] On Behalf Of Ben Pfaff
> Sent: Wednesday, November 8, 2017 11:46 PM
> To: Bodireddy, Bhanuprakash <bhanuprakash.bodireddy at intel.com>
> Cc: dev at openvswitch.org
> Subject: Re: [ovs-dev] [PATCH 0/7] Introduce high resolution sleep support.
> 
> On Wed, Nov 08, 2017 at 08:54:11PM +0000, Bodireddy, Bhanuprakash wrote:
> > HI Ben,
> > >On Wed, Nov 08, 2017 at 04:35:52PM +0000, Bhanuprakash Bodireddy
> wrote:
> > >> This patchset introduces high resolution sleep support for linux
> > >> and
> > >windows.
> > >> Also time macros are introduced to replace the numbers with
> > >> meaningful names.
> > >
> > >Thank you very much for the series.
> > >
> > >Did you test that the Windows version of the code compiles (e.g. via
> > >appveyor)?
> >
> > I cross checked with appveyor and the build was successful. I replied to
> another thread where we were discussing about the windows
> implementation.
> >
> > >
> > >I would normally squash patch 3 (the Windows version of xnanosleep)
> > >into patch 2 (the Linux version).
> >
> > I couldn't verify the functionality of windows implementation and hence
> posted it as a separate patch for now.
> > I will squash it once I receive feedback from Alin.
> 
> OK.  Once we have feedback from Alin, I think that v2 of the series will be
> ready to go.  Thanks again!
[Alin Serdean] Sorry for the delay.
I sent out my review on the patch: https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340889.html. 
I'll try to run some benchmarks on the accuracy when I have time.


More information about the dev mailing list