[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Remove debugging messages.
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [PATCH] Remove debugging messages. |
Date: |
Tue, 19 Jul 2011 23:49:22 +0400 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.1.16) Gecko/20110704 Icedove/3.0.11 |
19.07.2011 23:23, Richard W.M. Jones wrote:
> From: "Richard W.M. Jones" <address@hidden>
>
> When using qemu -machine accel=kvm:tcg, if KVM is not available you
> get a load of debugging output:
>
> Could not access KVM kernel module: No such file or directory
> failed to initialize KVM: No such file or directory
> Back to tcg accelerator.
>
> Signed-off-by: Richard W.M. Jones <address@hidden>
> ---
> kvm-all.c | 1 -
> vl.c | 14 --------------
> 2 files changed, 0 insertions(+), 15 deletions(-)
>
> diff --git a/kvm-all.c b/kvm-all.c
> index cbc2532..35d698b 100644
> --- a/kvm-all.c
> +++ b/kvm-all.c
> @@ -713,7 +713,6 @@ int kvm_init(void)
> s->vmfd = -1;
> s->fd = qemu_open("/dev/kvm", O_RDWR);
> if (s->fd == -1) {
> - fprintf(stderr, "Could not access KVM kernel module: %m\n");
Don't do this. qemu error reporting sucks badly already,
and this very error message gives us at least some details
(the %m part) about what's missing. There's no other place
where this information is printed in case this open fails.
We already have to resort to strace in way too many places... :(
Thanks,
/mjt
- [Qemu-devel] [PATCH 1/1] Remove debugging messages., Richard W.M. Jones, 2011/07/19
- [Qemu-devel] [PATCH] Remove debugging messages., Richard W.M. Jones, 2011/07/19
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Jan Kiszka, 2011/07/19
- Re: [Qemu-devel] [PATCH] Remove debugging messages.,
Michael Tokarev <=
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Anthony Liguori, 2011/07/19
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Richard W.M. Jones, 2011/07/19
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Anthony Liguori, 2011/07/19
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Richard W.M. Jones, 2011/07/20
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Anthony Liguori, 2011/07/20
- Re: [Qemu-devel] [PATCH] Remove debugging messages., Richard W.M. Jones, 2011/07/20