qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Filip Hejsek <filip.hejsek@gmail.com>
To: Dominique Martinet <asmadeus@codewreck.org>
Cc: Szymon Lukasz <noh4hss@gmail.com>,
	qemu-devel@nongnu.org,  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: Thu, 11 Sep 2025 16:08:49 +0200	[thread overview]
Message-ID: <7eadc2c7030c79923303a5b1b3f2bce0f27ded44.camel@gmail.com> (raw)
In-Reply-To: <aMAwL8q0i6pqcAOn@codewreck.org>

On Tue, 2025-09-09 at 22:48 +0900, Dominique Martinet wrote:
> Filip Hejsek wrote on Mon, Oct 23, 2023 at 05:12:14AM +0200:
> > 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.
> 
> It's been a couple more years... :)
> 
> Have you had a look at this, or did you find another way of handling
> terminal resizes cleanly enough?

I had indeed looked into this back in 2023 and even did some work on
it, but unfortunately I had abandoned it before getting to the finish
line.

From what I remember, I found that the patches didn't merge cleanly, so
I attempted to reconcile them with the changes made in QEMU master
since the original patches were submitted. As part of that work, I felt
that I needed to take responsibility for the patches, the same way I
would for a patch I wrote on my own.

Due to my yak shaving tendencies, I got distracted by several things. I
ended up investigating a test failure caused by a bug in unrelated code
(which I reported as issue #1969). I wrote terminal size detection code
for Windows. And I remember there being something in the patches that I
wasn't really happy about and intended to rewrite, but I never got
around to doing that. After a while, I lost my enthusiasm, and because
I had other things to worry about, this work ended up just sitting
there on my SSD for almost two years.

So anyway, for now I have now uploaded my version of the patches here:
https://gitlab.com/filip-hejsek/qemu/-/commits/console-resize-old

I will try to merge it with the current master again, which should be
ready in a short time. I can't promise much more than that, though. :)

Best regards,
Filip Hejsek


  parent reply	other threads:[~2025-09-11 14:09 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 ` [PATCH v3 0/9] virtio-console: notify about the terminal size Filip Hejsek
2023-10-23  8:09   ` Amit Shah
2025-09-09 13:48   ` Dominique Martinet
2025-09-09 19:24     ` Michael S. Tsirkin
2025-09-11 14:08     ` Filip Hejsek [this message]
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=7eadc2c7030c79923303a5b1b3f2bce0f27ded44.camel@gmail.com \
    --to=filip.hejsek@gmail.com \
    --cc=amit@kernel.org \
    --cc=asmadeus@codewreck.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).