Commit 48e09084 authored by Wez's avatar Wez Committed by Commit Bot

[base] Only log zx_vmar_map() failure for non-hinted allocations.

Bug: 927411
Change-Id: Ia2fabf76ef8faa3b8d875b8625572c81eef20946
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911488
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: default avatarChris Palmer <palmer@chromium.org>
Auto-Submit: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714668}
parent 6c42d2c2
...@@ -109,7 +109,10 @@ void* SystemAllocPagesInternal(void* hint, ...@@ -109,7 +109,10 @@ void* SystemAllocPagesInternal(void* hint,
zx::vmar::root_self()->map(vmar_offset, vmo, zx::vmar::root_self()->map(vmar_offset, vmo,
/*vmo_offset=*/0, length, options, &address); /*vmo_offset=*/0, length, options, &address);
if (status != ZX_OK) { if (status != ZX_OK) {
ZX_DLOG(INFO, status) << "zx_vmar_map"; // map() is expected to fail if |hint| is set to an already-in-use location.
if (!hint) {
ZX_DLOG(ERROR, status) << "zx_vmar_map";
}
return nullptr; return nullptr;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment