Commit 972d512a authored by Sean Hefty's avatar Sean Hefty Committed by Roland Dreier

[IB] Add MAD data field size definitions

Clean up code by using enums instead of hard-coded magic numbers.
Signed-off-by: default avatarSean Hefty <sean.hefty@intel.com>
Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
parent 2fe9f798
...@@ -583,6 +583,7 @@ static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr) ...@@ -583,6 +583,7 @@ static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr)
{ {
struct ib_rmpp_mad *rmpp_mad; struct ib_rmpp_mad *rmpp_mad;
int timeout; int timeout;
u32 paylen;
rmpp_mad = (struct ib_rmpp_mad *)mad_send_wr->send_wr.wr.ud.mad_hdr; rmpp_mad = (struct ib_rmpp_mad *)mad_send_wr->send_wr.wr.ud.mad_hdr;
ib_set_rmpp_flags(&rmpp_mad->rmpp_hdr, IB_MGMT_RMPP_FLAG_ACTIVE); ib_set_rmpp_flags(&rmpp_mad->rmpp_hdr, IB_MGMT_RMPP_FLAG_ACTIVE);
...@@ -590,11 +591,9 @@ static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr) ...@@ -590,11 +591,9 @@ static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr)
if (mad_send_wr->seg_num == 1) { if (mad_send_wr->seg_num == 1) {
rmpp_mad->rmpp_hdr.rmpp_rtime_flags |= IB_MGMT_RMPP_FLAG_FIRST; rmpp_mad->rmpp_hdr.rmpp_rtime_flags |= IB_MGMT_RMPP_FLAG_FIRST;
rmpp_mad->rmpp_hdr.paylen_newwin = paylen = mad_send_wr->total_seg * IB_MGMT_RMPP_DATA -
cpu_to_be32(mad_send_wr->total_seg * mad_send_wr->pad;
(sizeof(struct ib_rmpp_mad) - rmpp_mad->rmpp_hdr.paylen_newwin = cpu_to_be32(paylen);
offsetof(struct ib_rmpp_mad, data)) -
mad_send_wr->pad);
mad_send_wr->sg_list[0].length = sizeof(struct ib_rmpp_mad); mad_send_wr->sg_list[0].length = sizeof(struct ib_rmpp_mad);
} else { } else {
mad_send_wr->send_wr.num_sge = 2; mad_send_wr->send_wr.num_sge = 2;
...@@ -608,10 +607,8 @@ static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr) ...@@ -608,10 +607,8 @@ static int send_next_seg(struct ib_mad_send_wr_private *mad_send_wr)
if (mad_send_wr->seg_num == mad_send_wr->total_seg) { if (mad_send_wr->seg_num == mad_send_wr->total_seg) {
rmpp_mad->rmpp_hdr.rmpp_rtime_flags |= IB_MGMT_RMPP_FLAG_LAST; rmpp_mad->rmpp_hdr.rmpp_rtime_flags |= IB_MGMT_RMPP_FLAG_LAST;
rmpp_mad->rmpp_hdr.paylen_newwin = paylen = IB_MGMT_RMPP_DATA - mad_send_wr->pad;
cpu_to_be32(sizeof(struct ib_rmpp_mad) - rmpp_mad->rmpp_hdr.paylen_newwin = cpu_to_be32(paylen);
offsetof(struct ib_rmpp_mad, data) -
mad_send_wr->pad);
} }
/* 2 seconds for an ACK until we can find the packet lifetime */ /* 2 seconds for an ACK until we can find the packet lifetime */
......
...@@ -108,6 +108,13 @@ ...@@ -108,6 +108,13 @@
#define IB_QP1_QKEY 0x80010000 #define IB_QP1_QKEY 0x80010000
#define IB_QP_SET_QKEY 0x80000000 #define IB_QP_SET_QKEY 0x80000000
enum {
IB_MGMT_MAD_DATA = 232,
IB_MGMT_RMPP_DATA = 220,
IB_MGMT_VENDOR_DATA = 216,
IB_MGMT_SA_DATA = 200
};
struct ib_mad_hdr { struct ib_mad_hdr {
u8 base_version; u8 base_version;
u8 mgmt_class; u8 mgmt_class;
...@@ -149,20 +156,20 @@ struct ib_sa_hdr { ...@@ -149,20 +156,20 @@ struct ib_sa_hdr {
struct ib_mad { struct ib_mad {
struct ib_mad_hdr mad_hdr; struct ib_mad_hdr mad_hdr;
u8 data[232]; u8 data[IB_MGMT_MAD_DATA];
}; };
struct ib_rmpp_mad { struct ib_rmpp_mad {
struct ib_mad_hdr mad_hdr; struct ib_mad_hdr mad_hdr;
struct ib_rmpp_hdr rmpp_hdr; struct ib_rmpp_hdr rmpp_hdr;
u8 data[220]; u8 data[IB_MGMT_RMPP_DATA];
}; };
struct ib_sa_mad { struct ib_sa_mad {
struct ib_mad_hdr mad_hdr; struct ib_mad_hdr mad_hdr;
struct ib_rmpp_hdr rmpp_hdr; struct ib_rmpp_hdr rmpp_hdr;
struct ib_sa_hdr sa_hdr; struct ib_sa_hdr sa_hdr;
u8 data[200]; u8 data[IB_MGMT_SA_DATA];
} __attribute__ ((packed)); } __attribute__ ((packed));
struct ib_vendor_mad { struct ib_vendor_mad {
...@@ -170,7 +177,7 @@ struct ib_vendor_mad { ...@@ -170,7 +177,7 @@ struct ib_vendor_mad {
struct ib_rmpp_hdr rmpp_hdr; struct ib_rmpp_hdr rmpp_hdr;
u8 reserved; u8 reserved;
u8 oui[3]; u8 oui[3];
u8 data[216]; u8 data[IB_MGMT_VENDOR_DATA];
}; };
struct ib_class_port_info struct ib_class_port_info
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment