[Spice-devel] [PATCH 1/3] Add a DeferredFPS option.

Christophe Fergeau cfergeau at redhat.com
Fri Sep 28 10:56:11 PDT 2012


This one looks good, but is useless without the rest of the series, which
I'm not sure I'll be able to make a meaningful review of :-/

Christophe

On Sat, Sep 15, 2012 at 11:09:20AM -0500, Jeremy White wrote:
> ---
>  src/qxl.h        |    3 +++
>  src/qxl_driver.c |    8 ++++++++
>  2 files changed, 11 insertions(+)
> 
> diff --git a/src/qxl.h b/src/qxl.h
> index 33b0fb7..f26ddd0 100644
> --- a/src/qxl.h
> +++ b/src/qxl.h
> @@ -125,6 +125,7 @@ enum {
>      OPTION_SPICE_TLS_CIPHERS,
>      OPTION_SPICE_CACERT_FILE,
>      OPTION_SPICE_DH_FILE,
> +    OPTION_SPICE_DEFERRED_FPS,
>  #endif
>      OPTION_COUNT,
>  };
> @@ -258,6 +259,8 @@ struct _qxl_screen_t
>          uint32_t       bytes_pp;
>          uint8_t        *data, *flipped;
>      } guest_primary;
> +
> +    uint32_t           deferred_fps;
>  #endif /* XSPICE */
>  };
>  
> diff --git a/src/qxl_driver.c b/src/qxl_driver.c
> index bfdb51a..1c79060 100644
> --- a/src/qxl_driver.c
> +++ b/src/qxl_driver.c
> @@ -124,6 +124,8 @@ const OptionInfoRec DefaultOptions[] =
>        "SpiceCacertFile",          OPTV_STRING,    {0}, FALSE},
>      { OPTION_SPICE_DH_FILE,
>        "SpiceDhFile",              OPTV_STRING,    {0}, FALSE},
> +    { OPTION_SPICE_DEFERRED_FPS,
> +      "SpiceDeferredFPS",         OPTV_INTEGER,   {0}, FALSE},
>  #endif
>      
>      { -1, NULL, OPTV_NONE, {0}, FALSE }
> @@ -2405,6 +2407,12 @@ qxl_pre_init (ScrnInfoPtr pScrn, int flags)
>      qxl->num_heads =
>          get_int_option (qxl->options, OPTION_NUM_HEADS, "QXL_NUM_HEADS");
>      
> +    qxl->deferred_fps = get_int_option(qxl->options, OPTION_SPICE_DEFERRED_FPS, "XSPICE_DEFERRED_FPS");
> +    if (qxl->deferred_fps > 0)
> +        xf86DrvMsg(scrnIndex, X_INFO, "Deferred FPS: %d\n", qxl->deferred_fps);
> +    else
> +        xf86DrvMsg(scrnIndex, X_INFO, "Deferred Frames: Disabled\n");
> +
>      xf86DrvMsg (scrnIndex, X_INFO, "Offscreen Surfaces: %s\n",
>                  qxl->enable_surfaces ? "Enabled" : "Disabled");
>      xf86DrvMsg (scrnIndex, X_INFO, "Image Cache: %s\n",
> -- 
> 1.7.9.5
> 
> 
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20120928/ce384a8e/attachment.pgp>


More information about the Spice-devel mailing list