qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 20/24] xen: re-enable refresh interval reporting


From: Stefano Stabellini
Subject: Re: [Qemu-devel] [PATCH 20/24] xen: re-enable refresh interval reporting for xenfb
Date: Fri, 5 Apr 2013 11:07:34 +0100
User-agent: Alpine 2.02 (DEB 1266 2009-07-14)

On Fri, 5 Apr 2013, Gerd Hoffmann wrote:
> >> -#else
> >>    ; /* nothing */
> >> -#endif
> >>      } else {
> >>    /* we don't get update notifications, thus use the
> >>     * sledge hammer approach ... */
> > 
> > You might as well remove the if () nothing; case.
> 
> Yep, will do.
> 
> >> @@ -785,6 +755,20 @@ static void xenfb_update(void *opaque)
> >>      xenfb->up_fullscreen = 0;
> >>  }
> >>  
> >> +static void xenfb_update_interval(void *opaque, uint64_t interval)
> >> +{
> >> +    struct XenFB *xenfb = opaque;
> >> +
> >> +    if (xenfb->feature_update) {
> >> +#ifdef XENFB_TYPE_REFRESH_PERIOD
> >> +        if (xenfb_queue_full(xenfb)) {
> >> +            return;
> >> +        }
> >> +        xenfb_send_refresh_period(xenfb, interval);
> > 
> > Shouldn't we be updating xenfb->refresh_period here? And shouldn't we
> > call xenfb_send_refresh_period only if interval != 
> > xenfb->refresh_period?
> 
> > On the other hand if refresh_period is not useful anymore, shouldn't
> > we remove it from struct XenFB?
> 
> xenfb_update_interval is only called when interval changes, which I
> think means we don't need refresh_period any more, correct?
 
that's right



reply via email to

[Prev in Thread] Current Thread [Next in Thread]