diff options
author | Victor Toso <me@victortoso.com> | 2019-03-07 07:24:52 +0000 |
---|---|---|
committer | Victor Toso <me@victortoso.com> | 2019-03-11 08:53:25 +0000 |
commit | 679b63fe6ea153d234c876c8a97393870e8212e7 (patch) | |
tree | ba99b6c5780d19b8361cecea581e5f76ea28b3a2 | |
parent | 105e63dd8149230b3f164567aecd7421c61e6b92 (diff) |
display-channel: monitors config debug: add head number
The difference is subtle but compared to what client receives, this
could help identify values set to the wrong head, e.g:
First we received:
| display-channel.c:180:monitors_config_debug: monitors config count:2 max:4
| display-channel.c:184:monitors_config_debug: +0+0 1015x805
| display-channel.c:184:monitors_config_debug: +1015+0 1024x740
And then:
| display-channel.c:180:monitors_config_debug: monitors config count:3 max:4
| display-channel.c:184:monitors_config_debug: +0+0 1015x805
| display-channel.c:184:monitors_config_debug: +0+0 0x0
| display-channel.c:184:monitors_config_debug: +1015+0 1024x740
In the first debug it would be helpful to have "head 0" and "head 1",
to point out the temporary error in monitor's config message.
Signed-off-by: Victor Toso <victortoso@redhat.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
-rw-r--r-- | server/display-channel.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/server/display-channel.c b/server/display-channel.c index e68ed10f..9bb7fa44 100644 --- a/server/display-channel.c +++ b/server/display-channel.c @@ -179,10 +179,11 @@ static void monitors_config_debug(MonitorsConfig *mc) int i; spice_debug("monitors config count:%d max:%d", mc->count, mc->max_allowed); - for (i = 0; i < mc->count; i++) - spice_debug("+%d+%d %dx%d", + for (i = 0; i < mc->count; i++) { + spice_debug("head #%d +%d+%d %dx%d", i, mc->heads[i].x, mc->heads[i].y, mc->heads[i].width, mc->heads[i].height); + } } static MonitorsConfig* monitors_config_new(QXLHead *heads, ssize_t nheads, ssize_t max) |