xen/balloon: Remove pr_info's and don't alter retry_count
In the past the retry_count (in other form of this code) was zero.
Meaning retry forever. Do the same thing here without changing it to
the value 16.
Also remove some of the pr_info as there is no need to spam
the user.
Acked-by: Ian Campbell <Ian.Campbell@eu.citrix.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
index 6cd2530..77e6ad3 100644
--- a/drivers/xen/balloon.c
+++ b/drivers/xen/balloon.c
@@ -199,15 +199,10 @@
return BP_DONE;
}
- pr_info("xen_balloon: Retry count: %lu/%lu\n", balloon_stats.retry_count,
- balloon_stats.max_retry_count);
-
++balloon_stats.retry_count;
if (balloon_stats.max_retry_count != RETRY_UNLIMITED &&
balloon_stats.retry_count > balloon_stats.max_retry_count) {
- pr_info("xen_balloon: Retry count limit exceeded\n"
- "xen_balloon: Balloon operation canceled\n");
balloon_stats.schedule_delay = 1;
balloon_stats.retry_count = 1;
return BP_ECANCELED;
@@ -260,10 +255,8 @@
set_xen_guest_handle(reservation.extent_start, frame_list);
reservation.nr_extents = nr_pages;
rc = HYPERVISOR_memory_op(XENMEM_populate_physmap, &reservation);
- if (rc <= 0) {
- pr_info("xen_balloon: %s: Cannot allocate memory\n", __func__);
+ if (rc <= 0)
return BP_EAGAIN;
- }
for (i = 0; i < rc; i++) {
page = balloon_retrieve();
@@ -313,7 +306,6 @@
for (i = 0; i < nr_pages; i++) {
if ((page = alloc_page(GFP_BALLOON)) == NULL) {
- pr_info("xen_balloon: %s: Cannot allocate memory\n", __func__);
nr_pages = i;
state = BP_EAGAIN;
break;
@@ -456,7 +448,7 @@
balloon_stats.schedule_delay = 1;
balloon_stats.max_schedule_delay = 32;
balloon_stats.retry_count = 1;
- balloon_stats.max_retry_count = 16;
+ balloon_stats.max_retry_count = RETRY_UNLIMITED;
register_balloon(&balloon_sysdev);