From patchwork Thu Oct 29 21:46:35 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sasha Khapyorsky X-Patchwork-Id: 56525 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n9TLicnD009642 for ; Thu, 29 Oct 2009 21:44:38 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753079AbZJ2Voc (ORCPT ); Thu, 29 Oct 2009 17:44:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753359AbZJ2Voc (ORCPT ); Thu, 29 Oct 2009 17:44:32 -0400 Received: from mail-bw0-f227.google.com ([209.85.218.227]:38167 "EHLO mail-bw0-f227.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753079AbZJ2Vob (ORCPT ); Thu, 29 Oct 2009 17:44:31 -0400 Received: by bwz27 with SMTP id 27so2847068bwz.21 for ; Thu, 29 Oct 2009 14:44:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:sender:received:date:from:to :cc:subject:message-id:references:mime-version:content-type :content-disposition:in-reply-to:user-agent; bh=gwjV2oDRwoBcuSSmKHpw3EX1mMw5QG3Cb/MrUI3bjzA=; b=tcUYeT1l9r3GzbG6beyHloEYb9Z8wYzikNs1x7nZanY3D1hL1ze5rXzQg7qmQqEIJ8 ohle+JJZ7e14ua6DFgUXdMW6sB8ivR/nsfE3U4QwkkDv6cUtmlkixZY8f41umORPz7iA QNl7DFdW/MGC25z8z2jFxWyuJhtmj6k2rG1mw= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=HyGMKXcd2KxUt0TR2bKKLqhSGdkiYmsT9HqUxDNFhRtqxv97z1HkNELaGjZUqogqcb h6YH9WjTZZiYK5FaOwRLBcMK5oSuigvQj/016dakdXFfr8xJEQIKXyEiNSxeY1HBO+Gf b00r5t5roT7kaR3QTmsSEwAFSNmorPZEWVMWk= Received: by 10.204.157.24 with SMTP id z24mr423700bkw.208.1256852670629; Thu, 29 Oct 2009 14:44:30 -0700 (PDT) Received: from me.localdomain (85.64.35.106.dynamic.barak-online.net [85.64.35.106]) by mx.google.com with ESMTPS id 18sm268767fkq.37.2009.10.29.14.44.29 (version=TLSv1/SSLv3 cipher=RC4-MD5); Thu, 29 Oct 2009 14:44:30 -0700 (PDT) Received: by me.localdomain (Postfix, from userid 1000) id 3995F11E89; Thu, 29 Oct 2009 23:46:35 +0200 (IST) Date: Thu, 29 Oct 2009 23:46:35 +0200 From: Sasha Khapyorsky To: Al Chu Cc: linux-rdma@vger.kernel.org Subject: [PATCH] libibnetdisc: rename ibnd_scan variable to scan Message-ID: <20091029214635.GI20136@me> References: <1256773929.4819.190.camel@auk31.llnl.gov> <20091029213900.GG20136@me> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20091029213900.GG20136@me> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-rdma-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org diff --git a/infiniband-diags/libibnetdisc/src/chassis.c b/infiniband-diags/libibnetdisc/src/chassis.c index dcf15b4..ffae29b 100644 --- a/infiniband-diags/libibnetdisc/src/chassis.c +++ b/infiniband-diags/libibnetdisc/src/chassis.c @@ -782,19 +782,19 @@ static void voltaire_portmap(ibnd_port_t * port) port->ext_portnum = int2ext_map_slb8[chipnum][portnum]; } -static int add_chassis(ibnd_scan_t *ibnd_scan) +static int add_chassis(ibnd_scan_t *scan) { - if (!(ibnd_scan->current_chassis = calloc(1, sizeof(ibnd_chassis_t)))) { + if (!(scan->current_chassis = calloc(1, sizeof(ibnd_chassis_t)))) { IBND_ERROR("OOM: failed to allocate chassis object\n"); return (-1); } - if (ibnd_scan->first_chassis == NULL) { - ibnd_scan->first_chassis = ibnd_scan->current_chassis; - ibnd_scan->last_chassis = ibnd_scan->current_chassis; + if (scan->first_chassis == NULL) { + scan->first_chassis = scan->current_chassis; + scan->last_chassis = scan->current_chassis; } else { - ibnd_scan->last_chassis->next = ibnd_scan->current_chassis; - ibnd_scan->last_chassis = ibnd_scan->current_chassis; + scan->last_chassis->next = scan->current_chassis; + scan->last_chassis = scan->current_chassis; } return (0); } @@ -818,7 +818,7 @@ static void add_node_to_chassis(ibnd_chassis_t * chassis, ibnd_node_t * node) Returns: 0 on success, -1 on failure */ -int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) +int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *scan) { ibnd_node_t *node; int dist; @@ -826,16 +826,16 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) ibnd_chassis_t *chassis; ibnd_chassis_t *ch, *ch_next; - ibnd_scan->first_chassis = NULL; - ibnd_scan->current_chassis = NULL; - ibnd_scan->last_chassis = NULL; + scan->first_chassis = NULL; + scan->current_chassis = NULL; + scan->last_chassis = NULL; /* first pass on switches and build for every Voltaire node */ /* an appropriate chassis record (slotnum and position) */ /* according to internal connectivity */ /* not very efficient but clear code so... */ for (dist = 0; dist <= fabric->maxhops_discovered; dist++) { - for (node = ibnd_scan->nodesdist[dist]; node; node = node->dnext) { + for (node = scan->nodesdist[dist]; node; node = node->dnext) { if (mad_get_field(node->info, 0, IB_NODE_VENDORID_F) == VTR_VENDOR_ID) if (fill_voltaire_chassis_record(node)) @@ -846,7 +846,7 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) /* separate every Voltaire chassis from each other and build linked list of them */ /* algorithm: catch spine and find all surrounding nodes */ for (dist = 0; dist <= fabric->maxhops_discovered; dist++) { - for (node = ibnd_scan->nodesdist[dist]; node; node = node->dnext) { + for (node = scan->nodesdist[dist]; node; node = node->dnext) { if (mad_get_field(node->info, 0, IB_NODE_VENDORID_F) != VTR_VENDOR_ID) continue; @@ -854,10 +854,10 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) || (node->chassis && node->chassis->chassisnum) || !is_spine(node)) continue; - if (add_chassis(ibnd_scan)) + if (add_chassis(scan)) goto cleanup; - ibnd_scan->current_chassis->chassisnum = ++chassisnum; - if (build_chassis(node, ibnd_scan->current_chassis)) + scan->current_chassis->chassisnum = ++chassisnum; + if (build_chassis(node, scan->current_chassis)) goto cleanup; } } @@ -865,7 +865,7 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) /* now make pass on nodes for chassis which are not Voltaire */ /* grouped by common SystemImageGUID */ for (dist = 0; dist <= fabric->maxhops_discovered; dist++) { - for (node = ibnd_scan->nodesdist[dist]; node; node = node->dnext) { + for (node = scan->nodesdist[dist]; node; node = node->dnext) { if (mad_get_field(node->info, 0, IB_NODE_VENDORID_F) == VTR_VENDOR_ID) continue; @@ -878,12 +878,12 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) chassis->nodecount++; else { /* Possible new chassis */ - if (add_chassis(ibnd_scan)) + if (add_chassis(scan)) goto cleanup; - ibnd_scan->current_chassis->chassisguid = + scan->current_chassis->chassisguid = get_chassisguid((ibnd_node_t *) node); - ibnd_scan->current_chassis->nodecount = 1; + scan->current_chassis->nodecount = 1; } } } @@ -892,7 +892,7 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) /* now, make another pass to see which nodes are part of chassis */ /* (defined as chassis->nodecount > 1) */ for (dist = 0; dist <= MAXHOPS;) { - for (node = ibnd_scan->nodesdist[dist]; node; node = node->dnext) { + for (node = scan->nodesdist[dist]; node; node = node->dnext) { if (mad_get_field(node->info, 0, IB_NODE_VENDORID_F) == VTR_VENDOR_ID) continue; @@ -920,18 +920,18 @@ int group_nodes(ibnd_fabric_t * fabric, ibnd_scan_t *ibnd_scan) dist++; } - fabric->chassis = ibnd_scan->first_chassis; + fabric->chassis = scan->first_chassis; return (0); cleanup: - ch = ibnd_scan->first_chassis; + ch = scan->first_chassis; while (ch) { ch_next = ch->next; free(ch); ch = ch_next; } - ibnd_scan->first_chassis = NULL; - ibnd_scan->current_chassis = NULL; - ibnd_scan->last_chassis = NULL; + scan->first_chassis = NULL; + scan->current_chassis = NULL; + scan->last_chassis = NULL; return (-1); } diff --git a/infiniband-diags/libibnetdisc/src/ibnetdisc.c b/infiniband-diags/libibnetdisc/src/ibnetdisc.c index 047b705..d2ec9b2 100644 --- a/infiniband-diags/libibnetdisc/src/ibnetdisc.c +++ b/infiniband-diags/libibnetdisc/src/ibnetdisc.c @@ -180,20 +180,20 @@ static void retract_dpath(ib_portid_t * path) } static int extend_dpath(struct ibmad_port *ibmad_port, ibnd_fabric_t * fabric, - ibnd_scan_t *ibnd_scan, ib_portid_t * portid, int nextport) + ibnd_scan_t *scan, ib_portid_t * portid, int nextport) { int rc = 0; if (portid->lid) { /* If we were LID routed we need to set up the drslid */ - if (!ibnd_scan->selfportid.lid) - if (ib_resolve_self_via(&ibnd_scan->selfportid, NULL, NULL, + if (!scan->selfportid.lid) + if (ib_resolve_self_via(&scan->selfportid, NULL, NULL, ibmad_port) < 0) { IBND_ERROR("Failed to resolve self\n"); return -1; } - portid->drpath.drslid = (uint16_t) ibnd_scan->selfportid.lid; + portid->drpath.drslid = (uint16_t) scan->selfportid.lid; portid->drpath.drdlid = 0xFFFF; } @@ -387,16 +387,16 @@ static void add_to_type_list(ibnd_node_t * node, ibnd_fabric_t * fabric) } } -static void add_to_nodedist(ibnd_node_t * node, ibnd_scan_t * ibnd_scan, int dist) +static void add_to_nodedist(ibnd_node_t * node, ibnd_scan_t * scan, int dist) { if (node->type != IB_NODE_SWITCH) dist = MAXHOPS; /* special Ca list */ - node->dnext = ibnd_scan->nodesdist[dist]; - ibnd_scan->nodesdist[dist] = node; + node->dnext = scan->nodesdist[dist]; + scan->nodesdist[dist] = node; } -static ibnd_node_t *create_node(ibnd_fabric_t * fabric, ibnd_scan_t * ibnd_scan, +static ibnd_node_t *create_node(ibnd_fabric_t * fabric, ibnd_scan_t * scan, ibnd_node_t * temp, ib_portid_t * path, int dist) { @@ -418,7 +418,7 @@ static ibnd_node_t *create_node(ibnd_fabric_t * fabric, ibnd_scan_t * ibnd_scan, fabric->nodes = (ibnd_node_t *) node; add_to_type_list(node, fabric); - add_to_nodedist(node, ibnd_scan, dist); + add_to_nodedist(node, scan, dist); return node; } @@ -478,7 +478,7 @@ static void link_ports(ibnd_node_t * node, ibnd_port_t * port, } static int get_remote_node(struct ibmad_port *ibmad_port, - ibnd_fabric_t * fabric, ibnd_scan_t * ibnd_scan, + ibnd_fabric_t * fabric, ibnd_scan_t * scan, ibnd_node_t * node, ibnd_port_t * port, ib_portid_t * path, int portnum, int dist) { @@ -498,7 +498,7 @@ static int get_remote_node(struct ibmad_port *ibmad_port, != IB_PORT_PHYS_STATE_LINKUP) return 1; /* positive == non-fatal error */ - if (portnum > 0 && extend_dpath(ibmad_port, fabric, ibnd_scan, + if (portnum > 0 && extend_dpath(ibmad_port, fabric, scan, path, portnum) < 0) return -1; @@ -512,8 +512,8 @@ static int get_remote_node(struct ibmad_port *ibmad_port, oldnode = find_existing_node(fabric, &node_buf); if (oldnode) remotenode = oldnode; - else if (!(remotenode = create_node(fabric, ibnd_scan, &node_buf, - path, dist + 1))) { + else if (!(remotenode = create_node(fabric, scan, &node_buf, path, + dist + 1))) { rc = -1; goto error; } @@ -552,7 +552,7 @@ ibnd_fabric_t *ibnd_discover_fabric(struct ibmad_port * ibmad_port, int dist = 0; ib_portid_t *path; int max_hops = MAXHOPS - 1; /* default find everything */ - ibnd_scan_t ibnd_scan; + ibnd_scan_t scan; if (_check_ibmad_port(ibmad_port) < 0) return (NULL); @@ -575,7 +575,7 @@ ibnd_fabric_t *ibnd_discover_fabric(struct ibmad_port * ibmad_port, memset(fabric, 0, sizeof(*fabric)); - memset(&ibnd_scan, '\0', sizeof(ibnd_scan_t)); + memset(&scan, '\0', sizeof(ibnd_scan_t)); IBND_DEBUG("from %s\n", portid2str(from)); @@ -587,7 +587,7 @@ ibnd_fabric_t *ibnd_discover_fabric(struct ibmad_port * ibmad_port, goto error; } - node = create_node(fabric, &ibnd_scan, &node_buf, from, 0); + node = create_node(fabric, &scan, &node_buf, from, 0); if (!node) goto error; @@ -597,7 +597,7 @@ ibnd_fabric_t *ibnd_discover_fabric(struct ibmad_port * ibmad_port, if (!port) goto error; - rc = get_remote_node(ibmad_port, fabric, &ibnd_scan, node, port, from, + rc = get_remote_node(ibmad_port, fabric, &scan, node, port, from, mad_get_field(node->info, 0, IB_NODE_LOCAL_PORT_F), 0); if (rc < 0) @@ -607,7 +607,7 @@ ibnd_fabric_t *ibnd_discover_fabric(struct ibmad_port * ibmad_port, for (dist = 0; dist <= max_hops; dist++) { - for (node = ibnd_scan.nodesdist[dist]; node; node = node->dnext) { + for (node = scan.nodesdist[dist]; node; node = node->dnext) { path = &node->path_portid; @@ -643,14 +643,14 @@ ibnd_fabric_t *ibnd_discover_fabric(struct ibmad_port * ibmad_port, IB_NODE_PORT_GUID_F); } - if (get_remote_node(ibmad_port, fabric, &ibnd_scan, + if (get_remote_node(ibmad_port, fabric, &scan, node, port, path, i, dist) < 0) goto error; } } } - if (group_nodes(fabric, &ibnd_scan)) + if (group_nodes(fabric, &scan)) goto error; return ((ibnd_fabric_t *) fabric);