This commit fixes return type of online_page(), which shall be int to include correct error value.
Signed-off-by: Penny Zheng <penny.zh...@amd.com> --- xen/common/page_alloc.c | 2 +- xen/include/xen/mm.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/common/page_alloc.c b/xen/common/page_alloc.c index 1bf070c8c5..b5ec4bda7d 100644 --- a/xen/common/page_alloc.c +++ b/xen/common/page_alloc.c @@ -1764,7 +1764,7 @@ int offline_page(mfn_t mfn, int broken, uint32_t *status) * The caller should make sure end_pfn <= max_page, * if not, expand_pages() should be called prior to online_page(). */ -unsigned int online_page(mfn_t mfn, uint32_t *status) +int online_page(mfn_t mfn, uint32_t *status) { unsigned long x, nx, y; struct page_info *pg; diff --git a/xen/include/xen/mm.h b/xen/include/xen/mm.h index 16f733281a..ae1c48a615 100644 --- a/xen/include/xen/mm.h +++ b/xen/include/xen/mm.h @@ -144,7 +144,7 @@ unsigned long avail_domheap_pages(void); unsigned long avail_node_heap_pages(unsigned int nodeid); #define alloc_domheap_page(d,f) (alloc_domheap_pages(d,0,f)) #define free_domheap_page(p) (free_domheap_pages(p,0)) -unsigned int online_page(mfn_t mfn, uint32_t *status); +int online_page(mfn_t mfn, uint32_t *status); int offline_page(mfn_t mfn, int broken, uint32_t *status); int query_page_offline(mfn_t mfn, uint32_t *status); -- 2.34.1