[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v2.1 12/15] tmem: cleanup: refactor function tmemc_shared_pool_auth()
From: Bob Liu <lliubbo@xxxxxxxxx> Make function tmemc_shared_pool_auth() more readable. Note that the previous check for free being set the first time '(free == -1)' in the loop is now removed. That is OK because when we set free the first time ('free = i;') we follow it immediately with a break to get out of the loop. Signed-off-by: Bob Liu <bob.liu@xxxxxxxxxx> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> --- xen/common/tmem.c | 38 +++++++++++++++++++++++++------------- 1 file changed, 25 insertions(+), 13 deletions(-) diff --git a/xen/common/tmem.c b/xen/common/tmem.c index add236e..bad1bb9 100644 --- a/xen/common/tmem.c +++ b/xen/common/tmem.c @@ -2257,27 +2257,39 @@ static int tmemc_shared_pool_auth(domid_t cli_id, uint64_t uuid_lo, client = tmem_client_from_cli_id(cli_id); if ( client == NULL ) return -EINVAL; + for ( i = 0; i < MAX_GLOBAL_SHARED_POOLS; i++) { - if ( (client->shared_auth_uuid[i][0] == uuid_lo) && - (client->shared_auth_uuid[i][1] == uuid_hi) ) + if ( auth == 0 ) { - if ( auth == 0 ) - client->shared_auth_uuid[i][0] = - client->shared_auth_uuid[i][1] = -1L; - return 1; + if ( (client->shared_auth_uuid[i][0] == uuid_lo) && + (client->shared_auth_uuid[i][1] == uuid_hi) ) + { + client->shared_auth_uuid[i][0] = -1L; + client->shared_auth_uuid[i][1] = -1L; + return 1; + } } - if ( (auth == 1) && (client->shared_auth_uuid[i][0] == -1L) && - (client->shared_auth_uuid[i][1] == -1L) && (free == -1) ) - free = i; + else + { + if ( (client->shared_auth_uuid[i][0] == -1L) && + (client->shared_auth_uuid[i][1] == -1L) ) + { + free = i; + break; + } + } } if ( auth == 0 ) return 0; - if ( auth == 1 && free == -1 ) + else if ( free == -1) return -ENOMEM; - client->shared_auth_uuid[free][0] = uuid_lo; - client->shared_auth_uuid[free][1] = uuid_hi; - return 1; + else + { + client->shared_auth_uuid[free][0] = uuid_lo; + client->shared_auth_uuid[free][1] = uuid_hi; + return 1; + } } static int tmemc_save_subop(int cli_id, uint32_t pool_id, -- 1.8.5.3 _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |