[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] util: Improve os_mem_prealloc er
From: |
Eric Blake |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH] util: Improve os_mem_prealloc error message |
Date: |
Thu, 16 Oct 2014 08:52:38 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.1 |
On 10/16/2014 07:13 AM, Michal Privoznik wrote:
> Currently, when the preallocating guest memory process fails, an not
s/an/a/
> so helpful error message is printed out:
>
> # virsh start migt10
> error: Failed to start domain migt10
> error: internal error: process exited while connecting to monitor:
> os_mem_prealloc: failed to preallocate pages
>
>>From the error message it's not clear at the first glance where the
> problem lies. However, changing the error message might give uses a
s/uses/users/
> clue.
>
> Signed-off-by: Michal Privoznik <address@hidden>
> ---
> util/oslib-posix.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
Reviewed-by: Eric Blake <address@hidden>
> diff --git a/util/oslib-posix.c b/util/oslib-posix.c
> index 016a047..8c9d80e 100644
> --- a/util/oslib-posix.c
> +++ b/util/oslib-posix.c
> @@ -390,7 +390,8 @@ void os_mem_prealloc(int fd, char *area, size_t memory)
> pthread_sigmask(SIG_UNBLOCK, &set, &oldset);
>
> if (sigsetjmp(sigjump, 1)) {
> - fprintf(stderr, "os_mem_prealloc: failed to preallocate pages\n");
> + fprintf(stderr, "os_mem_prealloc: Insufficient free host memory "
> + "pages available to allocate guest RAM\n");
> exit(1);
> } else {
> int i;
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature