Re: [PATCH spice] server: use a warning when disconnecting unresponsive client

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

 



Hi

----- Original Message -----
> On Thu, May 15, 2014 at 01:42:21PM +0200, Marc-André Lureau wrote:
> > The debug level is not visible by default, since it is an undesirable
> > server behaviour, make it a warning.
> > ---
> >  server/red_channel.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/server/red_channel.c b/server/red_channel.c
> > index 2e467af..887daa9 100644
> > --- a/server/red_channel.c
> > +++ b/server/red_channel.c
> > @@ -818,7 +818,7 @@ static void red_channel_client_connectivity_timer(void
> > *opaque)
> >                                          rcc->connectivity_monitor.timeout);
> >      } else {
> >          monitor->state = CONNECTIVITY_STATE_DISCONNECTED;
> > -        spice_debug("rcc %p has not been responsive for more than %u ms,
> > disconnecting",
> > +        spice_warning("rcc %p has not been responsive for more than %u ms,
> > disconnecting",
> >                      rcc, monitor->timeout);
> >          red_channel_client_disconnect(rcc);
> 
> ACK, maybe we could print rcc->channel->type and rcc->channel->id while
> at it?

>From what I got reading the code, it was only checked on main channel.

But I'll add that too.

thanks
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]