diff options
| author | Or Gerlitz <ogerlitz@mellanox.com> | 2012-01-11 19:03:51 +0200 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2012-03-08 12:34:27 -0800 |
| commit | d927d505c59a0c7353343174e6225c43c61fba6d (patch) | |
| tree | 8fe928291acb9c6975f7c03514566933edeef7d5 /include/rdma | |
| parent | 3eae7c9f97415ac1f5ab5db4eeb03cde689eb427 (diff) | |
| download | op-kernel-dev-d927d505c59a0c7353343174e6225c43c61fba6d.zip op-kernel-dev-d927d505c59a0c7353343174e6225c43c61fba6d.tar.gz | |
IB: Change CQE "csum_ok" field to a bit flag
Use a bit in wc_flags rather then a whole integer to hold the
"checksum OK" flag. By itself, this change doesn't reduce the size of
struct ib_wc on 64bit machines -- it stays on 56 bytes because of
padding. However, it will allow to add more fields in the future
without enlarging the struct. Also, it will let us have a unified
approach with future libibverbs checksum offload reporting, because a
bit flag doesn't break the library ABI.
This patch was suggested during conversation with Liran Liss
<liranl@mellanox.com>.
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Reviewed-by: Sean Hefty <sean.hefty@intel.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'include/rdma')
| -rw-r--r-- | include/rdma/ib_verbs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h index ff22a73..c3cca5a 100644 --- a/include/rdma/ib_verbs.h +++ b/include/rdma/ib_verbs.h @@ -518,6 +518,7 @@ enum ib_wc_flags { IB_WC_GRH = 1, IB_WC_WITH_IMM = (1<<1), IB_WC_WITH_INVALIDATE = (1<<2), + IB_WC_IP_CSUM_OK = (1<<3), }; struct ib_wc { @@ -538,7 +539,6 @@ struct ib_wc { u8 sl; u8 dlid_path_bits; u8 port_num; /* valid only for DR SMPs on switches */ - int csum_ok; }; enum ib_cq_notify_flags { |
