[CRIU] [PATCH v3] restore: Fetch mmap_minimal_address runtime
Pavel Emelyanov
xemul at virtuozzo.com
Mon Dec 19 03:45:18 PST 2016
On 12/08/2016 04:28 PM, Cyrill Gorcunov wrote:
> On Thu, Dec 08, 2016 at 04:25:48PM +0300, Pavel Emelyanov wrote:
>>> +
>>> + if (value < default_mmap_min_addr) {
>>> + pr_debug("Adjust mmap_min_addr %#lx -> %#lx\n",
>>> + value, default_mmap_min_addr);
>>
>> This doesn't build on ARM.
>>
>> In file included from criu/kerndat.c:16:0:
>> criu/kerndat.c: In function 'kerndat_get_mmap_min_addr':
>> criu/kerndat.c:136:12: error: format '%lx' expects argument of type 'long unsigned int', but argument 3 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=]
>> pr_debug("Adjust mmap_min_addr %#lx -> %#lx\n",
>> ^
>> /criu/criu/include/log.h:68:21: note: in definition of macro 'pr_debug'
>> LOG_PREFIX fmt, ##__VA_ARGS__)
>> ^~~
>> CC criu/log.o
>
> Could you please just merge in explicit (unsigned long) conversion?
>
> pr_debug("Adjust mmap_min_addr %#lx -> %#lx\n",
> (unsigned long)value, default_mmap_min_addr);
> .
>
It's not just this place, I suppose...
-- Pavel
More information about the CRIU
mailing list