[Spice-devel] [PATCH] Make monitors config debug output more clear
Fabiano Fidencio
ffidenci at redhat.com
Tue Jun 30 12:17:14 PDT 2015
ACK!
----- Original Message -----
> From: "Jonathon Jongsma" <jjongsma at redhat.com>
> To: spice-devel at freedesktop.org
> Sent: Tuesday, June 30, 2015 7:26:38 PM
> Subject: [Spice-devel] [PATCH] Make monitors config debug output more clear
>
> Indicate whether the monitors config debug output is from sending or
> receiving new monitors configuration. You can tell this by looking at
> which channel is involved (main vs display), but making it more explicit
> is helpful for glancing through logs.
> ---
> src/channel-display.c | 2 +-
> src/channel-main.c | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/src/channel-display.c b/src/channel-display.c
> index efe2259..568aa2e 100644
> --- a/src/channel-display.c
> +++ b/src/channel-display.c
> @@ -1713,7 +1713,7 @@ static void display_handle_monitors_config(SpiceChannel
> *channel, SpiceMsgIn *in
> g_return_if_fail(config != NULL);
> g_return_if_fail(config->count > 0);
>
> - CHANNEL_DEBUG(channel, "monitors config: n: %d/%d", config->count,
> config->max_allowed);
> + CHANNEL_DEBUG(channel, "received new monitors config from guest: n:
> %d/%d", config->count, config->max_allowed);
>
> c->monitors_max = config->max_allowed;
> if (CLAMP_CHECK(c->monitors_max, 1, MONITORS_MAX)) {
> diff --git a/src/channel-main.c b/src/channel-main.c
> index c55d097..35c5b2f 100644
> --- a/src/channel-main.c
> +++ b/src/channel-main.c
> @@ -1071,6 +1071,7 @@ gboolean
> spice_main_send_monitor_config(SpiceMainChannel *channel)
> c->disable_display_align == FALSE)
> mon->flags |= VD_AGENT_CONFIG_MONITORS_FLAG_USE_POS;
>
> + CHANNEL_DEBUG(channel, "sending new monitors config to guest");
> j = 0;
> for (i = 0; i < SPICE_N_ELEMENTS(c->display); i++) {
> if (!c->display[i].enabled) {
> @@ -1084,7 +1085,7 @@ gboolean
> spice_main_send_monitor_config(SpiceMainChannel *channel)
> mon->monitors[j].height = c->display[i].height;
> mon->monitors[j].x = c->display[i].x;
> mon->monitors[j].y = c->display[i].y;
> - CHANNEL_DEBUG(channel, "monitor config: #%d %dx%d+%d+%d @ %d bpp",
> j,
> + CHANNEL_DEBUG(channel, "monitor #%d: %dx%d+%d+%d @ %d bpp", j,
> mon->monitors[j].width, mon->monitors[j].height,
> mon->monitors[j].x, mon->monitors[j].y,
> mon->monitors[j].depth);
> --
> 2.1.0
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>
More information about the Spice-devel
mailing list