mad: Restructure smi as shared between mad and agent
(adds new files smi.h, smi.c, and agent.h)

Index: agent.c
===================================================================
--- agent.c     (revision 1161)
+++ agent.c     (working copy)
@@ -24,6 +24,7 @@
 */
 
 #include <ib_smi.h>
+#include "smi.h"
 #include "agent_priv.h"
 #include "mad_priv.h"
 #include <asm/bug.h>
@@ -32,210 +33,7 @@
 static spinlock_t ib_agent_port_list_lock = SPIN_LOCK_UNLOCKED;
 static LIST_HEAD(ib_agent_port_list);
 
-/*
- * Fixup a directed route SMP for sending.  Return 0 if the SMP should be
- * discarded.
- */
-int smi_handle_dr_smp_send(struct ib_smp *smp,
-                          u8 node_type,
-                          int port_num)
-{
-       u8 hop_ptr, hop_cnt;
 
-       hop_ptr = smp->hop_ptr;
-       hop_cnt = smp->hop_cnt;
-
-       /* See section 14.2.2.2, Vol 1 IB spec */
-       if (!ib_get_smp_direction(smp)) {
-               /* C14-9:1 */
-               if (hop_cnt && hop_ptr == 0) {
-                       smp->hop_ptr++;
-                       return (smp->initial_path[smp->hop_ptr] == 
-                               port_num);
-               }
-
-               /* C14-9:2 */
-               if (hop_ptr && hop_ptr < hop_cnt) {
-                       if (node_type != IB_NODE_SWITCH)
-                               return 0;
-                       
-                       /* smp->return_path set when received */
-                       smp->hop_ptr++;
-                       return (smp->initial_path[smp->hop_ptr] == 
-                               port_num);
-               }
-
-               /* C14-9:3 -- We're at the end of the DR segment of path */
-               if (hop_ptr == hop_cnt) {
-                       /* smp->return_path set when received */
-                       smp->hop_ptr++;
-                       return (node_type == IB_NODE_SWITCH ||
-                               smp->dr_dlid == IB_LID_PERMISSIVE);
-               }
-
-               /* C14-9:4 -- hop_ptr = hop_cnt + 1 -> give to SMA/SM. */
-               /* C14-9:5 -- Fail unreasonable hop pointer. */
-               return (hop_ptr == hop_cnt + 1);
-
-       } else {
-               /* C14-13:1 */
-               if (hop_cnt && hop_ptr == hop_cnt + 1) {
-                       smp->hop_ptr--;
-                       return (smp->return_path[smp->hop_ptr] == 
-                               port_num);
-               }
-
-               /* C14-13:2 */
-               if (2 <= hop_ptr && hop_ptr <= hop_cnt) {
-                       if (node_type != IB_NODE_SWITCH)
-                               return 0;
-
-                       smp->hop_ptr--;
-                       return (smp->return_path[smp->hop_ptr] == 
-                               port_num);
-               }
-
-               /* C14-13:3 -- at the end of the DR segment of path */
-               if (hop_ptr == 1) {
-                       smp->hop_ptr--;
-                       /* C14-13:3 -- SMPs destined for SM shouldn't be here */
-                       return (node_type == IB_NODE_SWITCH ||
-                               smp->dr_slid == IB_LID_PERMISSIVE);
-               }
-
-               /* C14-13:4 -- hop_ptr = 0 -> should have gone to SM. */
-               /* C14-13:5 -- Check for unreasonable hop pointer. */
-               return 0;
-       }
-}
-
-/*
- * Return 1 if the SMP should be handled by the local SMA via process_mad.
- */
-static inline int smi_check_local_smp(struct ib_mad_agent *mad_agent,
-                                     struct ib_smp *smp)
-{
-       /* C14-9:3 -- We're at the end of the DR segment of path */
-       /* C14-9:4 -- Hop Pointer = Hop Count + 1 -> give to SMA/SM. */
-       return ((mad_agent->device->process_mad &&
-               !ib_get_smp_direction(smp) &&
-               (smp->hop_ptr == smp->hop_cnt + 1)));
-}
-
-/*
- * Adjust information for a received SMP.  Return 0 if the SMP should be
- * dropped.
- */
-int smi_handle_dr_smp_recv(struct ib_smp *smp,
-                          u8 node_type,
-                          int port_num,
-                          int phys_port_cnt)
-{
-       u8 hop_ptr, hop_cnt;
-
-       hop_ptr = smp->hop_ptr;
-       hop_cnt = smp->hop_cnt;
-
-       /* See section 14.2.2.2, Vol 1 IB spec */
-       if (!ib_get_smp_direction(smp)) {
-               /* C14-9:1 -- sender should have incremented hop_ptr */
-               if (hop_cnt && hop_ptr == 0)
-                       return 0;
-
-               /* C14-9:2 -- intermediate hop */
-               if (hop_ptr && hop_ptr < hop_cnt) {
-                       if (node_type != IB_NODE_SWITCH)
-                               return 0;
-
-                       smp->return_path[hop_ptr] = port_num;
-                       /* smp->hop_ptr updated when sending */
-                       return (smp->initial_path[hop_ptr+1] <= phys_port_cnt);
-               }
-
-               /* C14-9:3 -- We're at the end of the DR segment of path */
-               if (hop_ptr == hop_cnt) {
-                       if (hop_cnt)
-                               smp->return_path[hop_ptr] = port_num;
-                       /* smp->hop_ptr updated when sending */
-
-                       return (node_type == IB_NODE_SWITCH ||
-                               smp->dr_dlid == IB_LID_PERMISSIVE);
-               }
-               
-               /* C14-9:4 -- hop_ptr = hop_cnt + 1 -> give to SMA/SM. */
-               /* C14-9:5 -- fail unreasonable hop pointer. */
-               return (hop_ptr == hop_cnt + 1);
-
-       } else {
-
-               /* C14-13:1 */
-               if (hop_cnt && hop_ptr == hop_cnt + 1) {
-                       smp->hop_ptr--;
-                       return (smp->return_path[smp->hop_ptr] ==
-                               port_num);
-               }
-
-               /* C14-13:2 */
-               if (2 <= hop_ptr && hop_ptr <= hop_cnt) {
-                       if (node_type != IB_NODE_SWITCH)
-                               return 0;
-
-                       /* smp->hop_ptr updated when sending */
-                       return (smp->return_path[hop_ptr-1] <= phys_port_cnt);
-               }
-
-               /* C14-13:3 -- We're at the end of the DR segment of path */
-               if (hop_ptr == 1) {
-                       if (smp->dr_slid == IB_LID_PERMISSIVE) {
-                               /* giving SMP to SM - update hop_ptr */
-                               smp->hop_ptr--;
-                               return 1;
-                       }
-                       /* smp->hop_ptr updated when sending */
-                       return (node_type == IB_NODE_SWITCH);
-               }
-
-               /* C14-13:4 -- hop_ptr = 0 -> give to SM. */
-               /* C14-13:5 -- Check for unreasonable hop pointer. */
-               return (hop_ptr == 0);
-       }
-}
-
-/*
- * Return 1 if the received DR SMP should be forwarded to the send queue.
- * Return 0 if the SMP should be completed up the stack.
- */
-int smi_check_forward_dr_smp(struct ib_smp *smp)
-{
-       u8 hop_ptr, hop_cnt;
-
-       hop_ptr = smp->hop_ptr;
-       hop_cnt = smp->hop_cnt;
-
-       if (!ib_get_smp_direction(smp)) {
-               /* C14-9:2 -- intermediate hop */
-               if (hop_ptr && hop_ptr < hop_cnt)
-                       return 1;
-
-               /* C14-9:3 -- at the end of the DR segment of path */
-               if (hop_ptr == hop_cnt)
-                       return (smp->dr_dlid == IB_LID_PERMISSIVE);
-
-               /* C14-9:4 -- hop_ptr = hop_cnt + 1 -> give to SMA/SM. */
-               if (hop_ptr == hop_cnt + 1)
-                       return 1;
-       } else {
-               /* C14-13:2 */
-               if (2 <= hop_ptr && hop_ptr <= hop_cnt)
-                       return 1;
-
-               /* C14-13:3 -- at the end of the DR segment of path */
-               if (hop_ptr == 1)
-                       return (smp->dr_slid != IB_LID_PERMISSIVE);
-       }
-       return 0;
-}
-
 static inline struct ib_agent_port_private *
 __ib_get_agent_mad(struct ib_device *device, int port_num,
                   struct ib_mad_agent *mad_agent)
Index: mad.c
===================================================================
--- mad.c       (revision 1161)
+++ mad.c       (working copy)
@@ -56,6 +56,8 @@
 
 #include <ib_mad.h>
 #include "mad_priv.h"
+#include "smi.h"
+#include "agent.h"
 
 #include <linux/smp_lock.h>
 #include <linux/interrupt.h>
@@ -922,23 +924,6 @@
        }
 }
 
-extern int smi_handle_dr_smp_recv(struct ib_smp *smp,
-                                 u8 node_type,
-                                 int port_num,
-                                 int phys_port_cnt);
-extern int smi_check_forward_dr_smp(struct ib_smp *smp);
-extern int smi_handle_dr_smp_send(struct ib_smp *smp,
-                                 u8 node_type,
-                                 int port_num);
-extern int smi_check_local_dr_smp(struct ib_smp *smp,
-                                 struct ib_device *device,
-                                 int port_num);
-extern int agent_send(struct ib_mad *mad,
-                     struct ib_grh *grh,
-                     struct ib_wc *wc,
-                     struct ib_device *device,
-                     int port_num);
-
 static void ib_mad_recv_done_handler(struct ib_mad_port_private *port_priv,
                                     struct ib_wc *wc)
 {
@@ -1877,12 +1862,6 @@
        return 0;
 }
 
-
-extern int ib_agent_port_open(struct ib_device *device, int port_num,
-                             int phys_port_cnt);
-extern int ib_agent_port_close(struct ib_device *device, int port_num);
-
-
 static void ib_mad_init_device(struct ib_device *device)
 {
        int ret, num_ports, cur_port, i, ret2;
Index: agent.h
===================================================================
--- agent.h     (revision 0)
+++ agent.h     (revision 0)
@@ -0,0 +1,41 @@
+/*
+  This software is available to you under a choice of one of two
+  licenses.  You may choose to be licensed under the terms of the GNU
+  General Public License (GPL) Version 2, available at
+  <http://www.fsf.org/copyleft/gpl.html>, or the OpenIB.org BSD
+  license, available in the LICENSE.TXT file accompanying this
+  software.  These details are also available at
+  <http://openib.org/license.html>.
+
+  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+  EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+  MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+  NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+  BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+  ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+  SOFTWARE.
+
+  Copyright (c) 2004 Mellanox Technologies Ltd.  All rights reserved.
+  Copyright (c) 2004 Infinicon Corporation.  All rights reserved.
+  Copyright (c) 2004 Intel Corporation.  All rights reserved.
+  Copyright (c) 2004 Topspin Corporation.  All rights reserved.
+  Copyright (c) 2004 Voltaire Corporation.  All rights reserved.
+*/
+
+#ifndef __AGENT_H_
+#define __AGENT_H_
+
+extern int ib_agent_port_open(struct ib_device *device,
+                             int port_num,
+                             int phys_port_cnt);
+
+extern int ib_agent_port_close(struct ib_device *device, int port_num);
+
+extern int agent_send(struct ib_mad *mad,
+                     struct ib_grh *grh,
+                     struct ib_wc *wc,
+                     struct ib_device *device,
+                     int port_num);
+
+#endif /* __AGENT_H_ */
Index: smi.c
===================================================================
--- smi.c       (revision 0)
+++ smi.c       (revision 0)
@@ -0,0 +1,219 @@
+/*
+  This software is available to you under a choice of one of two
+  licenses.  You may choose to be licensed under the terms of the GNU
+  General Public License (GPL) Version 2, available at
+  <http://www.fsf.org/copyleft/gpl.html>, or the OpenIB.org BSD
+  license, available in the LICENSE.TXT file accompanying this
+  software.  These details are also available at
+  <http://openib.org/license.html>.
+
+  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+  EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+  MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+  NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+  BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+  ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+  SOFTWARE.
+
+  Copyright (c) 2004 Mellanox Technologies Ltd.  All rights reserved.
+  Copyright (c) 2004 Infinicon Corporation.  All rights reserved.
+  Copyright (c) 2004 Intel Corporation.  All rights reserved.
+  Copyright (c) 2004 Topspin Corporation.  All rights reserved.
+  Copyright (c) 2004 Voltaire Corporation.  All rights reserved.
+*/
+
+#include <ib_smi.h>
+
+
+/*
+ * Fixup a directed route SMP for sending.  Return 0 if the SMP should be
+ * discarded.
+ */
+int smi_handle_dr_smp_send(struct ib_smp *smp,
+                          u8 node_type,
+                          int port_num)
+{
+       u8 hop_ptr, hop_cnt;
+
+       hop_ptr = smp->hop_ptr;
+       hop_cnt = smp->hop_cnt;
+
+       /* See section 14.2.2.2, Vol 1 IB spec */
+       if (!ib_get_smp_direction(smp)) {
+               /* C14-9:1 */
+               if (hop_cnt && hop_ptr == 0) {
+                       smp->hop_ptr++;
+                       return (smp->initial_path[smp->hop_ptr] == 
+                               port_num);
+               }
+
+               /* C14-9:2 */
+               if (hop_ptr && hop_ptr < hop_cnt) {
+                       if (node_type != IB_NODE_SWITCH)
+                               return 0;
+                       
+                       /* smp->return_path set when received */
+                       smp->hop_ptr++;
+                       return (smp->initial_path[smp->hop_ptr] == 
+                               port_num);
+               }
+
+               /* C14-9:3 -- We're at the end of the DR segment of path */
+               if (hop_ptr == hop_cnt) {
+                       /* smp->return_path set when received */
+                       smp->hop_ptr++;
+                       return (node_type == IB_NODE_SWITCH ||
+                               smp->dr_dlid == IB_LID_PERMISSIVE);
+               }
+
+               /* C14-9:4 -- hop_ptr = hop_cnt + 1 -> give to SMA/SM. */
+               /* C14-9:5 -- Fail unreasonable hop pointer. */
+               return (hop_ptr == hop_cnt + 1);
+
+       } else {
+               /* C14-13:1 */
+               if (hop_cnt && hop_ptr == hop_cnt + 1) {
+                       smp->hop_ptr--;
+                       return (smp->return_path[smp->hop_ptr] == 
+                               port_num);
+               }
+
+               /* C14-13:2 */
+               if (2 <= hop_ptr && hop_ptr <= hop_cnt) {
+                       if (node_type != IB_NODE_SWITCH)
+                               return 0;
+
+                       smp->hop_ptr--;
+                       return (smp->return_path[smp->hop_ptr] == 
+                               port_num);
+               }
+
+               /* C14-13:3 -- at the end of the DR segment of path */
+               if (hop_ptr == 1) {
+                       smp->hop_ptr--;
+                       /* C14-13:3 -- SMPs destined for SM shouldn't be here */
+                       return (node_type == IB_NODE_SWITCH ||
+                               smp->dr_slid == IB_LID_PERMISSIVE);
+               }
+
+               /* C14-13:4 -- hop_ptr = 0 -> should have gone to SM. */
+               /* C14-13:5 -- Check for unreasonable hop pointer. */
+               return 0;
+       }
+}
+
+/*
+ * Adjust information for a received SMP.  Return 0 if the SMP should be
+ * dropped.
+ */
+int smi_handle_dr_smp_recv(struct ib_smp *smp,
+                          u8 node_type,
+                          int port_num,
+                          int phys_port_cnt)
+{
+       u8 hop_ptr, hop_cnt;
+
+       hop_ptr = smp->hop_ptr;
+       hop_cnt = smp->hop_cnt;
+
+       /* See section 14.2.2.2, Vol 1 IB spec */
+       if (!ib_get_smp_direction(smp)) {
+               /* C14-9:1 -- sender should have incremented hop_ptr */
+               if (hop_cnt && hop_ptr == 0)
+                       return 0;
+
+               /* C14-9:2 -- intermediate hop */
+               if (hop_ptr && hop_ptr < hop_cnt) {
+                       if (node_type != IB_NODE_SWITCH)
+                               return 0;
+
+                       smp->return_path[hop_ptr] = port_num;
+                       /* smp->hop_ptr updated when sending */
+                       return (smp->initial_path[hop_ptr+1] <= phys_port_cnt);
+               }
+
+               /* C14-9:3 -- We're at the end of the DR segment of path */
+               if (hop_ptr == hop_cnt) {
+                       if (hop_cnt)
+                               smp->return_path[hop_ptr] = port_num;
+                       /* smp->hop_ptr updated when sending */
+
+                       return (node_type == IB_NODE_SWITCH ||
+                               smp->dr_dlid == IB_LID_PERMISSIVE);
+               }
+               
+               /* C14-9:4 -- hop_ptr = hop_cnt + 1 -> give to SMA/SM. */
+               /* C14-9:5 -- fail unreasonable hop pointer. */
+               return (hop_ptr == hop_cnt + 1);
+
+       } else {
+
+               /* C14-13:1 */
+               if (hop_cnt && hop_ptr == hop_cnt + 1) {
+                       smp->hop_ptr--;
+                       return (smp->return_path[smp->hop_ptr] ==
+                               port_num);
+               }
+
+               /* C14-13:2 */
+               if (2 <= hop_ptr && hop_ptr <= hop_cnt) {
+                       if (node_type != IB_NODE_SWITCH)
+                               return 0;
+
+                       /* smp->hop_ptr updated when sending */
+                       return (smp->return_path[hop_ptr-1] <= phys_port_cnt);
+               }
+
+               /* C14-13:3 -- We're at the end of the DR segment of path */
+               if (hop_ptr == 1) {
+                       if (smp->dr_slid == IB_LID_PERMISSIVE) {
+                               /* giving SMP to SM - update hop_ptr */
+                               smp->hop_ptr--;
+                               return 1;
+                       }
+                       /* smp->hop_ptr updated when sending */
+                       return (node_type == IB_NODE_SWITCH);
+               }
+
+               /* C14-13:4 -- hop_ptr = 0 -> give to SM. */
+               /* C14-13:5 -- Check for unreasonable hop pointer. */
+               return (hop_ptr == 0);
+       }
+}
+
+/*
+ * Return 1 if the received DR SMP should be forwarded to the send queue.
+ * Return 0 if the SMP should be completed up the stack.
+ */
+int smi_check_forward_dr_smp(struct ib_smp *smp)
+{
+       u8 hop_ptr, hop_cnt;
+
+       hop_ptr = smp->hop_ptr;
+       hop_cnt = smp->hop_cnt;
+
+       if (!ib_get_smp_direction(smp)) {
+               /* C14-9:2 -- intermediate hop */
+               if (hop_ptr && hop_ptr < hop_cnt)
+                       return 1;
+
+               /* C14-9:3 -- at the end of the DR segment of path */
+               if (hop_ptr == hop_cnt)
+                       return (smp->dr_dlid == IB_LID_PERMISSIVE);
+
+               /* C14-9:4 -- hop_ptr = hop_cnt + 1 -> give to SMA/SM. */
+               if (hop_ptr == hop_cnt + 1)
+                       return 1;
+       } else {
+               /* C14-13:2 */
+               if (2 <= hop_ptr && hop_ptr <= hop_cnt)
+                       return 1;
+
+               /* C14-13:3 -- at the end of the DR segment of path */
+               if (hop_ptr == 1)
+                       return (smp->dr_slid != IB_LID_PERMISSIVE);
+       }
+       return 0;
+}
+
Index: Makefile
===================================================================
--- Makefile    (revision 1161)
+++ Makefile    (working copy)
@@ -17,6 +17,7 @@
 
 ib_mad-objs := \
     mad.o \
+    smi.o \
     agent.o
 
 ib_sa-objs := sa_query.o
Index: smi.h
===================================================================
--- smi.h       (revision 0)
+++ smi.h       (revision 0)
@@ -0,0 +1,54 @@
+/*
+  This software is available to you under a choice of one of two
+  licenses.  You may choose to be licensed under the terms of the GNU
+  General Public License (GPL) Version 2, available at
+  <http://www.fsf.org/copyleft/gpl.html>, or the OpenIB.org BSD
+  license, available in the LICENSE.TXT file accompanying this
+  software.  These details are also available at
+  <http://openib.org/license.html>.
+
+  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+  EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+  MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+  NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+  BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+  ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+  SOFTWARE.
+
+  Copyright (c) 2004 Mellanox Technologies Ltd.  All rights reserved.
+  Copyright (c) 2004 Infinicon Corporation.  All rights reserved.
+  Copyright (c) 2004 Intel Corporation.  All rights reserved.
+  Copyright (c) 2004 Topspin Corporation.  All rights reserved.
+  Copyright (c) 2004 Voltaire Corporation.  All rights reserved.
+*/
+
+#ifndef __SMI_H_
+#define __SMI_H_
+
+int smi_handle_dr_smp_recv(struct ib_smp *smp,
+                          u8 node_type,
+                          int port_num,
+                          int phys_port_cnt);
+extern int smi_check_forward_dr_smp(struct ib_smp *smp);
+extern int smi_handle_dr_smp_send(struct ib_smp *smp,
+                                 u8 node_type,
+                                 int port_num);
+extern int smi_check_local_dr_smp(struct ib_smp *smp,
+                                 struct ib_device *device,
+                                 int port_num);
+
+/*
+ * Return 1 if the SMP should be handled by the local SMA via process_mad.
+ */
+static inline int smi_check_local_smp(struct ib_mad_agent *mad_agent,
+                                     struct ib_smp *smp)
+{
+       /* C14-9:3 -- We're at the end of the DR segment of path */
+       /* C14-9:4 -- Hop Pointer = Hop Count + 1 -> give to SMA/SM. */
+       return ((mad_agent->device->process_mad &&
+               !ib_get_smp_direction(smp) &&
+               (smp->hop_ptr == smp->hop_cnt + 1)));
+}
+
+#endif /* __SMI_H_ */



_______________________________________________
openib-general mailing list
[EMAIL PROTECTED]
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to