From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1O7SzM-0003Xv-P8
	for qemu-devel@nongnu.org; Thu, 29 Apr 2010 08:37:08 -0400
Received: from [140.186.70.92] (port=48333 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1O7SzK-0003V6-CY
	for qemu-devel@nongnu.org; Thu, 29 Apr 2010 08:37:08 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69)
	(envelope-from <slash@ac.auone-net.jp>) id 1O7Syo-0001Vt-1i
	for qemu-devel@nongnu.org; Thu, 29 Apr 2010 08:36:35 -0400
Received: from msa102.auone-net.jp ([61.117.18.162]:59939)
	by eggs.gnu.org with esmtp (Exim 4.69)
	(envelope-from <slash@ac.auone-net.jp>) id 1O7Syn-0001Vl-Nj
	for qemu-devel@nongnu.org; Thu, 29 Apr 2010 08:36:33 -0400
Date: Thu, 29 Apr 2010 21:36:30 +0900
From: Kusanagi Kouichi <slash@ac.auone-net.jp>
Subject: Re: [Qemu-devel] [PATCH 2/2] virtio-console: Notify resize to the
	guest.
References: <20100416114248.A001562C03A@msa106.auone-net.jp>
	<20100416130219.GE21268@amit-x200.redhat.com>
	<20100429071239.6854F6FC039@msa105.auone-net.jp>
	<20100429104718.GB21309@amit-x200.redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20100429104718.GB21309@amit-x200.redhat.com>
Message-Id: <20100429123631.130366AC03A@msa102.auone-net.jp>
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Amit Shah <amit.shah@redhat.com>
Cc: qemu-devel@nongnu.org

On 2010-04-29 16:17:18 +0530, Amit Shah wrote:
> On (Thu) Apr 29 2010 [16:12:38], Kusanagi Kouichi wrote:
> > On 2010-04-16 18:32:19 +0530, Amit Shah wrote:
> > > On (Fri) Apr 16 2010 [20:42:47], Kusanagi Kouichi wrote:
> > > > 
> > > > Signed-off-by: Kusanagi Kouichi <slash@ac.auone-net.jp>
> > > > ---
> > > >  hw/virtio-console.c    |    3 +++
> > > >  hw/virtio-serial-bus.c |    8 ++++++++
> > > >  hw/virtio-serial.h     |    9 +++++++--
> > > >  3 files changed, 18 insertions(+), 2 deletions(-)
> > > 
> > > Hello,
> > > 
> > > The patches look good. How did you test them?
> > > 
> > > Just one comment here:
> > > 
> > > > diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c
> > > > index 17c1ec1..66891ee 100644
> > > > --- a/hw/virtio-serial-bus.c
> > > > +++ b/hw/virtio-serial-bus.c
> > > > @@ -163,6 +163,13 @@ int virtio_serial_close(VirtIOSerialPort *port)
> > > >      return 0;
> > > >  }
> > > >  
> > > > +void virtio_serial_resize(VirtIOSerialPort *port, int rows, int cols)
> > > 
> > > Would virtio_serial_resize_console() be better, since this is only to be
> > > used for console ports?
> > > 
> > 
> > It is better, I think. I will update the patch. Thanks.
> 
> Thanks.
> 
> Please include me in the CC when you send it. Also please mention how
> you test this feature.
> 
> 		Amit

I put printf()s into involved functions. Then ran qemu on a terminal
emulator, and resized it.
That's how I did. Is this what you need?