From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:49944) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Qkheg-0001nV-Mr for qemu-devel@nongnu.org; Sat, 23 Jul 2011 15:14:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Qkhef-0004Dt-G8 for qemu-devel@nongnu.org; Sat, 23 Jul 2011 15:14:30 -0400 Received: from mail-yi0-f45.google.com ([209.85.218.45]:47206) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Qkhef-0004Do-CP for qemu-devel@nongnu.org; Sat, 23 Jul 2011 15:14:29 -0400 Received: by yia25 with SMTP id 25so2062683yia.4 for ; Sat, 23 Jul 2011 12:14:28 -0700 (PDT) Message-ID: <4E2B1D93.1090205@codemonkey.ws> Date: Sat, 23 Jul 2011 14:14:27 -0500 From: Anthony Liguori MIME-Version: 1.0 References: <1311108115-27128-4-git-send-email-mdroth@linux.vnet.ibm.com> <1311193177-22282-1-git-send-email-mdroth@linux.vnet.ibm.com> <81810A74-E616-4199-BB69-E7DE4AAE9C0C@suse.de> <4E2AF189.7040403@linux.vnet.ibm.com> <4E2AF277.6060001@codemonkey.ws> <4E2AFA2D.9070706@linux.vnet.ibm.com> <0651B83D-B47C-4CA1-B6E8-36DB8A5DA00C@suse.de> In-Reply-To: <0651B83D-B47C-4CA1-B6E8-36DB8A5DA00C@suse.de> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH] guest agent: qemu-ga daemon List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: agl@linux.vnet.ibm.com, "qemu-devel@nongnu.org Developers" , Jes Sorensen , Michael Roth , Luiz Capitulino , Blue Swirl , Anthony Liguori On 07/23/2011 01:34 PM, Alexander Graf wrote: > > On 23.07.2011, at 18:43, Michael Roth wrote: > >> On 07/23/2011 11:10 AM, Anthony Liguori wrote: >>> On 07/23/2011 11:06 AM, Michael Roth wrote: >>>> On 07/23/2011 05:07 AM, Alexander Graf wrote: >>>>> >>>>> On 20.07.2011, at 22:19, Michael Roth wrote: >>>>> >>>>>> This is the actual guest daemon, it listens for requests over a >>>>>> virtio-serial/isa-serial/unix socket channel and routes them through >>>>>> to dispatch routines, and writes the results back to the channel in >>>>>> a manner similar to QMP. >>>>>> >>>>>> A shorthand invocation: >>>>>> >>>>>> qemu-ga -d >>>>>> >>>>>> Is equivalent to: >>>>>> >>>>>> qemu-ga -m virtio-serial -p /dev/virtio-ports/org.qemu.guest_agent.0 \ >>>>>> -f /var/run/qemu-ga.pid -d >>>>>> >>>>>> Signed-off-by: Michael Roth >>>>> >>>>> A rebase on top of current HEAD gave me the following on openSUSE 11.1 >>>>> PPC: >>>>> >>>>> >>>>> agraf@lychee:/home/agraf/release/qemu> make >>>>> CC qemu-ga.o >>>>> qemu-ga.c:40: error: expected specifier-qualifier-list before ‘GSocket’ >>> >>> GIO is fairly new. It may not be available on openSUSE. >>> >>> Mike, you probably need to do a configure test for GIO and if it's not >>> present, don't build qemu-ga. >> >> It should've failed the glib probe in that case. I think we might need a compile test to catch this GSocket issue. >> >> Rather than building qemu-ga when possible, should we just go ahead and add a configure option and only run the probes when it's set? At least until QMP/QEMU start formally using glib? If so, on or off by default? > > In general, I like the workflow of adding a feature with default off and then enabling it after it has been in for a couple of weeks. Since this got pushed so late for 0.15, I'd personally prefer to see it as preview (disabled by default) in 0.15 and only enabled by default if the requirements are there on 0.16. The only way something like this gets tested is to default it on. We default off'd the I/O thread even after years we still don't have it enabled. With respect to 0.15, this bit of code is totally isolated from everything else. Worst case scenario, we just disable it on platforms where it doesn't work. It presents no real risk to the stability of the release. Regards, Anthony Liguori > > > Alex > >