diff options
| author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2017-08-18 08:35:33 +0200 |
|---|---|---|
| committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2017-08-29 16:29:02 +0200 |
| commit | eb304e800d491d5168df61a999beebe8042e7e58 (patch) | |
| tree | 276b135b5ce0f4ec9fc1e53eaf142e07e3f4e91f /drivers/s390 | |
| parent | 41b0dbfac0da515ad62edf9256414c56ee217364 (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>
Diffstat (limited to 'drivers/s390')
| -rw-r--r-- | drivers/s390/char/vmcp.c | 5 |
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; } |