summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHeiko Carstens <heiko.carstens@de.ibm.com>2017-08-18 08:35:33 +0200
committerMartin Schwidefsky <schwidefsky@de.ibm.com>2017-08-29 16:29:02 +0200
commiteb304e800d491d5168df61a999beebe8042e7e58 (patch)
tree276b135b5ce0f4ec9fc1e53eaf142e07e3f4e91f
parent41b0dbfac0da515ad62edf9256414c56ee217364 (diff)
s390/vmcp: simplify vmcp_response_free()
Get rid of the goto and "out" label within vmcp_response_free() which I added. This just makes the code harder to read than necessary. Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
-rw-r--r--drivers/s390/char/vmcp.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/s390/char/vmcp.c b/drivers/s390/char/vmcp.c
index 0aa50afa5063..7898bbcc28fc 100644
--- a/drivers/s390/char/vmcp.c
+++ b/drivers/s390/char/vmcp.c
@@ -90,10 +90,9 @@ static void vmcp_response_free(struct vmcp_session *session)
page = phys_to_page((unsigned long)session->response);
cma_release(vmcp_cma, page, nr_pages);
session->cma_alloc = 0;
- goto out;
+ } else {
+ free_pages((unsigned long)session->response, order);
}
- free_pages((unsigned long)session->response, order);
-out:
session->response = NULL;
}