tmp_opt is memset to 0. Thus, no need for another call to tcp_clear_options which simply sets some fields explicitly to 0 again.
Cc: Steve Wise <sw...@chelsio.com> Cc: Roland Dreier <rol...@kernel.org> Cc: Sean Hefty <sean.he...@intel.com> Cc: Hal Rosenstock <hal.rosenst...@gmail.com> Cc: linux-rdma@vger.kernel.org Signed-off-by: Christoph Paasch <christoph.paa...@uclouvain.be> --- drivers/infiniband/hw/cxgb4/cm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c index 96d7131ab974..2106a6dc62b3 100644 --- a/drivers/infiniband/hw/cxgb4/cm.c +++ b/drivers/infiniband/hw/cxgb4/cm.c @@ -3326,7 +3326,6 @@ static void build_cpl_pass_accept_req(struct sk_buff *skb, int stid , u8 tos) * to generate cpl_pass_accept_req. */ memset(&tmp_opt, 0, sizeof(tmp_opt)); - tcp_clear_options(&tmp_opt); tcp_parse_options(skb, &tmp_opt, 0, NULL); req = (struct cpl_pass_accept_req *)__skb_push(skb, sizeof(*req)); -- 1.9.3 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html