No problem to merge them, thank you. On Mon, Apr 3, 2017 at 1:36 PM, Frediano Ziglio <[email protected]> wrote:
> > > > From: "[email protected]" <[email protected]> > > > > 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 <[email protected]> > > 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 > > > --- > > 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); > > } > >
_______________________________________________ Spice-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/spice-devel
