diff options
author | Claudio Imbrenda <imbrenda@linux.ibm.com> | 2020-06-22 18:21:40 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2020-06-22 13:49:43 -0400 |
commit | 4a4f8af26f3c1491843ca700d05a534c0e767980 (patch) | |
tree | 69768a1441febd6c1a0bda307d7dcc6bc48d23f2 | |
parent | 4aabe7c09a6be56ca6a115b5f4775ae3050be06e (diff) | |
download | kvm-unit-tests-4a4f8af26f3c1491843ca700d05a534c0e767980.tar.gz |
lib/alloc_page: make get_order return unsigned int
Since get_order never returns a negative value, it makes sense to make
it return an unsigned int.
The returned value will be in practice always very small, a u8 would
probably also do the trick.
Signed-off-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
Message-Id: <20200622162141.279716-8-imbrenda@linux.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
-rw-r--r-- | lib/alloc_page.c | 2 | ||||
-rw-r--r-- | lib/alloc_page.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/alloc_page.c b/lib/alloc_page.c index 7c8461a..8769c3f 100644 --- a/lib/alloc_page.c +++ b/lib/alloc_page.c @@ -176,7 +176,7 @@ void page_alloc_ops_enable(void) alloc_ops = &page_alloc_ops; } -int get_order(size_t size) +unsigned int get_order(size_t size) { return is_power_of_2(size) ? fls(size) : fls(size) + 1; } diff --git a/lib/alloc_page.h b/lib/alloc_page.h index e6a51d2..6181299 100644 --- a/lib/alloc_page.h +++ b/lib/alloc_page.h @@ -15,6 +15,6 @@ void *alloc_pages(unsigned long order); void free_page(void *page); void free_pages(void *mem, unsigned long size); void free_pages_by_order(void *mem, unsigned long order); -int get_order(size_t size); +unsigned int get_order(size_t size); #endif |