From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:42370)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1TeR7D-0002Fp-DZ
	for qemu-devel@nongnu.org; Fri, 30 Nov 2012 08:58:52 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1TeR7C-00035E-9A
	for qemu-devel@nongnu.org; Fri, 30 Nov 2012 08:58:51 -0500
Received: from mx1.redhat.com ([209.132.183.28]:51748)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1TeR7C-00035A-1E
	for qemu-devel@nongnu.org; Fri, 30 Nov 2012 08:58:50 -0500
Message-ID: <50B8BB94.8070302@redhat.com>
Date: Fri, 30 Nov 2012 14:58:44 +0100
From: Gerd Hoffmann <kraxel@redhat.com>
MIME-Version: 1.0
References: <1354281947-20227-1-git-send-email-marcandre.lureau@redhat.com>
In-Reply-To: <1354281947-20227-1-git-send-email-marcandre.lureau@redhat.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 0/6] RFC: add "spiceport" chardev
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: =?UTF-8?B?TWFyYy1BbmRyw6kgTHVyZWF1?= <marcandre.lureau@gmail.com>
Cc: spice-devel@lists.freedesktop.org, alevy@redhat.com, qemu-devel@nongnu.org, =?UTF-8?B?TWFyYy1BbmRyw6kgTHVyZWF1?= <marcandre.lureau@redhat.com>

  Hi,

> This allows the Spice client to have a special port for the qemu
> monitor:
> 
> ... -chardev spiceport,name=org.qemu.monitor,id=monitorport
>     -mon chardev=monitorport

Nice.  I think we should have a registry of names, at least for the
org.qemu.* namespace.  A simple text file in docs/ should do.  Monitor
needs to specify the protocol (hmp vs qmp).  Also adding a number
doesn't hurt.  So ...

   org.qemu.monitor.hmp.0
   org.qemu.monitor.qmp.0

Also nice to have:

   org.qemu.console.serial.0

... and spice client redirecting that to a vte.  You might have to add
some termios control messages to spiceport, so one can turn on/off echo,
send breaks, xon/xoff flow control, etc to make it work really nicely.

> Or to allow arbitrary communication outside of qemu:
> 
> ... -chardev spiceport,name=org.ovirt.controller,id=...,chardev=ovcsocket
>     -chardev socket,server,host=0.0.0.0,port=4242,id=ovcsocket,nowait

Hmm, so that will make qemu just hook those chardevs back-to-back and
forward data without looking at it?  I'm not sure we want that ...

What is the use case?  Any reason why the spice client can not (or
should not) speak to ovirt directly?

cheers,
  Gerd