* [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices [not found] <200909041443.n84EhZaA005101@d01av04.pok.ibm.com> @ 2009-09-09 11:30 ` Avi Kivity 2009-09-09 11:32 ` Jan Kiszka 0 siblings, 1 reply; 5+ messages in thread From: Avi Kivity @ 2009-09-09 11:30 UTC (permalink / raw) To: qemu-devel; +Cc: Jan Kiszka, KVM list On 09/04/2009 05:43 PM, Anthony Liguori wrote: > From: Jan Kiszka<jan.kiszka@siemens.com> > > Rebased version of Anthony's patch: Allow to specify more than one > monitor terminal via the -monitor command line switch. This is > particularly useful when libvirt or some other management tool already > occupies the primary monitor but you need another one for debugging. > The current clumsy workaround is to multiplex such additional terminals > over a qemu character device (e.g. -serial mon:<device>). > > This (as merged into kvm-next) breaks autotest. Autotest starts qemu-kvm with -monitor unix:/tmp/blah,server,nowait; with this patch it no longer opens the socket (as seen with lsof). I haven't yet found out why. -- error compiling committee.c: too many arguments to function ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices 2009-09-09 11:30 ` [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices Avi Kivity @ 2009-09-09 11:32 ` Jan Kiszka 2009-09-09 11:42 ` Avi Kivity 0 siblings, 1 reply; 5+ messages in thread From: Jan Kiszka @ 2009-09-09 11:32 UTC (permalink / raw) To: Avi Kivity; +Cc: qemu-devel, KVM list Avi Kivity wrote: > On 09/04/2009 05:43 PM, Anthony Liguori wrote: >> From: Jan Kiszka<jan.kiszka@siemens.com> >> >> Rebased version of Anthony's patch: Allow to specify more than one >> monitor terminal via the -monitor command line switch. This is >> particularly useful when libvirt or some other management tool already >> occupies the primary monitor but you need another one for debugging. >> The current clumsy workaround is to multiplex such additional terminals >> over a qemu character device (e.g. -serial mon:<device>). >> >> > > This (as merged into kvm-next) breaks autotest. Autotest starts > qemu-kvm with -monitor unix:/tmp/blah,server,nowait; with this patch it > no longer opens the socket (as seen with lsof). I haven't yet found out > why. Ok, will have a look. Jan -- Siemens AG, Corporate Technology, CT SE 2 Corporate Competence Center Embedded Linux ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices 2009-09-09 11:32 ` Jan Kiszka @ 2009-09-09 11:42 ` Avi Kivity 2009-09-09 11:47 ` Avi Kivity 0 siblings, 1 reply; 5+ messages in thread From: Avi Kivity @ 2009-09-09 11:42 UTC (permalink / raw) To: Jan Kiszka; +Cc: qemu-devel, KVM list On 09/09/2009 02:32 PM, Jan Kiszka wrote: > >> This (as merged into kvm-next) breaks autotest. Autotest starts >> qemu-kvm with -monitor unix:/tmp/blah,server,nowait; with this patch it >> no longer opens the socket (as seen with lsof). I haven't yet found out >> why. >> > Ok, will have a look. > > I think it's a merge error, not a but in the commit itself. -- error compiling committee.c: too many arguments to function ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices 2009-09-09 11:42 ` Avi Kivity @ 2009-09-09 11:47 ` Avi Kivity 2009-09-09 11:50 ` Jan Kiszka 0 siblings, 1 reply; 5+ messages in thread From: Avi Kivity @ 2009-09-09 11:47 UTC (permalink / raw) To: Jan Kiszka; +Cc: qemu-devel, KVM list On 09/09/2009 02:42 PM, Avi Kivity wrote: > On 09/09/2009 02:32 PM, Jan Kiszka wrote: >> >>> This (as merged into kvm-next) breaks autotest. Autotest starts >>> qemu-kvm with -monitor unix:/tmp/blah,server,nowait; with this patch it >>> no longer opens the socket (as seen with lsof). I haven't yet found >>> out >>> why. >> Ok, will have a look. >> > > I think it's a merge error, not a but in the commit itself. > It is (totally unrelated to monitors, except that it was introduced when I merged that commit); fixed. -- error compiling committee.c: too many arguments to function ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices 2009-09-09 11:47 ` Avi Kivity @ 2009-09-09 11:50 ` Jan Kiszka 0 siblings, 0 replies; 5+ messages in thread From: Jan Kiszka @ 2009-09-09 11:50 UTC (permalink / raw) To: Avi Kivity; +Cc: qemu-devel, KVM list Avi Kivity wrote: > On 09/09/2009 02:42 PM, Avi Kivity wrote: >> On 09/09/2009 02:32 PM, Jan Kiszka wrote: >>>> This (as merged into kvm-next) breaks autotest. Autotest starts >>>> qemu-kvm with -monitor unix:/tmp/blah,server,nowait; with this patch it >>>> no longer opens the socket (as seen with lsof). I haven't yet found >>>> out >>>> why. >>> Ok, will have a look. >>> >> I think it's a merge error, not a but in the commit itself. >> > > It is (totally unrelated to monitors, except that it was introduced when > I merged that commit); fixed. Fine, tasks-- :) Jan -- Siemens AG, Corporate Technology, CT SE 2 Corporate Competence Center Embedded Linux ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-09-09 11:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <200909041443.n84EhZaA005101@d01av04.pok.ibm.com>
2009-09-09 11:30 ` [Qemu-devel] Re: [Qemu-commits] [COMMIT ddd9bbd] Support for multiple -monitor devices Avi Kivity
2009-09-09 11:32 ` Jan Kiszka
2009-09-09 11:42 ` Avi Kivity
2009-09-09 11:47 ` Avi Kivity
2009-09-09 11:50 ` Jan Kiszka
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).