<div dir="ltr">No problem to merge them, thank you.</div><div class="gmail_extra"> <div class="gmail_quote">On Mon, Apr 3, 2017 at 1:36 PM, Frediano Ziglio <<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> > From: "<a href="mailto:yuri.benditovich@daynix.com">yuri.benditovich@daynix.com</a>" <<a href="mailto:yuri.benditovich@daynix.com">yuri.benditovich@daynix.com</a>> > > Default indication period (13-17ms) from the driver to OS > increases total CPU consumption during presentation operations. > Reducing indication frequency to 200 ms reduces cost of this > feature. Till now we do not see any negative impact of this. > > Signed-off-by: Yuri Benditovich <<a href="mailto:yuri.benditovich@daynix.com">yuri.benditovich@daynix.com</a>> Till 7/12 I ack them. I have additional consideration/comments for the next ones. Can I merge till 7/12 or do you want to wait for all of them? Frediano <div class="HOEnZb"><div class="h5"> > --- > qxldod/QxlDod.cpp | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp > index 7341389..aa70f39 100755 > --- a/qxldod/QxlDod.cpp > +++ b/qxldod/QxlDod.cpp > @@ -19,6 +19,7 @@ > (QXL_INTERRUPT_CURSOR) | \ > (QXL_INTERRUPT_IO_CMD)) > > +#define VSYNC_PERIOD 200 // ms, use 0 for auto > #define VSYNC_RATE 75 > > BOOLEAN g_bSupportVSync; > @@ -5006,8 +5007,9 @@ VOID QxlDod::EnableVsync(BOOLEAN bEnable) > else > { > LARGE_INTEGER li; > - LONG period = 1000 / VSYNC_RATE; > - DbgPrint(TRACE_LEVEL_WARNING, ("Enabled VSync(fired %d)\n", > m_VsyncFiredCounter)); > + LONG period = VSYNC_PERIOD; > + if (!period) period = 1000 / VSYNC_RATE; > + DbgPrint(TRACE_LEVEL_WARNING, ("Enabled VSync %d ms(fired > %d)\n", period, m_VsyncFiredCounter)); > li.QuadPart = -10000000 / VSYNC_RATE; > KeSetTimerEx(&m_VsyncTimer, li, period, &m_VsyncTimerDpc); > } </div></div></blockquote></div> </div>