diff mbox

kvm tools: Implement exit kvm tools on Ctrl+a+x

Message ID 1302418231-7364-1-git-send-email-asias.hejun@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Asias He April 10, 2011, 6:50 a.m. UTC
This patch makes Ctrl+a escape key. Ctrl+a+x will terminate kvm tools.

It works for both serial and virtio console.
If you want to input Ctrl+a to guest, type Ctrl+a twice.

Signed-off-by: Asias He <asias.hejun@gmail.com>
---
 tools/kvm/term.c |   41 ++++++++++++++++++++++++++++++++++++-----
 1 files changed, 36 insertions(+), 5 deletions(-)

Comments

Pekka Enberg April 10, 2011, 7:09 a.m. UTC | #1
On Sun, Apr 10, 2011 at 9:50 AM, Asias He <asias.hejun@gmail.com> wrote:
> This patch makes Ctrl+a escape key. Ctrl+a+x will terminate kvm tools.
>
> It works for both serial and virtio console.
> If you want to input Ctrl+a to guest, type Ctrl+a twice.
>
> Signed-off-by: Asias He <asias.hejun@gmail.com>

Cyrill, does this patch work for you? It doesn't do anything for me.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Cyrill Gorcunov April 10, 2011, 7:13 a.m. UTC | #2
On 04/10/2011 11:09 AM, Pekka Enberg wrote:
> On Sun, Apr 10, 2011 at 9:50 AM, Asias He <asias.hejun@gmail.com> wrote:
>> This patch makes Ctrl+a escape key. Ctrl+a+x will terminate kvm tools.
>>
>> It works for both serial and virtio console.
>> If you want to input Ctrl+a to guest, type Ctrl+a twice.
>>
>> Signed-off-by: Asias He <asias.hejun@gmail.com>
> 
> Cyrill, does this patch work for you? It doesn't do anything for me.

Nope, doesn't work for me either.
Amos Kong April 10, 2011, 8:16 a.m. UTC | #3
On Sun, Apr 10, 2011 at 11:13:54AM +0400, Cyrill Gorcunov wrote:
> On 04/10/2011 11:09 AM, Pekka Enberg wrote:
> > On Sun, Apr 10, 2011 at 9:50 AM, Asias He <asias.hejun@gmail.com> wrote:
> >> This patch makes Ctrl+a escape key. Ctrl+a+x will terminate kvm tools.
> >>
> >> It works for both serial and virtio console.
> >> If you want to input Ctrl+a to guest, type Ctrl+a twice.
> >>
> >> Signed-off-by: Asias He <asias.hejun@gmail.com>
> > 
> > Cyrill, does this patch work for you? It doesn't do anything for me.
> 
> Nope, doesn't work for me either.

This patch works for me.


It should be:
Ctrl+a 
x
------------
Not
Ctrl+a
Ctrl+x
------------
Not 
Ctrl+a+x (hit three keys at the same time)
 
Amos.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Pekka Enberg April 10, 2011, 8:20 a.m. UTC | #4
On Sun, Apr 10, 2011 at 11:16 AM, Amos Kong <akong@redhat.com> wrote:
> On Sun, Apr 10, 2011 at 11:13:54AM +0400, Cyrill Gorcunov wrote:
>> On 04/10/2011 11:09 AM, Pekka Enberg wrote:
>> > On Sun, Apr 10, 2011 at 9:50 AM, Asias He <asias.hejun@gmail.com> wrote:
>> >> This patch makes Ctrl+a escape key. Ctrl+a+x will terminate kvm tools.
>> >>
>> >> It works for both serial and virtio console.
>> >> If you want to input Ctrl+a to guest, type Ctrl+a twice.
>> >>
>> >> Signed-off-by: Asias He <asias.hejun@gmail.com>
>> >
>> > Cyrill, does this patch work for you? It doesn't do anything for me.
>>
>> Nope, doesn't work for me either.
>
> This patch works for me.
>
> It should be:
> Ctrl+a
> x

Aah, works for me too. I guess I was doing Ctrl-a + Ctlr+x
unconsciously. Thanks, I'll apply the patch!
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/tools/kvm/term.c b/tools/kvm/term.c
index 9b235d0..d862998 100644
--- a/tools/kvm/term.c
+++ b/tools/kvm/term.c
@@ -9,9 +9,12 @@ 
 #include "kvm/term.h"
 #include "kvm/util.h"
 
-static struct termios orig_term;
+static struct termios	orig_term;
 
-int active_console = CONSOLE_8250;
+int term_escape_char	= 0x01; /* ctrl-a is used for escape */
+bool term_got_escape	= false;
+
+int active_console	= CONSOLE_8250;
 
 int term_getc(int who)
 {
@@ -22,6 +25,24 @@  int term_getc(int who)
 
 	if (read_in_full(STDIN_FILENO, &c, 1) < 0)
 		return -1;
+
+	c &= 0xff;
+
+	if (term_got_escape) {
+		term_got_escape = false;
+		if (c == 'x') {
+			printf("\nKVM TOOLS: Terminated\n");
+			exit(1);
+		}
+		if (c == term_escape_char)
+			return c;
+	}
+
+	if (c == term_escape_char) {
+		term_got_escape = true;
+		return -1;
+	}
+
 	return c;
 }
 
@@ -39,16 +60,26 @@  int term_putc(int who, char *addr, int cnt)
 
 int term_getc_iov(int who, struct iovec *iov, int iovcnt)
 {
+	int c;
+
 	if (who != active_console)
-		return -1;
+		return 0;
+
+	c = term_getc(who);
 
-	return readv(STDIN_FILENO, iov, iovcnt);
+	if (c < 0)
+		return 0;
+
+	*((int *)iov[0].iov_base)	= c;
+	iov[0].iov_len			= sizeof(int);
+
+	return sizeof(int);
 }
 
 int term_putc_iov(int who, struct iovec *iov, int iovcnt)
 {
 	if (who != active_console)
-		return -1;
+		return 0;
 
 	return writev(STDOUT_FILENO, iov, iovcnt);
 }