The following renames 'multipacket' to 'rmpp' for consistency.
The most notable name changes are:

        ib_mad_multipacket_seg --> ib_rmpp_segment
        ib_mad_get_multipacket_seg() --> ib_get_rmpp_segment()

Signed-off-by: Sean Hefty <[EMAIL PROTECTED]>

---

Index: include/rdma/ib_mad.h
===================================================================
--- include/rdma/ib_mad.h       (revision 5455)
+++ include/rdma/ib_mad.h       (working copy)
@@ -141,7 +141,7 @@ struct ib_rmpp_hdr {
        __be32  paylen_newwin;
 };
 
-struct ib_mad_multipacket_seg {
+struct ib_rmpp_segment {
        struct list_head list;
        u32 num;
        u16 size;
@@ -597,14 +597,14 @@ struct ib_mad_send_buf * ib_create_send_
                                            gfp_t gfp_mask);
 
 /**
- * *ib_mad_get_multipacket_seg - returns a given RMPP segment.
+ * *ib_get_rmpp_segment - returns a given RMPP segment.
  * @send_buf: Previously allocated send data buffer.
  * @seg_num: number of segment to return
  *
- * This routine returns a pointer to a segment of a multipacket RMPP message.
+ * This routine returns a pointer to a segment of an RMPP message.
  */
-struct ib_mad_multipacket_seg
-*ib_mad_get_multipacket_seg(struct ib_mad_send_buf *send_buf, int seg_num);
+struct ib_rmpp_segment *ib_get_rmpp_segment(struct ib_mad_send_buf *send_buf,
+                                           int seg_num);
 
 /**
  * ib_free_send_mad - Returns data buffers used to send a MAD.
Index: core/mad_rmpp.c
===================================================================
--- core/mad_rmpp.c     (revision 5455)
+++ core/mad_rmpp.c     (working copy)
@@ -535,7 +535,7 @@ start_rmpp(struct ib_mad_agent_private *
 static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr)
 {
        struct ib_rmpp_mad *rmpp_mad;
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
        int timeout;
        u32 paylen;
 
@@ -549,8 +549,7 @@ static int send_next_seg(struct ib_mad_s
                         mad_send_wr->pad;
                rmpp_mad->rmpp_hdr.paylen_newwin = cpu_to_be32(paylen);
        } else {
-               seg = ib_rmpp_get_multipacket_seg(mad_send_wr,
-                                                 mad_send_wr->seg_num);
+               seg = ib_get_segment(mad_send_wr, mad_send_wr->seg_num);
                if (!seg) {
                        printk(KERN_ERR PFX "send_next_seg: "
                               "could not find segment %d\n",
@@ -605,12 +604,12 @@ out:
 
 static inline void adjust_last_ack(struct ib_mad_send_wr_private *wr)
 {
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
 
        if (wr->last_ack < 2)
                return;
        else if (!wr->last_ack_seg)
-               list_for_each_entry(seg, &wr->multipacket_list, list) {
+               list_for_each_entry(seg, &wr->rmpp_list, list) {
                        if (wr->last_ack == seg->num) {
                                wr->last_ack_seg = seg;
                                break;
@@ -902,7 +901,7 @@ int ib_retry_rmpp(struct ib_mad_send_wr_
                return IB_RMPP_RESULT_PROCESSED;
 
        mad_send_wr->seg_num = mad_send_wr->last_ack + 1;
-       mad_send_wr->seg_num_seg = mad_send_wr->last_ack_seg;
+       mad_send_wr->cur_seg = mad_send_wr->last_ack_seg;
 
        ret = send_next_seg(mad_send_wr);
        if (ret)
Index: core/user_mad.c
===================================================================
--- core/user_mad.c     (revision 5455)
+++ core/user_mad.c     (working copy)
@@ -194,7 +194,7 @@ static int copy_recv_mad(struct ib_mad_r
        struct ib_mad_recv_buf *seg_buf;
        struct ib_rmpp_mad *rmpp_mad;
        void *data;
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
        int size, len, offset;
        u8 flags;
 
@@ -220,7 +220,7 @@ static int copy_recv_mad(struct ib_mad_r
                                size = len;
                        else
                                size = sizeof(*rmpp_mad) - offset;
-                       seg = kmalloc(sizeof(struct ib_mad_multipacket_seg) +
+                       seg = kmalloc(sizeof(struct ib_rmpp_segment) +
                                      sizeof(struct ib_rmpp_mad) - offset,
                                      GFP_KERNEL);
                        if (!seg)
@@ -235,7 +235,7 @@ static int copy_recv_mad(struct ib_mad_r
 
 static void free_packet(struct ib_umad_packet *packet)
 {
-       struct ib_mad_multipacket_seg *seg, *tmp;
+       struct ib_rmpp_segment *seg, *tmp;
 
        list_for_each_entry_safe(seg, tmp, &packet->seg_list, list) {
                list_del(&seg->list);
@@ -322,7 +322,7 @@ static ssize_t ib_umad_read(struct file 
                            size_t count, loff_t *pos)
 {
        struct ib_umad_file *file = filp->private_data;
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
        struct ib_umad_packet *packet;
        ssize_t ret;
 
@@ -411,7 +411,7 @@ static ssize_t ib_umad_write(struct file
        int ret, length, hdr_len, copy_offset;
        int rmpp_active, has_rmpp_header;
        int s, seg_num;
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
 
        if (count < sizeof (struct ib_user_mad) + IB_MGMT_RMPP_HDR)
                return -EINVAL;
@@ -523,7 +523,7 @@ static ssize_t ib_umad_write(struct file
        if (length > 0) {
                buf +=  sizeof (struct ib_user_mad) + sizeof(struct ib_mad);
                for (seg_num = 2; length > 0; ++seg_num, buf += s, length -= s) 
{
-                       seg = ib_mad_get_multipacket_seg(packet->msg, seg_num);
+                       seg = ib_get_rmpp_segment(packet->msg, seg_num);
                        BUG_ON(!seg);
                        s = min_t(int, length, seg->size);
                        if (copy_from_user(seg->data, buf, s)) {
Index: core/mad.c
===================================================================
--- core/mad.c  (revision 5455)
+++ core/mad.c  (working copy)
@@ -779,23 +779,22 @@ static int get_buf_length(int hdr_len, i
        return hdr_len + data_len + pad;
 }
 
-static void free_send_multipacket_list(struct ib_mad_send_wr_private *
-                                      mad_send_wr)
+static void free_send_rmpp_list(struct ib_mad_send_wr_private *mad_send_wr)
 {
-       struct ib_mad_multipacket_seg *s, *t;
+       struct ib_rmpp_segment *s, *t;
 
-       list_for_each_entry_safe(s, t, &mad_send_wr->multipacket_list, list) {
+       list_for_each_entry_safe(s, t, &mad_send_wr->rmpp_list, list) {
                list_del(&s->list);
                kfree(s);
        }
 }
 
-static inline int alloc_send_rmpp_segs(struct ib_mad_send_wr_private *send_wr,
+static inline int alloc_send_rmpp_list(struct ib_mad_send_wr_private *send_wr,
                                       int message_size, int hdr_len,
                                       int data_len, u8 rmpp_version,
                                       gfp_t gfp_mask)
 {
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
        struct ib_rmpp_mad *rmpp_mad = send_wr->send_buf.mad;
        int seg_size, i = 2;
 
@@ -809,19 +808,19 @@ static inline int alloc_send_rmpp_segs(s
        message_size -= sizeof(struct ib_mad);
        seg_size = sizeof(struct ib_mad) - hdr_len;
        while (message_size > 0) {
-               seg = kmalloc(sizeof(struct ib_mad_multipacket_seg) + seg_size,
+               seg = kmalloc(sizeof(struct ib_rmpp_segment) + seg_size,
                                     gfp_mask);
                if (!seg) {
                        printk(KERN_ERR "ib_create_send_mad: RMPP mem "
                               "alloc failed for len %zd, gfp %#x\n",
-                              sizeof(struct ib_mad_multipacket_seg) + seg_size,
+                              sizeof(struct ib_rmpp_segment) + seg_size,
                               gfp_mask);
-                       free_send_multipacket_list(send_wr);
+                       free_send_rmpp_list(send_wr);
                        return -ENOMEM;
                }
                seg->size = seg_size;
                seg->num = i++;
-               list_add_tail(&seg->list, &send_wr->multipacket_list);
+               list_add_tail(&seg->list, &send_wr->rmpp_list);
                message_size -= seg_size;
        }
        return 0;
@@ -854,7 +853,7 @@ struct ib_mad_send_buf * ib_create_send_
                return ERR_PTR(-ENOMEM);
 
        mad_send_wr = buf + sizeof(struct ib_mad);
-       INIT_LIST_HEAD(&mad_send_wr->multipacket_list);
+       INIT_LIST_HEAD(&mad_send_wr->rmpp_list);
        mad_send_wr->send_buf.mad = buf;
        mad_send_wr->mad_payload = buf + hdr_len;
 
@@ -873,10 +872,10 @@ struct ib_mad_send_buf * ib_create_send_
        mad_send_wr->send_wr.wr.ud.remote_qkey = IB_QP_SET_QKEY;
        mad_send_wr->send_wr.wr.ud.pkey_index = pkey_index;
        mad_send_wr->last_ack_seg = NULL;
-       mad_send_wr->seg_num_seg = NULL;
+       mad_send_wr->cur_seg = NULL;
 
        if (rmpp_active) {
-               ret = alloc_send_rmpp_segs(mad_send_wr, message_size, hdr_len,
+               ret = alloc_send_rmpp_list(mad_send_wr, message_size, hdr_len,
                                           data_len, mad_agent->rmpp_version,
                                           gfp_mask);
                if (ret) {
@@ -891,48 +890,45 @@ struct ib_mad_send_buf * ib_create_send_
 }
 EXPORT_SYMBOL(ib_create_send_mad);
 
-struct ib_mad_multipacket_seg
-*ib_rmpp_get_multipacket_seg(struct ib_mad_send_wr_private *wr, int seg_num)
+struct ib_rmpp_segment *ib_get_segment(struct ib_mad_send_wr_private 
*mad_send_wr,
+                                      int seg_num)
 {
-       struct ib_mad_multipacket_seg *seg;
+       struct ib_rmpp_segment *seg;
 
        if (seg_num == 2) {
-               wr->seg_num_seg =
-                       container_of(wr->multipacket_list.next,
-                                    struct ib_mad_multipacket_seg, list);
-               return wr->seg_num_seg;
+               mad_send_wr->cur_seg = container_of(mad_send_wr->rmpp_list.next,
+                                                   struct ib_rmpp_segment, 
list);
+               return mad_send_wr->cur_seg;
        }
 
        /* get first list entry if was not already done */
-       if (!wr->seg_num_seg)
-               wr->seg_num_seg =
-                       container_of(wr->multipacket_list.next,
-                                    struct ib_mad_multipacket_seg, list);
-
-       if (wr->seg_num_seg->num == seg_num)
-               return wr->seg_num_seg;
-       else if (wr->seg_num_seg->num < seg_num) {
-               list_for_each_entry(seg, &wr->seg_num_seg->list, list) {
+       if (!mad_send_wr->cur_seg)
+               mad_send_wr->cur_seg = container_of(mad_send_wr->rmpp_list.next,
+                                                   struct ib_rmpp_segment, 
list);
+
+       if (mad_send_wr->cur_seg->num == seg_num)
+               return mad_send_wr->cur_seg;
+       else if (mad_send_wr->cur_seg->num < seg_num) {
+               list_for_each_entry(seg, &mad_send_wr->cur_seg->list, list) {
                        if (seg->num == seg_num) {
-                               wr->seg_num_seg = seg;
-                               return wr->seg_num_seg;
+                               mad_send_wr->cur_seg = seg;
+                               return mad_send_wr->cur_seg;
                        }
                }
-               return NULL;
        } else {
-               list_for_each_entry_reverse(seg, &wr->seg_num_seg->list, list) {
+               list_for_each_entry_reverse(seg, &mad_send_wr->cur_seg->list,
+                                           list) {
                        if (seg->num == seg_num) {
-                               wr->seg_num_seg = seg;
-                               return wr->seg_num_seg;
+                               mad_send_wr->cur_seg = seg;
+                               return mad_send_wr->cur_seg;
                        }
                }
-               return NULL;
        }
        return NULL;
 }
 
-struct ib_mad_multipacket_seg
-*ib_mad_get_multipacket_seg(struct ib_mad_send_buf *send_buf, int seg_num)
+struct ib_rmpp_segment *ib_get_rmpp_segment(struct ib_mad_send_buf *send_buf,
+                                           int seg_num)
 {
        struct ib_mad_send_wr_private *wr;
 
@@ -940,9 +936,9 @@ struct ib_mad_multipacket_seg
                return NULL;
 
        wr = container_of(send_buf, struct ib_mad_send_wr_private, send_buf);
-       return ib_rmpp_get_multipacket_seg(wr, seg_num);
+       return ib_get_segment(wr, seg_num);
 }
-EXPORT_SYMBOL(ib_mad_get_multipacket_seg);
+EXPORT_SYMBOL(ib_get_rmpp_segment);
 
 void ib_free_send_mad(struct ib_mad_send_buf *send_buf)
 {
@@ -954,7 +950,7 @@ void ib_free_send_mad(struct ib_mad_send
        mad_send_wr = container_of(send_buf, struct ib_mad_send_wr_private,
                                   send_buf);
 
-       free_send_multipacket_list(mad_send_wr);
+       free_send_rmpp_list(mad_send_wr);
        kfree(send_buf->mad);
        if (atomic_dec_and_test(&mad_agent_priv->refcount))
                wake_up(&mad_agent_priv->wait);
Index: core/mad_priv.h
===================================================================
--- core/mad_priv.h     (revision 5455)
+++ core/mad_priv.h     (working copy)
@@ -132,9 +132,9 @@ struct ib_mad_send_wr_private {
        enum ib_wc_status status;
 
        /* RMPP control */
-       struct list_head multipacket_list;
-       struct ib_mad_multipacket_seg *last_ack_seg;
-       struct ib_mad_multipacket_seg *seg_num_seg;
+       struct list_head rmpp_list;
+       struct ib_rmpp_segment *last_ack_seg;
+       struct ib_rmpp_segment *cur_seg;
        int last_ack;
        int seg_num;
        int newwin;
@@ -224,7 +224,7 @@ void ib_mark_mad_done(struct ib_mad_send
 void ib_reset_mad_timeout(struct ib_mad_send_wr_private *mad_send_wr,
                          int timeout_ms);
 
-struct ib_mad_multipacket_seg
-*ib_rmpp_get_multipacket_seg(struct ib_mad_send_wr_private *wr, int seg_num);
+struct ib_rmpp_segment *ib_get_segment(struct ib_mad_send_wr_private 
*mad_send_wr,
+                                      int seg_num);
 
 #endif /* __IB_MAD_PRIV_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