[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] migration: initialize RAM to zero
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2] migration: initialize RAM to zero |
Date: |
Tue, 09 Apr 2013 17:04:02 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> Il 09/04/2013 15:20, Markus Armbruster ha scritto:
>>> > -#if defined(CONFIG_VALGRIND)
>>> > - if (running_on_valgrind < 0) {
>>> > - /* First call, test whether we are running on Valgrind.
>>> > - This is a substitute for RUNNING_ON_VALGRIND from
>>> > valgrind.h. */
>>> > - const char *ld = getenv("LD_PRELOAD");
>>> > - running_on_valgrind = (ld != NULL && strstr(ld, "vgpreload"));
>>> > + if ((intptr_t) ptr == -1) {
>> Recommend ptr == MAP_FAILED
>>
>
> Worth respinning?
I'd do it, but it's really your choice.