qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Filip Hejsek <filip.hejsek@gmail.com>
To: Szymon Lukasz <noh4hss@gmail.com>, qemu-devel@nongnu.org
Cc: lvivier@redhat.com, berrange@redhat.com, amit@kernel.org,
	mst@redhat.com,  marcandre.lureau@redhat.com,
	pbonzini@redhat.com
Subject: Re: [PATCH v3 0/9] virtio-console: notify about the terminal size
Date: Mon, 23 Oct 2023 05:12:14 +0200	[thread overview]
Message-ID: <3913e8227c343a5d9bfcc2ac1f01d9bd8eceac7a.camel@gmail.com> (raw)
In-Reply-To: <20200629164041.472528-1-noh4hss@gmail.com>

I have found this 3 year old patch adding virtio console resize
support, which seems to have been forgotten at that time. I would like
to see this feature merged.

Szymon, you have originally submitted this patch, would you be willing
to update it for the latest development tree and resubmit it?

If it turns out that Szymon doesn't want to work on this anymore for
any reason (or doesn't respond), could I resubmit the patch myself?
The contributing guidelines don't say anything about this. I'm
especially unsure how to deal with Signed-off-by, as in such case I
would be (re)submitting sombody else's work, but potentially with my
modifications. I'm willing to update and test the patch, and if
necessary, address any review feedback.

 - Filip Hejsek


> The goal of this series is to have a resizable terminal into a guest
> without having to set up networking and using, e.g. ssh.
> 
> The virtio spec allows a virtio-console device to notify the guest
> about
> terminal resizes in the host. Linux Kernel implements the driver part
> of
> the spec. This series implement the device part in QEMU.
> 
> This series adds support for a resizable terminal if a virtio console
> device is connected to the stdio backend. 
> 
> This series also introduces resize messages that can be sent over QMP
> to
> notify QEMU about the size of the terminal connented to some chardev.
> In the libvirt setting, it will allow to implement a resizable
> terminal 
> for virsh console and other libvirt clients.
> 
> 
> v3:
> add resize messages over QMP, as suggested by Daniel
> 
> v2:
> fix adding a new virtio feature bit to the virtio console device
> 
> Szymon Lukasz (9):
>   chardev: add cols, rows fields
>   chardev: add CHR_EVENT_RESIZE
>   chardev: add qemu_chr_resize()
>   char-mux: add support for the terminal size
>   main-loop: change the handling of SIGWINCH
>   char-stdio: add support for the terminal size
>   qmp: add chardev-resize command
>   virtio-serial-bus: add terminal resize messages
>   virtio-console: notify the guest about terminal resizes
> 
>  backends/cryptodev-vhost-user.c   |  1 +
>  chardev/char-fe.c                 | 13 +++++++
>  chardev/char-mux.c                | 18 ++++++++-
>  chardev/char-stdio.c              | 29 ++++++++++++++
>  chardev/char.c                    | 26 +++++++++++++
>  hw/block/vhost-user-blk.c         |  1 +
>  hw/char/terminal3270.c            |  1 +
>  hw/char/trace-events              |  1 +
>  hw/char/virtio-console.c          | 63
> +++++++++++++++++++++++++++++--
>  hw/char/virtio-serial-bus.c       | 42 ++++++++++++++++++++-
>  hw/core/machine.c                 |  1 +
>  hw/ipmi/ipmi_bmc_extern.c         |  1 +
>  hw/usb/ccid-card-passthru.c       |  1 +
>  hw/usb/dev-serial.c               |  1 +
>  hw/usb/redirect.c                 |  1 +
>  include/chardev/char-fe.h         | 10 +++++
>  include/chardev/char.h            |  7 ++++
>  include/hw/virtio/virtio-serial.h |  5 +++
>  include/qemu/main-loop.h          |  4 ++
>  monitor/hmp.c                     |  1 +
>  monitor/qmp.c                     |  1 +
>  net/vhost-user.c                  |  1 +
>  qapi/char.json                    | 25 ++++++++++++
>  ui/curses.c                       | 11 +++---
>  util/main-loop.c                  | 21 +++++++++++
>  25 files changed, 274 insertions(+), 12 deletions(-)
> 


  parent reply	other threads:[~2023-10-23 12:46 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-29 16:40 [PATCH v3 0/9] virtio-console: notify about the terminal size Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 1/9] chardev: add cols, rows fields Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 2/9] chardev: add CHR_EVENT_RESIZE Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 3/9] chardev: add qemu_chr_resize() Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 4/9] char-mux: add support for the terminal size Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 5/9] main-loop: change the handling of SIGWINCH Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 6/9] char-stdio: add support for the terminal size Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 7/9] qmp: add chardev-resize command Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 8/9] virtio-serial-bus: add terminal resize messages Szymon Lukasz
2020-06-29 16:40 ` [PATCH v3 9/9] virtio-console: notify the guest about terminal resizes Szymon Lukasz
2023-10-23  3:12 ` Filip Hejsek [this message]
2023-10-23  8:09   ` [PATCH v3 0/9] virtio-console: notify about the terminal size Amit Shah
2025-09-09 13:48   ` Dominique Martinet
2025-09-09 19:24     ` Michael S. Tsirkin
2025-09-11 14:08     ` Filip Hejsek
2025-09-11 16:43       ` Filip Hejsek
2025-09-11 21:19         ` Filip Hejsek
2025-09-11 21:53           ` Dominique Martinet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3913e8227c343a5d9bfcc2ac1f01d9bd8eceac7a.camel@gmail.com \
    --to=filip.hejsek@gmail.com \
    --cc=amit@kernel.org \
    --cc=berrange@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=mst@redhat.com \
    --cc=noh4hss@gmail.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).