diff options
author | Eli Cohen <eli@mellanox.co.il> | 2010-02-08 11:40:37 +0000 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2010-02-24 14:54:10 -0800 |
commit | 920d706c892e8f8cfff95f46aeb95fc6344f0bd5 (patch) | |
tree | a4219e9b0bc285bb080b4239107cf9e391017c66 /include/rdma/ib_pack.h | |
parent | ccbe9f0b11b137c9453771a7ca3bf417dc7ce152 (diff) | |
download | op-kernel-dev-920d706c892e8f8cfff95f46aeb95fc6344f0bd5.zip op-kernel-dev-920d706c892e8f8cfff95f46aeb95fc6344f0bd5.tar.gz |
IB/core: Fix and clean up ib_ud_header_init()
ib_ud_header_init() first clears header and then fills up the various
fields. Later on, it tests header->immediate_present, which it has
already cleared, so the condition is always false. Fix this by adding
an immediate_present parameter and setting header->immediate_present
as is done with grh_present. Also remove unused calculation of
header_len.
Signed-off-by: Eli Cohen <eli@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'include/rdma/ib_pack.h')
-rw-r--r-- | include/rdma/ib_pack.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/rdma/ib_pack.h b/include/rdma/ib_pack.h index d7fc45c..cbb50f4 100644 --- a/include/rdma/ib_pack.h +++ b/include/rdma/ib_pack.h @@ -232,6 +232,7 @@ void ib_unpack(const struct ib_field *desc, void ib_ud_header_init(int payload_bytes, int grh_present, + int immediate_present, struct ib_ud_header *header); int ib_ud_header_pack(struct ib_ud_header *header, |